Skip to content
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

Extend CI with KMS-based key tests #655

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

atanzu
Copy link
Contributor

@atanzu atanzu commented Jan 29, 2025

Issue #, if available:

Description of changes:

This PR adds tests that use a KMS key for signing EIFs. This new test suite requires to set environment variables TEST_KMS_KEY_ARN and TEST_CERTIFICATE_PATH.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@atanzu atanzu force-pushed the feature/tests-with-kms-keys branch from 46f656e to c33782a Compare January 29, 2025 15:43
@atanzu atanzu marked this pull request as ready for review January 31, 2025 07:15
@atanzu atanzu force-pushed the feature/tests-with-kms-keys branch from c33782a to 8a696e4 Compare February 18, 2025 08:46
This commit modifies the CI script to obtain parameters from SSM which
are needed to run tests with KMS keys.

Signed-off-by: Mark Kirichenko <[email protected]>
This commit adds tests which use KMS keys to sign EIFs.

Signed-off-by: Mark Kirichenko <[email protected]>
@atanzu atanzu force-pushed the feature/tests-with-kms-keys branch from 8a696e4 to 1ef3c41 Compare February 20, 2025 12:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants