Skip to content

ci tarball creation action - #5367

Open
dangome3 wants to merge 3 commits into
cilium:mainfrom
dangome3:pr/daniel/ci-tarball-creation
Open

ci tarball creation action#5367
dangome3 wants to merge 3 commits into
cilium:mainfrom
dangome3:pr/daniel/ci-tarball-creation

Conversation

@dangome3

@dangome3 dangome3 commented Jul 31, 2026

Copy link
Copy Markdown
Contributor

Description

There is some duplication for building the tarball step in .github/workflows/packages-e2e-tests.yaml and .github/workflows/vmpolicytests.yaml. This PR contains a reusable action so we avoid duplicated code

dangome3 added 3 commits July 31, 2026 12:13
There are several workflows that duplicates building tarballs along
with the required dependencies installation. This action will
allow reducing that repeated code.

Signed-off-by: Daniel Gomez <dangome3@cisco.com>
Signed-off-by: Daniel Gomez <dangome3@cisco.com>
Signed-off-by: Daniel Gomez <dangome3@cisco.com>
@dangome3
dangome3 force-pushed the pr/daniel/ci-tarball-creation branch from 70bed6e to f7f943d Compare July 31, 2026 18:18
@dangome3 dangome3 changed the title Pr/daniel/ci tarball creation ci tarball creation action Jul 31, 2026
@dangome3

Copy link
Copy Markdown
Contributor Author

I'm thinking about using this action in build-images-release.yml too, by removing the tarball-release make target and moving the mv+sha256sum commands into the workflow. Is there any hard reason not to make this change and keep it as is?
cc: @mtardy @kkourt

@dangome3
dangome3 marked this pull request as ready for review July 31, 2026 19:17
@dangome3
dangome3 requested a review from a team as a code owner July 31, 2026 19:17
@dangome3
dangome3 requested a review from FedeDP July 31, 2026 19:17

@FedeDP FedeDP left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I like it!

@FedeDP FedeDP added kind/enhancement This improves or streamlines existing functionality area/ci Related to CI release-note/ci This PR makes changes to the CI. labels Aug 3, 2026
id: tarball
uses: ./go/src/github.com/cilium/tetragon/.github/actions/build-tarball
with:
working-directory: go/src/github.com/cilium/tetragon

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Does this mean that we are building tarball multiple times? If so, would it make sense to only build it once and then reuse it across all workflows that needed?

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.

Sure! that makes more sense. Let me re-organize the workflows so we let this happen

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/ci Related to CI kind/enhancement This improves or streamlines existing functionality release-note/ci This PR makes changes to the CI.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants