From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail02.groups.io (mail02.groups.io [66.175.222.108]) by spool.mail.gandi.net (Postfix) with ESMTPS id 7BB2DD8027F for ; Wed, 14 Feb 2024 21:54:31 +0000 (UTC) DKIM-Signature: a=rsa-sha256; bh=ecKH8U9ixLNtKvErh8vh51ZEHlhdOcxW5eD06Oq2O/s=; c=relaxed/simple; d=groups.io; h=MIME-Version:References:In-Reply-To:From:Date:Message-ID:Subject:To:Cc:Precedence:List-Subscribe:List-Help:Sender:List-Id:Mailing-List:Delivered-To:Reply-To:List-Unsubscribe-Post:List-Unsubscribe:Content-Type; s=20140610; t=1707947670; v=1; b=jISFAH6mKAPo/Q79slv9U3h1lE3g3vX64iLWQlrxE/OOJUJmWwXmZzy+WCD/VFxT+xJpLQGm VC/9+02kgdeGm/xvlEN3uN4tKZlBRVl/yusdGTu6grXAuDmDY5syxd+tZMFNyUDA4MEaJ2Tm5Am gDMry591ZgwCJliUPmWcxWeA= X-Received: by 127.0.0.2 with SMTP id mP9yYY7687511xf2z6qBEFrK; Wed, 14 Feb 2024 13:54:30 -0800 X-Received: from sin.source.kernel.org (sin.source.kernel.org [145.40.73.55]) by mx.groups.io with SMTP id smtpd.web10.14.1707947669121650192 for ; Wed, 14 Feb 2024 13:54:29 -0800 X-Received: from smtp.kernel.org (transwarp.subspace.kernel.org [100.75.92.58]) by sin.source.kernel.org (Postfix) with ESMTP id 27319CE22A6 for ; Wed, 14 Feb 2024 21:54:26 +0000 (UTC) X-Received: by smtp.kernel.org (Postfix) with ESMTPSA id 5432CC43390 for ; Wed, 14 Feb 2024 21:54:25 +0000 (UTC) X-Received: by mail-lf1-f42.google.com with SMTP id 2adb3069b0e04-51182f8590bso253113e87.0 for ; Wed, 14 Feb 2024 13:54:25 -0800 (PST) X-Forwarded-Encrypted: i=1; AJvYcCVFrNkZSpsAh+xNGNGFSH7cBQAxupCd8+jfL+0oDPE48b9iYbs5aVS+176PiTZ+q8luNDYMzteJGycl/6fW5Pkpn8496w== X-Gm-Message-State: dpu3sQrbcQScj6rUFIZgzpqux7686176AA= X-Google-Smtp-Source: AGHT+IFCdS6jKUQWA+u6T0Mf86YH5voGG2+f05W08PUeiNsHFiROsML/zLygfo5T7z9p4vnqHjnTeLp8WkZH7lj5Ntw= X-Received: by 2002:a19:2d16:0:b0:511:aae2:e5e8 with SMTP id k22-20020a192d16000000b00511aae2e5e8mr43116lfj.52.1707947663528; Wed, 14 Feb 2024 13:54:23 -0800 (PST) MIME-Version: 1.0 References: <20240214011751.2529-1-michael.d.kinney@intel.com> In-Reply-To: From: "Ard Biesheuvel" Date: Wed, 14 Feb 2024 22:54:12 +0100 X-Gmail-Original-Message-ID: Message-ID: Subject: Re: [edk2-devel] [Patch 1/1] BaseTools/Scripts/PatchCheck: Error if commit modifies multiple packages To: "Kinney, Michael D" Cc: Leif Lindholm , "devel@edk2.groups.io" , Rebecca Cran , Liming Gao , "Feng, Bob C" , "Chen, Christine" , Michael Kubacki Precedence: Bulk List-Subscribe: List-Help: Sender: devel@edk2.groups.io List-Id: Mailing-List: list devel@edk2.groups.io; contact devel+owner@edk2.groups.io Reply-To: devel@edk2.groups.io,ardb@kernel.org List-Unsubscribe-Post: List-Unsubscribe=One-Click List-Unsubscribe: Content-Type: text/plain; charset="UTF-8" X-GND-Status: LEGIT Authentication-Results: spool.mail.gandi.net; dkim=pass header.d=groups.io header.s=20140610 header.b=jISFAH6m; spf=pass (spool.mail.gandi.net: domain of bounce@groups.io designates 66.175.222.108 as permitted sender) smtp.mailfrom=bounce@groups.io; dmarc=fail reason="SPF not aligned (relaxed), DKIM not aligned (relaxed)" header.from=kernel.org (policy=none) On Wed, 14 Feb 2024 at 18:16, Kinney, Michael D wrote: > > Hi Ard and Leif, > > Today in CI we do not have fine grain control to ignore specific > CI check failures when merging. > I have been asking for this for a while now. > CI checks can either be informative only in the log or blocking. > > For cases where a commit really should be broken up into multiple > commits, the informative approach in a log can be easily missed > by authors/reviewers/maintainers. > > For the suggestion to override a CI check, we would need a way > to re-run CI with input from a maintainer to relax specific CI > check(s). The default could be all checks enabled. Some checks > can be enabled/disabled at the package scope through ci.yaml > file settings. This specific check is for the contents of one > of more commits under review, so the ci.yaml at package scope > does not apply. > I turned off some CI checks in packages entirely, simply because I cannot override a single CI fail from the dashboard. > A couple ideas > > 1) Make this initial version of this check informative only and > figure out how to make the results more visible to the > author/reviewers/maintainers. Making this informative only for all PRs likely defeats the purpose. > 2) Investigate a mechanism for a maintainer to disable a specific > check and re-run CI with that check disabled. > a) Perhaps a flag in the commit message > b) Perhaps a label in the PR > I'd lean towards a), as it will be logged in the commit history. It also requires some extra work to respin the PR, and this should make abuse of this feature less likely in cases where the change can easily just be split up instead. > It would also be helpful if a few examples from the edk2 commit > history where this proposed CI check extension would report a > failure and it would not be possible to reorganize the commits into > a passing condition. That would help support the hard requirement > for the need to bypass the check. > I didn't look at all the patches in the ticket, but there are at least two which cannot be split up without breaking the build. 103fa647d159e3d76be2634d2653c2d215dd0d46 ArmPkg: Replace CoreId and ClusterId with Mpidr in ARM_CORE_INFO struct 4c55f6394fafe0494ec24e7c05cb68c938d7852d MdePkg: IORT header update for IORT Rev E.d spec -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#115470): https://edk2.groups.io/g/devel/message/115470 Mute This Topic: https://groups.io/mt/104345509/7686176 Group Owner: devel+owner@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [rebecca@openfw.io] -=-=-=-=-=-=-=-=-=-=-=-