Skip to content

Bump kueue submodule to 92abdcbd3#2167

Open
kannon92 wants to merge 1 commit into
openshift:mainfrom
kannon92:bump-kueue-submodule-jul-10
Open

Bump kueue submodule to 92abdcbd3#2167
kannon92 wants to merge 1 commit into
openshift:mainfrom
kannon92:bump-kueue-submodule-jul-10

Conversation

@kannon92

@kannon92 kannon92 commented Jul 10, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Updates upstream/kueue/src submodule from ace3d7a2f to 92abdcbd33d0cbc63671ddf688bdd55355b57db3
  • Regenerates bindata/assets/kueue-operator/crds/crd-workloads.kueue.x-k8s.io-v1beta2.yaml via make submodule-workflow — Workload PodSet maxItems raised from 10 to 18 (upstream #12819)

Commits included

  • 92abdcbd3 Release team and release automation - sync release notes step. (#12848)
  • b39a67710 Increase Workload PodSet limit to 18 (#12819)
  • cdc242b3c chart: disable internal cert management when cert-manager is enabled (#12859)
  • d65602f14 kueueviz: fix CSWSH by removing permissive CheckOrigin (#12734)
  • 45d895f5d Use strong typing for scheduling equivalence hashes (#12871)
  • 792e21303 Don't admit workload when WaitingForReplacementPods=True. (#12768)
  • 61c3245af [Cleanup] Use structured errors in TAS Flavorassigner (#12679)
  • fd65bcdc5 Replace connecting/disconnected with single enum (#12837)
  • f5fea343a Mention incr dispatcher config in docs. (#12853)
  • e9ca10e50 KueueViz: Fix missing http server timeouts (#12590)
  • c77a0966a cleanup tls config unit tests (#12861)
  • bd2b4f75f Remove @tenzen-y CyberAgent, Inc contact information (#12860)
  • 0ab6a8c33 add curve support to kueue (#11832)
  • bf7118ca0 kueueviz: avoid per-workload pod lists in dashboard (#12777)
  • 239291343 Handle SchedulingEquivalenceHashing skipped workloads in UnadmittedWorkloadsObservability (#12821)
  • 8bf23d956 Fix the flaky perf tas test (#12840)
  • 7a83d66eb Deleting an entire block in job_test.go (#12850)
  • 5781b5763 add OnHold reason (#12812)
  • d3f122956 Test: Converting the remaining priorityclass with workloadpriorityclass in visibility_test.go (#12842)
  • da1706fba set empty underlying_cause for non QuotaReserved admission issues (#12814)
  • 9ed4d7dbc Increase CI build times (#12839)
  • 6473bbe19 Fix race condition in multikueue watcher establishment (#12824)
  • b133b72b5 Fix flaky LeaderWorkerSet restart policy tests by increasing timeout (#12831)
  • 1050ee04c build(deps-dev): bump vitest (#12816)
  • fc4a09f27 fix(test): ignore multiKueue teardown cleanup errors to prevent flakiness (#12811)
  • 1091f9286 Add e2e regression test for elasticJob ResourceFlavor injection (#12421)
  • e688ba1f5 feat: try to measure timings for individual bash commands (#12693)

Test plan

  • make build
  • make test-unit
  • CI

🤖 Generated with Claude Code

Summary by CodeRabbit

  • Chores
    • Updated the tracked upstream component revision to incorporate the latest upstream changes.

@openshift-ci openshift-ci Bot added the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Jul 10, 2026
@coderabbitai

coderabbitai Bot commented Jul 10, 2026

Copy link
Copy Markdown

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: Enterprise

Run ID: 32d17f6f-319d-49fe-bb50-cab37fbea45f

📥 Commits

Reviewing files that changed from the base of the PR and between 2f8c289 and 6170ed6.

📒 Files selected for processing (1)
  • upstream/kueue/src
✅ Files skipped from review due to trivial changes (1)
  • upstream/kueue/src

Walkthrough

The tracked upstream/kueue/src subproject revision changes from 92abdcbd... to cf778f69.... No other code, configuration, or logic changes are included.

Changes

Upstream revision update

Layer / File(s) Summary
Update upstream subproject revision
upstream/kueue/src
The subproject commit pointer is updated to cf778f69df02f6068cb35854cb4902f6f937b798.

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

Possibly related PRs

🚥 Pre-merge checks | ✅ 14 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Title check ⚠️ Warning The title names the old submodule revision, but the PR updates kueue to cf778f69df02f6068cb35854cb4902f6f937b798. Rename it to reflect the new kueue submodule commit, e.g. "Bump kueue submodule to cf778f69d".
✅ Passed checks (14 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
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.
Stable And Deterministic Test Names ✅ Passed The PR only changes the upstream/kueue/src submodule pointer; no test files or Ginkgo titles are modified in this repo.
Test Structure And Quality ✅ Passed PR only updates the upstream submodule pointer; no Ginkgo test files or test code changed in this repo, so the test-quality check is not applicable.
Microshift Test Compatibility ✅ Passed PR only updates a submodule pointer; no Ginkgo e2e tests were added or modified in the diff, so MicroShift-specific compatibility checks don’t apply.
Single Node Openshift (Sno) Test Compatibility ✅ Passed PR only updates the upstream/kueue/src gitlink; no Ginkgo e2e test files or SNO-sensitive test logic were added or changed.
Topology-Aware Scheduling Compatibility ✅ Passed This PR only updates the upstream/kueue/src gitlink; no deployment manifests, operator code, or controllers were changed in the parent repo.
Ote Binary Stdout Contract ✅ Passed Only the submodule gitlink changed; no tracked process-level code or suite setup was modified, so no stdout-to-stdout contract violation is introduced.
Ipv6 And Disconnected Network Test Compatibility ✅ Passed This PR only bumps the kueue submodule pointer; no new Ginkgo e2e tests or network-sensitive test code were added in the superproject diff.
No-Weak-Crypto ✅ Passed Diff only updates the kueue submodule gitlink; no weak-crypto, custom-crypto, or secret-comparison code changed.
Container-Privileges ✅ Passed PR diff only updates the submodule gitlink; no container/K8s manifest changes add privileged settings.
No-Sensitive-Data-In-Logs ✅ Passed Only a submodule SHA changed; the repo diff contains no logging code or sensitive-data exposure.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Comment @coderabbitai help to get the list of available commands.

@openshift-ci

openshift-ci Bot commented Jul 10, 2026

Copy link
Copy Markdown

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: kannon92

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 Jul 10, 2026
Updates upstream/kueue/src from ace3d7a2f to 92abdcbd33d0cbc63671ddf688bdd55355b57db3 and regenerates the workloads CRD (PodSet maxItems raised from 10 to 18).

Commits included:
- 92abdcbd3 Release team and release automation - sync release notes step. (#12848)
- b39a67710 Increase Workload PodSet limit to 18 (#12819)
- cdc242b3c chart: disable internal cert management when cert-manager is enabled (#12859)
- d65602f14 kueueviz: fix CSWSH by removing permissive CheckOrigin (#12734)
- 45d895f5d Use strong typing for scheduling equivalence hashes (#12871)
- 792e21303 Don't admit workload when WaitingForReplacementPods=True. (#12768)
- 61c3245af [Cleanup] Use structured errors in TAS Flavorassigner (#12679)
- fd65bcdc5 Replace connecting/disconnected with single enum (#12837)
- f5fea343a Mention incr dispatcher config in docs. (#12853)
- e9ca10e50 KueueViz: Fix missing http server timeouts (#12590)
- c77a0966a cleanup tls config unit tests (#12861)
- bd2b4f75f Remove @tenzen-y CyberAgent, Inc contact information (#12860)
- 0ab6a8c33 add curve support to kueue (#11832)
- bf7118ca0 kueueviz: avoid per-workload pod lists in dashboard (#12777)
- 239291343 Handle SchedulingEquivalenceHashing skipped workloads in UnadmittedWorkloadsObservability (#12821)
- 8bf23d956 Fix the flaky perf tas test (#12840)
- 7a83d66eb Deleting an entire block in job_test.go (#12850)
- 5781b5763 add OnHold reason (#12812)
- d3f122956 Test: Converting the remaining priorityclass with workloadpriorityclass in visibility_test.go (#12842)
- da1706fba set empty underlying_cause for non QuotaReserved admission issues (#12814)
- 9ed4d7dbc Increase CI build times (#12839)
- 6473bbe19 Fix race condition in multikueue watcher establishment (#12824)
- b133b72b5 Fix flaky LeaderWorkerSet restart policy tests by increasing timeout (#12831)
- 1050ee04c build(deps-dev): bump vitest (#12816)
- fc4a09f27 fix(test): ignore multiKueue teardown cleanup errors to prevent flakiness (#12811)
- 1091f9286 Add e2e regression test for elasticJob ResourceFlavor injection (#12421)
- e688ba1f5 feat: try to measure timings for individual bash commands (#12693)

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
@kannon92 kannon92 force-pushed the bump-kueue-submodule-jul-10 branch from 2f8c289 to 6170ed6 Compare July 10, 2026 19:11
@openshift-ci openshift-ci Bot removed the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Jul 10, 2026
@openshift-ci

openshift-ci Bot commented Jul 10, 2026

Copy link
Copy Markdown

@kannon92: 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/prow/test-e2e-ci-build-downstream-4-22 6170ed6 link true /test test-e2e-ci-build-downstream-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.

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.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant