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

Add thick_eager disk_format option #385

Merged
merged 1 commit into from
Aug 1, 2019

Conversation

d-m-u
Copy link
Contributor

@d-m-u d-m-u commented Apr 2, 2019

For vmware disk_formats, we're missing an option at the moment.

The "... dialog field option should be updated so that the exist "thick" option remains, but the display value is updated to read "Thick - Lazy Zero". This supports backward-compatibility. Then a new options for "thick_eager" should be added."

#384 and ManageIQ/manageiq#18614 moved the vmware prov dialogs from main to the vmware repo, and this adds the option in the yaml files.

First half of work for https://bugzilla.redhat.com/show_bug.cgi?id=1633867

@d-m-u
Copy link
Contributor Author

d-m-u commented Apr 2, 2019

@miq-bot add_label enhancement
@miq-bot add_reviewer @gmcculloug

@gmcculloug gmcculloug self-assigned this Apr 2, 2019
@gmcculloug gmcculloug added the wip label Apr 2, 2019
@gmcculloug gmcculloug changed the title Add thick_eager disk_format option [WIP] Add thick_eager disk_format option Apr 2, 2019
@miq-bot miq-bot requested a review from gmcculloug April 2, 2019 15:17
@miq-bot
Copy link
Member

miq-bot commented Apr 2, 2019

Checked commit d-m-u@3b5edab with ruby 2.3.3, rubocop 0.52.1, haml-lint 0.20.0, and yamllint 1.10.0
5 files checked, 0 offenses detected
Everything looks fine. ⭐

agrare pushed a commit to agrare/manageiq-providers-vmware that referenced this pull request Apr 15, 2019
…vmattributes

Add specs for Transformation - RestoreVmAttributes
@d-m-u d-m-u changed the title [WIP] Add thick_eager disk_format option Add thick_eager disk_format option Aug 1, 2019
@d-m-u
Copy link
Contributor Author

d-m-u commented Aug 1, 2019

@miq-bot remove_label wip
@agrare could you 👀 please?

@agrare agrare removed the wip label Aug 1, 2019
@agrare agrare merged commit 3b5edab into ManageIQ:master Aug 1, 2019
agrare added a commit that referenced this pull request Aug 1, 2019
@agrare agrare assigned agrare and unassigned gmcculloug Aug 1, 2019
@agrare agrare added this to the Sprint 117 Ending Aug 5, 2019 milestone Aug 1, 2019
@d-m-u
Copy link
Contributor Author

d-m-u commented Aug 1, 2019

@miq-bot add_label ivanchuk/yes

@simaishi
Copy link
Contributor

simaishi commented Aug 1, 2019

Ivanchuk backport details:

$ git log -1
commit 80d2c301c8f0e222476eb73b88fb72e27150e2e8
Author: Adam Grare <agrare@redhat.com>
Date:   Thu Aug 1 10:01:50 2019 -0400

    Merge pull request #385 from d-m-u/adding_prov_option_disks
    
    Add thick_eager disk_format option
    
    (cherry picked from commit 88a141130e7f7217b5c676ba93ffd8d28fa92a30)
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1633867

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