Skip to content

SPLAT-2337: Added OTE binary for ccm-aws#30525

Merged
openshift-merge-bot[bot] merged 1 commit intoopenshift:mainfrom
mtulio:SPLAT-2337-ote-ccm-aws
Jan 31, 2026
Merged

SPLAT-2337: Added OTE binary for ccm-aws#30525
openshift-merge-bot[bot] merged 1 commit intoopenshift:mainfrom
mtulio:SPLAT-2337-ote-ccm-aws

Conversation

@mtulio
Copy link
Copy Markdown
Contributor

@mtulio mtulio commented Nov 25, 2025

Re-adding OTE for CCM-AWS to validate hypershift: #30235

CCM-AWS e2e (OTE) on hypershift workload cluster is working and must skip two tests that is failing due an possible issue on retries, documented in:

@openshift-ci-robot
Copy link
Copy Markdown

Pipeline controller notification
This repository is configured to use the pipeline controller. Second-stage tests will be triggered either automatically or after lgtm label is added, depending on the repository configuration. The pipeline controller will automatically detect which contexts are required and will utilize /test Prow commands to trigger the second stage.

For optional jobs, comment /test ? to see a list of all defined jobs. To trigger manually all jobs from second stage use /pipeline required command.

This repository is configured in: automatic mode

@openshift-ci openshift-ci bot added the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Nov 25, 2025
@openshift-ci-robot openshift-ci-robot added the jira/valid-reference Indicates that this PR references a valid Jira ticket of any type. label Nov 25, 2025
@openshift-ci
Copy link
Copy Markdown
Contributor

openshift-ci bot commented Nov 25, 2025

Skipping CI for Draft Pull Request.
If you want CI signal for your change, please convert it to an actual PR.
You can still manually trigger a test run with /test all

@openshift-ci-robot
Copy link
Copy Markdown

openshift-ci-robot commented Nov 25, 2025

@mtulio: This pull request references SPLAT-2337 which is a valid jira issue.

Details

In response to this:

Re-adding OTE for CCM-AWS to validate hypershift

openshift/cloud-provider-aws#122

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.

@mtulio
Copy link
Copy Markdown
Contributor Author

mtulio commented Nov 25, 2025

/help

@mtulio
Copy link
Copy Markdown
Contributor Author

mtulio commented Nov 25, 2025

/test ?

@mtulio
Copy link
Copy Markdown
Contributor Author

mtulio commented Nov 25, 2025

/test e2e-hypershift-conformance

@openshift-ci-robot
Copy link
Copy Markdown

openshift-ci-robot commented Nov 25, 2025

@mtulio: This pull request references SPLAT-2337 which is a valid jira issue.

Details

In response to this:

Re-adding OTE for CCM-AWS to validate hypershift: #30235

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-robot
Copy link
Copy Markdown

openshift-ci-robot commented Nov 25, 2025

@mtulio: This pull request references SPLAT-2337 which is a valid jira issue.

Details

In response to this:

Re-adding OTE for CCM-AWS to validate hypershift: #30235

DNM while proofing working upstream CCM-AWS e2e (OTE) on hypershift workload cluster.

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-merge-robot openshift-merge-robot added the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Nov 28, 2025
@mtulio
Copy link
Copy Markdown
Contributor Author

mtulio commented Dec 4, 2025

/testwith openshift/origin/main/e2e-hypershift-conformance openshift/hypershift/pull/7339

@openshift-ci
Copy link
Copy Markdown
Contributor

openshift-ci bot commented Dec 4, 2025

@mtulio, testwith: Error processing request. ERROR:

could not determine job runs: invalid format for additional PR: openshift/hypershift/pull/7339

@mtulio mtulio force-pushed the SPLAT-2337-ote-ccm-aws branch from 8f3a0a7 to c8a18aa Compare December 4, 2025 20:40
@openshift-merge-robot openshift-merge-robot removed the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Dec 4, 2025
@mtulio
Copy link
Copy Markdown
Contributor Author

mtulio commented Dec 4, 2025

/testwith openshift/origin/main/e2e-hypershift-conformance openshift/hypershift/pull/7339

@openshift-ci
Copy link
Copy Markdown
Contributor

openshift-ci bot commented Dec 4, 2025

@mtulio, testwith: Error processing request. ERROR:

could not determine job runs: invalid format for additional PR: openshift/hypershift/pull/7339

@mtulio
Copy link
Copy Markdown
Contributor Author

mtulio commented Dec 4, 2025

/testwith openshift/origin/main/e2e-hypershift-conformance openshift/hypershift#7339

1 similar comment
@mtulio
Copy link
Copy Markdown
Contributor Author

mtulio commented Dec 10, 2025

/testwith openshift/origin/main/e2e-hypershift-conformance openshift/hypershift#7339

@mtulio
Copy link
Copy Markdown
Contributor Author

mtulio commented Dec 10, 2025

/testwith openshift/origin/main/e2e-hypershift-conformance openshift/hypershift#7339

re-testing after updates in the hypershift repo. I am not having success running /testwith from there.

@mtulio
Copy link
Copy Markdown
Contributor Author

mtulio commented Dec 15, 2025

/testwith openshift/origin/main/e2e-hypershift-conformance openshift/hypershift#7339

re-testing after updates in the hypershift repo. I am not having success running /testwith from there.

We've been providing inline patch to the target PR, as aligned with Hypershift team. Let's see the new results:

@mtulio
Copy link
Copy Markdown
Contributor Author

mtulio commented Dec 15, 2025

/testwith openshift/origin/main/e2e-hypershift-conformance openshift/hypershift#7339

1 similar comment
@mtulio
Copy link
Copy Markdown
Contributor Author

mtulio commented Jan 7, 2026

/testwith openshift/origin/main/e2e-hypershift-conformance openshift/hypershift#7339

@mtulio
Copy link
Copy Markdown
Contributor Author

mtulio commented Jan 12, 2026

testing with new changes forcing policy - without checking flag (that seems to be not set on create cluster nor iam):

@mtulio
Copy link
Copy Markdown
Contributor Author

mtulio commented Jan 12, 2026

/testwith openshift/origin/main/e2e-hypershift-conformance openshift/hypershift#7339

@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 12, 2026
@mtulio mtulio force-pushed the SPLAT-2337-ote-ccm-aws branch from c8a18aa to 7379dc1 Compare January 12, 2026 20:16
@mtulio
Copy link
Copy Markdown
Contributor Author

mtulio commented Jan 12, 2026

/testwith openshift/origin/main/e2e-hypershift-conformance openshift/hypershift#7339

@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 12, 2026
@mtulio
Copy link
Copy Markdown
Contributor Author

mtulio commented Jan 28, 2026

/validated by mtulio on job e2e-hypershift-conformance #2016524990978985984

@mtulio
Copy link
Copy Markdown
Contributor Author

mtulio commented Jan 28, 2026

/validated by mtulio on job e2e-hypershift-conformance #2016524990978985984

/verified by mtulio on job e2e-hypershift-conformance #2016524990978985984

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

