Skip to content

kueue-operator: expand e2e test timeout to 4 hours - #83835

Open
redhat-chai-bot wants to merge 1 commit into
openshift:mainfrom
redhat-chai-bot:kueue-operator-e2e-timeout-4h
Open

kueue-operator: expand e2e test timeout to 4 hours#83835
redhat-chai-bot wants to merge 1 commit into
openshift:mainfrom
redhat-chai-bot:kueue-operator-e2e-timeout-4h

Conversation

@redhat-chai-bot

@redhat-chai-bot redhat-chai-bot commented Aug 21, 2026

Copy link
Copy Markdown
Contributor

Summary

Expand the timeout for all kueue-operator e2e tests to 4 hours on both main and release-1.4 branches.

Changes

  • Added timeout: 4h0m0s at the test definition level for all 20 test-e2e-* tests in each config file (40 tests total)
  • Preserved cluster_claim.timeout: 1h0m0s values (cluster acquisition wait time, separate concern)
  • Regenerated prowjob definitions via make ci-operator-config and make jobs

Files modified

  • ci-operator/config/openshift/kueue-operator/openshift-kueue-operator-main.yaml
  • ci-operator/config/openshift/kueue-operator/openshift-kueue-operator-release-1.4.yaml
  • 4 generated prowjob definition files

AI-generated. Review for accuracy.

@kannon92 requested in Slack thread

Summary by CodeRabbit

  • Increased the timeout for 40 Kueue operator end-to-end tests to 4 hours across the main and release-1.4 branches.
  • Updated OpenShift CI configurations and regenerated Prow job definitions.
  • Preserved the existing 1-hour cluster_claim.timeout values.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@coderabbitai

coderabbitai Bot commented Aug 21, 2026

Copy link
Copy Markdown
Contributor

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Repository YAML (base), Central YAML (inherited)

Review profile: CHILL

Plan: Pro Plus

Run ID: df6cf9fa-850c-4d06-96f5-614bc67e7f6a

📥 Commits

Reviewing files that changed from the base of the PR and between 6e1fc8d and 7e94981.

