-
Notifications
You must be signed in to change notification settings - Fork 26
Add new mode for vault-unsealer using ssm parameter with type SecureString #27
base: master
Are you sure you want to change the base?
Conversation
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: If they are not already assigned, you can assign the PR to them by writing The full list of commands accepted by this bot can be found here. The pull request process is described here DetailsNeeds approval from an approver in each of these files:Approvers can indicate their approval by writing |
|
/ok-to-test |
Signed-off-by: Adam Hamsik <adam.hamsik@chillisys.com>
…ing which can be automatically decrypted by AWS KMS key. Add example terraform templates which can be used to create these from env variables. Signed-off-by: Adam Hamsik <adam.hamsik@chillisys.com>
Signed-off-by: Adam Hamsik <adam.hamsik@chillisys.com>
Signed-off-by: Adam Hamsik <adam.hamsik@chillisys.com>
|
@JoshVanL here you go. |
|
@haad Sorry, should have been more clear, it needs to be included in the root PR as described here https://github.com/kubernetes/community/blob/master/contributors/guide/release-notes.md and will look something like this #24 There looks to also be some problems with |
Signed-off-by: Adam Hamsik <adam.hamsik@chillisys.com>
Signed-off-by: Adam Hamsik <adam.hamsik@chillisys.com>
|
/assign @haad |
|
@JoshVanL This should do it :) |
|
@haad Thanks, I'll take a look when I get the chance :) |
|
@haad: PR needs rebase. DetailsInstructions 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. |
Uh oh!
There was an error while loading. Please reload this page.