From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-pj1-f47.google.com (mail-pj1-f47.google.com [209.85.216.47]) by mx.groups.io with SMTP id smtpd.web10.26986.1688656986502430479 for ; Thu, 06 Jul 2023 08:23:06 -0700 Authentication-Results: mx.groups.io; dkim=fail reason="signature has expired" header.i=@gmail.com header.s=20221208 header.b=Da22uIvn; spf=pass (domain: gmail.com, ip: 209.85.216.47, mailfrom: joey.vagedes@gmail.com) Received: by mail-pj1-f47.google.com with SMTP id 98e67ed59e1d1-262e44b8bf6so545611a91.1 for ; Thu, 06 Jul 2023 08:23:06 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20221208; t=1688656985; x=1691248985; h=cc:to:subject:message-id:date:from:in-reply-to:references :mime-version:from:to:cc:subject:date:message-id:reply-to; bh=xTLOjtW5X89gDShYVZv44bM+fThhxb4GIMLgdUIUsXY=; b=Da22uIvnO4FDJApozh3Lbo8b8troWje0evjXM5zGc25BojAD6f6h0XFrMuhQKxqlBg /HqZUnWlf3+9kZDEPP7jA5pfe042mBfZ1SYDo+nJRAY4eep7DU7Y1srVGz6TloKM6AMX uDT5HLJXXHoHVJXCfKApchxZ8CDFSv7e0LegghE7h+eUxsspeOYAhbJRqagMmTu+VKrG MgtJuVAAZp0Kl71qmjqnsMPlSQNocUu5pAaBmObrSWjA0QIqo9fAIVXS4bMx4FVWuha6 HJ4TEgkp5kKCAwr5mSu/m3uGWzKwzoLyzIFO0lK4ECT8bv5De2mHqrEghp4q4gDhbNbK jWzQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20221208; t=1688656985; x=1691248985; h=cc:to:subject:message-id:date:from:in-reply-to:references :mime-version:x-gm-message-state:from:to:cc:subject:date:message-id :reply-to; bh=xTLOjtW5X89gDShYVZv44bM+fThhxb4GIMLgdUIUsXY=; b=ddLO5X8q0K4m08azWeNz+lhbkgpN7TndL9tnBVlcPaUPEYAeHLUgqxi2G3mdglquFN SyAg2c5MvcSzIZcDR+C2M6MAwG2yulGHyAwan//IKxBLLnWdZFYdfxmOXoooq5alqytp SK+CGPODBmGX5fXbiNmhARvFKH7KOSz8HlcG6Isd3RjtOcjJsSmWRMIwQzdnCfgdmb1B GMMnOTK+UmksGULZcPW6WKqPAk2kVHks4Ua3Ty2ujK+/9T1a1gO4ZwvOeXESefIc4HF5 RIlSbI7FZqlb1LpmbRVhLJXW3MdduNf43e54+hWSZ25wu0uW6zHq1hMof1f4t8Y82o8M 8g4Q== X-Gm-Message-State: ABy/qLacvMmkn7xxxhtyvezBx8A8qNDGknwtb+9MuVEYAlP9FnhSttuA Yp3vNa3dHB4roIZoRu3jPxd+VNSUjH7QKuwnZr3JuPTuUwiUfA== X-Google-Smtp-Source: APBJJlFjsFjESAX8P6zd9FLbJS6/God1NRZe2PbBVZilhZV5WBCAAuaaeCeO1h3EjWnRKBj5qOUFD+KJezvXFwnKQ/0= X-Received: by 2002:a17:90a:4a01:b0:262:de1a:6028 with SMTP id e1-20020a17090a4a0100b00262de1a6028mr1662230pjh.13.1688656985331; Thu, 06 Jul 2023 08:23:05 -0700 (PDT) MIME-Version: 1.0 References: <20230626213434.8-1-joey.vagedes@gmail.com> In-Reply-To: <20230626213434.8-1-joey.vagedes@gmail.com> From: "Joey Vagedes" Date: Thu, 6 Jul 2023 08:22:52 -0700 Message-ID: Subject: Re: [PATCH v1 00/24] Update Edk2-pytools to latest versions To: devel@edk2.groups.io Cc: Alexei Fedorov , Ard Biesheuvel , Ashraf Ali S , Chasel Chiu , Chen Gang C , Daniel Schaefer , Duggapu Chinni B , Eric Dong , Gerd Hoffmann , Guo Dong , Guomin Jiang , Hao A Wu , Igor Kulchytskyy , James Lu , Jian J Wang , Jiewen Yao , Leif Lindholm , Liming Gao , Maciej Rabeda , Michael D Kinney , Michael Kubacki , Nate DeSimone , Nickle Wang , Pierre Gondois , Rahul Kumar , Ray Han Lim Ng , Ray Ni , Sami Mujawar , Sean Brogan , Sean Rhodes , Siyuan Fu , Star Zeng , Susovan Mohapatra , Ted Kuo , Wei6 Xu , Xiaoyu Lu , Yi Li , Zhichao Gao , Zhiguang Liu Content-Type: multipart/alternative; boundary="000000000000c21cae05ffd31796" --000000000000c21cae05ffd31796 Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable Hello everyone, Just following up on this patch series; looking for reviews from each package owner. Just to reiterate, this patch series enables a new policy for detecting which packages should be built for any given PR / Patch on github. The policy is used when an INF is changed. It requires that any package consuming that inf (generally a library instance) be built and tested. I'm happy to help answer any questions! Thanks, Joey On Mon, Jun 26, 2023 at 2:34=E2=80=AFPM Joey Vagedes wrote: > Updates Edk2-pytools to the latest versions, which introduces a new Polic= y > for detecting which packages must be built given a set of changes. This > policy > is used when a library class instance INF is changed. When this occurs, i= t > will > review the PrEval:DscPath provided in the CI YAML file and determine if t= he > package uses the library instance or not. If it does, it will build and > test > the package. > > Due to these changes, each package must now include an entry in it's CI > YAML > file (PrEval:DscPath). If the package does not provide this entry, this > policy > will be skipped, which could lead to a library instance breaking packages > outside > of it's parent package. > > Cc: Abner Chang > Cc: Alexei Fedorov > Cc: Ard Biesheuvel > Cc: Ashraf Ali S > Cc: Chasel Chiu > Cc: Chen Gang C > Cc: Daniel Schaefer > Cc: Duggapu Chinni B > Cc: Eric Dong > Cc: Gerd Hoffmann > Cc: Guo Dong > Cc: Guomin Jiang > Cc: Hao A Wu > Cc: Igor Kulchytskyy > Cc: James Lu > Cc: Jian J Wang > Cc: Jiewen Yao > Cc: Leif Lindholm > Cc: Liming Gao > Cc: Maciej Rabeda > Cc: Michael D Kinney > Cc: Michael Kubacki > Cc: Nate DeSimone > Cc: Nickle Wang > Cc: Pierre Gondois > Cc: Rahul Kumar > Cc: Ray Han Lim Ng > Cc: Ray Ni > Cc: Sami Mujawar > Cc: Sean Brogan > Cc: Sean Rhodes > Cc: Siyuan Fu > Cc: Star Zeng > Cc: Susovan Mohapatra > Cc: Ted Kuo > Cc: Wei6 Xu > Cc: Xiaoyu Lu > Cc: Yi Li > Cc: Zhichao Gao > Cc: Zhiguang Liu > > Joey Vagedes (24): > Edk2: edk2-pytools: Update to latest versions > UnitTestFrameworkPkg: CI: Add PrEval entry > UefiPayloadPkg: CI: Add PrEval entry > ArmPkg: CI: Add PrEval entry > UefiCpuPkg: CI: Add PrEval entry > ArmPlatformPkg: CI: Add PrEval entry > CryptoPkg: CI: Add PrEval entry > DynamicTablesPkg: CI: Add PrEval entry > EmbeddedPkg: CI: Add PrEval entry > FatPkg: CI: Add PrEval entry > FmpDevicePkg: CI: Add PrEval entry > IntelFsp2Pkg: CI: Add PrEval entry > CryptoPkg: CI: Add PrEval entry > SourceLevelDebugPkg: CI: Add PrEval entry > SignedCapsulePkg: CI: Add PrEval entry > ShellPkg: CI: Add PrEval entry > SecurityPkg: CI: Add PrEval entry > RedfishPkg: CI: Add PrEval entry > PrmPkg: CI: Add PrEval entry > PcAtChipsetPkg: CI: Add PrEval entry > NetworkPkg: CI: Add PrEval entry > MdePkg: CI: Add PrEval entry > MdeModulePkg: CI: Add PrEval entry > IntelFsp2WrapperPkg: CI: Add PrEval entry > > ArmPkg/ArmPkg.ci.yaml | 3 +++ > ArmPlatformPkg/ArmPlatformPkg.ci.yaml | 3 +++ > CryptoPkg/CryptoPkg.ci.yaml | 3 +++ > DynamicTablesPkg/DynamicTablesPkg.ci.yaml | 3 +++ > EmbeddedPkg/EmbeddedPkg.ci.yaml | 3 +++ > FatPkg/FatPkg.ci.yaml | 3 +++ > FmpDevicePkg/FmpDevicePkg.ci.yaml | 3 +++ > IntelFsp2Pkg/IntelFsp2Pkg.ci.yaml | 3 +++ > IntelFsp2WrapperPkg/IntelFsp2WrapperPkg.ci.yaml | 3 +++ > MdeModulePkg/MdeModulePkg.ci.yaml | 3 +++ > MdePkg/MdePkg.ci.yaml | 3 +++ > NetworkPkg/NetworkPkg.ci.yaml | 3 +++ > PcAtChipsetPkg/PcAtChipsetPkg.ci.yaml | 3 +++ > PrmPkg/PrmPkg.ci.yaml | 3 +++ > RedfishPkg/RedfishPkg.ci.yaml | 3 +++ > SecurityPkg/SecurityPkg.ci.yaml | 3 +++ > ShellPkg/ShellPkg.ci.yaml | 3 +++ > SignedCapsulePkg/SignedCapsulePkg.ci.yaml | 3 +++ > SourceLevelDebugPkg/SourceLevelDebugPkg.ci.yaml | 3 +++ > StandaloneMmPkg/StandaloneMmPkg.ci.yaml | 3 +++ > UefiCpuPkg/UefiCpuPkg.ci.yaml | 3 +++ > UefiPayloadPkg/UefiPayloadPkg.ci.yaml | 3 +++ > UnitTestFrameworkPkg/UnitTestFrameworkPkg.ci.yaml | 3 +++ > pip-requirements.txt | 4 ++-- > 24 files changed, 71 insertions(+), 2 deletions(-) > > -- > 2.41.0.windows.1 > > --000000000000c21cae05ffd31796 Content-Type: text/html; charset="UTF-8" Content-Transfer-Encoding: quoted-printable
Hello everyone,

