feat: introduce serviceMonitor and labels to services#6340
feat: introduce serviceMonitor and labels to services#6340oilbeater merged 1 commit intokubeovn:masterfrom
Conversation
Summary of ChangesHello @oujonny, I'm Gemini Code Assist1! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed! This pull request significantly enhances the observability of Kube-OVN services by integrating them with Prometheus through the introduction of ServiceMonitor resources. It also improves Kubernetes resource management by applying consistent, standardized labels to the core Kube-OVN services, making them easier to identify and manage within a Kubernetes cluster. Highlights
Changelog
Activity
Using Gemini Code AssistThe full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips. Invoking Gemini You can request assistance from Gemini at any point by creating a comment using either
Customization To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a Limitations & Feedback Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for Github and other Google products, sign up here. You can also get AI-powered code generation, chat, as well as code reviews directly in the IDE at no cost with the Gemini Code Assist IDE Extension. Footnotes
|
Signed-off-by: oujonny <jonny@immerda.ch>
869a182 to
a3a8709
Compare
There was a problem hiding this comment.
Code Review
This pull request introduces ServiceMonitor resources for various Kube-OVN components and adds standard labels to the corresponding services. The changes are well-structured and follow Helm best practices. However, I've found a critical issue in the label selectors for the controller, monitor, and pinger ServiceMonitor resources. They use an incorrect label key, which will prevent them from discovering their target services. I've left specific comments with suggestions for fixing this. Once these are addressed, the PR should be good to go.
Signed-off-by: oujonny <jonny@immerda.ch> (cherry picked from commit ca8b1c8)
Signed-off-by: oujonny <jonny@immerda.ch> (cherry picked from commit ca8b1c8)
|
@oujonny I have backport it to release-1.15 and release-1.14 |
|
Awsome! Thanks for the nice collaborations:) Do you plan to release a v1.14.x version with the chart updates and the live-migration fixes? |
|
@oujonny v1.14.31 includes these patches. |
Pull Request
What type of this PR
Examples of user facing changes:
This PR introduces serviceMonitors to the v2 helm chart and sets proper labels for the services.
We would love to see backports into v1.14.x and v1.15.x, but we can also easy do it just on our side.
Which issue(s) this PR fixes
Fixes #6339