From 4fd5bdca9294cce4d25c2bb3a2201d806c2aa322 Mon Sep 17 00:00:00 2001 From: E M <5089238+emizzle@users.noreply.github.com> Date: Wed, 29 Apr 2026 16:46:24 +1000 Subject: [PATCH] refactor: remove allow-tests-pods node label from GKE node pools The `allow-tests-pods` boolean label was used by the test framework to steer pods away from runner nodes via a node affinity exclusion. Pod scheduling now uses the existing `workload-type` label directly as a nodeSelector, making the boolean label redundant. --- .../main.tf | 14 ++++++-------- 1 file changed, 6 insertions(+), 8 deletions(-) diff --git a/.github/release/clusters/logos-storage-rel-tests-gcp-europe-west4/main.tf b/.github/release/clusters/logos-storage-rel-tests-gcp-europe-west4/main.tf index 294c2915..ebec59d1 100644 --- a/.github/release/clusters/logos-storage-rel-tests-gcp-europe-west4/main.tf +++ b/.github/release/clusters/logos-storage-rel-tests-gcp-europe-west4/main.tf @@ -12,19 +12,17 @@ module "gke" { node_pool_min = 1 node_pool_max = 5 node_pool_labels = { - allow-tests-pods = "false" - default-pool = "true" - scaling-type = "auto" - workload-type = "tests-runners-ci" + default-pool = "true" + scaling-type = "auto" + workload-type = "tests-runners-ci" } tests_pool_name = "tests-e2-medium" tests_pool_machine_type = "e2-medium" tests_pool_max = 10 tests_pool_labels = { - allow-tests-pods = "true" - default-pool = "false" - scaling-type = "auto" - workload-type = "tests-pods" + default-pool = "false" + scaling-type = "auto" + workload-type = "tests-pods" } }