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

Allow any EMS to create cloud volume #600

Merged

Conversation

gberginc
Copy link
Contributor

@gberginc gberginc commented Mar 6, 2017

Contrary to OpenStack where CloudVolume is part of the OpenStack cloud
manager, Amazon provides a separate EMS for block storage management.
This is not based on EmsCloud. Therefore, when only Amazon provider is
available, the button for creating a new cloud volume is disabled.

This patch checks all CloudVolumes that are defined by any
ExtManagementSystem and enables the button as soon as any of them
supports volume creation.

New requirement has been described in the updated spec that fails with
the current version. Applying this patch all tests pass.

@miq-bot add_label storage,enhancement

Contrary to OpenStack where `CloudVolume` is part of the OpenStack cloud
manager, Amazon provides a separate EMS for block storage management.
This is not based on `EmsCloud`. Therefore, when only Amazon provider is
available, the button for creating a new cloud volume is disabled.

This patch checks all `CloudVolume`s that are defined by any
`ExtManagementSystem` and enables the button as soon as any of them
supports volume creation.

New requirement has been described in the updated spec that fails with
the current version. Applying this patch all tests pass.
@miq-bot
Copy link
Member

miq-bot commented Mar 6, 2017

Checked commit gberginc@0cd62ac with ruby 2.2.6, rubocop 0.47.1, and haml-lint 0.20.0
2 files checked, 0 offenses detected
Everything looks good. 🏆

@mzazrivec mzazrivec self-assigned this Mar 7, 2017
@mzazrivec mzazrivec added this to the Sprint 56 Ending Mar 13, 2017 milestone Mar 7, 2017
@mzazrivec mzazrivec merged commit 8e77b45 into ManageIQ:master Mar 7, 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.

3 participants