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

[EUWE] Fix Host getting disconnected from Cluster when migrating a VM in RHEV #13836

Conversation

agrare
Copy link
Member

@agrare agrare commented Feb 9, 2017

Euwe backport of #13815

When building a minimal host hash for migrated vm targeted refresh include the ems_cluster with the host so that it doesn't clear the ems_cluster_id when saving.

#13511 fixed saving a VM when it gets migrated to a new host but as a result the cluster link to that host got cleared when it was saved. This adds the cluster link to the minimal host hash so that it is saved properly.

https://bugzilla.redhat.com/show_bug.cgi?id=1420003

@agrare
Copy link
Member Author

agrare commented Feb 9, 2017

@borod108 please review

@agrare agrare force-pushed the euwe_bz_1420003_rhev_fix_missing_host_link_targeted_refresh branch from 1b893de to 50c3388 Compare February 9, 2017 14:04
@miq-bot
Copy link
Member

miq-bot commented Feb 9, 2017

Checked commit agrare@50c3388 with ruby 2.2.6, rubocop 0.47.1, and haml-lint 0.20.0
2 files checked, 0 offenses detected
Everything looks good. 🏆

@borod108
Copy link

borod108 commented Feb 9, 2017

LGTM!

@simaishi simaishi removed the euwe/yes label Feb 9, 2017
@agrare
Copy link
Member Author

agrare commented Feb 9, 2017

@simaishi upstream PR was merged

@simaishi
Copy link
Contributor

simaishi commented Feb 9, 2017

@simaishi simaishi merged commit d92ab15 into ManageIQ:euwe Feb 9, 2017
@simaishi simaishi added this to the Sprint 54 Ending Feb 13, 2017 milestone Feb 9, 2017
@agrare agrare deleted the euwe_bz_1420003_rhev_fix_missing_host_link_targeted_refresh branch February 9, 2017 19:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants