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

feat(monitor): support to monitor vm and expose influxdb svc as nodeport type #2116

Merged
merged 7 commits into from
Nov 14, 2022
Merged

feat(monitor): support to monitor vm and expose influxdb svc as nodeport type #2116

merged 7 commits into from
Nov 14, 2022

Conversation

willzgli
Copy link
Collaborator

support to collect vm metrics from libvirt level

Signed-off-by: willzgli [email protected]

What type of PR is this?

Uncomment only one /kind <> line, hit enter to put that in a new line, and remove leading whitespace from that line:

/kind api-change
/kind bug
/kind cleanup
/kind deprecation
/kind design
/kind documentation
/kind failing-test
/kind feature
/kind flake

What this PR does / why we need it:

Which issue(s) this PR fixes:

Fixes #

Special notes for your reviewer:

Does this PR introduce a user-facing change?:


Additional documentation e.g., KEPs (Kubernetes Enhancement Proposals), usage docs, etc.:


@willzgli willzgli changed the title feat(monitor): support monitor kubevirt vms feat(monitor): support to monitor vm and expose influxdb svc as nodeport type Oct 18, 2022
willzgli added 7 commits November 11, 2022 18:19
support to collect vm metrics from libvirt level

Signed-off-by: willzgli <[email protected]>
1. fix alert rule expression for vm
2. add vm name in alert message
3. round values in prometheus alert annotation to two decimals

Signed-off-by: willzgli <[email protected]>
expose influxdb svc as nodeport in non-ha mode  global cluster
and ha mode global cluster. it will unify the remote writing
address for promethues in global cluster and business cluster.

Signed-off-by: willzgli <[email protected]>
@leoryu leoryu merged commit e1d681d into tkestack:master Nov 14, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants