Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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
integrating external secrets operator with gen3 helm charts #138
integrating external secrets operator with gen3 helm charts #138
Changes from all commits
3ab8426
a1e5c7a
a7f368d
a65340d
1239a36
e28fd34
c38dcb8
33f702e
ca7f84c
8e7eec0
7da2bc1
f0fd53a
703b341
f88cb72
4131806
e45b280
bb6235c
20b3379
e8ecba5
6f8a43c
8f2632c
6f0c0b2
32feb90
8bf1983
33f033e
590eec0
558bbce
ff4d840
ab7ce28
f7e6685
19a2fef
d51b8fd
a31e125
503155e
d002976
df99089
5472197
c84b653
b14af76
cba5b51
4a34967
339ef9c
efc7dbb
bcc1c5d
3c54bba
792a061
4f69cf3
de399c0
89d3686
6367637
013e420
a49d333
bafcd7f
98c677f
6b4c7f6
da2e79b
9bbdc9b
9a06579
28ad27f
2fdb1ec
05d8700
1160987
1e3d7e1
File filter
Filter by extension
Conversations
Jump to
There are no files selected for viewing
Large diffs are not rendered by default.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I feel like we need to organize this a little differently to emphasize the ability to be cloud agnostic, but that we have an example implementation with AWS Secrets Manager. Something like:
high level overview of the External Secrets Operator
Definition of abstraction layer where secrets can come from elsewhere
SKIP the details about the AWS Secrets Manager Implementation
Show how you can use the External Secrets Operator secrets in our charts (once you have them)
Now we can go into detail about 1 of the currently implemented options for housing original secrets: AWS Secrets Manager
Detail steps for how to use an alternative (even if they're broad steps, also suggest providing such code back to the project if applicable)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think we should wait on changing this part of the md until we build this out to work with other cloud providers. I believe it won't be a large fix, but the ClusterSecretStore will need to be edited in order to do this. I will note this down and make sure to update the doc when we make this cloud agnostic!