feat: run CAPA smoke test suite - #82477
Conversation
|
Note Reviews pausedIt looks like this branch is under active development. To avoid overwhelming you with review comments due to an influx of new commits, CodeRabbit has automatically paused this review. You can configure this behavior by changing the Use the following commands to manage reviews:
Use the checkboxes below for quick actions:
WalkthroughThe ROSA E2E ChangesROSA E2E capa workflow
Estimated code review effort: 2 (Simple) | ~10 minutes Suggested labels: Suggested reviewers: 🚥 Pre-merge checks | ✅ 14 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (14 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
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. Comment |
|
/pj-rehearse periodic-ci-openshift-online-rosa-e2e-main_capa-e2e-capa-e2e |
|
@andclt: now processing your pj-rehearse request. Please allow up to 10 minutes for jobs to trigger or cancel. |
41d282f to
35ca52e
Compare
|
/pj-rehearse periodic-ci-openshift-online-rosa-e2e-main_capa-e2e-capa-e2e |
|
@andclt: now processing your pj-rehearse request. Please allow up to 10 minutes for jobs to trigger or cancel. |
35ca52e to
af31e9b
Compare
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 2
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@ci-operator/step-registry/rosa/e2e/capa/rosa-e2e-capa-commands.sh`:
- Around line 50-51: Update both curl downloads in the Helm setup flow to use
bounded connection and total timeouts, and add limited retries for transient
failures. Apply the same timeout and retry options to the archive and its
checksum downloads while preserving their existing URLs and output paths.
- Around line 50-55: Update the Helm download verification in the rosa e2e setup
so the downloaded checksum file is not the sole trust anchor. Pin the expected
archive SHA-256 in repository-controlled configuration or verify Helm’s official
detached GPG signature, then validate the downloaded archive against that
trusted value before continuing.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Repository YAML (base), Central YAML (inherited)
Review profile: CHILL
Plan: Pro Plus
Run ID: ecb1c5b1-c534-422e-bd0b-4dbf7b4fba1c
📒 Files selected for processing (2)
ci-operator/config/openshift-online/rosa-e2e/openshift-online-rosa-e2e-main_capa-e2e.yamlci-operator/step-registry/rosa/e2e/capa/rosa-e2e-capa-commands.sh
🚧 Files skipped from review as they are similar to previous changes (1)
- ci-operator/config/openshift-online/rosa-e2e/openshift-online-rosa-e2e-main_capa-e2e.yaml
af31e9b to
87558a8
Compare
|
/pj-rehearse periodic-ci-openshift-online-rosa-e2e-main_capa-e2e-capa-e2e |
|
@andclt: now processing your pj-rehearse request. Please allow up to 10 minutes for jobs to trigger or cancel. |
3f90fca to
caf4b7f
Compare
|
/pj-rehearse periodic-ci-openshift-online-rosa-e2e-main_capa-e2e-capa-e2e |
|
@andclt: now processing your pj-rehearse request. Please allow up to 10 minutes for jobs to trigger or cancel. |
caf4b7f to
7a87a61
Compare
|
/pj-rehearse periodic-ci-openshift-online-rosa-e2e-main_capa-e2e-capa-e2e |
|
@andclt: now processing your pj-rehearse request. Please allow up to 10 minutes for jobs to trigger or cancel. |
7a87a61 to
ee3d8e2
Compare
|
/pj-rehearse periodic-ci-openshift-online-rosa-e2e-main_capa-e2e-capa-e2e |
|
@andclt: now processing your pj-rehearse request. Please allow up to 10 minutes for jobs to trigger or cancel. |
ee3d8e2 to
71c7a70
Compare
|
/pj-rehearse periodic-ci-openshift-online-rosa-e2e-main_capa-e2e-capa-e2e |
|
@andclt: now processing your pj-rehearse request. Please allow up to 10 minutes for jobs to trigger or cancel. |
|
/pj-rehearse periodic-ci-openshift-online-rosa-e2e-main_capa-e2e-capa-e2e |
|
@andclt: now processing your pj-rehearse request. Please allow up to 10 minutes for jobs to trigger or cancel. |
|
/pj-rehearse periodic-ci-openshift-online-rosa-e2e-main_capa-e2e-capa-e2e |
|
@andclt: your |
|
@andclt: now processing your pj-rehearse request. Please allow up to 10 minutes for jobs to trigger or cancel. |
|
/pj-rehearse periodic-ci-openshift-online-rosa-e2e-main_capa-e2e-capa-e2e |
|
@andclt: now processing your pj-rehearse request. Please allow up to 10 minutes for jobs to trigger or cancel. |
|
/pj-rehearse periodic-ci-openshift-online-rosa-e2e-main_capa-e2e-capa-e2e |
|
@andclt: now processing your pj-rehearse request. Please allow up to 10 minutes for jobs to trigger or cancel. |
71c7a70 to
712d7be
Compare
|
/pj-rehearse periodic-ci-openshift-online-rosa-e2e-main_capa-e2e-capa-e2e |
|
@andclt: now processing your pj-rehearse request. Please allow up to 10 minutes for jobs to trigger or cancel. |
712d7be to
c5cfb22
Compare
|
/pj-rehearse periodic-ci-openshift-online-rosa-e2e-main_capa-e2e-capa-e2e |
|
@andclt: now processing your pj-rehearse request. Please allow up to 10 minutes for jobs to trigger or cancel. |
c5cfb22 to
577545e
Compare
|
/pj-rehearse periodic-ci-openshift-online-rosa-e2e-main_capa-e2e-capa-e2e |
|
@andclt: now processing your pj-rehearse request. Please allow up to 10 minutes for jobs to trigger or cancel. |
|
[REHEARSALNOTIFIER]
Interacting with pj-rehearseComment: Once you are satisfied with the results of the rehearsals, comment: |
|
@andclt: all tests passed! Full PR test history. Your PR dashboard. DetailsInstructions 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. |
|
/lgtm |
|
@dustman9000: now processing your pj-rehearse request. Please allow up to 10 minutes for jobs to trigger or cancel. |
openshift-merge-bot
Bot
added
the
rehearsals-ack
|
/lgtm |
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: andclt, dustman9000, lucasponce The full list of commands accepted by this bot can be found here. The pull request process is described here DetailsNeeds approval from an approver in each of these files:
Approvers can indicate their approval by writing |
openshift-ci
Bot
added
the
approved
![openshift-prow[bot]](https://github.githubassets.com/images/gravatars/gravatar-user-420.png){kind=avatar}
3 participants
Summary
Summary by CodeRabbit
capa-e2eCI job configuration to run the smoke E2E subset by settingTEST_SUITE: --tag smokein the job’senv(so the job executes tests viarun-test-suite.pyusing that selector).ci-operator/step-registry/rosa/e2e/capa/rosa-e2e-capa-commands.shto read AWS credentials from the mountedawsEncodedCredentials/awsAccountIdfiles and export them asAWS_ACCESS_KEY_ID,AWS_SECRET_ACCESS_KEY, andAWS_ACCOUNT_ID(replacing the priorAWS_B64ENCODED_CREDENTIALSexport)./tmp/binand prepends it toPATHwhenhelmis not already available, then reportshelm version --short.