From be0588099f78ccf8126ee949a02c7ff3309657f7 Mon Sep 17 00:00:00 2001 From: jgrusewski Date: Tue, 28 Apr 2026 22:00:47 +0200 Subject: [PATCH] fix(argo): quote test-name parameter values to satisfy strict YAML parsers The unquoted test name string contains '::' (Rust path separator). Some strict YAML implementations (Go go-yaml when used by certain kustomize paths) interpret it ambiguously. Quoting makes it always a string scalar. Templates were applied directly via 'kubectl apply -f' which uses a permissive parser; the unquoted form worked. Quoting makes them also kustomize-compatible. --- infra/k8s/argo/nsys-test-template.yaml | 2 +- infra/k8s/argo/sanitizer-test-template.yaml | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/infra/k8s/argo/nsys-test-template.yaml b/infra/k8s/argo/nsys-test-template.yaml index 4689d1a59..4862bc78f 100644 --- a/infra/k8s/argo/nsys-test-template.yaml +++ b/infra/k8s/argo/nsys-test-template.yaml @@ -50,7 +50,7 @@ spec: - name: cuda-compute-cap value: "89" - name: test-name - value: performance::test_real_data_single_epoch + value: "performance::test_real_data_single_epoch" # nsys capture options. Defaults capture the full kernel timeline. # For range-based capture, use `--capture-range=cudaProfilerApi` and # call cudaProfilerStart/Stop in the test (advanced). diff --git a/infra/k8s/argo/sanitizer-test-template.yaml b/infra/k8s/argo/sanitizer-test-template.yaml index 85aae1537..252b730b3 100644 --- a/infra/k8s/argo/sanitizer-test-template.yaml +++ b/infra/k8s/argo/sanitizer-test-template.yaml @@ -48,7 +48,7 @@ spec: - name: cuda-compute-cap value: "89" # L40S Ada Lovelace - name: test-name - value: iqn_quantile_monotonicity::iqn_multi_quantile_heads_produce_monotonic_estimates + value: "iqn_quantile_monotonicity::iqn_multi_quantile_heads_produce_monotonic_estimates" - name: sanitizer-tool value: memcheck # memcheck | racecheck | synccheck | initcheck - name: sanitizer-extra-args