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

Add bullseye & bullseye-slim versions #357

Merged
merged 1 commit into from
Aug 16, 2021

Conversation

ojab
Copy link
Contributor

@ojab ojab commented Aug 15, 2021

bullseye is out, let's add it.

Closes #356

@ojab
Copy link
Contributor Author

ojab commented Aug 15, 2021

+ echo '62eefa55030788d409003eccd945ccc408f02fe0d71616ed1c1bdcaf7a2f8a54 *autoconf-2.70.patch'
+ sha256sum --check --strict
sha256sum: WARNING: 1 computed checksum did NOT match

in 2.6.8-alpine3.14 CI and looks like I can't restart workflow to recheck. Unrelated, sending to review.

@ojab ojab marked this pull request as ready for review August 15, 2021 07:40
@ojab ojab mentioned this pull request Aug 15, 2021
@tianon
Copy link
Member

tianon commented Aug 16, 2021

As in docker-library/python#633, this should also drop all traces of stretch (since we build from source, every variant is a non-trivial amount of added load, so we only maintain max two per distro).

@tianon
Copy link
Member

tianon commented Aug 16, 2021

(You'll also need to update/run versions.sh)

@ojab
Copy link
Contributor Author

ojab commented Aug 16, 2021

Done, CI is green except the same 2.6.8-alpine3.14 failure.

Copy link
Member

@tianon tianon left a comment

Choose a reason for hiding this comment

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

Very close, just a couple minor nits -- happy to take over from here if you'd prefer. 👍

Dockerfile.template Outdated Show resolved Hide resolved
versions.sh Outdated
Comment on lines 71 to 72
"buster",
if env.rcVersion == "2.6" then
"stretch"
else empty end
"bullseye"
Copy link
Member

Choose a reason for hiding this comment

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

These should be in the opposite order, and ought to have matching whitespace 😅

Suggested change
"buster",
if env.rcVersion == "2.6" then
"stretch"
else empty end
"bullseye"
"bullseye",
"buster"

Copy link
Contributor Author

Choose a reason for hiding this comment

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

oh, whitespaces 🤦
So AFAIU we want bullseye to be the default one?

Copy link
Member

Choose a reason for hiding this comment

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

Yep!

Copy link
Member

@tianon tianon left a comment

Choose a reason for hiding this comment

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

LGTM, thanks!

@tianon tianon merged commit 301b52c into docker-library:master Aug 16, 2021
docker-library-bot added a commit to docker-library-bot/official-images that referenced this pull request Aug 16, 2021
Changes:

- docker-library/ruby@301b52c: Merge pull request docker-library/ruby#357 from ojab/add_bullseye
- docker-library/ruby@4916859: Add `bullseye` & `bullseye-slim` versions, drop `stretch` & `stretch-slim`
- docker-library/ruby@fc5c093: Merge pull request docker-library/ruby#358 from infosiftr/confused
- docker-library/ruby@e439599: Update patch file shasum
p8 added a commit to p8/buildkite-config that referenced this pull request Aug 19, 2021
The latest Ruby Docker image upgraded from Debian to Bullseye:
docker-library/ruby#357

Bullseye no longer supports Node v10.x, which cause NPM to not be
installed, resulting in failing builds.

As Node v10.x is no longer supported we should switch to a supported
version. Active support for v14 ends in November 2021, so we might as
well switch to v16. See: https://endoflife.date/nodejs

Steps to reproduce:

    $ git clone https://github.com/rails/rails
    $ cd rails
    $ git clone https://github.com/rails/buildkite-config .buildkite/
    $ RUBY_IMAGE=ruby:3.0.2 docker-compose -f .buildkite/docker-compose.yml build base && CI=1 docker-compose -f .buildkite/docker-compose.yml run actionview runner actionview 'rake test:ujs'
@ojab ojab mentioned this pull request Jun 13, 2023
@ojab ojab deleted the add_bullseye branch June 13, 2023 12:59
atosbucket added a commit to atosbucket/rails-buildkit that referenced this pull request Mar 22, 2024
The latest Ruby Docker image upgraded from Debian to Bullseye:
docker-library/ruby#357

Bullseye no longer supports Node v10.x, which cause NPM to not be
installed, resulting in failing builds.

As Node v10.x is no longer supported we should switch to a supported
version. Active support for v14 ends in November 2021, so we might as
well switch to v16. See: https://endoflife.date/nodejs

Steps to reproduce:

    $ git clone https://github.com/rails/rails
    $ cd rails
    $ git clone https://github.com/rails/buildkite-config .buildkite/
    $ RUBY_IMAGE=ruby:3.0.2 docker-compose -f .buildkite/docker-compose.yml build base && CI=1 docker-compose -f .buildkite/docker-compose.yml run actionview runner actionview 'rake test:ujs'
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.

Plan to 3-bullseye?
2 participants