-
Notifications
You must be signed in to change notification settings - Fork 719
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
[quorum] integrate aws secrets manager #2623
[quorum] integrate aws secrets manager #2623
Conversation
platforms/quorum/charts/quorum-genesis/templates/genesis-job-init.yaml
Outdated
Show resolved
Hide resolved
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.
condition needs to be checked.
The keys will conflict if same platforms are deployed using the same secret manager region. See if the keys can be made unique using the namespace |
9e3e8f1
to
b315d2b
Compare
b315d2b
to
909e108
Compare
platforms/quorum/charts/quorum-node/templates/node-hooks-pre-install.yaml
Outdated
Show resolved
Hide resolved
909e108
to
192b6d3
Compare
This PR will allow the use of the AWS service called Secrets Manager to store sensitive information, similar to how we use HashiCorp Vault for the same purpose. - A guide named "integrate-aws-secrets-manager-with-eks.md" has been introduced to help users securely connect their EKS cluster with Secrets Manager using OIDC. - The Quorum master README has been updated to guide users on how to deploy a network with AWS Secrets Manager. - A Python script has been added that contains the CRUD operation code for AWS Secrets Manager, injecting the script into the container via ConfigMap. - The Quorum Genesis and Node charts code have been updated to support Secrets Manager. fixes hyperledger-bevel#2200 Signed-off-by: saurabhkumarkardam <[email protected]>
192b6d3
to
ed7bbde
Compare
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.
LGTM
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.
LGTM
Commit to be reviewed
feat(quorum): integrate aws secrets manager
fixes #2200