EDK II CI failing - Do not submit PRs with "push" label set

EDK II CI failing - Do not submit PRs with "push" label set

[Michael D Kinney]

Hello,

The repo for the cmocka submodule is not available right now, and this is failing 
CI checks that run unit tests.  Pre-commit checks that do not have any unit tests
will pass, but the post commit checks that run everything will always fail.

    PROGRESS - ## Checking Git repository: UnitTestFrameworkPkg/Library/CmockaLib/cmocka... 
    ERROR - Failed to fetch UnitTestFrameworkPkg/Library/CmockaLib/cmocka 
    ERROR - FAILED! 
    ERROR - Failed to fetch required repository! 
    ERROR - Unable to checkout repo due to error 

This repo access error to the cmocka has occurred one other time in the past.  Bret had
suggested the idea of using a different repo for cmocka, perhaps a fork in GitHub,
to avoid this issue.

We are hoping the issue will be resolved very soon. but until then, please do not
submit any PRs with a “push” label set.

Best regards,

Mike

Re: EDK II CI failing - Do not submit PRs with "push" label set

[Michael D Kinney]

This appears to be a good mirror of cmocka that is being kept up to date.

	https://github.com/neverware-mirrors/cmocka

Mike

> -----Original Message-----
> From: Kinney, Michael D <michael.d.kinney@intel.com>
> Sent: Thursday, December 10, 2020 10:23 PM
> To: devel@edk2.groups.io; Kinney, Michael D <michael.d.kinney@intel.com>; bret.barkelew@microsoft.com
> Subject: EDK II CI failing - Do not submit PRs with "push" label set
> 
> Hello,
> 
> The repo for the cmocka submodule is not available right now, and this is failing
> CI checks that run unit tests.  Pre-commit checks that do not have any unit tests
> will pass, but the post commit checks that run everything will always fail.
> 
>     PROGRESS - ## Checking Git repository: UnitTestFrameworkPkg/Library/CmockaLib/cmocka...
>     ERROR - Failed to fetch UnitTestFrameworkPkg/Library/CmockaLib/cmocka
>     ERROR - FAILED!
>     ERROR - Failed to fetch required repository!
>     ERROR - Unable to checkout repo due to error
> 
> This repo access error to the cmocka has occurred one other time in the past.  Bret had
> suggested the idea of using a different repo for cmocka, perhaps a fork in GitHub,
> to avoid this issue.
> 
> We are hoping the issue will be resolved very soon. but until then, please do not
> submit any PRs with a “push” label set.
> 
> Best regards,
> 
> Mike
> 

Re: EDK II CI failing - Do not submit PRs with "push" label set

[Michael D Kinney]

The issue is resolved.

PRs are being processed correctly now.

Mike

> -----Original Message-----
> From: Kinney, Michael D <michael.d.kinney@intel.com>
> Sent: Thursday, December 10, 2020 10:23 PM
> To: devel@edk2.groups.io; Kinney, Michael D <michael.d.kinney@intel.com>; bret.barkelew@microsoft.com
> Subject: EDK II CI failing - Do not submit PRs with "push" label set
> 
> Hello,
> 
> The repo for the cmocka submodule is not available right now, and this is failing
> CI checks that run unit tests.  Pre-commit checks that do not have any unit tests
> will pass, but the post commit checks that run everything will always fail.
> 
>     PROGRESS - ## Checking Git repository: UnitTestFrameworkPkg/Library/CmockaLib/cmocka...
>     ERROR - Failed to fetch UnitTestFrameworkPkg/Library/CmockaLib/cmocka
>     ERROR - FAILED!
>     ERROR - Failed to fetch required repository!
>     ERROR - Unable to checkout repo due to error
> 
> This repo access error to the cmocka has occurred one other time in the past.  Bret had
> suggested the idea of using a different repo for cmocka, perhaps a fork in GitHub,
> to avoid this issue.
> 
> We are hoping the issue will be resolved very soon. but until then, please do not
> submit any PRs with a “push” label set.
> 
> Best regards,
> 
> Mike
> 

Re: EDK II CI failing - Do not submit PRs with "push" label set

