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 provisioning independent VM #277

Merged
merged 1 commit into from Aug 21, 2018

Conversation

borod108
Copy link
Contributor

We ignored the clone_type parameter during provisioning from template.

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

@borod108 borod108 added the bug label Aug 20, 2018
@borod108 borod108 added the wip label Aug 20, 2018
@borod108 borod108 force-pushed the bugs/1598747provisioning_clone branch from 87b0ab4 to 53d3308 Compare August 20, 2018 15:46
@borod108 borod108 closed this Aug 20, 2018
@borod108 borod108 reopened this Aug 20, 2018
@borod108
Copy link
Contributor Author

@agrare Hi, do you have any idea why the ci is failing?

@borod108 borod108 removed the wip label Aug 20, 2018
@masayag
Copy link
Contributor

masayag commented Aug 20, 2018

@borod108 please add doc_text to the bug explaining the semantics of the flow.

@borod108
Copy link
Contributor Author

@masayag thank you, done.

@mwperina
Copy link

LGTM, so it can be included in 5.10 build

@borod108 borod108 force-pushed the bugs/1598747provisioning_clone branch from 53d3308 to cc01e4b Compare August 21, 2018 16:43
We ignored the clone_type parameter during provisioning from template.

Fixes: https://bugzilla.redhat.com/show_bug.cgi?id=1598747
@borod108 borod108 force-pushed the bugs/1598747provisioning_clone branch from cc01e4b to 8dd133f Compare August 21, 2018 16:51
@borod108
Copy link
Contributor Author

borod108 commented Aug 21, 2018

@masayag @pkliczewski can you please review again? I added a change to respect the linked_clone parameter if it is supplied.
And specs :)

@miq-bot
Copy link
Member

miq-bot commented Aug 21, 2018

Checked commit borod108@8dd133f with ruby 2.3.3, rubocop 0.52.1, haml-lint 0.20.0, and yamllint 1.10.0
3 files checked, 0 offenses detected
Everything looks fine. 🍪

@masayag masayag merged commit b88ddd1 into ManageIQ:master Aug 21, 2018
@agrare agrare added this to the Sprint 93 Ending Aug 27, 2018 milestone Aug 22, 2018
simaishi pushed a commit that referenced this pull request Aug 31, 2018
@simaishi
Copy link
Contributor

Gaprindashvili backport details:

$ git log -1
commit 1df8d77e5154b17d06278d2a0e9fc9e0aa530711
Author: Moti Asayag <masayag@redhat.com>
Date:   Tue Aug 21 20:27:23 2018 +0300

    Merge pull request #277 from borod108/bugs/1598747provisioning_clone
    
    Fix provisioning independent VM
    (cherry picked from commit b88ddd1f6a16df20f7ea4367205e84ddb1414ea8)
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1621449

@borod108 borod108 deleted the bugs/1598747provisioning_clone branch December 4, 2018 14:57
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

7 participants