Traces: Remove serviceMap feature flag (#52375)

* Remove serviceMap feature flag

* Remove import

* Add serviceMap to queryTypeOptions
pull/51797/head^2
Joey Tawadrous 3 years ago committed by GitHub
parent 46eec85b1a
commit 11b743ecd9
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
  1. 1
      packages/grafana-data/src/types/featureToggles.gen.ts
  2. 6
      pkg/services/featuremgmt/registry.go
  3. 4
      pkg/services/featuremgmt/toggles_gen.go
  4. 12
      public/app/plugins/datasource/tempo/QueryEditor/QueryField.tsx
  5. 8
      public/app/plugins/datasource/tempo/configuration/ConfigEditor.tsx

@ -26,7 +26,6 @@ export interface FeatureToggles {
['live-service-web-worker']?: boolean;
queryOverLive?: boolean;
panelTitleSearch?: boolean;
tempoServiceGraph?: boolean;
tempoApmTable?: boolean;
prometheus_azure_auth?: boolean;
prometheusAzureOverrideAudience?: boolean;

@ -62,12 +62,6 @@ var (
Description: "Search for dashboards using panel title",
State: FeatureStateAlpha,
},
{
Name: "tempoServiceGraph",
Description: "show service",
State: FeatureStateBeta,
FrontendOnly: true,
},
{
Name: "tempoApmTable",
Description: "Show APM table",

@ -47,10 +47,6 @@ const (
// Search for dashboards using panel title
FlagPanelTitleSearch = "panelTitleSearch"
// FlagTempoServiceGraph
// show service
FlagTempoServiceGraph = "tempoServiceGraph"
// FlagTempoApmTable
// Show APM table
FlagTempoApmTable = "tempoApmTable"

@ -3,7 +3,7 @@ import React from 'react';
import useAsync from 'react-use/lib/useAsync';
import { QueryEditorProps, SelectableValue } from '@grafana/data';
import { config, reportInteraction } from '@grafana/runtime';
import { reportInteraction } from '@grafana/runtime';
import {
FileDropzone,
InlineField,
@ -75,14 +75,12 @@ class TempoQueryFieldComponent extends React.PureComponent<Props> {
const queryTypeOptions: Array<SelectableValue<TempoQueryType>> = [
{ value: 'traceId', label: 'TraceID' },
{ value: 'upload', label: 'JSON file' },
{ value: 'serviceMap', label: 'Service Graph' },
];
if (config.featureToggles.tempoServiceGraph) {
queryTypeOptions.push({ value: 'serviceMap', label: 'Service Graph' });
// span names in Tempo search links (generated on the service graph page) are in camel case (for Prometheus queries)
// but the span name dropdown menu in the search tab is lower case
query.spanName = query.spanName?.toLowerCase();
}
// span names in Tempo search links (generated on the service graph page) are in camel case (for Prometheus queries)
// but the span name dropdown menu in the search tab is lower case
query.spanName = query.spanName?.toLowerCase();
if (!datasource?.search?.hide) {
queryTypeOptions.unshift({ value: 'nativeSearch', label: 'Search' });

@ -34,11 +34,9 @@ export const ConfigEditor: React.FC<Props> = ({ options, onOptionsChange }) => {
</div>
) : null}
{config.featureToggles.tempoServiceGraph && (
<div className="gf-form-group">
<ServiceGraphSettings options={options} onOptionsChange={onOptionsChange} />
</div>
)}
<div className="gf-form-group">
<ServiceGraphSettings options={options} onOptionsChange={onOptionsChange} />
</div>
<div className="gf-form-group">
<SearchSettings options={options} onOptionsChange={onOptionsChange} />

Loading…
Cancel
Save