-
Notifications
You must be signed in to change notification settings - Fork 120
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
Requested Quota for instance_type dialog override. #333
Requested Quota for instance_type dialog override. #333
Conversation
Updated requested method to allow for Service dialogs to override instance_type for cloud provisioning. Added tests for Google service and instance_type override This requires updated quota_helper method in ManageIQ repo.
@miq-bot add_label enhancement |
Checked commit billfitzgerald0120@641b8d0 with ruby 2.3.3, rubocop 0.52.1, haml-lint 0.20.0, and yamllint 1.10.0 |
@tinaafitz Please review |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@billfitzgerald0120 Looks good.
@mkanoor Please review.
@@ -263,7 +264,7 @@ def cloud_storage(args_hash) | |||
flavor.root_disk_size.to_i + flavor.ephemeral_disk_size.to_i + flavor.swap_disk_size.to_i | |||
end | |||
|
|||
storage += cloud_volume_storage(args_hash) | |||
storage += cloud_volume_storage(args_hash) if @reconfigure_request |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@billfitzgerald0120
Is this related to this PR? or related to configure request
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@mkanoor I added the "if @reconfigure_request" because it caused the test to fail without it.
@mkanoor The dependent PR is merged, anything else for this PR? |
Updated requested method to allow for Service dialogs to override instance_type for cloud provisioning.
Added tests for Google service and instance_type override.
This requires updated quota_helper method in ManageIQ repo.
ManageIQ/manageiq#17597
https://bugzilla.redhat.com/show_bug.cgi?id=1581288