-
Notifications
You must be signed in to change notification settings - Fork 73
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
Adding Windows 2004 #25
Conversation
c8c9cac
to
7d2bdd9
Compare
@bradrydzewski there's a weird one on this.... i got core dumps from servers when i used images built from this Dockerfile. It worked on almost all of our builds except one was a doozy so I'm just warning you that the fix for me was to remove all the nanoserver and leave everything in servercore. Wish I knew why that fixed it but after a LOT of head banging against a wall that's the only thing that did it for me. |
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.
Seems fine but can't merge until there's a builder
7d2bdd9
to
ff486ac
Compare
@donny-dont @bradrydzewski update on the BSOD or whatever issue... if you have a large repo this seems to be the cause during the git fetch process. I back traced it to pulling packfiles somewhere deep in git.exe and I don't think it's anything any of us are going to solve, seems like something for someone at msft or docker and I'm trying to push my dump files and logs to someone who might be able to fix it. the basics for running drone is this... you HAVE to set the DRONE_DEPTH to something small on larger repos or it will die a miserable death during the clone process. This worked...
Hope it helps you guys and hope you get a 2004 runner soon. |
@luthermonson just curious but did you try a later git at all in this process? As of now we're up to git 2.29 |
@donny-dont yup, i did... it's not that so i reverted back to your version |
@donny-dont @bradrydzewski with 12/9 container AMIs in Amazon this problem discussed has pretty much gone away and our builds are 100% consistent. We're merging and building 2004 containers every build going forward and will tell you if we have any more issues. im quite confident this was never drone and this PR should be merged. I will have 20h2 PRs shortly too |
@luthermonson again the problem is there's no build for |
i fully understand the build servers are the issue for not merging this... that's not my problem. we have our own running with drone-git built in 2004 with no issues and I'm trying to relay they my confidence in 2004 + drone is high and you should get the servers and move forward. that's all i can do... let me check into your powershell comment |
@donny-dont this Powershell stuff you linked is for the images in hub |
Invoke-WebRequest -UseBasicParsing https://github.com/git-for-windows/git/releases/download/v2.21.0.windows.1/MinGit-2.21.0-64-bit.zip -OutFile git.zip; ` | ||
Expand-Archive git.zip -DestinationPath C:\git; | ||
|
||
FROM mcr.microsoft.com/powershell:nanoserver-2004 |
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.
@luthermonson you're using the powershell image as the base. All the scripts are powershell so there needs to be a 20H2 before you can do anything.
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.
ah yes, indeed. you win so much copy pasta i didnt even notice. that's a shame but for my purposes ill just keep it a servercore, large image size doesnt bother me. for your purposes you are correct
|
lols |
1 similar comment
@StillJodes yes i did try later versions from git for windows. as of nov/dec the 2004 AMI in windows no longer has this problem. It was some patch that had to come from msft to fix this it appears |
Added to support drone-plugins/drone-docker#284
For this to work fully it will need Windows 2004 hosts to build the images.