openshift / installer

Install an OpenShift 4.x cluster
https://try.openshift.com
Apache License 2.0
1.44k stars 1.39k forks source link

CORS-3282: IBMCloud: CAPI Machines #9203

Closed cjschaef closed 1 week ago

cjschaef commented 2 weeks ago

Update the IBM Cloud CAPI version to include the new IBM Cloud VPC Load Balancer support, completing the major functional changes for VPC Infrastructure. This bump consists of:

cjschaef commented 2 weeks ago

/test altinfra-e2e-ibmcloud-capi-ovn

Power team likely wants to confirm the bump as well. I'll let them decide if they want to rely on CI or another method to be safe.

cjschaef commented 2 weeks ago

e2e-ibmcloud-ovn failure appears to be https://issues.redhat.com/browse/OCPBUGS-42869

: [Jira:"Networking / router"] monitor test service-type-load-balancer-availability setup expand_less | 50s
-- | --
{  failed during setup error waiting for replicaset: failed to create RC "service-test": Internal error occurred: resource quota evaluation timed out}

altinfra-e2e-ibmcloud-capi-ovn failure occurs due to 15 minute timeout (Infra takes roughly 20 mins)

level=error msg=failed to fetch Cluster: failed to generate asset "Cluster": failed to create cluster: failed to provision control-plane machines within 15m0s: context deadline exceeded

This should be addressed as part of https://github.com/openshift/installer/pull/9199

mjturek commented 2 weeks ago

/test e2e-powervs-capi-ovn

openshift-ci[bot] commented 2 weeks ago

@cjschaef: 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/altinfra-e2e-ibmcloud-capi-ovn 3fe3a9223702e9ce4c4c6024f3824a108253fc14 link false /test altinfra-e2e-ibmcloud-capi-ovn
ci/prow/e2e-ibmcloud-ovn 3fe3a9223702e9ce4c4c6024f3824a108253fc14 link false /test e2e-ibmcloud-ovn
ci/prow/e2e-powervs-capi-ovn 3fe3a9223702e9ce4c4c6024f3824a108253fc14 link false /test e2e-powervs-capi-ovn

Full PR test history. Your PR dashboard.

Instructions for interacting with me using PR comments are available [here](https://git.k8s.io/community/contributors/guide/pull-requests.md). If you have questions or suggestions related to my behavior, please file an issue against the [kubernetes-sigs/prow](https://github.com/kubernetes-sigs/prow/issues/new?title=Prow%20issue:) repository. I understand the commands that are listed [here](https://go.k8s.io/bot-commands).
mjturek commented 1 week ago

The powervs failure seems unrelated (couldn't get third master up). I also deployed this locally fine.

/lgtm

sadasu commented 1 week ago

/approve

openshift-ci[bot] commented 1 week ago

[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

Needs approval from an approver in each of these files: - ~~[OWNERS](https://github.com/openshift/installer/blob/master/OWNERS)~~ [sadasu] Approvers can indicate their approval by writing `/approve` in a comment Approvers can cancel approval by writing `/approve cancel` in a comment
jeffnowicki commented 1 week ago

/retitle CORS-3282: IBMCloud: CAPI Machines

openshift-ci-robot commented 1 week ago

@cjschaef: This pull request references CORS-3282 which is a valid jira issue.

Warning: The referenced jira issue has an invalid target version for the target branch this PR targets: expected the story to target either version "4.18." or "openshift-4.18.", but it targets "4.16.0" instead.

In response to [this](https://github.com/openshift/installer/pull/9203): >Update the IBM Cloud CAPI version to include the new >IBM Cloud VPC Load Balancer support, completing the >major functional changes for VPC Infrastructure. This >bump consists of: > >- IBMCloud: Bump CAPI and IBM Cloud CAPI go mod versions >- IBMCloud: Update IBM Cloud CAPI Infra components/manifests >- IBMCloud: Update vendor for CAPI and IBM Cloud CAPI bumps Instructions for interacting with me using PR comments are available [here](https://prow.ci.openshift.org/command-help?repo=openshift%2Finstaller). If you have questions or suggestions related to my behavior, please file an issue against the [openshift-eng/jira-lifecycle-plugin](https://github.com/openshift-eng/jira-lifecycle-plugin/issues/new) repository.
jeffnowicki commented 1 week ago

Retitling to progress PR.

jeffnowicki commented 1 week ago

/jira refresh

openshift-ci-robot commented 1 week ago

@jeffnowicki: This pull request references CORS-3282 which is a valid jira issue.

In response to [this](https://github.com/openshift/installer/pull/9203#issuecomment-2483349452): >/jira refresh Instructions for interacting with me using PR comments are available [here](https://prow.ci.openshift.org/command-help?repo=openshift%2Finstaller). If you have questions or suggestions related to my behavior, please file an issue against the [openshift-eng/jira-lifecycle-plugin](https://github.com/openshift-eng/jira-lifecycle-plugin/issues/new) repository.
openshift-bot commented 1 week ago

[ART PR BUILD NOTIFIER]

Distgit: ose-installer-altinfra This PR has been included in build ose-installer-altinfra-container-v4.18.0-202411182236.p0.ga892cbd.assembly.stream.el9. All builds following this will include this PR.

openshift-bot commented 1 week ago

[ART PR BUILD NOTIFIER]

Distgit: ose-installer-terraform-providers This PR has been included in build ose-installer-terraform-providers-container-v4.18.0-202411182236.p0.ga892cbd.assembly.stream.el9. All builds following this will include this PR.

openshift-bot commented 1 week ago

[ART PR BUILD NOTIFIER]

Distgit: ose-baremetal-installer This PR has been included in build ose-baremetal-installer-container-v4.18.0-202411182236.p0.ga892cbd.assembly.stream.el9. All builds following this will include this PR.

openshift-bot commented 1 week ago

[ART PR BUILD NOTIFIER]

Distgit: ose-installer-artifacts This PR has been included in build ose-installer-artifacts-container-v4.18.0-202411182236.p0.ga892cbd.assembly.stream.el9. All builds following this will include this PR.