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

Add vm migration Grafana dashboard (backport #953) #958

Merged
merged 1 commit into from
Feb 12, 2025

Conversation

mergify[bot]
Copy link

@mergify mergify bot commented Feb 11, 2025

Problem:

Solution:

Related Issue:
related to harvester/harvester#4352
upgrade path pr: harvester/harvester#7551

Test plan:

  • Prepare a 3 node harvester cluster
  • Enable rancher-monitoring addon
  • Migration normally goes fast, so please restrict migration bandwidth to 2Mi first
    kubectl patch kubevirt kubevirt -n harvester-system --type merge -p '{"spec":{"configuration":{"migrations":{"bandwidthPerMigration":"2Mi"}}}}'
    
  • Create 2 vm, test, test2 (2 cpu, 2GB ram for each)
  • Migrate vm test to another node.
  • On VM test2, install stress and run a memory stress test to verify that the dirty memory rate is nonzero. Then, migrate VM test2 to another node.
sudo apt install stress
stress --vm 2 --vm-bytes 100M --timeout 86400s
  • Go to grafana “Harvester VM Migration Details” dashboard
    • click Add-ons > rancher-monitoring > Grafana > click Grafana icon > Dashboards > Harvester VM Migration Details

Here is the sample screenshot:

vm test migration
image

vm test2 migration
image


This is an automatic backport of pull request #953 done by [Mergify](https://mergify.com).

Copy link
Member

@w13915984028 w13915984028 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thanks.

@ihcsim
Copy link
Contributor

ihcsim commented Feb 11, 2025

Per slack chat, rebasing this on top of #957 should fix the CI error.

Signed-off-by: Cooper Tseng <cooper.tseng@suse.com>
(cherry picked from commit abed4ed)
@brandboat brandboat force-pushed the mergify/bp/v1.5/pr-953 branch from 35c86ad to 007396d Compare February 12, 2025 05:30
@brandboat brandboat merged commit ce0bd4b into v1.5 Feb 12, 2025
7 checks passed
@brandboat brandboat deleted the mergify/bp/v1.5/pr-953 branch February 12, 2025 07:45
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.

3 participants