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

Require cluster for RVH VM provision #16281

Merged

Conversation

masayag
Copy link
Contributor

@masayag masayag commented Oct 24, 2017

In order to provision VM on RHV, when not selecting the auto-placement,
user should specify the destination cluster as well.

The PR enforces the selection of the destination cluster.

Bug-Url:
https://bugzilla.redhat.com/show_bug.cgi?id=1469364

@masayag
Copy link
Contributor Author

masayag commented Oct 24, 2017

@masayag masayag changed the title Require cluster for RVH VM provision [WIP] Require cluster for RVH VM provision Oct 24, 2017
@miq-bot miq-bot added the wip label Oct 24, 2017
In order to provision VM on RHV, when not selecting the auto-placement,
user should specify the destination cluster as well.

The PR enforces the selection of the destination cluster.

https://bugzilla.redhat.com/show_bug.cgi?id=1469364
@masayag masayag force-pushed the require_cluster_for_vm_provisioning branch from 1ca93af to 21180e2 Compare October 24, 2017 14:10
@masayag masayag changed the title [WIP] Require cluster for RVH VM provision Require cluster for RVH VM provision Oct 24, 2017
@miq-bot
Copy link
Member

miq-bot commented Oct 24, 2017

Checked commit masayag@21180e2 with ruby 2.3.3, rubocop 0.47.1, and haml-lint 0.20.0
0 files checked, 0 offenses detected
Everything looks fine. 👍

@miq-bot miq-bot removed the wip label Oct 24, 2017
@gmcculloug gmcculloug merged commit 5dfe415 into ManageIQ:master Oct 24, 2017
@gmcculloug gmcculloug added this to the Sprint 72 Ending Oct 30, 2017 milestone Oct 24, 2017
@masayag masayag deleted the require_cluster_for_vm_provisioning branch June 27, 2018 20:29
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.

6 participants