Skip to content

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

Merged
merged 13 commits into from
Dec 7, 2020

Conversation

garethjevans
Copy link
Contributor

No description provided.

@garethjevans garethjevans requested a review from a team as a code owner November 5, 2020 14:47
@daniel-beck
Copy link
Member

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.

@slide
Copy link
Member

slide commented Nov 20, 2020

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.

@daniel-beck
Copy link
Member

In order to run on Google Cloud, Windows docker images have to be at least 1909.

So this could replace the 1809 with 2019, easy. We'd just end up in another #1033 if we keep things around forever.

@slide
Copy link
Member

slide commented Nov 20, 2020

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

@slide
Copy link
Member

slide commented Nov 20, 2020

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.

@daniel-beck
Copy link
Member

daniel-beck commented Nov 20, 2020

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?

@slide
Copy link
Member

slide commented Nov 20, 2020

Yes, there is a blue ocean image that is old and not maintained that could possibly be in that category

@MarkEWaite
Copy link
Contributor

MarkEWaite commented Nov 20, 2020 via email

@slide
Copy link
Member

slide commented Nov 20, 2020

FYI, I am adding 20H2 images for Adoptium.

AdoptOpenJDK/openjdk-docker#472

@garethjevans
Copy link
Contributor Author

@slide 2019 ltsc also appears to be supported on Google Cloud - https://cloud.google.com/compute/docs/images/os-details#windows_server

@slide
Copy link
Member

slide commented Nov 23, 2020

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?

@garethjevans
Copy link
Contributor Author

This is only "jdk11 hotspot" at the moment, I've removed the other images

@slide
Copy link
Member

slide commented Nov 23, 2020

@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.

@daniel-beck
Copy link
Member

Is this even being built? I just see a file added, are we iterating over directories for this?

@slide
Copy link
Member

slide commented Dec 3, 2020

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.

@slide
Copy link
Member

slide commented Dec 4, 2020

@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.

@garethjevans garethjevans merged commit a1b42da into jenkinsci:master Dec 7, 2020
@@ -0,0 +1,71 @@
# escape=`
FROM jenkins4eval/openjdk:11-hotspot-windowsservercore-ltsc2019
Copy link
Member

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?

Copy link
Contributor Author

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

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