Compare commits

...
Author SHA1 Message Date
Sonia Aguilar 7264ca9acf Merge remote-tracking branch 'origin/main' into alerting/UI-for-imported-timeintervals 2026-01-14 17:47:21 +01:00
Adela Almasan 99acd3766d Suggestions: Update empty state (#116172) 2026-01-14 10:37:42 -06:00
Sonia Aguilar 77ccbeb543 Refactor: Use isImportedResource and isProvisionedResource utilities
- Replace manual provenance comparisons with existing utility functions
- Use isImportedResource() instead of comparing with 'prometheus_convert' directly
- Use isProvisionedResource() instead of manual PROVENANCE_NONE checks
- Update tests to use correct provenance value 'converted_prometheus' (from KnownProvenance enum)
- Remove redundant constant definitions
2026-01-14 16:36:40 +01:00
Sonia Aguilar 11835de49f Merge remote-tracking branch 'origin/main' into alerting/UI-for-imported-timeintervals 2026-01-14 16:16:25 +01:00
Sonia Aguilar 41168594f3 Merge remote-tracking branch 'origin/main' into alerting/UI-for-imported-timeintervals 2026-01-14 12:53:04 +01:00
Sonia Aguilar 3a1f19bdf3 lint 2026-01-14 12:44:26 +01:00
Sonia Aguilar f395a749a4 Alerting: Add UI for imported time intervals
- Add ImportedTimeIntervalAlert component for time intervals imported from external Alertmanager
- Filter imported time intervals (provenance: prometheus_convert) from notification policy selector
- Refactor MuteTimingForm to use provenance prop instead of separate provisioned/imported props
- Add comprehensive tests for MuteTimingsSelector filtering behavior
- Add tests for MuteTimingForm with different provenance values
- Add i18n translations for imported time interval alerts
2026-01-14 12:02:24 +01:00
13 changed files with 405 additions and 26 deletions
-10
View File
@@ -1392,11 +1392,6 @@
"count": 2 "count": 2
} }
}, },
"public/app/features/alerting/unified/components/mute-timings/MuteTimingForm.tsx": {
"no-restricted-syntax": {
"count": 1
}
},
"public/app/features/alerting/unified/components/mute-timings/MuteTimingTimeInterval.tsx": { "public/app/features/alerting/unified/components/mute-timings/MuteTimingTimeInterval.tsx": {
"no-restricted-syntax": { "no-restricted-syntax": {
"count": 5 "count": 5
@@ -1521,11 +1516,6 @@
"count": 1 "count": 1
} }
}, },
"public/app/features/alerting/unified/components/rule-editor/alert-rule-form/simplifiedRouting/route-settings/MuteTimingFields.tsx": {
"no-restricted-syntax": {
"count": 1
}
},
"public/app/features/alerting/unified/components/rule-editor/alert-rule-form/simplifiedRouting/route-settings/RouteSettings.tsx": { "public/app/features/alerting/unified/components/rule-editor/alert-rule-form/simplifiedRouting/route-settings/RouteSettings.tsx": {
"no-restricted-syntax": { "no-restricted-syntax": {
"count": 1 "count": 1
@@ -56,6 +56,24 @@ export const ImportedContactPointAlert = (props: ExtraAlertProps) => {
); );
}; };
export const ImportedTimeIntervalAlert = (props: ExtraAlertProps) => {
return (
<Alert
title={t(
'alerting.provisioning.title-imported-time-interval',
'This time interval was imported and cannot be edited through the UI'
)}
severity="info"
{...props}
>
<Trans i18nKey="alerting.provisioning.body-imported-time-interval">
This time interval was imported from an external Alertmanager and is currently read-only. The time interval will
become editable after the migration process is complete.
</Trans>
</Alert>
);
};
export const ProvisioningBadge = ({ export const ProvisioningBadge = ({
tooltip, tooltip,
provenance, provenance,
@@ -0,0 +1,130 @@
import { render, screen, userEvent } from 'test/test-utils';
import { setupMswServer } from 'app/features/alerting/unified/mockApi';
import { grantUserPermissions } from 'app/features/alerting/unified/mocks';
import { setTimeIntervalsList } from 'app/features/alerting/unified/mocks/server/configure';
import { AlertmanagerProvider } from 'app/features/alerting/unified/state/AlertmanagerContext';
import { GRAFANA_RULES_SOURCE_NAME } from 'app/features/alerting/unified/utils/datasource';
import { AccessControlAction } from 'app/types/accessControl';
import MuteTimingsSelector from './MuteTimingsSelector';
const renderWithProvider = (alertManagerSource = GRAFANA_RULES_SOURCE_NAME) => {
return render(
<AlertmanagerProvider accessType={'notification'} alertmanagerSourceName={alertManagerSource}>
<MuteTimingsSelector
alertmanager={alertManagerSource}
selectProps={{
onChange: () => {},
}}
/>
</AlertmanagerProvider>
);
};
setupMswServer();
describe('MuteTimingsSelector', () => {
beforeEach(() => {
grantUserPermissions([
AccessControlAction.AlertingNotificationsRead,
AccessControlAction.AlertingNotificationsWrite,
]);
});
it('should show all non-imported time intervals', async () => {
const user = userEvent.setup();
setTimeIntervalsList([
{ name: 'regular-interval', provenance: 'none' },
{ name: 'file-provisioned', provenance: 'file' },
{ name: 'another-regular', provenance: 'none' },
]);
renderWithProvider();
// Click to open the dropdown
const selector = await screen.findByRole('combobox', { name: /time intervals/i });
await user.click(selector);
// All non-imported intervals should be visible
expect(await screen.findByText('regular-interval')).toBeInTheDocument();
expect(screen.getByText('file-provisioned')).toBeInTheDocument();
expect(screen.getByText('another-regular')).toBeInTheDocument();
});
it('should filter out imported time intervals (provenance: converted_prometheus)', async () => {
const user = userEvent.setup();
setTimeIntervalsList([
{ name: 'regular-interval', provenance: 'none' },
{ name: 'imported-interval', provenance: 'converted_prometheus' },
{ name: 'file-provisioned', provenance: 'file' },
]);
renderWithProvider();
// Click to open the dropdown
const selector = await screen.findByRole('combobox', { name: /time intervals/i });
await user.click(selector);
// Regular and file-provisioned should be visible
expect(await screen.findByText('regular-interval')).toBeInTheDocument();
expect(screen.getByText('file-provisioned')).toBeInTheDocument();
// Imported interval should NOT be in the list
expect(screen.queryByText('imported-interval')).not.toBeInTheDocument();
});
it('should show only non-imported intervals when all types are present', async () => {
const user = userEvent.setup();
setTimeIntervalsList([
{ name: 'normal-1', provenance: 'none' },
{ name: 'imported-1', provenance: 'converted_prometheus' },
{ name: 'normal-2', provenance: 'none' },
{ name: 'imported-2', provenance: 'converted_prometheus' },
{ name: 'file-1', provenance: 'file' },
]);
renderWithProvider();
// Click to open the dropdown
const selector = await screen.findByRole('combobox', { name: /time intervals/i });
await user.click(selector);
// Non-imported intervals should be visible
expect(await screen.findByText('normal-1')).toBeInTheDocument();
expect(screen.getByText('normal-2')).toBeInTheDocument();
expect(screen.getByText('file-1')).toBeInTheDocument();
// Imported intervals should NOT be visible
expect(screen.queryByText('imported-1')).not.toBeInTheDocument();
expect(screen.queryByText('imported-2')).not.toBeInTheDocument();
});
it('should handle empty list', async () => {
setTimeIntervalsList([]);
renderWithProvider();
// Selector should be present but have no options
const selector = await screen.findByRole('combobox', { name: /time intervals/i });
expect(selector).toBeInTheDocument();
});
it('should handle list with only imported intervals', async () => {
const user = userEvent.setup();
setTimeIntervalsList([
{ name: 'imported-1', provenance: 'converted_prometheus' },
{ name: 'imported-2', provenance: 'converted_prometheus' },
]);
renderWithProvider();
// Click to open the dropdown
const selector = await screen.findByRole('combobox', { name: /time intervals/i });
await user.click(selector);
// No intervals should be visible
expect(screen.queryByText('imported-1')).not.toBeInTheDocument();
expect(screen.queryByText('imported-2')).not.toBeInTheDocument();
});
});
@@ -1,17 +1,24 @@
import { SelectableValue } from '@grafana/data'; import { SelectableValue } from '@grafana/data';
import { t } from '@grafana/i18n'; import { t } from '@grafana/i18n';
import { MultiSelect, MultiSelectCommonProps } from '@grafana/ui'; import { MultiSelect, MultiSelectCommonProps } from '@grafana/ui';
import { useMuteTimings } from 'app/features/alerting/unified/components/mute-timings/useMuteTimings'; import { MuteTiming, useMuteTimings } from 'app/features/alerting/unified/components/mute-timings/useMuteTimings';
import { BaseAlertmanagerArgs } from 'app/features/alerting/unified/types/hooks'; import { BaseAlertmanagerArgs } from 'app/features/alerting/unified/types/hooks';
import { timeIntervalToString } from 'app/features/alerting/unified/utils/alertmanager'; import { timeIntervalToString } from 'app/features/alerting/unified/utils/alertmanager';
import { MuteTimeInterval } from 'app/plugins/datasource/alertmanager/types'; import { K8sAnnotations } from 'app/features/alerting/unified/utils/k8s/constants';
import { isImportedResource } from 'app/features/alerting/unified/utils/k8s/utils';
const mapTimeInterval = ({ name, time_intervals }: MuteTimeInterval): SelectableValue<string> => ({ const mapTimeInterval = ({ name, time_intervals }: MuteTiming): SelectableValue<string> => ({
value: name, value: name,
label: name, label: name,
description: time_intervals.map((interval) => timeIntervalToString(interval)).join(', AND '), description: time_intervals.map((interval) => timeIntervalToString(interval)).join(', AND '),
}); });
/** Check if a time interval was imported from an external Alertmanager */
const isImportedTimeInterval = (timing: MuteTiming): boolean => {
const provenance = timing.metadata?.annotations?.[K8sAnnotations.Provenance];
return isImportedResource(provenance);
};
/** Provides a MultiSelect with available time intervals for the given alertmanager */ /** Provides a MultiSelect with available time intervals for the given alertmanager */
const TimeIntervalSelector = ({ const TimeIntervalSelector = ({
alertmanager, alertmanager,
@@ -19,7 +26,9 @@ const TimeIntervalSelector = ({
}: BaseAlertmanagerArgs & { selectProps: MultiSelectCommonProps<string> }) => { }: BaseAlertmanagerArgs & { selectProps: MultiSelectCommonProps<string> }) => {
const { data } = useMuteTimings({ alertmanager, skip: selectProps.disabled }); const { data } = useMuteTimings({ alertmanager, skip: selectProps.disabled });
const timeIntervalOptions = data?.map((value) => mapTimeInterval(value)) || []; // Filter out imported time intervals (provenance === 'prometheus_convert')
const availableTimings = data?.filter((timing) => !isImportedTimeInterval(timing)) || [];
const timeIntervalOptions = availableTimings.map((value) => mapTimeInterval(value));
return ( return (
<MultiSelect <MultiSelect
@@ -3,6 +3,7 @@ import { Navigate } from 'react-router-dom-v5-compat';
import { t } from '@grafana/i18n'; import { t } from '@grafana/i18n';
import { useGetMuteTiming } from 'app/features/alerting/unified/components/mute-timings/useMuteTimings'; import { useGetMuteTiming } from 'app/features/alerting/unified/components/mute-timings/useMuteTimings';
import { useURLSearchParams } from 'app/features/alerting/unified/hooks/useURLSearchParams'; import { useURLSearchParams } from 'app/features/alerting/unified/hooks/useURLSearchParams';
import { K8sAnnotations } from 'app/features/alerting/unified/utils/k8s/constants';
import { useAlertmanager } from '../../state/AlertmanagerContext'; import { useAlertmanager } from '../../state/AlertmanagerContext';
import { withPageErrorBoundary } from '../../withPageErrorBoundary'; import { withPageErrorBoundary } from '../../withPageErrorBoundary';
@@ -28,13 +29,15 @@ const EditTimingRoute = () => {
return <Navigate replace to="/alerting/routes" />; return <Navigate replace to="/alerting/routes" />;
} }
const provenance = timeInterval?.metadata?.annotations?.[K8sAnnotations.Provenance];
return ( return (
<MuteTimingForm <MuteTimingForm
editMode editMode
loading={isLoading} loading={isLoading}
showError={isError} showError={isError}
muteTiming={timeInterval} muteTiming={timeInterval}
provisioned={timeInterval?.provisioned} provenance={provenance}
/> />
); );
}; };
@@ -0,0 +1,103 @@
import { render, screen } from 'test/test-utils';
import { setupMswServer } from 'app/features/alerting/unified/mockApi';
import { grantUserPermissions } from 'app/features/alerting/unified/mocks';
import { AlertmanagerProvider } from 'app/features/alerting/unified/state/AlertmanagerContext';
import { GRAFANA_RULES_SOURCE_NAME } from 'app/features/alerting/unified/utils/datasource';
import { AccessControlAction } from 'app/types/accessControl';
import MuteTimingForm from './MuteTimingForm';
import { muteTimeInterval } from './mocks';
const renderWithProvider = (provenance?: string, editMode = false) => {
return render(
<AlertmanagerProvider accessType={'notification'} alertmanagerSourceName={GRAFANA_RULES_SOURCE_NAME}>
<MuteTimingForm
muteTiming={{ id: 'mock-id', ...muteTimeInterval }}
provenance={provenance}
editMode={editMode}
loading={false}
showError={false}
/>
</AlertmanagerProvider>
);
};
setupMswServer();
describe('MuteTimingForm', () => {
beforeEach(() => {
grantUserPermissions([
AccessControlAction.AlertingNotificationsRead,
AccessControlAction.AlertingNotificationsWrite,
]);
});
it('should not show any alert when provenance is none', async () => {
renderWithProvider('none');
expect(screen.queryByText(/imported and cannot be edited/i)).not.toBeInTheDocument();
expect(screen.queryByText(/provisioned/i)).not.toBeInTheDocument();
});
it('should not show any alert when provenance is undefined', async () => {
renderWithProvider(undefined);
expect(screen.queryByText(/imported and cannot be edited/i)).not.toBeInTheDocument();
expect(screen.queryByText(/provisioned/i)).not.toBeInTheDocument();
});
it('should show imported alert when provenance is converted_prometheus', async () => {
renderWithProvider('converted_prometheus');
expect(
await screen.findByText(/This time interval was imported and cannot be edited through the UI/i)
).toBeInTheDocument();
expect(
screen.getByText(/This time interval was imported from an external Alertmanager and is currently read-only/i)
).toBeInTheDocument();
});
it('should show provisioning alert when provenance is file', async () => {
renderWithProvider('file');
expect(await screen.findByText(/This time interval cannot be edited through the UI/i)).toBeInTheDocument();
expect(
screen.getByText(/This time interval has been provisioned, that means it was created by config/i)
).toBeInTheDocument();
});
it('should show provisioning alert for other provenance types', async () => {
renderWithProvider('api');
expect(await screen.findByText(/This time interval cannot be edited through the UI/i)).toBeInTheDocument();
});
it('should disable form when provenance is converted_prometheus', async () => {
renderWithProvider('converted_prometheus', true);
const nameInput = await screen.findByTestId('mute-timing-name');
expect(nameInput).toBeDisabled();
});
it('should disable form when provenance is file', async () => {
renderWithProvider('file', true);
const nameInput = await screen.findByTestId('mute-timing-name');
expect(nameInput).toBeDisabled();
});
it('should enable form when provenance is none', async () => {
renderWithProvider('none', true);
const nameInput = await screen.findByTestId('mute-timing-name');
expect(nameInput).toBeEnabled();
});
it('should enable form when provenance is undefined', async () => {
renderWithProvider(undefined, true);
const nameInput = await screen.findByTestId('mute-timing-name');
expect(nameInput).toBeEnabled();
});
});
@@ -14,9 +14,10 @@ import {
import { useAlertmanager } from '../../state/AlertmanagerContext'; import { useAlertmanager } from '../../state/AlertmanagerContext';
import { MuteTimingFields } from '../../types/mute-timing-form'; import { MuteTimingFields } from '../../types/mute-timing-form';
import { isImportedResource, isProvisionedResource } from '../../utils/k8s/utils';
import { makeAMLink } from '../../utils/misc'; import { makeAMLink } from '../../utils/misc';
import { createMuteTiming, defaultTimeInterval, isTimeIntervalDisabled } from '../../utils/mute-timings'; import { createMuteTiming, defaultTimeInterval, isTimeIntervalDisabled } from '../../utils/mute-timings';
import { ProvisionedResource, ProvisioningAlert } from '../Provisioning'; import { ImportedTimeIntervalAlert, ProvisionedResource, ProvisioningAlert } from '../Provisioning';
import { MuteTimingTimeInterval } from './MuteTimingTimeInterval'; import { MuteTimingTimeInterval } from './MuteTimingTimeInterval';
@@ -24,8 +25,8 @@ interface Props {
muteTiming?: MuteTiming; muteTiming?: MuteTiming;
showError?: boolean; showError?: boolean;
loading?: boolean; loading?: boolean;
/** Is the current mute timing provisioned? If so, will disable editing via UI */ /** Provenance of the mute timing - indicates how it was created (e.g., 'file', 'prometheus_convert', 'none') */
provisioned?: boolean; provenance?: string;
/** Are we editing an existing time interval? */ /** Are we editing an existing time interval? */
editMode?: boolean; editMode?: boolean;
} }
@@ -56,7 +57,7 @@ const useDefaultValues = (muteTiming?: MuteTiming): MuteTimingFields => {
}; };
}; };
const MuteTimingForm = ({ muteTiming, showError, loading, provisioned, editMode }: Props) => { const MuteTimingForm = ({ muteTiming, showError, loading, provenance, editMode }: Props) => {
const { selectedAlertmanager } = useAlertmanager(); const { selectedAlertmanager } = useAlertmanager();
const hookArgs = { alertmanager: selectedAlertmanager! }; const hookArgs = { alertmanager: selectedAlertmanager! };
@@ -105,14 +106,19 @@ const MuteTimingForm = ({ muteTiming, showError, loading, provisioned, editMode
); );
} }
const isProvisioned = isProvisionedResource(provenance);
const isImported = isImportedResource(provenance);
return ( return (
<> <>
{provisioned && <ProvisioningAlert resource={ProvisionedResource.MuteTiming} />} {isProvisioned && isImported && <ImportedTimeIntervalAlert />}
{isProvisioned && !isImported && <ProvisioningAlert resource={ProvisionedResource.MuteTiming} />}
<FormProvider {...formApi}> <FormProvider {...formApi}>
<form onSubmit={formApi.handleSubmit(onSubmit)} data-testid="mute-timing-form"> <form onSubmit={formApi.handleSubmit(onSubmit)} data-testid="mute-timing-form">
<FieldSet disabled={provisioned || updating}> <FieldSet disabled={isProvisioned || updating}>
<Field <Field
required required
noMargin
label={t('alerting.mute-timing-form.label-name', 'Name')} label={t('alerting.mute-timing-form.label-name', 'Name')}
description={t( description={t(
'alerting.time-interval-form.description-unique-time-interval', 'alerting.time-interval-form.description-unique-time-interval',
@@ -27,6 +27,7 @@ export function MuteTimingFields({ alertmanager }: BaseAlertmanagerArgs) {
)} )}
className={styles.muteTimingField} className={styles.muteTimingField}
invalid={!!errors.contactPoints?.[alertmanager]?.muteTimeIntervals} invalid={!!errors.contactPoints?.[alertmanager]?.muteTimeIntervals}
noMargin
> >
<Controller <Controller
render={({ field: { onChange, ref, ...field } }) => ( render={({ field: { onChange, ref, ...field } }) => (
@@ -175,6 +175,36 @@ export const setTimeIntervalsListEmpty = () => {
return handler; return handler;
}; };
interface TimeIntervalConfig {
name: string;
provenance?: string;
}
/**
* Makes the mock server respond with custom time intervals
*/
export const setTimeIntervalsList = (intervals: TimeIntervalConfig[]) => {
const listMuteTimingsPath = listNamespacedTimeIntervalHandler().info.path;
const handler = http.get(listMuteTimingsPath, () => {
const items = intervals.map((interval) => ({
metadata: {
annotations: {
'grafana.com/provenance': interval.provenance ?? 'none',
},
name: interval.name,
uid: `uid-${interval.name}`,
namespace: 'default',
resourceVersion: 'e0270bfced786660',
},
spec: { name: interval.name, time_intervals: [] },
}));
return HttpResponse.json(getK8sResponse('TimeIntervalList', items));
});
server.use(handler);
return handler;
};
export function mimirDataSource() { export function mimirDataSource() {
const dataSource = mockDataSource( const dataSource = mockDataSource(
{ {
@@ -65,3 +65,7 @@ export const stringifyFieldSelector = (fieldSelectors: FieldSelector[]): string
export function isProvisionedResource(provenance?: string): boolean { export function isProvisionedResource(provenance?: string): boolean {
return Boolean(provenance && provenance !== KnownProvenance.None); return Boolean(provenance && provenance !== KnownProvenance.None);
} }
export function isImportedResource(provenance?: string): boolean {
return provenance === KnownProvenance.ConvertedPrometheus;
}
@@ -2,8 +2,9 @@ import { render, screen } from '@testing-library/react';
import { defaultsDeep } from 'lodash'; import { defaultsDeep } from 'lodash';
import { Provider } from 'react-redux'; import { Provider } from 'react-redux';
import { FieldType, getDefaultTimeRange, LoadingState } from '@grafana/data'; import { CoreApp, EventBusSrv, FieldType, getDefaultTimeRange, LoadingState } from '@grafana/data';
import { PanelDataErrorViewProps } from '@grafana/runtime'; import { config, PanelDataErrorViewProps } from '@grafana/runtime';
import { usePanelContext } from '@grafana/ui';
import { configureStore } from 'app/store/configureStore'; import { configureStore } from 'app/store/configureStore';
import { PanelDataErrorView } from './PanelDataErrorView'; import { PanelDataErrorView } from './PanelDataErrorView';
@@ -16,7 +17,24 @@ jest.mock('app/features/dashboard/services/DashboardSrv', () => ({
}, },
})); }));
jest.mock('@grafana/ui', () => ({
...jest.requireActual('@grafana/ui'),
usePanelContext: jest.fn(),
}));
const mockUsePanelContext = jest.mocked(usePanelContext);
const RUN_QUERY_MESSAGE = 'Run a query to visualize it here or go to all visualizations to add other panel types';
const panelContextRoot = {
app: CoreApp.Dashboard,
eventsScope: 'global',
eventBus: new EventBusSrv(),
};
describe('PanelDataErrorView', () => { describe('PanelDataErrorView', () => {
beforeEach(() => {
mockUsePanelContext.mockReturnValue(panelContextRoot);
});
it('show No data when there is no data', () => { it('show No data when there is no data', () => {
renderWithProps(); renderWithProps();
@@ -70,6 +88,45 @@ describe('PanelDataErrorView', () => {
expect(screen.getByText('Query returned nothing')).toBeInTheDocument(); expect(screen.getByText('Query returned nothing')).toBeInTheDocument();
}); });
it('should show "Run a query..." message when no query is configured and feature toggle is enabled', () => {
mockUsePanelContext.mockReturnValue(panelContextRoot);
const originalFeatureToggle = config.featureToggles.newVizSuggestions;
config.featureToggles.newVizSuggestions = true;
renderWithProps({
data: {
state: LoadingState.Done,
series: [],
timeRange: getDefaultTimeRange(),
},
});
expect(screen.getByText(RUN_QUERY_MESSAGE)).toBeInTheDocument();
config.featureToggles.newVizSuggestions = originalFeatureToggle;
});
it('should show "No data" message when feature toggle is disabled even without queries', () => {
mockUsePanelContext.mockReturnValue(panelContextRoot);
const originalFeatureToggle = config.featureToggles.newVizSuggestions;
config.featureToggles.newVizSuggestions = false;
renderWithProps({
data: {
state: LoadingState.Done,
series: [],
timeRange: getDefaultTimeRange(),
},
});
expect(screen.getByText('No data')).toBeInTheDocument();
expect(screen.queryByText(RUN_QUERY_MESSAGE)).not.toBeInTheDocument();
config.featureToggles.newVizSuggestions = originalFeatureToggle;
});
}); });
function renderWithProps(overrides?: Partial<PanelDataErrorViewProps>) { function renderWithProps(overrides?: Partial<PanelDataErrorViewProps>) {
@@ -5,14 +5,15 @@ import {
FieldType, FieldType,
getPanelDataSummary, getPanelDataSummary,
GrafanaTheme2, GrafanaTheme2,
PanelData,
PanelDataSummary, PanelDataSummary,
PanelPluginVisualizationSuggestion, PanelPluginVisualizationSuggestion,
} from '@grafana/data'; } from '@grafana/data';
import { selectors } from '@grafana/e2e-selectors'; import { selectors } from '@grafana/e2e-selectors';
import { t, Trans } from '@grafana/i18n'; import { t, Trans } from '@grafana/i18n';
import { PanelDataErrorViewProps, locationService } from '@grafana/runtime'; import { PanelDataErrorViewProps, locationService, config } from '@grafana/runtime';
import { VizPanel } from '@grafana/scenes'; import { VizPanel } from '@grafana/scenes';
import { usePanelContext, useStyles2 } from '@grafana/ui'; import { Icon, usePanelContext, useStyles2 } from '@grafana/ui';
import { CardButton } from 'app/core/components/CardButton'; import { CardButton } from 'app/core/components/CardButton';
import { LS_VISUALIZATION_SELECT_TAB_KEY } from 'app/core/constants'; import { LS_VISUALIZATION_SELECT_TAB_KEY } from 'app/core/constants';
import store from 'app/core/store'; import store from 'app/core/store';
@@ -24,6 +25,11 @@ import { findVizPanelByKey, getVizPanelKeyForPanelId } from 'app/features/dashbo
import { useDispatch } from 'app/types/store'; import { useDispatch } from 'app/types/store';
import { changePanelPlugin } from '../state/actions'; import { changePanelPlugin } from '../state/actions';
import { hasData } from '../suggestions/utils';
function hasNoQueryConfigured(data: PanelData): boolean {
return !data.request?.targets || data.request.targets.length === 0;
}
export function PanelDataErrorView(props: PanelDataErrorViewProps) { export function PanelDataErrorView(props: PanelDataErrorViewProps) {
const styles = useStyles2(getStyles); const styles = useStyles2(getStyles);
@@ -93,8 +99,14 @@ export function PanelDataErrorView(props: PanelDataErrorViewProps) {
} }
}; };
const noData = !hasData(props.data);
const noQueryConfigured = hasNoQueryConfigured(props.data);
const showEmptyState =
config.featureToggles.newVizSuggestions && context.app === CoreApp.PanelEditor && noQueryConfigured && noData;
return ( return (
<div className={styles.wrapper}> <div className={styles.wrapper}>
{showEmptyState && <Icon name="chart-line" size="xxxl" className={styles.emptyStateIcon} />}
<div className={styles.message} data-testid={selectors.components.Panels.Panel.PanelDataErrorMessage}> <div className={styles.message} data-testid={selectors.components.Panels.Panel.PanelDataErrorMessage}>
{message} {message}
</div> </div>
@@ -131,7 +143,17 @@ function getMessageFor(
return message; return message;
} }
if (!data.series || data.series.length === 0 || data.series.every((frame) => frame.length === 0)) { const noData = !hasData(data);
const noQueryConfigured = hasNoQueryConfigured(data);
if (config.featureToggles.newVizSuggestions && noQueryConfigured && noData) {
return t(
'dashboard.new-panel.empty-state-message',
'Run a query to visualize it here or go to all visualizations to add other panel types'
);
}
if (noData) {
return fieldConfig?.defaults.noValue ?? t('panel.panel-data-error-view.no-value.default', 'No data'); return fieldConfig?.defaults.noValue ?? t('panel.panel-data-error-view.no-value.default', 'No data');
} }
@@ -176,5 +198,9 @@ const getStyles = (theme: GrafanaTheme2) => {
width: '100%', width: '100%',
maxWidth: '600px', maxWidth: '600px',
}), }),
emptyStateIcon: css({
color: theme.colors.text.secondary,
marginBottom: theme.spacing(2),
}),
}; };
}; };
+2
View File
@@ -2178,8 +2178,10 @@
"badge-tooltip-provenance": "This resource has been provisioned via {{provenance}} and cannot be edited through the UI", "badge-tooltip-provenance": "This resource has been provisioned via {{provenance}} and cannot be edited through the UI",
"badge-tooltip-standard": "This resource has been provisioned and cannot be edited through the UI", "badge-tooltip-standard": "This resource has been provisioned and cannot be edited through the UI",
"body-imported": "This contact point contains integrations that were imported from an external Alertmanager and is currently read-only. The integrations will become editable after the migration process is complete.", "body-imported": "This contact point contains integrations that were imported from an external Alertmanager and is currently read-only. The integrations will become editable after the migration process is complete.",
"body-imported-time-interval": "This time interval was imported from an external Alertmanager and is currently read-only. The time interval will become editable after the migration process is complete.",
"body-provisioned": "This {{resource}} has been provisioned, that means it was created by config. Please contact your server admin to update this {{resource}}.", "body-provisioned": "This {{resource}} has been provisioned, that means it was created by config. Please contact your server admin to update this {{resource}}.",
"title-imported": "This contact point was imported and cannot be edited through the UI", "title-imported": "This contact point was imported and cannot be edited through the UI",
"title-imported-time-interval": "This time interval was imported and cannot be edited through the UI",
"title-provisioned": "This {{resource}} cannot be edited through the UI" "title-provisioned": "This {{resource}} cannot be edited through the UI"
}, },
"provisioning-badge": { "provisioning-badge": {