Skip to content

[release-4.21] OCPBUGS-84225: ibmcloud: bump vpc-go-sdk and capibm#10514

Merged
openshift-merge-bot[bot] merged 4 commits intoopenshift:release-4.21from
Pacho20:4-21-vpc-sgr-fix
May 6, 2026
Merged

[release-4.21] OCPBUGS-84225: ibmcloud: bump vpc-go-sdk and capibm#10514
openshift-merge-bot[bot] merged 4 commits intoopenshift:release-4.21from
Pacho20:4-21-vpc-sgr-fix

Conversation

@Pacho20
Copy link
Copy Markdown
Contributor

@Pacho20 Pacho20 commented Apr 22, 2026

Bumping the vpc go sdk to v0.71.2. This contains a fix for a recent VPC API breaking change.
Also Bumping the capibm version to the latest commit on the https://github.com/kubernetes-sigs/cluster-api-provider-ibmcloud/tree/release-0.10 branch. This is the same release of the currently used capibm with some patches that also contains the VPC API fix.

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Apr 22, 2026

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Repository: openshift/coderabbit/.coderabbit.yaml

Review profile: CHILL

Plan: Enterprise

Run ID: 24b514f6-2a7a-4844-b50d-1dceea7a45df

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@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 Apr 22, 2026
@openshift-ci
Copy link
Copy Markdown
Contributor

openshift-ci Bot commented Apr 22, 2026

Hi @Pacho20. 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.

Tip

We noticed you've done this a few times! Consider joining the org to skip this step and gain /lgtm and other bot rights. We recommend asking approvers on your previous PRs to sponsor you.

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 barbacbd and rvanderp3 April 22, 2026 12:53
@Pacho20
Copy link
Copy Markdown
Contributor Author

Pacho20 commented Apr 23, 2026

/jira cherrypick OCPBUGS-84088

@openshift-ci-robot
Copy link
Copy Markdown
Contributor

@Pacho20: Jira Issue OCPBUGS-84088 has been cloned as Jira Issue OCPBUGS-84225. Will retitle bug to link to clone.
/retitle OCPBUGS-84225: ibmcloud: bump vpc-go-sdk and capibm

Details

In response to this:

/jira cherrypick OCPBUGS-84088

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 openshift-eng/jira-lifecycle-plugin repository.

@openshift-ci openshift-ci Bot changed the title ibmcloud: bump vpc-go-sdk and capibm OCPBUGS-84225: ibmcloud: bump vpc-go-sdk and capibm Apr 23, 2026
@openshift-ci-robot openshift-ci-robot added jira/valid-reference Indicates that this PR references a valid Jira ticket of any type. jira/invalid-bug Indicates that a referenced Jira bug is invalid for the branch this PR is targeting. labels Apr 23, 2026
@openshift-ci-robot
Copy link
Copy Markdown
Contributor

@Pacho20: This pull request references Jira Issue OCPBUGS-84225, which is invalid:

Comment /jira refresh to re-evaluate validity if changes to the Jira bug are made, or edit the title of this pull request to link to a different bug.

The bug has been updated to refer to the pull request using the external bug tracker.

Details

In response to this:

Bumping the vpc go sdk to v0.71.2. This contains a fix for a recent VPC API breaking change.
Also Bumping the capibm version to the latest commit on the https://github.com/kubernetes-sigs/cluster-api-provider-ibmcloud/tree/release-0.10 branch. This is the same release of the currently used capibm with some patches that also contains the VPC API fix.

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 openshift-eng/jira-lifecycle-plugin repository.

@Pacho20
Copy link
Copy Markdown
Contributor Author

Pacho20 commented Apr 23, 2026

/jira refresh

@openshift-ci-robot openshift-ci-robot added jira/valid-bug Indicates that a referenced Jira bug is valid for the branch this PR is targeting. and removed jira/invalid-bug Indicates that a referenced Jira bug is invalid for the branch this PR is targeting. labels Apr 23, 2026
@openshift-ci-robot
Copy link
Copy Markdown
Contributor

@Pacho20: This pull request references Jira Issue OCPBUGS-84225, which is valid. The bug has been moved to the POST state.

7 validation(s) were run on this bug
  • bug is open, matching expected state (open)
  • bug target version (4.21.z) matches configured target version for branch (4.21.z)
  • bug is in the state New, which is one of the valid states (NEW, ASSIGNED, POST)
  • release note text is set and does not match the template
  • dependent bug Jira Issue OCPBUGS-84088 is in the state Verified, which is one of the valid states (VERIFIED, RELEASE PENDING, CLOSED (ERRATA), CLOSED (CURRENT RELEASE), CLOSED (DONE), CLOSED (DONE-ERRATA))
  • dependent Jira Issue OCPBUGS-84088 targets the "4.22.0" version, which is one of the valid target versions: 4.22.0
  • bug has dependents
Details

In response to this:

/jira refresh

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 openshift-eng/jira-lifecycle-plugin repository.

@Pacho20 Pacho20 changed the title OCPBUGS-84225: ibmcloud: bump vpc-go-sdk and capibm [release-4-21] OCPBUGS-84225: ibmcloud: bump vpc-go-sdk and capibm Apr 23, 2026
@Pacho20 Pacho20 changed the title [release-4-21] OCPBUGS-84225: ibmcloud: bump vpc-go-sdk and capibm [release-4.21] OCPBUGS-84225: ibmcloud: bump vpc-go-sdk and capibm Apr 23, 2026
@mjturek
Copy link
Copy Markdown
Contributor

mjturek commented Apr 23, 2026

/test e2e-powervs-capi-ovn

@TwoDCube
Copy link
Copy Markdown
Member

