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

Support publish Vm by RHV #15981

Merged
merged 1 commit into from
Sep 26, 2017
Merged

Conversation

masayag
Copy link
Contributor

@masayag masayag commented Sep 18, 2017

RHV provider supports creating a template from a virtual machine.
The dialog introduces the supported options for RHV, including sealing a
template after its creation.

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

Screenshots are available here

@masayag
Copy link
Contributor Author

masayag commented Sep 18, 2017

Copy link
Member

@gmcculloug gmcculloug left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Overall this looks good, just a few minor questions.

true: 1
:description: Choose Automatically
:required: false
:display: :hide
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Does this option need to be hidden? Can't auto-placement still be used to select the target cluster?

Also, why was :placement_dc_name removed from this section? I would think that could still be used for filtering.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

On RHV the user can create a template from the VM only on the same DC (restricted due to resources shared between the same DC, i.e. networks), therefore showing the DC might be misleading to the user.

I'll enable the auto-placements.

true: 1
:description: Power on virtual machines after creation
:required: false
:display: :edit
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Seems like this field should not be displayed, suggest using :display: :hide

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

correct. it shouldn't be displayed. Seems that it is hidden in the attached screenshots regardless of the value of :display.
Nevertheless, I'll change its value to :hide

RHV provider supports creating a template from a virtual machine.
The dialog introduces the supported options for RHV, including sealing a
template after its creation.

https://bugzilla.redhat.com/show_bug.cgi?id=1373076
@miq-bot
Copy link
Member

miq-bot commented Sep 25, 2017

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

@gmcculloug gmcculloug merged commit bc7ad9b into ManageIQ:master Sep 26, 2017
@gmcculloug gmcculloug added this to the Sprint 70 Ending Oct 2, 2017 milestone Sep 26, 2017
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.

5 participants