Skip to content

Commit

Permalink
Upgrade logzio-telemetry chart to v4.1.0 (#438)
Browse files Browse the repository at this point in the history
* Fix bug with AKS K8S 360 metrics filter (#436)

* Remove unified_status_code label from SPM (#437)

* updated prometheus-node-exporter (#435)

* Update changelog

---------

Co-authored-by: Bar <51476543+bardabun@users.noreply.github.com>
  • Loading branch information
ralongit and bardabun authored Feb 20, 2024
1 parent ff2a781 commit 4a5d463
Show file tree
Hide file tree
Showing 4 changed files with 9 additions and 13 deletions.
4 changes: 2 additions & 2 deletions charts/logzio-telemetry/Chart.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@ dependencies:
condition: kubeStateMetrics.enabled

- name: prometheus-node-exporter
version: "4.23.2"
version: "4.29.0"
repository: "https://prometheus-community.github.io/helm-charts"
condition: nodeExporter.enabled

Expand All @@ -24,7 +24,7 @@ dependencies:
# This is the chart version. This version number should be incremented each time you make changes
# to the chart and its templates, including the app version.
# Versions are expected to follow Semantic Versioning (https://semver.org/)
version: 4.0.0
version: 4.1.0

# This is the version number of the application being deployed. This version number should be
# incremented each time you make changes to the application. Versions are not expected to
Expand Down
4 changes: 4 additions & 0 deletions charts/logzio-telemetry/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -394,6 +394,10 @@ If you don't want the sub charts to installed add the relevant flag per sub char


## Change log
* 4.1.0
- Upgraded prometheus-node-exporter version to `4.29.0`
- Fixed bug with AKS metrics filter
- Remove unified_status_code label from SPM
* 4.0.0
- **BREAKING CHANGES**:
- Removed the `kubernetes-360-metrics` key from the `logzio-secret`.
Expand Down
2 changes: 1 addition & 1 deletion charts/logzio-telemetry/templates/_helpers.tpl
Original file line number Diff line number Diff line change
Expand Up @@ -137,7 +137,7 @@ If any OOB filters is being used the function return the OOB filter concatenated
{{- define "opentelemetry-collector.k360Metrics" -}}
{{- $metrics := "" }}
{{- if .Values.enableMetricsFilter.aks }}
{{ $metrics = .Values.prometheusFilters.metrics.infrastructure.keep.aks }}
{{- $metrics = .Values.prometheusFilters.metrics.infrastructure.keep.aks }}
{{- else if .Values.enableMetricsFilter.gke}}
{{- $metrics = .Values.prometheusFilters.metrics.infrastructure.keep.gke }}
{{- else }}
Expand Down
12 changes: 2 additions & 10 deletions charts/logzio-telemetry/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -627,15 +627,7 @@ spanMetricsAgregator:
operations:
- action: update_label
label: span.name
new_label: operation
attributes/spm:
actions:
- action: upsert
from_attribute: rpc.grpc.status_code
key: unified_status_code
- action: upsert
from_attribute: http.status_code
key: unified_status_code
new_label: operation
connectors:
spanmetrics:
histogram:
Expand Down Expand Up @@ -714,7 +706,7 @@ spanMetricsAgregator:
exporters: [prometheus/spm]
metrics/spm-logzio:
receivers: [prometheus/spm-logzio, spanmetrics]
processors: [metricstransform,attributes/spm]
processors: [metricstransform]
exporters: [prometheusremotewrite/spm-logzio]
# service values
service:
Expand Down

0 comments on commit 4a5d463

Please sign in to comment.