diff --git a/pkg/registry/apis/dashboard/legacy/sql_dashboards.go b/pkg/registry/apis/dashboard/legacy/sql_dashboards.go index b7a660f53e6..3c032613b05 100644 --- a/pkg/registry/apis/dashboard/legacy/sql_dashboards.go +++ b/pkg/registry/apis/dashboard/legacy/sql_dashboards.go @@ -303,19 +303,15 @@ func (a *dashboardSqlAccess) scanRow(rows *sql.Rows, history bool) (*dashboardRo } if origin_name.String != "" { - // if the reader cannot be found, it may be an orphaned provisioned dashboard - resolvedPath := a.provisioning.GetDashboardProvisionerResolvedPath(origin_name.String) - if resolvedPath != "" { - meta.SetSourceProperties(utils.SourceProperties{ - Path: origin_path.String, - Checksum: origin_hash.String, - TimestampMillis: origin_ts.Int64, - }) - meta.SetManagerProperties(utils.ManagerProperties{ - Kind: utils.ManagerKindClassicFP, // nolint:staticcheck - Identity: origin_name.String, - }) - } + meta.SetSourceProperties(utils.SourceProperties{ + Path: origin_path.String, + Checksum: origin_hash.String, + TimestampMillis: origin_ts.Int64, + }) + meta.SetManagerProperties(utils.ManagerProperties{ + Kind: utils.ManagerKindClassicFP, // nolint:staticcheck + Identity: origin_name.String, + }) } else if plugin_id.String != "" { meta.SetManagerProperties(utils.ManagerProperties{ Kind: utils.ManagerKindPlugin, diff --git a/pkg/services/dashboards/service/dashboard_service.go b/pkg/services/dashboards/service/dashboard_service.go index e9e3f695850..1206e1944cd 100644 --- a/pkg/services/dashboards/service/dashboard_service.go +++ b/pkg/services/dashboards/service/dashboard_service.go @@ -1161,8 +1161,11 @@ func (dr *DashboardServiceImpl) UnprovisionDashboard(ctx context.Context, dashbo UpdatedAt: time.Now(), Dashboard: dash.Data, }, nil, true) + if err != nil { + return err + } - return err + return dr.dashboardStore.UnprovisionDashboard(ctx, dashboardId) } return dashboards.ErrDashboardNotFound diff --git a/pkg/services/dashboards/service/dashboard_service_test.go b/pkg/services/dashboards/service/dashboard_service_test.go index 15179f52a2e..a6607d36800 100644 --- a/pkg/services/dashboards/service/dashboard_service_test.go +++ b/pkg/services/dashboards/service/dashboard_service_test.go @@ -1119,6 +1119,7 @@ func TestUnprovisionDashboard(t *testing.T) { }, "spec": map[string]any{}, }} + fakeStore.On("UnprovisionDashboard", mock.Anything, int64(1)).Return(nil).Once() k8sCliMock.On("Get", mock.Anything, mock.Anything, mock.Anything, mock.Anything, mock.Anything).Return(dash, nil) dashWithoutAnnotations := &unstructured.Unstructured{Object: map[string]any{ "apiVersion": dashboardv0.APIVERSION, @@ -1169,6 +1170,7 @@ func TestUnprovisionDashboard(t *testing.T) { err := service.UnprovisionDashboard(ctx, 1) require.NoError(t, err) k8sCliMock.AssertExpectations(t) + fakeStore.AssertExpectations(t) }) }