Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update controller-runtime to v0.16.3 #4

Merged
merged 7 commits into from
Dec 13, 2023
Merged

Conversation

benwh
Copy link

@benwh benwh commented Dec 13, 2023

Why have we done this? To move past old versions of libraries, such as k8s.io/api, which have transitive dependencies on the pre-1.0 go.opentelemetry.io/otel library.

Commits in order:


commit 4fa0ddf Update controller-runtime to v0.16.3

This will include the large change mentioned here: https://github.com/kubernetes-sigs/controller-runtime/releases/tag/v0.15.0

While here, also change the module path, as this looks to be the proper
way of doing things.


commit de88551 Update controllers for new controller-runtime APIs

Make this compile again, as some options have been moved around.


commit 7969e20 Regenerate manifests

This brings in the new k8s 1.28.0 fields values.


commit 2e7963f Update kind version

We're using k8s v1.28 APIs, so let's go to the latest available Kind
image, to run a 1.27 cluster.


commit b384c1c Fix race condition in acceptance tests

Our app was being deployed before cert-manager was fully up and
running, leading to theatre pods stuck in a ContainerCreating state,
due to a missing secret volume.


commit aeda687 Fix logger warning in acceptance tests


commit ee83746 v5.0.0

Bump the major version, as we've upgraded many things, and may have lost
compatibility with older clusters.


This will include the large change mentioned here: https://github.com/kubernetes-sigs/controller-runtime/releases/tag/v0.15.0

While here, also change the module path, as this looks to be the proper
way of doing things.
Make this compile again, as some options have been moved around.
This brings in the new k8s 1.28.0 fields values.
We're using k8s v1.28 APIs, so let's go to the latest available Kind
image, to run a 1.27 cluster.
Our app was being deployed before `cert-manager` was fully up and
running, leading to theatre pods stuck in a `ContainerCreating` state,
due to a missing secret volume.
Bump the major version, as we've upgraded many things, and may have lost
compatibility with older clusters.
@benwh benwh merged commit 27f49dc into master Dec 13, 2023
4 checks passed
@benwh benwh deleted the upgrade-controller-runtime branch December 13, 2023 11:41
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.

1 participant