/ok-to-test

@openshift-ci openshift-ci Bot added ok-to-test Indicates a non-member PR verified by an org member that is safe to test. and removed needs-ok-to-test Indicates a PR that requires an org member to verify it is safe to test. labels Apr 23, 2026
@prb112
Copy link
Copy Markdown
Contributor

prb112 commented Apr 23, 2026

/cc @hamzy @mjturek

Could you look at this PR?

@openshift-ci openshift-ci Bot requested review from hamzy and mjturek April 23, 2026 13:42
@mjturek
Copy link
Copy Markdown
Contributor

mjturek commented Apr 23, 2026

/cc @hamzy @mjturek

Could you look at this PR?

I am awaiting the powervs job results

@hamzy
Copy link
Copy Markdown
Contributor

hamzy commented Apr 23, 2026

The installer team prefers the commits to be separated into two commits

  • only go.* changes
  • all the vendor/ files
    This makes reviewing the PR much easier

@sadasu
Copy link
Copy Markdown
Contributor

sadasu commented Apr 26, 2026

/test e2e-powervs-capi-ovn

Pacho20 added 2 commits April 28, 2026 14:39
The newer API versions of the vpc contains some breaking changes for
SecurityGroupRuleProtocol. Some older SDKs was patched to handle additional
protocol values introduced in newer API versions, ensuring older SDK
releases do not fail when reading security group or network ACL rules
created with expanded protocol support. The latest release of v0.10 of
the cluster-api-provider-ibmcloud uses one of the patched SDKs.
This update allows the installer to continue functioning with the changes.
Pacho20 added 2 commits April 28, 2026 14:44
SDK Compatibility Patch - Protocol Handling
The SDK was patched to handle additional protocol values introduced in
newer API versions, ensuring older SDK releases do not fail when reading
security group or network ACL rules created with expanded protocol support.
@mjturek
Copy link
Copy Markdown
Contributor

mjturek commented Apr 28, 2026

/test e2e-powervs-capi-ovn

1 similar comment
@mjturek
Copy link
Copy Markdown
Contributor

mjturek commented Apr 28, 2026

/test e2e-powervs-capi-ovn

@openshift-ci
Copy link
Copy Markdown
Contributor

openshift-ci Bot commented Apr 28, 2026

@Pacho20: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name Commit Details Required Rerun command
ci/prow/e2e-powervs-capi-ovn 75f444e link false /test e2e-powervs-capi-ovn
ci/prow/e2e-ibmcloud-ovn 75f444e link false /test e2e-ibmcloud-ovn

Full PR test history. Your PR dashboard.

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. I understand the commands that are listed here.

@mjturek
Copy link
Copy Markdown
Contributor

mjturek commented Apr 28, 2026

/lgtm

Failure seems unrelated. Thanks for the patience.

@openshift-ci openshift-ci Bot added the lgtm Indicates that a PR is ready to be merged. label Apr 28, 2026
@sadasu
Copy link
Copy Markdown
Contributor

sadasu commented Apr 29, 2026

/approve

@openshift-ci
Copy link
Copy Markdown
Contributor

openshift-ci Bot commented Apr 29, 2026

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: sadasu

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

The pull request process is described 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-ci openshift-ci Bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Apr 29, 2026
@sadasu
Copy link
Copy Markdown
Contributor

sadasu commented Apr 29, 2026

/verified by CI.
More specifically, the presubmit job e2e-powervs-capi-ovn.

@openshift-ci-robot openshift-ci-robot added the verified Signifies that the PR passed pre-merge verification criteria label Apr 29, 2026
@openshift-ci-robot
Copy link
Copy Markdown
Contributor

@sadasu: This PR has been marked as verified by CI..

Details

In response to this:

/verified by CI.
More specifically, the presubmit job e2e-powervs-capi-ovn.

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 openshift-eng/jira-lifecycle-plugin repository.

@sadasu
Copy link
Copy Markdown
Contributor

sadasu commented May 6, 2026

/label backport-risk-assessed

@openshift-ci openshift-ci Bot added the backport-risk-assessed Indicates a PR to a release branch has been evaluated and considered safe to accept. label May 6, 2026
@openshift-merge-bot openshift-merge-bot Bot merged commit 9a415c4 into openshift:release-4.21 May 6, 2026
15 of 17 checks passed
@openshift-ci-robot
Copy link
Copy Markdown
Contributor

@Pacho20: Jira Issue Verification Checks: Jira Issue OCPBUGS-84225
✔️ This pull request was pre-merge verified.
✔️ All associated pull requests have merged.
✔️ All associated, merged pull requests were pre-merge verified.

Jira Issue OCPBUGS-84225 has been moved to the MODIFIED state and will move to the VERIFIED state when the change is available in an accepted nightly payload. 🕓

Details

In response to this:

Bumping the vpc go sdk to v0.71.2. This contains a fix for a recent VPC API breaking change.
Also Bumping the capibm version to the latest commit on the https://github.com/kubernetes-sigs/cluster-api-provider-ibmcloud/tree/release-0.10 branch. This is the same release of the currently used capibm with some patches that also contains the VPC API fix.

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 openshift-eng/jira-lifecycle-plugin repository.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. backport-risk-assessed Indicates a PR to a release branch has been evaluated and considered safe to accept. jira/valid-bug Indicates that a referenced Jira bug is valid for the branch this PR is targeting. jira/valid-reference Indicates that this PR references a valid Jira ticket of any type. lgtm Indicates that a PR is ready to be merged. ok-to-test Indicates a non-member PR verified by an org member that is safe to test. verified Signifies that the PR passed pre-merge verification criteria

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants