steve/databricks-cli
steve
/
databricks-cli
mirror of https://github.com/databricks/cli.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Move dry run check before possible reutrns

This commit is contained in:
Wojciech Pratkowiecki 2024-10-03 08:48:45 +02:00
parent 12259b9837
commit 9e53f117d2
1 changed files with 9 additions and 9 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

18
bundle/phases/deploy.go
View File

@ -63,6 +63,15 @@ func approvalForDeploy(ctx context.Context, b *bundle.Bundle) (bool, error) {
return false, err
}
if b.DryRun {
cmdio.LogString(ctx, "Following changes would be deployed:")
_, err := tf.ShowPlanFile(ctx, b.Plan.Path)
if err != nil {
return false, err
}
return false, nil
}
schemaActions := parseTerraformActions(plan.ResourceChanges, func(typ string, actions tfjson.Actions) bool {
// Filter in only UC schema resources.
if typ != "databricks_schema" {
@ -115,15 +124,6 @@ properties such as the 'catalog' or 'storage' are changed:`
return true, nil
}
if b.DryRun {
cmdio.LogString(ctx, "Following changes would be deployed:")
_, err := tf.ShowPlanFile(ctx, b.Plan.Path)
if err != nil {
return false, err
}
return false, nil
}
if !cmdio.IsPromptSupported(ctx) {
return false, fmt.Errorf("the deployment requires destructive actions, but current console does not support prompting. Please specify --auto-approve if you would like to skip prompts and proceed")
}