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 PM Joey Vagedes <joey.vagedes@gmail.com> wrote:

Updates Edk2-pytools to the latest versions, which introduces a new Policy
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, 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 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 <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 <guo.dong@intel.com>
Cc: Guomin Jiang <guomin.jiang@intel.com>
Cc: Hao A Wu <hao.a.wu@intel.com>
Cc: Igor Kulchytskyy <igork@ami.com>
Cc: James Lu <james.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):
  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