@mtulio: This PR has been marked as verified by mtulio on job [e2e-hypershift-conformance #2016524990978985984 ](https://prow.ci.openshift.org/view/gs/test-platform-results/pr-logs/pull/30525/pull-ci-openshift-origin-main-e2e-hypershift-conformance/2016524990978985984).

Details

In response to this:

/validated by mtulio on job e2e-hypershift-conformance #2016524990978985984

/verified by mtulio on job e2e-hypershift-conformance #2016524990978985984

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.

@mtulio
Copy link
Copy Markdown
Contributor Author

mtulio commented Jan 28, 2026

/verified by mtulio on job e2e-hypershift-conformance #2016524990978985984

/verified by mtulio on job e2e-hypershift-conformance #2016524990978985984 - and all tests provided by OTE is passing (prefix [cloud-provider-aws-e2e])

@openshift-ci-robot
Copy link
Copy Markdown

@mtulio: This PR has been marked as verified by mtulio on job [e2e-hypershift-conformance #2016524990978985984](https://prow.ci.openshift.org/view/gs/test-platform-results/pr-logs/pull/30525/pull-ci-openshift-origin-main-e2e-hypershift-conformance/2016524990978985984) - and all tests provided by OTE is passing (prefix [cloud-provider-aws-e2e]).

Details

In response to this:

/verified by mtulio on job e2e-hypershift-conformance #2016524990978985984

/verified by mtulio on job e2e-hypershift-conformance #2016524990978985984 - and all tests provided by OTE is passing (prefix [cloud-provider-aws-e2e])

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.

@mtulio
Copy link
Copy Markdown
Contributor Author

mtulio commented Jan 28, 2026

/test e2e-vsphere-ovn

@smg247
Copy link
Copy Markdown
Member

smg247 commented Jan 29, 2026

/approve

@openshift-ci
Copy link
Copy Markdown
Contributor

openshift-ci bot commented Jan 29, 2026

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: mtulio, rvanderp3, smg247

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 Jan 29, 2026
@mtulio
Copy link
Copy Markdown
Contributor Author

mtulio commented Jan 29, 2026

The tests are passing in those tests (filter by test prefix [cloud-provider-aws-e2e] in the pass list):

And skipped in non-AWS job, used the job as example: e2e-vsphere-ovn #2016594179487961088:

time="2026-01-28T20:38:36Z" level=info 
msg="Listed 0 tests in 58.926864ms" 
binary=aws-cloud-controller-manager-tests-ext

@mtulio
Copy link
Copy Markdown
Contributor Author

mtulio commented Jan 29, 2026

/verified by mtulio #30525 (comment)

@openshift-ci-robot
Copy link
Copy Markdown

@mtulio: This PR has been marked as verified by mtulio https://github.com/openshift/origin/pull/30525#issuecomment-3817704811.

Details

In response to this:

/verified by mtulio #30525 (comment)

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.

@mtulio
Copy link
Copy Markdown
Contributor Author

mtulio commented Jan 29, 2026

/test e2e-aws-ovn-microshift

@openshift-ci-robot
Copy link
Copy Markdown

/retest-required

Remaining retests: 0 against base HEAD be1e34a and 2 for PR HEAD 1a50012 in total

@mtulio
Copy link
Copy Markdown
Contributor Author

mtulio commented Jan 29, 2026

Job failing on build (registry issues).

/test e2e-aws-ovn-microshift

@mtulio
Copy link
Copy Markdown
Contributor Author

mtulio commented Jan 29, 2026

/retest-required

@openshift-ci-robot
Copy link
Copy Markdown

/retest-required

Remaining retests: 0 against base HEAD e713d4e and 1 for PR HEAD 1a50012 in total

@mtulio
Copy link
Copy Markdown
Contributor Author

mtulio commented Jan 30, 2026

The job is reporting permanent failure in many jobs starting yesterday. I see one green signal this morning from 017143090124427264, let me try again:prow.ci.openshift.org/view/gs/test-platform-results/pr-logs/pull/30657/pull-ci-openshift-origin-main-e2e-aws-ovn-microshift/2017143090124427264), let me try again:

/retest-required

@mtulio
Copy link
Copy Markdown
Contributor Author

mtulio commented Jan 30, 2026

/pipeline required

@openshift-ci-robot
Copy link
Copy Markdown

Scheduling required tests:
/test e2e-aws-csi
/test e2e-aws-ovn-fips
/test e2e-aws-ovn-microshift
/test e2e-aws-ovn-microshift-serial
/test e2e-aws-ovn-serial-1of2
/test e2e-aws-ovn-serial-2of2
/test e2e-gcp-csi
/test e2e-gcp-ovn
/test e2e-gcp-ovn-upgrade
/test e2e-metal-ipi-ovn-ipv6
/test e2e-vsphere-ovn
/test e2e-vsphere-ovn-upi

@openshift-trt
Copy link
Copy Markdown

openshift-trt bot commented Jan 30, 2026

Risk analysis has seen new tests most likely introduced by this PR.
Please ensure that new tests meet guidelines for naming and stability.

New Test Risks for sha: 1a50012

Job Name New Test Risk
pull-ci-openshift-origin-main-e2e-aws-ovn-microshift-serial Medium - "[Jira:"Cluster Version Operator"] cluster-version-operator-tests should support passing serial tests [Serial]" is a new test, and was only seen in one job.

New tests seen in this PR at sha: 1a50012

  • "[Jira:"Cluster Version Operator"] cluster-version-operator-tests should support passing serial tests [Serial]" [Total: 1, Pass: 1, Fail: 0, Flake: 0]

1 similar comment
@openshift-trt
Copy link
Copy Markdown

openshift-trt bot commented Jan 30, 2026

Risk analysis has seen new tests most likely introduced by this PR.
Please ensure that new tests meet guidelines for naming and stability.

New Test Risks for sha: 1a50012

Job Name New Test Risk
pull-ci-openshift-origin-main-e2e-aws-ovn-microshift-serial Medium - "[Jira:"Cluster Version Operator"] cluster-version-operator-tests should support passing serial tests [Serial]" is a new test, and was only seen in one job.

New tests seen in this PR at sha: 1a50012

  • "[Jira:"Cluster Version Operator"] cluster-version-operator-tests should support passing serial tests [Serial]" [Total: 1, Pass: 1, Fail: 0, Flake: 0]

@openshift-trt
Copy link
Copy Markdown

openshift-trt bot commented Jan 30, 2026

Risk analysis has seen new tests most likely introduced by this PR.
Please ensure that new tests meet guidelines for naming and stability.

New tests seen in this PR at sha: 1a50012

  • "[Jira:"Cluster Version Operator"] cluster-version-operator-tests should support passing serial tests [Serial]" [Total: 2, Pass: 2, Fail: 0, Flake: 0]

@mtulio
Copy link
Copy Markdown
Contributor Author

mtulio commented Jan 31, 2026

unrelated test failure

/test e2e-aws-ovn-serial-1of2

@openshift-ci
Copy link
Copy Markdown
Contributor

openshift-ci bot commented Jan 31, 2026

@mtulio: all tests passed!

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.

@openshift-merge-bot openshift-merge-bot bot merged commit e00df4c into openshift:main Jan 31, 2026
21 of 22 checks passed
mehabhalodiya pushed a commit to mehabhalodiya/hypershift that referenced this pull request Feb 25, 2026
This fix[1] include updates in the policy cloud-controller used by
KubeCloudControllerARN required by upstream CCM and exposed by OTE
when enabled in the OCP payload, exercised in hypershift through job
presubmit e2e-conformance (requires OTE binary[2]).

There is also a hotfix on ROSA managed policy used on CI while managed policies is
updated by AWS[3].

The policy ensure CCM AWS have enough permissions to deliver the feautre
to HCP - already added in upstream[2].

[1] https://issues.redhat.com/browse/OCPBUGS-65885
[2] https://issues.redhat.com/browse/OCPSTRAT-2743
openshift/origin#30525
`https://github.com/kubernetes/cloud-provider-aws/pull/1214`
[3] https://issues.redhat.com/browse/SREP-2895
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. 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. verified Signifies that the PR passed pre-merge verification criteria

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants