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

release-20.2: docker: Switch to using ubi8/ubi-minimal #55519

Merged
merged 1 commit into from
Oct 13, 2020

Conversation

jlinder
Copy link
Collaborator

@jlinder jlinder commented Oct 13, 2020

Backport 1/1 commits from #55467.

/cc @cockroachdb/release


Before: We were using ubi8/ubi because we hadn't figured out how to get
tzdata into the ubi8/ubi-minimal image.

Why: The ubi8/ubi-minimal image is smaller and now has everything we
need.

Now: The docker image is based on ubi8/ubi-minimal. The required tzdata
files are acquired by rpm erasing the tzdata package and then
reinstalling it (there is no microdnf reinstall command in the current
image).

Release note (backward-incompatible change): The docker image is now
based on RedHat's ubi8/ubi-minimal image (instead of ubi8/ubi). This
image is smaller.

Before: We were using ubi8/ubi because we hadn't figured out how to get
tzdata into the ubi8/ubi-minimal image.

Why: The ubi8/ubi-minimal image is smaller and now has everything we
need.

Now: The docker image is based on ubi8/ubi-minimal. The required tzdata
files are acquired by rpm erasing the tzdata package and then
reinstalling it (there is no `microdnf reinstall` command in the current
image).

Release note (backward-incompatible change): The docker image is now
based on RedHat's ubi8/ubi-minimal image (instead of ubi8/ubi). This
image is smaller.
@jlinder jlinder requested a review from bdarnell October 13, 2020 17:58
@cockroach-teamcity
Copy link
Member

This change is Reviewable

@jlinder jlinder merged commit dbd9335 into cockroachdb:release-20.2 Oct 13, 2020
@jlinder
Copy link
Collaborator Author

jlinder commented Oct 13, 2020

TFTR!

@jlinder jlinder deleted the backport20.2-55467 branch October 13, 2020 18:37
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