Skip to content

Conversation

@PeterDaveHello
Copy link
Member

@PeterDaveHello PeterDaveHello requested a review from a team August 2, 2019 09:53
@LaurentGoderre
Copy link
Member

@PeterDaveHello I would vote for waiting until we can support multiple version, which we are so close to getting.

@PeterDaveHello PeterDaveHello requested review from a team, SimenB, Starefossen and pesho August 10, 2019 09:06
@PeterDaveHello
Copy link
Member Author

Hmm, but no ETA actually, I'm not sure if it should be blocking.

@LaurentGoderre
Copy link
Member

@PeteeDaveHello we're sooo close and it would avoid us ton of grief!

@bbortt
Copy link

bbortt commented Sep 9, 2019

Any news on this (respectively the multi-version-support)? Would be nice to have an alpine:3.10 image as well.

@PeterDaveHello
Copy link
Member Author

@nodejs/docker what do you think?

@tpopov94
Copy link

Is there any update on this MR? Currently, there is an issue in Alpine 3.9 related to openssl, which is resolven in Alpine 3.10?

@PeterDaveHello
Copy link
Member Author

As nobody from the team replies, I can't not merge my own PR, however, I suggest if anyone who's interested in help us deliver the next release with Alpine 3.10, just send us a pull request and I'll help review and merge it.

@nschonni
Copy link
Member

I think getting #1026 landed first might be a good idea. That way if people still need to use 3.9, they will have a fallback

@shantanugadgil
Copy link

Security scanning tools keep flagging warnings for older OS software packages. Having a "clean" security scan report is necessary for many orgs. I am adding my +1 to the OP.

@PeterDaveHello
Copy link
Member Author

#1026 is not ready yet.

@LinusU
Copy link

LinusU commented Oct 2, 2019

I'm also running into a bug in Alpine 3.9 in regards to a missing dependency on the perl-git-svn package, which is fixed in Alpine 3.10...

I'm not sure how #1026 is related to this though?

That way if people still need to use 3.9, they will have a fallback

How would this work? 🤔

@shantanugadgil
Copy link

@LinusU this hack works for me:
#1061 (comment)

@LaurentGoderre
Copy link
Member

This should be merged after #1125

@LaurentGoderre
Copy link
Member

I think this is ready to merge!

@PeterDaveHello PeterDaveHello merged commit 8f85325 into nodejs:master Nov 14, 2019
@PeterDaveHello PeterDaveHello deleted the update-alpine-linux-version-in-config branch November 14, 2019 16:45
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.

8 participants