enable list deprecations for workspace and resolve existing

Signed-off-by: Kurt King <[email protected]>

Author: kurtaking

workspaces/grafana/.changeset/empty-oranges-bake.md

@@ -0,0 +1,5 @@
+---
+'@backstage-community/plugin-grafana': minor
+---
+
+All existing deprecated references have been removed.

workspaces/grafana/bcp.json

@@ -1,4 +1,5 @@
 {
   "autoVersionBump": true,
-  "knipReports": false
+  "knipReports": false,
+  "listDeprecations": true
 }

workspaces/grafana/plugins/grafana/report.api.md

@@ -105,9 +105,7 @@ export const grafanaPlugin: BackstagePlugin<{}, {}, {}>;
 export const isAlertSelectorAvailable: (entity: Entity) => boolean;
 
 // @public
-export const isDashboardSelectorAvailable: (
-  entity: Entity,
-) => string | undefined;
+export const isDashboardSelectorAvailable: (entity: Entity) => string;
 
 // @public
 export const isOverviewDashboardAvailable: (entity: Entity) => boolean;

workspaces/grafana/plugins/grafana/src/__fixtures__/entity.ts

@@ -17,7 +17,6 @@ import {
   GRAFANA_ANNOTATION_ALERT_LABEL_SELECTOR,
   GRAFANA_ANNOTATION_OVERVIEW_DASHBOARD,
   GRAFANA_ANNOTATION_DASHBOARD_SELECTOR,
-  GRAFANA_ANNOTATION_TAG_SELECTOR,
 } from '../constants';
 
 export const sampleEntity = {
@@ -28,7 +27,6 @@ export const sampleEntity = {
       name: 'awesome-service',
       annotations: {
         [GRAFANA_ANNOTATION_ALERT_LABEL_SELECTOR]: 'service=awesome-service',
-        [GRAFANA_ANNOTATION_TAG_SELECTOR]: 'awesome-tag',
         [GRAFANA_ANNOTATION_DASHBOARD_SELECTOR]:
           "(tags @> 'my-service' || tags @> 'my-service-slo') && tags @> 'generated'",
         [GRAFANA_ANNOTATION_OVERVIEW_DASHBOARD]:

workspaces/grafana/plugins/grafana/src/components/AlertsCard/AlertsCard.tsx

@@ -37,12 +37,12 @@ import useAsync from 'react-use/lib/useAsync';
 import { Alert } from '@material-ui/lab';
 import { AlertsCardOpts, Alert as GrafanaAlert } from '../../types';
 import {
-  GRAFANA_ANNOTATION_TAG_SELECTOR,
   GRAFANA_ANNOTATION_ALERT_LABEL_SELECTOR,
   isAlertSelectorAvailable,
   isDashboardSelectorAvailable,
-  tagSelectorFromEntity,
   alertSelectorFromEntity,
+  GRAFANA_ANNOTATION_DASHBOARD_SELECTOR,
+  dashboardSelectorFromEntity,
 } from '../../constants';
 
 const AlertStatusBadge = ({ alert }: { alert: GrafanaAlert }) => {
@@ -128,7 +128,7 @@ const Alerts = ({ entity, opts }: { entity: Entity; opts: AlertsCardOpts }) => {
     configApi.getOptionalBoolean('grafana.unifiedAlerting') || false;
   const alertSelector = unifiedAlertingEnabled
     ? alertSelectorFromEntity(entity)
-    : tagSelectorFromEntity(entity);
+    : dashboardSelectorFromEntity(entity);
 
   const { value, loading, error } = useAsync(
     async () => await grafanaApi.alertsForSelector(alertSelector),
@@ -152,7 +152,7 @@ export const AlertsCard = (opts?: AlertsCardOpts) => {
   if (!unifiedAlertingEnabled && !isDashboardSelectorAvailable(entity)) {
     return (
       <MissingAnnotationEmptyState
-        annotation={GRAFANA_ANNOTATION_TAG_SELECTOR}
+        annotation={GRAFANA_ANNOTATION_DASHBOARD_SELECTOR}
       />
     );
   }

workspaces/grafana/plugins/grafana/src/constants.ts

@@ -49,8 +49,7 @@ export const GRAFANA_ANNOTATION_OVERVIEW_DASHBOARD =
  * @public
  */
 export const isDashboardSelectorAvailable = (entity: Entity) =>
-  entity?.metadata.annotations?.[GRAFANA_ANNOTATION_DASHBOARD_SELECTOR] ||
-  entity?.metadata.annotations?.[GRAFANA_ANNOTATION_TAG_SELECTOR];
+  entity?.metadata.annotations?.[GRAFANA_ANNOTATION_DASHBOARD_SELECTOR] ?? '';
 
 /**
  * Returns if the alert selector annotation for an entity is set
@@ -75,9 +74,7 @@ export const isOverviewDashboardAvailable = (entity: Entity) =>
  * @public
  */
 export const dashboardSelectorFromEntity = (entity: Entity) =>
-  entity?.metadata.annotations?.[GRAFANA_ANNOTATION_DASHBOARD_SELECTOR] ??
-  entity?.metadata.annotations?.[GRAFANA_ANNOTATION_TAG_SELECTOR] ??
-  '';
+  entity?.metadata.annotations?.[GRAFANA_ANNOTATION_DASHBOARD_SELECTOR] ?? '';
 /**
  * Returns the alert selector annotation for an entity
  * @public