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

Remove OpenStack reference from PowerVC #1587

Merged
merged 1 commit into from
Aug 6, 2021

Conversation

jaywcarman
Copy link
Member

There is now a standalone PowerVC provider:
https://github.com/ManageIQ/manageiq-providers-ibm_power_vc

The new PowerVC provider is a subclass of the OpenStack provider. Though
it does still use the OpenStack API this is now transparent to the user.

There is now a standalone PowerVC provider:
https://github.com/ManageIQ/manageiq-providers-ibm_power_vc

The new PowerVC provider is a subclass of the OpenStack provider. Though
it does still use the OpenStack API this is now transparent to the user.
@jaywcarman
Copy link
Member Author

I'm working through #1562 to finally close out all of the items, and this was included. I had thought that wording was more like "via the OpenStack provider", but now I see that what's there, "via the OpenStack API", is still accurate even with the new provider (https://github.com/ManageIQ/manageiq-providers-ibm_power_vc). I thinks it's better to remove it (hence this patch) but don't feel strongly about it.

@chessbyte chessbyte merged commit 1fea0df into ManageIQ:master Aug 6, 2021
@jaywcarman jaywcarman deleted the powervs_openstack_api_ref branch August 6, 2021 04:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants