Gauge: Update test dashboard to round two of the segment panels to whole numbers

.github/CODEOWNERS

@@ -425,7 +425,6 @@ i18next.config.ts @grafana/grafana-frontend-platform
 /public/locales/enterprise/i18next.config.ts @grafana/grafana-frontend-platform
 /public/app/core/internationalization/ @grafana/grafana-frontend-platform
 /e2e/ @grafana/grafana-frontend-platform
-/e2e-playwright/alerting-suite/ @grafana/alerting-frontend
 /e2e-playwright/cloud-plugins-suite/ @grafana/partner-datasources
 /e2e-playwright/dashboard-new-layouts/ @grafana/dashboards-squad
 /e2e-playwright/dashboard-cujs/ @grafana/dashboards-squad
@@ -501,6 +500,7 @@ i18next.config.ts @grafana/grafana-frontend-platform
 /e2e-playwright/various-suite/filter-annotations.spec.ts @grafana/dashboards-squad
 /e2e-playwright/various-suite/frontend-sandbox-app.spec.ts @grafana/plugins-platform-frontend
 /e2e-playwright/various-suite/frontend-sandbox-datasource.spec.ts @grafana/plugins-platform-frontend
+/e2e-playwright/various-suite/gauge.spec.ts @grafana/dataviz-squad
 /e2e-playwright/various-suite/grafana-datasource-random-walk.spec.ts @grafana/grafana-frontend-platform
 /e2e-playwright/various-suite/graph-auto-migrate.spec.ts @grafana/dataviz-squad
 /e2e-playwright/various-suite/inspect-drawer.spec.ts @grafana/dashboards-squad
@@ -519,7 +519,7 @@ i18next.config.ts @grafana/grafana-frontend-platform
 /e2e-playwright/various-suite/solo-route.spec.ts @grafana/dashboards-squad
 /e2e-playwright/various-suite/trace-view-scrolling.spec.ts @grafana/observability-traces-and-profiling
 /e2e-playwright/various-suite/verify-i18n.spec.ts @grafana/grafana-frontend-platform
-/e2e-playwright/various-suite/visualization-suggestions*.spec.ts @grafana/dataviz-squad
+/e2e-playwright/various-suite/visualization-suggestions.spec.ts @grafana/dataviz-squad
 /e2e-playwright/various-suite/perf-test.spec.ts @grafana/grafana-frontend-platform
 
 # Packages

.github/actions/change-detection/action.yml

@@ -95,11 +95,9 @@ runs:
             - 'nx.json'
             - 'tsconfig.json'
             - '.yarn/**'
-            - 'apps/dashboard/pkg/migration/**'
             - '${{ inputs.self }}'
           e2e:
             - 'e2e/**'
-            - 'e2e-playwright/**'
             - '.github/actions/setup-enterprise/**'
             - '.github/actions/checkout/**'
             - 'emails/**'

apps/advisor/go.mod

@@ -157,7 +157,7 @@ require (
 	github.com/google/go-querystring v1.1.0 // indirect
 	github.com/google/uuid v1.6.0 // indirect
 	github.com/google/wire v0.7.0 // indirect
-	github.com/grafana/alerting v0.0.0-20251231150637-b7821017d69f // indirect
+	github.com/grafana/alerting v0.0.0-20251212143239-491433b332b7 // indirect
 	github.com/grafana/authlib v0.0.0-20250930082137-a40e2c2b094f // indirect
 	github.com/grafana/dataplane/sdata v0.0.9 // indirect
 	github.com/grafana/dskit v0.0.0-20250908063411-6b6da59b5cc4 // indirect

apps/advisor/go.sum

@@ -619,8 +619,8 @@ github.com/gorilla/mux v1.8.1 h1:TuBL49tXwgrFYWhqrNgrUNEY92u81SPhu7sTdzQEiWY=
 github.com/gorilla/mux v1.8.1/go.mod h1:AKf9I4AEqPTmMytcMc0KkNouC66V3BtZ4qD5fmWSiMQ=
 github.com/gorilla/websocket v1.5.4-0.20250319132907-e064f32e3674 h1:JeSE6pjso5THxAzdVpqr6/geYxZytqFMBCOtn/ujyeo=
 github.com/gorilla/websocket v1.5.4-0.20250319132907-e064f32e3674/go.mod h1:r4w70xmWCQKmi1ONH4KIaBptdivuRPyosB9RmPlGEwA=
-github.com/grafana/alerting v0.0.0-20251231150637-b7821017d69f h1:Br4SaUL3dnVopKKNhDavCLgehw60jdtl/sIxdfzmVts=
-github.com/grafana/alerting v0.0.0-20251231150637-b7821017d69f/go.mod h1:l7v67cgP7x72ajB9UPZlumdrHqNztpKoqQ52cU8T3LU=
+github.com/grafana/alerting v0.0.0-20251212143239-491433b332b7 h1:ZzG/gCclEit9w0QUfQt9GURcOycAIGcsQAhY1u0AEX0=
+github.com/grafana/alerting v0.0.0-20251212143239-491433b332b7/go.mod h1:l7v67cgP7x72ajB9UPZlumdrHqNztpKoqQ52cU8T3LU=
 github.com/grafana/authlib v0.0.0-20250930082137-a40e2c2b094f h1:Cbm6OKkOcJ+7CSZsGsEJzktC/SIa5bxVeYKQLuYK86o=
 github.com/grafana/authlib v0.0.0-20250930082137-a40e2c2b094f/go.mod h1:axY0cdOg3q0TZHwpHnIz5x16xZ8ZBxJHShsSHHXcHQg=
 github.com/grafana/authlib/types v0.0.0-20251119142549-be091cf2f4d4 h1:Muoy+FMGrHj3GdFbvsMzUT7eusgii9PKf9L1ZaXDDbY=

apps/alerting/historian/go.mod

@@ -4,7 +4,7 @@ go 1.25.5
 
 require (
 	github.com/go-kit/log v0.2.1
-	github.com/grafana/alerting v0.0.0-20251231150637-b7821017d69f
+	github.com/grafana/alerting v0.0.0-20251212143239-491433b332b7
 	github.com/grafana/dskit v0.0.0-20250908063411-6b6da59b5cc4
 	github.com/grafana/grafana-app-sdk v0.48.7
 	github.com/grafana/grafana-app-sdk/logging v0.48.7

apps/alerting/historian/go.sum

@@ -243,8 +243,8 @@ github.com/google/uuid v1.6.0 h1:NIvaJDMOsjHA8n1jAhLSgzrAzy1Hgr+hNrb57e+94F0=
 github.com/google/uuid v1.6.0/go.mod h1:TIyPZe4MgqvfeYDBFedMoGGpEw/LqOeaOT+nhxU+yHo=
 github.com/googleapis/gax-go/v2 v2.0.4/go.mod h1:0Wqv26UfaUD9n4G6kQubkQ+KchISgw+vpHVxEJEs9eg=
 github.com/googleapis/gax-go/v2 v2.0.5/go.mod h1:DWXyrwAJ9X0FpwwEdw+IPEYBICEFu5mhpdKc/us6bOk=
-github.com/grafana/alerting v0.0.0-20251231150637-b7821017d69f h1:Br4SaUL3dnVopKKNhDavCLgehw60jdtl/sIxdfzmVts=
-github.com/grafana/alerting v0.0.0-20251231150637-b7821017d69f/go.mod h1:l7v67cgP7x72ajB9UPZlumdrHqNztpKoqQ52cU8T3LU=
+github.com/grafana/alerting v0.0.0-20251212143239-491433b332b7 h1:ZzG/gCclEit9w0QUfQt9GURcOycAIGcsQAhY1u0AEX0=
+github.com/grafana/alerting v0.0.0-20251212143239-491433b332b7/go.mod h1:l7v67cgP7x72ajB9UPZlumdrHqNztpKoqQ52cU8T3LU=
 github.com/grafana/dskit v0.0.0-20250908063411-6b6da59b5cc4 h1:jSojuc7njleS3UOz223WDlXOinmuLAIPI0z2vtq8EgI=
 github.com/grafana/dskit v0.0.0-20250908063411-6b6da59b5cc4/go.mod h1:VahT+GtfQIM+o8ht2StR6J9g+Ef+C2Vokh5uuSmOD/4=
 github.com/grafana/grafana-app-sdk v0.48.7 h1:9mF7nqkqP0QUYYDlznoOt+GIyjzj45wGfUHB32u2ZMo=

apps/alerting/historian/pkg/app/notification/lokireader.go

@@ -31,10 +31,6 @@ const (
 	maxLimit           = 1000
 	Namespace          = "grafana"
 	Subsystem          = "alerting"
-
-	// LogQL field path for alert rule UID after JSON parsing.
-	// Loki flattens nested JSON fields with underscores: alert.labels.__alert_rule_uid__ -> alert_labels___alert_rule_uid__
-	lokiAlertRuleUIDField = "alert_labels___alert_rule_uid__"
 )
 
 var (
@@ -115,13 +111,13 @@ func buildQuery(query Query) (string, error) {
 		fmt.Sprintf(`%s=%q`, historian.LabelFrom, historian.LabelFromValue),
 	}
 
-	logql := fmt.Sprintf(`{%s} | json`, strings.Join(selectors, `,`))
-
-	// Add ruleUID filter as JSON line filter if specified.
-	if query.RuleUID != nil && *query.RuleUID != "" {
-		logql += fmt.Sprintf(` | %s = %q`, lokiAlertRuleUIDField, *query.RuleUID)
+	if query.RuleUID != nil {
+		selectors = append(selectors,
+			fmt.Sprintf(`%s=%q`, historian.LabelRuleUID, *query.RuleUID))
 	}
 
+	logql := fmt.Sprintf(`{%s} | json`, strings.Join(selectors, `,`))
+
 	// Add receiver filter if specified.
 	if query.Receiver != nil && *query.Receiver != "" {
 		logql += fmt.Sprintf(` | receiver = %q`, *query.Receiver)
@@ -215,13 +211,16 @@ func parseLokiEntry(s lokiclient.Sample) (Entry, error) {
 		groupLabels = make(map[string]string)
 	}
 
-	alerts := []EntryAlert{{
-		Status:      lokiEntry.Alert.Status,
-		Labels:      lokiEntry.Alert.Labels,
-		Annotations: lokiEntry.Alert.Annotations,
-		StartsAt:    lokiEntry.Alert.StartsAt,
-		EndsAt:      lokiEntry.Alert.EndsAt,
-	}}
+	alerts := make([]EntryAlert, len(lokiEntry.Alerts))
+	for i, a := range lokiEntry.Alerts {
+		alerts[i] = EntryAlert{
+			Status:      a.Status,
+			Labels:      a.Labels,
+			Annotations: a.Annotations,
+			StartsAt:    a.StartsAt,
+			EndsAt:      a.EndsAt,
+		}
+	}
 
 	return Entry{
 		Timestamp:    s.T,

apps/alerting/historian/pkg/app/notification/lokireader_test.go

@@ -7,7 +7,6 @@ import (
 	"testing"
 	"time"
 
-	"github.com/grafana/alerting/models"
 	"github.com/grafana/alerting/notify/historian"
 	"github.com/grafana/alerting/notify/historian/lokiclient"
 	"github.com/grafana/grafana-app-sdk/logging"
@@ -134,8 +133,9 @@ func TestBuildQuery(t *testing.T) {
 			query: Query{
 				RuleUID: stringPtr("test-rule-uid"),
 			},
-			expected: fmt.Sprintf(`{%s=%q} | json | alert_labels___alert_rule_uid__ = "test-rule-uid"`,
-				historian.LabelFrom, historian.LabelFromValue),
+			expected: fmt.Sprintf(`{%s=%q,%s=%q} | json`,
+				historian.LabelFrom, historian.LabelFromValue,
+				historian.LabelRuleUID, "test-rule-uid"),
 		},
 		{
 			name: "query with receiver filter",
@@ -143,8 +143,9 @@ func TestBuildQuery(t *testing.T) {
 				RuleUID:  stringPtr("test-rule-uid"),
 				Receiver: stringPtr("email-receiver"),
 			},
-			expected: fmt.Sprintf(`{%s=%q} | json | alert_labels___alert_rule_uid__ = "test-rule-uid" | receiver = "email-receiver"`,
-				historian.LabelFrom, historian.LabelFromValue),
+			expected: fmt.Sprintf(`{%s=%q,%s=%q} | json | receiver = "email-receiver"`,
+				historian.LabelFrom, historian.LabelFromValue,
+				historian.LabelRuleUID, "test-rule-uid"),
 		},
 		{
 			name: "query with status filter",
@@ -152,8 +153,9 @@ func TestBuildQuery(t *testing.T) {
 				RuleUID: stringPtr("test-rule-uid"),
 				Status:  createStatusPtr(v0alpha1.CreateNotificationqueryRequestNotificationStatusFiring),
 			},
-			expected: fmt.Sprintf(`{%s=%q} | json | alert_labels___alert_rule_uid__ = "test-rule-uid" | status = "firing"`,
-				historian.LabelFrom, historian.LabelFromValue),
+			expected: fmt.Sprintf(`{%s=%q,%s=%q} | json | status = "firing"`,
+				historian.LabelFrom, historian.LabelFromValue,
+				historian.LabelRuleUID, "test-rule-uid"),
 		},
 		{
 			name: "query with success outcome filter",
@@ -161,8 +163,9 @@ func TestBuildQuery(t *testing.T) {
 				RuleUID: stringPtr("test-rule-uid"),
 				Outcome: outcomePtr(v0alpha1.CreateNotificationqueryRequestNotificationOutcomeSuccess),
 			},
-			expected: fmt.Sprintf(`{%s=%q} | json | alert_labels___alert_rule_uid__ = "test-rule-uid" | error = ""`,
-				historian.LabelFrom, historian.LabelFromValue),
+			expected: fmt.Sprintf(`{%s=%q,%s=%q} | json | error = ""`,
+				historian.LabelFrom, historian.LabelFromValue,
+				historian.LabelRuleUID, "test-rule-uid"),
 		},
 		{
 			name: "query with error outcome filter",
@@ -170,8 +173,9 @@ func TestBuildQuery(t *testing.T) {
 				RuleUID: stringPtr("test-rule-uid"),
 				Outcome: outcomePtr(v0alpha1.CreateNotificationqueryRequestNotificationOutcomeError),
 			},
-			expected: fmt.Sprintf(`{%s=%q} | json | alert_labels___alert_rule_uid__ = "test-rule-uid" | error != ""`,
-				historian.LabelFrom, historian.LabelFromValue),
+			expected: fmt.Sprintf(`{%s=%q,%s=%q} | json | error != ""`,
+				historian.LabelFrom, historian.LabelFromValue,
+				historian.LabelRuleUID, "test-rule-uid"),
 		},
 		{
 			name: "query with many filters",
@@ -181,8 +185,9 @@ func TestBuildQuery(t *testing.T) {
 				Status:   createStatusPtr(v0alpha1.CreateNotificationqueryRequestNotificationStatusResolved),
 				Outcome:  outcomePtr(v0alpha1.CreateNotificationqueryRequestNotificationOutcomeSuccess),
 			},
-			expected: fmt.Sprintf(`{%s=%q} | json | alert_labels___alert_rule_uid__ = "test-rule-uid" | receiver = "email-receiver" | status = "resolved" | error = ""`,
-				historian.LabelFrom, historian.LabelFromValue),
+			expected: fmt.Sprintf(`{%s=%q,%s=%q} | json | receiver = "email-receiver" | status = "resolved" | error = ""`,
+				historian.LabelFrom, historian.LabelFromValue,
+				historian.LabelRuleUID, "test-rule-uid"),
 		},
 		{
 			name: "query with group label matcher",
@@ -272,19 +277,19 @@ func TestParseLokiEntry(t *testing.T) {
 					GroupLabels: map[string]string{
 						"alertname": "test-alert",
 					},
-					Alert: historian.NotificationHistoryLokiEntryAlert{
-						Status: "firing",
-						Labels: map[string]string{
-							"severity": "critical",
+					Alerts: []historian.NotificationHistoryLokiEntryAlert{
+						{
+							Status: "firing",
+							Labels: map[string]string{
+								"severity": "critical",
+							},
+							Annotations: map[string]string{
+								"summary": "Test alert",
+							},
+							StartsAt: now,
+							EndsAt:   now.Add(1 * time.Hour),
 						},
-						Annotations: map[string]string{
-							"summary": "Test alert",
-						},
-						StartsAt: now,
-						EndsAt:   now.Add(1 * time.Hour),
 					},
-					AlertIndex:   0,
-					AlertCount:   1,
 					Retry:        false,
 					Duration:     100,
 					PipelineTime: now,
@@ -330,9 +335,7 @@ func TestParseLokiEntry(t *testing.T) {
 					Error:         "notification failed",
 					GroupKey:      "key:thing",
 					GroupLabels:   map[string]string{},
-					Alert:         historian.NotificationHistoryLokiEntryAlert{},
-					AlertIndex:    0,
-					AlertCount:    1,
+					Alerts:        []historian.NotificationHistoryLokiEntryAlert{},
 					PipelineTime:  now,
 				}),
 			},
@@ -344,7 +347,7 @@ func TestParseLokiEntry(t *testing.T) {
 				Outcome:      OutcomeError,
 				GroupKey:     "key:thing",
 				GroupLabels:  map[string]string{},
-				Alerts:       []EntryAlert{{}},
+				Alerts:       []EntryAlert{},
 				Error:        stringPtr("notification failed"),
 				PipelineTime: now,
 			},
@@ -362,7 +365,7 @@ func TestParseLokiEntry(t *testing.T) {
 				Status:       Status("firing"),
 				Outcome:      OutcomeSuccess,
 				GroupLabels:  map[string]string{},
-				Alerts:       []EntryAlert{{}},
+				Alerts:       []EntryAlert{},
 				PipelineTime: now,
 			},
 		},
@@ -445,9 +448,7 @@ func TestLokiReader_RunQuery(t *testing.T) {
 								Receiver:      "receiver-1",
 								Status:        "firing",
 								GroupLabels:   map[string]string{},
-								Alert:         historian.NotificationHistoryLokiEntryAlert{},
-								AlertIndex:    0,
-								AlertCount:    1,
+								Alerts:        []historian.NotificationHistoryLokiEntryAlert{},
 								PipelineTime:  now,
 							}),
 						},
@@ -458,9 +459,7 @@ func TestLokiReader_RunQuery(t *testing.T) {
 								Receiver:      "receiver-3",
 								Status:        "firing",
 								GroupLabels:   map[string]string{},
-								Alert:         historian.NotificationHistoryLokiEntryAlert{},
-								AlertIndex:    0,
-								AlertCount:    1,
+								Alerts:        []historian.NotificationHistoryLokiEntryAlert{},
 								PipelineTime:  now,
 							}),
 						},
@@ -475,9 +474,7 @@ func TestLokiReader_RunQuery(t *testing.T) {
 								Receiver:      "receiver-2",
 								Status:        "firing",
 								GroupLabels:   map[string]string{},
-								Alert:         historian.NotificationHistoryLokiEntryAlert{},
-								AlertIndex:    0,
-								AlertCount:    1,
+								Alerts:        []historian.NotificationHistoryLokiEntryAlert{},
 								PipelineTime:  now,
 							}),
 						},
@@ -549,19 +546,19 @@ func createMockLokiResponse(timestamp time.Time) lokiclient.QueryRes {
 								GroupLabels: map[string]string{
 									"alertname": "test-alert",
 								},
-								Alert: historian.NotificationHistoryLokiEntryAlert{
-									Status: "firing",
-									Labels: map[string]string{
-										"severity": "critical",
+								Alerts: []historian.NotificationHistoryLokiEntryAlert{
+									{
+										Status: "firing",
+										Labels: map[string]string{
+											"severity": "critical",
+										},
+										Annotations: map[string]string{
+											"summary": "Test alert",
+										},
+										StartsAt: timestamp,
+										EndsAt:   timestamp.Add(1 * time.Hour),
 									},
-									Annotations: map[string]string{
-										"summary": "Test alert",
-									},
-									StartsAt: timestamp,
-									EndsAt:   timestamp.Add(1 * time.Hour),
 								},
-								AlertIndex:   0,
-								AlertCount:   1,
 								Retry:        false,
 								Duration:     100,
 								PipelineTime: timestamp,
@@ -590,19 +587,10 @@ func createLokiEntryJSONWithNilLabels(t *testing.T, timestamp time.Time) string
 		"status": "firing",
 		"error": "",
 		"groupLabels": null,
-		"alert": {},
-		"alertIndex": 0,
-		"alertCount": 1,
+		"alerts": [],
 		"retry": false,
 		"duration": 0,
 		"pipelineTime": "%s"
 	}`, timestamp.Format(time.RFC3339Nano))
 	return jsonStr
 }
-
-func TestRuleUIDLabelConstant(t *testing.T) {
-	// Verify that models.RuleUIDLabel has the expected value.
-	// If this changes in the alerting module, our LogQL field path constant will be incorrect
-	// and filtering for a single alert rule by its UID will break.
-	assert.Equal(t, "__alert_rule_uid__", models.RuleUIDLabel)
-}

apps/dashboard/pkg/migration/conversion/conversion_data_loss_detection.go

@@ -180,15 +180,12 @@ func countAnnotationsV0V1(spec map[string]interface{}) int {
 		return 0
 	}
 
-	// Handle both []interface{} (from JSON unmarshaling) and []map[string]interface{} (from programmatic creation)
-	if annotationList, ok := annotations["list"].([]interface{}); ok {
-		return len(annotationList)
-	}
-	if annotationList, ok := annotations["list"].([]map[string]interface{}); ok {
-		return len(annotationList)
+	annotationList, ok := annotations["list"].([]interface{})
+	if !ok {
+		return 0
 	}
 
-	return 0
+	return len(annotationList)
 }
 
 // countLinksV0V1 counts dashboard links in v0alpha1 or v1beta1 dashboard spec
@@ -197,15 +194,12 @@ func countLinksV0V1(spec map[string]interface{}) int {
 		return 0
 	}
 
-	// Handle both []interface{} (from JSON unmarshaling) and []map[string]interface{} (from programmatic creation)
-	if links, ok := spec["links"].([]interface{}); ok {
-		return len(links)
-	}
-	if links, ok := spec["links"].([]map[string]interface{}); ok {
-		return len(links)
+	links, ok := spec["links"].([]interface{})
+	if !ok {
+		return 0
 	}
 
-	return 0
+	return len(links)
 }
 
 // countVariablesV0V1 counts template variables in v0alpha1 or v1beta1 dashboard spec
@@ -219,15 +213,12 @@ func countVariablesV0V1(spec map[string]interface{}) int {
 		return 0
 	}
 
-	// Handle both []interface{} (from JSON unmarshaling) and []map[string]interface{} (from programmatic creation)
-	if variableList, ok := templating["list"].([]interface{}); ok {
-		return len(variableList)
-	}
-	if variableList, ok := templating["list"].([]map[string]interface{}); ok {
-		return len(variableList)
+	variableList, ok := templating["list"].([]interface{})
+	if !ok {
+		return 0
 	}
 
-	return 0
+	return len(variableList)
 }
 
 // collectStatsV0V1 collects statistics from v0alpha1 or v1beta1 dashboard

apps/dashboard/pkg/migration/conversion/testdata/input/v1beta1.bom-in-links.json

@@ -1,142 +0,0 @@
-{
-  "kind": "Dashboard",
-  "apiVersion": "dashboard.grafana.app/v1beta1",
-  "metadata": {
-    "name": "bom-in-links-test",
-    "namespace": "org-1",
-    "labels": {
-      "test": "bom-stripping"
-    }
-  },
-  "spec": {
-    "title": "BOM Stripping Test Dashboard",
-    "description": "Testing that BOM characters are stripped from URLs during conversion",
-    "schemaVersion": 42,
-    "tags": ["test", "bom"],
-    "editable": true,
-    "links": [
-      {
-        "title": "Dashboard link with BOM",
-        "type": "link",
-        "url": "http://example.com?var=${datasource}&other=value",
-        "targetBlank": true,
-        "icon": "external link"
-      }
-    ],
-    "panels": [
-      {
-        "id": 1,
-        "type": "table",
-        "title": "Panel with BOM in field config override links",
-        "gridPos": {
-          "h": 8,
-          "w": 12,
-          "x": 0,
-          "y": 0
-        },
-        "fieldConfig": {
-          "defaults": {
-            "custom": {},
-            "mappings": [],
-            "thresholds": {
-              "mode": "absolute",
-              "steps": [
-                {"color": "green"},
-                {"color": "red", "value": 80}
-              ]
-            }
-          },
-          "overrides": [
-            {
-              "matcher": {
-                "id": "byName",
-                "options": "server"
-              },
-              "properties": [
-                {
-                  "id": "links",
-                  "value": [
-                    {
-                      "title": "Override link with BOM",
-                      "url": "http://localhost:3000/d/test?var-datacenter=${__data.fields[datacenter]}&var-server=${__value.raw}"
-                    }
-                  ]
-                }
-              ]
-            }
-          ]
-        },
-        "links": [
-          {
-            "title": "Panel data link with BOM",
-            "url": "http://example.com/${__data.fields.cluster}&var=value",
-            "targetBlank": true
-          }
-        ],
-        "targets": [
-          {
-            "refId": "A",
-            "datasource": {
-              "type": "prometheus",
-              "uid": "test-ds"
-            }
-          }
-        ]
-      },
-      {
-        "id": 2,
-        "type": "timeseries",
-        "title": "Panel with BOM in options dataLinks",
-        "gridPos": {
-          "h": 8,
-          "w": 12,
-          "x": 12,
-          "y": 0
-        },
-        "options": {
-          "legend": {
-            "showLegend": true,
-            "displayMode": "list",
-            "placement": "bottom"
-          },
-          "dataLinks": [
-            {
-              "title": "Options data link with BOM",
-              "url": "http://example.com?series=${__series.name}&time=${__value.time}",
-              "targetBlank": true
-            }
-          ]
-        },
-        "fieldConfig": {
-          "defaults": {
-            "links": [
-              {
-                "title": "Field config default link with BOM",
-                "url": "http://example.com?field=${__field.name}&value=${__value.raw}",
-                "targetBlank": false
-              }
-            ]
-          },
-          "overrides": []
-        },
-        "targets": [
-          {
-            "refId": "A",
-            "datasource": {
-              "type": "prometheus",
-              "uid": "test-ds"
-            }
-          }
-        ]
-      }
-    ],
-    "time": {
-      "from": "now-6h",
-      "to": "now"
-    },
-    "timepicker": {
-      "refresh_intervals": ["5s", "10s", "30s", "1m", "5m"]
-    }
-  }
-}
-

apps/dashboard/pkg/migration/conversion/testdata/input/v1beta1.groupby.json

@@ -1,166 +0,0 @@
-{
-    "kind": "DashboardWithAccessInfo",
-    "apiVersion": "dashboard.grafana.app/v1beta1",
-    "metadata": {
-      "name": "groupby-test"
-    },
-    "spec": {
-      "annotations": {
-        "list": [
-          {
-            "builtIn": 1,
-            "datasource": {
-              "type": "grafana",
-              "uid": "-- Grafana --"
-            },
-            "enable": true,
-            "hide": true,
-            "iconColor": "rgba(0, 211, 255, 1)",
-            "name": "Annotations \u0026 Alerts",
-            "type": "dashboard"
-          }
-        ]
-      },
-      "editable": true,
-      "fiscalYearStartMonth": 0,
-      "graphTooltip": 0,
-      "links": [],
-      "panels": [
-        {
-          "datasource": {
-            "type": "prometheus",
-            "uid": "test-uid"
-          },
-          "fieldConfig": {
-            "defaults": {
-              "color": {
-                "mode": "palette-classic"
-              },
-              "custom": {
-                "axisBorderShow": false,
-                "axisCenteredZero": false,
-                "axisColorMode": "text",
-                "axisLabel": "",
-                "axisPlacement": "auto",
-                "barAlignment": 0,
-                "barWidthFactor": 0.6,
-                "drawStyle": "line",
-                "fillOpacity": 0,
-                "gradientMode": "none",
-                "hideFrom": {
-                  "legend": false,
-                  "tooltip": false,
-                  "viz": false
-                },
-                "insertNulls": false,
-                "lineInterpolation": "linear",
-                "lineWidth": 1,
-                "pointSize": 5,
-                "scaleDistribution": {
-                  "type": "linear"
-                },
-                "showPoints": "auto",
-                "showValues": false,
-                "spanNulls": false,
-                "stacking": {
-                  "group": "A",
-                  "mode": "none"
-                },
-                "thresholdsStyle": {
-                  "mode": "off"
-                }
-              },
-              "mappings": [],
-              "thresholds": {
-                "mode": "absolute",
-                "steps": [
-                  {
-                    "color": "green",
-                    "value": 0
-                  },
-                  {
-                    "color": "red",
-                    "value": 80
-                  }
-                ]
-              }
-            },
-            "overrides": []
-          },
-          "gridPos": {
-            "h": 8,
-            "w": 12,
-            "x": 12,
-            "y": 0
-          },
-          "id": 2,
-          "options": {
-            "legend": {
-              "calcs": [],
-              "displayMode": "list",
-              "placement": "bottom",
-              "showLegend": true
-            },
-            "tooltip": {
-              "hideZeros": false,
-              "mode": "single",
-              "sort": "none"
-            }
-          },
-          "pluginVersion": "12.4.0-pre",
-          "targets": [
-            {
-              "datasource": {
-                "type": "prometheus",
-                "uid": "test-uid"
-              },
-              "editorMode": "code",
-              "expr": "sum(counters_requests)",
-              "legendFormat": "__auto",
-              "range": true,
-              "refId": "A"
-            }
-          ],
-          "title": "works with group by var",
-          "type": "timeseries"
-        }
-      ],
-      "preload": false,
-      "schemaVersion": 42,
-      "tags": [],
-      "templating": {
-        "list": [
-          {
-            "current": {
-              "text": [
-                "a_legacy_label",
-                "app",
-                "exported_instance",
-                "exported_job"
-              ],
-              "value": [
-                "a_legacy_label",
-                "app",
-                "exported_instance",
-                "exported_job"
-              ]
-            },
-            "datasource": {
-              "type": "prometheus",
-              "uid": "test-uid"
-            },
-            "name": "Group by",
-            "type": "groupby"
-          }
-        ]
-      },
-      "time": {
-        "from": "now-6h",
-        "to": "now"
-      },
-      "timepicker": {},
-      "timezone": "browser",
-      "title": "groupby test",
-      "weekStart": ""
-    }
-  }

apps/dashboard/pkg/migration/conversion/testdata/migrated_dashboards_output/v1beta1-mig-v15.mimir_rollout_debugging.v42.v0alpha1.json

@@ -743,9 +743,7 @@
             "text": "prod",
             "value": "prod"
           },
-          "datasource": {
-            "uid": "$datasource"
-          },
+          "datasource": "$datasource",
           "hide": 0,
           "includeAll": true,
           "label": "cluster",
@@ -766,9 +764,7 @@
             "text": "prod",
             "value": "prod"
           },
-          "datasource": {
-            "uid": "$datasource"
-          },
+          "datasource": "$datasource",
           "hide": 0,
           "includeAll": false,
           "label": "namespace",

apps/dashboard/pkg/migration/conversion/testdata/migrated_dashboards_output/v1beta1-mig-v15.mimir_rollout_debugging.v42.v2alpha1.json

@@ -961,12 +961,8 @@
           "hide": "dontHide",
           "refresh": "onDashboardLoad",
           "skipUrlSync": false,
-          "datasource": {
-            "type": "",
-            "uid": "$datasource"
-          },
           "query": {
-            "kind": "",
+            "kind": "prometheus",
             "spec": {
               "__legacyStringValue": "label_values(up, job)"
             }
@@ -992,12 +988,8 @@
           "hide": "dontHide",
           "refresh": "onDashboardLoad",
           "skipUrlSync": false,
-          "datasource": {
-            "type": "",
-            "uid": "$datasource"
-          },
           "query": {
-            "kind": "",
+            "kind": "prometheus",
             "spec": {
               "__legacyStringValue": "label_values(up{job=~\"$cluster\"}, instance)"
             }

apps/dashboard/pkg/migration/conversion/testdata/migrated_dashboards_output/v1beta1-mig-v15.mimir_rollout_debugging.v42.v2beta1.json

@@ -978,11 +978,8 @@
           "skipUrlSync": false,
           "query": {
             "kind": "DataQuery",
-            "group": "",
+            "group": "prometheus",
             "version": "v0",
-            "datasource": {
-              "name": "$datasource"
-            },
             "spec": {
               "__legacyStringValue": "label_values(up, job)"
             }
@@ -1010,11 +1007,8 @@
           "skipUrlSync": false,
           "query": {
             "kind": "DataQuery",
-            "group": "",
+            "group": "prometheus",
             "version": "v0",
-            "datasource": {
-              "name": "$datasource"
-            },
             "spec": {
               "__legacyStringValue": "label_values(up{job=~\"$cluster\"}, instance)"
             }

apps/dashboard/pkg/migration/conversion/testdata/output/migrated_dev_dashboards/feature-templating/v0alpha1.testdata-nested-variables.v42.v2alpha1.json

@@ -120,7 +120,7 @@
                         "value": [
                           {
                             "title": "filter",
-                            "url": "http://localhost:3000/d/-Y-tnEDWk/templating-nested-template-variables?var-datacenter=${__data.fields[datacenter]}\u0026var-server=${__value.raw}"
+                            "url": "http://localhost:3000/d/-Y-tnEDWk/templating-nested-template-variables?var-datacenter=${__data.fields[datacenter]}\u0026var-server=${__value.raw}"
                           }
                         ]
                       }

apps/dashboard/pkg/migration/conversion/testdata/output/migrated_dev_dashboards/feature-templating/v0alpha1.testdata-nested-variables.v42.v2beta1.json

@@ -124,7 +124,7 @@
                         "value": [
                           {
                             "title": "filter",
-                            "url": "http://localhost:3000/d/-Y-tnEDWk/templating-nested-template-variables?var-datacenter=${__data.fields[datacenter]}\u0026var-server=${__value.raw}"
+                            "url": "http://localhost:3000/d/-Y-tnEDWk/templating-nested-template-variables?var-datacenter=${__data.fields[datacenter]}\u0026var-server=${__value.raw}"
                           }
                         ]
                       }

apps/dashboard/pkg/migration/conversion/testdata/output/migrated_dev_dashboards/panel-gauge/v0alpha1.gauge_tests_new.v42.v1beta1.json

@@ -2051,4 +2051,4 @@
       "storedVersion": "v0alpha1"
     }
   }
-}
+}

apps/dashboard/pkg/migration/conversion/testdata/output/migrated_dev_dashboards/panel-gauge/v0alpha1.gauge_tests_new.v42.v2alpha1.json

@@ -2691,4 +2691,4 @@
       "storedVersion": "v0alpha1"
     }
   }
-}
+}

apps/dashboard/pkg/migration/conversion/testdata/output/migrated_dev_dashboards/panel-gauge/v0alpha1.gauge_tests_new.v42.v2beta1.json

@@ -2764,4 +2764,4 @@
       "storedVersion": "v0alpha1"
     }
   }
-}
+}

apps/dashboard/pkg/migration/conversion/testdata/output/migrated_dev_dashboards/panel-gauge/v0alpha1.gauge_tests_old_to_new.v42.v1beta1.json

@@ -1173,4 +1173,4 @@
       "storedVersion": "v0alpha1"
     }
   }
-}
+}

apps/dashboard/pkg/migration/conversion/testdata/output/migrated_dev_dashboards/panel-gauge/v0alpha1.gauge_tests_old_to_new.v42.v2alpha1.json

@@ -1618,4 +1618,4 @@
       "storedVersion": "v0alpha1"
     }
   }
-}
+}

apps/dashboard/pkg/migration/conversion/testdata/output/migrated_dev_dashboards/panel-gauge/v0alpha1.gauge_tests_old_to_new.v42.v2beta1.json

@@ -1670,4 +1670,4 @@
       "storedVersion": "v0alpha1"
     }
   }
-}
+}

apps/dashboard/pkg/migration/conversion/testdata/output/v1beta1.bom-in-links.v0alpha1.json

@@ -1,161 +0,0 @@
-{
-  "kind": "Dashboard",
-  "apiVersion": "dashboard.grafana.app/v0alpha1",
-  "metadata": {
-    "name": "bom-in-links-test",
-    "namespace": "org-1",
-    "labels": {
-      "test": "bom-stripping"
-    }
-  },
-  "spec": {
-    "description": "Testing that BOM characters are stripped from URLs during conversion",
-    "editable": true,
-    "links": [
-      {
-        "icon": "external link",
-        "targetBlank": true,
-        "title": "Dashboard link with BOM",
-        "type": "link",
-        "url": "http://example.com?var=${datasource}\u0026other=value"
-      }
-    ],
-    "panels": [
-      {
-        "fieldConfig": {
-          "defaults": {
-            "custom": {},
-            "mappings": [],
-            "thresholds": {
-              "mode": "absolute",
-              "steps": [
-                {
-                  "color": "green"
-                },
-                {
-                  "color": "red",
-                  "value": 80
-                }
-              ]
-            }
-          },
-          "overrides": [
-            {
-              "matcher": {
-                "id": "byName",
-                "options": "server"
-              },
-              "properties": [
-                {
-                  "id": "links",
-                  "value": [
-                    {
-                      "title": "Override link with BOM",
-                      "url": "http://localhost:3000/d/test?var-datacenter=${__data.fields[datacenter]}\u0026var-server=${__value.raw}"
-                    }
-                  ]
-                }
-              ]
-            }
-          ]
-        },
-        "gridPos": {
-          "h": 8,
-          "w": 12,
-          "x": 0,
-          "y": 0
-        },
-        "id": 1,
-        "links": [
-          {
-            "targetBlank": true,
-            "title": "Panel data link with BOM",
-            "url": "http://example.com/${__data.fields.cluster}\u0026var=value"
-          }
-        ],
-        "targets": [
-          {
-            "datasource": {
-              "type": "prometheus",
-              "uid": "test-ds"
-            },
-            "refId": "A"
-          }
-        ],
-        "title": "Panel with BOM in field config override links",
-        "type": "table"
-      },
-      {
-        "fieldConfig": {
-          "defaults": {
-            "links": [
-              {
-                "targetBlank": false,
-                "title": "Field config default link with BOM",
-                "url": "http://example.com?field=${__field.name}\u0026value=${__value.raw}"
-              }
-            ]
-          },
-          "overrides": []
-        },
-        "gridPos": {
-          "h": 8,
-          "w": 12,
-          "x": 12,
-          "y": 0
-        },
-        "id": 2,
-        "options": {
-          "dataLinks": [
-            {
-              "targetBlank": true,
-              "title": "Options data link with BOM",
-              "url": "http://example.com?series=${__series.name}\u0026time=${__value.time}"
-            }
-          ],
-          "legend": {
-            "displayMode": "list",
-            "placement": "bottom",
-            "showLegend": true
-          }
-        },
-        "targets": [
-          {
-            "datasource": {
-              "type": "prometheus",
-              "uid": "test-ds"
-            },
-            "refId": "A"
-          }
-        ],
-        "title": "Panel with BOM in options dataLinks",
-        "type": "timeseries"
-      }
-    ],
-    "schemaVersion": 42,
-    "tags": [
-      "test",
-      "bom"
-    ],
-    "time": {
-      "from": "now-6h",
-      "to": "now"
-    },
-    "timepicker": {
-      "refresh_intervals": [
-        "5s",
-        "10s",
-        "30s",
-        "1m",
-        "5m"
-      ]
-    },
-    "title": "BOM Stripping Test Dashboard"
-  },
-  "status": {
-    "conversion": {
-      "failed": false,
-      "storedVersion": "v1beta1"
-    }
-  }
-}

apps/dashboard/pkg/migration/conversion/testdata/output/v1beta1.bom-in-links.v2alpha1.json

@@ -1,242 +0,0 @@
-{
-  "kind": "Dashboard",
-  "apiVersion": "dashboard.grafana.app/v2alpha1",
-  "metadata": {
-    "name": "bom-in-links-test",
-    "namespace": "org-1",
-    "labels": {
-      "test": "bom-stripping"
-    }
-  },
-  "spec": {
-    "annotations": [],
-    "cursorSync": "Off",
-    "description": "Testing that BOM characters are stripped from URLs during conversion",
-    "editable": true,
-    "elements": {
-      "panel-1": {
-        "kind": "Panel",
-        "spec": {
-          "id": 1,
-          "title": "Panel with BOM in field config override links",
-          "description": "",
-          "links": [
-            {
-              "title": "Panel data link with BOM",
-              "url": "http://example.com/${__data.fields.cluster}\u0026var=value",
-              "targetBlank": true
-            }
-          ],
-          "data": {
-            "kind": "QueryGroup",
-            "spec": {
-              "queries": [
-                {
-                  "kind": "PanelQuery",
-                  "spec": {
-                    "query": {
-                      "kind": "prometheus",
-                      "spec": {}
-                    },
-                    "datasource": {
-                      "type": "prometheus",
-                      "uid": "test-ds"
-                    },
-                    "refId": "A",
-                    "hidden": false
-                  }
-                }
-              ],
-              "transformations": [],
-              "queryOptions": {}
-            }
-          },
-          "vizConfig": {
-            "kind": "table",
-            "spec": {
-              "pluginVersion": "",
-              "options": {},
-              "fieldConfig": {
-                "defaults": {
-                  "thresholds": {
-                    "mode": "absolute",
-                    "steps": [
-                      {
-                        "value": null,
-                        "color": "green"
-                      },
-                      {
-                        "value": 80,
-                        "color": "red"
-                      }
-                    ]
-                  }
-                },
-                "overrides": [
-                  {
-                    "matcher": {
-                      "id": "byName",
-                      "options": "server"
-                    },
-                    "properties": [
-                      {
-                        "id": "links",
-                        "value": [
-                          {
-                            "title": "Override link with BOM",
-                            "url": "http://localhost:3000/d/test?var-datacenter=${__data.fields[datacenter]}\u0026var-server=${__value.raw}"
-                          }
-                        ]
-                      }
-                    ]
-                  }
-                ]
-              }
-            }
-          }
-        }
-      },
-      "panel-2": {
-        "kind": "Panel",
-        "spec": {
-          "id": 2,
-          "title": "Panel with BOM in options dataLinks",
-          "description": "",
-          "links": [],
-          "data": {
-            "kind": "QueryGroup",
-            "spec": {
-              "queries": [
-                {
-                  "kind": "PanelQuery",
-                  "spec": {
-                    "query": {
-                      "kind": "prometheus",
-                      "spec": {}
-                    },
-                    "datasource": {
-                      "type": "prometheus",
-                      "uid": "test-ds"
-                    },
-                    "refId": "A",
-                    "hidden": false
-                  }
-                }
-              ],
-              "transformations": [],
-              "queryOptions": {}
-            }
-          },
-          "vizConfig": {
-            "kind": "timeseries",
-            "spec": {
-              "pluginVersion": "",
-              "options": {
-                "dataLinks": [
-                  {
-                    "targetBlank": true,
-                    "title": "Options data link with BOM",
-                    "url": "http://example.com?series=${__series.name}\u0026time=${__value.time}"
-                  }
-                ],
-                "legend": {
-                  "displayMode": "list",
-                  "placement": "bottom",
-                  "showLegend": true
-                }
-              },
-              "fieldConfig": {
-                "defaults": {
-                  "links": [
-                    {
-                      "targetBlank": false,
-                      "title": "Field config default link with BOM",
-                      "url": "http://example.com?field=${__field.name}\u0026value=${__value.raw}"
-                    }
-                  ]
-                },
-                "overrides": []
-              }
-            }
-          }
-        }
-      }
-    },
-    "layout": {
-      "kind": "GridLayout",
-      "spec": {
-        "items": [
-          {
-            "kind": "GridLayoutItem",
-            "spec": {
-              "x": 0,
-              "y": 0,
-              "width": 12,
-              "height": 8,
-              "element": {
-                "kind": "ElementReference",
-                "name": "panel-1"
-              }
-            }
-          },
-          {
-            "kind": "GridLayoutItem",
-            "spec": {
-              "x": 12,
-              "y": 0,
-              "width": 12,
-              "height": 8,
-              "element": {
-                "kind": "ElementReference",
-                "name": "panel-2"
-              }
-            }
-          }
-        ]
-      }
-    },
-    "links": [
-      {
-        "title": "Dashboard link with BOM",
-        "type": "link",
-        "icon": "external link",
-        "tooltip": "",
-        "url": "http://example.com?var=${datasource}\u0026other=value",
-        "tags": [],
-        "asDropdown": false,
-        "targetBlank": true,
-        "includeVars": false,
-        "keepTime": false
-      }
-    ],
-    "liveNow": false,
-    "preload": false,
-    "tags": [
-      "test",
-      "bom"
-    ],
-    "timeSettings": {
-      "timezone": "browser",
-      "from": "now-6h",
-      "to": "now",
-      "autoRefresh": "",
-      "autoRefreshIntervals": [
-        "5s",
-        "10s",
-        "30s",
-        "1m",
-        "5m"
-      ],
-      "hideTimepicker": false,
-      "fiscalYearStartMonth": 0
-    },
-    "title": "BOM Stripping Test Dashboard",
-    "variables": []
-  },
-  "status": {
-    "conversion": {
-      "failed": false,
-      "storedVersion": "v1beta1"
-    }
-  }
-}

apps/dashboard/pkg/migration/conversion/testdata/output/v1beta1.bom-in-links.v2beta1.json

@@ -1,246 +0,0 @@
-{
-  "kind": "Dashboard",
-  "apiVersion": "dashboard.grafana.app/v2beta1",
-  "metadata": {
-    "name": "bom-in-links-test",
-    "namespace": "org-1",
-    "labels": {
-      "test": "bom-stripping"
-    }
-  },
-  "spec": {
-    "annotations": [],
-    "cursorSync": "Off",
-    "description": "Testing that BOM characters are stripped from URLs during conversion",
-    "editable": true,
-    "elements": {
-      "panel-1": {
-        "kind": "Panel",
-        "spec": {
-          "id": 1,
-          "title": "Panel with BOM in field config override links",
-          "description": "",
-          "links": [
-            {
-              "title": "Panel data link with BOM",
-              "url": "http://example.com/${__data.fields.cluster}\u0026var=value",
-              "targetBlank": true
-            }
-          ],
-          "data": {
-            "kind": "QueryGroup",
-            "spec": {
-              "queries": [
-                {
-                  "kind": "PanelQuery",
-                  "spec": {
-                    "query": {
-                      "kind": "DataQuery",
-                      "group": "prometheus",
-                      "version": "v0",
-                      "datasource": {
-                        "name": "test-ds"
-                      },
-                      "spec": {}
-                    },
-                    "refId": "A",
-                    "hidden": false
-                  }
-                }
-              ],
-              "transformations": [],
-              "queryOptions": {}
-            }
-          },
-          "vizConfig": {
-            "kind": "VizConfig",
-            "group": "table",
-            "version": "",
-            "spec": {
-              "options": {},
-              "fieldConfig": {
-                "defaults": {
-                  "thresholds": {
-                    "mode": "absolute",
-                    "steps": [
-                      {
-                        "value": null,
-                        "color": "green"
-                      },
-                      {
-                        "value": 80,
-                        "color": "red"
-                      }
-                    ]
-                  }
-                },
-                "overrides": [
-                  {
-                    "matcher": {
-                      "id": "byName",
-                      "options": "server"
-                    },
-                    "properties": [
-                      {
-                        "id": "links",
-                        "value": [
-                          {
-                            "title": "Override link with BOM",
-                            "url": "http://localhost:3000/d/test?var-datacenter=${__data.fields[datacenter]}\u0026var-server=${__value.raw}"
-                          }
-                        ]
-                      }
-                    ]
-                  }
-                ]
-              }
-            }
-          }
-        }
-      },
-      "panel-2": {
-        "kind": "Panel",
-        "spec": {
-          "id": 2,
-          "title": "Panel with BOM in options dataLinks",
-          "description": "",
-          "links": [],
-          "data": {
-            "kind": "QueryGroup",
-            "spec": {
-              "queries": [
-                {
-                  "kind": "PanelQuery",
-                  "spec": {
-                    "query": {
-                      "kind": "DataQuery",
-                      "group": "prometheus",
-                      "version": "v0",
-                      "datasource": {
-                        "name": "test-ds"
-                      },
-                      "spec": {}
-                    },
-                    "refId": "A",
-                    "hidden": false
-                  }
-                }
-              ],
-              "transformations": [],
-              "queryOptions": {}
-            }
-          },
-          "vizConfig": {
-            "kind": "VizConfig",
-            "group": "timeseries",
-            "version": "",
-            "spec": {
-              "options": {
-                "dataLinks": [
-                  {
-                    "targetBlank": true,
-                    "title": "Options data link with BOM",
-                    "url": "http://example.com?series=${__series.name}\u0026time=${__value.time}"
-                  }
-                ],
-                "legend": {
-                  "displayMode": "list",
-                  "placement": "bottom",
-                  "showLegend": true
-                }
-              },
-              "fieldConfig": {
-                "defaults": {
-                  "links": [
-                    {
-                      "targetBlank": false,
-                      "title": "Field config default link with BOM",
-                      "url": "http://example.com?field=${__field.name}\u0026value=${__value.raw}"
-                    }
-                  ]
-                },
-                "overrides": []
-              }
-            }
-          }
-        }
-      }
-    },
-    "layout": {
-      "kind": "GridLayout",
-      "spec": {
-        "items": [
-          {
-            "kind": "GridLayoutItem",
-            "spec": {
-              "x": 0,
-              "y": 0,
-              "width": 12,
-              "height": 8,
-              "element": {
-                "kind": "ElementReference",
-                "name": "panel-1"
-              }
-            }
-          },
-          {
-            "kind": "GridLayoutItem",
-            "spec": {
-              "x": 12,
-              "y": 0,
-              "width": 12,
-              "height": 8,
-              "element": {
-                "kind": "ElementReference",
-                "name": "panel-2"
-              }
-            }
-          }
-        ]
-      }
-    },
-    "links": [
-      {
-        "title": "Dashboard link with BOM",
-        "type": "link",
-        "icon": "external link",
-        "tooltip": "",
-        "url": "http://example.com?var=${datasource}\u0026other=value",
-        "tags": [],
-        "asDropdown": false,
-        "targetBlank": true,
-        "includeVars": false,
-        "keepTime": false
-      }
-    ],
-    "liveNow": false,
-    "preload": false,
-    "tags": [
-      "test",
-      "bom"
-    ],
-    "timeSettings": {
-      "timezone": "browser",
-      "from": "now-6h",
-      "to": "now",
-      "autoRefresh": "",
-      "autoRefreshIntervals": [
-        "5s",
-        "10s",
-        "30s",
-        "1m",
-        "5m"
-      ],
-      "hideTimepicker": false,
-      "fiscalYearStartMonth": 0
-    },
-    "title": "BOM Stripping Test Dashboard",
-    "variables": []
-  },
-  "status": {
-    "conversion": {
-      "failed": false,
-      "storedVersion": "v1beta1"
-    }
-  }
-}

apps/dashboard/pkg/migration/conversion/testdata/output/v1beta1.groupby.v0alpha1.json

@@ -1,172 +0,0 @@
-{
-  "kind": "DashboardWithAccessInfo",
-  "apiVersion": "dashboard.grafana.app/v0alpha1",
-  "metadata": {
-    "name": "groupby-test"
-  },
-  "spec": {
-    "annotations": {
-      "list": [
-        {
-          "builtIn": 1,
-          "datasource": {
-            "type": "grafana",
-            "uid": "-- Grafana --"
-          },
-          "enable": true,
-          "hide": true,
-          "iconColor": "rgba(0, 211, 255, 1)",
-          "name": "Annotations \u0026 Alerts",
-          "type": "dashboard"
-        }
-      ]
-    },
-    "editable": true,
-    "fiscalYearStartMonth": 0,
-    "graphTooltip": 0,
-    "links": [],
-    "panels": [
-      {
-        "datasource": {
-          "type": "prometheus",
-          "uid": "test-uid"
-        },
-        "fieldConfig": {
-          "defaults": {
-            "color": {
-              "mode": "palette-classic"
-            },
-            "custom": {
-              "axisBorderShow": false,
-              "axisCenteredZero": false,
-              "axisColorMode": "text",
-              "axisLabel": "",
-              "axisPlacement": "auto",
-              "barAlignment": 0,
-              "barWidthFactor": 0.6,
-              "drawStyle": "line",
-              "fillOpacity": 0,
-              "gradientMode": "none",
-              "hideFrom": {
-                "legend": false,
-                "tooltip": false,
-                "viz": false
-              },
-              "insertNulls": false,
-              "lineInterpolation": "linear",
-              "lineWidth": 1,
-              "pointSize": 5,
-              "scaleDistribution": {
-                "type": "linear"
-              },
-              "showPoints": "auto",
-              "showValues": false,
-              "spanNulls": false,
-              "stacking": {
-                "group": "A",
-                "mode": "none"
-              },
-              "thresholdsStyle": {
-                "mode": "off"
-              }
-            },
-            "mappings": [],
-            "thresholds": {
-              "mode": "absolute",
-              "steps": [
-                {
-                  "color": "green",
-                  "value": 0
-                },
-                {
-                  "color": "red",
-                  "value": 80
-                }
-              ]
-            }
-          },
-          "overrides": []
-        },
-        "gridPos": {
-          "h": 8,
-          "w": 12,
-          "x": 12,
-          "y": 0
-        },
-        "id": 2,
-        "options": {
-          "legend": {
-            "calcs": [],
-            "displayMode": "list",
-            "placement": "bottom",
-            "showLegend": true
-          },
-          "tooltip": {
-            "hideZeros": false,
-            "mode": "single",
-            "sort": "none"
-          }
-        },
-        "pluginVersion": "12.4.0-pre",
-        "targets": [
-          {
-            "datasource": {
-              "type": "prometheus",
-              "uid": "test-uid"
-            },
-            "editorMode": "code",
-            "expr": "sum(counters_requests)",
-            "legendFormat": "__auto",
-            "range": true,
-            "refId": "A"
-          }
-        ],
-        "title": "works with group by var",
-        "type": "timeseries"
-      }
-    ],
-    "preload": false,
-    "schemaVersion": 42,
-    "tags": [],
-    "templating": {
-      "list": [
-        {
-          "current": {
-            "text": [
-              "a_legacy_label",
-              "app",
-              "exported_instance",
-              "exported_job"
-            ],
-            "value": [
-              "a_legacy_label",
-              "app",
-              "exported_instance",
-              "exported_job"
-            ]
-          },
-          "datasource": {
-            "type": "prometheus",
-            "uid": "test-uid"
-          },
-          "name": "Group by",
-          "type": "groupby"
-        }
-      ]
-    },
-    "time": {
-      "from": "now-6h",
-      "to": "now"
-    },
-    "timepicker": {},
-    "timezone": "browser",
-    "title": "groupby test",
-    "weekStart": ""
-  },
-  "status": {
-    "conversion": {
-      "failed": false,
-      "storedVersion": "v1beta1"
-    }
-  }
-}

apps/dashboard/pkg/migration/conversion/testdata/output/v1beta1.groupby.v2alpha1.json

@@ -1,229 +0,0 @@
-{
-  "kind": "DashboardWithAccessInfo",
-  "apiVersion": "dashboard.grafana.app/v2alpha1",
-  "metadata": {
-    "name": "groupby-test"
-  },
-  "spec": {
-    "annotations": [
-      {
-        "kind": "AnnotationQuery",
-        "spec": {
-          "datasource": {
-            "type": "grafana",
-            "uid": "-- Grafana --"
-          },
-          "query": {
-            "kind": "grafana",
-            "spec": {}
-          },
-          "enable": true,
-          "hide": true,
-          "iconColor": "rgba(0, 211, 255, 1)",
-          "name": "Annotations \u0026 Alerts",
-          "builtIn": true,
-          "legacyOptions": {
-            "type": "dashboard"
-          }
-        }
-      }
-    ],
-    "cursorSync": "Off",
-    "editable": true,
-    "elements": {
-      "panel-2": {
-        "kind": "Panel",
-        "spec": {
-          "id": 2,
-          "title": "works with group by var",
-          "description": "",
-          "links": [],
-          "data": {
-            "kind": "QueryGroup",
-            "spec": {
-              "queries": [
-                {
-                  "kind": "PanelQuery",
-                  "spec": {
-                    "query": {
-                      "kind": "prometheus",
-                      "spec": {
-                        "editorMode": "code",
-                        "expr": "sum(counters_requests)",
-                        "legendFormat": "__auto",
-                        "range": true
-                      }
-                    },
-                    "datasource": {
-                      "type": "prometheus",
-                      "uid": "test-uid"
-                    },
-                    "refId": "A",
-                    "hidden": false
-                  }
-                }
-              ],
-              "transformations": [],
-              "queryOptions": {}
-            }
-          },
-          "vizConfig": {
-            "kind": "timeseries",
-            "spec": {
-              "pluginVersion": "12.4.0-pre",
-              "options": {
-                "legend": {
-                  "calcs": [],
-                  "displayMode": "list",
-                  "placement": "bottom",
-                  "showLegend": true
-                },
-                "tooltip": {
-                  "hideZeros": false,
-                  "mode": "single",
-                  "sort": "none"
-                }
-              },
-              "fieldConfig": {
-                "defaults": {
-                  "thresholds": {
-                    "mode": "absolute",
-                    "steps": [
-                      {
-                        "value": 0,
-                        "color": "green"
-                      },
-                      {
-                        "value": 80,
-                        "color": "red"
-                      }
-                    ]
-                  },
-                  "color": {
-                    "mode": "palette-classic"
-                  },
-                  "custom": {
-                    "axisBorderShow": false,
-                    "axisCenteredZero": false,
-                    "axisColorMode": "text",
-                    "axisLabel": "",
-                    "axisPlacement": "auto",
-                    "barAlignment": 0,
-                    "barWidthFactor": 0.6,
-                    "drawStyle": "line",
-                    "fillOpacity": 0,
-                    "gradientMode": "none",
-                    "hideFrom": {
-                      "legend": false,
-                      "tooltip": false,
-                      "viz": false
-                    },
-                    "insertNulls": false,
-                    "lineInterpolation": "linear",
-                    "lineWidth": 1,
-                    "pointSize": 5,
-                    "scaleDistribution": {
-                      "type": "linear"
-                    },
-                    "showPoints": "auto",
-                    "showValues": false,
-                    "spanNulls": false,
-                    "stacking": {
-                      "group": "A",
-                      "mode": "none"
-                    },
-                    "thresholdsStyle": {
-                      "mode": "off"
-                    }
-                  }
-                },
-                "overrides": []
-              }
-            }
-          }
-        }
-      }
-    },
-    "layout": {
-      "kind": "GridLayout",
-      "spec": {
-        "items": [
-          {
-            "kind": "GridLayoutItem",
-            "spec": {
-              "x": 12,
-              "y": 0,
-              "width": 12,
-              "height": 8,
-              "element": {
-                "kind": "ElementReference",
-                "name": "panel-2"
-              }
-            }
-          }
-        ]
-      }
-    },
-    "links": [],
-    "liveNow": false,
-    "preload": false,
-    "tags": [],
-    "timeSettings": {
-      "timezone": "browser",
-      "from": "now-6h",
-      "to": "now",
-      "autoRefresh": "",
-      "autoRefreshIntervals": [
-        "5s",
-        "10s",
-        "30s",
-        "1m",
-        "5m",
-        "15m",
-        "30m",
-        "1h",
-        "2h",
-        "1d"
-      ],
-      "hideTimepicker": false,
-      "fiscalYearStartMonth": 0
-    },
-    "title": "groupby test",
-    "variables": [
-      {
-        "kind": "GroupByVariable",
-        "spec": {
-          "name": "Group by",
-          "datasource": {
-            "type": "prometheus",
-            "uid": "test-uid"
-          },
-          "current": {
-            "text": [
-              "a_legacy_label",
-              "app",
-              "exported_instance",
-              "exported_job"
-            ],
-            "value": [
-              "a_legacy_label",
-              "app",
-              "exported_instance",
-              "exported_job"
-            ]
-          },
-          "options": [],
-          "multi": true,
-          "hide": "dontHide",
-          "skipUrlSync": false
-        }
-      }
-    ]
-  },
-  "status": {
-    "conversion": {
-      "failed": false,
-      "storedVersion": "v1beta1"
-    }
-  }
-}

apps/dashboard/pkg/migration/conversion/testdata/output/v1beta1.groupby.v2beta1.json

@@ -1,232 +0,0 @@
-{
-  "kind": "DashboardWithAccessInfo",
-  "apiVersion": "dashboard.grafana.app/v2beta1",
-  "metadata": {
-    "name": "groupby-test"
-  },
-  "spec": {
-    "annotations": [
-      {
-        "kind": "AnnotationQuery",
-        "spec": {
-          "query": {
-            "kind": "DataQuery",
-            "group": "grafana",
-            "version": "v0",
-            "datasource": {
-              "name": "-- Grafana --"
-            },
-            "spec": {}
-          },
-          "enable": true,
-          "hide": true,
-          "iconColor": "rgba(0, 211, 255, 1)",
-          "name": "Annotations \u0026 Alerts",
-          "builtIn": true,
-          "legacyOptions": {
-            "type": "dashboard"
-          }
-        }
-      }
-    ],
-    "cursorSync": "Off",
-    "editable": true,
-    "elements": {
-      "panel-2": {
-        "kind": "Panel",
-        "spec": {
-          "id": 2,
-          "title": "works with group by var",
-          "description": "",
-          "links": [],
-          "data": {
-            "kind": "QueryGroup",
-            "spec": {
-              "queries": [
-                {
-                  "kind": "PanelQuery",
-                  "spec": {
-                    "query": {
-                      "kind": "DataQuery",
-                      "group": "prometheus",
-                      "version": "v0",
-                      "datasource": {
-                        "name": "test-uid"
-                      },
-                      "spec": {
-                        "editorMode": "code",
-                        "expr": "sum(counters_requests)",
-                        "legendFormat": "__auto",
-                        "range": true
-                      }
-                    },
-                    "refId": "A",
-                    "hidden": false
-                  }
-                }
-              ],
-              "transformations": [],
-              "queryOptions": {}
-            }
-          },
-          "vizConfig": {
-            "kind": "VizConfig",
-            "group": "timeseries",
-            "version": "12.4.0-pre",
-            "spec": {
-              "options": {
-                "legend": {
-                  "calcs": [],
-                  "displayMode": "list",
-                  "placement": "bottom",
-                  "showLegend": true
-                },
-                "tooltip": {
-                  "hideZeros": false,
-                  "mode": "single",
-                  "sort": "none"
-                }
-              },
-              "fieldConfig": {
-                "defaults": {
-                  "thresholds": {
-                    "mode": "absolute",
-                    "steps": [
-                      {
-                        "value": 0,
-                        "color": "green"
-                      },
-                      {
-                        "value": 80,
-                        "color": "red"
-                      }
-                    ]
-                  },
-                  "color": {
-                    "mode": "palette-classic"
-                  },
-                  "custom": {
-                    "axisBorderShow": false,
-                    "axisCenteredZero": false,
-                    "axisColorMode": "text",
-                    "axisLabel": "",
-                    "axisPlacement": "auto",
-                    "barAlignment": 0,
-                    "barWidthFactor": 0.6,
-                    "drawStyle": "line",
-                    "fillOpacity": 0,
-                    "gradientMode": "none",
-                    "hideFrom": {
-                      "legend": false,
-                      "tooltip": false,
-                      "viz": false
-                    },
-                    "insertNulls": false,
-                    "lineInterpolation": "linear",
-                    "lineWidth": 1,
-                    "pointSize": 5,
-                    "scaleDistribution": {
-                      "type": "linear"
-                    },
-                    "showPoints": "auto",
-                    "showValues": false,
-                    "spanNulls": false,
-                    "stacking": {
-                      "group": "A",
-                      "mode": "none"
-                    },
-                    "thresholdsStyle": {
-                      "mode": "off"
-                    }
-                  }
-                },
-                "overrides": []
-              }
-            }
-          }
-        }
-      }
-    },
-    "layout": {
-      "kind": "GridLayout",
-      "spec": {
-        "items": [
-          {
-            "kind": "GridLayoutItem",
-            "spec": {
-              "x": 12,
-              "y": 0,
-              "width": 12,
-              "height": 8,
-              "element": {
-                "kind": "ElementReference",
-                "name": "panel-2"
-              }
-            }
-          }
-        ]
-      }
-    },
-    "links": [],
-    "liveNow": false,
-    "preload": false,
-    "tags": [],
-    "timeSettings": {
-      "timezone": "browser",
-      "from": "now-6h",
-      "to": "now",
-      "autoRefresh": "",
-      "autoRefreshIntervals": [
-        "5s",
-        "10s",
-        "30s",
-        "1m",
-        "5m",
-        "15m",
-        "30m",
-        "1h",
-        "2h",
-        "1d"
-      ],
-      "hideTimepicker": false,
-      "fiscalYearStartMonth": 0
-    },
-    "title": "groupby test",
-    "variables": [
-      {
-        "kind": "GroupByVariable",
-        "group": "prometheus",
-        "datasource": {
-          "name": "test-uid"
-        },
-        "spec": {
-          "name": "Group by",
-          "current": {
-            "text": [
-              "a_legacy_label",
-              "app",
-              "exported_instance",
-              "exported_job"
-            ],
-            "value": [
-              "a_legacy_label",
-              "app",
-              "exported_instance",
-              "exported_job"
-            ]
-          },
-          "options": [],
-          "multi": true,
-          "hide": "dontHide",
-          "skipUrlSync": false
-        }
-      }
-    ]
-  },
-  "status": {
-    "conversion": {
-      "failed": false,
-      "storedVersion": "v1beta1"
-    }
-  }
-}

apps/dashboard/pkg/migration/conversion/v0alpha1_to_v1beta1.go

@@ -2,7 +2,6 @@ package conversion
 
 import (
 	"context"
-	"strings"
 
 	"k8s.io/apimachinery/pkg/conversion"
 	"k8s.io/apiserver/pkg/endpoints/request"
@@ -80,57 +79,5 @@ func ConvertDashboard_V0_to_V1beta1(in *dashv0.Dashboard, out *dashv1.Dashboard,
 		return schemaversion.NewMigrationError(err.Error(), schemaversion.GetSchemaVersion(in.Spec.Object), schemaversion.LATEST_VERSION, "Convert_V0_to_V1")
 	}
 
-	// Normalize template variable datasources from string to object format
-	// This handles legacy dashboards where query variables have datasource: "$datasource" (string)
-	// instead of datasource: { uid: "$datasource" } (object)
-	// our migration pipeline in v36 doesn't address because this was not addressed historically
-	// in DashboardMigrator - see public/app/features/dashboard/state/DashboardMigrator.ts#L607
-	// Which means that we have schemaVersion: 42 dashboards where datasource variable references are still strings
-	normalizeTemplateVariableDatasources(out.Spec.Object)
-
 	return nil
 }
-
-// normalizeTemplateVariableDatasources converts template variable string datasources to object format.
-// Legacy dashboards may have query variables with datasource: "$datasource" (string).
-// This normalizes them to datasource: { uid: "$datasource" } for consistent V1→V2 conversion.
-func normalizeTemplateVariableDatasources(dashboard map[string]interface{}) {
-	templating, ok := dashboard["templating"].(map[string]interface{})
-	if !ok {
-		return
-	}
-
-	list, ok := templating["list"].([]interface{})
-	if !ok {
-		return
-	}
-
-	for _, variable := range list {
-		varMap, ok := variable.(map[string]interface{})
-		if !ok {
-			continue
-		}
-
-		varType, _ := varMap["type"].(string)
-		if varType != "query" {
-			continue
-		}
-
-		ds := varMap["datasource"]
-		if dsStr, ok := ds.(string); ok && isTemplateVariableRef(dsStr) {
-			// Convert string template variable reference to object format
-			varMap["datasource"] = map[string]interface{}{
-				"uid": dsStr,
-			}
-		}
-	}
-}
-
-// isTemplateVariableRef checks if a string is a Grafana template variable reference.
-// Template variables can be in the form: $varname or ${varname}
-func isTemplateVariableRef(s string) bool {
-	if s == "" {
-		return false
-	}
-	return strings.HasPrefix(s, "$") || strings.HasPrefix(s, "${")
-}

apps/dashboard/pkg/migration/conversion/v1beta1_to_v2alpha1.go

@@ -229,36 +229,6 @@ func getBoolField(m map[string]interface{}, key string, defaultValue bool) bool
 	return defaultValue
 }
 
-// stripBOM removes Byte Order Mark (BOM) characters from a string.
-// BOMs (U+FEFF) can be introduced through copy/paste from certain editors
-// and cause CUE validation errors ("illegal byte order mark").
-func stripBOM(s string) string {
-	return strings.ReplaceAll(s, "\ufeff", "")
-}
-
-// stripBOMFromInterface recursively strips BOM characters from all strings
-// in an interface{} value (map, slice, or string).
-func stripBOMFromInterface(v interface{}) interface{} {
-	switch val := v.(type) {
-	case string:
-		return stripBOM(val)
-	case map[string]interface{}:
-		result := make(map[string]interface{}, len(val))
-		for k, v := range val {
-			result[k] = stripBOMFromInterface(v)
-		}
-		return result
-	case []interface{}:
-		result := make([]interface{}, len(val))
-		for i, item := range val {
-			result[i] = stripBOMFromInterface(item)
-		}
-		return result
-	default:
-		return v
-	}
-}
-
 func getUnionField[T ~string](m map[string]interface{}, key string) *T {
 	if val, ok := m[key]; ok {
 		if str, ok := val.(string); ok && str != "" {
@@ -423,8 +393,7 @@ func transformLinks(dashboard map[string]interface{}) []dashv2alpha1.DashboardDa
 				// Optional field - only set if present
 				if url, exists := linkMap["url"]; exists {
 					if urlStr, ok := url.(string); ok {
-						cleanUrl := stripBOM(urlStr)
-						dashLink.Url = &cleanUrl
+						dashLink.Url = &urlStr
 					}
 				}
 
@@ -1216,10 +1185,6 @@ func buildQueryVariable(ctx context.Context, varMap map[string]interface{}, comm
 			// If no UID and no type, use default
 			datasourceType = getDefaultDatasourceType(ctx, dsIndexProvider)
 		}
-	} else if dsStr, ok := datasource.(string); ok && isTemplateVariable(dsStr) {
-		// Handle datasource variable reference (e.g., "$datasource")
-		// Only process template variables - other string values are not supported in V2 format
-		datasourceUID = dsStr
 	} else {
 		datasourceType = getDefaultDatasourceType(ctx, dsIndexProvider)
 	}
@@ -1567,10 +1532,6 @@ func buildAdhocVariable(ctx context.Context, varMap map[string]interface{}, comm
 			// If no UID and no type, use default
 			datasourceType = getDefaultDatasourceType(ctx, dsIndexProvider)
 		}
-	} else if dsStr, ok := datasource.(string); ok && isTemplateVariable(dsStr) {
-		// Handle datasource variable reference (e.g., "$datasource")
-		// Only process template variables - other string values are not supported in V2 format
-		datasourceUID = dsStr
 	} else {
 		datasourceType = getDefaultDatasourceType(ctx, dsIndexProvider)
 	}
@@ -1748,10 +1709,6 @@ func buildGroupByVariable(ctx context.Context, varMap map[string]interface{}, co
 
 		// Resolve Grafana datasource UID when type is "datasource" and UID is empty
 		datasourceUID = resolveGrafanaDatasourceUID(datasourceType, datasourceUID)
-	} else if dsStr, ok := datasource.(string); ok && isTemplateVariable(dsStr) {
-		// Handle datasource variable reference (e.g., "$datasource")
-		// Only process template variables - other string values are not supported in V2 format
-		datasourceUID = dsStr
 	} else {
 		datasourceType = getDefaultDatasourceType(ctx, dsIndexProvider)
 	}
@@ -1765,9 +1722,7 @@ func buildGroupByVariable(ctx context.Context, varMap map[string]interface{}, co
 			Hide:        commonProps.Hide,
 			SkipUrlSync: commonProps.SkipUrlSync,
 			Current:     buildVariableCurrent(varMap["current"]),
-			// We set it to true by default because GroupByVariable
-			// constructor defaults to multi: true
-			Multi: getBoolField(varMap, "multi", true),
+			Multi:       getBoolField(varMap, "multi", false),
 		},
 	}
 
@@ -2270,7 +2225,7 @@ func transformDataLinks(panelMap map[string]interface{}) []dashv2alpha1.Dashboar
 		if linkMap, ok := link.(map[string]interface{}); ok {
 			dataLink := dashv2alpha1.DashboardDataLink{
 				Title: schemaversion.GetStringValue(linkMap, "title"),
-				Url:   stripBOM(schemaversion.GetStringValue(linkMap, "url")),
+				Url:   schemaversion.GetStringValue(linkMap, "url"),
 			}
 			if _, exists := linkMap["targetBlank"]; exists {
 				targetBlank := getBoolField(linkMap, "targetBlank", false)
@@ -2362,12 +2317,6 @@ func buildVizConfig(panelMap map[string]interface{}) dashv2alpha1.DashboardVizCo
 		}
 	}
 
-	// Strip BOMs from options (may contain dataLinks with URLs that have BOMs)
-	cleanedOptions := stripBOMFromInterface(options)
-	if cleanedMap, ok := cleanedOptions.(map[string]interface{}); ok {
-		options = cleanedMap
-	}
-
 	// Build field config by mapping each field individually
 	fieldConfigSource := extractFieldConfigSource(fieldConfig)
 
@@ -2511,14 +2460,9 @@ func extractFieldConfigDefaults(defaults map[string]interface{}) dashv2alpha1.Da
 		hasDefaults = true
 	}
 
-	// Extract array field - strip BOMs from link URLs
+	// Extract array field
 	if linksArray, ok := extractArrayField(defaults, "links"); ok {
-		cleanedLinks := stripBOMFromInterface(linksArray)
-		if cleanedArray, ok := cleanedLinks.([]interface{}); ok {
-			fieldConfigDefaults.Links = cleanedArray
-		} else {
-			fieldConfigDefaults.Links = linksArray
-		}
+		fieldConfigDefaults.Links = linksArray
 		hasDefaults = true
 	}
 
@@ -2804,11 +2748,9 @@ func extractFieldConfigOverrides(fieldConfig map[string]interface{}) []dashv2alp
 				fieldOverride.Properties = make([]dashv2alpha1.DashboardDynamicConfigValue, 0, len(propertiesArray))
 				for _, property := range propertiesArray {
 					if propertyMap, ok := property.(map[string]interface{}); ok {
-						// Strip BOMs from property values (may contain links with URLs)
-						cleanedValue := stripBOMFromInterface(propertyMap["value"])
 						fieldOverride.Properties = append(fieldOverride.Properties, dashv2alpha1.DashboardDynamicConfigValue{
 							Id:    schemaversion.GetStringValue(propertyMap, "id"),
-							Value: cleanedValue,
+							Value: propertyMap["value"],
 						})
 					}
 				}

apps/dashboard/pkg/migration/testdata/dev-dashboards-output/panel-gauge/gauge_tests.v42.json

@@ -628,20 +628,6 @@
         }
       ],
       "title": "Only nulls and no user set min \u0026 max",
-      "transformations": [
-        {
-          "id": "convertFieldType",
-          "options":  {
-            "conversions": [
-              {
-                "destinationType": "number",
-                "targetField": "A-series"
-              }
-            ],
-            "fields": {}
-          }
-        }
-      ],
       "type": "gauge"
     },
     {
@@ -1193,4 +1179,4 @@
   "title": "Panel Tests - Gauge",
   "uid": "_5rDmaQiz",
   "weekStart": ""
-}
+}

apps/dashboard/pkg/migration/testdata/dev-dashboards-output/panel-gauge/gauge_tests_old_to_new.v42.json

@@ -485,7 +485,6 @@
       },
       "id": 12,
       "options": {
-        "displayName": "My gauge",
         "minVizHeight": 75,
         "minVizWidth": 75,
         "orientation": "auto",

apps/dashboard/pkg/migration/testdata/input/v15.mimir_rollout_debugging.json

@@ -654,9 +654,7 @@
           "text": "prod",
           "value": "prod"
         },
-        "datasource": {
-          "uid": "$datasource"
-        },
+        "datasource": "$datasource",
         "hide": 0,
         "includeAll": true,
         "label": "cluster",
@@ -679,9 +677,7 @@
           "text": "prod",
           "value": "prod"
         },
-        "datasource": {
-          "uid": "$datasource"
-        },
+        "datasource": "$datasource",
         "hide": 0,
         "includeAll": false,
         "label": "namespace",

apps/dashboard/pkg/migration/testdata/output/latest_version/v15.mimir_rollout_debugging.v42.json

@@ -737,9 +737,7 @@
           "text": "prod",
           "value": "prod"
         },
-        "datasource": {
-          "uid": "$datasource"
-        },
+        "datasource": "$datasource",
         "hide": 0,
         "includeAll": true,
         "label": "cluster",
@@ -760,9 +758,7 @@
           "text": "prod",
           "value": "prod"
         },
-        "datasource": {
-          "uid": "$datasource"
-        },
+        "datasource": "$datasource",
         "hide": 0,
         "includeAll": false,
         "label": "namespace",

apps/dashboard/pkg/migration/testdata/output/single_version/v15.mimir_rollout_debugging.v15.json

@@ -717,9 +717,7 @@
           "text": "prod",
           "value": "prod"
         },
-        "datasource": {
-          "uid": "$datasource"
-        },
+        "datasource": "$datasource",
         "hide": 0,
         "includeAll": true,
         "label": "cluster",
@@ -741,9 +739,7 @@
           "text": "prod",
           "value": "prod"
         },
-        "datasource": {
-          "uid": "$datasource"
-        },
+        "datasource": "$datasource",
         "hide": 0,
         "includeAll": false,
         "label": "namespace",

apps/iam/go.mod

@@ -223,7 +223,7 @@ require (
 	github.com/googleapis/enterprise-certificate-proxy v0.3.6 // indirect
 	github.com/googleapis/gax-go/v2 v2.15.0 // indirect
 	github.com/gorilla/mux v1.8.1 // indirect
-	github.com/grafana/alerting v0.0.0-20251231150637-b7821017d69f // indirect
+	github.com/grafana/alerting v0.0.0-20251212143239-491433b332b7 // indirect
 	github.com/grafana/authlib v0.0.0-20250930082137-a40e2c2b094f // indirect
 	github.com/grafana/authlib/types v0.0.0-20251119142549-be091cf2f4d4 // indirect
 	github.com/grafana/dataplane/sdata v0.0.9 // indirect

apps/iam/go.sum

@@ -827,8 +827,8 @@ github.com/gorilla/mux v1.8.1 h1:TuBL49tXwgrFYWhqrNgrUNEY92u81SPhu7sTdzQEiWY=
 github.com/gorilla/mux v1.8.1/go.mod h1:AKf9I4AEqPTmMytcMc0KkNouC66V3BtZ4qD5fmWSiMQ=
 github.com/gorilla/websocket v1.5.4-0.20250319132907-e064f32e3674 h1:JeSE6pjso5THxAzdVpqr6/geYxZytqFMBCOtn/ujyeo=
 github.com/gorilla/websocket v1.5.4-0.20250319132907-e064f32e3674/go.mod h1:r4w70xmWCQKmi1ONH4KIaBptdivuRPyosB9RmPlGEwA=
-github.com/grafana/alerting v0.0.0-20251231150637-b7821017d69f h1:Br4SaUL3dnVopKKNhDavCLgehw60jdtl/sIxdfzmVts=
-github.com/grafana/alerting v0.0.0-20251231150637-b7821017d69f/go.mod h1:l7v67cgP7x72ajB9UPZlumdrHqNztpKoqQ52cU8T3LU=
+github.com/grafana/alerting v0.0.0-20251212143239-491433b332b7 h1:ZzG/gCclEit9w0QUfQt9GURcOycAIGcsQAhY1u0AEX0=
+github.com/grafana/alerting v0.0.0-20251212143239-491433b332b7/go.mod h1:l7v67cgP7x72ajB9UPZlumdrHqNztpKoqQ52cU8T3LU=
 github.com/grafana/authlib v0.0.0-20250930082137-a40e2c2b094f h1:Cbm6OKkOcJ+7CSZsGsEJzktC/SIa5bxVeYKQLuYK86o=
 github.com/grafana/authlib v0.0.0-20250930082137-a40e2c2b094f/go.mod h1:axY0cdOg3q0TZHwpHnIz5x16xZ8ZBxJHShsSHHXcHQg=
 github.com/grafana/authlib/types v0.0.0-20251119142549-be091cf2f4d4 h1:Muoy+FMGrHj3GdFbvsMzUT7eusgii9PKf9L1ZaXDDbY=

apps/plugins/README.md

@@ -1,20 +0,0 @@
-# Plugins App
-
-API documentation is available at http://localhost:3000/swagger?api=plugins.grafana.app-v0alpha1
-
-## Codegen
-
-- Go: `make generate`
-- Frontend: Follow instructions in this [README](../..//packages/grafana-api-clients/README.md)
-
-## Plugin sync
-
-The plugin sync pushes the plugins loaded from disk to the plugins API.
-
-To enable, add these feature toggles in your `custom.ini`:
-
-```ini
-[feature_toggles]
-pluginInstallAPISync = true
-pluginStoreServiceLoading = true
-```

apps/plugins/go.mod

@@ -90,7 +90,7 @@ require (
 	github.com/google/gnostic-models v0.7.1 // indirect
 	github.com/google/go-cmp v0.7.0 // indirect
 	github.com/google/uuid v1.6.0 // indirect
-	github.com/grafana/alerting v0.0.0-20251231150637-b7821017d69f // indirect
+	github.com/grafana/alerting v0.0.0-20251212143239-491433b332b7 // indirect
 	github.com/grafana/authlib v0.0.0-20250930082137-a40e2c2b094f // indirect
 	github.com/grafana/authlib/types v0.0.0-20251119142549-be091cf2f4d4 // indirect
 	github.com/grafana/dataplane/sdata v0.0.9 // indirect

apps/plugins/go.sum

@@ -213,8 +213,8 @@ github.com/google/uuid v1.6.0 h1:NIvaJDMOsjHA8n1jAhLSgzrAzy1Hgr+hNrb57e+94F0=
 github.com/google/uuid v1.6.0/go.mod h1:TIyPZe4MgqvfeYDBFedMoGGpEw/LqOeaOT+nhxU+yHo=
 github.com/gorilla/websocket v1.5.4-0.20250319132907-e064f32e3674 h1:JeSE6pjso5THxAzdVpqr6/geYxZytqFMBCOtn/ujyeo=
 github.com/gorilla/websocket v1.5.4-0.20250319132907-e064f32e3674/go.mod h1:r4w70xmWCQKmi1ONH4KIaBptdivuRPyosB9RmPlGEwA=
-github.com/grafana/alerting v0.0.0-20251231150637-b7821017d69f h1:Br4SaUL3dnVopKKNhDavCLgehw60jdtl/sIxdfzmVts=
-github.com/grafana/alerting v0.0.0-20251231150637-b7821017d69f/go.mod h1:l7v67cgP7x72ajB9UPZlumdrHqNztpKoqQ52cU8T3LU=
+github.com/grafana/alerting v0.0.0-20251212143239-491433b332b7 h1:ZzG/gCclEit9w0QUfQt9GURcOycAIGcsQAhY1u0AEX0=
+github.com/grafana/alerting v0.0.0-20251212143239-491433b332b7/go.mod h1:l7v67cgP7x72ajB9UPZlumdrHqNztpKoqQ52cU8T3LU=
 github.com/grafana/authlib v0.0.0-20250930082137-a40e2c2b094f h1:Cbm6OKkOcJ+7CSZsGsEJzktC/SIa5bxVeYKQLuYK86o=
 github.com/grafana/authlib v0.0.0-20250930082137-a40e2c2b094f/go.mod h1:axY0cdOg3q0TZHwpHnIz5x16xZ8ZBxJHShsSHHXcHQg=
 github.com/grafana/authlib/types v0.0.0-20251119142549-be091cf2f4d4 h1:Muoy+FMGrHj3GdFbvsMzUT7eusgii9PKf9L1ZaXDDbY=

apps/provisioning/pkg/auth/access_checker.go

@@ -1,22 +0,0 @@
-package auth
-
-import (
-	"context"
-
-	authlib "github.com/grafana/authlib/types"
-	"github.com/grafana/grafana/pkg/apimachinery/identity"
-)
-
-//go:generate mockery --name AccessChecker --structname MockAccessChecker --inpackage --filename access_checker_mock.go --with-expecter
-
-// AccessChecker provides access control checks with optional role-based fallback.
-type AccessChecker interface {
-	// Check performs an access check and returns nil if allowed, or an appropriate
-	// API error if denied. If req.Namespace is empty, it will be filled from the
-	// identity's namespace.
-	Check(ctx context.Context, req authlib.CheckRequest, folder string) error
-
-	// WithFallbackRole returns an AccessChecker configured with the specified fallback role.
-	// Whether the fallback is actually applied depends on the implementation.
-	WithFallbackRole(role identity.RoleType) AccessChecker
-}

apps/provisioning/pkg/auth/access_checker_mock.go

@@ -1,135 +0,0 @@
-// Code generated by mockery v2.53.4. DO NOT EDIT.
-
-package auth
-
-import (
-	context "context"
-
-	identity "github.com/grafana/grafana/pkg/apimachinery/identity"
-	mock "github.com/stretchr/testify/mock"
-
-	types "github.com/grafana/authlib/types"
-)
-
-// MockAccessChecker is an autogenerated mock type for the AccessChecker type
-type MockAccessChecker struct {
-	mock.Mock
-}
-
-type MockAccessChecker_Expecter struct {
-	mock *mock.Mock
-}
-
-func (_m *MockAccessChecker) EXPECT() *MockAccessChecker_Expecter {
-	return &MockAccessChecker_Expecter{mock: &_m.Mock}
-}
-
-// Check provides a mock function with given fields: ctx, req, folder
-func (_m *MockAccessChecker) Check(ctx context.Context, req types.CheckRequest, folder string) error {
-	ret := _m.Called(ctx, req, folder)
-
-	if len(ret) == 0 {
-		panic("no return value specified for Check")
-	}
-
-	var r0 error
-	if rf, ok := ret.Get(0).(func(context.Context, types.CheckRequest, string) error); ok {
-		r0 = rf(ctx, req, folder)
-	} else {
-		r0 = ret.Error(0)
-	}
-
-	return r0
-}
-
-// MockAccessChecker_Check_Call is a *mock.Call that shadows Run/Return methods with type explicit version for method 'Check'
-type MockAccessChecker_Check_Call struct {
-	*mock.Call
-}
-
-// Check is a helper method to define mock.On call
-//   - ctx context.Context
-//   - req types.CheckRequest
-//   - folder string
-func (_e *MockAccessChecker_Expecter) Check(ctx interface{}, req interface{}, folder interface{}) *MockAccessChecker_Check_Call {
-	return &MockAccessChecker_Check_Call{Call: _e.mock.On("Check", ctx, req, folder)}
-}
-
-func (_c *MockAccessChecker_Check_Call) Run(run func(ctx context.Context, req types.CheckRequest, folder string)) *MockAccessChecker_Check_Call {
-	_c.Call.Run(func(args mock.Arguments) {
-		run(args[0].(context.Context), args[1].(types.CheckRequest), args[2].(string))
-	})
-	return _c
-}
-
-func (_c *MockAccessChecker_Check_Call) Return(_a0 error) *MockAccessChecker_Check_Call {
-	_c.Call.Return(_a0)
-	return _c
-}
-
-func (_c *MockAccessChecker_Check_Call) RunAndReturn(run func(context.Context, types.CheckRequest, string) error) *MockAccessChecker_Check_Call {
-	_c.Call.Return(run)
-	return _c
-}
-
-// WithFallbackRole provides a mock function with given fields: role
-func (_m *MockAccessChecker) WithFallbackRole(role identity.RoleType) AccessChecker {
-	ret := _m.Called(role)
-
-	if len(ret) == 0 {
-		panic("no return value specified for WithFallbackRole")
-	}
-
-	var r0 AccessChecker
-	if rf, ok := ret.Get(0).(func(identity.RoleType) AccessChecker); ok {
-		r0 = rf(role)
-	} else {
-		if ret.Get(0) != nil {
-			r0 = ret.Get(0).(AccessChecker)
-		}
-	}
-
-	return r0
-}
-
-// MockAccessChecker_WithFallbackRole_Call is a *mock.Call that shadows Run/Return methods with type explicit version for method 'WithFallbackRole'
-type MockAccessChecker_WithFallbackRole_Call struct {
-	*mock.Call
-}
-
-// WithFallbackRole is a helper method to define mock.On call
-//   - role identity.RoleType
-func (_e *MockAccessChecker_Expecter) WithFallbackRole(role interface{}) *MockAccessChecker_WithFallbackRole_Call {
-	return &MockAccessChecker_WithFallbackRole_Call{Call: _e.mock.On("WithFallbackRole", role)}
-}
-
-func (_c *MockAccessChecker_WithFallbackRole_Call) Run(run func(role identity.RoleType)) *MockAccessChecker_WithFallbackRole_Call {
-	_c.Call.Run(func(args mock.Arguments) {
-		run(args[0].(identity.RoleType))
-	})
-	return _c
-}
-
-func (_c *MockAccessChecker_WithFallbackRole_Call) Return(_a0 AccessChecker) *MockAccessChecker_WithFallbackRole_Call {
-	_c.Call.Return(_a0)
-	return _c
-}
-
-func (_c *MockAccessChecker_WithFallbackRole_Call) RunAndReturn(run func(identity.RoleType) AccessChecker) *MockAccessChecker_WithFallbackRole_Call {
-	_c.Call.Return(run)
-	return _c
-}
-
-// NewMockAccessChecker creates a new instance of MockAccessChecker. It also registers a testing interface on the mock and a cleanup function to assert the mocks expectations.
-// The first argument is typically a *testing.T value.
-func NewMockAccessChecker(t interface {
-	mock.TestingT
-	Cleanup(func())
-}) *MockAccessChecker {
-	mock := &MockAccessChecker{}
-	mock.Mock.Test(t)
-
-	t.Cleanup(func() { mock.AssertExpectations(t) })
-
-	return mock
-}

apps/provisioning/pkg/auth/round_tripper.go

@@ -1,4 +1,3 @@
-// Package auth provides authentication utilities for the provisioning API.
 package auth
 
 import (
@@ -7,6 +6,7 @@ import (
 	"net/http"
 
 	"github.com/grafana/authlib/authn"
+	"github.com/grafana/grafana/apps/provisioning/pkg/apis/provisioning/v0alpha1"
 	utilnet "k8s.io/apimachinery/pkg/util/net"
 )
 
@@ -15,61 +15,29 @@ type tokenExchanger interface {
 	Exchange(ctx context.Context, req authn.TokenExchangeRequest) (*authn.TokenExchangeResponse, error)
 }
 
-// RoundTripperOption configures optional behavior for the RoundTripper.
-type RoundTripperOption func(*RoundTripper)
-
-// ExtraAudience appends an additional audience to the token exchange request.
-//
-// This is primarily used by operators connecting to the multitenant aggregator,
-// where the token must include both the target API server's audience (e.g., dashboards,
-// folders) and the provisioning group audience. The provisioning group audience is
-// required so that the token passes the enforceManagerProperties check, which prevents
-// unauthorized updates to provisioned resources.
-//
-// Example:
-//
-//	authrt.NewRoundTripper(client, rt, "dashboards.grafana.app", authrt.ExtraAudience("provisioning.grafana.app"))
-func ExtraAudience(audience string) RoundTripperOption {
-	return func(rt *RoundTripper) {
-		rt.extraAudience = audience
-	}
-}
-
-// RoundTripper is an http.RoundTripper that performs token exchange before each request.
-// It exchanges the service's credentials for an access token scoped to the configured
-// audience(s), then injects that token into the outgoing request's X-Access-Token header.
+// RoundTripper injects an exchanged access token for the provisioning API into outgoing requests.
 type RoundTripper struct {
-	client        tokenExchanger
-	transport     http.RoundTripper
-	audience      string
-	extraAudience string
+	client    tokenExchanger
+	transport http.RoundTripper
+	audience  string
 }
 
-// NewRoundTripper creates a RoundTripper that exchanges tokens for each outgoing request.
-//
-// Parameters:
-//   - tokenExchangeClient: the client used to exchange credentials for access tokens
-//   - base: the underlying transport to delegate requests to after token injection
-//   - audience: the primary audience for the token (typically the target API server's group)
-//   - opts: optional configuration (e.g., ExtraAudience to include additional audiences)
-func NewRoundTripper(tokenExchangeClient tokenExchanger, base http.RoundTripper, audience string, opts ...RoundTripperOption) *RoundTripper {
-	rt := &RoundTripper{
+// NewRoundTripper constructs a RoundTripper that exchanges the provided token per request
+// and forwards the request to the provided base transport.
+func NewRoundTripper(tokenExchangeClient tokenExchanger, base http.RoundTripper, audience string) *RoundTripper {
+	return &RoundTripper{
 		client:    tokenExchangeClient,
 		transport: base,
 		audience:  audience,
 	}
-	for _, opt := range opts {
-		opt(rt)
-	}
-	return rt
 }
 
-// RoundTrip exchanges credentials for an access token and injects it into the request.
-// The token is scoped to all configured audiences and the wildcard namespace ("*").
 func (t *RoundTripper) RoundTrip(req *http.Request) (*http.Response, error) {
+	// when we want to write resources with the provisioning API, the audience needs to include provisioning
+	// so that it passes the check in enforceManagerProperties, which prevents others from updating provisioned resources
 	audiences := []string{t.audience}
-	if t.extraAudience != "" && t.extraAudience != t.audience {
-		audiences = append(audiences, t.extraAudience)
+	if t.audience != v0alpha1.GROUP {
+		audiences = append(audiences, v0alpha1.GROUP)
 	}
 
 	tokenResponse, err := t.client.Exchange(req.Context(), authn.TokenExchangeRequest{

apps/provisioning/pkg/auth/round_tripper_test.go

@@ -71,29 +71,16 @@ func TestRoundTripper_AudiencesAndNamespace(t *testing.T) {
 	tests := []struct {
 		name          string
 		audience      string
-		extraAudience string
 		wantAudiences []string
 	}{
 		{
-			name:          "uses only provided audience by default",
+			name:          "adds group when custom audience",
 			audience:      "example-audience",
-			wantAudiences: []string{"example-audience"},
-		},
-		{
-			name:          "uses only group audience by default",
-			audience:      v0alpha1.GROUP,
-			wantAudiences: []string{v0alpha1.GROUP},
-		},
-		{
-			name:          "extra audience adds provisioning group",
-			audience:      "example-audience",
-			extraAudience: v0alpha1.GROUP,
 			wantAudiences: []string{"example-audience", v0alpha1.GROUP},
 		},
 		{
-			name:          "extra audience no duplicate when same as primary",
+			name:          "no duplicate when group audience",
 			audience:      v0alpha1.GROUP,
-			extraAudience: v0alpha1.GROUP,
 			wantAudiences: []string{v0alpha1.GROUP},
 		},
 	}
@@ -101,15 +88,11 @@ func TestRoundTripper_AudiencesAndNamespace(t *testing.T) {
 	for _, tt := range tests {
 		t.Run(tt.name, func(t *testing.T) {
 			fx := &fakeExchanger{resp: &authn.TokenExchangeResponse{Token: "abc123"}}
-			var opts []RoundTripperOption
-			if tt.extraAudience != "" {
-				opts = append(opts, ExtraAudience(tt.extraAudience))
-			}
 			tr := NewRoundTripper(fx, roundTripperFunc(func(_ *http.Request) (*http.Response, error) {
 				rr := httptest.NewRecorder()
 				rr.WriteHeader(http.StatusOK)
 				return rr.Result(), nil
-			}), tt.audience, opts...)
+			}), tt.audience)
 
 			req, _ := http.NewRequestWithContext(context.Background(), http.MethodGet, "http://example", nil)
 			resp, err := tr.RoundTrip(req)

apps/provisioning/pkg/auth/session_access_checker.go

@@ -1,153 +0,0 @@
-package auth
-
-import (
-	"context"
-	"fmt"
-	"strings"
-
-	apierrors "k8s.io/apimachinery/pkg/api/errors"
-	"k8s.io/apimachinery/pkg/runtime/schema"
-
-	authlib "github.com/grafana/authlib/types"
-	"github.com/grafana/grafana-app-sdk/logging"
-	"github.com/grafana/grafana/pkg/apimachinery/identity"
-)
-
-// sessionAccessChecker implements AccessChecker using Grafana session identity.
-type sessionAccessChecker struct {
-	inner        authlib.AccessChecker
-	fallbackRole identity.RoleType
-}
-
-// NewSessionAccessChecker creates an AccessChecker that gets identity from Grafana
-// sessions via GetRequester(ctx). Supports optional role-based fallback via
-// WithFallbackRole for backwards compatibility.
-func NewSessionAccessChecker(inner authlib.AccessChecker) AccessChecker {
-	return &sessionAccessChecker{
-		inner:        inner,
-		fallbackRole: "",
-	}
-}
-
-// WithFallbackRole returns a new AccessChecker with the specified fallback role.
-func (c *sessionAccessChecker) WithFallbackRole(role identity.RoleType) AccessChecker {
-	return &sessionAccessChecker{
-		inner:        c.inner,
-		fallbackRole: role,
-	}
-}
-
-// Check performs an access check with optional role-based fallback.
-// Returns nil if access is allowed, or an appropriate API error if denied.
-func (c *sessionAccessChecker) Check(ctx context.Context, req authlib.CheckRequest, folder string) error {
-	logger := logging.FromContext(ctx).With("logger", "sessionAccessChecker")
-
-	// Get identity from Grafana session
-	requester, err := identity.GetRequester(ctx)
-	if err != nil {
-		logger.Debug("failed to get requester",
-			"resource", req.Resource,
-			"verb", req.Verb,
-			"error", err.Error(),
-		)
-		return apierrors.NewUnauthorized(fmt.Sprintf("failed to get requester: %v", err))
-	}
-
-	logger.Debug("checking access",
-		"identityType", requester.GetIdentityType(),
-		"orgRole", requester.GetOrgRole(),
-		"namespace", requester.GetNamespace(),
-		"resource", req.Resource,
-		"verb", req.Verb,
-		"group", req.Group,
-		"name", req.Name,
-		"folder", folder,
-		"fallbackRole", c.fallbackRole,
-	)
-
-	// Fill in namespace from identity if not provided
-	if req.Namespace == "" {
-		req.Namespace = requester.GetNamespace()
-	}
-
-	// Perform the access check
-	rsp, err := c.inner.Check(ctx, requester, req, folder)
-
-	// Build the GroupResource for error messages
-	gr := schema.GroupResource{Group: req.Group, Resource: req.Resource}
-
-	// No fallback configured, return result directly
-	if c.fallbackRole == "" {
-		if err != nil {
-			logger.Debug("access check error (no fallback)",
-				"resource", req.Resource,
-				"verb", req.Verb,
-				"error", err.Error(),
-			)
-			return apierrors.NewForbidden(gr, req.Name, fmt.Errorf("%s.%s is forbidden: %w", req.Resource, req.Group, err))
-		}
-		if !rsp.Allowed {
-			logger.Debug("access check denied (no fallback)",
-				"resource", req.Resource,
-				"verb", req.Verb,
-				"group", req.Group,
-				"allowed", rsp.Allowed,
-			)
-			return apierrors.NewForbidden(gr, req.Name, fmt.Errorf("permission denied"))
-		}
-		logger.Debug("access allowed",
-			"resource", req.Resource,
-			"verb", req.Verb,
-		)
-		return nil
-	}
-
-	// Fallback is configured - apply fallback logic
-	if err != nil {
-		if requester.GetOrgRole().Includes(c.fallbackRole) {
-			logger.Debug("access allowed via role fallback (after error)",
-				"resource", req.Resource,
-				"verb", req.Verb,
-				"fallbackRole", c.fallbackRole,
-				"orgRole", requester.GetOrgRole(),
-			)
-			return nil // Fallback succeeded
-		}
-		logger.Debug("access check error (fallback failed)",
-			"resource", req.Resource,
-			"verb", req.Verb,
-			"error", err.Error(),
-			"fallbackRole", c.fallbackRole,
-			"orgRole", requester.GetOrgRole(),
-		)
-		return apierrors.NewForbidden(gr, req.Name, fmt.Errorf("%s.%s is forbidden: %w", req.Resource, req.Group, err))
-	}
-
-	if rsp.Allowed {
-		logger.Debug("access allowed",
-			"resource", req.Resource,
-			"verb", req.Verb,
-		)
-		return nil
-	}
-
-	// Fall back to role for backwards compatibility
-	if requester.GetOrgRole().Includes(c.fallbackRole) {
-		logger.Debug("access allowed via role fallback",
-			"resource", req.Resource,
-			"verb", req.Verb,
-			"fallbackRole", c.fallbackRole,
-			"orgRole", requester.GetOrgRole(),
-		)
-		return nil // Fallback succeeded
-	}
-
-	logger.Debug("access denied (fallback role not met)",
-		"resource", req.Resource,
-		"verb", req.Verb,
-		"group", req.Group,
-		"fallbackRole", c.fallbackRole,
-		"orgRole", requester.GetOrgRole(),
-	)
-	return apierrors.NewForbidden(gr, req.Name, fmt.Errorf("%s role is required", strings.ToLower(string(c.fallbackRole))))
-}

apps/provisioning/pkg/auth/session_access_checker_test.go

@@ -1,244 +0,0 @@
-package auth
-
-import (
-	"context"
-	"errors"
-	"testing"
-
-	apierrors "k8s.io/apimachinery/pkg/api/errors"
-
-	authlib "github.com/grafana/authlib/types"
-	"github.com/grafana/grafana/pkg/apimachinery/identity"
-	"github.com/grafana/grafana/pkg/services/user"
-	"github.com/stretchr/testify/assert"
-	"github.com/stretchr/testify/require"
-)
-
-// mockRequester implements identity.Requester for testing.
-type mockRequester struct {
-	identity.Requester
-	orgRole      identity.RoleType
-	identityType authlib.IdentityType
-	namespace    string
-}
-
-func (m *mockRequester) GetOrgRole() identity.RoleType {
-	return m.orgRole
-}
-
-func (m *mockRequester) GetIdentityType() authlib.IdentityType {
-	return m.identityType
-}
-
-func (m *mockRequester) GetNamespace() string {
-	return m.namespace
-}
-
-func TestSessionAccessChecker_Check(t *testing.T) {
-	ctx := context.Background()
-	req := authlib.CheckRequest{
-		Verb:      "get",
-		Group:     "provisioning.grafana.app",
-		Resource:  "repositories",
-		Name:      "test-repo",
-		Namespace: "default",
-	}
-
-	tests := []struct {
-		name          string
-		fallbackRole  identity.RoleType
-		innerResponse authlib.CheckResponse
-		innerErr      error
-		requester     *mockRequester
-		expectAllow   bool
-	}{
-		{
-			name:          "allowed by checker",
-			fallbackRole:  identity.RoleAdmin,
-			innerResponse: authlib.CheckResponse{Allowed: true},
-			requester:     &mockRequester{orgRole: identity.RoleViewer, identityType: authlib.TypeUser},
-			expectAllow:   true,
-		},
-		{
-			name:          "denied by checker, fallback to admin role succeeds",
-			fallbackRole:  identity.RoleAdmin,
-			innerResponse: authlib.CheckResponse{Allowed: false},
-			requester:     &mockRequester{orgRole: identity.RoleAdmin, identityType: authlib.TypeUser},
-			expectAllow:   true,
-		},
-		{
-			name:          "denied by checker, fallback to admin role fails for viewer",
-			fallbackRole:  identity.RoleAdmin,
-			innerResponse: authlib.CheckResponse{Allowed: false},
-			requester:     &mockRequester{orgRole: identity.RoleViewer, identityType: authlib.TypeUser},
-			expectAllow:   false,
-		},
-		{
-			name:         "error from checker, fallback to admin role succeeds",
-			fallbackRole: identity.RoleAdmin,
-			innerErr:     errors.New("access check failed"),
-			requester:    &mockRequester{orgRole: identity.RoleAdmin, identityType: authlib.TypeUser},
-			expectAllow:  true,
-		},
-		{
-			name:         "error from checker, fallback fails for viewer",
-			fallbackRole: identity.RoleAdmin,
-			innerErr:     errors.New("access check failed"),
-			requester:    &mockRequester{orgRole: identity.RoleViewer, identityType: authlib.TypeUser},
-			expectAllow:  false,
-		},
-		{
-			name:          "denied, editor fallback succeeds for editor",
-			fallbackRole:  identity.RoleEditor,
-			innerResponse: authlib.CheckResponse{Allowed: false},
-			requester:     &mockRequester{orgRole: identity.RoleEditor, identityType: authlib.TypeUser},
-			expectAllow:   true,
-		},
-		{
-			name:          "denied, editor fallback fails for viewer",
-			fallbackRole:  identity.RoleEditor,
-			innerResponse: authlib.CheckResponse{Allowed: false},
-			requester:     &mockRequester{orgRole: identity.RoleViewer, identityType: authlib.TypeUser},
-			expectAllow:   false,
-		},
-		{
-			name:          "no fallback configured, denied stays denied",
-			fallbackRole:  "", // no fallback
-			innerResponse: authlib.CheckResponse{Allowed: false},
-			requester:     &mockRequester{orgRole: identity.RoleAdmin, identityType: authlib.TypeUser},
-			expectAllow:   false,
-		},
-	}
-
-	for _, tt := range tests {
-		t.Run(tt.name, func(t *testing.T) {
-			mock := &mockInnerAccessChecker{
-				response: tt.innerResponse,
-				err:      tt.innerErr,
-			}
-
-			checker := NewSessionAccessChecker(mock)
-			if tt.fallbackRole != "" {
-				checker = checker.WithFallbackRole(tt.fallbackRole)
-			}
-
-			// Add requester to context
-			testCtx := identity.WithRequester(ctx, tt.requester)
-
-			err := checker.Check(testCtx, req, "")
-
-			if tt.expectAllow {
-				require.NoError(t, err)
-			} else {
-				require.Error(t, err)
-				assert.True(t, apierrors.IsForbidden(err), "expected Forbidden error, got: %v", err)
-			}
-		})
-	}
-}
-
-func TestSessionAccessChecker_NoRequester(t *testing.T) {
-	mock := &mockInnerAccessChecker{
-		response: authlib.CheckResponse{Allowed: true},
-	}
-
-	checker := NewSessionAccessChecker(mock)
-	err := checker.Check(context.Background(), authlib.CheckRequest{}, "")
-
-	require.Error(t, err)
-	assert.True(t, apierrors.IsUnauthorized(err), "expected Unauthorized error")
-}
-
-func TestSessionAccessChecker_WithFallbackRole_ImmutableOriginal(t *testing.T) {
-	mock := &mockInnerAccessChecker{
-		response: authlib.CheckResponse{Allowed: false},
-	}
-
-	original := NewSessionAccessChecker(mock)
-	withAdmin := original.WithFallbackRole(identity.RoleAdmin)
-	withEditor := original.WithFallbackRole(identity.RoleEditor)
-
-	ctx := identity.WithRequester(context.Background(), &mockRequester{
-		orgRole:      identity.RoleEditor,
-		identityType: authlib.TypeUser,
-	})
-
-	req := authlib.CheckRequest{}
-
-	// Original should deny (no fallback)
-	err := original.Check(ctx, req, "")
-	require.Error(t, err, "original should deny without fallback")
-
-	// WithAdmin should deny for editor
-	err = withAdmin.Check(ctx, req, "")
-	require.Error(t, err, "admin fallback should deny for editor")
-
-	// WithEditor should allow for editor
-	err = withEditor.Check(ctx, req, "")
-	require.NoError(t, err, "editor fallback should allow for editor")
-}
-
-func TestSessionAccessChecker_WithFallbackRole_ChainedCalls(t *testing.T) {
-	mock := &mockInnerAccessChecker{
-		response: authlib.CheckResponse{Allowed: false},
-	}
-
-	// Ensure chained WithFallbackRole calls work correctly
-	checker := NewSessionAccessChecker(mock).
-		WithFallbackRole(identity.RoleAdmin).
-		WithFallbackRole(identity.RoleEditor) // This should override admin
-
-	ctx := identity.WithRequester(context.Background(), &mockRequester{
-		orgRole:      identity.RoleEditor,
-		identityType: authlib.TypeUser,
-	})
-
-	err := checker.Check(ctx, authlib.CheckRequest{}, "")
-	require.NoError(t, err, "last fallback (editor) should be used")
-}
-
-func TestSessionAccessChecker_RealSignedInUser(t *testing.T) {
-	mock := &mockInnerAccessChecker{
-		response: authlib.CheckResponse{Allowed: false},
-	}
-
-	checker := NewSessionAccessChecker(mock).WithFallbackRole(identity.RoleAdmin)
-
-	// Use a real SignedInUser
-	signedInUser := &user.SignedInUser{
-		UserID:  1,
-		OrgID:   1,
-		OrgRole: identity.RoleAdmin,
-	}
-
-	ctx := identity.WithRequester(context.Background(), signedInUser)
-
-	err := checker.Check(ctx, authlib.CheckRequest{}, "")
-	require.NoError(t, err, "admin user should be allowed via fallback")
-}
-
-func TestSessionAccessChecker_FillsNamespace(t *testing.T) {
-	mock := &mockInnerAccessChecker{
-		response: authlib.CheckResponse{Allowed: true},
-	}
-
-	checker := NewSessionAccessChecker(mock)
-
-	ctx := identity.WithRequester(context.Background(), &mockRequester{
-		orgRole:      identity.RoleAdmin,
-		identityType: authlib.TypeUser,
-		namespace:    "org-123",
-	})
-
-	// Request without namespace
-	req := authlib.CheckRequest{
-		Verb:     "get",
-		Group:    "provisioning.grafana.app",
-		Resource: "repositories",
-		Name:     "test-repo",
-		// Namespace intentionally empty
-	}
-
-	err := checker.Check(ctx, req, "")
-	require.NoError(t, err)
-}

apps/provisioning/pkg/auth/token_access_checker.go

@@ -1,92 +0,0 @@
-package auth
-
-import (
-	"context"
-	"fmt"
-
-	apierrors "k8s.io/apimachinery/pkg/api/errors"
-	"k8s.io/apimachinery/pkg/runtime/schema"
-
-	authlib "github.com/grafana/authlib/types"
-	"github.com/grafana/grafana-app-sdk/logging"
-	"github.com/grafana/grafana/pkg/apimachinery/identity"
-)
-
-// tokenAccessChecker implements AccessChecker using access tokens from context.
-type tokenAccessChecker struct {
-	inner authlib.AccessChecker
-}
-
-// NewTokenAccessChecker creates an AccessChecker that gets identity from access tokens
-// via AuthInfoFrom(ctx). Role-based fallback is not supported.
-func NewTokenAccessChecker(inner authlib.AccessChecker) AccessChecker {
-	return &tokenAccessChecker{inner: inner}
-}
-
-// WithFallbackRole returns the same checker since fallback is not supported.
-func (c *tokenAccessChecker) WithFallbackRole(_ identity.RoleType) AccessChecker {
-	return c
-}
-
-// Check performs an access check using AuthInfo from context.
-// Returns nil if access is allowed, or an appropriate API error if denied.
-func (c *tokenAccessChecker) Check(ctx context.Context, req authlib.CheckRequest, folder string) error {
-	logger := logging.FromContext(ctx).With("logger", "tokenAccessChecker")
-
-	// Get identity from access token in context
-	id, ok := authlib.AuthInfoFrom(ctx)
-	if !ok {
-		logger.Debug("no auth info in context",
-			"resource", req.Resource,
-			"verb", req.Verb,
-			"namespace", req.Namespace,
-		)
-		return apierrors.NewUnauthorized("no auth info in context")
-	}
-
-	logger.Debug("checking access",
-		"identityType", id.GetIdentityType(),
-		"namespace", id.GetNamespace(),
-		"resource", req.Resource,
-		"verb", req.Verb,
-		"group", req.Group,
-		"name", req.Name,
-		"folder", folder,
-	)
-
-	// Fill in namespace from identity if not provided
-	if req.Namespace == "" {
-		req.Namespace = id.GetNamespace()
-	}
-
-	// Perform the access check
-	rsp, err := c.inner.Check(ctx, id, req, folder)
-
-	// Build the GroupResource for error messages
-	gr := schema.GroupResource{Group: req.Group, Resource: req.Resource}
-
-	if err != nil {
-		logger.Debug("access check error",
-			"resource", req.Resource,
-			"verb", req.Verb,
-			"error", err.Error(),
-		)
-		return apierrors.NewForbidden(gr, req.Name, fmt.Errorf("%s.%s is forbidden: %w", req.Resource, req.Group, err))
-	}
-	if !rsp.Allowed {
-		logger.Debug("access check denied",
-			"resource", req.Resource,
-			"verb", req.Verb,
-			"group", req.Group,
-			"identityType", id.GetIdentityType(),
-			"allowed", rsp.Allowed,
-		)
-		return apierrors.NewForbidden(gr, req.Name, fmt.Errorf("permission denied"))
-	}
-
-	logger.Debug("access allowed",
-		"resource", req.Resource,
-		"verb", req.Verb,
-	)
-	return nil
-}

apps/provisioning/pkg/auth/token_access_checker_test.go

@@ -1,137 +0,0 @@
-package auth
-
-import (
-	"context"
-	"errors"
-	"testing"
-
-	apierrors "k8s.io/apimachinery/pkg/api/errors"
-
-	authlib "github.com/grafana/authlib/types"
-	"github.com/grafana/grafana/pkg/apimachinery/identity"
-	"github.com/stretchr/testify/assert"
-	"github.com/stretchr/testify/require"
-)
-
-func TestTokenAccessChecker_Check(t *testing.T) {
-	req := authlib.CheckRequest{
-		Verb:      "get",
-		Group:     "provisioning.grafana.app",
-		Resource:  "repositories",
-		Name:      "test-repo",
-		Namespace: "default",
-	}
-
-	tests := []struct {
-		name          string
-		innerResponse authlib.CheckResponse
-		innerErr      error
-		authInfo      *identity.StaticRequester
-		expectAllow   bool
-	}{
-		{
-			name:          "allowed by checker",
-			innerResponse: authlib.CheckResponse{Allowed: true},
-			authInfo:      &identity.StaticRequester{Type: authlib.TypeUser},
-			expectAllow:   true,
-		},
-		{
-			name:          "denied by checker",
-			innerResponse: authlib.CheckResponse{Allowed: false},
-			authInfo:      &identity.StaticRequester{Type: authlib.TypeUser},
-			expectAllow:   false,
-		},
-		{
-			name:        "error from checker",
-			innerErr:    errors.New("access check failed"),
-			authInfo:    &identity.StaticRequester{Type: authlib.TypeUser},
-			expectAllow: false,
-		},
-	}
-
-	for _, tt := range tests {
-		t.Run(tt.name, func(t *testing.T) {
-			mock := &mockInnerAccessChecker{
-				response: tt.innerResponse,
-				err:      tt.innerErr,
-			}
-
-			checker := NewTokenAccessChecker(mock)
-
-			// Add auth info to context
-			testCtx := authlib.WithAuthInfo(context.Background(), tt.authInfo)
-
-			err := checker.Check(testCtx, req, "")
-
-			if tt.expectAllow {
-				require.NoError(t, err)
-			} else {
-				require.Error(t, err)
-				assert.True(t, apierrors.IsForbidden(err), "expected Forbidden error, got: %v", err)
-			}
-		})
-	}
-}
-
-func TestTokenAccessChecker_NoAuthInfo(t *testing.T) {
-	mock := &mockInnerAccessChecker{
-		response: authlib.CheckResponse{Allowed: true},
-	}
-
-	checker := NewTokenAccessChecker(mock)
-	err := checker.Check(context.Background(), authlib.CheckRequest{}, "")
-
-	require.Error(t, err)
-	assert.True(t, apierrors.IsUnauthorized(err), "expected Unauthorized error")
-}
-
-func TestTokenAccessChecker_WithFallbackRole_IsNoOp(t *testing.T) {
-	mock := &mockInnerAccessChecker{
-		response: authlib.CheckResponse{Allowed: false},
-	}
-
-	checker := NewTokenAccessChecker(mock)
-	checkerWithFallback := checker.WithFallbackRole(identity.RoleAdmin)
-
-	// They should be the same instance
-	assert.Same(t, checker, checkerWithFallback, "WithFallbackRole should return same instance")
-}
-
-func TestTokenAccessChecker_FillsNamespace(t *testing.T) {
-	mock := &mockInnerAccessChecker{
-		response: authlib.CheckResponse{Allowed: true},
-	}
-
-	checker := NewTokenAccessChecker(mock)
-
-	ctx := authlib.WithAuthInfo(context.Background(), &identity.StaticRequester{
-		Type:      authlib.TypeUser,
-		Namespace: "org-123",
-	})
-
-	// Request without namespace
-	req := authlib.CheckRequest{
-		Verb:     "get",
-		Group:    "provisioning.grafana.app",
-		Resource: "repositories",
-		Name:     "test-repo",
-		// Namespace intentionally empty
-	}
-
-	err := checker.Check(ctx, req, "")
-	require.NoError(t, err)
-}
-
-// mockInnerAccessChecker implements authlib.AccessChecker for testing.
-type mockInnerAccessChecker struct {
-	response authlib.CheckResponse
-	err      error
-}
-
-func (m *mockInnerAccessChecker) Check(_ context.Context, _ authlib.AuthInfo, _ authlib.CheckRequest, _ string) (authlib.CheckResponse, error) {
-	return m.response, m.err
-}
-
-func (m *mockInnerAccessChecker) Compile(_ context.Context, _ authlib.AuthInfo, _ authlib.ListRequest) (authlib.ItemChecker, authlib.Zookie, error) {
-	return nil, nil, nil
-}

apps/provisioning/pkg/repository/tester.go

@@ -23,13 +23,7 @@ func NewSimpleRepositoryTester(validator RepositoryValidator) SimpleRepositoryTe
 
 // TestRepository validates the repository and then runs a health check
 func (t *SimpleRepositoryTester) TestRepository(ctx context.Context, repo Repository) (*provisioning.TestResults, error) {
-	// Determine if this is a CREATE or UPDATE operation
-	// If the repository has been observed by the controller (ObservedGeneration > 0),
-	// it's an existing repository and we should treat it as UPDATE
-	cfg := repo.Config()
-	isCreate := cfg.Status.ObservedGeneration == 0
-
-	errors := t.validator.ValidateRepository(repo, isCreate)
+	errors := t.validator.ValidateRepository(repo)
 	if len(errors) > 0 {
 		rsp := &provisioning.TestResults{
 			Code:    http.StatusUnprocessableEntity, // Invalid

apps/provisioning/pkg/repository/validator.go

@@ -32,9 +32,7 @@ func NewValidator(minSyncInterval time.Duration, allowedTargets []provisioning.S
 }
 
 // ValidateRepository solely does configuration checks on the repository object. It does not run a health check or compare against existing repositories.
-// isCreate indicates whether this is a CREATE operation (true) or UPDATE operation (false).
-// When isCreate is false, allowedTargets validation is skipped to allow existing repositories to continue working.
-func (v *RepositoryValidator) ValidateRepository(repo Repository, isCreate bool) field.ErrorList {
+func (v *RepositoryValidator) ValidateRepository(repo Repository) field.ErrorList {
 	list := repo.Validate()
 	cfg := repo.Config()
 
@@ -46,7 +44,7 @@ func (v *RepositoryValidator) ValidateRepository(repo Repository, isCreate bool)
 		if cfg.Spec.Sync.Target == "" {
 			list = append(list, field.Required(field.NewPath("spec", "sync", "target"),
 				"The target type is required when sync is enabled"))
-		} else if isCreate && !slices.Contains(v.allowedTargets, cfg.Spec.Sync.Target) {
+		} else if !slices.Contains(v.allowedTargets, cfg.Spec.Sync.Target) {
 			list = append(list,
 				field.Invalid(
 					field.NewPath("spec", "target"),

apps/provisioning/pkg/repository/validator_test.go

@@ -303,8 +303,7 @@ func TestValidateRepository(t *testing.T) {
 	validator := NewValidator(10*time.Second, []provisioning.SyncTargetType{provisioning.SyncTargetTypeFolder}, false)
 	for _, tt := range tests {
 		t.Run(tt.name, func(t *testing.T) {
-			// Tests validate new configurations, so always pass isCreate=true
-			errors := validator.ValidateRepository(tt.repository, true)
+			errors := validator.ValidateRepository(tt.repository)
 			require.Len(t, errors, tt.expectedErrs)
 			if tt.validateError != nil {
 				tt.validateError(t, errors)

conf/defaults.ini

@@ -1653,11 +1653,6 @@ loki_basic_auth_password =
 # Accepts duration formats like: 30s, 1m, 1h.
 rule_query_offset = 1m
 
-# Default data source UID to use for query execution when importing Prometheus rules.
-# This default is used when the X-Grafana-Alerting-Datasource-UID header is not provided.
-# If not set, the header becomes required.
-default_datasource_uid =
-
 [recording_rules]
 # Enable recording rules.
 enabled = true
@@ -2269,7 +2264,7 @@ fail_tests_on_console = true
 # List of targets that can be controlled by a repository, separated by |.
 # Instance means the whole grafana instance will be controlled by a repository.
 # Folder limits it to a folder within the grafana instance.
-allowed_targets = folder
+allowed_targets = instance|folder
 
 # Whether image rendering is allowed for dashboard previews.
 # Requires image rendering service to be configured.

conf/sample.ini

@@ -1615,11 +1615,6 @@ max_annotations_to_keep =
 # Accepts duration formats like: 30s, 1m, 1h.
 rule_query_offset = 1m
 
-# Default data source UID to use for query execution when importing Prometheus rules.
-# This default is used when the X-Grafana-Alerting-Datasource-UID header is not provided.
-# If not set, the header becomes required.
-default_datasource_uid =
-
 #################################### Recording Rules #####################
 [recording_rules]
 # Enable recording rules.

devenv/dev-dashboards/panel-gauge/gauge_tests.json

@@ -600,20 +600,6 @@
           "stringInput": "null,null"
         }
       ],
-      "transformations": [
-        {
-          "id": "convertFieldType",
-          "options": {
-            "fields": {},
-            "conversions": [
-              {
-                "targetField": "A-series",
-                "destinationType": "number"
-              }
-            ]
-          }
-        }
-      ],
       "title": "Only nulls and no user set min & max",
       "type": "gauge"
     },

devenv/dev-dashboards/panel-gauge/gauge_tests_old_to_new.json

@@ -474,7 +474,6 @@
       },
       "id": 12,
       "options": {
-        "displayName": "My gauge",
         "minVizHeight": 75,
         "minVizWidth": 75,
         "orientation": "auto",

docs/sources/alerting/alerting-rules/alerting-migration.md

@@ -134,7 +134,7 @@ To convert data source-managed alert rules to Grafana managed alerts:
 
    Pausing stops alert rule evaluation behavior for the newly created Grafana-managed alert rules.
 
-9. (Optional) In the **Target data source** of the **Recording rules** section, you can select the data source to which the imported recording rules will write metrics. By default, it is the data source selected in the **Data source** dropdown.
+9. (Optional) In the **Target data source** of the **Recording rules** section, you can select the data source that the imported recording rules will query. By default, it is the data source selected in the **Data source** dropdown.
 
 10. Click **Import**.
 
@@ -242,8 +242,6 @@ Set to `true` to import recording rules in paused state.
 
 The UID of the data source to use for alert rule queries.
 
-If not specified in the header, Grafana uses the configured default from `unified_alerting.prometheus_conversion.default_datasource_uid`. If neither the header nor the configuration option is provided, the request fails.
-
 #### `X-Grafana-Alerting-Target-Datasource-UID`
 
 The UID of the target data source for recording rules. If not specified, the value from `X-Grafana-Alerting-Datasource-UID` is used.

docs/sources/as-code/observability-as-code/provision-resources/file-path-setup.md

@@ -128,48 +128,35 @@ The set up process verifies the path and provides an error message if a problem
 
 #### Synchronization limitations
 
-{{< admonition type="caution" >}}
+Full instance sync is not available in Grafana Cloud.
 
-Full instance sync is not available in Grafana Cloud and is experimental and unsupported in Grafana OSS/Enterprise.
-
-{{< /admonition >}}
-
-To have access to full instance sync you must explicitly enable the option.
-
-The following applies:
+In Grafana OSS/Enterprise:
 
+- If you try to perform a full instance sync with resources that contain alerts or panels, the connection will be blocked.
 - You won't be able to create new alerts or library panels after setup is completed.
 - If you opted for full instance sync and want to use alerts and library panels, you'll have to delete the provisioned repository and connect again with folder sync.
 
 #### Set up synchronization
 
-You can sync external resources into a new folder without affecting the rest of your instance.
+Choose to either sync your entire organization resources with external storage, or to sync certain resources to a new Grafana folder (with up to 10 connections).
 
-To set up synchronization:
+- Choose **Sync all resources with external storage** if you want to sync and manage your entire Grafana instance through external storage. With this option, all of your dashboards are synced to that one repository. You can only have one provisioned connection with this selection, and you won't have the option of setting up additional repositories to connect to.
 
-1. Select which resources you want to sync.
+- Choose **Sync external storage to new Grafana folder** to sync external resources into a new folder without affecting the rest of your instance. You can repeat this process for up to 10 connections.
 
-1. Enter a **Display name** for the repository connection. Resources stored in this connection appear under the chosen display name in the Grafana UI.
+Next, enter a **Display name** for the repository connection. Resources stored in this connection appear under the chosen display name in the Grafana UI.
 
-1. Click **Synchronize** to continue.
-
-1. You can repeat this process for up to 10 connections.
-
-{{< admonition type="note" >}}
-
-Optionally, you can export any unmanaged resources into the provisioned folder. See how in [Synchronize with external storage](#synchronize-with-external-storage).
-
-{{< /admonition >}}
+Click **Synchronize** to continue.
 
 ### Synchronize with external storage
 
-In this step you proceed to synchronize the resources selected in the previous step. Optionally, you can check the **Migrate existing resources** box to migrate your unmanaged dashboards to the provisioned folder.
+After this one time step, all future updates are automatically saved to the local file path and provisioned back to the instance.
 
-Select **Begin synchronization** to start the process. After this one time step, all future updates are automatically saved to the local file path and provisioned back to the instance.
-
-Note that during the initial synchronization, your dashboards will be temporarily unavailable. No data or configurations will be lost.
+During the initial synchronization, your dashboards will be temporarily unavailable. No data or configurations will be lost.
 How long the process takes depends upon the number of resources involved.
 
+Select **Begin synchronization** to start the process.
+
 ### Choose additional settings
 
 If you wish, you can make any files synchronized as as **Read only** so no changes can be made to the resources through Grafana.

docs/sources/as-code/observability-as-code/provision-resources/git-sync-setup.md

@@ -132,35 +132,17 @@ To connect your GitHub repository:
 
 ### Choose what to synchronize
 
-You can sync external resources into a new folder without affecting the rest of your instance.
+In this step, you can decide which elements to synchronize. The available options depend on the status of your Grafana instance:
+
+- If the instance contains resources in an incompatible data format, you'll have to migrate all the data using instance sync. Folder sync won't be supported.
+- If there's already another connection using folder sync, instance sync won't be offered.
 
 To set up synchronization:
 
-1. Select which resources you want to sync.
+- Choose **Sync all resources with external storage** if you want to sync and manage your entire Grafana instance through external storage. With this option, all of your dashboards are synced to that one repository. You can only have one provisioned connection with this selection, and you won't have the option of setting up additional repositories to connect to.
+- Choose **Sync external storage to new Grafana folder** to sync external resources into a new folder without affecting the rest of your instance. You can repeat this process for up to 10 connections.
 
-1. Enter a **Display name** for the repository connection. Resources stored in this connection appear under the chosen display name in the Grafana UI.
-
-1. Click **Synchronize** to continue.
-
-1. You can repeat this process for up to 10 connections.
-
-{{< admonition type="note" >}}
-
-Optionally, you can export any unmanaged resources into the provisioned folder. See how in [Synchronize with external storage](#synchronize-with-external-storage).
-
-{{< /admonition >}}
-
-#### Full instance sync
-
-Full instance sync is not available in Grafana Cloud and is experimental and unsupported in Grafana OSS/Enterprise.
-
-To have access to this option you must enable experimental instance sync on purpose.
-
-### Synchronize with external storage
-
-After this one time step, all future updates are automatically saved to the Git repository and provisioned back to the instance.
-
-Check the **Migrate existing resources** box to migrate your unmanaged dashboards to the provisioned folder.
+Next, enter a **Display name** for the repository connection. Resources stored in this connection appear under the chosen display name in the Grafana UI. Click **Synchronize** to continue.
 
 ### Choose additional settings
 

docs/sources/as-code/observability-as-code/provision-resources/intro-git-sync.md

@@ -47,7 +47,7 @@ Using Git Sync, you can:
 
 {{< admonition type="caution" >}}
 
-Full instance sync is not available in Grafana Cloud and is experimental and unsupported in Grafana OSS/Enterprise.
+Git Sync only works with specific folders for the moment. Full-instance sync is not currently supported.
 
 {{< /admonition >}}
 
@@ -84,7 +84,7 @@ Refer to [Requirements](https://grafana.com/docs/grafana/<GRAFANA_VERSION>/obser
 
 - You can only sync dashboards and folders. Refer to [Supported resources](#supported-resources) for more information.
 - If you're using Git Sync in Grafana OSS and Grafana Enterprise, some resources might be in an incompatible data format and won't be synced.
-- Full-instance sync is not available in Grafana Cloud and is experimental in Grafana OSS and Grafana Enterprise. Refer to [Choose what to synchronize](../git-sync-setup/#choose-what-to-synchronize) for more details.
+- Full-instance sync is not available in Grafana Cloud and has limitations in Grafana OSS and Grafana Enterprise. Refer to [Choose what to synchronize](../git-sync-setup/#choose-what-to-synchronize) for more details.
 - When migrating to full instance sync, during the synchronization process your resources will be temporarily unavailable. No one will be able to create, edit, or delete resources during this process.
 - If you want to manage existing resources with Git Sync, you need to save them as JSON files and commit them to the synced repository. Open a PR to import, copy, move, or save a dashboard.
 - Restoring resources from the UI is currently not possible. As an alternative, you can restore dashboards directly in your GitHub repository by raising a PR, and they will be updated in Grafana.

docs/sources/datasources/_index.md

@@ -112,12 +112,6 @@ For example, this video demonstrates the visual Prometheus query builder:
 
 For general information about querying in Grafana, and common options and user interface elements across all query editors, refer to [Query and transform data](ref:query-transform-data).
 
-## Build a dashboard from the data source
-
-After you've configured a data source, you can start creating a dashboard directly from it, by clicking the **Build a dashboard** button.
-
-For more information, refer to [Begin dashboard creation from data source configuration](https://grafana.com/docs/grafana/<GRAFANA_VERSION>/visualizations/dashboards/build-dashboards/create-dashboard/#begin-dashboard-creation-from-connections).
-
 ## Special data sources
 
 Grafana includes three special data sources:

docs/sources/setup-grafana/configure-grafana/_index.md

@@ -2052,10 +2052,6 @@ This section applies only to rules imported as Grafana-managed rules. For more i
 
 Set the query offset to imported Grafana-managed rules when `query_offset` is not defined in the original rule group configuration. The default value is `1m`.
 
-#### `default_datasource_uid`
-
-Set the default data source UID to use for query execution when importing Prometheus rules. Grafana uses this default when the `X-Grafana-Alerting-Datasource-UID` header isn't provided during import. If this option isn't set, the header becomes required. The default value is empty.
-
 <hr>
 
 ### `[annotations]`

docs/sources/setup-grafana/configure-grafana/feature-toggles/index.md

@@ -31,6 +31,7 @@ Most [generally available](https://grafana.com/docs/release-life-cycle/#general-
 | `logsContextDatasourceUi`             | Allow datasource to provide custom UI for context view                                                                                                        | Yes                |
 | `lokiQuerySplitting`                  | Split large interval queries into subqueries with smaller time intervals                                                                                      | Yes                |
 | `influxdbBackendMigration`            | Query InfluxDB InfluxQL without the proxy                                                                                                                     | Yes                |
+| `unifiedRequestLog`                   | Writes error logs to the request logger                                                                                                                       | Yes                |
 | `logsExploreTableVisualisation`       | A table visualisation for logs in Explore                                                                                                                     | Yes                |
 | `awsDatasourcesTempCredentials`       | Support temporary security credentials in AWS plugins for Grafana Cloud customers                                                                             | Yes                |
 | `awsAsyncQueryCaching`                | Enable caching for async queries for Redshift and Athena. Requires that the datasource has caching and async query support enabled                            | Yes                |

docs/sources/visualizations/dashboards/build-dashboards/annotate-visualizations/index.md

@@ -163,9 +163,9 @@ To add a new annotation query to a dashboard, follow these steps:
 
 1. To create a query, do one of the following:
    - Write or construct a query in the query language of your data source. The annotation query options are different for each data source. For information about annotations in a specific data source, refer to the specific [data source](ref:data-source) topic.
-   - Open the **Saved queries** drop-down menu and click **Replace query** to reuse a [saved query](ref:saved-queries).
+   - Click **Replace with saved query** to reuse a [saved query](ref:saved-queries).
 
-1. (Optional) To [save the query](ref:save-query) for reuse, open the **Saved queries** drop-down menu and click the **Save query** option.
+1. (Optional) To [save the query](ref:save-query) for reuse, click the **Save query** button (or icon).
 1. (Optional) Click **Test annotation query** to ensure that the query is working properly.
 1. (Optional) To add subsequent queries, click **+ Add query** or **+ Add from saved queries**, and test them as many times as needed.
 

docs/sources/visualizations/dashboards/build-dashboards/create-dashboard/index.md

@@ -99,7 +99,7 @@ Dashboards and panels allow you to show your data in visual form. Each panel nee
 - Understand the query language of the target data source.
 - Ensure that data source for which you are writing a query has been added. For more information about adding a data source, refer to [Add a data source](ref:add-a-data-source) if you need instructions.
 
-To create a dashboard, follow these steps:
+**To create a dashboard**:
 
 {{< shared id="create-dashboard" >}}
 
@@ -125,9 +125,9 @@ To create a dashboard, follow these steps:
 
 1. To create a query, do one of the following:
    - Write or construct a query in the query language of your data source.
-   - Open the **Saved queries** drop-down menu and click **Replace query** to reuse a [saved query](ref:saved-queries).
+   - Click **Replace with saved query** to reuse a [saved query](ref:saved-queries).
 
-1. (Optional) To [save the query](ref:save-query) for reuse, open the **Saved queries** drop-down menu and click the **Save query** option.
+1. (Optional) To [save the query](ref:save-query) for reuse, click the **Save query** button (or icon).
 1. Click **Refresh** to query the data source.
 1. (Optional) To add subsequent queries, click **+ Add query** or **+ Add from saved queries**, and refresh the data source as many times as needed.
 
@@ -171,28 +171,6 @@ To create a dashboard, follow these steps:
 
    Now, when you want to make more changes to the saved dashboard, click **Edit** in the top-right corner.
 
-### Begin dashboard creation from data source configuration
-
-You can start the process of creating a dashboard directly from a data source rather than from the **Dashboards** page.
-
-To begin building a dashboard directly from a data source, follow these steps:
-
-1. Navigate to **Connections > Data sources**.
-1. On the row of the data source for which you want to build a dashboard, click **Build a dashboard**.
-
-   The empty dashboard page opens.
-
-1. Do one of the following:
-   - Click **+Add visualization** to configure all the elements of the new dashboard.
-   - Select one of the suggested dashboards by clicking its **Use dashboard** button. This can be helpful when you're not sure how to most effectively visualize your data.
-     The suggested dashboards are specific to your data source type (for example, Prometheus, Loki, or Elasticsearch). If there are more than three dashboard suggestions, you can click **View all** to see the rest of them.
-
-   ![Empty dashboard with add visualization and suggested dashboard options](/media/docs/grafana/dashboards/screenshot-suggested-dashboards-v12.3.png)
-
-   {{< docs/public-preview product="Suggested dashboards" >}}
-
-1. Complete the rest of the dashboard configuration. For more detailed steps, refer to [Create a dashboard](#create-a-dashboard), beginning at step five.
-
 ## Copy a dashboard
 
 To copy a dashboard, follow these steps:

docs/sources/visualizations/dashboards/share-dashboards-panels/_index.md

@@ -98,7 +98,7 @@ You can share dashboards in the following ways:
 - [As a report](#schedule-a-report)
 - [As a snapshot](#share-a-snapshot)
 - [As a PDF export](#export-a-dashboard-as-pdf)
-- [As a JSON file export](#export-a-dashboard-as-code)
+- [As a JSON file export](#export-a-dashboard-as-json)
 - [As an image export](#export-a-dashboard-as-an-image)
 
 When you share a dashboard externally as a link or by email, those dashboards are included in a list of your shared dashboards. To view the list and manage these dashboards, navigate to **Dashboards > Shared dashboards**.

docs/sources/visualizations/explore/get-started-with-explore.md

@@ -71,9 +71,8 @@ Explore consists of a toolbar, outline, query editor, the ability to add multipl
   - **Run query** - Click to run your query.
 
 - **Query editor** - Interface where you construct the query for a specific data source. Query editor elements differ based on data source. In order to run queries across multiple data sources you need to select **Mixed** from the data source picker.
-  - **Saved queries**:
-    - **Save query** - To [save the query](ref:save-query) for reuse, click the **Save query** button (or icon).
-    - **Replace query** - Reuse a saved query.
+  - **Save query** - To [save the query](ref:save-query) for reuse, click the **Save query** button (or icon).
+  - **Replace with saved query** - Reuse a saved query.
   - **+ Add query** - Add an additional query.
   - **+ Add from saved queries** - Add an additional query by reusing a saved query.
 

docs/sources/visualizations/panels-visualizations/panel-editor-overview/index.md

@@ -88,9 +88,8 @@ The data section contains tabs where you enter queries, transform your data, and
 
 - **Queries**
   - Select your data source. You can also set or update the data source in existing dashboards using the drop-down menu in the **Queries** tab.
-  - **Saved queries**:
-    - **Save query** - To [save the query](ref:save-query) for reuse, click the **Save query** button (or icon).
-    - **Replace query** - Reuse a saved query.
+  - **Save query** - To [save the query](ref:save-query) for reuse, click the **Save query** button (or icon).
+  - **Replace with saved query** - Reuse a saved query.
   - **+ Add query** - Add an additional query.
   - **+ Add from saved queries** - Add an additional query by reusing a saved query.
 

docs/sources/visualizations/panels-visualizations/query-transform-data/_index.md

@@ -156,11 +156,11 @@ In the **Saved queries** drawer, you can:
 - Edit a query title, description, tags, or the availability of the query to other users in your organization. By default, saved queries are locked for editing.
 - When you access the **Saved queries** drawer from Explore, you can use the **Edit in Explore** option to edit the body of a query.
 
-To access your saved queries, click **+ Add from saved queries** or open the **Saved queries** drop-down menu and click **Replace query** in the query editor:
+To access your saved queries, click **+ Add from saved queries** or **Replace with saved query** in the query editor:
 
 {{< figure src="/media/docs/grafana/dashboards/screenshot-use-saved-queries-v12.3.png" max-width="750px" alt="Access saved queries" >}}
 
-Clicking **+ Add from saved queries** adds an additional query, while clicking **Replace query** in the **Saved queries** drop-down menu updates your existing query.
+Clicking **+ Add from saved queries** adds an additional query, while clicking **Replace with saved query** updates your existing query.
 
 {{< admonition type="note" >}}
 Users with Admin and Editor roles can create and save queries for reuse.
@@ -172,7 +172,7 @@ Viewers can only reuse queries.
 
 To save a query you've created:
 
-1. From the query editor, open the **Saved queries** drop-down menu and click the **Save query** option:
+1. From the query editor, click the **Save query** icon:
 
    {{< figure src="/media/docs/grafana/panels-visualizations/screenshot-save-query-v12.2.png" max-width="750px" alt="Save a query" >}}
 
@@ -227,7 +227,7 @@ To add a query, follow these steps:
 
 1. To create a query, do one of the following:
    - Write or construct a query in the query language of your data source.
-   - Open the **Saved queries** drop-down menu and click **Replace query** to reuse a saved query.
+   - Click **Replace with saved query** to reuse a saved query.
 
    {{< admonition type="note" >}}
    [Saved queries](#saved-queries) is currently in [public preview](https://grafana.com/docs/release-life-cycle/). Grafana Labs offers limited support, and breaking changes might occur prior to the feature being made generally available.
@@ -235,7 +235,7 @@ To add a query, follow these steps:
    This feature is only available on Grafana Enterprise and Grafana Cloud.
    {{< /admonition >}}
 
-1. (Optional) To [save the query](#save-a-query) for reuse, click the **Save query** option in the **Saved queries** drop-down menu.
+1. (Optional) To [save the query](#save-a-query) for reuse, click the **Save query** button (or icon).
 1. (Optional) Click **+ Add query** or **Add from saved queries** to add more queries as needed.
 
 1. Click **Run queries**.

e2e-playwright/alerting-suite/saved-searches.spec.ts

@@ -1,271 +0,0 @@
-import { Page } from '@playwright/test';
-
-import { test, expect } from '@grafana/plugin-e2e';
-
-/**
- * UI selectors for Saved Searches e2e tests.
- * Each selector is a function that takes the page and returns a locator.
- */
-const ui = {
-  // Main elements
-  savedSearchesButton: (page: Page) => page.getByRole('button', { name: /saved searches/i }),
-  dropdown: (page: Page) => page.getByRole('dialog', { name: /saved searches/i }),
-  searchInput: (page: Page) => page.getByTestId('search-query-input'),
-
-  // Save functionality
-  saveButton: (page: Page) => page.getByRole('button', { name: /save current search/i }),
-  saveConfirmButton: (page: Page) => page.getByRole('button', { name: /^save$/i }),
-  saveNameInput: (page: Page) => page.getByPlaceholder(/enter a name/i),
-
-  // Action menu
-  actionsButton: (page: Page) => page.getByRole('button', { name: /actions/i }),
-  renameMenuItem: (page: Page) => page.getByText(/rename/i),
-  deleteMenuItem: (page: Page) => page.getByText(/^delete$/i),
-  setAsDefaultMenuItem: (page: Page) => page.getByText(/set as default/i),
-  deleteConfirmButton: (page: Page) => page.getByRole('button', { name: /^delete$/i }),
-
-  // Indicators
-  emptyState: (page: Page) => page.getByText(/no saved searches/i),
-  defaultIcon: (page: Page) => page.locator('[title="Default search"]'),
-  duplicateError: (page: Page) => page.getByText(/already exists/i),
-};
-
-/**
- * Helper to clear saved searches storage.
- * UserStorage uses localStorage as fallback, so we clear both potential keys.
- */
-async function clearSavedSearches(page: Page) {
-  await page.evaluate(() => {
-    // Clear localStorage keys that might contain saved searches
-    // UserStorage stores under 'grafana.userstorage.alerting' pattern
-    const keysToRemove = Object.keys(localStorage).filter(
-      (key) => key.includes('alerting') && (key.includes('savedSearches') || key.includes('userstorage'))
-    );
-    keysToRemove.forEach((key) => localStorage.removeItem(key));
-
-    // Also clear session storage visited flag
-    const sessionKeysToRemove = Object.keys(sessionStorage).filter((key) => key.includes('alerting'));
-    sessionKeysToRemove.forEach((key) => sessionStorage.removeItem(key));
-  });
-}
-
-test.describe(
-  'Alert Rules - Saved Searches',
-  {
-    tag: ['@alerting'],
-  },
-  () => {
-    test.beforeEach(async ({ page }) => {
-      // Clear any saved searches from previous tests before navigating
-      await page.goto('/alerting/list');
-      await clearSavedSearches(page);
-      await page.reload();
-    });
-
-    test.afterEach(async ({ page }) => {
-      // Clean up saved searches after each test
-      await clearSavedSearches(page);
-    });
-
-    test('should display Saved searches button', async ({ page }) => {
-      await expect(ui.savedSearchesButton(page)).toBeVisible();
-    });
-
-    test('should open dropdown when clicking Saved searches button', async ({ page }) => {
-      await ui.savedSearchesButton(page).click();
-
-      await expect(ui.dropdown(page)).toBeVisible();
-    });
-
-    test('should show empty state when no saved searches exist', async ({ page }) => {
-      // Storage is cleared in beforeEach, so we should see empty state
-      await ui.savedSearchesButton(page).click();
-
-      await expect(ui.emptyState(page)).toBeVisible();
-    });
-
-    test('should enable Save current search button when search query is entered', async ({ page }) => {
-      // Enter a search query
-      await ui.searchInput(page).fill('state:firing');
-      await ui.searchInput(page).press('Enter');
-
-      // Open saved searches
-      await ui.savedSearchesButton(page).click();
-
-      await expect(ui.saveButton(page)).toBeEnabled();
-    });
-
-    test('should disable Save current search button when search query is empty', async ({ page }) => {
-      await ui.savedSearchesButton(page).click();
-
-      await expect(ui.saveButton(page)).toBeDisabled();
-    });
-
-    test('should save a new search', async ({ page }) => {
-      // Enter a search query
-      await ui.searchInput(page).fill('state:firing');
-      await ui.searchInput(page).press('Enter');
-
-      // Open saved searches
-      await ui.savedSearchesButton(page).click();
-
-      // Click save button
-      await ui.saveButton(page).click();
-
-      // Enter name and save
-      await ui.saveNameInput(page).fill('My Firing Rules');
-      await ui.saveConfirmButton(page).click();
-
-      // Verify the saved search appears in the list
-      await expect(page.getByText('My Firing Rules')).toBeVisible();
-    });
-
-    test('should show validation error for duplicate name', async ({ page }) => {
-      // First save a search
-      await ui.searchInput(page).fill('state:firing');
-      await ui.searchInput(page).press('Enter');
-
-      await ui.savedSearchesButton(page).click();
-
-      await ui.saveButton(page).click();
-
-      await ui.saveNameInput(page).fill('Duplicate Test');
-      await ui.saveConfirmButton(page).click();
-
-      // Try to save another with the same name
-      await ui.saveButton(page).click();
-      await ui.saveNameInput(page).fill('Duplicate Test');
-      await ui.saveConfirmButton(page).click();
-
-      // Verify validation error
-      await expect(ui.duplicateError(page)).toBeVisible();
-    });
-
-    test('should apply a saved search', async ({ page }) => {
-      // Create a saved search first
-      await ui.searchInput(page).fill('state:firing');
-      await ui.searchInput(page).press('Enter');
-
-      await ui.savedSearchesButton(page).click();
-
-      await ui.saveButton(page).click();
-
-      await ui.saveNameInput(page).fill('Apply Test');
-      await ui.saveConfirmButton(page).click();
-
-      // Clear the search
-      await ui.searchInput(page).clear();
-      await ui.searchInput(page).press('Enter');
-
-      // Apply the saved search
-      await ui.savedSearchesButton(page).click();
-      await page.getByRole('button', { name: /apply search.*apply test/i }).click();
-
-      // Verify the search input is updated
-      await expect(ui.searchInput(page)).toHaveValue('state:firing');
-    });
-
-    test('should rename a saved search', async ({ page }) => {
-      // Create a saved search
-      await ui.searchInput(page).fill('state:firing');
-      await ui.searchInput(page).press('Enter');
-
-      await ui.savedSearchesButton(page).click();
-
-      await ui.saveButton(page).click();
-
-      await ui.saveNameInput(page).fill('Original Name');
-      await ui.saveConfirmButton(page).click();
-
-      // Open action menu and click rename
-      await ui.actionsButton(page).click();
-      await ui.renameMenuItem(page).click();
-
-      // Enter new name
-      const renameInput = page.getByDisplayValue('Original Name');
-      await renameInput.clear();
-      await renameInput.fill('Renamed Search');
-      await page.keyboard.press('Enter');
-
-      // Verify the name was updated
-      await expect(page.getByText('Renamed Search')).toBeVisible();
-      await expect(page.getByText('Original Name')).not.toBeVisible();
-    });
-
-    test('should delete a saved search', async ({ page }) => {
-      // Create a saved search
-      await ui.searchInput(page).fill('state:firing');
-      await ui.searchInput(page).press('Enter');
-
-      await ui.savedSearchesButton(page).click();
-
-      await ui.saveButton(page).click();
-
-      await ui.saveNameInput(page).fill('To Delete');
-      await ui.saveConfirmButton(page).click();
-
-      // Verify it was saved
-      await expect(page.getByText('To Delete')).toBeVisible();
-
-      // Open action menu and click delete
-      await ui.actionsButton(page).click();
-      await ui.deleteMenuItem(page).click();
-
-      // Confirm delete
-      await ui.deleteConfirmButton(page).click();
-
-      // Verify it was deleted
-      await expect(page.getByText('To Delete')).not.toBeVisible();
-    });
-
-    test('should set a search as default', async ({ page }) => {
-      // Create a saved search
-      await ui.searchInput(page).fill('state:firing');
-      await ui.searchInput(page).press('Enter');
-
-      await ui.savedSearchesButton(page).click();
-
-      await ui.saveButton(page).click();
-
-      await ui.saveNameInput(page).fill('Default Test');
-      await ui.saveConfirmButton(page).click();
-
-      // Set as default
-      await ui.actionsButton(page).click();
-      await ui.setAsDefaultMenuItem(page).click();
-
-      // Verify the star icon appears (indicating default)
-      await expect(ui.defaultIcon(page)).toBeVisible();
-    });
-
-    test('should close dropdown when pressing Escape', async ({ page }) => {
-      await ui.savedSearchesButton(page).click();
-
-      await expect(ui.dropdown(page)).toBeVisible();
-
-      await page.keyboard.press('Escape');
-
-      await expect(ui.dropdown(page)).not.toBeVisible();
-    });
-
-    test('should cancel save mode when pressing Escape', async ({ page }) => {
-      // Enter a search query
-      await ui.searchInput(page).fill('state:firing');
-      await ui.searchInput(page).press('Enter');
-
-      await ui.savedSearchesButton(page).click();
-
-      // Start save mode
-      await ui.saveButton(page).click();
-
-      await expect(ui.saveNameInput(page)).toBeVisible();
-
-      // Press Escape to cancel
-      await page.keyboard.press('Escape');
-
-      // Verify we're back to list mode
-      await expect(ui.saveNameInput(page)).not.toBeVisible();
-      await expect(ui.saveButton(page)).toBeVisible();
-    });
-  }
-);

e2e-playwright/dashboards-suite/dashboard-browse-nested.spec.ts

@@ -10,7 +10,7 @@ const NUM_NESTED_DASHBOARDS = 60;
 test.use({
   featureToggles: {
     kubernetesDashboards: process.env.FORCE_V2_DASHBOARDS_API === 'true',
-    dashboardNewLayouts: process.env.FORCE_V2_DASHBOARDS_API === 'true',
+    kubernetesDashboardsV2: process.env.FORCE_V2_DASHBOARDS_API === 'true',
   },
 });
 

e2e-playwright/dashboards-suite/dashboard-browse.spec.ts

@@ -5,7 +5,7 @@ import testDashboard from '../dashboards/TestDashboard.json';
 test.use({
   featureToggles: {
     kubernetesDashboards: process.env.FORCE_V2_DASHBOARDS_API === 'true',
-    dashboardNewLayouts: process.env.FORCE_V2_DASHBOARDS_API === 'true',
+    kubernetesDashboardsV2: process.env.FORCE_V2_DASHBOARDS_API === 'true',
   },
 });
 

e2e-playwright/dashboards-suite/dashboard-export-image.spec.ts

@@ -7,7 +7,7 @@ test.use({
     scenes: true,
     sharingDashboardImage: true, // Enable the export image feature
     kubernetesDashboards: process.env.FORCE_V2_DASHBOARDS_API === 'true',
-    dashboardNewLayouts: process.env.FORCE_V2_DASHBOARDS_API === 'true',
+    kubernetesDashboardsV2: process.env.FORCE_V2_DASHBOARDS_API === 'true',
   },
 });
 

e2e-playwright/dashboards-suite/dashboard-export-json.spec.ts

@@ -3,7 +3,7 @@ import { test, expect } from '@grafana/plugin-e2e';
 test.use({
   featureToggles: {
     kubernetesDashboards: process.env.FORCE_V2_DASHBOARDS_API === 'true',
-    dashboardNewLayouts: process.env.FORCE_V2_DASHBOARDS_API === 'true',
+    kubernetesDashboardsV2: process.env.FORCE_V2_DASHBOARDS_API === 'true',
   },
 });
 

e2e-playwright/dashboards-suite/dashboard-keybindings.spec.ts

@@ -3,7 +3,7 @@ import { test, expect } from '@grafana/plugin-e2e';
 test.use({
   featureToggles: {
     kubernetesDashboards: process.env.FORCE_V2_DASHBOARDS_API === 'true',
-    dashboardNewLayouts: process.env.FORCE_V2_DASHBOARDS_API === 'true',
+    kubernetesDashboardsV2: process.env.FORCE_V2_DASHBOARDS_API === 'true',
   },
 });
 

e2e-playwright/dashboards-suite/dashboard-links-without-slug.spec.ts

@@ -5,7 +5,7 @@ import testDashboard from '../dashboards/DataLinkWithoutSlugTest.json';
 test.use({
   featureToggles: {
     kubernetesDashboards: process.env.FORCE_V2_DASHBOARDS_API === 'true',
-    dashboardNewLayouts: process.env.FORCE_V2_DASHBOARDS_API === 'true',
+    kubernetesDashboardsV2: process.env.FORCE_V2_DASHBOARDS_API === 'true',
   },
 });
 

e2e-playwright/dashboards-suite/dashboard-live-streaming.spec.ts

@@ -5,7 +5,7 @@ import testDashboard from '../dashboards/DashboardLiveTest.json';
 test.use({
   featureToggles: {
     kubernetesDashboards: process.env.FORCE_V2_DASHBOARDS_API === 'true',
-    dashboardNewLayouts: process.env.FORCE_V2_DASHBOARDS_API === 'true',
+    kubernetesDashboardsV2: process.env.FORCE_V2_DASHBOARDS_API === 'true',
   },
 });
 

e2e-playwright/dashboards-suite/dashboard-public-create.spec.ts

@@ -3,7 +3,7 @@ import { test, expect } from '@grafana/plugin-e2e';
 test.use({
   featureToggles: {
     kubernetesDashboards: process.env.FORCE_V2_DASHBOARDS_API === 'true',
-    dashboardNewLayouts: process.env.FORCE_V2_DASHBOARDS_API === 'true',
+    kubernetesDashboardsV2: process.env.FORCE_V2_DASHBOARDS_API === 'true',
     dashboardScene: false, // this test is for the old sharing modal only used when scenes is turned off
   },
 });

e2e-playwright/dashboards-suite/dashboard-public-templating.spec.ts

@@ -3,7 +3,7 @@ import { test, expect } from '@grafana/plugin-e2e';
 test.use({
   featureToggles: {
     kubernetesDashboards: process.env.FORCE_V2_DASHBOARDS_API === 'true',
-    dashboardNewLayouts: process.env.FORCE_V2_DASHBOARDS_API === 'true',
+    kubernetesDashboardsV2: process.env.FORCE_V2_DASHBOARDS_API === 'true',
     dashboardScene: false, // this test is for the old sharing modal only used when scenes is turned off
   },
 });

e2e-playwright/dashboards-suite/dashboard-share-externally-create.spec.ts

@@ -4,7 +4,7 @@ test.use({
   featureToggles: {
     scenes: true,
     kubernetesDashboards: process.env.FORCE_V2_DASHBOARDS_API === 'true',
-    dashboardNewLayouts: process.env.FORCE_V2_DASHBOARDS_API === 'true',
+    kubernetesDashboardsV2: process.env.FORCE_V2_DASHBOARDS_API === 'true',
   },
 });
 

e2e-playwright/dashboards-suite/dashboard-share-internally.spec.ts

@@ -4,7 +4,7 @@ test.use({
   featureToggles: {
     scenes: true,
     kubernetesDashboards: process.env.FORCE_V2_DASHBOARDS_API === 'true',
-    dashboardNewLayouts: process.env.FORCE_V2_DASHBOARDS_API === 'true',
+    kubernetesDashboardsV2: process.env.FORCE_V2_DASHBOARDS_API === 'true',
   },
 });
 

e2e-playwright/dashboards-suite/dashboard-share-snapshot-create.spec.ts

@@ -6,7 +6,7 @@ test.use({
   featureToggles: {
     scenes: true,
     kubernetesDashboards: process.env.FORCE_V2_DASHBOARDS_API === 'true',
-    dashboardNewLayouts: process.env.FORCE_V2_DASHBOARDS_API === 'true',
+    kubernetesDashboardsV2: process.env.FORCE_V2_DASHBOARDS_API === 'true',
   },
 });
 

e2e-playwright/dashboards-suite/dashboard-templating.spec.ts

@@ -6,7 +6,7 @@ test.use({
   timezoneId: 'Pacific/Easter',
   featureToggles: {
     kubernetesDashboards: process.env.FORCE_V2_DASHBOARDS_API === 'true',
-    dashboardNewLayouts: process.env.FORCE_V2_DASHBOARDS_API === 'true',
+    kubernetesDashboardsV2: process.env.FORCE_V2_DASHBOARDS_API === 'true',
   },
 });
 

e2e-playwright/dashboards-suite/dashboard-time-zone.spec.ts

@@ -8,7 +8,7 @@ const TIMEZONE_DASHBOARD_UID = 'd41dbaa2-a39e-4536-ab2b-caca52f1a9c8';
 test.use({
   featureToggles: {
     kubernetesDashboards: process.env.FORCE_V2_DASHBOARDS_API === 'true',
-    dashboardNewLayouts: process.env.FORCE_V2_DASHBOARDS_API === 'true',
+    kubernetesDashboardsV2: process.env.FORCE_V2_DASHBOARDS_API === 'true',
   },
 });
 

e2e-playwright/dashboards-suite/dashboard-timepicker.spec.ts

@@ -17,7 +17,7 @@ test.use({
   },
   featureToggles: {
     kubernetesDashboards: process.env.FORCE_V2_DASHBOARDS_API === 'true',
-    dashboardNewLayouts: process.env.FORCE_V2_DASHBOARDS_API === 'true',
+    kubernetesDashboardsV2: process.env.FORCE_V2_DASHBOARDS_API === 'true',
   },
 });
 

e2e-playwright/dashboards-suite/embedded-dashboard.spec.ts

@@ -3,7 +3,7 @@ import { test, expect } from '@grafana/plugin-e2e';
 test.use({
   featureToggles: {
     kubernetesDashboards: process.env.FORCE_V2_DASHBOARDS_API === 'true',
-    dashboardNewLayouts: process.env.FORCE_V2_DASHBOARDS_API === 'true',
+    kubernetesDashboardsV2: process.env.FORCE_V2_DASHBOARDS_API === 'true',
   },
 });
 

e2e-playwright/dashboards-suite/general-dashboards.spec.ts

@@ -5,7 +5,7 @@ const PAGE_UNDER_TEST = 'edediimbjhdz4b/a-tall-dashboard';
 test.use({
   featureToggles: {
     kubernetesDashboards: process.env.FORCE_V2_DASHBOARDS_API === 'true',
-    dashboardNewLayouts: process.env.FORCE_V2_DASHBOARDS_API === 'true',
+    kubernetesDashboardsV2: process.env.FORCE_V2_DASHBOARDS_API === 'true',
   },
 });
 

e2e-playwright/dashboards-suite/import-dashboard.spec.ts

@@ -5,7 +5,7 @@ import testDashboard from '../dashboards/TestDashboard.json';
 test.use({
   featureToggles: {
     kubernetesDashboards: process.env.FORCE_V2_DASHBOARDS_API === 'true',
-    dashboardNewLayouts: process.env.FORCE_V2_DASHBOARDS_API === 'true',
+    kubernetesDashboardsV2: process.env.FORCE_V2_DASHBOARDS_API === 'true',
   },
 });
 

e2e-playwright/dashboards-suite/load-options-from-url.spec.ts

@@ -5,7 +5,7 @@ const PAGE_UNDER_TEST = '-Y-tnEDWk/templating-nested-template-variables';
 test.use({
   featureToggles: {
     kubernetesDashboards: process.env.FORCE_V2_DASHBOARDS_API === 'true',
-    dashboardNewLayouts: process.env.FORCE_V2_DASHBOARDS_API === 'true',
+    kubernetesDashboardsV2: process.env.FORCE_V2_DASHBOARDS_API === 'true',
   },
 });
 

e2e-playwright/dashboards-suite/new-constant-variable.spec.ts

@@ -6,7 +6,7 @@ const DASHBOARD_NAME = 'Test variable output';
 test.use({
   featureToggles: {
     kubernetesDashboards: process.env.FORCE_V2_DASHBOARDS_API === 'true',
-    dashboardNewLayouts: process.env.FORCE_V2_DASHBOARDS_API === 'true',
+    kubernetesDashboardsV2: process.env.FORCE_V2_DASHBOARDS_API === 'true',
   },
 });
 

e2e-playwright/dashboards-suite/new-custom-variable.spec.ts

@@ -53,7 +53,7 @@ async function assertPreviewValues(
 test.use({
   featureToggles: {
     kubernetesDashboards: process.env.FORCE_V2_DASHBOARDS_API === 'true',
-    dashboardNewLayouts: process.env.FORCE_V2_DASHBOARDS_API === 'true',
+    kubernetesDashboardsV2: process.env.FORCE_V2_DASHBOARDS_API === 'true',
   },
 });
 

e2e-playwright/dashboards-suite/new-datasource-variable.spec.ts

@@ -6,7 +6,7 @@ const DASHBOARD_NAME = 'Test variable output';
 test.use({
   featureToggles: {
     kubernetesDashboards: process.env.FORCE_V2_DASHBOARDS_API === 'true',
-    dashboardNewLayouts: process.env.FORCE_V2_DASHBOARDS_API === 'true',
+    kubernetesDashboardsV2: process.env.FORCE_V2_DASHBOARDS_API === 'true',
   },
 });
 

e2e-playwright/dashboards-suite/new-interval-variable.spec.ts

@@ -19,7 +19,7 @@ async function assertPreviewValues(
 test.use({
   featureToggles: {
     kubernetesDashboards: process.env.FORCE_V2_DASHBOARDS_API === 'true',
-    dashboardNewLayouts: process.env.FORCE_V2_DASHBOARDS_API === 'true',
+    kubernetesDashboardsV2: process.env.FORCE_V2_DASHBOARDS_API === 'true',
   },
 });
 

e2e-playwright/dashboards-suite/new-query-variable.spec.ts

@@ -6,7 +6,7 @@ const DASHBOARD_NAME = 'Templating - Nested Template Variables';
 test.use({
   featureToggles: {
     kubernetesDashboards: process.env.FORCE_V2_DASHBOARDS_API === 'true',
-    dashboardNewLayouts: process.env.FORCE_V2_DASHBOARDS_API === 'true',
+    kubernetesDashboardsV2: process.env.FORCE_V2_DASHBOARDS_API === 'true',
   },
 });
 

e2e-playwright/dashboards-suite/new-text-box-variable.spec.ts

@@ -6,7 +6,7 @@ const DASHBOARD_NAME = 'Test variable output';
 test.use({
   featureToggles: {
     kubernetesDashboards: process.env.FORCE_V2_DASHBOARDS_API === 'true',
-    dashboardNewLayouts: process.env.FORCE_V2_DASHBOARDS_API === 'true',
+    kubernetesDashboardsV2: process.env.FORCE_V2_DASHBOARDS_API === 'true',
   },
 });
 

e2e-playwright/dashboards-suite/repeating-a-panel-horizontally.spec.ts

@@ -5,7 +5,7 @@ const PAGE_UNDER_TEST = 'WVpf2jp7z/repeating-a-panel-horizontally';
 test.use({
   featureToggles: {
     kubernetesDashboards: process.env.FORCE_V2_DASHBOARDS_API === 'true',
-    dashboardNewLayouts: process.env.FORCE_V2_DASHBOARDS_API === 'true',
+    kubernetesDashboardsV2: process.env.FORCE_V2_DASHBOARDS_API === 'true',
   },
 });
 

e2e-playwright/dashboards-suite/repeating-a-panel-vertically.spec.ts

@@ -5,7 +5,7 @@ const PAGE_UNDER_TEST = 'OY8Ghjt7k/repeating-a-panel-vertically';
 test.use({
   featureToggles: {
     kubernetesDashboards: process.env.FORCE_V2_DASHBOARDS_API === 'true',
-    dashboardNewLayouts: process.env.FORCE_V2_DASHBOARDS_API === 'true',
+    kubernetesDashboardsV2: process.env.FORCE_V2_DASHBOARDS_API === 'true',
   },
 });
 

e2e-playwright/dashboards-suite/repeating-an-empty-row.spec.ts

@@ -5,7 +5,7 @@ const PAGE_UNDER_TEST = 'dtpl2Ctnk/repeating-an-empty-row';
 test.use({
   featureToggles: {
     kubernetesDashboards: process.env.FORCE_V2_DASHBOARDS_API === 'true',
-    dashboardNewLayouts: process.env.FORCE_V2_DASHBOARDS_API === 'true',
+    kubernetesDashboardsV2: process.env.FORCE_V2_DASHBOARDS_API === 'true',
   },
 });
 

e2e-playwright/dashboards-suite/set-options-from-ui.spec.ts

@@ -5,7 +5,7 @@ const PAGE_UNDER_TEST = '-Y-tnEDWk/templating-nested-template-variables';
 test.use({
   featureToggles: {
     kubernetesDashboards: process.env.FORCE_V2_DASHBOARDS_API === 'true',
-    dashboardNewLayouts: process.env.FORCE_V2_DASHBOARDS_API === 'true',
+    kubernetesDashboardsV2: process.env.FORCE_V2_DASHBOARDS_API === 'true',
   },
 });
 

e2e-playwright/dashboards-suite/snapshot-create.spec.ts

@@ -5,7 +5,7 @@ const DASHBOARD_UID = 'ZqZnVvFZz';
 test.use({
   featureToggles: {
     kubernetesDashboards: process.env.FORCE_V2_DASHBOARDS_API === 'true',
-    dashboardNewLayouts: process.env.FORCE_V2_DASHBOARDS_API === 'true',
+    kubernetesDashboardsV2: process.env.FORCE_V2_DASHBOARDS_API === 'true',
     dashboardScene: false, // this test is for the old sharing modal only used when scenes is turned off
   },
 });

e2e-playwright/dashboards-suite/templating-dashboard-links-and-variables.spec.ts

@@ -5,7 +5,7 @@ const DASHBOARD_UID = 'yBCC3aKGk';
 test.use({
   featureToggles: {
     kubernetesDashboards: process.env.FORCE_V2_DASHBOARDS_API === 'true',
-    dashboardNewLayouts: process.env.FORCE_V2_DASHBOARDS_API === 'true',
+    kubernetesDashboardsV2: process.env.FORCE_V2_DASHBOARDS_API === 'true',
   },
 });