Upgrade k8s dependencies to 1.27

neondatabase / autoscaling

Postgres vertical autoscaling in k8s

Apache License 2.0

144 stars 18 forks source link

Upgrade k8s dependencies to 1.27 #929

Closed SergeyMelnikov closed 1 week ago

SergeyMelnikov commented 2 months ago

Updating whole autoscaling to k8s version 1.27

In particular:

Update all k8s dependency to 1.27.15
latest k3d/k3s is 1.27.14
latest kind is 1.27.13
Update sigs.k8s.io/controller-runtime to 0.15.1
Update Validate methods of the hooks to new interface, they need to return warnings, returning always nil for now.
Update all other versions in Makefile
Update cluster-autoscaler to be based on 1.27.8
Update crictl to 1.27 ( https://neondb.slack.com/archives/C03TN5G758R/p1718632719461499 )

cc: https://github.com/neondatabase/cloud/issues/7922

SergeyMelnikov commented 4 weeks ago

Bumped to 1.27.15 latest k3d/k3s is 1.27.14 latest kind is 1.27.13

sharnoff commented 3 weeks ago

Rebased onto #987 (which has a scheduler fix; more info here), which in turn is based on v0.32.0 -- so that it can be put onto staging and we can check everything's happy.

Compiler's unhappy right now though - will do the fixes & post updates in slack.

sharnoff commented 3 weeks ago

Rebased onto main; this one's been proving somewhat difficult to debug, actually... cc @Omrigan

Omrigan commented 3 weeks ago

@SergeyMelnikov I fixed the tests!

The reason it was failing is that controller-runtime>v0.15.0 now requires to pass WithStatusSubresource when creating fake client.

See https://github.com/kubernetes-sigs/controller-runtime/commit/0bff6fc8af438727a2a9e5c8a2050fdad25952b4