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

Bump slsa-framework/slsa-github-generator from 1.10.0 to 2.0.0 #3385

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

Conversation

dependabot[bot]
Copy link
Contributor

@dependabot dependabot bot commented on behalf of github Apr 22, 2024

Bumps slsa-framework/slsa-github-generator from 1.10.0 to 2.0.0.

Release notes

Sourced from slsa-framework/slsa-github-generator's releases.

v2.0.0

This is an un-finalized release.

See the CHANGELOG for details.

v2.0.0-rc.0

See the CHANGELOG for details.

Changelog

Sourced from slsa-framework/slsa-github-generator's changelog.

v2.0.0

v2.0.0: Breaking Change: upload-artifact and download-artifact

  • Our workflows now use the new @v4s of actions/upload-artifact and actions/download-artifact, which are incompatiblle with the prior @v3. See Our docs on the generic generator for more information and how to upgrade.

v2.0.0: Breaking Change: attestation-name Workflow Input and Output

  • attestation-name as a workflow input to .github/workflows/generator_generic_slsa3.yml is now removed. Use provenance-name instead.

v2.0.0: DSSE Rekor Type

  • When uploading signed provenance to the log, the entry created in the log is now a DSSE Rekor type. This fixes a bug where the current intoto type does not persist provenance signatures. The attestation will no longer be persisted in Rekor (#3299)
Commits
  • 41733f7 chore: v2.0.0-rc.0: update tags (#3578)
  • 3789345 docs: v.2.0.0: finalize CHANGELOG.md (#3577)
  • 02fc78b fix: deadlock and improve debugging experience (#3570)
  • 4534a0b break: Revert "chore: Revert "fix: upload-artifact and download-artifact v4""...
  • e8c2dcf fix(deps): Update Sigstore Dep to Sigstore 2.2.2 (#3491)
  • 2512315 feat(breaking): remove attestation-name input and output (#3456)
  • 4fbc6a9 chore: add ramonpetgrave64 to CODEOWNERS (#3490)
  • 8869c8a fix: Switch to newer DSSE rekor type (#3299)
  • 9d81ca7 chore: Update slsa-verifier version (#3454)
  • d6b8c9f chore: Ref to main after v1.10.0 release (#3421)
  • See full diff in compare view

Dependabot compatibility score

Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting @dependabot rebase.


Dependabot commands and options

You can trigger Dependabot actions by commenting on this PR:

  • @dependabot rebase will rebase this PR
  • @dependabot recreate will recreate this PR, overwriting any edits that have been made to it
  • @dependabot merge will merge this PR after your CI passes on it
  • @dependabot squash and merge will squash and merge this PR after your CI passes on it
  • @dependabot cancel merge will cancel a previously requested merge and block automerging
  • @dependabot reopen will reopen this PR if it is closed
  • @dependabot close will close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually
  • @dependabot show <dependency name> ignore conditions will show all of the ignore conditions of the specified dependency
  • @dependabot ignore this major version will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this minor version will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this dependency will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)

@dependabot dependabot bot added the dependencies Pull requests that update a dependency file label Apr 22, 2024
@dependabot dependabot bot requested a review from shazow as a code owner April 22, 2024 20:12
@dependabot dependabot bot added the Skip Changelog Pull requests that don't require a changelog entry label Apr 22, 2024
@dependabot dependabot bot requested a review from illia-v as a code owner April 22, 2024 20:12
Copy link
Contributor Author

dependabot bot commented on behalf of github Apr 22, 2024

The following labels could not be found: github_actions.

@illia-v
Copy link
Member

illia-v commented Apr 23, 2024

This upgrade may break our publish workflow because the new version uses actions/upload-artifact v4 and other jobs of our workflow use v3

@dependabot dependabot bot force-pushed the dependabot/github_actions/slsa-framework/slsa-github-generator-2.0.0 branch 2 times, most recently from 0614735 to 52bd188 Compare April 26, 2024 03:07
@pquentin
Copy link
Member

Thanks. I think it's safe to move to v4 now. David Lord switched his repos this week without issues: https://mas.to/@davidism/112323427394208633. I'm not clear on whether we need to switch everything at once or not.

@dependabot dependabot bot force-pushed the dependabot/github_actions/slsa-framework/slsa-github-generator-2.0.0 branch from 52bd188 to 83c29cc Compare May 10, 2024 19:05
Bumps [slsa-framework/slsa-github-generator](https://github.com/slsa-framework/slsa-github-generator) from 1.10.0 to 2.0.0.
- [Release notes](https://github.com/slsa-framework/slsa-github-generator/releases)
- [Changelog](https://github.com/slsa-framework/slsa-github-generator/blob/main/CHANGELOG.md)
- [Commits](slsa-framework/slsa-github-generator@v1.10.0...v2.0.0)

---
updated-dependencies:
- dependency-name: slsa-framework/slsa-github-generator
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <support@github.com>
@dependabot dependabot bot force-pushed the dependabot/github_actions/slsa-framework/slsa-github-generator-2.0.0 branch from 83c29cc to f33b40e Compare May 13, 2024 18:42
Copy link
Member

@pquentin pquentin left a comment

Choose a reason for hiding this comment

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

Looking at the changelog (https://github.com/slsa-framework/slsa-github-generator/blob/main/CHANGELOG.md#v200), I only updated download-artifact and upload-artifact as it's the only thing that I think affects us. (And also removed a workaround for an old version.)

Any suggestions for how to test this?

with:
base64-subjects: "${{ needs.build.outputs.hashes }}"
upload-assets: true
compile-generator: true # Workaround for https://github.com/slsa-framework/slsa-github-generator/issues/1163
Copy link
Member

Choose a reason for hiding this comment

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

@pquentin
Copy link
Member

pquentin commented May 16, 2024

Sigh, coverage failed due to ECONNRESET in download-artifacts v4, retrying.

@illia-v
Copy link
Member

illia-v commented May 16, 2024

Any suggestions for how to test this?

I remember @sethmlarson tested changes to the publish workflow in a separate repo
There is another option: what do you think about publishing commits to https://test.pypi.org/ to ease making future changes to the workflow?

@pquentin
Copy link
Member

That would be great. Do you know of any project that does that and that I could copy from? Or any other suggestion?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file Skip Changelog Pull requests that don't require a changelog entry
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants