kube-state-metrics icon indicating copy to clipboard operation
kube-state-metrics copied to clipboard

Presubmit checks for stable metrics

Open CatherineF-dev opened this issue 3 years ago • 11 comments

What would you like to be added: A presubmit to validate stable metrics don't change metric name, add labels or delete labels.

Similar to k8s stable metrics presubmit.

Why is this needed: Guarantee stable metrics are stable.

Due to lack of presubmit check, some stable metrics are changed. For example,

  • stable metric kube_persistentvolume_info is changed in https://github.com/kubernetes/kube-state-metrics/pull/1814
  • stable metric kube_service_status_load_balancer_ingress is changed in https://github.com/kubernetes/kube-state-metrics/pull/1687

Describe the solution you'd like

  1. Import k8s metrics stability framework https://github.com/kubernetes/kube-state-metrics/pull/1844 and https://github.com/kubernetes/kube-state-metrics/pull/1976
  2. Presubmit check https://github.com/kubernetes/kube-state-metrics/pull/1836

Additional context

CatherineF-dev avatar Sep 14 '22 00:09 CatherineF-dev

/open

CatherineF-dev avatar Oct 14 '22 12:10 CatherineF-dev

/reopen

CatherineF-dev avatar Feb 06 '23 19:02 CatherineF-dev

@CatherineF-dev: Reopened this issue.

In response to this:

/reopen

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

k8s-ci-robot avatar Feb 06 '23 19:02 k8s-ci-robot

/reopen

CatherineF-dev avatar Feb 07 '23 16:02 CatherineF-dev

@CatherineF-dev: Reopened this issue.

In response to this:

/reopen

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

k8s-ci-robot avatar Feb 07 '23 16:02 k8s-ci-robot

/triage accepted /assign @CatherineF-dev

logicalhan avatar Feb 09 '23 17:02 logicalhan

This issue has not been updated in over 1 year, and should be re-triaged.

You can:

  • Confirm that this issue is still relevant with /triage accepted (org members only)
  • Close this issue with /close

For more details on the triage process, see https://www.kubernetes.dev/docs/guide/issue-triage/

/remove-triage accepted

k8s-triage-robot avatar Feb 09 '24 18:02 k8s-triage-robot

/triage accepted

CatherineF-dev avatar Feb 09 '24 19:02 CatherineF-dev