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

[regression] /metrics port down when not existing CRD are listed in config file #2354

Open
bmm-alc opened this issue Mar 29, 2024 · 5 comments
Labels
kind/bug Categorizes issue or PR as related to a bug. needs-triage Indicates an issue or PR lacks a `triage/foo` label and requires one.

Comments

@bmm-alc
Copy link

bmm-alc commented Mar 29, 2024

What happened:
we use https://github.com/fluxcd/flux2-monitoring-example/blob/bcae1ef44963da89caabe962a93280c0dfad3381/monitoring/controllers/kube-prometheus-stack/kube-state-metrics-config.yaml as part of our config for FluxCD, but we didn´t deploy resourcesimagepolicies and imageupdateautomations from apiGroup image.toolkit.fluxcd.io (they are optional and we don't use them).
Before 2.11.0 release it would not cause any problem, except recurring warning messages.

since version 2.11.0, as long as theses resources are listed, the '/metrics' is down from prometheus POV
removing them, brought back the endpoint up in prometheus.

What you expected to happen:
/metrics to be up and running

How to reproduce it (as minimally and precisely as possible):
declaring not deployed resources should trigger the issue.

Anything else we need to know?:

Environment:

  • kube-state-metrics version: 2.11.0
  • Kubernetes version (use kubectl version): 1.29.1
  • Cloud provider or hardware configuration: scaleway
@bmm-alc bmm-alc added the kind/bug Categorizes issue or PR as related to a bug. label Mar 29, 2024
@k8s-ci-robot k8s-ci-robot added the needs-triage Indicates an issue or PR lacks a `triage/foo` label and requires one. label Mar 29, 2024
@k8s-ci-robot
Copy link
Contributor

This issue is currently awaiting triage.

If kube-state-metrics contributors determine this is a relevant issue, they will accept it by applying the triage/accepted label and provide further guidance.

The triage/accepted label can be added by org members by writing /triage accepted in a comment.

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.

@CatherineF-dev
Copy link
Contributor

Could you provide steps to reproduce this issue?

@bmm-alc
Copy link
Author

bmm-alc commented Apr 8, 2024

closing because I don't have time to investigate this

@bmm-alc bmm-alc closed this as not planned Won't fix, can't repro, duplicate, stale Apr 8, 2024
@CatherineF-dev
Copy link
Contributor

Okay, we can keep it open and wait others to provide steps to reproduce. Feel it might be an issue.

/reopen

@k8s-ci-robot
Copy link
Contributor

@CatherineF-dev: Reopened this issue.

In response to this:

Okay, we can keep it open and wait others to provide steps to reproduce. Feel it might be an issue.

/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 k8s-ci-robot reopened this Apr 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/bug Categorizes issue or PR as related to a bug. needs-triage Indicates an issue or PR lacks a `triage/foo` label and requires one.
Projects
None yet
Development

No branches or pull requests

3 participants