Skip to content

chore: Upgrade k8s.io core libs to 1.36 (0.36.z) - #3387

Merged
knative-prow[bot] merged 4 commits into
knative:mainfrom
dsimansk:pr/kube-136
Oct 9, 2026
Merged

knative-prow[bot] merged 4 commits into
knative:mainfrom
dsimansk:pr/kube-136

Conversation

@dsimansk

@dsimansk dsimansk commented Aug 27, 2026 •

Copy link
Copy Markdown
Contributor

Changes

  • chore: Upgrade k8s.io core libs to 1.36/0.36.z
Upgrade k8s.io core libs to 1.36/0.36.z

/cc @dprotaso

/hold

@knative-prow
knative-prow Bot requested a review from dprotaso August 27, 2026 12:41
@knative-prow knative-prow Bot added do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files. labels Aug 27, 2026
@dsimansk dsimansk changed the title chore: Upgrade k8s.io core libs to 1.36/0.36.4 chore: Upgrade k8s.io core libs to 1.36 (0.36.4) Aug 27, 2026
@codecov

codecov Bot commented Aug 27, 2026 •

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 75.08%. Comparing base (2f3769f) to head (73a9b9b).
⚠️ Report is 2 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #3387      +/-   ##
==========================================
+ Coverage   74.85%   75.08%   +0.22%     
==========================================
  Files         189      189              
  Lines        8349     8352       +3     
==========================================
+ Hits         6250     6271      +21     
+ Misses       1854     1837      -17     
+ Partials      245      244       -1     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@dsimansk
dsimansk force-pushed the pr/kube-136 branch 2 times, most recently from b8ad3b5 to 40918c0 Compare August 31, 2026 13:04
@dsimansk

Copy link
Copy Markdown
Contributor Author

/test unit-tests

@shashankxrm

Copy link
Copy Markdown

I've been looking into #3383 as the upstream prerequisite for Tekton's server-side sharded List/Watch work (tektoncd/pipeline#10610). It looks like this Kubernetes 0.36 / Go 1.26 upgrade is the prerequisite for exposing ListOptions.ShardSelector.

Once this upgrade is settled, I can work on the informer injection plumbing in #3383 as a follow-up. Just mentioning the dependency here for context.
@vdemeester

@dsimansk dsimansk changed the title chore: Upgrade k8s.io core libs to 1.36 (0.36.4) chore: Upgrade k8s.io core libs to 1.36 (0.36.5) Oct 9, 2026
@dsimansk

dsimansk commented Oct 9, 2026

Copy link
Copy Markdown
Contributor Author

/retest

@knative-prow-robot knative-prow-robot added the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Oct 9, 2026
@knative-prow-robot knative-prow-robot removed the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Oct 9, 2026
@dsimansk dsimansk changed the title chore: Upgrade k8s.io core libs to 1.36 (0.36.5) chore: Upgrade k8s.io core libs to 1.36 (0.36.z) Oct 9, 2026
@dsimansk

dsimansk commented Oct 9, 2026

Copy link
Copy Markdown
Contributor Author

PTAL
/cc @knative/steering-committee @dprotaso @creydr @aliok

@knative-prow
knative-prow Bot requested review from a team, aliok and creydr October 9, 2026 11:40
@dsimansk

dsimansk commented Oct 9, 2026 •

Copy link
Copy Markdown
Contributor Author

Per downstream test only Eventing is failing due to api changes, that can be addressed alongside auto-update PR once k8s libs are bumped.

PR opened: knative/eventing#9422

@dsimansk

dsimansk commented Oct 9, 2026

Copy link
Copy Markdown
Contributor Author

/unhold

@knative-prow knative-prow Bot removed the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Oct 9, 2026
@aliok

aliok commented Oct 9, 2026

Copy link
Copy Markdown
Member

/lgtm
/approve

Looks good. Claude also couldn't find anything major.

Thanks!

@knative-prow knative-prow Bot added the lgtm Indicates that a PR is ready to be merged. label Oct 9, 2026
@knative-prow

knative-prow Bot commented Oct 9, 2026

Copy link
Copy Markdown

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: aliok, dsimansk

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

@knative-prow knative-prow Bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Oct 9, 2026
@knative-prow
knative-prow Bot merged commit 795a0f6 into knative:main Oct 9, 2026
37 of 38 checks passed
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. area/test-and-release lgtm Indicates that a PR is ready to be merged. size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants