Skip to content

Use internal fork of slsa-github-generator action#164

Merged
cpetersen5 merged 1 commit into
mainfrom
slsa-github-generator
Mar 13, 2026
Merged

Use internal fork of slsa-github-generator action#164
cpetersen5 merged 1 commit into
mainfrom
slsa-github-generator

Conversation

@cpetersen5

@cpetersen5 cpetersen5 commented Mar 12, 2026

Copy link
Copy Markdown
Contributor

What does this PR change?

This PR replaces slsa-framework/slsa-github-generator with an internally maintained fork

Does this PR rely on any other PRs?

N/A

How does this PR impact users?

N/A

Links to Issues or tickets this PR addresses or fixes

https://apptio.atlassian.net/browse/KCM-5050

What risks are associated with merging this PR? What is required to fully test this PR?

Minimal - This is just an internally maintained fork for supply chain security.

How was this PR tested?

@cpetersen5 cpetersen5 marked this pull request as ready for review March 12, 2026 20:24
packages: write
needs: build
uses: slsa-framework/slsa-github-generator/.github/workflows/generator_container_slsa3.yml@v2.0.0
uses: kubecost/slsa-github-generator/.github/workflows/generator_container_slsa3.yml@main

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Double check whether this is properly referencing the "kubecost/github-actions" repo?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Hey @thomasvn, this Action references a full Go project, so instead of forking it inside of the github-actions repo, I thought it made more sense to fork it to its own, independent repo: https://github.com/kubecost/slsa-github-generator. Let me know if you think it makes more sense to keep it in the github-actions repo though!

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

@cpetersen5 Ok! I'm good with this approach for now. We can always adjust in the future if needed.

@cpetersen5 cpetersen5 merged commit a99c421 into main Mar 13, 2026
1 of 3 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