Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Tempo-distributed helm values typo in comment #3514

Open
houserx-jmcc opened this issue Jan 7, 2025 · 0 comments
Open

Tempo-distributed helm values typo in comment #3514

houserx-jmcc opened this issue Jan 7, 2025 · 0 comments

Comments

@houserx-jmcc
Copy link

houserx-jmcc commented Jan 7, 2025

After struggling through problems stemming from #2802, I believe there was a mistake in one of the final 2 PRs from that issue:

# -- For processors to be enabled and generate metrics, pass the names of the processors to `global_overrides.defaults.metrics_generator_processors` value like `[service-graphs, span-metrics]`.

Indicates this structure, which doesn't work (crash loops):

global_overrides:
  defaults:
    metrics_generator_processors: ['service-graphs', 'span-metrics']

but shinebayar-g's comment at the end does work:

global_overrides:
    defaults:
        metrics_generator:
            processors: ['service-graphs', 'span-metrics']
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant