Skip to content

tetragon: tarball deployment #647

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

Merged
merged 4 commits into from
Jan 25, 2023
Merged

tetragon: tarball deployment #647

merged 4 commits into from
Jan 25, 2023

Conversation

tixxdz
Copy link
Member

@tixxdz tixxdz commented Jan 19, 2023

Signed-off-by: Djalal Harouni [email protected]

@tixxdz tixxdz requested a review from a team as a code owner January 19, 2023 09:27
@tixxdz tixxdz requested review from jrfastab and removed request for a team January 19, 2023 09:27
@tixxdz tixxdz marked this pull request as draft January 19, 2023 09:27
@tixxdz tixxdz force-pushed the pr/tixxdz/tarball-deployment branch from 7f98d49 to dab135c Compare January 19, 2023 09:33
@tixxdz tixxdz marked this pull request as ready for review January 19, 2023 13:00
@tixxdz tixxdz force-pushed the pr/tixxdz/tarball-deployment branch from 98c10ec to 782c0d8 Compare January 19, 2023 13:00
Copy link
Contributor

@kkourt kkourt left a comment

Choose a reason for hiding this comment

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

LGTM, assuming @olsajiri's feedback is addressed.

Thanks! (especially for the nice documentation!)

Base automatically changed from pr/tixxdz/better-config-handling to main January 23, 2023 14:24
@tixxdz tixxdz force-pushed the pr/tixxdz/tarball-deployment branch 5 times, most recently from 1f67c23 to 968be86 Compare January 25, 2023 10:59
When building tarballs we want to produce files in following format:
tetragon-$(version)-{amd64|arm64}.tar.gz, to achieve this let's set
some architecture variables inside make.

Define exported TARGET_ARCH as the main target architecture of main
Makefile, and use BPF_TARGET_ARCH to set the BPF ARCH.

While we are it add an non exported ARCH var that defaults to amd64
for Golang binaries. For arm64/aarch64 builds setting this to arm64
should just work.
For bpf the corresponding BPF_TARGET_ARCH=arm64 must also be set,
this can be achieved by getting parent TARGET_ARCH value.

Signed-off-by: Djalal Harouni <[email protected]>
This adds the linux-tarball deployment resource files with their
Dockerfile.tarball file to make a docker image with one layer that
can be dumped to a tar -> tar.gz

Include sane and secure configurations for a non k8s deployment.

Signed-off-by: Djalal Harouni <[email protected]>
This explicitly uses make image as a base so we have same workflow.

Signed-off-by: Djalal Harouni <[email protected]>
@tixxdz tixxdz force-pushed the pr/tixxdz/tarball-deployment branch from 7a17a45 to ae2d3ed Compare January 25, 2023 13:13
@tixxdz tixxdz merged commit d0f4f56 into main Jan 25, 2023
@tixxdz tixxdz deleted the pr/tixxdz/tarball-deployment branch January 25, 2023 13:35
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.

3 participants