Skip to content

Conversation

@danlishka
Copy link
Member

Adding the end-to-end tests with KMS and autogenerated keys. All tests were conducted on Alpine, Ubuntu, and UBI9.

Screenshot 2024-06-06 at 23 31 57

danlishka added 2 commits June 6, 2024 23:37
Signed-off-by: Daniel Liszka <daniel@chainloop.dev>
Signed-off-by: Daniel Liszka <daniel@chainloop.dev>
@danlishka danlishka force-pushed the integration_tests branch from c8ab04d to e4d9ee7 Compare June 6, 2024 21:41
Signed-off-by: Daniel Liszka <daniel@chainloop.dev>
@danlishka danlishka requested review from jiparis and migmartri June 6, 2024 21:50
DIR="$( cd "$( dirname "${BATS_TEST_FILENAME}" )/.." >/dev/null 2>&1 && pwd )"
PATH="$DIR/:$PATH"
export PATH="$DIR/:$PATH:/usr/local/bin/chainloop_bin"
export DO_NOT_TRACK=1
Copy link
Member

Choose a reason for hiding this comment

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

thanks!

and btw, what organization are you running these tests against? cc/ @javirln

Copy link
Member Author

Choose a reason for hiding this comment

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

demo org


# Abats test_tags=bats:focus
@test "full attestation flow" {
export CHAINLOOP_WORKFLOW_NAME="chainloop-labs-tests"
Copy link
Member

Choose a reason for hiding this comment

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

is this env variable something that the chainloop_attestation_init wrapper understands? I am asking because we've removed a similar (or same) env variable from the init command itself in the CLI.

Copy link
Member Author

Choose a reason for hiding this comment

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

yes, this is in the wrapper added by @javirln at some point

@danlishka danlishka merged commit 797374f into main Jun 7, 2024
@danlishka danlishka deleted the integration_tests branch June 7, 2024 09:46
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