From de971ad5e66778bf2678724b9faf69c40c2194ca Mon Sep 17 00:00:00 2001 From: Kefu Chai Date: Fri, 26 Jul 2019 10:55:25 +0800 Subject: [PATCH] ceph-dev-new-trigger: only use positive match for filter there is no need to put ``` grep -v '\(jewel\|kraken\|luminous\)' | grep '\(mimic\|nautilus\)' ``` as we are not likely to have both "jewel" and "mimic" in a branch name. this change is a follow-up of the discussion when reviewing #1345 Signed-off-by: Kefu Chai --- .../config/definitions/ceph-dev-new-trigger.yml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/ceph-dev-new-trigger/config/definitions/ceph-dev-new-trigger.yml b/ceph-dev-new-trigger/config/definitions/ceph-dev-new-trigger.yml index 86225dae..9db68967 100644 --- a/ceph-dev-new-trigger/config/definitions/ceph-dev-new-trigger.yml +++ b/ceph-dev-new-trigger/config/definitions/ceph-dev-new-trigger.yml @@ -51,7 +51,7 @@ - conditional-step: condition-kind: shell condition-command: | - echo "${GIT_BRANCH}" | grep -v '\(jewel\|kraken\|luminous\)' | grep '\(mimic\|nautilus\)' + echo "${GIT_BRANCH}" | grep '\(mimic\|nautilus\)' on-evaluation-failure: dont-run steps: - shell: -- 2.47.3