-
Notifications
You must be signed in to change notification settings - Fork 25
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
CI: "Generate provenance" fails #545
Comments
Hey, thanks for raising this. I hadn't noticed the failing builds on master. For anyone looking at this, the error seems (ex: this build) to be related to the cosign.pub that is pulled from the slsa-provenance-action:
This same issue was seen in philips-labs/slsa-provenance-action#161. Our makefile does use The suggested fix is to disable COSIGN_EXPERIMENTAL: - name: Generate provenance
uses: philips-labs/[email protected]
with:
command: generate
subcommand: files
arguments: --artifact-path mock
+ env:
+ COSIGN_EXPERIMENTAL: 0 |
Remember to also remove @maelvls 's warning note from the release process when we eventually fix this. |
I tried to fix this in #553 but the simple fix @maelvls suggested does not seem to fix this. I have then tried to run this thing locally: # Try recreating locally like the release file
echo "foobar" > mock
/Users/peter.fiddes/projects/philips-labs/slsa-provenance-action/bin/slsa-provenance generate files --artifact-path mock
# Try doing it properly
/Users/peter.fiddes/projects/philips-labs/slsa-provenance-action/bin/slsa-provenance generate container --repository "quay.io/jetstack/preflight" --output-path "provenance.json" --digest "sha256:48e30b96726a8e0dc09c68bdf9e6153cee10d5874f988745519b6ea75c579192" --tags "v0.1.49" All getting a different error because it's running outside of GH actions:
I know nothing about this, but any suggestions on how I can provide the "github-context" this tool requires? If there is no easy way to run it locally to attempt to rectify this... I suggest we just nuke these build steps, because they haven't worked in so long and not a single person has complained. That should be the fall back plan. Also I realise that this issue is a duplicate of #472. |
I wonder if
and then you can use the flag:
But I'd probably try |
The following step in the workflow always seems to fail for quite a while: https://github.com/jetstack/jetstack-secure/blob/master/.github/workflows/release-master.yml#L95-L100
We don't see it because it only runs on master branch. Given no-one has noticed or reported I take it is not a huge issue.
But perhaps we should check why it is failing and look to fix?
The text was updated successfully, but these errors were encountered: