-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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
Introduce tbot-distroless
image
#38259
Conversation
I'll rebase out the "Release" commit once this is all approved ;) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This looks good to me, but I would rather have someone from Tooling review this as well
examples/chart/teleport-cluster/tests/__snapshot__/auth_deployment_test.yaml.snap
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
How quickly do you need to get this in? I'd very much prefer to avoid adding to Drone right now, as I am currently moving everything off it. It's a slow and painful process but I hope to have it complete by the end of the week. If you can hold off a little, I'll work with you to get this into the GHA build/publish workflows instead.
More than happy to hold off - especially if it means this can go straight into GHA instead of drone. I'll regroup with you next week on this - thank you Cam. |
2a812ee
to
fd5c68d
Compare
Successful build in the pure GHA environment: https://github.com/gravitational/teleport.e/actions/runs/8055743348 Successful publish in a pure GHA environment: https://github.com/gravitational/teleport.e/actions/runs/8056429084
|
Running v16.0.0-dev.noah.tbr.6 build to test b326428 |
Once I've got approval on both sides, I'll drop the release commits from this branch. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This has still got drone stuff in it that should be dropped. All this PR needs is the Dockerfile-tbot-distroless
file.
To keep the version stuff from a dev tag build getting on the PR, what I often do is create a test branch on top of the PR branch (strideynet/test/tbot-docker-image-remastered
in this case) and run make update-version; make update-tag
on that. The test branch does not even need to be pushed, since the tag is pushed. Then I throw away the branch when I'm done.
I don't like to approve PRs that have the version changes in it in case some accidentally gets merged - I think it's cleaner and easier to leave it off the PR.
fa2a8f3
to
0326745
Compare
Running https://github.com/gravitational/teleport.e/actions/runs/8062741799 to validate deb extraction changes and changes on e to use version action. Failed due to incorrect config Fixed config and rerunning https://github.com/gravitational/teleport.e/actions/runs/8064136416 |
Successful build, publish and run based on the last commit! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks really good now. I'm pleased with how simple it's ended up.
@strideynet See the table below for backport results.
|
Closes #21280
Paired with https://github.com/gravitational/teleport.e/pull/3465
changelog: tbot-distroless image is now published. This contains just the tbot binary and therefore has a smaller image size.
Successful build: https://drone.platform.teleport.sh/gravitational/teleport/34023
Image: public.ecr.aws/gravitational/tbot-distroless:16.0.0-dev.noah.tbr.1
Working
docker run
: