Skip to content

Conversation

@taliandre49
Copy link

Problem

The installer was defaulting to deprecated Power9 systems (e.g., s922, e980) in certain PowerVS regions like dal14, even though our hardcoded region mappings correctly specify s1022 as the supported system type.

Root Cause

The issue stems from how we retrieve supported system types via the PowerVS API. The installer currently uses GetDatacenterCapabilities(), which calls a regional bulk endpoint that returns cached/stale data. For dal14, this endpoint still includes deprecated Power9 systems in its response. Since the default selection logic picks the first system type returned, the installer ends up defaulting to an unsupported system.
The IBM Cloud CLI avoids this issue by using a datacenter-specific v1 endpoint (datacenterClient.Get(region)), which returns accurate, up-to-date supported systems.

Solution

Updated GetDatacenterSupportedSystems() to use the datacenter-specific endpoint via instance.NewIBMPIDatacenterClient and datacenterClient.Get(region). This ensures we retrieve accurate, non-cached supported systems data for all regions. (code found in the pkg/asset/installconfig/powervs/client.go file

Impact

Set dal14 to defaults to s1022 instead of s1222 systems, dal14 no longer considers deprecated s922 a valid supported sysType option
All other regions continue to work as expected with accurate system type data
Removes reliance on stale cached data from bulk endpoints

Testing

Validated that dal14 returns only s1022 (no deprecated systems) and defaults correctly. Verified other regions remain unaffected.

eshulman2 and others added 4 commits January 9, 2026 13:02
- Added dal14 region support for PowerVS
- Added logic to pin default sysType version for dal14 to s1022

Signed-off-by: Natalia Jordan <natalia.jordan@ibm.com>
…s- dal14 now returns the corrcect supported systems, identifies power 9 no longer supported

Signed-off-by: Natalia Jordan <natalia.jordan@ibm.com>
Signed-off-by: Natalia Jordan <natalia.jordan@ibm.com>
@openshift-merge-robot openshift-merge-robot added the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Jan 28, 2026
@openshift-ci openshift-ci bot added the needs-ok-to-test Indicates a PR that requires an org member to verify it is safe to test. label Jan 28, 2026
@openshift-ci
Copy link
Contributor

openshift-ci bot commented Jan 28, 2026

Hi @taliandre49. Thanks for your PR.

I'm waiting for a openshift member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.

Once the patch is verified, the new status will be reflected by the ok-to-test label.

I understand the commands that are listed here.

Details

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.

@openshift-ci openshift-ci bot requested review from bfournie and mjturek January 28, 2026 18:06
@openshift-ci
Copy link
Contributor

openshift-ci bot commented Jan 28, 2026

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
Once this PR has been reviewed and has the lgtm label, please assign pawanpinjarkar for approval. For more information see the Code Review Process.

The full list of commands accepted by this bot can be found here.

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@openshift-merge-robot openshift-merge-robot removed the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Jan 28, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

needs-ok-to-test Indicates a PR that requires an org member to verify it is safe to test.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants