Replace PowerVS System Pools API with Datacenters API #536
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
jaywcarman
commented
Jun 25, 2025
| X-Ibmcloud-Sdk-Analytics: | ||
| - service_name=resource_controller;service_version=V2;operation_id=get_resource_instance | ||
| Accept: | ||
| - application/json | ||
| Authorization: | ||
| - Bearer IAMBEARERTOKEN | ||
| Authorization: Bearer xxxxxx | ||
| Connection: | ||
| - close | ||
| response: | ||
| status: | ||
| code: 200 |
Member
Author
There was a problem hiding this comment.
For some reason this resource controller API response always has a code of 34 but the body include the requested data 😕. I manually wrote the code to 200, continued recording the rest of the specs and everything looked good. I'll push another PR that shows this issue more clearly. (done: #537)
jaywcarman
force-pushed
the
powervs_replace_system_pools_api_with_datacenters_api
branch
from
ab496c4 to
9b3fd67
Compare
Member
|
Checked commits jaywcarman/manageiq-providers-ibm_cloud@2f912cb~...9b3fd67 with ruby 3.1.7, rubocop 1.56.3, haml-lint 0.64.0, and yamllint lib/tasks_private/power_virtual_servers.rake
|
jaywcarman
force-pushed
the
powervs_replace_system_pools_api_with_datacenters_api
branch
from
9b3fd67 to
8df982b
Compare
The System Pools is no longer available. The Datacenters API now providers a list of available system types (stored in MIQ as a flavor).
The default IAM uri for power_virtual_servers is 'https://iam.cloud.ibm.com/identity/token#apikey'. The resource controller URI starts with 'https://resource-controller.cloud.ibm.com' (no 'iaas' subdomain). There's no harm in sanitizing _any_ domain with cloud.ibm in it.
After creating a PVM instance, the pvm_instances_api.pcloud_pvminstances_get() for the new instance returns an HTTP 500 response for a short time. It takes a few minutes for the instance to be created anyways, so adding a 1m sleep prevents the bad response without slowing down the setup process.
Update images to newer versions that are available in the PowerVS image catalog.
The new PowerVS workspace used to record specs requires some check counts/values to be updated.
jaywcarman
force-pushed
the
powervs_replace_system_pools_api_with_datacenters_api
branch
from
8df982b to
414d847
Compare
jaywcarman
deleted the
powervs_replace_system_pools_api_with_datacenters_api
branch
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
5 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
The PowerVS System Pools API has been deprecated and is no longer available. It was only used to get a list of available host types, which is also available in the Datacenters API.
Depends on:
ibm_cloud_powerv3.0.0 gem