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
AWS KMS Support #74
AWS KMS Support #74
Conversation
This looks awesome! Thanks! |
Signed-off-by: Cody Soyland <cody.soyland@solarwinds.com>
209eda7
to
2cebefe
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.
just a couple questions - looking great overall
Signed-off-by: Cody Soyland <cody.soyland@solarwinds.com>
- Fix a few test issues - Resolve linter warnings - Add WithRemoteVerification option - Use https with AWS custom endpoints - Add env var to disable AWS TLS verification Signed-off-by: Cody Soyland <cody.soyland@solarwinds.com>
LGTM - will approve once the e2e tests pass |
Signed-off-by: Cody Soyland <cody.soyland@solarwinds.com>
Thanks @bobcallaway for your quick reviews! My last patch should fix the e2e test failures. |
Signed-off-by: Cody Soyland <cody.soyland@solarwinds.com>
After that last failure, I set up GHA on my fork and realized it wasn't running the e2e-test.sh script, so I updated the GHA config with the localstack settings and got it passing there. It should be good now! 🤞 |
This adds AWS as a supported KMS backend.