-
Notifications
You must be signed in to change notification settings - Fork 897
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 to use azure-armrest version 0.0.6 #5073
Conversation
@@ -21,7 +21,7 @@ def self.make_create_options(template, options) | |||
create_options[:parameters] = create_options[:parameters].map { |k, v| [k, {'value' => v}] }.to_h | |||
end | |||
|
|||
return resource_group, create_options | |||
return resource_group, 'properties' => create_options |
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.
armrest gem no longer automatically insert properties
key if missing.
bf5438a
to
2c3b0fa
Compare
@@ -235,27 +249,29 @@ def add_instance_disk(disks, size, name, location) | |||
super(disks, size, name, location, "azure") | |||
end | |||
|
|||
def populate_hardware_hash_with_networks(hardware_networks_array, instance) | |||
nics = instance.fetch_path('properties', 'networkProfile', 'networkInterfaces') | |||
def populate_hardware_hash_with_networks(networks_array, instance) |
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.
Rewrote the whole implementation. The armrest gem no longer queries network ip addresses and inserts to the vm object. Make explicit calls here to get them.
Looks good to me Bill. I think the rubocop issues are the only thing you'll need to deal with. |
LGTM |
Update Azure refresh logic due to the changes in armrest gem
2c3b0fa
to
93bb6a2
Compare
<github_pr_commenter_batch />Some comments on commit bzwei@93bb6a2 |
Checked commit bzwei@93bb6a2 with ruby 1.9.3, rubocop 0.34.2, and haml-lint 0.13.0 app/models/manageiq/providers/azure/cloud_manager/refresh_parser.rb
spec/models/manageiq/providers/azure/cloud_manager/orchestration_stack_spec.rb
spec/models/manageiq/providers/azure/cloud_manager/refresher_spec.rb
spec/models/manageiq/providers/azure/cloud_manager_spec.rb
|
Update to use azure-armrest version 0.0.6
Update Azure refresh logic due to the changes in armrest gem
Add links between cloud networks and stacks.