[PATCH 0/1] MR4362: gitlab: Add windows tests to MR pipeline.

[Rémi Bernon (＠rbernon)]

Since we don't have the bridge anymore, and as I understand it's not coming back (?), it's becoming very easy to miss windows tests regressions. This should at least make sure we get some direct visibility of tests running on Win10.

[PATCH 1/1] gitlab: Add windows tests to MR pipeline.

[Rémi Bernon]

From: Rémi Bernon

---
 tools/gitlab/build.yml | 20 +++++++++++++++++++-
 tools/gitlab/test.yml  | 33 +++++++++++++++++++++++++++++++--
 2 files changed, 50 insertions(+), 3 deletions(-)

diff --git a/tools/gitlab/build.yml b/tools/gitlab/build.yml
index 8a5d5b40ac1..af9abb7ebc0 100644
--- a/tools/gitlab/build.yml
+++ b/tools/gitlab/build.yml
@@ -52,6 +52,24 @@ build-mac:
     - export PATH="/usr/local/opt/ccache/libexec:$PATH"
     - git rebase $CI_MERGE_REQUEST_DIFF_BASE_SHA --exec ./tools/gitlab/build-mac

+build-winetest:
+  stage: build
+  image: $CI_REGISTRY/wine/wine:debian-bookworm
+  rules:
+    - if: $CI_PIPELINE_SOURCE == 'merge_request_event'
+  needs:
+    - job: build-linux
+  variables:
+    GIT_STRATEGY: none
+  artifacts:
+    name: winetest
+    paths:
+      - winetest.exe
+      - winetest64.exe
+  script:
+    - mv usr/local/lib/wine/i386-windows/winetest.exe winetest.exe
+    - mv usr/local/lib/wine/x86_64-windows/winetest.exe winetest64.exe
+
 build-daily-linux:
   extends: .wine-build
   rules:
@@ -63,7 +81,7 @@ build-daily-linux:
   script:
     - ./tools/gitlab/build-linux

-build-winetest:
+build-daily-winetest:
   stage: build
   image: $CI_REGISTRY/wine/wine:debian-bookworm
   rules:
diff --git a/tools/gitlab/test.yml b/tools/gitlab/test.yml
index 590eacb2438..831114af00c 100644
--- a/tools/gitlab/test.yml
+++ b/tools/gitlab/test.yml
@@ -67,6 +67,35 @@ test-linux-32:
     - export WINETEST_COLOR=1
     - wine usr/local/lib/wine/i386-windows/winetest.exe -q -q -o - -t gitlab -u $CI_JOB_URL -n $EXCLUDE_TESTS

+test-win10-21h2-32:
+  stage: test
+  interruptible: true
+  variables:
+    GIT_STRATEGY: none
+  rules:
+    - if: $CI_PIPELINE_SOURCE == 'merge_request_event'
+  needs:
+    - job: build-winetest
+  tags:
+    - win10-21h2
+  script:
+    - ./winetest.exe -q -q -o - -t gitlab -u $CI_JOB_URL
+
+test-win10-21h2-64:
+  stage: test
+  interruptible: true
+  variables:
+    GIT_STRATEGY: none
+    INCLUDE_TESTS: "dinput"
+  rules:
+    - if: $CI_PIPELINE_SOURCE == 'merge_request_event'
+  needs:
+    - job: build-winetest
+  tags:
+    - win10-21h2
+  script:
+    - ./winetest64.exe -q -q -o - -t gitlab -u $CI_JOB_URL $INCLUDE_TESTS
+
 debian-32:
   extends: .wine-test
   rules:
@@ -93,7 +122,7 @@ win10-21h2-32:
   rules:
     - if: $CI_PIPELINE_SOURCE == 'trigger' && $CI_COMMIT_BRANCH == $CI_DEFAULT_BRANCH
   needs:
-    - job: build-winetest
+    - job: build-daily-winetest
   tags:
     - win10-21h2
   script:
@@ -107,7 +136,7 @@ win10-21h2-64:
   rules:
     - if: $CI_PIPELINE_SOURCE == 'trigger' && $CI_COMMIT_BRANCH == $CI_DEFAULT_BRANCH
   needs:
-    - job: build-winetest
+    - job: build-daily-winetest
   tags:
     - win10-21h2
   script:

[Rémi Bernon (＠rbernon)]

```
Traceback (most recent call last):
  File "/home/gitlab-runner/tools/gitlab/virtrunner/qemu-agent.py", line 190, in <module>
    main()
  File "/home/gitlab-runner/tools/gitlab/virtrunner/qemu-agent.py", line 179, in main
    ret = run(virsh, args)
  File "/home/gitlab-runner/tools/gitlab/virtrunner/qemu-agent.py", line 112, in run
    print(data.decode("utf-8"))
UnicodeDecodeError: 'utf-8' codec can't decode byte 0x82 in position 56598: invalid start byte
```

Don't know where this comes from, maybe winetest stdout that fails to be decoded? It's only hapenning on the 32bit tests and I'm not sure what I can do about it.