-
-
Notifications
You must be signed in to change notification settings - Fork 383
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
Docker Backend: fully support windows container #4381
Docker Backend: fully support windows container #4381
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #4381 +/- ##
==========================================
+ Coverage 27.57% 27.64% +0.06%
==========================================
Files 381 382 +1
Lines 27923 27952 +29
==========================================
+ Hits 7700 7726 +26
- Misses 19538 19540 +2
- Partials 685 686 +1 ☔ View full report in Codecov by Sentry. |
look's like it works :) the only thing is i had to modify the clone plugin: --- a/plugins/plugin-git/Dockerfile.Windows
+++ b/plugins/plugin-git/Dockerfile.Windows
@@ -1,7 +1,7 @@
# syntax = docker/dockerfile:1
# escape=`
-FROM mcr.microsoft.com/windows/servercore:ltsc2022
+FROM mcr.microsoft.com/windows/servercore:1809
# Busybox Unicode https://github.com/rmyorston/busybox-w32 |
so yes woodpecker itselve wont support windows container fully ...
|
my test config: labels:
backend: docker
platform: windows/amd64
skip_clone: true
depends_on:
- echo
steps:
check_echo:
image: mcr.microsoft.com/windows/servercore:1809
commands:
- echo some checks
- sleep 2s
labels:
backend: docker
platform: windows/amd64
when:
- event: pull_request
- event: [push, deployment]
branch:
- ${CI_REPO_DEFAULT_BRANCH}
clone:
clone:
image: woodpecker-windows-git
steps:
createFile:
image: mcr.microsoft.com/windows/servercore:1809
commands:
- echo FROM alpine:edge >> Dockerfile
- echo RUN apk update >> Dockerfile
echo:
image: mcr.microsoft.com/windows/servercore:1809
commands:
- echo buildZ
- sleep 10s
- ls
depl:
image: bash
commands: echo deploy
when:
- event: deployment
branch: ${CI_REPO_DEFAULT_BRANCH}
|
@cduchenoy ☝️ ...? |
Hi, It's excellent! On Windows, we must take the tag which corresponds to the kernel version see https://hub.docker.com/r/microsoft/windows-servercore So I think I should check :)
did you also test the agent build? https://github.com/GECO-IT/woodpecker-windows/blob/main/agent/backend-docker/Dockerfile.Windows |
For test a service image, you can use as a base: https://github.com/GECO-IT/woodpecker-windows/blob/main/base/windows-base/Dockerfile.base-chocolatey.Windows Base image are in docker-hub: https://hub.docker.com/r/gecoit84/woodpecker-windows-base-chocolatey and add:
I haven't tried so I don't know if it launches the service correctly |
no as I only wana test if the engine works ... so the deployment of the agent should not alter the test ... |
ah nice ... will do so |
labels:
backend: docker
platform: windows/amd64
skip_clone: true
steps:
create:
image: mcr.microsoft.com/windows/servercore:1809
commands:
- echo some checks > NEW
- sleep 2s
curl:
image: a6543/woodpecker-windows-base-chocolatey:latest
commands:
- choco install -y curl
- curl.exe -I http://caddy:80
- curl.exe http://caddy:80/NEW
services:
caddy:
image: a6543/woodpecker-windows-caddy-static:latest
ports:
- 80 services work just fine ... |
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.
no clue about windows but lgtm WRT to the diff.
Tearing down https://woodpecker-ci-woodpecker-pr-4381.surge.sh |
-> #4464 |
close #4286
address one bug described in #4274
TODO: actuall testing of: