Skip to content

[v0.36.0] release prep: bump k8s/go deps, manifests, docs, and CI matrix#1874

Open
a7i wants to merge 3 commits into
kubernetes-sigs:masterfrom
a7i:descheduler-v0.36.0
Open

[v0.36.0] release prep: bump k8s/go deps, manifests, docs, and CI matrix#1874
a7i wants to merge 3 commits into
kubernetes-sigs:masterfrom
a7i:descheduler-v0.36.0

Conversation

@a7i
Copy link
Copy Markdown
Contributor

@a7i a7i commented May 16, 2026

This PR prepares descheduler for the v0.36.0 release by combining dependency/toolchain updates with release-facing docs and manifest alignment in one change set.

What changed

  • Bumped Kubernetes dependency stack to 1.36 line and updated related Go modules.
  • Updated Go toolchain/runtime settings to 1.26 and aligned build/test scripts.
  • Updated CI manifest workflow matrix to target:
    • Kubernetes v1.36.1
    • descheduler v0.36.0
    • kind v0.35.1
  • Updated release manifests to use descheduler image v0.36.0:
    • Deployment
    • Job
    • CronJob
  • Updated release-facing documentation for v0.36:
    • Added v0.36 docs branch mapping
    • Added v0.36 compatibility matrix entry
    • Added v0.36 container image entry in user guide
  • Included generated/API updates related to new eviction-per-namespace policy typing and conversion/deepcopy updates.

Why

This follows the same release-prep pattern used in:

and keeps all release-cut prep in one PR for easier review and traceability.

Validation

  • Verified YAML/doc edits are structurally clean in editor checks.
  • Local tree is internally consistent for release versions across workflow, manifests, and docs.

Suggested reviewer focus

  • Dependency/toolchain bump impact and generated file diffs.
  • API type/conversion changes around namespace eviction limits.
  • Release version consistency across CI, manifests, and docs.

Release impact

  • No behavioral feature additions intended.
  • Primary impact is version alignment and release readiness for v0.36.0.

Copilot AI review requested due to automatic review settings May 16, 2026 02:18
Copilot AI reviewed May 16, 2026
View reviewed changes
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot wasn't able to review this pull request because it exceeds the maximum number of files (300). Try reducing the number of changed files and requesting a review from Copilot again.

@k8s-ci-robot k8s-ci-robot added cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files. labels May 16, 2026
@k8s-ci-robot
Copy link
Copy Markdown
Contributor

k8s-ci-robot commented May 16, 2026

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
Once this PR has been reviewed and has the lgtm label, please assign ricardomaraschini 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

@a7i a7i force-pushed the descheduler-v0.36.0 branch from 1020379 to 4b309fe Compare May 16, 2026 18:56
Copilot AI review requested due to automatic review settings May 17, 2026 01:39
@a7i a7i force-pushed the descheduler-v0.36.0 branch from 4b309fe to b1686ea Compare May 17, 2026 01:39
Copilot AI reviewed May 17, 2026
View reviewed changes
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot wasn't able to review this pull request because it exceeds the maximum number of files (300). Try reducing the number of changed files and requesting a review from Copilot again.

Copilot AI review requested due to automatic review settings May 17, 2026 01:55
Copilot AI reviewed May 17, 2026
View reviewed changes
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot wasn't able to review this pull request because it exceeds the maximum number of files (300). Try reducing the number of changed files and requesting a review from Copilot again.

@a7i a7i force-pushed the descheduler-v0.36.0 branch 2 times, most recently from be07fd4 to c5caedb Compare May 17, 2026 02:05
Copilot AI review requested due to automatic review settings May 17, 2026 02:05
Copilot AI reviewed May 17, 2026
View reviewed changes
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot wasn't able to review this pull request because it exceeds the maximum number of files (300). Try reducing the number of changed files and requesting a review from Copilot again.

@a7i
Copy link
Copy Markdown
Contributor Author

a7i commented May 17, 2026

/retest-required

@a7i a7i force-pushed the descheduler-v0.36.0 branch from aef6ccc to e860517 Compare May 17, 2026 14:14
Copilot AI review requested due to automatic review settings May 17, 2026 14:14
Copilot AI reviewed May 17, 2026
View reviewed changes
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot wasn't able to review this pull request because it exceeds the maximum number of files (300). Try reducing the number of changed files and requesting a review from Copilot again.