[Bret Barkelew]

[-- Attachment #1: Type: text/plain, Size: 1810 bytes --]

Should we just create a Tianocore mirror, rather than relying on someone else?

- Bret

From: Michael D Kinney via groups.io<mailto:michael.d.kinney=intel.com@groups.io>
Sent: Friday, December 11, 2020 12:16 AM
To: devel@edk2.groups.io<mailto:devel@edk2.groups.io>; Bret Barkelew<mailto:Bret.Barkelew@microsoft.com>; Kinney, Michael D<mailto:michael.d.kinney@intel.com>
Subject: [EXTERNAL] Re: [edk2-devel] EDK II CI failing - Do not submit PRs with "push" label set

The issue is resolved.

PRs are being processed correctly now.

Mike

> -----Original Message-----
> From: Kinney, Michael D <michael.d.kinney@intel.com>
> Sent: Thursday, December 10, 2020 10:23 PM
> To: devel@edk2.groups.io; Kinney, Michael D <michael.d.kinney@intel.com>; bret.barkelew@microsoft.com
> Subject: EDK II CI failing - Do not submit PRs with "push" label set
>
> Hello,
>
> The repo for the cmocka submodule is not available right now, and this is failing
> CI checks that run unit tests.  Pre-commit checks that do not have any unit tests
> will pass, but the post commit checks that run everything will always fail.
>
>     PROGRESS - ## Checking Git repository: UnitTestFrameworkPkg/Library/CmockaLib/cmocka...
>     ERROR - Failed to fetch UnitTestFrameworkPkg/Library/CmockaLib/cmocka
>     ERROR - FAILED!
>     ERROR - Failed to fetch required repository!
>     ERROR - Unable to checkout repo due to error
>
> This repo access error to the cmocka has occurred one other time in the past.  Bret had
> suggested the idea of using a different repo for cmocka, perhaps a fork in GitHub,
> to avoid this issue.
>
> We are hoping the issue will be resolved very soon. but until then, please do not
> submit any PRs with a “push” label set.
>
> Best regards,
>
> Mike
>







[-- Attachment #2: Type: text/html, Size: 4018 bytes --]

Re: [edk2-devel] EDK II CI failing - Do not submit PRs with "push" label set

[Rebecca Cran]

On 12/11/20 12:30 AM, Michael D Kinney wrote:
> This appears to be a good mirror of cmocka that is being kept up to date.
> 
> 	https://github.com/neverware-mirrors/cmocka

There's also the official mirror on Gitlab, which should be reliable: 
https://gitlab.com/cmocka/cmocka .

-- 
Rebecca Cran

Re: [EXTERNAL] Re: [edk2-devel] EDK II CI failing - Do not submit PRs with "push" label set

[Bret Barkelew]

[-- Attachment #1: Type: text/plain, Size: 1637 bytes --]

Agreed. We moved to GitLab in Mu.

mu_basecore/.gitmodules at release/202008 · microsoft/mu_basecore (github.com)<https://github.com/microsoft/mu_basecore/blob/release/202008/.gitmodules>

- Bret

From: Rebecca Cran<mailto:rebecca@nuviainc.com>
Sent: Friday, December 11, 2020 1:16 PM
To: devel@edk2.groups.io<mailto:devel@edk2.groups.io>; Kinney, Michael D<mailto:michael.d.kinney@intel.com>; Bret Barkelew<mailto:Bret.Barkelew@microsoft.com>
Subject: [EXTERNAL] Re: [edk2-devel] EDK II CI failing - Do not submit PRs with "push" label set

On 12/11/20 12:30 AM, Michael D Kinney wrote:
> This appears to be a good mirror of cmocka that is being kept up to date.
>
>        https://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fneverware-mirrors%2Fcmocka&amp;data=04%7C01%7Cbret.barkelew%40microsoft.com%7Cfb033c777225465ed6c808d89e19f87d%7C72f988bf86f141af91ab2d7cd011db47%7C1%7C0%7C637433181665011693%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C1000&amp;sdata=k0i2E%2F%2FAz5717rfQxroJjcG4%2F4qGnjPy6ImC5H4Z3gI%3D&amp;reserved=0

There's also the official mirror on Gitlab, which should be reliable:
https://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgitlab.com%2Fcmocka%2Fcmocka&amp;data=04%7C01%7Cbret.barkelew%40microsoft.com%7Cfb033c777225465ed6c808d89e19f87d%7C72f988bf86f141af91ab2d7cd011db47%7C1%7C0%7C637433181665011693%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C1000&amp;sdata=cqm512EE4Ra7XxUsndzAeISb%2FSYxZ7sQ%2BL%2Bs1w8a2GA%3D&amp;reserved=0 .

--
Rebecca Cran


[-- Attachment #2: Type: text/html, Size: 4517 bytes --]

Re: [edk2-devel] EDK II CI failing - Do not submit PRs with "push" label set

[Laszlo Ersek]

On 12/11/20 22:12, Bret Barkelew via groups.io wrote:
> Should we just create a Tianocore mirror, rather than relying on someone
> else?

(1) TianoCore mirror is a good idea; some other projects too mirror the
submodules they depend on. (Example, IIUC: <https://git.qemu.org/>.)

(2) This announcement from Mike, and the associated discussion, are
dated Dec 11th (last Friday, as of this writing), and I can also see
Mike's other (later) announcement "EDK II CI Working - Resume normal
activities". But perhaps I'm still hitting something similar -- in my
current PR <https://github.com/tianocore/edk2/pull/1224>, there are no
CI failures, but the Mergify bot isn't taking action, despite the "push"
label. I've tried it twice (initial submission, then close+reopen).

Thanks,
Laszlo

> 
>  
> 
> - Bret
> 
>  
> 
> *From: *Michael D Kinney via groups.io
> <mailto:michael.d.kinney=intel.com@groups.io>
> *Sent: *Friday, December 11, 2020 12:16 AM
> *To: *devel@edk2.groups.io <mailto:devel@edk2.groups.io>; Bret Barkelew
> <mailto:Bret.Barkelew@microsoft.com>; Kinney, Michael D
> <mailto:michael.d.kinney@intel.com>
> *Subject: *[EXTERNAL] Re: [edk2-devel] EDK II CI failing - Do not submit
> PRs with "push" label set
> 
>  
> 
> The issue is resolved.
> 
> PRs are being processed correctly now.
> 
> Mike
> 
>> -----Original Message-----
>> From: Kinney, Michael D <michael.d.kinney@intel.com>
>> Sent: Thursday, December 10, 2020 10:23 PM
>> To: devel@edk2.groups.io; Kinney, Michael D
> <michael.d.kinney@intel.com>; bret.barkelew@microsoft.com
>> Subject: EDK II CI failing - Do not submit PRs with "push" label set
>>
>> Hello,
>>
>> The repo for the cmocka submodule is not available right now, and this
> is failing
>> CI checks that run unit tests.  Pre-commit checks that do not have any
> unit tests
>> will pass, but the post commit checks that run everything will always
> fail.
>>
>>     PROGRESS - ## Checking Git repository:
> UnitTestFrameworkPkg/Library/CmockaLib/cmocka...
>>     ERROR - Failed to fetch UnitTestFrameworkPkg/Library/CmockaLib/cmocka
>>     ERROR - FAILED!
>>     ERROR - Failed to fetch required repository!
>>     ERROR - Unable to checkout repo due to error
>>
>> This repo access error to the cmocka has occurred one other time in
> the past.  Bret had
>> suggested the idea of using a different repo for cmocka, perhaps a
> fork in GitHub,
>> to avoid this issue.
>>
>> We are hoping the issue will be resolved very soon. but until then,
> please do not
>> submit any PRs with a “pushâ€&#65533; label set.
>>
>> Best regards,
>>
>> Mike
>>
> 
> 
> 
> 
> 
>  
> 
> 

Re: [edk2-devel] EDK II CI failing - Do not submit PRs with "push" label set

[Michael D Kinney]

Hi Laszlo,

I used the following comments to ask Mergify to re-evaluate the status.  The PR has been merged.

    @Mergifyio refresh

https://docs.mergify.io/commands.html

Mike

> -----Original Message-----
> From: Laszlo Ersek <lersek@redhat.com>
> Sent: Monday, December 14, 2020 6:54 AM
> To: devel@edk2.groups.io; bret.barkelew@microsoft.com; Kinney, Michael D <michael.d.kinney@intel.com>
> Subject: Re: [edk2-devel] EDK II CI failing - Do not submit PRs with "push" label set
> 
> On 12/11/20 22:12, Bret Barkelew via groups.io wrote:
> > Should we just create a Tianocore mirror, rather than relying on someone
> > else?
> 
> (1) TianoCore mirror is a good idea; some other projects too mirror the
> submodules they depend on. (Example, IIUC: <https://git.qemu.org/>.)
> 
> (2) This announcement from Mike, and the associated discussion, are
> dated Dec 11th (last Friday, as of this writing), and I can also see
> Mike's other (later) announcement "EDK II CI Working - Resume normal
> activities". But perhaps I'm still hitting something similar -- in my
> current PR <https://github.com/tianocore/edk2/pull/1224>, there are no
> CI failures, but the Mergify bot isn't taking action, despite the "push"
> label. I've tried it twice (initial submission, then close+reopen).
> 
> Thanks,
> Laszlo
> 
> >
> > Â
> >
> > - Bret
> >
> > Â
> >
> > *From: *Michael D Kinney via groups.io
> > <mailto:michael.d.kinney=intel.com@groups.io>
> > *Sent: *Friday, December 11, 2020 12:16 AM
> > *To: *devel@edk2.groups.io <mailto:devel@edk2.groups.io>; Bret Barkelew
> > <mailto:Bret.Barkelew@microsoft.com>; Kinney, Michael D
> > <mailto:michael.d.kinney@intel.com>
> > *Subject: *[EXTERNAL] Re: [edk2-devel] EDK II CI failing - Do not submit
> > PRs with "push" label set
> >
> > Â
> >
> > The issue is resolved.
> >
> > PRs are being processed correctly now.
> >
> > Mike
> >
> >> -----Original Message-----
> >> From: Kinney, Michael D <michael.d.kinney@intel.com>
> >> Sent: Thursday, December 10, 2020 10:23 PM
> >> To: devel@edk2.groups.io; Kinney, Michael D
> > <michael.d.kinney@intel.com>; bret.barkelew@microsoft.com
> >> Subject: EDK II CI failing - Do not submit PRs with "push" label set
> >>
> >> Hello,
> >>
> >> The repo for the cmocka submodule is not available right now, and this
> > is failing
> >> CI checks that run unit tests.  Pre-commit checks that do not have any
> > unit tests
> >> will pass, but the post commit checks that run everything will always
> > fail.
> >>
> >>     PROGRESS - ## Checking Git repository:
> > UnitTestFrameworkPkg/Library/CmockaLib/cmocka...
> >>     ERROR - Failed to fetch UnitTestFrameworkPkg/Library/CmockaLib/cmocka
> >>     ERROR - FAILED!
> >>     ERROR - Failed to fetch required repository!
> >>     ERROR - Unable to checkout repo due to error
> >>
> >> This repo access error to the cmocka has occurred one other time in
> > the past.  Bret had
> >> suggested the idea of using a different repo for cmocka, perhaps a
> > fork in GitHub,
> >> to avoid this issue.
> >>
> >> We are hoping the issue will be resolved very soon. but until then,
> > please do not
> >> submit any PRs with a “push” label set.
> >>
> >> Best regards,
> >>
> >> Mike
> >>
> >
> >
> >
> >
> >
> > Â
> >
> > 

Re: [edk2-devel] EDK II CI failing - Do not submit PRs with "push" label set

[Laszlo Ersek]

On 12/14/20 18:30, Kinney, Michael D wrote:
> Hi Laszlo,
> 
> I used the following comments to ask Mergify to re-evaluate the status.  The PR has been merged.

Thank you!

> 
>     @Mergifyio refresh
> 
> https://docs.mergify.io/commands.html

Meh, I'm sure you've told me about this before, likely several times.
I'm sorry for not remembering it! :(

Laszlo