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

Vm provision handle ems_ref properly #11285

Merged
merged 1 commit into from Sep 15, 2016

Conversation

pkliczewski
Copy link
Contributor

@pkliczewski pkliczewski commented Sep 15, 2016

During vm clone we did not parse ems_ref propely.

During vm clone from template we need to make sure that cloned vm uses
properly built ems_ref.
@pkliczewski
Copy link
Contributor Author

This PR fixes:

https://bugzilla.redhat.com/1374995

@pkliczewski
Copy link
Contributor Author

@miq-bot assign @blomquisg

@miq-bot
Copy link
Member

miq-bot commented Sep 15, 2016

Checked commit pkliczewski@8d4be21 with ruby 2.2.5, rubocop 0.37.2, and haml-lint 0.16.1
1 file checked, 0 offenses detected
Everything looks good. ⭐

@gmcculloug
Copy link
Member

cc @bdunne

@oourfali
Copy link

Looks good to me.

@oourfali
Copy link

@miq-bot add_label darga/yes

@gmcculloug
Copy link
Member

@pkliczewski Please add a link to the BZ in the commit message.

@bdunne Please review.

@bdunne
Copy link
Member

bdunne commented Sep 15, 2016

👍 LGTM. Please restart the affected tests, the unrelated failure has been corrected on master.

@gmcculloug gmcculloug assigned gmcculloug and unassigned blomquisg Sep 15, 2016
@gmcculloug gmcculloug merged commit 8f42493 into ManageIQ:master Sep 15, 2016
@gmcculloug gmcculloug added this to the Sprint 47 Ending Oct 3, 2016 milestone Sep 15, 2016
chessbyte pushed a commit that referenced this pull request Sep 19, 2016
Vm provision handle ems_ref properly
(cherry picked from commit 8f42493)
@chessbyte
Copy link
Member

Darga Backport details:

$ git cherry-pick -x -m 1 8f42493 
[darga 7a3e697] Merge pull request #11285 from pkliczewski/vm_provision
 Author: Greg McCullough <gmccullo@redhat.com>
 Date: Thu Sep 15 16:15:08 2016 -0400
 1 file changed, 1 insertion(+), 1 deletion(-)

$ git log
commit 7a3e697d7a98d7747e7de75650cbbad4b29bd50f
Author: Greg McCullough <gmccullo@redhat.com>
Date:   Thu Sep 15 16:15:08 2016 -0400

    Merge pull request #11285 from pkliczewski/vm_provision

    Vm provision handle ems_ref properly
    (cherry picked from commit 8f42493f86386121a5286ef49c30de0ee713e214)

@chessbyte
Copy link
Member

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.

None yet

9 participants