@a7i a7i force-pushed the descheduler-v0.36.0 branch from e860517 to d4f3c20 Compare May 17, 2026 14:17
Copilot AI review requested due to automatic review settings May 17, 2026 14:22
@a7i a7i force-pushed the descheduler-v0.36.0 branch from d4f3c20 to 09af9a0 Compare May 17, 2026 14:22
Copilot AI reviewed May 17, 2026
View reviewed changes
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot wasn't able to review this pull request because it exceeds the maximum number of files (300). Try reducing the number of changed files and requesting a review from Copilot again.

@a7i a7i force-pushed the descheduler-v0.36.0 branch from 09af9a0 to aae77a8 Compare May 17, 2026 14:23
@a7i
Copy link
Copy Markdown
Contributor Author

a7i commented May 17, 2026

/cc @jklaw90 @ingvagabund

@k8s-ci-robot k8s-ci-robot requested a review from ingvagabund May 17, 2026 19:02
@k8s-ci-robot k8s-ci-robot requested a review from jklaw90 May 17, 2026 19:02
@jklaw90
Copy link
Copy Markdown
Contributor

jklaw90 commented May 17, 2026

/lgtm

@k8s-ci-robot k8s-ci-robot added the lgtm "Looks good to me", indicates that a PR is ready to be merged. label May 17, 2026
@ingvagabund
Copy link
Copy Markdown
Contributor

ingvagabund commented May 18, 2026

/test all

@a7i
Copy link
Copy Markdown
Contributor Author

a7i commented May 18, 2026

/retest-required

@a7i
Copy link
Copy Markdown
Contributor Author

a7i commented May 18, 2026

TestKubeClientSandboxRestoreEvictedPods seems to be a flaky test and needs fixing

@a7i
Copy link
Copy Markdown
Contributor Author

a7i commented May 18, 2026

/retest-required

@k8s-ci-robot
Copy link
Copy Markdown
Contributor

k8s-ci-robot commented May 18, 2026 •
edited
Loading

@a7i: 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
pull-descheduler-test-e2e-k8s-master-1-36 aae77a8 link true /test pull-descheduler-test-e2e-k8s-master-1-36

Full PR test history. Your PR dashboard. Please help us cut down on flakes by linking to an open issue when you hit one in your PR.

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.

@ingvagabund
Copy link
Copy Markdown
Contributor

ingvagabund commented May 18, 2026

/retest-required

@a7i a7i force-pushed the descheduler-v0.36.0 branch from aae77a8 to 89e3199 Compare May 18, 2026 14:37
Copilot AI review requested due to automatic review settings May 18, 2026 14:37
@k8s-ci-robot k8s-ci-robot removed the lgtm "Looks good to me", indicates that a PR is ready to be merged. label May 18, 2026
@k8s-ci-robot
Copy link
Copy Markdown
Contributor

k8s-ci-robot commented May 18, 2026

New changes are detected. LGTM label has been removed.

Copilot AI reviewed May 18, 2026
View reviewed changes
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot wasn't able to review this pull request because it exceeds the maximum number of files (300). Try reducing the number of changed files and requesting a review from Copilot again.

@a7i
Copy link
Copy Markdown
Contributor Author

a7i commented May 18, 2026

Added a commit (wait) to address the test flakiness

@ingvagabund ingvagabund mentioned this pull request May 18, 2026
Open
14 tasks
@ingvagabund
Copy link
Copy Markdown
Contributor

ingvagabund commented May 18, 2026

@a7i #1878 might help. Up to you which fix you will choose.

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

Reviewers

Copilot code review Copilot Copilot left review comments

@googs1025 googs1025 Awaiting requested review from googs1025

@knelasevero knelasevero Awaiting requested review from knelasevero

@ingvagabund ingvagabund Awaiting requested review from ingvagabund

@jklaw90 jklaw90 Awaiting requested review from jklaw90

Assignees

@jklaw90 jklaw90

Labels

cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@a7i @k8s-ci-robot @jklaw90 @ingvagabund