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

Issue while updating upjet to 1.4.2 (crossplane-runtime v1.18.0) #467

Open
remyleone opened this issue Feb 6, 2025 · 4 comments
Open

Issue while updating upjet to 1.4.2 (crossplane-runtime v1.18.0) #467

remyleone opened this issue Feb 6, 2025 · 4 comments
Labels
bug Something isn't working

Comments

@remyleone
Copy link

What happened?

I was trying to update upjet to v1.4.2 and stumbled upon the following error:

github.com/crossplane/upjet/pkg/controller/handler
# github.com/crossplane/upjet/pkg/controller/handler
../../go/pkg/mod/github.com/crossplane/[email protected]/pkg/controller/handler/eventhandler.go:114:33: cannot use limitingInterface (variable of type workqueue.RateLimitingInterface) as workqueue.TypedRateLimitingInterface[reconcile.Request] value in argument to e.innerHandler.Create: workqueue.RateLimitingInterface does not implement workqueue.TypedRateLimitingInterface[reconcile.Request] (wrong type for method Add)
                have Add(any)
                want Add(reconcile.Request)
../../go/pkg/mod/github.com/crossplane/[email protected]/pkg/controller/handler/eventhandler.go:120:33: cannot use limitingInterface (variable of type workqueue.RateLimitingInterface) as workqueue.TypedRateLimitingInterface[reconcile.Request] value in argument to e.innerHandler.Update: workqueue.RateLimitingInterface does not implement workqueue.TypedRateLimitingInterface[reconcile.Request] (wrong type for method Add)
                have Add(any)
                want Add(reconcile.Request)
../../go/pkg/mod/github.com/crossplane/[email protected]/pkg/controller/handler/eventhandler.go:126:33: cannot use limitingInterface (variable of type workqueue.RateLimitingInterface) as workqueue.TypedRateLimitingInterface[reconcile.Request] value in argument to e.innerHandler.Delete: workqueue.RateLimitingInterface does not implement workqueue.TypedRateLimitingInterface[reconcile.Request] (wrong type for method Add)
                have Add(any)
                want Add(reconcile.Request)
../../go/pkg/mod/github.com/crossplane/[email protected]/pkg/controller/handler/eventhandler.go:132:34: cannot use limitingInterface (variable of type workqueue.RateLimitingInterface) as workqueue.TypedRateLimitingInterface[reconcile.Request] value in argument to e.innerHandler.Generic: workqueue.RateLimitingInterface does not implement workqueue.TypedRateLimitingInterface[reconcile.Request] (wrong type for method Add)
                have Add(any)
                want Add(reconcile.Request)

How can we reproduce it?

You can checkout my branch and try to run a make build: scaleway/crossplane-provider-scaleway#123

@remyleone remyleone added the bug Something isn't working label Feb 6, 2025
@remyleone remyleone changed the title Issue while updating upjet to 1.4.2 Issue while updating upjet to 1.4.2 (crossplane-runtime v1.18.0) Feb 6, 2025
@chlunde
Copy link

chlunde commented Feb 6, 2025

Try controller-runtime 0.18, see breaking changes in 0.19 here: https://github.com/kubernetes-sigs/controller-runtime/releases/tag/v0.19.0

@remyleone
Copy link
Author

remyleone commented Feb 6, 2025

I've tried and it still give me the same error :/ beside doing a go mod tidy revert this change :/

@chlunde
Copy link

chlunde commented Feb 6, 2025

This seems to work, you also need to use k8s 0.30 and controller-tools v0.14.0

require (
        dario.cat/mergo v1.0.1
        github.com/alecthomas/kingpin/v2 v2.4.0
        github.com/crossplane/crossplane-runtime v1.17.0
        github.com/crossplane/crossplane-tools v0.0.0-20230925130601-628280f8bf79
        github.com/crossplane/upjet v1.4.1-0.20250130134024-ce71033d450f
        github.com/pkg/errors v0.9.1
        k8s.io/apimachinery v0.30.0
        k8s.io/client-go v0.30.0
        sigs.k8s.io/controller-runtime v0.18.2
        sigs.k8s.io/controller-tools v0.14.0
)

@chlunde
Copy link

chlunde commented Feb 6, 2025

It looks like 0.31/0.18/1.18 will be supported by #452

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants