We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
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
-config.file
values.yaml
It is extremely confusing what config is relevant in the Helm chart today.
The config in the Helm chart today is for a dependency "Phlare", and can seem like it is part of the Pyroscope configuration.
pyroscope/operations/pyroscope/helm/pyroscope/values.yaml
Lines 155 to 169 in bbe05cc
But when you look closer, this is not the Pyroscope configuration file. The example uses:
storage: backend: s3 s3: endpoint: "minio:9000"
But Pyroscope uses:
s3_storage_backend: endpoint: "minio:9000"
This means that most of the available configuration parameters are left to be specified in the extraArgs section as CLI params.
extraArgs
There should be a structured Pyroscope configuration that aligns with the Pyroscope documentation in the Helm chart values.
The text was updated successfully, but these errors were encountered:
No branches or pull requests
Is your feature request related to a problem? Please describe.
It is extremely confusing what config is relevant in the Helm chart today.
The config in the Helm chart today is for a dependency "Phlare", and can seem like it is part of the Pyroscope configuration.
pyroscope/operations/pyroscope/helm/pyroscope/values.yaml
Lines 155 to 169 in bbe05cc
But when you look closer, this is not the Pyroscope configuration file. The example uses:
But Pyroscope uses:
This means that most of the available configuration parameters are left to be specified in the
extraArgs
section as CLI params.Describe the solution you'd like
There should be a structured Pyroscope configuration that aligns with the Pyroscope documentation in the Helm chart values.
The text was updated successfully, but these errors were encountered: