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

Update distroless images to use Debian 12 #31620

Merged
merged 1 commit into from
Sep 13, 2023
Merged

Update distroless images to use Debian 12 #31620

merged 1 commit into from
Sep 13, 2023

Conversation

reedloden
Copy link
Contributor

@reedloden reedloden commented Sep 8, 2023

e companion -- https://github.com/gravitational/teleport.e/pull/2154

https://github.com/GoogleContainerTools/distroless#debian-12

Note that the debian12 images no longer include OpenSSL, which we don't need anyway, as we statically link our own copy for tsh and other purposes.

Also, add Debian 12 to various places, as a supported OS.

Other included changes:

  • Standardize how we select which distroless release to use.
  • Ensure a specific distroless version is used instead of latest.
  • Add new Amazon Linux and Oracle Linux releases to compatibility testing.
  • Correct s/MacOS/macOS/ in install script.

build.assets/build-test-compat.sh Show resolved Hide resolved
https://github.com/GoogleContainerTools/distroless#debian-12

Note that the debian12 images no longer include OpenSSL, which we
don't need anyway, as we statically link our own copy for tsh and
other purposes.

Also, add Debian 12 to various places, as a supported OS.

Other included changes:
* Remove Debian 8 and 9, as they aren't receiving any security updates.
* Standardize how we select which distroless release to use.
* Ensure a specific distroless version is used instead of latest.
* Add new Amazon Linux and Oracle Linux releases to compatibility testing.
* Correct s/MacOS/macOS/ in install script.

`e` companion -- https://github.com/gravitational/teleport.e/pull/2154
@ptgott
Copy link
Contributor

ptgott commented Sep 8, 2023

@reedloden Does this require backporting?

@reedloden
Copy link
Contributor Author

@reedloden Does this require backporting?

Likely yes. Want to get this into master and just make sure it looks good before backporting.

# images that are also Debian 11 based: https://github.com/GoogleContainerTools/distroless
FROM docker.io/library/debian:11
# images that are also Debian 12 based: https://github.com/GoogleContainerTools/distroless
FROM docker.io/library/debian:12
Copy link
Contributor

Choose a reason for hiding this comment

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

This may break a few things as we're using this container for our releases for ARM (tar.gz packages). Normally I'd panic before updating it, but I think we may try to increase the GLibc version for ARM and see how it goes.
CC @tcsc @r0mant

@reedloden reedloden added this pull request to the merge queue Sep 13, 2023
Merged via the queue into master with commit b8feb63 Sep 13, 2023
33 checks passed
@reedloden reedloden deleted the reed/debian12 branch September 13, 2023 19:14
camscale added a commit that referenced this pull request Dec 8, 2023
Revert part of #31620 that updated the arm 32-bit buildbox from Debian
11 to Debian 12, rolling back to Debian 11. This increased the required
version of glibc above what we say we support. The update of the
distroless image is fine as that is self-contained. The buildbox builds
the stand-alone binaries and OS packages so must be conservative in that
it uses.

Reverts: #31620 (partial)
Issue: #35476
camscale added a commit that referenced this pull request Dec 8, 2023
Revert part of #31620 that updated the arm 32-bit buildbox from Debian
11 to Debian 12, rolling back to Debian 11. This increased the required
version of glibc above what we say we support. The update of the
distroless image is fine as that is self-contained. The buildbox builds
the stand-alone binaries and OS packages so must be conservative in that
it uses.

Reverts: #31620 (partial)
Issue: #35476
github-merge-queue bot pushed a commit that referenced this pull request Dec 8, 2023
Revert part of #31620 that updated the arm 32-bit buildbox from Debian
11 to Debian 12, rolling back to Debian 11. This increased the required
version of glibc above what we say we support. The update of the
distroless image is fine as that is self-contained. The buildbox builds
the stand-alone binaries and OS packages so must be conservative in that
it uses.

Reverts: #31620 (partial)
Issue: #35476
github-actions bot pushed a commit that referenced this pull request Dec 8, 2023
Revert part of #31620 that updated the arm 32-bit buildbox from Debian
11 to Debian 12, rolling back to Debian 11. This increased the required
version of glibc above what we say we support. The update of the
distroless image is fine as that is self-contained. The buildbox builds
the stand-alone binaries and OS packages so must be conservative in that
it uses.

Reverts: #31620 (partial)
Issue: #35476
github-merge-queue bot pushed a commit that referenced this pull request Dec 8, 2023
Revert part of #31620 that updated the arm 32-bit buildbox from Debian
11 to Debian 12, rolling back to Debian 11. This increased the required
version of glibc above what we say we support. The update of the
distroless image is fine as that is self-contained. The buildbox builds
the stand-alone binaries and OS packages so must be conservative in that
it uses.

Reverts: #31620 (partial)
Issue: #35476
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants