-
-
Notifications
You must be signed in to change notification settings - Fork 4.7k
feat: add windows images for LTSC 2019 #1020
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
Conversation
I would like to see the data that says there's even a single user for each of the images you're adding here. I suspect there is not. |
In order to run on Google Cloud, Windows docker images have to be at least 1909. We have received requests to run on Google Cloud, so that is where this is coming from. We could probably reduce to just jdk11 and so forth to reduce the amount of time it takes. |
So this could replace the 1809 with 2019, easy. We'd just end up in another #1033 if we keep things around forever. |
not necessarily containers work a little differently on Windows sometimes you need the actual version of Windows that you're running on to be able to use a container |
I would like to propose that we follow something similar to what Adoptium (the artiest formerly known as AdoptOpenJDK). We should have a small set of "official" images that we publish that are the ones that you would need to worry about for security releases and other "unofficial" images that you don't. I'll start a topic about this with the platform SIG. |
That seems like an interesting approach. Thank you for taking this on! Don't we have a weird blueocean image around somewhere that would probably be unofficial? |
Yes, there is a blue ocean image that is old and not maintained that could possibly be in that category |
The blueocean image was used by the "Installing Docker" instructions on
jenkins.io and by three tutorials on jenkins.io until Vlad Silverman's pull
request to use the jenkins/jenkins images about a month ago.
It is not maintained but as far as I know, it is kept current by automated
processes that use the most recent LTS and the most recent Blue Ocean
plugin releases.
I believe the blueocean image should be officially declared "deprecated"
now that we have install instructions and tutorials dependencies fixed to
use jenkins/jenkins instead of the blueocean image.
…On Fri, Nov 20, 2020 at 4:04 PM Alex Earl ***@***.***> wrote:
Yes, there is a blue ocean image that is old and not maintained that could
possibly be in that category
—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHub
<#1020 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AABGIDLIHFG26ULZCUXS7ZTSQ3KWFANCNFSM4TLOPXJA>
.
|
FYI, I am adding 20H2 images for Adoptium. |
@slide 2019 ltsc also appears to be supported on Google Cloud - https://cloud.google.com/compute/docs/images/os-details#windows_server |
I think we can reduce this to just one added image if we just provide a jdk11 hotspot image instead of all four variants. Thoughts? |
This is only "jdk11 hotspot" at the moment, I've removed the other images |
@daniel-beck Are you ok adding this image since we actually reduced the number of windows images with my other PR? We would be at two images instead of four if we merge this. I will also look at parallelizing the Windows builds like we have for the Linux side. |
Is this even being built? I just see a file added, are we iterating over directories for this? |
Yes, it is being built. The make.ps1 file finds all the Windows Dockerfiles and builds them. See https://ci.jenkins.io/blue/organizations/jenkins/Packaging%2Fdocker/detail/PR-1020/98/pipeline/28#step-29-log-169 for instance. |
@daniel-beck We've reduced the number of Windows builds, are you ok with merging this? This would bring us to 2 when we had 4 before. You have "Changed Requested" in your review and I would like to move forward with this. |
@@ -0,0 +1,71 @@ | |||
# escape=` | |||
FROM jenkins4eval/openjdk:11-hotspot-windowsservercore-ltsc2019 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why are we pretending that jenkins4eval
is "unofficial" if we end up just repackaging it into jenkins
images?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@daniel-beck these images are not yet published by AdoptOpenJDK, there is an open PR for this but we've had little progress in getting it merged. When AdoptOpenJDK start supporting these base images we can remove these.
Would you prefer these were published under a different org? I'm not entirely sure what you are recommending here.
For reference - AdoptOpenJDK/openjdk-docker#365
No description provided.