⛔ Files ignored due to path filters (4)
  • ci-operator/jobs/openshift/kueue-operator/openshift-kueue-operator-main-periodics.yaml is excluded by !ci-operator/jobs/**
  • ci-operator/jobs/openshift/kueue-operator/openshift-kueue-operator-main-presubmits.yaml is excluded by !ci-operator/jobs/**
  • ci-operator/jobs/openshift/kueue-operator/openshift-kueue-operator-release-1.4-periodics.yaml is excluded by !ci-operator/jobs/**
  • ci-operator/jobs/openshift/kueue-operator/openshift-kueue-operator-release-1.4-presubmits.yaml is excluded by !ci-operator/jobs/**
📒 Files selected for processing (2)
  • ci-operator/config/openshift/kueue-operator/openshift-kueue-operator-main.yaml
  • ci-operator/config/openshift/kueue-operator/openshift-kueue-operator-release-1.4.yaml

Included review availability: Your plan provides up to 2 included reviews per hour; 1 remains after this review.


Walkthrough

The OpenShift CI configurations for the Kueue operator now set a four-hour timeout on 20 e2e, disruptive, disconnected, and DRA test jobs across the main and release-1.4 branches.

Changes

Kueue e2e timeout configuration

Layer / File(s) Summary
Configure four-hour e2e job timeouts
ci-operator/config/openshift/kueue-operator/openshift-kueue-operator-main.yaml, ci-operator/config/openshift/kueue-operator/openshift-kueue-operator-release-1.4.yaml
The listed e2e job definitions now include timeout: 4h0m0s. No other job parameters changed.

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

Merge Risk: 🔵 Low · up to 7e949

The PR only extends e2e test timeouts and updates generated CI definitions, so it is broadly mergeable with owner awareness. Before merging, confirm that the required update step generated all 40 timeout changes consistently across both branches.

Suggested reviewers: pannagarao

🚥 Pre-merge checks | ✅ 15
✅ Passed checks (15 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 main change: increasing Kueue operator end-to-end test timeouts to four hours.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0 files. (2 skipped: 2 unsupported.)
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.
Stable And Deterministic Test Names ✅ Passed The PR diff only adds 4-hour CI timeout fields; it changes no Ginkgo It, Describe, Context, or When declarations, and existing job identifiers remain unchanged.
Test Structure And Quality ✅ Passed The commit changes only six YAML CI configuration files; no Ginkgo It blocks, setup, cleanup, waits, or assertions changed. This test-code check is not applicable.
Microshift Test Compatibility ✅ Passed The commit changes only Kueue CI timeout and generated prowjob configuration; it adds no Ginkgo tests or OpenShift API usage to assess.
Single Node Openshift (Sno) Test Compatibility ✅ Passed The exact PR diff changes only CI timeout fields in two config files and four generated job files; it adds no Ginkgo tests or multi-node assumptions.
Topology-Aware Scheduling Compatibility ✅ Passed HEAD^..HEAD changes only CI test/job timeout fields; no deployment manifests, operator code, controllers, or topology scheduling constraints were modified, so this check is inapplicable.
Ote Binary Stdout Contract ✅ Passed The PR changes only six YAML configuration/job files, adding 4-hour timeouts; it introduces no OTE binary or process-level stdout writes.
Ipv6 And Disconnected Network Test Compatibility ✅ Passed The HEAD^..HEAD diff changes only six YAML CI configuration files and adds timeout fields; it adds no Ginkgo tests or networking code, so this check is inapplicable.
No-Weak-Crypto ✅ Passed The complete diff adds only timeout: 4h0m0s fields and expands generated decoration_config; it introduces no weak-crypto algorithm, custom crypto, or secret comparison.
Container-Privileges ✅ Passed The PR diff adds only 4-hour timeout fields and generated decoration_config timeouts; no privileged, host namespace, SYS_ADMIN, root, or allowPrivilegeEscalation settings were introduced.
No-Sensitive-Data-In-Logs ✅ Passed The HEAD^..HEAD diff adds only 4h0m0s timeout settings and generated decoration_config entries; it adds no logging, secrets, credentials, or sensitive-data indicators.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

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 requested review from PannagaRao and cpmeadors August 21, 2026 18:02
@openshift-ci

openshift-ci Bot commented Aug 21, 2026

Copy link
Copy Markdown
Contributor

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: redhat-chai-bot
Once this PR has been reviewed and has the lgtm label, please assign rphillips for approval. For more information see the Code Review Process.

The full list of commands accepted by this bot can be found 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-merge-bot

Copy link
Copy Markdown
Contributor

[REHEARSALNOTIFIER]
@redhat-chai-bot: the pj-rehearse plugin accommodates running rehearsal tests for the changes in this PR. Expand 'Interacting with pj-rehearse' for usage details. The following rehearsable tests have been affected by this change:

Test name Repo Type Reason
pull-ci-openshift-kueue-operator-main-test-e2e-ci-build-downstream-4-22 openshift/kueue-operator presubmit Ci-operator config changed
pull-ci-openshift-kueue-operator-main-test-e2e-ci-build-upstream-4-22 openshift/kueue-operator presubmit Ci-operator config changed
pull-ci-openshift-kueue-operator-main-test-e2e-ci-build-upstream-dra-4-22 openshift/kueue-operator presubmit Ci-operator config changed
pull-ci-openshift-kueue-operator-main-test-e2e-downstream-4-20-disruptive openshift/kueue-operator presubmit Ci-operator config changed
pull-ci-openshift-kueue-operator-main-test-e2e-downstream-4-21-disruptive openshift/kueue-operator presubmit Ci-operator config changed
pull-ci-openshift-kueue-operator-main-test-e2e-dra-gpu-4-22 openshift/kueue-operator presubmit Ci-operator config changed
pull-ci-openshift-kueue-operator-release-1.4-test-e2e-ci-build-downstream-4-22 openshift/kueue-operator presubmit Ci-operator config changed
pull-ci-openshift-kueue-operator-release-1.4-test-e2e-ci-build-upstream-4-22 openshift/kueue-operator presubmit Ci-operator config changed
pull-ci-openshift-kueue-operator-release-1.4-test-e2e-ci-build-upstream-dra-4-22 openshift/kueue-operator presubmit Ci-operator config changed
pull-ci-openshift-kueue-operator-release-1.4-test-e2e-downstream-4-20-disruptive openshift/kueue-operator presubmit Ci-operator config changed
pull-ci-openshift-kueue-operator-release-1.4-test-e2e-downstream-4-21-disruptive openshift/kueue-operator presubmit Ci-operator config changed
pull-ci-openshift-kueue-operator-release-1.4-test-e2e-dra-gpu-4-22 openshift/kueue-operator presubmit Ci-operator config changed
periodic-ci-openshift-kueue-operator-main-test-e2e-4-19 N/A periodic Ci-operator config changed
periodic-ci-openshift-kueue-operator-main-test-e2e-downstream-4-20 N/A periodic Ci-operator config changed
periodic-ci-openshift-kueue-operator-release-1.4-test-e2e-downstream-4-20 N/A periodic Ci-operator config changed
periodic-ci-openshift-kueue-operator-release-1.4-test-e2e-downstream-4-20-disruptive N/A periodic Ci-operator config changed
periodic-ci-openshift-kueue-operator-release-1.4-test-e2e-dra-gpu-4-22 N/A periodic Ci-operator config changed
periodic-ci-openshift-kueue-operator-main-test-e2e-upstream-4-22 N/A periodic Ci-operator config changed
periodic-ci-openshift-kueue-operator-main-test-e2e-4-22-hypershift N/A periodic Ci-operator config changed
periodic-ci-openshift-kueue-operator-release-1.4-test-e2e-upstream-4-22 N/A periodic Ci-operator config changed
periodic-ci-openshift-kueue-operator-release-1.4-test-e2e-upstream-4-21 N/A periodic Ci-operator config changed
periodic-ci-openshift-kueue-operator-release-1.4-test-e2e-ci-build-upstream-dra-4-22 N/A periodic Ci-operator config changed
periodic-ci-openshift-kueue-operator-main-test-e2e-downstream-4-21-disruptive N/A periodic Ci-operator config changed
periodic-ci-openshift-kueue-operator-main-test-e2e-4-22-arm64 N/A periodic Ci-operator config changed
periodic-ci-openshift-kueue-operator-release-1.4-test-e2e-downstream-4-21 N/A periodic Ci-operator config changed

A total of 48 jobs have been affected by this change. The above listing is non-exhaustive and limited to 25 jobs.

A full list of affected jobs can be found here
Prior to this PR being merged, you will need to either run and acknowledge or opt to skip these rehearsals.

Interacting with pj-rehearse

Comment: /pj-rehearse to run up to 5 rehearsals
Comment: /pj-rehearse skip to opt-out of rehearsals
Comment: /pj-rehearse {test-name}, with each test separated by a space, to run one or more specific rehearsals
Comment: /pj-rehearse more to run up to 10 rehearsals
Comment: /pj-rehearse max to run up to 25 rehearsals
Comment: /pj-rehearse auto-ack to run up to 5 rehearsals, and add the rehearsals-ack label on success
Comment: /pj-rehearse list to get an up-to-date list of affected jobs
Comment: /pj-rehearse abort to abort all active rehearsals
Comment: /pj-rehearse network-access-allowed to allow rehearsals of tests that have the restrict_network_access field set to false. This must be executed by an openshift org member who is not the PR author

Once you are satisfied with the results of the rehearsals, comment: /pj-rehearse ack to unblock merge. When the rehearsals-ack label is present on your PR, merge will no longer be blocked by rehearsals.
If you would like the rehearsals-ack label removed, comment: /pj-rehearse reject to re-block merging.

@kannon92

Copy link
Copy Markdown
Contributor

/pj-rehearse

@openshift-merge-bot

Copy link
Copy Markdown
Contributor

@kannon92: now processing your pj-rehearse request. Please allow up to 10 minutes for jobs to trigger or cancel.

@openshift-ci

openshift-ci Bot commented Aug 21, 2026

Copy link
Copy Markdown
Contributor

@redhat-chai-bot: The following test 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/rehearse/openshift/kueue-operator/release-1.4/test-e2e-ci-build-upstream-4-22 7e94981 link unknown /pj-rehearse pull-ci-openshift-kueue-operator-release-1.4-test-e2e-ci-build-upstream-4-22

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.

@redhat-chai-bot

Copy link
Copy Markdown
Contributor Author

/retest-required


AI-generated. Review for accuracy.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants