From 0c54f261eae2f19eb4cef5fa8b444d96a3515513 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Martin=20Roukala=20=28n=C3=A9=20Peres=29?= Date: Wed, 1 Feb 2023 12:56:08 +0200 Subject: [PATCH] ci/core-manual-rules: enclose the whole condition in quotes MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Quoting a condition is apparently an effective way of working around YAML parsing weirdness. However, the quotes need to surround the whole expression, not just parts of it. Fixes: f6c06ef2f66a ("ci: Add manual rules variations to disable.") Suggested-by: Michel Dänzer Reviewed-by: Eric Engestrom Signed-off-by: Martin Roukala (né Peres) Part-of: --- .gitlab-ci/test-source-dep.yml | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/.gitlab-ci/test-source-dep.yml b/.gitlab-ci/test-source-dep.yml index 7d2edd50e1d..2c790b07c7b 100644 --- a/.gitlab-ci/test-source-dep.yml +++ b/.gitlab-ci/test-source-dep.yml @@ -59,9 +59,9 @@ # We only want manual jobs to show up when it's not marge's pre-merge CI # run, otherwise she'll wait until her timeout. The exception is # performance jobs, see below. - - if: ('$GITLAB_USER_LOGIN == "marge-bot" && - $CI_PIPELINE_SOURCE == "merge_request_event"' && - $CI_JOB_NAME !~ "/performance$/") + - if: '($GITLAB_USER_LOGIN == "marge-bot" && + $CI_PIPELINE_SOURCE == "merge_request_event" && + $CI_JOB_NAME !~ "/performance$/")' when: never - !reference [.scheduled_pipeline-rules, rules] - changes: