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

EVEREST-107 Remove kube-rbac-proxy and update kubebuilder project to go/v4 #436

Merged
merged 2 commits into from
Dec 3, 2024

Conversation

recharte
Copy link
Collaborator

@recharte recharte commented Dec 3, 2024

In percona/everest-operator#597 we removed kube-rbac-proxy and migrated to go/v4 project. This commit reflects the manifest changes introduced in that PR.

@recharte recharte requested a review from a team as a code owner December 3, 2024 10:06
In percona/everest-operator#597 we removed
kube-rbac-proxy and migrated to go/v4 project. This commit reflects the
manifest changes introduced in that PR.
@recharte recharte force-pushed the EVEREST-107-remove-kube-rbac-proxy branch from 2c1dc62 to e17567a Compare December 3, 2024 10:06
@recharte recharte changed the title EVEREST-107 update everest operator manifests EVEREST-107 Remove kube-rbac-proxy and update kubebuilder project to go/v4 Dec 3, 2024
mayankshah1607
mayankshah1607 previously approved these changes Dec 3, 2024
Copy link
Member

@mayankshah1607 mayankshah1607 left a comment

Choose a reason for hiding this comment

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

LGTM, just one question

Also, you need to run make update-dev-chart in the everest repo after this is merged, otherwise the CI there will complain..

@recharte recharte merged commit 9b16ea7 into main Dec 3, 2024
3 checks passed
@recharte recharte deleted the EVEREST-107-remove-kube-rbac-proxy branch December 3, 2024 11:36
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.

2 participants