-
Notifications
You must be signed in to change notification settings - Fork 2k
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 stretch variant #465
Add stretch variant #465
Conversation
This is the first step to transitioning over to stretch for the base images
What should the next steps look like? Do we switch the default for all versions at some point, or do we need a way to switch the default separately for the different release branches? |
I'm still thinking about next steps. I guess, in terms of deadline, we'll need to switch to stretch either by the EOL date for full support (~June 6th 2018) or the EOL date for LTS support (~June 6th 2020): https://wiki.debian.org/DebianReleases#Production_Releases I lean more towards going with the full support EOL date. My interpretation of how Debian does LTS is that it's not "real" LTS since it's volunteer based and things might be looser in terms of security updates. I'm probably wrong on that but I find the distinction and their use of "LTS" a bit confusing. It would be good to know what the plan is for all the other Official Docker images that are Debian based. If all the official images are going to switch to stretch at some point it would be good to coordinate. For the docker-node image, I would prefer to switch the default for all versions rather than having a mix of jessie and stretch. In practical terms I think we should do the switch when we update Node so people can roll back or pin if they have to. Also it just occurred to me that we will probably want to create a jessie variant as well to help with the transition. How long we keep that around will also depend on which EOL date we to stick to. |
@chorrell, we are starting an effort to label all our images with the appropriate OS release: ( We have not figured out a general solution for changing the default from jessie to stretch (or alpine 3.5 to 3.6). I know for golang we are making the next release, |
Any reason not to merge this? Switching defaults can be done later, right? Can you update with the changes necessary after merging #452? Also move 8.1 to 8.2 |
Also updates yarn to 0.27.5
Update for 8.2.1. |
Stretch is just amd64? (I have no idea, just asking) |
Never mind, you just didn't generate new ones after updating the template 😄 That's what I get for being eager. |
Right, the arch updates haven't really landed in the docker files yet, just the templates. |
So, do I need to modify the architectures file to include stretch?
|
@yhwang ^ |
oh, in a recent change, I changed |
After merging this (and updating the dockerfiles), are we ready for a PR to docker hub with ppc64le and stretch? |
@yhwang, sounds good to me! |
Thanks @SimenB . I used @chorrell 's @chorrell since you are the author for the stretch variant (I just verified it on ppc64le for you), do you want to add |
Done. |
Red CI is fixed on master. Any reason not to merge this? |
Sounds good to me. |
Are we ready for docker hub pr with stretch and new arch? |
Stretch: yes I think so New arch: not sure. If upstream/Docker is good us adding the new arch stuff then I guess that's fine. |
In node download, the binary files for different architectures are already there. I think it's also good to support multiple architectures for docker user. One question about the process:
|
It's the infra of docker that builds, we just point them to the dockerfiles (with the stackbrew diff) |
thanks @SimenB . understood! Then no need to worry about the servers. great! |
@SimenB do you have an ETA of when it will be available on docker hub? :) |
@victornoel It's available now |
@SimenB thanks for notifying and for the link, I didn't understand the process actually, now I know :) |
@SimenB actually, that's strange, even though the stretch variant is present in ppc64le (https://hub.docker.com/r/ppc64le/node/), the amd64 one is not up to date… |
Is there a schedule when stretch will be the default image for node? |
No not yet. |
@victornoel stretch is here: https://hub.docker.com/_/node/ |
@SimenB that's my point, if you look at the tag list: https://hub.docker.com/r/library/node/tags/ it is not there, the last push is from a day ago while the PR was merged half a day ago. |
Ah! And |
@tianon ^ ideas? |
There was a hiccup or two in building/pushing, but all artifacts should be (long-since) up now: $ docker pull node:stretch
stretch: Pulling from library/node
Digest: sha256:2afc089d9902461f94df2c44e2595a4625c3994a4b18f305703ba9607e09886d
Status: Image is up to date for node:stretch
$ docker pull ppc64le/node:stretch
stretch: Pulling from ppc64le/node
Digest: sha256:f6cb450cc56f264e313d00874cef2e6e2d4c765a3606ac3499ef560816e77544
Status: Image is up to date for ppc64le/node:stretch
|
They're also here: https://hub.docker.com/r/library/node/tags/ Thanks! |
This is the first step to transitioning over to Debian Stretch (9) for the base images.
The yarn versions should match up with what we currently have set for a given Node.js version, but it would be great if someone could sanity check that for me :)
Also note that full support for Jessie will drop around June 2018. That might be the right timing for a transition or at least something to consider