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
Update armrest version to 0.7.0 #42
Conversation
@miq-bot add_label euwe/yes |
@miq-bot add_label darga/yes |
I think this will fail until ManageIQ/manageiq-gems-pending#100 is merged. |
Update flavor count and refresher cassettes. Updated for latest api-version change.
Checked commit https://github.com/djberg96/manageiq-providers-azure/commit/ba881682a2451d6ed4ac042f7e521441c48a1d06 with ruby 2.2.6, rubocop 0.47.1, and haml-lint 0.20.0 |
@miq-bot add_label fine/yes |
Update armrest version to 0.7.0 (cherry picked from commit c318b5d) https://bugzilla.redhat.com/show_bug.cgi?id=1438092
Fine backport details:
|
@djberg96 Can you please create an Euwe PR for this PR and ManageIQ/manageiq-gems-pending#100 combined? |
@simaishi Ok, please see ManageIQ/manageiq#14799 |
Backported to Euwe via ManageIQ/manageiq#14799 |
This bumps the azure-armrest version up to 0.7.0.
Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1434988
The major version changes shouldn't be scary. The change from 0.5.x to 0.6.x was caused by an incompatible change to Role classes that we don't use internally. The change from 0.6.x to 0.7.x is the backwards incompatible, but necessary, change for getting templates.
Other than a redundant
.to_s
call, there should be no issues on the ManageIQ side.