Skip to content

Update to k8s.io v0.33.5#17200

Merged
machine424 merged 3 commits intoprometheus:mainfrom
mjlshen:k8s-v0.33.5
Sep 26, 2025
Merged

Update to k8s.io v0.33.5#17200
machine424 merged 3 commits intoprometheus:mainfrom
mjlshen:k8s-v0.33.5

Conversation

@mjlshen
Copy link
Contributor

@mjlshen mjlshen commented Sep 17, 2025

Which issue(s) does the PR fix:

Continues the work of #16534 and #16890 by updating the k8s.io dependencies to v0.33.5, replacing deprecated code where relevant, and adding deprecation notices to the apiv1.Endpoints API as it has been deprecated in K8s starting in v1.33+. We will still support it for some time per #16890 (comment), but this will nudge downstream users to leverage the EndpointSlice API instead.

Does this PR introduce a user-facing change?

NONE

Copy link
Member

@machine424 machine424 left a comment

Choose a reason for hiding this comment

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

thanks, I was looking at taking care of this.
some suggestions

@mjlshen mjlshen force-pushed the k8s-v0.33.5 branch 2 times, most recently from 1c85e12 to f69f2b7 Compare September 18, 2025 03:42
Copy link
Member

@machine424 machine424 left a comment

Choose a reason for hiding this comment

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

some suggestions.
Normally, we only include changes that affect Prometheus binary users in the release notes (for the role deprecation, we already mention that in the docs). In this case, the deprecation is mainly relevant for Prometheus library users, who will already be warned via the // Deprecated comment. So, I’d recommend just putting NONE in the release-note block.

Signed-off-by: Michael Shen <mishen@umich.edu>
Signed-off-by: Michael Shen <mishen@umich.edu>
Signed-off-by: Michael Shen <mishen@umich.edu>
Copy link
Member

@machine424 machine424 left a comment

Choose a reason for hiding this comment

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

lgtm, thanks again!

@mjlshen
Copy link
Contributor Author

mjlshen commented Sep 26, 2025

@machine424 Just making sure, is there anything I need to do?

@machine424
Copy link
Member

@machine424 Just making sure, is there anything I need to do?

waited so others can double check it

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