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

build: Pin GitHub Actions #3323

Merged
merged 1 commit into from Nov 17, 2022
Merged

build: Pin GitHub Actions #3323

merged 1 commit into from Nov 17, 2022

Conversation

pjbgf
Copy link
Member

@pjbgf pjbgf commented Nov 16, 2022

The main benefit of pinning GitHub actions is the determinism it brings in terms of what version of a given action will be executed. This is a step towards having hermetic builds.

Once pinned to a commit, Dependabot will automatically issue PRs to update to newer versions. For the time being this will happen every Monday.

Pinned versions is the only security metric from OpenSSF scorecard that this repository currently have a zero score.

image


This PR focus on external actions. Flux actions that live in pkg/actions will be dealt with separately.

@pjbgf pjbgf added the area/ci CI related issues and pull requests label Nov 16, 2022
@stefanprodan
Copy link
Member

I think we need to add comments with semver, see sigstore/cosign@b7efcb9

@pjbgf pjbgf force-pushed the hermetic-actions branch 3 times, most recently from e70a427 to 3b5113a Compare November 17, 2022 13:43
@pjbgf
Copy link
Member Author

pjbgf commented Nov 17, 2022

The comments are not a hard requirement and the official documentation is quite slim on it.

My current understanding is that it can be used to pin to a specific semver. I added the comments across external dependencies, apart from the GitHub official actions, which we won't restrict updates to a given semver.

The main benefit of pinning GitHub actions is the determinism it brings
in terms of what version of a given action will be executed. This is
a step towards having hermetic builds.

Once pinned to a commit, dependabot will automatically issue PRs to update
to newer versions.

Pinned versions is the only security metric from OpenSSF scorecard that
this repository currently have a zero score.

Signed-off-by: Paulo Gomes <paulo.gomes@weave.works>
Copy link
Member

@stefanprodan stefanprodan left a comment

Choose a reason for hiding this comment

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

LGTM

Thanks @pjbgf

@pjbgf pjbgf merged commit 1fca76c into fluxcd:main Nov 17, 2022
@pjbgf pjbgf deleted the hermetic-actions branch November 17, 2022 16:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/ci CI related issues and pull requests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants