feat: add pod storage metrics #1047
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
petewall
approved these changes
Jan 2, 2025
bentonam
pushed a commit
that referenced
this pull request
Jan 6, 2025
* feat: add pod storage metrics * build: make clean build * fix: tests
bentonam
added a commit
that referenced
this pull request
Jan 6, 2025
* Added Grafana Integration * Updated Common Relabelings * Updated Loki Integration and Added Tests * Updated Build Dependencies * Updated deprecated stdlib env() w/ sys.env (#1027) * Fixed deprecated stdlib env() with sys.env() Resolves #1026 * Updated Build Dependencies and Artifacts * Rebuilt * Fix the check if the filters are enabled Signed-off-by: Pete Wall <[email protected]> * Fix the type of endpoint params (#1029) Signed-off-by: Pete Wall <[email protected]> * Bump version to 2.0.0-rc.11 Also fix k8s-monitoring-test branch Signed-off-by: Pete Wall <[email protected]> * fix input for filter and memory limiter processors (#1037) Signed-off-by: Pete Wall <[email protected]> * Bump version to 2.0.0-rc.12 Signed-off-by: Pete Wall <[email protected]> * chore(dev-deps): bump markdownlint-cli2 from 0.16.0 to 0.17.0 (#1041) Bumps [markdownlint-cli2](https://github.com/DavidAnson/markdownlint-cli2) from 0.16.0 to 0.17.0. - [Changelog](https://github.com/DavidAnson/markdownlint-cli2/blob/main/CHANGELOG.md) - [Commits](DavidAnson/markdownlint-cli2@v0.16.0...v0.17.0) --- updated-dependencies: - dependency-name: markdownlint-cli2 dependency-type: direct:development update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * Update KSM, Node Exporter, and OpenCost (#1045) * Update KSM, Node Exporter, and OpenCost Signed-off-by: Pete Wall <[email protected]> * fix deterministic ordering of the mysql integration Signed-off-by: Pete Wall <[email protected]> --------- Signed-off-by: Pete Wall <[email protected]> * feat: add pod storage metrics (#1047) * feat: add pod storage metrics * build: make clean build * fix: tests * Update example docs (#1023) * Update example docs Signed-off-by: Pete Wall <[email protected]> * Update charts/k8s-monitoring/docs/examples/platforms/openshift/README.md Co-authored-by: Beverly Buchanan <[email protected]> * Fix examples Signed-off-by: Pete Wall <[email protected]> --------- Signed-off-by: Pete Wall <[email protected]> Co-authored-by: Beverly Buchanan <[email protected]> * Move the feature charts as directories inside of the k8s-monitoring chart (#1046) * Move the feature charts as directories inside of the k8s-monitoring chart. Signed-off-by: Pete Wall <[email protected]> * Fix changed files and tests * remove extra subcharts that were added during a rebase Signed-off-by: Pete Wall <[email protected]> * Use local helm-schema-gen if it's available Signed-off-by: Pete Wall <[email protected]> --------- Signed-off-by: Pete Wall <[email protected]> * Fix pod annotation duplication (#1050) * Add DPM checks for control plane integration test and add a test for annotation autodiscovery Signed-off-by: Pete Wall <[email protected]> * Drop the __tmp_port label, which breaks autodiscovery deduplication Signed-off-by: Pete Wall <[email protected]> --------- Signed-off-by: Pete Wall <[email protected]> * Bump versions to 1.6.17 and 2.0.0-rc.13 Signed-off-by: Pete Wall <[email protected]> * chore(dev-deps): bump textlint from 14.4.0 to 14.4.2 (#1051) Bumps [textlint](https://github.com/textlint/textlint) from 14.4.0 to 14.4.2. - [Release notes](https://github.com/textlint/textlint/releases) - [Commits](textlint/textlint@v14.4.0...v14.4.2) --- updated-dependencies: - dependency-name: textlint dependency-type: direct:development update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * chore(dev-deps): bump markdownlint-cli2 from 0.17.0 to 0.17.1 (#1052) Bumps [markdownlint-cli2](https://github.com/DavidAnson/markdownlint-cli2) from 0.17.0 to 0.17.1. - [Changelog](https://github.com/DavidAnson/markdownlint-cli2/blob/main/CHANGELOG.md) - [Commits](DavidAnson/markdownlint-cli2@v0.17.0...v0.17.1) --- updated-dependencies: - dependency-name: markdownlint-cli2 dependency-type: direct:development update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * Updated Loki Integration and Added Tests * Updated Build Dependencies * Added Grafana Integration * Updated Meta-Monitoring * Updated Loki Tests * Updated ls command to use no color * Rebuilt * Sort the valueList * Updated Tests * Fixed typo and Rebuilt * Changed ls command to never use color * Regenerated Output * Updated vendir alloyModules * Updated stdlib join to be string.join * Rebuilt * Rebuilt * Removed Unused Default Allow List --------- Signed-off-by: Pete Wall <[email protected]> Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: Pete Wall <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Stephen Lang <[email protected]> Co-authored-by: Beverly Buchanan <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This PR adds metrics to the default allow lists for Kubelet and KSM required for pod storage observability. Also make the v1/v2 lists consistent with each other.