bundle/config/mutator/apply_presets.go

@@ -214,7 +214,7 @@ func (m *applyPresets) Apply(ctx context.Context, b *bundle.Bundle) diag.Diagnos
 
 	// Dashboards: Prefix
 	for key, dashboard := range r.Dashboards {
-		if dashboard == nil || dashboard.CreateDashboardRequest == nil {
+		if dashboard == nil {
 			diags = diags.Extend(diag.Errorf("dashboard %s s is not defined", key))
 			continue
 		}

bundle/config/mutator/configure_dashboard_defaults_test.go

@@ -10,7 +10,6 @@ import (
 	"github.com/databricks/cli/bundle/config/resources"
 	"github.com/databricks/cli/bundle/internal/bundletest"
 	"github.com/databricks/cli/libs/dyn"
-	"github.com/databricks/databricks-sdk-go/service/dashboards"
 	"github.com/stretchr/testify/assert"
 	"github.com/stretchr/testify/require"
 )
@@ -26,16 +25,12 @@ func TestConfigureDashboardDefaultsParentPath(t *testing.T) {
 					"d1": {
 						// Empty string is skipped.
 						// See below for how it is set.
-						CreateDashboardRequest: &dashboards.CreateDashboardRequest{
 						ParentPath: "",
 					},
-					},
 					"d2": {
 						// Non-empty string is skipped.
-						CreateDashboardRequest: &dashboards.CreateDashboardRequest{
 						ParentPath: "already-set",
 					},
-					},
 					"d3": {
 						// No parent path set.
 					},

bundle/config/mutator/initialize_urls_test.go

@@ -8,7 +8,6 @@ import (
 	"github.com/databricks/cli/bundle/config/resources"
 	"github.com/databricks/databricks-sdk-go/service/catalog"
 	"github.com/databricks/databricks-sdk-go/service/compute"
-	"github.com/databricks/databricks-sdk-go/service/dashboards"
 	"github.com/databricks/databricks-sdk-go/service/jobs"
 	"github.com/databricks/databricks-sdk-go/service/ml"
 	"github.com/databricks/databricks-sdk-go/service/pipelines"
@@ -89,13 +88,11 @@ func TestInitializeURLs(t *testing.T) {
 				Dashboards: map[string]*resources.Dashboard{
 					"dashboard1": {
 						ID:          "01ef8d56871e1d50ae30ce7375e42478",
-						CreateDashboardRequest: &dashboards.CreateDashboardRequest{
 						DisplayName: "My special dashboard",
 					},
 				},
 			},
 		},
-		},
 	}
 
 	expectedURLs := map[string]string{

bundle/config/mutator/process_target_mode_test.go

@@ -14,7 +14,6 @@ import (
 	sdkconfig "github.com/databricks/databricks-sdk-go/config"
 	"github.com/databricks/databricks-sdk-go/service/catalog"
 	"github.com/databricks/databricks-sdk-go/service/compute"
-	"github.com/databricks/databricks-sdk-go/service/dashboards"
 	"github.com/databricks/databricks-sdk-go/service/iam"
 	"github.com/databricks/databricks-sdk-go/service/jobs"
 	"github.com/databricks/databricks-sdk-go/service/ml"
@@ -125,11 +124,7 @@ func mockBundle(mode config.Mode) *bundle.Bundle {
 					"cluster1": {ClusterSpec: &compute.ClusterSpec{ClusterName: "cluster1", SparkVersion: "13.2.x", NumWorkers: 1}},
 				},
 				Dashboards: map[string]*resources.Dashboard{
-					"dashboard1": {
+					"dashboard1": {DisplayName: "dashboard1"},
-						CreateDashboardRequest: &dashboards.CreateDashboardRequest{
-							DisplayName: "dashboard1",
-						},
-					},
 				},
 			},
 		},

bundle/config/resources/dashboard.go

@@ -17,18 +17,27 @@ type Dashboard struct {
 	ModifiedStatus ModifiedStatus `json:"modified_status,omitempty" bundle:"internal"`
 	URL            string         `json:"url,omitempty" bundle:"internal"`
 
-	*dashboards.CreateDashboardRequest
+	// ===========================
+	// === BEGIN OF API FIELDS ===
+	// ===========================
 
-	// =========================
+	// DisplayName is the display name of the dashboard (both as title and as basename in the workspace).
-	// === Additional fields ===
+	DisplayName string `json:"display_name"`
-	// =========================
+
+	// WarehouseID is the ID of the SQL Warehouse used to run the dashboard's queries.
+	WarehouseID string `json:"warehouse_id"`
 
 	// SerializedDashboard holds the contents of the dashboard in serialized JSON form.
-	// We override the field's type from the SDK struct here to allow for inlining as YAML.
+	// Note: its type is any and not string such that it can be inlined as YAML.
-	// If the value is a string, it is used as is.
+	// If it is not a string, its contents will be marshalled as JSON.
-	// If it is not a string, its contents is marshalled as JSON.
 	SerializedDashboard any `json:"serialized_dashboard,omitempty"`
 
+	// ParentPath is the workspace path of the folder containing the dashboard.
+	// Includes leading slash and no trailing slash.
+	//
+	// Defaults to ${workspace.resource_path} if not set.
+	ParentPath string `json:"parent_path,omitempty"`
+
 	// EmbedCredentials is a flag to indicate if the publisher's credentials should
 	// be embedded in the published dashboard. These embedded credentials will be used
 	// to execute the published dashboard's queries.
@@ -36,6 +45,10 @@ type Dashboard struct {
 	// Defaults to false if not set.
 	EmbedCredentials bool `json:"embed_credentials,omitempty"`
 
+	// ===========================
+	// ==== END OF API FIELDS ====
+	// ===========================
+
 	// FilePath points to the local `.lvdash.json` file containing the dashboard definition.
 	FilePath string `json:"file_path,omitempty"`
 }

bundle/deploy/terraform/check_modified_dashboards.go

@@ -16,7 +16,7 @@ type dashboardState struct {
 	ETag string
 }
 
-func collectDashboardsFromState(ctx context.Context, b *bundle.Bundle) ([]dashboardState, error) {
+func collectDashboards(ctx context.Context, b *bundle.Bundle) ([]dashboardState, error) {
 	state, err := ParseResourcesState(ctx, b)
 	if err != nil && state == nil {
 		return nil, err
@@ -28,12 +28,22 @@ func collectDashboardsFromState(ctx context.Context, b *bundle.Bundle) ([]dashbo
 			continue
 		}
 		for _, instance := range resource.Instances {
+			id := instance.Attributes.ID
+			if id == "" {
+				continue
+			}
+
 			switch resource.Type {
 			case "databricks_dashboard":
+				etag := instance.Attributes.ETag
+				if etag == "" {
+					continue
+				}
+
 				dashboards = append(dashboards, dashboardState{
 					Name: resource.Name,
-					ID:   instance.Attributes.ID,
+					ID:   id,
-					ETag: instance.Attributes.ETag,
+					ETag: etag,
 				})
 			}
 		}
@@ -42,14 +52,14 @@ func collectDashboardsFromState(ctx context.Context, b *bundle.Bundle) ([]dashbo
 	return dashboards, nil
 }
 
-type checkDashboardsModifiedRemotely struct {
+type checkModifiedDashboards struct {
 }
 
-func (l *checkDashboardsModifiedRemotely) Name() string {
+func (l *checkModifiedDashboards) Name() string {
-	return "CheckDashboardsModifiedRemotely"
+	return "CheckModifiedDashboards"
 }
 
-func (l *checkDashboardsModifiedRemotely) Apply(ctx context.Context, b *bundle.Bundle) diag.Diagnostics {
+func (l *checkModifiedDashboards) Apply(ctx context.Context, b *bundle.Bundle) diag.Diagnostics {
 	// This mutator is relevant only if the bundle includes dashboards.
 	if len(b.Config.Resources.Dashboards) == 0 {
 		return nil
@@ -60,7 +70,7 @@ func (l *checkDashboardsModifiedRemotely) Apply(ctx context.Context, b *bundle.B
 		return nil
 	}
 
-	dashboards, err := collectDashboardsFromState(ctx, b)
+	dashboards, err := collectDashboards(ctx, b)
 	if err != nil {
 		return diag.FromErr(err)
 	}
@@ -112,6 +122,6 @@ func (l *checkDashboardsModifiedRemotely) Apply(ctx context.Context, b *bundle.B
 	return diags
 }
 
-func CheckDashboardsModifiedRemotely() *checkDashboardsModifiedRemotely {
+func CheckModifiedDashboards() *checkModifiedDashboards {
-	return &checkDashboardsModifiedRemotely{}
+	return &checkModifiedDashboards{}
 }

bundle/deploy/terraform/check_modified_dashboards_test.go

@@ -29,18 +29,16 @@ func mockDashboardBundle(t *testing.T) *bundle.Bundle {
 			Resources: config.Resources{
 				Dashboards: map[string]*resources.Dashboard{
 					"dash1": {
-						CreateDashboardRequest: &dashboards.CreateDashboardRequest{
 						DisplayName: "My Special Dashboard",
 					},
 				},
 			},
 		},
-		},
 	}
 	return b
 }
 
-func TestCheckDashboardsModifiedRemotely_NoDashboards(t *testing.T) {
+func TestCheckModifiedDashboards_NoDashboards(t *testing.T) {
 	dir := t.TempDir()
 	b := &bundle.Bundle{
 		BundleRootPath: dir,
@@ -52,17 +50,17 @@ func TestCheckDashboardsModifiedRemotely_NoDashboards(t *testing.T) {
 		},
 	}
 
-	diags := bundle.Apply(context.Background(), b, CheckDashboardsModifiedRemotely())
+	diags := bundle.Apply(context.Background(), b, CheckModifiedDashboards())
 	assert.Empty(t, diags)
 }
 
-func TestCheckDashboardsModifiedRemotely_FirstDeployment(t *testing.T) {
+func TestCheckModifiedDashboards_FirstDeployment(t *testing.T) {
 	b := mockDashboardBundle(t)
-	diags := bundle.Apply(context.Background(), b, CheckDashboardsModifiedRemotely())
+	diags := bundle.Apply(context.Background(), b, CheckModifiedDashboards())
 	assert.Empty(t, diags)
 }
 
-func TestCheckDashboardsModifiedRemotely_ExistingStateNoChange(t *testing.T) {
+func TestCheckModifiedDashboards_ExistingStateNoChange(t *testing.T) {
 	ctx := context.Background()
 
 	b := mockDashboardBundle(t)
@@ -81,11 +79,11 @@ func TestCheckDashboardsModifiedRemotely_ExistingStateNoChange(t *testing.T) {
 	b.SetWorkpaceClient(m.WorkspaceClient)
 
 	// No changes, so no diags.
-	diags := bundle.Apply(ctx, b, CheckDashboardsModifiedRemotely())
+	diags := bundle.Apply(ctx, b, CheckModifiedDashboards())
 	assert.Empty(t, diags)
 }
 
-func TestCheckDashboardsModifiedRemotely_ExistingStateChange(t *testing.T) {
+func TestCheckModifiedDashboards_ExistingStateChange(t *testing.T) {
 	ctx := context.Background()
 
 	b := mockDashboardBundle(t)
@@ -104,14 +102,14 @@ func TestCheckDashboardsModifiedRemotely_ExistingStateChange(t *testing.T) {
 	b.SetWorkpaceClient(m.WorkspaceClient)
 
 	// The dashboard has changed, so expect an error.
-	diags := bundle.Apply(ctx, b, CheckDashboardsModifiedRemotely())
+	diags := bundle.Apply(ctx, b, CheckModifiedDashboards())
 	if assert.Len(t, diags, 1) {
 		assert.Equal(t, diag.Error, diags[0].Severity)
 		assert.Equal(t, `dashboard "dash1" has been modified remotely`, diags[0].Summary)
 	}
 }
 
-func TestCheckDashboardsModifiedRemotely_ExistingStateFailureToGet(t *testing.T) {
+func TestCheckModifiedDashboards_ExistingStateFailureToGet(t *testing.T) {
 	ctx := context.Background()
 
 	b := mockDashboardBundle(t)
@@ -127,7 +125,7 @@ func TestCheckDashboardsModifiedRemotely_ExistingStateFailureToGet(t *testing.T)
 	b.SetWorkpaceClient(m.WorkspaceClient)
 
 	// Unable to get the dashboard, so expect an error.
-	diags := bundle.Apply(ctx, b, CheckDashboardsModifiedRemotely())
+	diags := bundle.Apply(ctx, b, CheckModifiedDashboards())
 	if assert.Len(t, diags, 1) {
 		assert.Equal(t, diag.Error, diags[0].Severity)
 		assert.Equal(t, `failed to get dashboard "dash1"`, diags[0].Summary)

bundle/deploy/terraform/convert_test.go

@@ -12,7 +12,6 @@ import (
 	"github.com/databricks/cli/libs/dyn/convert"
 	"github.com/databricks/databricks-sdk-go/service/catalog"
 	"github.com/databricks/databricks-sdk-go/service/compute"
-	"github.com/databricks/databricks-sdk-go/service/dashboards"
 	"github.com/databricks/databricks-sdk-go/service/jobs"
 	"github.com/databricks/databricks-sdk-go/service/ml"
 	"github.com/databricks/databricks-sdk-go/service/pipelines"
@@ -792,12 +791,10 @@ func TestTerraformToBundleEmptyRemoteResources(t *testing.T) {
 			},
 			Dashboards: map[string]*resources.Dashboard{
 				"test_dashboard": {
-					CreateDashboardRequest: &dashboards.CreateDashboardRequest{
 					DisplayName: "test_dashboard",
 				},
 			},
 		},
-		},
 	}
 	var tfState = resourcesState{
 		Resources: nil,
@@ -951,17 +948,13 @@ func TestTerraformToBundleModifiedResources(t *testing.T) {
 			},
 			Dashboards: map[string]*resources.Dashboard{
 				"test_dashboard": {
-					CreateDashboardRequest: &dashboards.CreateDashboardRequest{
 					DisplayName: "test_dashboard",
 				},
-				},
 				"test_dashboard_new": {
-					CreateDashboardRequest: &dashboards.CreateDashboardRequest{
 					DisplayName: "test_dashboard_new",
 				},
 			},
 		},
-		},
 	}
 	var tfState = resourcesState{
 		Resources: []stateResource{

bundle/deploy/terraform/tfdyn/convert_dashboard.go

@@ -2,7 +2,6 @@ package tfdyn
 
 import (
 	"context"
-	"encoding/json"
 	"fmt"
 
 	"github.com/databricks/cli/bundle/internal/tf/schema"
@@ -11,34 +10,6 @@ import (
 	"github.com/databricks/cli/libs/log"
 )
 
-const (
-	filePathFieldName            = "file_path"
-	serializedDashboardFieldName = "serialized_dashboard"
-)
-
-// Marshal "serialized_dashboard" as JSON if it is set in the input but not in the output.
-func marshalSerializedDashboard(vin dyn.Value, vout dyn.Value) (dyn.Value, error) {
-	// Skip if the "serialized_dashboard" field is already set.
-	if v := vout.Get(serializedDashboardFieldName); v.IsValid() {
-		return vout, nil
-	}
-
-	// Skip if the "serialized_dashboard" field on the input is not set.
-	v := vin.Get(serializedDashboardFieldName)
-	if !v.IsValid() {
-		return vout, nil
-	}
-
-	// Marshal the "serialized_dashboard" field as JSON.
-	data, err := json.Marshal(v.AsAny())
-	if err != nil {
-		return dyn.InvalidValue, fmt.Errorf("failed to marshal serialized_dashboard: %w", err)
-	}
-
-	// Set the "serialized_dashboard" field on the output.
-	return dyn.Set(vout, serializedDashboardFieldName, dyn.V(string(data)))
-}
-
 func convertDashboardResource(ctx context.Context, vin dyn.Value) (dyn.Value, error) {
 	var err error
 
@@ -51,8 +22,8 @@ func convertDashboardResource(ctx context.Context, vin dyn.Value) (dyn.Value, er
 	// Include "serialized_dashboard" field if "file_path" is set.
 	// Note: the Terraform resource supports "file_path" natively, but its
 	// change detection mechanism doesn't work as expected at the time of writing (Sep 30).
-	if path, ok := vout.Get(filePathFieldName).AsString(); ok {
+	if path, ok := vout.Get("file_path").AsString(); ok {
-		vout, err = dyn.Set(vout, serializedDashboardFieldName, dyn.V(fmt.Sprintf("${file(\"%s\")}", path)))
+		vout, err = dyn.Set(vout, "serialized_dashboard", dyn.V(fmt.Sprintf("${file(\"%s\")}", path)))
 		if err != nil {
 			return dyn.InvalidValue, fmt.Errorf("failed to set serialized_dashboard: %w", err)
 		}
@@ -62,7 +33,7 @@ func convertDashboardResource(ctx context.Context, vin dyn.Value) (dyn.Value, er
 			case 0:
 				return v, nil
 			case 1:
-				if p[0] == dyn.Key(filePathFieldName) {
+				if p[0] == dyn.Key("file_path") {
 					return v, dyn.ErrDrop
 				}
 			}
@@ -75,12 +46,6 @@ func convertDashboardResource(ctx context.Context, vin dyn.Value) (dyn.Value, er
 		}
 	}
 
-	// Marshal "serialized_dashboard" as JSON if it is set in the input but not in the output.
-	vout, err = marshalSerializedDashboard(vin, vout)
-	if err != nil {
-		return dyn.InvalidValue, err
-	}
-
 	return vout, nil
 }
 

bundle/deploy/terraform/tfdyn/convert_dashboard_test.go

@@ -8,19 +8,15 @@ import (
 	"github.com/databricks/cli/bundle/internal/tf/schema"
 	"github.com/databricks/cli/libs/dyn"
 	"github.com/databricks/cli/libs/dyn/convert"
-	"github.com/databricks/databricks-sdk-go/service/dashboards"
 	"github.com/stretchr/testify/assert"
 	"github.com/stretchr/testify/require"
 )
 
 func TestConvertDashboard(t *testing.T) {
 	var src = resources.Dashboard{
-		CreateDashboardRequest: &dashboards.CreateDashboardRequest{
 		DisplayName:      "my dashboard",
-			WarehouseId: "f00dcafe",
+		WarehouseID:      "f00dcafe",
 		ParentPath:       "/some/path",
-		},
-
 		EmbedCredentials: true,
 
 		Permissions: []resources.Permission{
@@ -82,48 +78,3 @@ func TestConvertDashboardFilePath(t *testing.T) {
 		"file_path": "some/path",
 	})
 }
-
-func TestConvertDashboardSerializedDashboardString(t *testing.T) {
-	var src = resources.Dashboard{
-		SerializedDashboard: `{ "json": true }`,
-	}
-
-	vin, err := convert.FromTyped(src, dyn.NilValue)
-	require.NoError(t, err)
-
-	ctx := context.Background()
-	out := schema.NewResources()
-	err = dashboardConverter{}.Convert(ctx, "my_dashboard", vin, out)
-	require.NoError(t, err)
-
-	// Assert that the "serialized_dashboard" is included.
-	assert.Subset(t, out.Dashboard["my_dashboard"], map[string]any{
-		"serialized_dashboard": `{ "json": true }`,
-	})
-}
-
-func TestConvertDashboardSerializedDashboardAny(t *testing.T) {
-	var src = resources.Dashboard{
-		SerializedDashboard: map[string]any{
-			"pages": []map[string]any{
-				{
-					"displayName": "New Page",
-					"layout":      []map[string]any{},
-				},
-			},
-		},
-	}
-
-	vin, err := convert.FromTyped(src, dyn.NilValue)
-	require.NoError(t, err)
-
-	ctx := context.Background()
-	out := schema.NewResources()
-	err = dashboardConverter{}.Convert(ctx, "my_dashboard", vin, out)
-	require.NoError(t, err)
-
-	// Assert that the "serialized_dashboard" is included.
-	assert.Subset(t, out.Dashboard["my_dashboard"], map[string]any{
-		"serialized_dashboard": `{"pages":[{"displayName":"New Page","layout":[]}]}`,
-	})
-}

bundle/phases/deploy.go

@@ -152,7 +152,7 @@ func Deploy(outputHandler sync.OutputHandler) bundle.Mutator {
 		bundle.Defer(
 			bundle.Seq(
 				terraform.StatePull(),
-				terraform.CheckDashboardsModifiedRemotely(),
+				terraform.CheckModifiedDashboards(),
 				deploy.StatePull(),
 				mutator.ValidateGitDetails(),
 				artifacts.CleanUp(),