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

Replace EOL Debian Buster with Bullseye in Dockerfile-node #47204

Merged
merged 2 commits into from
Oct 4, 2024

Conversation

gzdunek
Copy link
Contributor

@gzdunek gzdunek commented Oct 4, 2024

Electron upgrade to version 32.1.2 failed because Electron started requiring GCC v10:

g++: error: unrecognized command line option ‘-std=gnu++20’; did you mean ‘-std=gnu++2a’?

While we could probably install it, I noticed that the Buster image we use, went EOL some time ago nodejs/docker-node#2114.
Updating to Bullseye resolves the problem with GCC version.
The only downside is that it increases the glibc requirement, from 2.29 to 2.31. However, this should not have a minimal impact on users, for example, Ubuntu 20.04 LTS has glibc 2.31.

Successful tag build https://github.com/gravitational/teleport.e/actions/runs/11180298411/job/31081783865

changelog: Teleport Connect for Linux now requires glibc 2.31 or later

@avatus avatus added this pull request to the merge queue Oct 4, 2024
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Oct 4, 2024
@avatus avatus added this pull request to the merge queue Oct 4, 2024
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Oct 4, 2024
@avatus avatus added this pull request to the merge queue Oct 4, 2024
Merged via the queue into master with commit 62abc57 Oct 4, 2024
42 of 43 checks passed
@avatus avatus deleted the gzdunek/update-docker-node branch October 4, 2024 19:54
@public-teleport-github-review-bot

@gzdunek See the table below for backport results.

Branch Result
branch/v14 Create PR
branch/v15 Create PR
branch/v16 Create PR

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.

3 participants