From patchwork Mon Aug 21 08:54:06 2023 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Simone Ballarin X-Patchwork-Id: 13359277 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from lists.xenproject.org (lists.xenproject.org [192.237.175.120]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id AE0A1EE4996 for ; Mon, 21 Aug 2023 08:55:00 +0000 (UTC) Received: from list by lists.xenproject.org with outflank-mailman.587407.918791 (Exim 4.92) (envelope-from ) id 1qY0gQ-0007oA-Aj; Mon, 21 Aug 2023 08:54:46 +0000 X-Outflank-Mailman: Message body and most headers restored to incoming version Received: by outflank-mailman (output) from mailman id 587407.918791; Mon, 21 Aug 2023 08:54:46 +0000 Received: from localhost ([127.0.0.1] helo=lists.xenproject.org) by lists.xenproject.org with esmtp (Exim 4.92) (envelope-from ) id 1qY0gQ-0007o3-81; Mon, 21 Aug 2023 08:54:46 +0000 Received: by outflank-mailman (input) for mailman id 587407; Mon, 21 Aug 2023 08:54:45 +0000 Received: from se1-gles-sth1-in.inumbo.com ([159.253.27.254] helo=se1-gles-sth1.inumbo.com) by lists.xenproject.org with esmtp (Exim 4.92) (envelope-from ) id 1qY0gP-0007nI-Ag for xen-devel@lists.xenproject.org; Mon, 21 Aug 2023 08:54:45 +0000 Received: from support.bugseng.com (mail.bugseng.com [162.55.131.47]) by se1-gles-sth1.inumbo.com (Halon) with ESMTPS id 5f425fbf-4000-11ee-8782-cb3800f73035; Mon, 21 Aug 2023 10:54:44 +0200 (CEST) Received: from beta.station (net-93-66-137-131.cust.vodafonedsl.it [93.66.137.131]) by support.bugseng.com (Postfix) with ESMTPSA id 4F7894EE0739; Mon, 21 Aug 2023 10:54:43 +0200 (CEST) X-BeenThere: xen-devel@lists.xenproject.org List-Id: Xen developer discussion List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Errors-To: xen-devel-bounces@lists.xenproject.org Precedence: list Sender: "Xen-devel" X-Inumbo-ID: 5f425fbf-4000-11ee-8782-cb3800f73035 From: Simone Ballarin To: xen-devel@lists.xenproject.org Cc: consulting@bugseng.com, andrew.cooper3@citrix.com, Simone Ballarin , Doug Goldstein , Stefano Stabellini Subject: [XEN PATCH v2 1/2] automation/eclair: avoid unintentional ECLAIR analysis Date: Mon, 21 Aug 2023 10:54:06 +0200 Message-Id: <4de9a2e93015b12186675b9a3f7ebff7fcd71baa.1692607487.git.simone.ballarin@bugseng.com> X-Mailer: git-send-email 2.34.1 In-Reply-To: References: MIME-Version: 1.0 With this patch, ECLAIR jobs will need to be manually started for "people/.*" pipelines and will not be triggered if the WTOKEN variable is missing. This avoids occupying the runner on analyzes that might not be used by developers. If developers want to analyze their own repositories they need to launch them from GitLab. Signed-off-by: Simone Ballarin Reviewed-by: Stefano Stabellini --- Changes in v2: - avoid ECLAIR jobs if the WTOKEN variable is not defined. --- automation/gitlab-ci/analyze.yaml | 22 +++++++++++++++++----- automation/scripts/eclair | 5 ----- 2 files changed, 17 insertions(+), 10 deletions(-) diff --git a/automation/gitlab-ci/analyze.yaml b/automation/gitlab-ci/analyze.yaml index 4aa4abe2ee..bd9a68de31 100644 --- a/automation/gitlab-ci/analyze.yaml +++ b/automation/gitlab-ci/analyze.yaml @@ -18,28 +18,40 @@ - '*.log' reports: codequality: gl-code-quality-report.json + rules: + - if: $WTOKEN == null + when: never + - when: always needs: [] -eclair-x86_64: +.eclair-analysis:triggered: extends: .eclair-analysis + allow_failure: true + rules: + - if: $WTOKEN && $CI_PROJECT_PATH =~ /^xen-project\/people\/.*$/ + when: manual + - !reference [.eclair-analysis, rules] + +eclair-x86_64: + extends: .eclair-analysis:triggered variables: LOGFILE: "eclair-x86_64.log" VARIANT: "X86_64" RULESET: "Set1" - allow_failure: true eclair-ARM64: - extends: .eclair-analysis + extends: .eclair-analysis:triggered variables: LOGFILE: "eclair-ARM64.log" VARIANT: "ARM64" RULESET: "Set1" - allow_failure: true .eclair-analysis:on-schedule: extends: .eclair-analysis rules: - - if: $CI_PIPELINE_SOURCE == "schedule" + - if: $CI_PIPELINE_SOURCE != "schedule" + when: never + - !reference [.eclair-analysis, rules] eclair-x86_64-Set1:on-schedule: extends: .eclair-analysis:on-schedule diff --git a/automation/scripts/eclair b/automation/scripts/eclair index 813a56eb6a..14e47a6f97 100755 --- a/automation/scripts/eclair +++ b/automation/scripts/eclair @@ -4,11 +4,6 @@ ECLAIR_ANALYSIS_DIR=automation/eclair_analysis ECLAIR_DIR="${ECLAIR_ANALYSIS_DIR}/ECLAIR" ECLAIR_OUTPUT_DIR=$(realpath "${ECLAIR_OUTPUT_DIR}") -if [ -z "${WTOKEN:-}" ]; then - echo "Failure: the WTOKEN variable is not defined." >&2 - exit 1 -fi - "${ECLAIR_ANALYSIS_DIR}/prepare.sh" "${VARIANT}" ex=0 From patchwork Mon Aug 21 08:54:07 2023 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Simone Ballarin X-Patchwork-Id: 13359276 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from lists.xenproject.org (lists.xenproject.org [192.237.175.120]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id CEF2EEE49A5 for ; Mon, 21 Aug 2023 08:55:00 +0000 (UTC) Received: from list by lists.xenproject.org with outflank-mailman.587408.918801 (Exim 4.92) (envelope-from ) id 1qY0gR-000852-I1; Mon, 21 Aug 2023 08:54:47 +0000 X-Outflank-Mailman: Message body and most headers restored to incoming version Received: by outflank-mailman (output) from mailman id 587408.918801; Mon, 21 Aug 2023 08:54:47 +0000 Received: from localhost ([127.0.0.1] helo=lists.xenproject.org) by lists.xenproject.org with esmtp (Exim 4.92) (envelope-from ) id 1qY0gR-00084t-F2; Mon, 21 Aug 2023 08:54:47 +0000 Received: by outflank-mailman (input) for mailman id 587408; Mon, 21 Aug 2023 08:54:46 +0000 Received: from se1-gles-sth1-in.inumbo.com ([159.253.27.254] helo=se1-gles-sth1.inumbo.com) by lists.xenproject.org with esmtp (Exim 4.92) (envelope-from ) id 1qY0gP-0007nI-Vl for xen-devel@lists.xenproject.org; Mon, 21 Aug 2023 08:54:45 +0000 Received: from support.bugseng.com (mail.bugseng.com [162.55.131.47]) by se1-gles-sth1.inumbo.com (Halon) with ESMTPS id 60272ce1-4000-11ee-8782-cb3800f73035; Mon, 21 Aug 2023 10:54:45 +0200 (CEST) Received: from beta.station (net-93-66-137-131.cust.vodafonedsl.it [93.66.137.131]) by support.bugseng.com (Postfix) with ESMTPSA id C77444EE0737; Mon, 21 Aug 2023 10:54:44 +0200 (CEST) X-BeenThere: xen-devel@lists.xenproject.org List-Id: Xen developer discussion List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Errors-To: xen-devel-bounces@lists.xenproject.org Precedence: list Sender: "Xen-devel" X-Inumbo-ID: 60272ce1-4000-11ee-8782-cb3800f73035 From: Simone Ballarin To: xen-devel@lists.xenproject.org Cc: consulting@bugseng.com, andrew.cooper3@citrix.com, Simone Ballarin , Doug Goldstein , Stefano Stabellini Subject: [XEN PATCH v2 2/2] automation: avoid pipelines on specific branches Date: Mon, 21 Aug 2023 10:54:07 +0200 Message-Id: <8ca21d49e7b95e788a4c092c1b8dcd3d1245d638.1692607487.git.simone.ballarin@bugseng.com> X-Mailer: git-send-email 2.34.1 In-Reply-To: References: MIME-Version: 1.0 This patch avoids the execution of pipelines in the following branches: - master - smoke - coverirty-tested/.* - stable-.* The job-level exclusions have been removed as they are pointless with this new workspace-level exclusion. Signed-off-by: Simone Ballarin Reviewed-by: Stefano Stabellini --- Changes in v2: - remove useless except clause in .yocto-test. --- .gitlab-ci.yml | 6 ++++++ automation/gitlab-ci/build.yaml | 11 ----------- automation/gitlab-ci/test.yaml | 5 ----- 3 files changed, 6 insertions(+), 16 deletions(-) diff --git a/.gitlab-ci.yml b/.gitlab-ci.yml index ee5430b8b7..ef4484e09a 100644 --- a/.gitlab-ci.yml +++ b/.gitlab-ci.yml @@ -1,3 +1,9 @@ +workflow: + rules: + - if: $CI_COMMIT_BRANCH =~ /^(master|smoke|^coverity-tested\/.*|stable-.*)$/ + when: never + - when: always + stages: - analyze - build diff --git a/automation/gitlab-ci/build.yaml b/automation/gitlab-ci/build.yaml index 1a4a5e490d..b633facff4 100644 --- a/automation/gitlab-ci/build.yaml +++ b/automation/gitlab-ci/build.yaml @@ -12,11 +12,6 @@ - '*/*.log' when: always needs: [] - except: - - master - - smoke - - /^coverity-tested\/.*/ - - /^stable-.*/ .gcc-tmpl: variables: &gcc @@ -214,11 +209,6 @@ .yocto-test: stage: build image: registry.gitlab.com/xen-project/xen/${CONTAINER} - except: - - master - - smoke - - /^coverity-tested\/.*/ - - /^stable-.*/ script: - ./automation/build/yocto/build-yocto.sh -v --log-dir=./logs --xen-dir=`pwd` ${YOCTO_BOARD} ${YOCTO_OUTPUT} variables: @@ -269,7 +259,6 @@ .test-jobs-artifact-common: stage: build needs: [] - except: !reference [.test-jobs-common, except] # Arm test artifacts diff --git a/automation/gitlab-ci/test.yaml b/automation/gitlab-ci/test.yaml index 810631bc46..8737f367c8 100644 --- a/automation/gitlab-ci/test.yaml +++ b/automation/gitlab-ci/test.yaml @@ -1,11 +1,6 @@ .test-jobs-common: stage: test image: registry.gitlab.com/xen-project/xen/${CONTAINER} - except: - - master - - smoke - - /^coverity-tested\/.*/ - - /^stable-.*/ .arm64-test-needs: &arm64-test-needs - alpine-3.18-arm64-rootfs-export