[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] CI: Don't run Coverity on forks
- To: Andrew Cooper <andrew.cooper3@xxxxxxxxxx>
- From: Roger Pau Monné <roger.pau@xxxxxxxxxx>
- Date: Mon, 21 Mar 2022 16:04:47 +0100
- Arc-authentication-results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=citrix.com; dmarc=pass action=none header.from=citrix.com; dkim=pass header.d=citrix.com; arc=none
- Arc-message-signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=pyooVaNQncRIsW/50+tBknBTWUcjROFecYOeIpUgxG0=; b=bD27jpWB86Io/VYAUJgr9BdL1xDfKrQtZ85PU9aiI/GqJKJnZuaI/HK1+bzwKak4s6s8JWizh6jd216wLbsR314To5tc6tqBjJKwlinuFJE5VrTQpLwzEOoGYs1Q4zlGp4JK943v5MdKuIw8SrNxHh5QWfi4iOIuzWl57s0nVAphI6t3pZ5MGXgdlc12c5bdC9YpC72pz+yccWKLxNp4zzQOdpWR8euj3s0fhVYr2YAyOUoIyy03fjz6iuFlymn0vLM6/9rj9mwFMMXmocXyCXlv4KmROeHNSnhMV41d+71Y62mwtXn8GeWQxM7GbQTt9hsESRNgw+Cs0Y8InrJ2Ww==
- Arc-seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=WhkQxfNBtH3yKhVDFrxp+H0z98Lbw4VGswbakn9ocBkpQHUpdwueehckbcJ4rOenE1OecKQiOTz/KzF1vrqVJqlbqYZUlTb1DPznS4DYwacFg0bl/IobR0/aGLCggKhMCKPa+PQ56C6QEQadrwybvonqZ6u/9HAoBGu8GQZIZHes9lO6lVDiiC7VW2DSssgjDwnhu0b4mm2sj1QEPlRTvYqjO+8eRqohk+j8tKKE0YC94WM5TafWmA6nMWoOE6Sk5NJFhOnPTA85QO5gb+FKoSygfU01MVgJEzB65/1cPKvKPEc6tzPSEljvJT74o2qNLeNEaLPSb20C2S7oBPQkwA==
- Authentication-results: esa2.hc3370-68.iphmx.com; dkim=pass (signature verified) header.i=@citrix.onmicrosoft.com
- Cc: Xen-devel <xen-devel@xxxxxxxxxxxxxxxxxxxx>, George Dunlap <George.Dunlap@xxxxxxxxxxxxx>, Jan Beulich <JBeulich@xxxxxxxx>, Stefano Stabellini <sstabellini@xxxxxxxxxx>, Wei Liu <wl@xxxxxxx>, Julien Grall <julien@xxxxxxx>
- Delivery-date: Mon, 21 Mar 2022 15:05:05 +0000
- Ironport-data: A9a23:IlpOPquEphNhWJe6wAtFFCJSQufnVH1eMUV32f8akzHdYApBsoF/q tZmKWzTbvrYM2fyetl2PYTkpEkCuJSBndZnHQdk/ys1EiIX+JbJXdiXEBz9bniYRiHhoOOLz Cm8hv3odp1coqr0/0/1WlTZhSAgk/nOHNIQMcacUsxLbVYMpBwJ1FQyw4bVvqYy2YLjW1jX4 IuryyHiEATNNwBcYzp8B52r8HuDjNyq0N/PlgVjDRzjlAa2e0g9VPrzF4noR5fLatA88tqBb /TC1NmEElbxpH/BPD8HfoHTKSXmSpaKVeSHZ+E/t6KK2nCurQRquko32WZ1he66RFxlkvgoo Oihu6BcRi8qZ5aWksZMWCN2DhhMYrFj9+eeM0Cw5Jn7I03uKxMAwt1rBUAye4YZ5vx2ESdF8 vlwxDIlN07ZwbjsmfTiF7cq1p9LwMrDZevzvll6yj7UF7A+SI3rSKTW/95Imjw3g6iiGN6AO ZVGOGQ3MnwsZTUeJmsTEZkSndyzxVf/SBFCiw2b/bEotj27IAtZj+G2bYu9lsaxbdpRtlaVo CTB5WuRKhMHMN2SzxKV/3TqgfXA9QvgQ54bHrC88v9sgXWQy3YVBRlQUkG0ydGph0j7V99BJ kg8/is1sbN05EGtVsP6XRCzvDiDpBF0c/h6HvA+6QqN4rHJ+AvfDW8BJgOtc/R/6pVwH2Zzk AbUwZW5XlSDrYF5V1qer4zOpxHjMhI7MFYwOjMhQlNc8eHs9dRbYg30cv5vF6u8j9vQED72w iyXoCVWu4j/nfLnxI3gowmZ3mvESozhC1dsu16JBj7NAhZRPtbNWmC+1bTMAR+sxq69R0LJg nULktP2AAsmXcDUz3zlrAng8diUCxe53N/03AUH83oJrW3FF5ufkWd4um0WGauRGpxYEQIFm WeK0e+r2LddPWGxcYh8aJ+rBsIhwMDITIq5CaCNNoMXMsEqJWdrGR2CgmbKggjQfLUEy/lja f93j+7yZZrlNUiX5GXvHLpMuVPa7is/2XnSVfjGI+ePitKjiIquYe5dajOmN7lhhIvd+Vm92 4sPZqOilkQEOMWjM3a/zGLmBQ1TRZTNLcut8JI/my/qClcOJVzN/NeKmOJ/IdM5x/oJ/goKl 1nkMnJlJJPErSSvAS2Ba2x5aaOpWpB6rHkhOjcrM0ru0H8mCbtDJo9GH3frVdHLLNBe8MM=
- Ironport-hdrordr: A9a23:yfLtbKqZxsmqJZEn32KMtE8aV5vJL9V00zEX/kB9WHVpm5Oj+P xGzc526farslsssREb+OxpOMG7MBThHLpOkPMs1NCZLXTbUQqTXfpfBO7ZrQEIdBeOlNK1uZ 0QFpSWTeeAcWSS7vyKkTVQcexQueVvmZrA7Yy1rwYPcegpUdAZ0+4QMHfkLqQcfnghOXNWLu v52iIRzADQBkj/I/7LTkUtbqzmnZnmhZjmaRkJC1oO7xSPtyqh7PrfHwKD1hkTfjtTyfN6mF K12TDR1+GGibWW2xXc32jc49B/n8bg8MJKAIiphtIOIjvhpw60bMBKWqGEvhoyvOazgWxa2+ XkklMFBYBe+nnRdma6rV/E3BTh6i8n7zvYxVqRkRLY0LvEbQN/L/AEqZNScxPf5UZllsp7yr h302WQsIcSJQ/cnQzmjuK4GC1Cpw6Rmz4PgOQTh3tQXc81c7lKt7ES+0tTDdMpAD/60oY6C+ NjZfusqMq+SWnqLkwxg1MfgOBFBh8Ib1S7qwk5y4GoOgFt7T5EJxBy/r1cop8CnKhNP6Wsqd 60d5iAr4s+PfP+XZgNdNvpfvHHeFAlYSi8Rl56cm6XXZ3uBRr22uvKCfMOlaWXRKA=
- List-id: Xen developer discussion <xen-devel.lists.xenproject.org>
On Mon, Mar 21, 2022 at 01:58:28PM +0000, Andrew Cooper wrote:
> By default, workflows run in all forks, but the Coverity token is specific to
> us, causing all other runs to fail.
>
> Signed-off-by: Andrew Cooper <andrew.cooper3@xxxxxxxxxx>
Acked-by: Roger Pau Monné <roger.pau@xxxxxxxxxx>
Albeit I have a suggestion to make this more useful I think
> ---
> CC: Roger Pau Monné <roger.pau@xxxxxxxxxx>
> CC: George Dunlap <George.Dunlap@xxxxxxxxxxxxx>
> CC: Jan Beulich <JBeulich@xxxxxxxx>
> CC: Stefano Stabellini <sstabellini@xxxxxxxxxx>
> CC: Wei Liu <wl@xxxxxxx>
> CC: Julien Grall <julien@xxxxxxx>
> ---
> .github/workflows/coverity.yml | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/.github/workflows/coverity.yml b/.github/workflows/coverity.yml
> index 427fb86f947f..f613f9ed3652 100644
> --- a/.github/workflows/coverity.yml
> +++ b/.github/workflows/coverity.yml
> @@ -8,6 +8,7 @@ on:
>
> jobs:
> coverity:
> + if: github.repository_owner == 'xen-project'
Since I don't know anything else similar, why not make this a secret,
ie: ${{ secrets.RUN_COVERITY_SCAN }}? So that people could decide to
enable coverity on their own repos if desired.
We would also need to introduce a ${{ secrets.COVERITY_SCAN_PROJECT }}
To allow setting a different project name.
Thanks, Roger.
|
