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

Bump kubernetes to 1.27 operator and target allocator #1653

Merged
merged 10 commits into from
May 31, 2023

Conversation

jaronoff97
Copy link
Contributor

closes #1652

This upgrades all kubernetes dependencies in each module to the latest version 0.27.0 for Kubernetes 1.27.0

Some extra upgrades were included, the dependencies of which have been reviewed and should all be safe upgrades.

@jaronoff97 jaronoff97 requested review from a team April 13, 2023 18:33
@jaronoff97 jaronoff97 added the Skip Changelog PRs that do not require a CHANGELOG.md entry label Apr 13, 2023
@jaronoff97
Copy link
Contributor Author

I think we are blocked on this kubernetes-sigs/controller-runtime#2189 unfortunately, seems dependencies are failing because of incompatibilities between these libs.

Error: ../../../go/pkg/mod/sigs.k8s.io/controller-runtime@v0.14.6/pkg/cache/multi_namespace_cache.go:308:9: cannot use handles (variable of type map[string]"k8s.io/client-go/tools/cache".ResourceEventHandlerRegistration) as "k8s.io/client-go/tools/cache".ResourceEventHandlerRegistration value in return statement: map[string]"k8s.io/client-go/tools/cache".ResourceEventHandlerRegistration does not implement "k8s.io/client-go/tools/cache".ResourceEventHandlerRegistration (missing method HasSynced)
Error: ../../../go/pkg/mod/sigs.k8s.io/controller-runtime@v0.14.6/pkg/cache/multi_namespace_cache.go:321:9: cannot use handles (variable of type map[string]"k8s.io/client-go/tools/cache".ResourceEventHandlerRegistration) as "k8s.io/client-go/tools/cache".ResourceEventHandlerRegistration value in return statement: map[string]"k8s.io/client-go/tools/cache".ResourceEventHandlerRegistration does not implement "k8s.io/client-go/tools/cache".ResourceEventHandlerRegistration (missing method HasSynced)
Error: ../../../go/pkg/mod/sigs.k8s.io/controller-runtime@v0.14.6/pkg/cache/multi_namespace_cache.go:326:17: impossible type assertion: h.(map[string]toolscache.ResourceEventHandlerRegistration)
	map[string]"k8s.io/client-go/tools/cache".ResourceEventHandlerRegistration does not implement "k8s.io/client-go/tools/cache".ResourceEventHandlerRegistration (missing method HasSynced)

@jaronoff97
Copy link
Contributor Author

and now we wait on kubernetes-sigs/controller-runtime#2297

Copy link
Contributor

@yuriolisa yuriolisa left a comment

Choose a reason for hiding this comment

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

Only a reminder to add on top of the compatibility matrix on README.

@pavolloffay
Copy link
Member

It seems that the CI failed and the PR needs to be rebased.

@jaronoff97
Copy link
Contributor Author

Another unexpected update that broke us kubernetes-sigs/controller-runtime#2134... we also may need to wait on this kubebuilder PR before we can merge this.

apis/v1alpha1/instrumentation_webhook.go Outdated Show resolved Hide resolved
apis/v1alpha1/opentelemetrycollector_webhook.go Outdated Show resolved Hide resolved
main.go Outdated Show resolved Hide resolved
@pavolloffay
Copy link
Member

LGTM just the imports need to be sorted

@jaronoff97
Copy link
Contributor Author

ha yeah noticed that as soon as i pushed

@jaronoff97
Copy link
Contributor Author

note: i will need to separately update the kubernetes versions for opamp-bridge, i tried to get go workspaces working, but it wasn't happy w/ me.

@@ -177,21 +178,21 @@ func (r *Instrumentation) Default() {
var _ webhook.Validator = &Instrumentation{}

// ValidateCreate implements webhook.Validator so a webhook will be registered for the type.
func (r *Instrumentation) ValidateCreate() error {
func (r *Instrumentation) ValidateCreate() (admission.Warnings, error) {
Copy link
Member

Choose a reason for hiding this comment

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

this is cool, we can start using the warnings

@TylerHelmuth
Copy link
Member

TylerHelmuth commented May 25, 2023

Created #1769 to handle the build jobs

@jaronoff97 jaronoff97 changed the title Bump kubernetes to 1.27 everywhere Bump kubernetes to 1.27 operator and target allocator May 25, 2023
Copy link
Member

@TylerHelmuth TylerHelmuth left a comment

Choose a reason for hiding this comment

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

Exciting!

@jaronoff97 jaronoff97 merged commit d6d9f2f into open-telemetry:main May 31, 2023
@jaronoff97 jaronoff97 deleted the kube-bump branch May 31, 2023 16:14
ItielOlenick pushed a commit to ItielOlenick/opentelemetry-operator that referenced this pull request May 1, 2024
…#1653)

* Bump kubernetes to 1.27 everywhere

* fix broken updates

* Fix dep order

* rename

* upgrade controller-gen and kustomize

* bump allocator deps

* unbump opamp bridge for now

* upgrade deprecations

* fix kuttl test

* bundle
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Skip Changelog PRs that do not require a CHANGELOG.md entry
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Upgrade Kubernetes Compatability for 1.27.0
4 participants