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

wrap actions/attest-build-provenance #211

Merged
merged 2 commits into from
May 9, 2024
Merged

Conversation

bdehamer
Copy link
Collaborator

@bdehamer bdehamer commented May 7, 2024

Replace local implementation with a call to the public actions/attest-build-provenance action.

This will ensure that anyone who is still referencing the private beta version of the action is getting all of the updates and bug fixes of the public action.

Also includes a deprecation warning in the job summary:
image

@bdehamer bdehamer requested a review from a team as a code owner May 7, 2024 21:22
@bdehamer bdehamer force-pushed the bdehamer/wrap-public-action branch from 4a80625 to bf2e88e Compare May 7, 2024 21:24
@bdehamer bdehamer marked this pull request as draft May 7, 2024 21:34
@bdehamer bdehamer force-pushed the bdehamer/wrap-public-action branch 2 times, most recently from 26aef12 to 2db573f Compare May 7, 2024 21:43
@bdehamer bdehamer force-pushed the bdehamer/wrap-public-action branch from 2db573f to 3b758b9 Compare May 7, 2024 21:45
Comment on lines +53 to +58
- id: translate-output
if: runner.os != 'Windows'
shell: bash
run: |
read -r line < ${{ steps.attest.outputs.bundle-path }}
echo "bundle=$line" >> $GITHUB_OUTPUT
Copy link
Collaborator Author

@bdehamer bdehamer May 7, 2024

Choose a reason for hiding this comment

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

The primary difference between the early-access action and the public action is the way the action output is handled:

  • In the early-access action, the JSON-serialized bundle is passed back in the bundle output variable.
  • In the new action, any attestation bundles are written to a file and the path to that file is passed back in the bundle-path output variable.

This bit translates the new scheme to the old one for backward compatibility. We don't handle Windows cause there's no easy way to handle file paths on both Windows and non-Windows runners.

Copy link
Contributor

Choose a reason for hiding this comment

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

The only user I'd worry about for the Windows platform is Homebrew... but I believe yesterday they switched to actions/attest-build-provenance so I think we can proceed!

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Also, I checked and they are not using the action output

@bdehamer bdehamer marked this pull request as ready for review May 7, 2024 21:51
Copy link
Contributor

@steiza steiza left a comment

Choose a reason for hiding this comment

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

I did not review the 101,076 lines you deleted 😐

.github/workflows/e2e.yml Outdated Show resolved Hide resolved
Comment on lines +53 to +58
- id: translate-output
if: runner.os != 'Windows'
shell: bash
run: |
read -r line < ${{ steps.attest.outputs.bundle-path }}
echo "bundle=$line" >> $GITHUB_OUTPUT
Copy link
Contributor

Choose a reason for hiding this comment

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

The only user I'd worry about for the Windows platform is Homebrew... but I believe yesterday they switched to actions/attest-build-provenance so I think we can proceed!

@bdehamer bdehamer merged commit a727ae5 into main May 9, 2024
4 checks passed
@bdehamer bdehamer deleted the bdehamer/wrap-public-action branch May 9, 2024 15:15
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.

2 participants