Skip to content

Commit

Permalink
Fix Kubernetes360 filter for cAdvisor metrics in standalone (#446)
Browse files Browse the repository at this point in the history
- Added the `logzio_app` label to standalone collector so filter won't drop the cAdvisor metrics.
  • Loading branch information
ralongit authored Apr 1, 2024
1 parent ad3ba3a commit 1ef591c
Show file tree
Hide file tree
Showing 5 changed files with 11 additions and 7 deletions.
8 changes: 2 additions & 6 deletions charts/logzio-monitoring/Chart.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@ apiVersion: v2
name: logzio-monitoring
description: logzio-monitoring allows you to ship logs, metrics, traces and security reports from your Kubernetes cluster using the OpenTelemetry collector for metrics and traces, Fluentd for logs, and Trivy for security reports.
type: application
version: 5.2.0
version: 5.2.1

sources:
- https://github.com/logzio/logzio-helm
Expand All @@ -12,7 +12,7 @@ dependencies:
repository: "https://logzio.github.io/logzio-helm/"
condition: logs.enabled
- name: logzio-k8s-telemetry
version: "4.1.0"
version: "4.1.1"
repository: "https://logzio.github.io/logzio-helm/"
condition: metricsOrTraces.enabled
- name: logzio-trivy
Expand All @@ -29,11 +29,7 @@ dependencies:
condition: deployEvents.enabled

maintainers:
- name: tamirmich
email: tamir.michaeli@logz.io
- name: yotamloe
email: yotam.loewenbach@logz.io
- name: miriig
email: miri.ignatiev@logz.io
- name: ralongit
email: raul.gurshumo@logz.io
3 changes: 3 additions & 0 deletions charts/logzio-monitoring/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -206,6 +206,9 @@ There are two possible approaches to the upgrade you can choose from:
## Changelog
- **5.2.0**:
- Upgrade `logzio-k8s-telemetry` to `4.1.1`:
- Fixed bug with cAdvisor metrics filter.
- **5.2.0**:
- Upgrade `logzio-k8s-telemetry` to `4.1.0`:
- Upgraded prometheus-node-exporter version to `4.29.0`
Expand Down
2 changes: 1 addition & 1 deletion charts/logzio-telemetry/Chart.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -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.1.0
version: 4.1.1

# 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
2 changes: 2 additions & 0 deletions charts/logzio-telemetry/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -394,6 +394,8 @@ If you don't want the sub charts to installed add the relevant flag per sub char


## Change log
* 4.1.1
- Fixed bug with cAdvisor metrics filter.
* 4.1.0
- Upgraded prometheus-node-exporter version to `4.29.0`
- Fixed bug with AKS metrics filter
Expand Down
3 changes: 3 additions & 0 deletions charts/logzio-telemetry/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -399,6 +399,9 @@ metricsConfig:
relabel_configs:
- action: labelmap
regex: __meta_kubernetes_node_label_(.+)
- action: replace
replacement: kubernetes360
target_label: logzio_app
metric_relabel_configs: []
prometheus/collector:
config:
Expand Down

0 comments on commit 1ef591c

Please sign in to comment.