From fb8209ad78085949e12a343e223c40f42cb875ee Mon Sep 17 00:00:00 2001 From: Chris Evich Date: Tue, 21 Jan 2020 14:39:46 -0500 Subject: Cirrus: No upload snap for docs job Previously, the upload-snap job was running during post-merge testing when the magic ci-docs string was present in the PR description. This fixes that and makes all the `only_if`'s more consistent. Signed-off-by: Chris Evich --- .cirrus.yml | 20 ++++++++++++++------ 1 file changed, 14 insertions(+), 6 deletions(-) diff --git a/.cirrus.yml b/.cirrus.yml index fb6dd4cdb..464455c87 100644 --- a/.cirrus.yml +++ b/.cirrus.yml @@ -514,6 +514,7 @@ special_testing_cgroupv2_task: always: <<: *standardlogs + special_testing_endpoint_task: depends_on: @@ -561,7 +562,8 @@ test_build_cache_images_task: only_if: >- $CIRRUS_BRANCH != $DEST_BRANCH && - $CIRRUS_CHANGE_MESSAGE =~ '.*CI:IMG.*' + $CIRRUS_CHANGE_MESSAGE =~ '.*CI:IMG.*' && + $CIRRUS_CHANGE_MESSAGE !=~ '.*CI:DOCS.*' depends_on: - "gating" @@ -593,7 +595,8 @@ verify_test_built_images_task: only_if: >- $CIRRUS_BRANCH != $DEST_BRANCH && - $CIRRUS_CHANGE_MESSAGE =~ '.*CI:IMG.*' + $CIRRUS_CHANGE_MESSAGE =~ '.*CI:IMG.*' && + $CIRRUS_CHANGE_MESSAGE !=~ '.*CI:DOCS.*' depends_on: @@ -637,11 +640,13 @@ verify_test_built_images_task: always: <<: *standardlogs - upload_snap_task: + only_if: >- + $CIRRUS_BRANCH != $DEST_BRANCH && + $CIRRUS_CHANGE_MESSAGE !=~ '.*CI:IMG.*' && + $CIRRUS_CHANGE_MESSAGE !=~ '.*CI:DOCS.*' # Only when PR or branch is merged into master - only_if: $CIRRUS_BRANCH == $DEST_BRANCH depends_on: - "test_building_snap" @@ -662,8 +667,11 @@ upload_snap_task: docs_task: - # Only run this for PRs on mention, but always run after merge - only_if: $CIRRUS_CHANGE_MESSAGE =~ '.*CI:DOCS.*' || $CIRRUS_BRANCH == $DEST_BRANCH + # Only run this for PRs on mention, and after merge + only_if: >- + $CIRRUS_BRANCH != $DEST_BRANCH && + $CIRRUS_CHANGE_MESSAGE !=~ '.*CI:IMG.*' && + $CIRRUS_CHANGE_MESSAGE =~ '.*CI:DOCS.*' depends_on: - "gating" -- cgit v1.2.3-54-g00ecf