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

fix: high cardinality policy_violations metrics #96

Open
wants to merge 8 commits into
base: main
Choose a base branch
from

Conversation

dwalker-sabiogroup
Copy link

  • fix: high cardinality policy_violations metrics

The exporter initialises all possible labels of dependency_track_project_policy_violations metrics which adds 72 metrics for each project.

#53 has details of the rational but there are usecases where it is desirable to only have non-zero values of the metrics available.

This change adds a new argument that enables only non-zero value metrics to be returned. This flag defaults to the current behaviour so is an opt in feature.

dwalker-sabiogroup and others added 8 commits August 3, 2024 13:21
* fix: high cardinality policy_violations metrics

The exporter initialises all possible labels of dependency_track_project_policy_violations
metrics which adds 72 metrics for each project.

jetstack#53 has details of the rational but there are usecases where
it is desirable to only have non-zero values of the metrics available.

This change adds a new argument that enables only non-zero value metrics to be returned.
This flag defaults to the current behaviour so is an opt in feature.

* docs: add flag to readme
* fix: imports

* fix: invalid goreleaser args

* fix: invalid goreleaser args

* fix: invalid goreleaser args
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

Successfully merging this pull request may close these issues.

1 participant