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

feat: node.js v16.0.0 #1465

Merged
merged 2 commits into from
Apr 22, 2021
Merged

feat: node.js v16.0.0 #1465

merged 2 commits into from
Apr 22, 2021

Conversation

dnalborczyk
Copy link
Contributor

@dnalborczyk dnalborczyk commented Apr 20, 2021

Description

  • removed alpine v3.10 as it is EOL very soon: 2021-05-01

Motivation and Context

Testing Details

Example Output(if appropriate)

Types of changes

  • Documentation
  • Version change (Update, remove or add more Node.js versions)
  • Variant change (Update, remove or add more variants, or versions of variants)
  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Others (non of above)

Checklist

  • My code follows the code style of this project.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have read the CONTRIBUTING.md document.
  • All new and existing tests passed.

Sorry, something went wrong.

&& case "${alpineArch##*-}" in \
x86_64) \
ARCH='x64' \
CHECKSUM="" \
Copy link
Member

Choose a reason for hiding this comment

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

We're blocked till the Alpine binaries are built https://unofficial-builds.nodejs.org/download/release/v16.0.0/

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I see, only skimmed over the Dockerfile. I thought I had it building locally. out of curiosity, what is the "Building from source" condition for?

Copy link
Member

Choose a reason for hiding this comment

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

I think mostly a fail safe for anyone doing a manual build. The dockerhub image layers are built using the binary version though, so we wait till they're ready

Copy link
Contributor

Choose a reason for hiding this comment

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

Also for the other architectures beyond the three that get built in the unofficial builds project. 😅

Copy link
Member

Choose a reason for hiding this comment

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

Yeah, I was going to caveat that around the s360 etc.., but it's a confusing story

@nschonni
Copy link
Member

Thanks! I did the same thing locally waiting for the binaries. I agree that 3.10 doesn't make sense to include this close to the EOL

@SimenB
Copy link
Member

SimenB commented Apr 21, 2021

@nschonni
Copy link
Member

@SimenB being tracked here nodejs/unofficial-builds#28 (comment)
It hit the similar Python 3 thing we did before

@rvagg
Copy link
Member

rvagg commented Apr 22, 2021

it be fixed now, thanks for your patience, sorry to hold up

@SimenB
Copy link
Member

SimenB commented Apr 22, 2021

@dnalborczyk can you regenerate?

Verified

This commit was signed with the committer’s verified signature.
nschonni Nick Schonning
Copy link
Member

@SimenB SimenB left a comment

Choose a reason for hiding this comment

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

Do we need 3 versions of alpine? Would it be enough with the last 2 (or even just 3.13)?

@SimenB
Copy link
Member

SimenB commented Apr 22, 2021

We should also remove stretch, it has been EOL since July 2020

@nschonni
Copy link
Member

Do we need 3 versions of alpine? Would it be enough with the last 2 (or even just 3.13)?

I was looking for my old thread trying to nail down policy on these ones, but since this seems to be the approach we have now, I'm going to go ahead an land

@nschonni nschonni merged commit 6943203 into nodejs:main Apr 22, 2021
@SimenB
Copy link
Member

SimenB commented Apr 22, 2021

why do we land with EOL distros?

@github-actions
Copy link

Created PR on the official-images repo (docker-library/official-images#10030). See https://github.com/docker-library/faq#an-images-source-changed-in-git-now-what if you are wondering when it will be available on the Docker Hub.

@SimenB
Copy link
Member

SimenB commented Apr 22, 2021

I don't think we should release v16 with stretch as v16 is not scheduled to be EOL until 2024

@nschonni
Copy link
Member

We should also remove stretch, it has been EOL since July 2020

image

Stretch LTS is still active till 2022

@SimenB
Copy link
Member

SimenB commented Apr 22, 2021

Security updates have been discontinued as of July 6th, 2020.

https://www.debian.org/releases/stretch/

And whenever they stop in 2022 we'll need to keep it for 2 more years

(not as bad as I was thinking since they have some sort of LTS, but I still think we should drop it for a release line that will become LTS)

@nschonni
Copy link
Member

I think we've dropped tags/distros like Alpine mid-way through release lines already. EX: 16.10.0 might be the last 16-stretch tag if we decide to drop it for 16.11

@nschonni
Copy link
Member

Probably better to open a separate dicussion though, as this exchange will never be findable again

@dnalborczyk dnalborczyk deleted the v16 branch April 22, 2021 14:47
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.

None yet

5 participants