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

chore(deps): update to Crossplane runtime 1.14.1, upjet 1.1.0 #39

Closed
wants to merge 6 commits into from

Conversation

displague
Copy link
Collaborator

@displague displague commented Feb 6, 2024

Description of your changes

Fixes #29

I have:

  • Read and followed Crossplane's contribution process.
  • Run make reviewable test to ensure this PR is ready for review.

How has this code been tested

go get github.com/crossplane/[email protected] && go mod tidy
make generate

Signed-off-by: Marques Johansson <[email protected]>
Updated from upbound/upjet tp crossplane/upjet

Signed-off-by: Marques Johansson <[email protected]>
For .github/ CI and Makefile:
- update go to 1.21
- build/ to latest
- golanci-lint to 1.54.2

Signed-off-by: Marques Johansson <[email protected]>
@displague
Copy link
Collaborator Author

My local build environment has 1.21. This may have led to the go.mod files including toolgen which is a 1.21 flag. I attempted to update the CI and Makefile to go 1.21 and then ran into linting and build errors on this line:

# github.com/crossplane-contrib/provider-jet-equinix/internal/controller/providerconfig
Error: internal/controller/providerconfig/config.go:45:12: source.Kind (value of type func(cache "sigs.k8s.io/controller-runtime/pkg/cache".Cache, object client.Object) source.SyncingSource) is not a type

I think the next step would be to revert to 1.19 to match upstream and regenerate go.mod using a 1.19 format

@displague
Copy link
Collaborator Author

displague commented Feb 6, 2024

$ go mod tidy -go=1.19
go: github.com/crossplane/[email protected] requires [email protected], but 1.19 is requested

💫

@displague
Copy link
Collaborator Author

Attempted to copy some enhancements over from provider-aws, which is already adapted to go 1.21, but there are many more features that would need to be copied over.

crossplane/upjet-provider-template#36

@displague
Copy link
Collaborator Author

Largely covered by #52

@displague displague closed this Jul 20, 2024
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.

Update to the latest upjet and crossplane-runtime versions
1 participant