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

Adding new attribute ReplicaFailure to deployment workload #725

Merged
merged 3 commits into from
Apr 25, 2023

Conversation

juanjjaramillo
Copy link
Contributor

@juanjjaramillo juanjjaramillo commented Apr 21, 2023

Description

The Kubernetes team is updating metrics for all supported workloads and services. Specifically, kube-state-metrics has upgraded several metrics and services from experimental to stable.

We noted in the Kubernetes repository that there are only three valid conditions of a deployment:

  • Available
  • Progressing
  • ReplicaFailure

While they are briefly mentioned in the documentation, there is no statement indicating they are the only valid conditions.

PR Changes

This PR makes changes to the agent to start scraping the attribute ReplicaFailure from KSM's metric kube_deployment_status_condition to make sure we scrape all valid conditions for Deployment workloads.

Related PRs

nri-kubernetes #725
infra-integration-specs #313
infra-metrics-utils #85
dirac #8087
dirac_async_query_service #3347
window-aggregation-merger #811
partial-aggregation-windower #529

Testing Plan

  • Run local testing environment and check metrics are ingested to NR1 in the respective K8s*Sample tables

@juanjjaramillo juanjjaramillo force-pushed the juanjjaramillo/new_deployment_attribute branch from 9d0d820 to 2e1967a Compare April 21, 2023 16:25
@juanjjaramillo juanjjaramillo force-pushed the juanjjaramillo/new_deployment_attribute branch from 8f09d67 to 80ec84a Compare April 25, 2023 19:36
@juanjjaramillo juanjjaramillo marked this pull request as ready for review April 25, 2023 20:16
@juanjjaramillo juanjjaramillo requested a review from a team as a code owner April 25, 2023 20:16
e2e/k8s.yml Show resolved Hide resolved
@juanjjaramillo juanjjaramillo merged commit a9a3e4b into main Apr 25, 2023
18 checks passed
@juanjjaramillo juanjjaramillo deleted the juanjjaramillo/new_deployment_attribute branch April 25, 2023 22:35
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.

None yet

4 participants