OCM-23803: migrate cs-osd-ccs-gcp-ad-staging-main FVT to Prow#78692
OCM-23803: migrate cs-osd-ccs-gcp-ad-staging-main FVT to Prow#78692dustman9000 wants to merge 1 commit intoopenshift:mainfrom
Conversation
|
@dustman9000: This pull request references OCM-23803 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 task to target the "5.0.0" version, but no target version was set. DetailsIn response to this:
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. |
|
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:
WalkthroughAdds a ci-operator YAML to configure a periodic OpenShift ROSA e2e job for the ChangesCI Job Configuration Addition
Estimated code review effort🎯 2 (Simple) | ⏱️ ~12 minutes 🚥 Pre-merge checks | ✅ 12✅ Passed checks (12 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
|
/pj-rehearse periodic-ci-openshift-online-rosa-e2e-main-ocm-fvt-periodic-cs-osd-ccs-gcp-ad-staging-main |
|
@dustman9000: now processing your pj-rehearse request. Please allow up to 10 minutes for jobs to trigger or cancel. |
|
/lgtm |
de64339 to
00342ac
Compare
|
/pj-rehearse periodic-ci-openshift-online-rosa-e2e-main-ocm-fvt-periodic-cs-osd-ccs-gcp-ad-staging-main |
|
@dustman9000: now processing your pj-rehearse request. Please allow up to 10 minutes for jobs to trigger or cancel. |
|
/lgtm |
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: dustman9000, jfrazierRedHat, tinaafitz 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 |
|
/pj-rehearse periodic-ci-openshift-online-rosa-e2e-main-ocm-fvt-periodic-cs-osd-ccs-gcp-ad-staging-main |
|
@dustman9000: now processing your pj-rehearse request. Please allow up to 10 minutes for jobs to trigger or cancel. |
|
/pj-rehearse ack |
|
@dustman9000: now processing your pj-rehearse request. Please allow up to 10 minutes for jobs to trigger or cancel. |
00342ac to
b5b1adf
Compare
|
New changes are detected. LGTM label has been removed. |
There was a problem hiding this comment.
Actionable comments posted: 1
🧹 Nitpick comments (1)
ci-operator/config/openshift-online/rosa-e2e/openshift-online-rosa-e2e-main__ocm-fvt-osd-gcp-staging.yaml (1)
29-44: ⚡ Quick winAdd strict shell flags in the heredoc setup block.
At Line 29, adding fail-fast options will surface missing credential files or sourcing errors immediately instead of failing later inside
ocmtest.Suggested fix
commands: | env -i bash --norc --noprofile << 'EOF' > /tmp/podman.env + set -euo pipefail export AWS_SHARED_CREDENTIALS_FILE=/credentials/aws-cred export SHARED_VPC_AWS_SHARED_CREDENTIALS_FILE=/credentials/aws-shared-vpc-credentials export ENABLE_JIRA_REPORTING=true🤖 Prompt for 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. In `@ci-operator/config/openshift-online/rosa-e2e/openshift-online-rosa-e2e-main__ocm-fvt-osd-gcp-staging.yaml` around lines 29 - 44, The heredoc that writes /tmp/podman.env (the block beginning with "env -i bash --norc --noprofile << 'EOF'") should enable strict shell flags so failures while exporting or sourcing credentials surface immediately; update the heredoc contents to set -euo pipefail (and optionally IFS=$'\n\t') near the top before exporting AWS_SHARED_CREDENTIALS_FILE and before the source lines (/usr/local/cs-qe-credentials/ocm-tokens and /usr/local/cs-qe-credentials/jira-cred) so missing files or errors cause an immediate exit rather than failing later in ocmtest.
🤖 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/config/openshift-online/rosa-e2e/openshift-online-rosa-e2e-main__ocm-fvt-osd-gcp-staging.yaml`:
- Line 57: The cron schedule currently set as "0 8 * * *" is off by one hour;
update the cron field in the YAML (the cron: line in the
openshift-online-rosa-e2e config) to "0 9 * * *" so it runs at 09:00 UTC daily
to match the migration objective. Ensure the single cron: entry is updated and
any related documentation or comments reflecting the schedule are adjusted to
"09:00 UTC" if present.
---
Nitpick comments:
In
`@ci-operator/config/openshift-online/rosa-e2e/openshift-online-rosa-e2e-main__ocm-fvt-osd-gcp-staging.yaml`:
- Around line 29-44: The heredoc that writes /tmp/podman.env (the block
beginning with "env -i bash --norc --noprofile << 'EOF'") should enable strict
shell flags so failures while exporting or sourcing credentials surface
immediately; update the heredoc contents to set -euo pipefail (and optionally
IFS=$'\n\t') near the top before exporting AWS_SHARED_CREDENTIALS_FILE and
before the source lines (/usr/local/cs-qe-credentials/ocm-tokens and
/usr/local/cs-qe-credentials/jira-cred) so missing files or errors cause an
immediate exit rather than failing later in ocmtest.
🪄 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: Enterprise
Run ID: 56c768fa-ca0a-44c9-b39b-88676319bcf7
📒 Files selected for processing (1)
ci-operator/config/openshift-online/rosa-e2e/openshift-online-rosa-e2e-main__ocm-fvt-osd-gcp-staging.yaml
ea02522 to
74c4f70
Compare
74c4f70 to
992f29d
Compare
|
[REHEARSALNOTIFIER]
Interacting with pj-rehearseComment: Once you are satisfied with the results of the rehearsals, comment: |
|
@dustman9000: The following tests failed, say
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. |
Summary
cs-osd-ccs-gcp-ad-staging-mainOCM FVT jobDetails
0 9 * * *(09:00 UTC daily)cms-gcp-ccs-gating-testTest plan
make jobsgenerates periodic job correctlySummary by CodeRabbit