search

ManageIQ / manageiq-providers-ibm_terraform

Apache License 2.0
2 stars 27 forks source link

Discover iaas resource vm as Configured system. #18

Closed k-chu closed 4 years ago

agrare commented 4 years ago

Hey @kenneth88888 it looks like this is a mix of collecting VMs and adding an identity URL to the params_for_create, could you split the identity endpoint out into a separate PR?

k-chu commented 4 years ago

@agrare sure, will do :)

miq-bot commented 4 years ago

Checked commit https://github.com/kenneth88888/manageiq-providers-ibm_terraform/commit/b266735f6983f786eea4fa15f4969142a983df8e with ruby 2.6.3, rubocop 0.69.0, haml-lint 0.28.0, and yamllint 4 files checked, 0 offenses detected Everything looks fine. :+1:

k-chu commented 4 years ago

Closing this PR - will split into 2 PR :)