From 9a8a7aaf1d17c4586ca7006a40b8ca53700dba05 Mon Sep 17 00:00:00 2001
From: Eric Engestrom <eric@igalia.com>
Date: Thu, 22 Jun 2023 17:46:28 +0100
Subject: [PATCH] ci: split valve farm in two

There are two physical locations, so it makes sense to manage them
separately.

Signed-off-by: Eric Engestrom <eric@igalia.com>
Reviewed-by: Martin Roukala <martin.roukala@mupuf.org>
Acked-by: David Heidelberg <david.heidelberg@collabora.com>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/23908>
---
 .ci-farms/{valve => valve-kws} |  0
 .ci-farms/valve-mupuf          |  0
 .gitlab-ci/test-source-dep.yml | 12 ++++++++++--
 src/amd/ci/gitlab-ci.yml       |  7 +++++++
 4 files changed, 17 insertions(+), 2 deletions(-)
 rename .ci-farms/{valve => valve-kws} (100%)
 create mode 100644 .ci-farms/valve-mupuf

diff --git a/.ci-farms/valve b/.ci-farms/valve-kws
similarity index 100%
rename from .ci-farms/valve
rename to .ci-farms/valve-kws
diff --git a/.ci-farms/valve-mupuf b/.ci-farms/valve-mupuf
new file mode 100644
index 00000000000..e69de29bb2d
diff --git a/.gitlab-ci/test-source-dep.yml b/.gitlab-ci/test-source-dep.yml
index 10a9e9666e0..a641c48079f 100644
--- a/.gitlab-ci/test-source-dep.yml
+++ b/.gitlab-ci/test-source-dep.yml
@@ -330,9 +330,17 @@
 
 .valve-farm-rules:
   rules:
-    - exists: [ .ci-farms-disabled/valve ]
+    - exists: [ .ci-farms-disabled/valve-mupuf ]
+      if: '$RUNNER_FARM_LOCATION == "mupuf"'
       when: never
-    - changes: [ .ci-farms-disabled/valve ]
+    - exists: [ .ci-farms-disabled/valve-kws ]
+      if: '$RUNNER_FARM_LOCATION == "keywords"'
+      when: never
+    - changes: [ .ci-farms-disabled/valve-mupuf ]
+      if: '$RUNNER_FARM_LOCATION == "mupuf"'
+      when: on_success
+    - changes: [ .ci-farms-disabled/valve-kws ]
+      if: '$RUNNER_FARM_LOCATION == "keywords"'
       when: on_success
     - changes: [ .ci-farms-disabled/* ]
       when: never
diff --git a/src/amd/ci/gitlab-ci.yml b/src/amd/ci/gitlab-ci.yml
index 6a44ac86141..77c59b87500 100644
--- a/src/amd/ci/gitlab-ci.yml
+++ b/src/amd/ci/gitlab-ci.yml
@@ -216,12 +216,14 @@ radeonsi-raven-va-full:x86_64:
 
 .kabini-test-valve:
   variables:
+    RUNNER_FARM_LOCATION: mupuf
     B2C_KERNEL_CMDLINE_EXTRAS: 'b2c.swap=8g'
   tags:
     - amdgpu:codename:KABINI
 
 .polaris10-test-valve:
   variables:
+    RUNNER_FARM_LOCATION: keywords
     FDO_CI_CONCURRENT: 16
     B2C_TIMEOUT_BOOT_RETRIES: 4
     # (2022-01) tsc=unstable was added in response to this message in kmsg,
@@ -232,6 +234,7 @@ radeonsi-raven-va-full:x86_64:
 
 .vega10-test-valve:
   variables:
+    RUNNER_FARM_LOCATION: keywords
     FDO_CI_CONCURRENT: 16
     B2C_KERNEL_CMDLINE_EXTRAS: 'b2c.swap=16g'
   tags:
@@ -239,6 +242,7 @@ radeonsi-raven-va-full:x86_64:
 
 .renoir-test-valve:
   variables:
+    RUNNER_FARM_LOCATION: mupuf
     FDO_CI_CONCURRENT: 24
     B2C_KERNEL_CMDLINE_EXTRAS: 'b2c.swap=16g'
     B2C_TIMEOUT_BOOT_MINUTES: 60
@@ -248,6 +252,7 @@ radeonsi-raven-va-full:x86_64:
 
 .navi10-test-valve:
   variables:
+    RUNNER_FARM_LOCATION: mupuf
     FDO_CI_CONCURRENT: 24
     # (2022-01) noapic is set because the serial adapter on this
     # machine is using non-MSI IRQs and is generating a lot of
@@ -259,6 +264,7 @@ radeonsi-raven-va-full:x86_64:
 
 .navi21-test-valve:
   variables:
+    RUNNER_FARM_LOCATION: keywords
     FDO_CI_CONCURRENT: 32
     B2C_KERNEL_CMDLINE_EXTRAS: 'b2c.swap=16g'
   tags:
@@ -266,6 +272,7 @@ radeonsi-raven-va-full:x86_64:
 
 .vangogh-test-valve:
   variables:
+    RUNNER_FARM_LOCATION: mupuf
     FDO_CI_CONCURRENT: 8
     B2C_TIMEOUT_BOOT_MINUTES: 85
     B2C_TIMEOUT_OVERALL_MINUTES: 120