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

Initial Windows support for maven docker image #6

Closed
wants to merge 21 commits into from
Closed

Initial Windows support for maven docker image #6

wants to merge 21 commits into from

Conversation

slide
Copy link
Member

@slide slide commented Aug 7, 2019

This depends on several other docker images being pushed to docker hub. Just wanted to get it in here to get some feedback started.

@rtyler
Copy link
Member

rtyler commented Aug 8, 2019

@slide does this require a Windows environment? Or can this be plugged into the Makefile tooling already present in this environment?

@rtyler rtyler self-assigned this Aug 8, 2019
@rtyler rtyler self-requested a review August 8, 2019 15:31
@slide
Copy link
Member Author

slide commented Aug 8, 2019

To build windows containers, you need a windows system. We're in kind of a chicken and egg problem as there are no Windows agents that have docker installed to build stuff like this yet, but we need these type of images to start some of that...

Copy link
Member

@jglick jglick left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

So not yet in maven/Makefile, because it could not be, hmm. So we need to amend the Windows agent VM templates to include a Docker installation, if that is even possible?

Seems like DockerHub automated builds still do not support Windows, so we could not switch to that.

@slide
Copy link
Member Author

slide commented Aug 15, 2019

@jglick, correct, you have to have a Windows system to build a Windows docker image. I think we can add docker to the Windows VM templates, but I will have to look into it.

AppVeyor does support Docker builds, however, it is using Windows Server 2016 instead of Windows Server 2019 (which has better container support).

@jglick
Copy link
Member

jglick commented Aug 19, 2019

Well in the short term I guess we would just manually publish images built from this work. There are not that many changes anyway.

@slide
Copy link
Member Author

slide commented Aug 19, 2019

Agreed, we just need someone with Windows platform(s) (2016 and 2019 would be the best if possible) who can build the images and then publish them.

@slide
Copy link
Member Author

slide commented Aug 20, 2019

I think I can fairly easily enable Windows docker nodes on ci.jenkins.io and trusted.ci. I am requesting access to ci.jenkins.io in order to enable it.

@slide
Copy link
Member Author

slide commented Aug 21, 2019

I have successfully setup a Windows docker agent on ci.jenkins.io, so we should be able to build these images somewhat soon. I will determine how it will work best to fit into the current Makefile like framework (won't be in Makefile, but I'll try and maintain a similar idea).

@slide
Copy link
Member Author

slide commented Nov 5, 2019

We have the capability to build these on both ci.j.io and trusted now. I need to upgrade the containers to use the "official" maven images now.

@slide slide changed the title [WIP] Initial Windows support for maven docker Initial Windows support for maven docker image Dec 13, 2019
@slide
Copy link
Member Author

slide commented Dec 18, 2019

@rtyler This is ready for review if you have some time.

@slide slide requested a review from a team as a code owner January 23, 2020 17:44
@slide
Copy link
Member Author

slide commented Jan 23, 2020

Close and resubmit because I messed something up

@slide slide closed this Jan 23, 2020
@jglick
Copy link
Member

jglick commented Jan 28, 2020

#12

timja pushed a commit to timja/jnlp-agents that referenced this pull request Oct 5, 2021
fix: correct JNLP path and installations
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.

6 participants