Skip to content

[release-1.4]: update submodule to july 6 for 0.18 - #2152

Merged
openshift-merge-bot[bot] merged 1 commit into
openshift:release-1.4from
kannon92:sync-submodule-july-6
Jul 6, 2026
Merged

[release-1.4]: update submodule to july 6 for 0.18#2152
openshift-merge-bot[bot] merged 1 commit into
openshift:release-1.4from
kannon92:sync-submodule-july-6

Conversation

@kannon92

@kannon92 kannon92 commented Jul 6, 2026

Copy link
Copy Markdown
Contributor

Summary by CodeRabbit

  • Chores
    • Updated the bundled upstream component to a newer revision.

@openshift-ci openshift-ci Bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Jul 6, 2026
@coderabbitai

coderabbitai Bot commented Jul 6, 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: 6c94c490-5a50-46ad-b3f6-87a02d457474

📥 Commits

Reviewing files that changed from the base of the PR and between 501dfc1 and 9190ff5.

📒 Files selected for processing (1)
  • upstream/kueue/src

Walkthrough

This change updates the pinned commit reference for the upstream/kueue/src submodule from one revision to a newer one, without any other code or documentation modifications.

Changes

Submodule Update

Layer / File(s) Summary
Submodule pointer update
upstream/kueue/src
Bumps the vendored submodule reference from commit 8ef04af4... to b43f6827....

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

Related PRs: None found.

Suggested labels: None

Suggested reviewers: None

🚥 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 matches the main change: updating the vendored upstream submodule on the release-1.4 branch.
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 PR only updates the upstream/kueue/src submodule gitlink; no Ginkgo test titles were added or edited in this repo.
Test Structure And Quality ✅ Passed PR only updates the submodule gitlink; no test files or Ginkgo code changed in this repo diff.
Microshift Test Compatibility ✅ Passed The submodule bump adds controller logic and plain Go tests only; no new Ginkgo e2e tests or unsupported MicroShift APIs/features were added.
Single Node Openshift (Sno) Test Compatibility ✅ Passed PASS: The submodule bump only tweaks existing e2e helpers/tests; no new Ginkgo e2e specs or multi-node/SNO assumptions were added.
Topology-Aware Scheduling Compatibility ✅ Passed Only the kueue submodule pointer changed; upstream diff shows no new topology-unsafe anti-affinity, spread, control-plane selectors, or PDB/replica assumptions.
Ote Binary Stdout Contract ✅ Passed Only the submodule gitlink changed; upstream patch search showed no main/TestMain/BeforeSuite or stdout print calls in changed code.
Ipv6 And Disconnected Network Test Compatibility ✅ Passed PR only bumps the submodule gitlink; no Ginkgo e2e tests or IPv4/external-network code changed in the parent diff.
No-Weak-Crypto ✅ Passed Only the upstream submodule gitlink changed; no source files or weak-crypto patterns were introduced in the PR diff.
Container-Privileges ✅ Passed PR only updates the upstream submodule pointer; I found no privileged manifest flags in the changed tree.
No-Sensitive-Data-In-Logs ✅ Passed Only a submodule pointer changed; searched upstream patch shows no password/token/secret/email/session logging, and added logs use operational identifiers.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

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

@sohankunkerkar

Copy link
Copy Markdown
Member

openshift/release#81490 will make these upgrade jobs optional.

@sohankunkerkar sohankunkerkar left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/lgtm

@openshift-ci openshift-ci Bot added the lgtm Indicates that a PR is ready to be merged. label Jul 6, 2026
@openshift-ci

openshift-ci Bot commented Jul 6, 2026

Copy link
Copy Markdown

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: kannon92, sohankunkerkar

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:
  • OWNERS [kannon92,sohankunkerkar]

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
openshift-merge-bot Bot merged commit c8ae8be into openshift:release-1.4 Jul 6, 2026
22 of 30 checks passed
@openshift-ci

openshift-ci Bot commented Jul 6, 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/upgrade-from-4.18-e2e-ocp-upgrade-4-18-to-4-19-kueue-1-4 9190ff5 link false /test upgrade-from-4.18-e2e-ocp-upgrade-4-18-to-4-19-kueue-1-4

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. lgtm Indicates that a PR is ready to be merged.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants