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 alpine variant for node v0.12 #280

Closed
wants to merge 1 commit into from
Closed

add alpine variant for node v0.12 #280

wants to merge 1 commit into from

Conversation

PeterDaveHello
Copy link
Member

No description provided.

@LaurentGoderre
Copy link
Member

v0.12 will be removed in a month as per #241. It wasn't included originally because of many build issues.

@PeterDaveHello
Copy link
Member Author

@LaurentGoderre I tested and didn't found any issue 😄

@chorrell
Copy link
Contributor

I'd like to pass on this. v0.12 will be EOL at the end of December and we deliberately did not add an alpine variant because of this reason.

@PeterDaveHello
Copy link
Member Author

Why not still provide it before the EOL if we didn't provide it originally by build issues? Didn't see an overhead or something bad.

@Starefossen
Copy link
Member

I'm hesitant to encourage the continued use of node v0.12 past its EOL.

@chorrell
Copy link
Contributor

Because we want to actively trying and get people to stop using v0.12 and switch to another LTS release like v4 or v6.

@PeterDaveHello
Copy link
Member Author

If we do so really want to push the new versions and drop the old ones, maybe we should remove all the v0.12 build there, not just prevent providing v0.12 based on Alpine?

@Starefossen
Copy link
Member

@PeterDaveHello yes, they will be removed as soon as v0.12 goes EOL in a month.

@chorrell
Copy link
Contributor

There's one more pending update to v0.12 in early December: nodejs/Release#159

@PeterDaveHello
Copy link
Member Author

PeterDaveHello commented Nov 28, 2016

As the other Debian based images still there, the users can still use v0.12 on that, even tons of other images and method to get v0.12, and v0.12 will still have a new release in next week, why not just make v0.12 as other versions have an alpine based build? I don't think this really help pushing v4 or v6, make only make the users one less choice.

@PeterDaveHello
Copy link
Member Author

Currently, I can only see these patterns actually:

  • Alpine and Debian build
  • not EOL versions

v0.12 mates the same patterns as others.

@Starefossen
Copy link
Member

Starefossen commented Nov 28, 2016 via email

@PeterDaveHello
Copy link
Member Author

hmmmm ... isn't the test script scales to the versions automatically?

@chorrell
Copy link
Contributor

He means as part of the Node.js testing CI setup, not the test script in this repo

Yeah, I think v4 might be the cutoff.

@PeterDaveHello
Copy link
Member Author

Any possibility to fix that? If no, maybe just close this PR.

@chorrell
Copy link
Contributor

At this point, I don't think so. It's too close to the EOL date for the build group to consider adding v0.12 Alpine Tests.

@PeterDaveHello
Copy link
Member Author

I thought it's tested here?
https://travis-ci.org/nodejs/docker-node/jobs/179510192#L356

@chorrell
Copy link
Contributor

No, I mean the Node.js ci infrastructure: https://ci.nodejs.org

I'm not too familiar with how things are setup there. They have a couple of Alpine nodes for testing, but I don't know the full details of what test are being run. I'm pretty sure, based on past conversations in some other GitHub issues that they were starting with v4 and up.

@PeterDaveHello
Copy link
Member Author

hmmm ... if the test here is not enough, okay let's close this.

@PeterDaveHello
Copy link
Member Author

BTW, thank you all for your time.

@PeterDaveHello PeterDaveHello deleted the alpine-node-v0.12 branch November 30, 2016 16:26
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.

4 participants