[edk2-devel] [RedfishPkg PATCH v5 3/4] .azurepipelines/templates: Add RedfishPkg to target build
Abner Chang
abner.chang at hpe.com
Wed Oct 7 03:04:03 UTC 2020
Ok, v6 sent. Should be the perfect one, I hope so.
From: devel at edk2.groups.io [mailto:devel at edk2.groups.io] On Behalf Of Bret Barkelew via groups.io
Sent: Wednesday, October 7, 2020 8:01 AM
To: Sean Brogan <spbrogan at outlook.com>; Chang, Abner (HPS SW/FW Technologist) <abner.chang at hpe.com>; devel at edk2.groups.io
Cc: Sean Brogan <sean.brogan at microsoft.com>; Kinney, Michael D <michael.d.kinney at intel.com>; Liming Gao <gaoliming at byosoft.com.cn>; Wang, Nickle (HPS SW) <nickle.wang at hpe.com>
Subject: Re: [EXTERNAL] Re: [edk2-devel] [RedfishPkg PATCH v5 3/4] .azurepipelines/templates: Add RedfishPkg to target build
Ah, yes. Good catch. Agree with Sean.
- Bret
From: Sean Brogan<mailto:spbrogan at outlook.com>
Sent: Tuesday, October 6, 2020 4:45 PM
To: Chang, Abner (HPS SW/FW Technologist)<mailto:abner.chang at hpe.com>; devel at edk2.groups.io<mailto:devel at edk2.groups.io>
Cc: Sean Brogan<mailto:sean.brogan at microsoft.com>; Bret Barkelew<mailto:Bret.Barkelew at microsoft.com>; Kinney, Michael D<mailto:michael.d.kinney at intel.com>; Liming Gao<mailto:gaoliming at byosoft.com.cn>; Wang, Nickle (HPS SW)<mailto:nickle.wang at hpe.com>
Subject: [EXTERNAL] Re: [edk2-devel] [RedfishPkg PATCH v5 3/4] .azurepipelines/templates: Add RedfishPkg to target build
I am just suggesting that for CI you should put redfishpkg into an
existing matrix item rather than create a new matrix item. Each one of
those is a build that spins up when CI/PR is invoked.
instead of this (your patch)
diff --git a/.azurepipelines/templates/pr-gate-build-job.yml
b/.azurepipelines/templates/pr-gate-build-job.yml
index 2683687a1c..b5b09c2cca 100644
--- a/.azurepipelines/templates/pr-gate-build-job.yml
+++ b/.azurepipelines/templates/pr-gate-build-job.yml
@@ -45,6 +45,9 @@ jobs:
TARGET_SECURITY:
Build.Pkgs: 'SecurityPkg'
Build.Targets: 'DEBUG,RELEASE,NO-TARGET'
+ TARGET_REDFISH:
+ Build.Pkgs: 'RedfishPkg'
+ Build.Targets: 'DEBUG,RELEASE,NO-TARGET'
TARGET_PLATFORMS:
# For Platforms only check code. Leave it to Platform CI
# to build them.
I would just do something like below. (i hand edited the diff so might
not be perfect but hopefully you get the idea).
@@ -45,6 +45,9 @@ jobs:
- TARGET_SECURITY:
+ TARGET_SECURITY_REDFISH:
- Build.Pkgs: 'SecurityPkg'
+ Build.Pkgs: 'SecurityPkg,RedfishPkg'
Build.Targets: 'DEBUG,RELEASE,NO-TARGET'
Thanks
Sean
On 10/6/2020 4:39 PM, Chang, Abner (HPS SW/FW Technologist) wrote:
>
>
>> -----Original Message-----
>> From: Sean Brogan [mailto:spbrogan at outlook.com]
>> Sent: Wednesday, October 7, 2020 7:05 AM
>> To: devel at edk2.groups.io<mailto:devel at edk2.groups.io>; Chang, Abner (HPS SW/FW Technologist)
>> <abner.chang at hpe.com<mailto:abner.chang at hpe.com>>
>> Cc: Sean Brogan <sean.brogan at microsoft.com<mailto:sean.brogan at microsoft.com>>; Bret Barkelew
>> <Bret.Barkelew at microsoft.com<mailto:Bret.Barkelew at microsoft.com>>; Michael D Kinney
>> <michael.d.kinney at intel.com<mailto:michael.d.kinney at intel.com>>; Liming Gao <gaoliming at byosoft.com.cn<mailto:gaoliming at byosoft.com.cn>>;
>> Wang, Nickle (HPS SW) <nickle.wang at hpe.com<mailto:nickle.wang at hpe.com>>
>> Subject: Re: [edk2-devel] [RedfishPkg PATCH v5
>> 3/4] .azurepipelines/templates: Add RedfishPkg to target build
>>
>> On this change i would recommend that you add RedFishPkg to another in
>> the list as i don't think the time it takes to compile and test this relatively
>> small package makes it worth spinning up another machine, syncing sources,
>> installing tools, etc.
>
> Sean, do you mean to add RedFishPkg to another package? Or move drivers (currently is none) under RedFishPkg to another package?
> This is the initial commit of RedFishPkg, more drivers will added to this package later on.
> https://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Ftianocore%2Fedk2-staging%2Fblob%2FUEFI_Redfish%2FReadme.md&data=02%7C01%7CBret.Barkelew%40microsoft.com%7C064c677f2cad4197e11608d86a51f2ab%7C72f988bf86f141af91ab2d7cd011db47%7C1%7C0%7C637376247502227698&sdata=L4wOv4IxFdOEoUV2IjBG3kUHlBW1Z%2FsQvTQGe67W980%3D&reserved=0<https://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Ftianocore%2Fedk2-staging%2Fblob%2FUEFI_Redfish%2FReadme.md&data=02%7C01%7CBret.Barkelew%40microsoft.com%7C064c677f2cad4197e11608d86a51f2ab%7C72f988bf86f141af91ab2d7cd011db47%7C1%7C0%7C637376247502227698&sdata=L4wOv4IxFdOEoUV2IjBG3kUHlBW1Z%2FsQvTQGe67W980%3D&reserved=0>
>
> That doesn't make sense to put RedFishPkg to any of another to save CI time, but what we can do now is remove RedFishPkg from CI test for the time being. Says don't add RedfishPkg to GetPackageSupported for now.
>
>
>>
>> Looking at the current matrix it looks like the SecurityPkg only takes 8
>> minutes and CRYPTO only takes ~9 minutes. I also see FMP and FAT is faster
>> than the slowest entry. We generally try to load balance this so that CI
>> completes quickly.
>>
>> vs2019
>> INVALID URI REMOVED
>> 3A__dev.azure.com_tianocore_edk2-2Dci_-5Fbuild_results-3FbuildId-
>> 3D12811-26view-
>> 3Dresults&d=DwICaQ&c=C5b8zRQO1miGmBeVZ2LFWg&r=_SN6FZBN4Vgi4Ul
>> kskz6qU3NYRO03nHp9P7Z5q59A3E&m=QmpPMx5HlLf978P4ICsd-
>> D4sw4IjDO8v_OJ74lDlU0M&s=3XIQ_TF_f0Ck_fkywiGXKd-
>> 2a23G5fMyOkEFrZgBz5M&e=
>>
>> gcc
>> INVALID URI REMOVED
>> 3A__dev.azure.com_tianocore_edk2-2Dci_-5Fbuild_results-3FbuildId-
>> 3D12810-26view-
>> 3Dresults&d=DwICaQ&c=C5b8zRQO1miGmBeVZ2LFWg&r=_SN6FZBN4Vgi4Ul
>> kskz6qU3NYRO03nHp9P7Z5q59A3E&m=QmpPMx5HlLf978P4ICsd-
>> D4sw4IjDO8v_OJ74lDlU0M&s=4VVVdhJiawmkngs8kXl6ENUdoVGHEueOUZ6
>> XbXB9yt4&e=
>>
>>
>> Thanks
>> Sean
>>
>>
>>
>> On 10/6/2020 3:48 AM, Abner Chang wrote:
>>> Signed-off-by: Abner Chang <abner.chang at hpe.com<mailto:abner.chang at hpe.com>>
>>> Cc: Sean Brogan <sean.brogan at microsoft.com<mailto:sean.brogan at microsoft.com>>
>>> Cc: Bret Barkelew <Bret.Barkelew at microsoft.com<mailto:Bret.Barkelew at microsoft.com>>
>>> Cc: Michael D Kinney <michael.d.kinney at intel.com<mailto:michael.d.kinney at intel.com>>
>>> Cc: Liming Gao <gaoliming at byosoft.com.cn<mailto:gaoliming at byosoft.com.cn>>
>>> Cc: Nickle Wang <nickle.wang at hpe.com<mailto:nickle.wang at hpe.com>>
>>> Reviewed-by: Bret Barkelew <Bret.Barkelew at microsoft.com<mailto:Bret.Barkelew at microsoft.com>>
>>> ---
>>> .azurepipelines/templates/pr-gate-build-job.yml | 3 +++
>>> 1 file changed, 3 insertions(+)
>>>
>>> diff --git a/.azurepipelines/templates/pr-gate-build-job.yml
>>> b/.azurepipelines/templates/pr-gate-build-job.yml
>>> index 2683687a1c..b5b09c2cca 100644
>>> --- a/.azurepipelines/templates/pr-gate-build-job.yml
>>> +++ b/.azurepipelines/templates/pr-gate-build-job.yml
>>> @@ -45,6 +45,9 @@ jobs:
>>> TARGET_SECURITY:
>>> Build.Pkgs: 'SecurityPkg'
>>> Build.Targets: 'DEBUG,RELEASE,NO-TARGET'
>>> + TARGET_REDFISH:
>>> + Build.Pkgs: 'RedfishPkg'
>>> + Build.Targets: 'DEBUG,RELEASE,NO-TARGET'
>>> TARGET_PLATFORMS:
>>> # For Platforms only check code. Leave it to Platform CI
>>> # to build them.
>>>
-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#65959): https://edk2.groups.io/g/devel/message/65959
Mute This Topic: https://groups.io/mt/77355658/1813853
Group Owner: devel+owner at edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [edk2-devel-archive at redhat.com]
-=-=-=-=-=-=-=-=-=-=-=-
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://listman.redhat.com/archives/edk2-devel-archive/attachments/20201007/4287b0e3/attachment.htm>
More information about the edk2-devel-archive
mailing list