Skip to content

Chore(deps): Bump the k8s-io group#16890

Closed
Duciwuci wants to merge 6 commits intoprometheus:mainfrom
Duciwuci:update-kubernetes-packages-and-api
Closed

Chore(deps): Bump the k8s-io group#16890
Duciwuci wants to merge 6 commits intoprometheus:mainfrom
Duciwuci:update-kubernetes-packages-and-api

Conversation

@Duciwuci
Copy link
Contributor

@Duciwuci Duciwuci commented Jul 17, 2025

This PR relates to #16534

This includes bumping all k8s packages to 0.33.x and adjust the breaking changes.

The test Go Test with previous Go Version is expected to fail, since new Kubernetes packages are already on Go 1.24.

Duciwuci added 4 commits July 17, 2025 13:04
Signed-off-by: Duciwuci <duciwuci@gmail.com>
Signed-off-by: Duciwuci <duciwuci@gmail.com>
Signed-off-by: Duciwuci <duciwuci@gmail.com>
Signed-off-by: Duciwuci <duciwuci@gmail.com>
@Duciwuci Duciwuci requested a review from brancz as a code owner July 17, 2025 11:11
Signed-off-by: Duciwuci <duciwuci@gmail.com>
@Duciwuci Duciwuci force-pushed the update-kubernetes-packages-and-api branch from 179d0aa to a4aa071 Compare July 17, 2025 11:18
Signed-off-by: Duciwuci <duciwuci@gmail.com>
@Duciwuci Duciwuci requested a review from juliusv as a code owner July 17, 2025 11:34
@bboreham
Copy link
Member

This needs to wait for Prometheus to update to Go 1.25.

@bboreham
Copy link
Member

Also we're not going to remove Endpoints right away - see #16684.

Can you explain more of the changes lumped under "adjust", please?

@Duciwuci
Copy link
Contributor Author

Also we're not going to remove Endpoints right away - see #16684.

Can you explain more of the changes lumped under "adjust", please?

Thanks. Then the golang-ci linter needs to ignore the whole validation in the file.
If we want to keep supporting Endpoints, then we actually needs none of my changes and just need an adjustment in the go.mod.

Then we also don't need to wait for Go 1.25?

@bboreham
Copy link
Member

The reason to wait for 1.25 is so the “previous version of Go” tests move to 1.24.

@Duciwuci
Copy link
Contributor Author

The reason to wait for 1.25 is so the “previous version of Go” tests move to 1.24.

Then we wait, thanks!

@Duciwuci
Copy link
Contributor Author

@bboreham Go 1.25 is released!

@Duciwuci
Copy link
Contributor Author

Now waiting for #17080

@vladst3f
Copy link

Now waiting for #17080

Hi @Duciwuci , it got merged a couple of days ago. Can this continue now ? There's a little train of dependencies waiting on this one now.
Thanks!

@mjlshen
Copy link
Contributor

mjlshen commented Sep 17, 2025

I attempted to address all existing feedback on this PR to push this update forward in #17200

@machine424
Copy link
Member

reviewed #17200 without knowing this one exists.

@mjlshen please check with @Duciwuci how you want to move forward.

As Bryan mentioned, we'll not be removing the Endpoints role, https://github.com/prometheus/prometheus/pull/16684/files added a warning about the deprecation in the docs.
another thing we can do is mark all exposed things e.g. NewEndpoints with a // Deprecated: to warn library users.

@mjlshen
Copy link
Contributor

mjlshen commented Sep 17, 2025

reviewed #17200 without knowing this one exists.

@\mjlshen please check with @\Duciwuci how you want to move forward.

Understood, I just realized it's been sitting for some time with unaddressed feedback and assumed Duciwuci was busy with life as has happened to myself many times. Just trying to help push it forward together so that we can upgrade corresponding downstream dependencies!

As Bryan mentioned, we'll not be removing the Endpoints role, https://github.com/prometheus/prometheus/pull/16684/files added a warning about the deprecation in the docs.

I've done this in #17200 because this MR has had that feedback for a while

another thing we can do is mark all exposed things e.g. NewEndpoints with a // Deprecated: to warn library users.

I'm happy to do this

@Duciwuci
Copy link
Contributor Author

Hi, I was on Vacation until today. In favor for #17200 we can close this.

@Duciwuci
Copy link
Contributor Author

Thanks @mjlshen

@Duciwuci Duciwuci closed this Sep 25, 2025
@mjlshen
Copy link
Contributor

mjlshen commented Sep 25, 2025

Thanks for the initial effort!

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.

5 participants