Just following up on th= is patch series; looking for reviews from each package owner. Just to reite= rate, this patch series=C2=A0enables a new policy for detecting which packa= ges should be built for any given PR / Patch on github. The policy is used = when an INF is changed. It requires that any package consuming that inf (ge= nerally a library instance) be built and tested. I'm happy to help answ= er any questions!

Thanks,
Joey

On M= on, Jun 26, 2023 at 2:34=E2=80=AFPM Joey Vagedes <joey.vagedes@gmail.com> wrote:
Updates Edk2-pytools to the lates= t versions, which introduces a new Policy
for detecting which packages must be built given a set of changes. This pol= icy
is used when a library class instance INF is changed. When this occurs, it = will
review the PrEval:DscPath provided in the CI YAML file and determine if the=
package uses the library instance or not. If it does, it will build and tes= t
the package.

Due to these changes, each package must now include an entry in it's CI= YAML
file (PrEval:DscPath). If the package does not provide this entry, this pol= icy
will be skipped, which could lead to a library instance breaking packages o= utside
of it's parent package.

Cc: Abner Chang <abner.chang@amd.com>
Cc: Alexei Fedorov <Alexei.Fedorov@arm.com>
Cc: Ard Biesheuvel <ardb+tianocore@kernel.org>
Cc: Ashraf Ali S <ashraf.ali.s@intel.com>
Cc: Chasel Chiu <chasel.chiu@intel.com>
Cc: Chen Gang C <gang.c.chen@intel.com>
Cc: Daniel Schaefer <git@danielschaefer.me>
Cc: Duggapu Chinni B <chinni.b.duggapu@intel.com>
Cc: Eric Dong <= eric.dong@intel.com>
Cc: Gerd Hoffmann <kraxel@redhat.com>
Cc: Guo Dong <gu= o.dong@intel.com>
Cc: Guomin Jiang <guomin.jiang@intel.com>
Cc: Hao A Wu <ha= o.a.wu@intel.com>
Cc: Igor Kulchytskyy <igork@ami.com>
Cc: James Lu <ja= mes.lu@intel.com>
Cc: Jian J Wang <jian.j.wang@intel.com>
Cc: Jiewen Yao <jiewen.yao@intel.com>
Cc: Leif Lindholm <quic_llindhol@quicinc.com>
Cc: Liming Gao <gaoliming@byosoft.com.cn>
Cc: Maciej Rabeda <maciej.rabeda@linux.intel.com>
Cc: Michael D Kinney <michael.d.kinney@intel.com>
Cc: Michael Kubacki <mikuback@linux.microsoft.com>
Cc: Nate DeSimone <nathaniel.l.desimone@intel.com>
Cc: Nickle Wang <nicklew@nvidia.com>
Cc: Pierre Gondois <pierre.gondois@arm.com>
Cc: Rahul Kumar <rahul1.kumar@intel.com>
Cc: Ray Han Lim Ng <ray.han.lim.ng@intel.com>
Cc: Ray Ni <ray.ni= @intel.com>
Cc: Sami Mujawar <sami.mujawar@arm.com>
Cc: Sean Brogan <sean.brogan@microsoft.com>
Cc: Sean Rhodes <sean@starlabs.systems>
Cc: Siyuan Fu <= siyuan.fu@intel.com>
Cc: Star Zeng <= star.zeng@intel.com>
Cc: Susovan Mohapatra <susovan.mohapatra@intel.com>
Cc: Ted Kuo <ted.= kuo@intel.com>
Cc: Wei6 Xu <wei6= .xu@intel.com>
Cc: Xiaoyu Lu <xiaoyu1.lu@intel.com>
Cc: Yi Li <yi1.li@= intel.com>
Cc: Zhichao Gao <zhichao.gao@intel.com>
Cc: Zhiguang Liu <zhiguang.liu@intel.com>

Joey Vagedes (24):
=C2=A0 Edk2: edk2-pytools: Update to latest versions
=C2=A0 UnitTestFrameworkPkg: CI: Add PrEval entry
=C2=A0 UefiPayloadPkg: CI: Add PrEval entry
=C2=A0 ArmPkg: CI: Add PrEval entry
=C2=A0 UefiCpuPkg: CI: Add PrEval entry
=C2=A0 ArmPlatformPkg: CI: Add PrEval entry
=C2=A0 CryptoPkg: CI: Add PrEval entry
=C2=A0 DynamicTablesPkg: CI: Add PrEval entry
=C2=A0 EmbeddedPkg: CI: Add PrEval entry
=C2=A0 FatPkg: CI: Add PrEval entry
=C2=A0 FmpDevicePkg: CI: Add PrEval entry
=C2=A0 IntelFsp2Pkg: CI: Add PrEval entry
=C2=A0 CryptoPkg: CI: Add PrEval entry
=C2=A0 SourceLevelDebugPkg: CI: Add PrEval entry
=C2=A0 SignedCapsulePkg: CI: Add PrEval entry
=C2=A0 ShellPkg: CI: Add PrEval entry
=C2=A0 SecurityPkg: CI: Add PrEval entry
=C2=A0 RedfishPkg: CI: Add PrEval entry
=C2=A0 PrmPkg: CI: Add PrEval entry
=C2=A0 PcAtChipsetPkg: CI: Add PrEval entry
=C2=A0 NetworkPkg: CI: Add PrEval entry
=C2=A0 MdePkg: CI: Add PrEval entry
=C2=A0 MdeModulePkg: CI: Add PrEval entry
=C2=A0 IntelFsp2WrapperPkg: CI: Add PrEval entry

