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

Bulk Dependency updates #1187

Merged
merged 4 commits into from
Aug 7, 2023
Merged

Bulk Dependency updates #1187

merged 4 commits into from
Aug 7, 2023

Conversation

HVBE
Copy link
Collaborator

@HVBE HVBE commented Aug 4, 2023

This Pull requests bundles a bunch of dependencies and upgrades, A lot of which were interdependent and were causing dependabot to 💩 when trying to update them separately.

This PR should allow dependabot to resume proper operation

Upgrade project go version to 1.20

Bump kustomize to v5

Refactoring of deprecated functions and fields

Regenerate documentation and manifests

Upgrade project go version to 1.20

Bump kustomize to v5

Refactoring of deprecated functions and fields

Regenerate documentation and manifests
@HVBE
Copy link
Collaborator Author

HVBE commented Aug 4, 2023

Looks like our tests are happy, I'm still going to test it out manually against kubernetes to make sure

@HVBE
Copy link
Collaborator Author

HVBE commented Aug 6, 2023

Verified on Minikube, looks to be working correctly.

No errors from the operator pod

Copy link
Collaborator

@NissesSenap NissesSenap left a comment

Choose a reason for hiding this comment

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

Have you verified kustomize installation? It contains a few breaking changes, I don't think I have used any of the old APIs but i can't guarantee it. LGTM

@HVBE
Copy link
Collaborator Author

HVBE commented Aug 7, 2023

Have you verified kustomize installation? It contains a few breaking changes, I don't think I have used any of the old APIs but i can't guarantee it. LGTM

Yep, this was tested using kustomize, deployed against minikube, the only thing was me overriding the image from the make recipe to point to my quay repo instead for testing.

@HVBE HVBE merged commit 2b8747a into master Aug 7, 2023
9 checks passed
@HVBE HVBE deleted the k8s.io-bundle-updates branch August 7, 2023 11:32
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.

None yet

3 participants