-
Notifications
You must be signed in to change notification settings - Fork 548
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
call e2e test for cosign attach #3112
Conversation
Codecov Report
@@ Coverage Diff @@
## main #3112 +/- ##
=======================================
Coverage 30.64% 30.64%
=======================================
Files 155 155
Lines 9791 9791
=======================================
Hits 3000 3000
Misses 6341 6341
Partials 450 450 |
It looks like the test itself has an issue. Would you be able to take a look? Or @Mukuls77? |
Hi @haydentherapper i checked the script for test case. it seems to work ok on my local setup. As i see the issue seems to be in the permission of the github workflow user to call the attach test script. Run ./test/e2e_test_attach.sh please find the logs of the test case i checked by invoking the script on my local setup. /home/mukul/cosign>./test/e2e_test_attach.sh
Verification for ttl.sh/cosign-ci/d442caf9@sha256:2376a0c12759aa1214ba83e771ff252c7b1663216b192fbe5e0fb364e952f85c --
[{"critical":{"identity":{"docker-reference":"ttl.sh/cosign-ci/d442caf9"},"image":{"docker-manifest-digest":"sha256:2376a0c12759aa1214ba83e771ff252c7b1663216b192fbe5e0fb364e952f85c"},"type":"cosign container image signature"},"optional":{"Subject":"[email protected]"}}]
|
@s1ntaxe770r Could you run |
Signed-off-by: s1ntaxe770r <[email protected]>
Signed-off-by: s1ntaxe770r <[email protected]>
🫡 done. |
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.
Thanks!
Summary
This pr adds a new step in the workflow to run the e2e test for
cosign attach
.Fixes #3111
Release Note
Documentation