-
Notifications
You must be signed in to change notification settings - Fork 632
Closed
Labels
triagePending triagePending triage
Description
Is there an existing issue already for this bug?
- I have searched for an existing issue, and could not find anything. I believe this is a new bug.
I have read the troubleshooting guide
- I have read the troubleshooting guide and I think this is a new bug.
I am running a supported version of CloudNativePG
- I have read the troubleshooting guide and I think this is a new bug.
Contact Details
No response
Version
1.24.0
What version of Kubernetes are you using?
1.30
What is your Kubernetes environment?
Cloud: Google GKE
How did you install the operator?
YAML manifest
What happened?
- [Bug]: Prometheus v2.52+ fails to scrape metrics: Error on ingesting samples with different value but same timestamp #6108 , decide to set
monitoring.enablePodMonitor: false. In the same commit, create aPodMonitorto be applied manually, with the same name (i.e. the cluster name) and namespace. - Apply both changes simultaneously
- Wonder why I don't have a
PodMonitor - Go spelunking in the code: gets a reference to the
client.ObjectKeyFromObject(expectedPodMonitor), PodMonitorbased on namespace and name only, not looking at the owner reference. Thereforeindeed finds thecase !manager.IsPodMonitorEnabled() && podMonitor != nil: PodMonitor(that it doesn't have an owner reference on) and deletes it.
I worked around the issue by adding a short suffix to the PodMonitor name so that it wouldn't be deleted, but this isn't ideal because it changes the job label that Prometheus attaches to it.
Cluster resource
N/ARelevant log output
N/ACode of Conduct
- I agree to follow this project's Code of Conduct
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
triagePending triagePending triage