Skip to content

OSAC-2162: Add periodic schedule to CaaS Netris E2E - #328

Merged
omer-vishlitzky merged 1 commit into
mainfrom
feat/netris-caas-periodic
Aug 9, 2026
Merged

omer-vishlitzky merged 1 commit into
mainfrom
feat/netris-caas-periodic

Conversation

@danmanor

@danmanor danmanor commented Aug 6, 2026 •

Copy link
Copy Markdown
Contributor

Summary

  • Add a schedule trigger (every 3 hours) to the CaaS Netris E2E caller workflow to measure success rate over time.

Assisted-by: Claude Code noreply@anthropic.com

Summary by CodeRabbit

  • Chores
    • Added an automated schedule to run the full-install end-to-end workflow every three hours.

@openshift-ci-robot

openshift-ci-robot commented Aug 6, 2026 •

Copy link
Copy Markdown

@danmanor: This pull request references OSAC-2162 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 the "5.0.0" version, but no target version was set.

Details

In response to this:

Summary

  • Add a schedule trigger (every 3 hours) to the CaaS Netris E2E caller workflow to measure success rate over time.

Test plan

  • Verify the workflow appears in the Actions tab with the schedule trigger
  • Confirm the first scheduled run fires and launches an EC2 instance correctly
  • Monitor runs over 24h to verify ~8 runs/day cadence

Assisted-by: Claude Code noreply@anthropic.com

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.

@coderabbitai

coderabbitai Bot commented Aug 6, 2026 •

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Repository: osac-project/coderabbit/.coderabbit.yaml

Review profile: CHILL

Plan: Pro Plus

Run ID: ba3d054a-51c2-4ca2-b4cd-beaf679e9380

📥 Commits

Reviewing files that changed from the base of the PR and between 555a68b and 6a0e86d.

📒 Files selected for processing (1)
  • .github/workflows/e2e-caas-netris-full-install-caller.yml

Walkthrough

The E2E CaaS Netris full-install workflow now runs automatically every three hours through a scheduled GitHub Actions trigger.

Changes

Netris E2E scheduling

Layer / File(s) Summary
Add recurring workflow trigger
.github/workflows/e2e-caas-netris-full-install-caller.yml
Adds a cron schedule that starts the workflow every three hours.

Estimated code review effort: 1 (Trivial) | ~2 minutes

Possibly related PRs

Suggested reviewers: rgolangh, eliorerz

🚥 Pre-merge checks | ✅ 11
✅ Passed checks (11 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly and concisely describes the added periodic schedule for the CaaS Netris E2E workflow.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
No-Hardcoded-Secrets ✅ Passed The commit adds only the schedule trigger and cron expression; no added line contains a secret, credential, token, password, key, or credential-shaped literal.
No-Weak-Crypto ✅ Passed The PR adds only a three-hour cron trigger; added lines contain no weak crypto algorithm, crypto implementation, or secret/token comparison indicators.
No-Injection-Vectors ✅ Passed The patch only adds the literal cron trigger 0 */3 * * *; added lines contain no SQL, shell, eval/exec, unsafe YAML, pickle, os.system, or HTML injection.
Container-Privileges ✅ Passed The PR changes only the workflow trigger by adding a three-hour cron schedule. It adds no privileged, hostPID, hostNetwork, hostIPC, SYS_ADMIN, or allowPrivilegeEscalation setting.
No-Sensitive-Data-In-Logs ✅ Passed The PR adds only a cron trigger; the caller has no logging commands, and the invoked secret-fetch paths write secrets to protected files without printing them.
Ai-Attribution ✅ Passed The PR commit names Claude Code and includes an Assisted-by trailer; the PR commit has no AI Co-Authored-By trailer.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch feat/netris-caas-periodic

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.

@openshift-ci

openshift-ci Bot commented Aug 6, 2026

Copy link
Copy Markdown

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: danmanor, eliorerz

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

@danmanor

danmanor commented Aug 7, 2026

Copy link
Copy Markdown
Contributor Author

/hold

@openshift-ci openshift-ci Bot added do-not-merge/hold Block merge until the label is removed needs-rebase labels Aug 7, 2026
Run every 3 hours to measure success rate of the netris-caas flow.

Assisted-by: Claude Code <noreply@anthropic.com>
Signed-off-by: Dan Manor <dmanor@redhat.com>
@danmanor
danmanor force-pushed the feat/netris-caas-periodic branch from 86cb42d to 6a0e86d Compare August 9, 2026 11:59
@openshift-ci openshift-ci Bot removed the lgtm label Aug 9, 2026
@openshift-ci

openshift-ci Bot commented Aug 9, 2026

Copy link
Copy Markdown

New changes are detected. LGTM label has been removed.

@danmanor danmanor added the lgtm label Aug 9, 2026
@danmanor

danmanor commented Aug 9, 2026

Copy link
Copy Markdown
Contributor Author

/unhold

@openshift-ci openshift-ci Bot removed the do-not-merge/hold Block merge until the label is removed label Aug 9, 2026
@github-actions

github-actions Bot commented Aug 9, 2026

Copy link
Copy Markdown

🚨 Credential leak found in e2e run logs/artifacts

Workflow: E2E BMaaS Full Install
E2E run: 31312176583
Scan run: Scan workflow logs
Redacted logs/artifacts: redacted-logs-31312176583

Tainted raw logs and/or artifacts for that e2e run were deleted to close the exposure window.

Inspect the redacted artifact (and the scan job summary) for finding details. Rotate any real credentials.

@github-actions

github-actions Bot commented Aug 9, 2026

Copy link
Copy Markdown

🚨 Credential leak found in e2e run logs/artifacts

Workflow: E2E VMaaS Full Install
E2E run: 31312176640
Scan run: Scan workflow logs
Redacted logs/artifacts: redacted-logs-31312176640

Tainted raw logs and/or artifacts for that e2e run were deleted to close the exposure window.

Inspect the redacted artifact (and the scan job summary) for finding details. Rotate any real credentials.

@danmanor

danmanor commented Aug 9, 2026

Copy link
Copy Markdown
Contributor Author

/retest

@github-actions

github-actions Bot commented Aug 9, 2026

Copy link
Copy Markdown

Re-triggered failed runs:

  • E2E CaaS Full Install (#31312176589)

@github-actions

github-actions Bot commented Aug 9, 2026

Copy link
Copy Markdown

🚨 Credential leak found in e2e run logs/artifacts

Workflow: E2E CaaS Full Install
E2E run: 31312176589
Scan run: Scan workflow logs
Redacted logs/artifacts: redacted-logs-31312176589

Tainted raw logs and/or artifacts for that e2e run were deleted to close the exposure window.

Inspect the redacted artifact (and the scan job summary) for finding details. Rotate any real credentials.

@omer-vishlitzky
omer-vishlitzky merged commit 625141b into main Aug 9, 2026
6 of 7 checks passed

This branch was previously deployed

1 inactive deployment
e2e-test — 6a0e86da Deployed Aug 9, 2026 by danmanor via e2e-caas-full-install / e2e #359
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants