Bret, Yes. Let’s start with build testing.
It will be good to boot the shell and run test suites like UEFI SCTs from the shell of OVMF in a CI agent, but that could be a scheduled post-commit task. Booting the shell is fast enough that we could consider it for pre-commit. Mike From: Bret Barkelew <[email protected]> Sent: Thursday, March 26, 2020 5:15 PM To: Kinney, Michael D <[email protected]>; Zhang, Shenglei <[email protected]>; [email protected] Cc: Sean Brogan <[email protected]>; Gao, Liming <[email protected]> Subject: RE: [EXTERNAL] [PATCH] .azurepipelines: Enable CI for OvmfPkg and EmulatorPkg So just clarifying, the only thing in scope right now is a simple build-test against these packages. There’s no desire to run tests within the Ovmf environment itself? - Bret From: Kinney, Michael D<mailto:[email protected]> Sent: Thursday, March 26, 2020 5:00 PM To: Bret Barkelew<mailto:[email protected]>; Zhang, Shenglei<mailto:[email protected]>; [email protected]<mailto:[email protected]>; Kinney, Michael D<mailto:[email protected]> Cc: Sean Brogan<mailto:[email protected]>; Gao, Liming<mailto:[email protected]> Subject: RE: [EXTERNAL] [PATCH] .azurepipelines: Enable CI for OvmfPkg and EmulatorPkg Bret, I would like to see issues with these platforms packages caught pre-commit. If a core package change breaks one of these platforms, the we can potentially prevent an issue in many other platforms. Mike From: Bret Barkelew <[email protected]<mailto:[email protected]>> Sent: Thursday, March 26, 2020 4:27 PM To: Zhang, Shenglei <[email protected]<mailto:[email protected]>>; [email protected]<mailto:[email protected]> Cc: Sean Brogan <[email protected]<mailto:[email protected]>>; Kinney, Michael D <[email protected]<mailto:[email protected]>>; Gao, Liming <[email protected]<mailto:[email protected]>> Subject: RE: [EXTERNAL] [PATCH] .azurepipelines: Enable CI for OvmfPkg and EmulatorPkg Taking a moment to look at this a different way… Is it expected at some point that we would want to run OvmfPkg-based integration tests as part of a “second-pass” automatic validation (maybe not a PR-gate, since those should be as fast as possible, but a nightly CI)? If so, I think I’d rather see these platforms covered under that pipeline. If anyone would be interested in pursuing that approach (which would also be more portable to other platforms in edk2-platforms, if they should want automated nightlys at some point), I’d be happy to put a pin in this topic and throw up a prototype of what that might look like. Thanks! - Bret From: Shenglei Zhang<mailto:[email protected]> Sent: Thursday, March 26, 2020 12:04 AM To: [email protected]<mailto:[email protected]> Cc: Sean Brogan<mailto:[email protected]>; Bret Barkelew<mailto:[email protected]>; Kinney, Michael D<mailto:[email protected]>; Liming Gao<mailto:[email protected]> Subject: [EXTERNAL] [PATCH] .azurepipelines: Enable CI for OvmfPkg and EmulatorPkg REF: https://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fbugzilla.tianocore.org%2Fshow_bug.cgi%3Fid%3D2570&data=02%7C01%7CBret.Barkelew%40microsoft.com%7C198058e322e7419c15d708d7d153eaff%7C72f988bf86f141af91ab2d7cd011db47%7C1%7C0%7C637208030668393225&sdata=TkqjqO7Fi%2BN7FPgJ0FlRcD4T59zCu7hfWQRbb%2FO5dKA%3D&reserved=0<https://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fbugzilla.tianocore.org%2Fshow_bug.cgi%3Fid%3D2570&data=02%7C01%7CBret.Barkelew%40microsoft.com%7C95bd9871827d4ac0a84708d7d1e1e394%7C72f988bf86f141af91ab2d7cd011db47%7C1%7C0%7C637208640427947736&sdata=paW9upRWw3kGWHGW7fXE1ecdQ%2FfxfXMt0s%2BYH15llW8%3D&reserved=0> OvmfPkg and EmulatorPkg are mostly used by the developers, so add them to target list. Cc: Sean Brogan <[email protected]<mailto:[email protected]>> Cc: Bret Barkelew <[email protected]<mailto:[email protected]>> Cc: Michael D Kinney <[email protected]<mailto:[email protected]>> Cc: Liming Gao <[email protected]<mailto:[email protected]>> Signed-off-by: Shenglei Zhang <[email protected]<mailto:[email protected]>> --- .azurepipelines/templates/pr-gate-build-job.yml | 6 ++++++ 1 file changed, 6 insertions(+) diff --git a/.azurepipelines/templates/pr-gate-build-job.yml b/.azurepipelines/templates/pr-gate-build-job.yml index 61868554d43c..34f03745cc70 100644 --- a/.azurepipelines/templates/pr-gate-build-job.yml +++ b/.azurepipelines/templates/pr-gate-build-job.yml @@ -44,6 +44,12 @@ jobs: TARGET_SECURITY: Build.Pkgs: 'SecurityPkg' Build.Targets: 'DEBUG,RELEASE,NO-TARGET' + TARGET_OVMF: + Build.Pkgs: 'OvmfPkg' + Build.Targets: 'DEBUG,RELEASE,NO-TARGET' + TARGET_EMULATOR: + Build.Pkgs: 'EmulatorPkg' + Build.Targets: 'DEBUG,RELEASE,NO-TARGET' workspace: clean: all -- 2.18.0.windows.1 -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#56449): https://edk2.groups.io/g/devel/message/56449 Mute This Topic: https://groups.io/mt/72576939/21656 Group Owner: [email protected] Unsubscribe: https://edk2.groups.io/g/devel/unsub [[email protected]] -=-=-=-=-=-=-=-=-=-=-=-
