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 src_vm_or_dest_host_refresh_target. #18715

Merged
merged 1 commit into from
May 21, 2019

Conversation

lfu
Copy link
Member

@lfu lfu commented May 1, 2019

If the VM was migrated off of the host before refresh adds the VM to the database then the src_vm target for this "/System/event_handlers/event_action_refresh?target=src_vm" won't work.
We need to refresh the dest_host.

Blocks ManageIQ/manageiq-content#531.

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

@miq-bot add_label bug, hammer/yes
@miq-bot assign @tinaafitz
cc @gmcculloug @agrare

If the VM was migrated off of the host before refresh adds the VM to the database then the src_vm target
for this "/System/event_handlers/event_action_refresh?target=src_vm" won't work.
We need to refresh the dest_host.

https://bugzilla.redhat.com/show_bug.cgi?id=1696889
@lfu lfu force-pushed the src_vm_or_dest_host_1696889 branch from 267daea to 702b5e1 Compare May 13, 2019 20:53
@miq-bot
Copy link
Member

miq-bot commented May 13, 2019

Checked commit lfu@702b5e1 with ruby 2.3.3, rubocop 0.52.1, haml-lint 0.20.0, and yamllint 1.10.0
2 files checked, 0 offenses detected
Everything looks fine. 🏆

@lfu
Copy link
Member Author

lfu commented May 20, 2019

@bzwei Could you please review this one?

@bzwei
Copy link
Contributor

bzwei commented May 20, 2019

LGTM

@Fryguy Fryguy merged commit 8a87d0f into ManageIQ:master May 21, 2019
@Fryguy Fryguy added this to the Sprint 112 Ending May 27, 2019 milestone May 21, 2019
@simaishi
Copy link
Contributor

Hammer backport details:

$ git log -1
commit 707a588b3da02d0908041b6d7b14f827e86c60d7
Author: Jason Frey <jfrey@redhat.com>
Date:   Tue May 21 12:31:27 2019 -0400

    Merge pull request #18715 from lfu/src_vm_or_dest_host_1696889
    
    Add src_vm_or_dest_host_refresh_target.
    
    (cherry picked from commit 8a87d0f3cba7f961aa02a16c19c333aaa7ae3920)
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1712595

simaishi pushed a commit that referenced this pull request May 22, 2019
@lfu lfu deleted the src_vm_or_dest_host_1696889 branch November 4, 2019 15:23
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.

7 participants