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

ci: disable build provenance by adding comments #4634

Merged
merged 1 commit into from
Dec 19, 2024

Conversation

giacob500
Copy link
Contributor

Summary

This PR addresses issue #4580 (ci: disable the build provenance stuff for now).

  • Disabled the build provenance steps in the GitHub Actions CI workflow.
  • Commented out the steps related to attest-build-provenance to prevent unnecessary updates for Dependabot.
  • Kept the relevant configuration in place to easily re-enable the build provenance steps in the future.

Details

While working on the CI setup, it was noted that the build provenance steps are not currently being utilized. These steps are meant to attest build provenance for the generated wheel and tar files, but there is no active use for them at the moment. As a result, we decided to comment them out to avoid unnecessary updates triggered by Dependabot, while preserving the configuration for future use.

The following steps were commented out in the .github/workflows/build-wheel.yml file:

- name: Attest Build Provenance for tar
  uses: actions/attest-build-provenance@ef244123eb79f2f7a7e75d99086184180e6d0018 # v1.4.4
  with:
    subject-path: "dist/${{ steps.filename.outputs.tar }}"

- name: Attest Build Provenance for whl
  uses: actions/attest-build-provenance@ef244123eb79f2f7a7e75d99086184180e6d0018 # v1.4.4
  with:
    subject-path: "dist/${{ steps.filename.outputs.whl }}"

Copy link
Contributor

@terriko terriko left a comment

Choose a reason for hiding this comment

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

Thank you! I think that will do the trick.

@terriko terriko merged commit f0de300 into intel:main Dec 19, 2024
24 checks passed
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