=C2=A0ArmPkg/ArmPkg.ci.yaml=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0= =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0| 3 +++
=C2=A0ArmPlatformPkg/ArmPlatformPkg.ci.yaml=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2= =A0 =C2=A0 =C2=A0| 3 +++
=C2=A0CryptoPkg/CryptoPkg.ci.yaml=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 = =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0| 3 +++
=C2=A0DynamicTablesPkg/DynamicTablesPkg.ci.yaml=C2=A0 =C2=A0 =C2=A0 =C2=A0 = =C2=A0| 3 +++
=C2=A0EmbeddedPkg/EmbeddedPkg.ci.yaml=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2= =A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0| 3 +++
=C2=A0FatPkg/FatPkg.ci.yaml=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0= =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0| 3 +++
=C2=A0FmpDevicePkg/FmpDevicePkg.ci.yaml=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 = =C2=A0 =C2=A0 =C2=A0 =C2=A0| 3 +++
=C2=A0IntelFsp2Pkg/IntelFsp2Pkg.ci.yaml=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 = =C2=A0 =C2=A0 =C2=A0 =C2=A0| 3 +++
=C2=A0IntelFsp2WrapperPkg/IntelFsp2WrapperPkg.ci.yaml=C2=A0 =C2=A0| 3 +++ =C2=A0MdeModulePkg/MdeModulePkg.ci.yaml=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 = =C2=A0 =C2=A0 =C2=A0 =C2=A0| 3 +++
=C2=A0MdePkg/MdePkg.ci.yaml=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0= =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0| 3 +++
=C2=A0NetworkPkg/NetworkPkg.ci.yaml=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2= =A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0| 3 +++
=C2=A0PcAtChipsetPkg/PcAtChipsetPkg.ci.yaml=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2= =A0 =C2=A0 =C2=A0| 3 +++
=C2=A0PrmPkg/PrmPkg.ci.yaml=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0= =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0| 3 +++
=C2=A0RedfishPkg/RedfishPkg.ci.yaml=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2= =A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0| 3 +++
=C2=A0SecurityPkg/SecurityPkg.ci.yaml=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2= =A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0| 3 +++
=C2=A0ShellPkg/ShellPkg.ci.yaml=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 = =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0| 3 +++
=C2=A0SignedCapsulePkg/SignedCapsulePkg.ci.yaml=C2=A0 =C2=A0 =C2=A0 =C2=A0 = =C2=A0| 3 +++
=C2=A0SourceLevelDebugPkg/SourceLevelDebugPkg.ci.yaml=C2=A0 =C2=A0| 3 +++ =C2=A0StandaloneMmPkg/StandaloneMmPkg.ci.yaml=C2=A0 =C2=A0 =C2=A0 =C2=A0 = =C2=A0 =C2=A0| 3 +++
=C2=A0UefiCpuPkg/UefiCpuPkg.ci.yaml=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2= =A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0| 3 +++
=C2=A0UefiPayloadPkg/UefiPayloadPkg.ci.yaml=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2= =A0 =C2=A0 =C2=A0| 3 +++
=C2=A0UnitTestFrameworkPkg/UnitTestFrameworkPkg.ci.yaml | 3 +++
=C2=A0pip-requirements.txt=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 = =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 | 4 ++--
=C2=A024 files changed, 71 insertions(+), 2 deletions(-)

--
2.41.0.windows.1

--000000000000c21cae05ffd31796--