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

Fix vm_migrate_task statemachine_task_status value. #38

Merged
merged 1 commit into from
Jun 8, 2017

Conversation

tinaafitz
Copy link
Member

Check status to determine return value.

Depends on: ManageIQ/manageiq#15332

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

@tinaafitz
Copy link
Member Author

@miq-bot assign @gmcculloug
@miq-bot add_label bug, fine/yes

@miq-bot
Copy link
Member

miq-bot commented Jun 7, 2017

Checked commit tinaafitz@aaf998e with ruby 2.2.6, rubocop 0.47.1, and haml-lint 0.20.0
2 files checked, 0 offenses detected
Everything looks fine. 👍

@gmcculloug
Copy link
Member

Opened issue #39 to address code client duplication issue.

@gmcculloug gmcculloug merged commit f4bc749 into ManageIQ:master Jun 8, 2017
@gmcculloug gmcculloug added this to the Sprint 63 Ending Jun 19, 2017 milestone Jun 8, 2017
@simaishi
Copy link
Contributor

simaishi commented Aug 8, 2017

Fine backport (to manageiq repo) details:

$ git log -1
commit c3d20fa84f18371729a486c0b4bfe22443d446d5
Author: Greg McCullough <gmccullo@redhat.com>
Date:   Thu Jun 8 15:51:18 2017 -0400

    Merge pull request #38 from tinaafitz/vm_migrate_fix
    
    Fix vm_migrate_task statemachine_task_status value.
    (cherry picked from commit f4bc749a055b9fff6b215ad62b83263fba29327b)
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1479478

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.

4 participants