Skip to content

sops/awskms: Fix AWS KMS config creation#667

Merged
hiddeco merged 1 commit intofluxcd:mainfrom
aryan9600:aws-kms-decryption
May 27, 2022
Merged

sops/awskms: Fix AWS KMS config creation#667
hiddeco merged 1 commit intofluxcd:mainfrom
aryan9600:aws-kms-decryption

Conversation

@aryan9600
Copy link
Copy Markdown
Member

@aryan9600 aryan9600 commented May 27, 2022

This PR is a follow up to #641 which fixes a few things around AWS KMS config creation and expands related tests.
Signed-off-by: Sanskar Jaiswal sanskar.jaiswal@weave.works

@aryan9600 aryan9600 requested a review from hiddeco May 27, 2022 18:15
Copy link
Copy Markdown
Member

@hiddeco hiddeco left a comment

Choose a reason for hiding this comment

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

Out of interest: any chance you can link me to the doc ref around profiles?

Comment thread internal/sops/awskms/keysource.go Outdated
Comment thread internal/sops/awskms/keysource.go Outdated
Comment thread internal/sops/awskms/keysource_test.go Outdated
@aryan9600 aryan9600 force-pushed the aws-kms-decryption branch from 6be4f5b to f3a7084 Compare May 27, 2022 18:34
@aryan9600 aryan9600 changed the title Add AwsProfile to SOPS AWSKMS implementation Fix AWS KMS config creation May 27, 2022
@aryan9600 aryan9600 force-pushed the aws-kms-decryption branch from f3a7084 to 01edc3b Compare May 27, 2022 18:41
Comment thread internal/sops/awskms/keysource.go
@aryan9600 aryan9600 force-pushed the aws-kms-decryption branch from 01edc3b to 928be40 Compare May 27, 2022 18:51
@hiddeco hiddeco added the area/sops SOPS related issues and pull requests label May 27, 2022
Signed-off-by: Sanskar Jaiswal <sanskar.jaiswal@weave.works>
@aryan9600 aryan9600 changed the title Fix AWS KMS config creation sops/awskms: Fix AWS KMS config creation May 27, 2022
@aryan9600 aryan9600 force-pushed the aws-kms-decryption branch from 928be40 to 10bc7ea Compare May 27, 2022 19:07
Copy link
Copy Markdown
Member

@hiddeco hiddeco left a comment

Choose a reason for hiding this comment

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

LGTM, brilliant @aryan9600 🙇

@hiddeco hiddeco merged commit 65af429 into fluxcd:main May 27, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/sops SOPS related issues and pull requests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants