-
Notifications
You must be signed in to change notification settings - Fork 96
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
Replace BEYLA_OTEL_USE_EXPONENTIAL_HISTOGRAMS by standard env var #635
Conversation
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## main #635 +/- ##
===========================================
- Coverage 78.95% 40.47% -38.48%
===========================================
Files 89 86 -3
Lines 7474 7288 -186
===========================================
- Hits 5901 2950 -2951
- Misses 1252 4164 +2912
+ Partials 321 174 -147
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM! Thanks Mario!
* test * chart.yaml * service-account * after helm create * adding rbac with changes for labels and annotations Signed-off-by: Syed Nihal <syed.nihal@nokia.com> * adding rbac with changes for labels and annotations Signed-off-by: Syed Nihal <syed.nihal@nokia.com> * adding rbac with changes for labels and annotations Signed-off-by: Syed Nihal <syed.nihal@nokia.com> * adding rbac with changes for labels and annotations Signed-off-by: Syed Nihal <syed.nihal@nokia.com> * test chart.yaml * service-account * after helm create adding rbac with changes for labels and annotations Signed-off-by: Syed Nihal <syed.nihal@nokia.com> adding rbac with changes for labels and annotations Signed-off-by: Syed Nihal <syed.nihal@nokia.com> adding rbac with changes for labels and annotations Signed-off-by: Syed Nihal <syed.nihal@nokia.com> * adding rbac with changes for labels and annotations Signed-off-by: Syed Nihal <syed.nihal@nokia.com> * Replace BEYLA_OTEL_USE_EXPONENTIAL_HISTOGRAMS by standard environment variable (#635) * Automatic update of offsets.json (#639) Co-authored-by: MattFrick <MattFrick@users.noreply.github.com> * Fix missing go_trace_map cleanup (#638) * add release namespace * Update cluster-role.yaml * Update cluster-role-binding.yaml * values.yaml * configmap and env * secret ref * values.yaml * Update configmap.yaml * Update configmap.yaml * Update values.yaml * Update configmap.yaml * values.yaml * configmap and daemonset changes * 101 * changed existing configmap * clusterolebinding * Update cluster-role-binding.yaml * added comments for external configmap * adding k8s service in helm chart to expose internal beyla metrics. Signed-off-by: Syed Nihal <syed.nihal@nokia.com> * affinity, updatestrategy, tolerations ... * Update values.yaml * Delete deployments/helm/templates/tests/test-connection.yaml * Update values.yaml * Update values.yaml * Delete deployments/helm/templates/NOTES.txt * add readme * remove autoscaling * remove autoscaling * remove ingress, update configmap * remove all ingress changes * addressed review comments Signed-off-by: Syed Nihal <syed.nihal@nokia.com> --------- Signed-off-by: Syed Nihal <syed.nihal@nokia.com> Co-authored-by: Syed Nihal <syed.nihal@nokia.com> Co-authored-by: Mario Macias <mario.macias@grafana.com> Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com> Co-authored-by: MattFrick <MattFrick@users.noreply.github.com> Co-authored-by: Nikola Grcevski <6207777+grcevski@users.noreply.github.com> Co-authored-by: Nihal <38865967+wasim-nihal@users.noreply.github.com>
Before releasing Beyla 1.3, I'm changing this environment variable by the standard OTEL variable, so we avoid having to later support and document two environment variables.