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

[backport v1.0] [BUG] VMIs created from VM Template don't have LiveMigrate evictionStrategy set #2414

Closed
harvesterhci-io-github-bot opened this issue Jun 23, 2022 · 2 comments
Assignees
Labels
area/ui kind/bug Issues that are defects reported by users or that we know have reached a real release not-require/test-plan Skip to create a e2e automation test issue priority/0 Must be fixed in this release require-ui/small estimate 1-2 working days
Milestone

Comments

@harvesterhci-io-github-bot

backport the issue #2357

@harvesterhci-io-github-bot harvesterhci-io-github-bot added area/ui kind/bug Issues that are defects reported by users or that we know have reached a real release not-require/test-plan Skip to create a e2e automation test issue priority/0 Must be fixed in this release require-ui/small estimate 1-2 working days labels Jun 23, 2022
@harvesterhci-io-github-bot harvesterhci-io-github-bot added this to the v1.0.3 milestone Jun 23, 2022
@harvesterhci-io-github-bot
Copy link
Author

harvesterhci-io-github-bot commented Jun 24, 2022

Pre Ready-For-Testing Checklist

* [ ] If labeled: require/HEP Has the Harvester Enhancement Proposal PR submitted?
The HEP PR is at:

* [ ] Is there a workaround for the issue? If so, where is it documented?
The workaround is at:

* [ ] Have the backend code been merged (harvester, harvester-installer, etc) (including backport-needed/*)?
The PR is at:

~* [ ] Does the PR include the explanation for the fix or the feature? ~

~ * [ ] Does the PR include deployment change (YAML/Chart)? If so, where are the PRs for both YAML file and Chart?
The PR for the YAML change is at:
The PR for the chart change is at:~

* [ ] If labeled: require/doc, require/knowledge-base Has the necessary document PR submitted or merged?
The documentation/KB PR is at:

* [ ] If NOT labeled: not-require/test-plan Has the e2e test plan been merged? Have QAs agreed on the automation test case? If only test case skeleton w/o implementation, have you created an implementation issue?
- The automation skeleton PR is at:
- The automation test case PR is at:

* [ ] If the fix introduces the code for backward compatibility Has a separate issue been filed with the label release/obsolete-compatibility?
The compatibility issue is filed at:

@lanfon72
Copy link
Member

Verified this bug has been fixed.

Test Information

Verify Steps:

  1. Install Harvester with at least 2 nodes
  2. Navigate to Advanced/Settings and update ui-index to https://releases.rancher.com/harvester-ui/dashboard/release-harvester-v1.0/index.html and force refresh to make it applied.
  3. Create Image for VM Creation
  4. Navigate to Advanced/Templates and create a template t1
  5. Create VM vm1 from template t1
  6. Edit YAML of vm1, field spec.template.spec.evictionStrategy should be LiveMigrate
  7. Enable Maintenance Mode on the host which hosting vm1
  8. vm1 should start migrating automatically
  9. Migration should success

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/ui kind/bug Issues that are defects reported by users or that we know have reached a real release not-require/test-plan Skip to create a e2e automation test issue priority/0 Must be fixed in this release require-ui/small estimate 1-2 working days
Projects
None yet
Development

No branches or pull requests

3 participants