-
Notifications
You must be signed in to change notification settings - Fork 2.6k
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
Add External Secret Operator configs #27932
Conversation
Generated by running helm repo add external-secrets-eso https://charts.external-secrets.io helm template --include-crds --output-dir ./output_dir external-secrets-eso external-secrets-eso/external-secrets The only modification is updating the service account name in the deployment.yaml file
b07ee5c
to
113e8d6
Compare
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: chaodaiG The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
This error occurred on generated yaml from External Secret Operator, figure it doesn't worth an error
113e8d6
to
393a94f
Compare
/lgtm Feel to remove the hold when you're ready. |
They are required for the external secret CR to be deployed in the cluster
New changes are detected. LGTM label has been removed. |
As discussed in #24869, CSI driver is probably preferred over ESO /close |
@chaodaiG: Closed this PR. In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
Part of #24869
The goal is to migrate from Kubernetes External Secret to External Secret Operator as the former was announced deprecated and superseded by the latter. This PR adds the configs with the replica set to 0 so it will be no effect.
For code reviewers, this PR consists of two commits: