Skip to content

Commit

Permalink
[APM] Provide * to the params.environment field when environment_all (#…
Browse files Browse the repository at this point in the history
…160103)

Bug fix for
#159958 (comment)

### What was done

Provide * to the params.environment field when environment_all,
actionable observability will handle it internally
  • Loading branch information
MiriamAparicio authored Jun 21, 2023
1 parent 834298c commit a705225
Showing 1 changed file with 2 additions and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -17,6 +17,7 @@ import { encode } from '@kbn/rison';
import React from 'react';
import { FormattedMessage } from '@kbn/i18n-react';
import { useApmPluginContext } from '../../../context/apm_plugin/use_apm_plugin_context';
import { ENVIRONMENT_ALL } from '../../../../common/environment_filter_values';

interface Props {
dismissCallout: () => void;
Expand Down Expand Up @@ -48,7 +49,7 @@ export function SloCallout({
type: 'sli.apm.transactionErrorRate',
params: {
service: serviceName,
environment,
environment: environment === ENVIRONMENT_ALL.value ? '*' : environment,
transactionName: transactionName ?? '',
transactionType: transactionType ?? '',
},
Expand Down

0 comments on commit a705225

Please sign in to comment.