-
Notifications
You must be signed in to change notification settings - Fork 242
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
[JENKINS-47703] - allow building dockerfile on Windows #354
[JENKINS-47703] - allow building dockerfile on Windows #354
Conversation
jetersen
commented
Oct 10, 2019
•
edited
edited
- JENKINS issue(s):
- JENKINS-47703
- Description:
- Support docker image build on Windows, related to Features/windows agents with pathing issues fixed docker-workflow-plugin#184
- Documentation changes:
- N/A
- Users/aliases to notify:
- @bitwiseman
Would be great to include in current release and the beta |
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.
Looks right
🙏 |
Poking @dwnusbaum and @bitwiseman 😓 |
I'm hoping this can go through soon. Access to building Dockerfiles on Windows will allow me to actually use the Windows Docker containers in Jenkins. It'll drastically allow me to simplify my pipelines and make them scale. |
Any update? Can this go through? |
@dwnusbaum @bitwiseman @oleg-nenashev should I go ahead and ask for commit access and release permission? |
@dwnusbaum @bitwiseman @oleg-nenashev any chance you could unblock this? Much appreciated. |
@bitwiseman, do you have any plans on reviewing this? |
@Casz @gegles I do not maintain this repo, but I will pass your ping to @bitwiseman and @dwnusbaum in my company's internal slack |
Pardon me, I've been head down on #355. |
@Casz Once again sorry for the delay. This look reasonable. From what I can see the Docker tests are not running (on linux or windows) in CI due to infra issues. I assume you've tested this manually locally? |
Before
after
|
@bitwiseman I demonstrated your assumption so what are the chances for getting a merge/release? 🤔 |
@bitwiseman any chance to get this into the 1.3.x branch? |
@Casz |
That works too @bitwiseman 🥰 |