-
Notifications
You must be signed in to change notification settings - Fork 5
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
Use service monitor #562
Use service monitor #562
Conversation
c3245d9
to
36063f4
Compare
99dddf2
to
08743fb
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Your PR says it makes use of a PodMonitor while you're actually using a ServiceMonitor? IMHO we can drop the service and really just use a PodMonitor as you proposed for the etcd-kubernetes-resources-count-exporter
.
I renamed thé PR sorry. I tried to use a podmonitor first but it does not work because this pod is using hostnetwork:true so kyverno is complaining when I add the ports information on thé deployment |
But that's probably more an issue because of not using CiliumNetworkPolicies, right? Like, plain NetworkPolicies can not be used for allowing traffic to host network. |
Hmm thé issue I had was really related to kyverno policy exceptions but you're free to try a Pod monitor instead :) I don't have the bandwith to do them all "properly", I really just need to get rid of the old annotations asap:) |
selector: | ||
{{- include "labels.selector" . | nindent 4 }} | ||
{{- include "labels.selector" . | nindent 4 }} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
{{- include "labels.selector" . | nindent 4 }} | |
{{- include "labels.selector" . | nindent 4 }} |
Adding the missing/removed newline at EOF.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ah, one more: Then please do not change the values (replace service.port
by just port
) if you keep using a ServiceMonitor.
Sure :) |
It's fixed @Gacko |
This configures the operator to be scraped using a service monitor instead of the old scheme