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

Upgrade dependencies #120

Merged
merged 6 commits into from
Oct 21, 2024
Merged

Upgrade dependencies #120

merged 6 commits into from
Oct 21, 2024

Conversation

phantomjinx
Copy link
Member

@phantomjinx phantomjinx commented Oct 21, 2024

Updates:

  • go version 1.21
  • sigs.k8s.io/controller-runtime v0.17.0
  • k8s.io/api v0.29.9
  • k8s.io/apiextensions-apiserver v0.29.0
  • k8s.io/apimachinery v0.29.9
  • k8s.io/client-go v0.29.9

* go.mod
 * Updates minimum go to 1.21.1
 * Updates controller-runtime to v0.17.0 (last version prior to go 1.22)
 * Updates k8s.io to v0.29.9 (last version prior to go 1.22)
 * Updates openshift/client-go to 2024 commits
 * Replaces gnostic indirect dependency to 0.7.0 which handles change of
   namespace in consumers to requiring gnostic-models instead

* cmd/manager/main.go
 * Updates method of defining namespaces that should be watched by the operator

* console_yaml_samples.go
 * Removal of logging package demands replacement with controller runtime logging

* fakeclient-test.go
* hawtio-controller.go
* hawtio-controller-test.go
 * Fixes due to controller client api changes

* go-get-update.sh
 * Script for updating dependencies automatically
* Drops testing with 1.20 given its no longer supported

* Cannot yet include golang 1.22 as requires upgrade to controller-gen
  version, which is not yet practical.
* Can only go up 1 increment because:
 * v0.7.0 -> v0.14.0 removes the 'preserveUnkownFields' option yet v0.15.0
   restores it. No point in changing our config or risk breaking older CRDs
   by needlessly removing only to restore it later
 * v0.15.0+ cannot yet upgrade as requires golang 1.22
@phantomjinx phantomjinx merged commit 9e196d9 into hawtio:main Oct 21, 2024
1 check passed
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