public inbox for devel@edk2.groups.io
 help / color / mirror / Atom feed
* [RFC] UnitTestFrameworkPkg cmocka submodule alternatives
@ 2020-12-16 18:45 Michael D Kinney
  2020-12-17  0:58 ` 回复: [edk2-devel] " gaoliming
                   ` (2 more replies)
  0 siblings, 3 replies; 16+ messages in thread
From: Michael D Kinney @ 2020-12-16 18:45 UTC (permalink / raw)
  To: rfc@edk2.groups.io, devel@edk2.groups.io, Kinney, Michael D,
	'Bret Barkelew'

Hello,

We have had at least three incidents in the last year where the link to the
cmocka submodule in the UnitTestFrameworkPkg has not been available, and this
impacted the EDK II CI system.  The following submodule link is the one that
is not reliable:

	https://git.cryptomilk.org/projects/cmocka.git

We have identified two potential mirrors of this repo:

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

The following patch provided a temporary fix for the EDK II CI agents, but
does not help other consumers of the edk2 repository.

	https://github.com/tianocore/edk2/commit/be746104d1766a8c363e74d6063144657820d688

I have seen one suggestion that TianoCore create its own
mirror of cmocka.  This does require monitoring and maintenance
by the  TianoCore community.  I would prefer to use a well
maintained mirror in github as long as we do not observe any
issues with the support of that mirror.

I propose we update the submodule in the UnitTestFrameworkPkg
to use the https://github.com/neverware-mirrors/cmocka.git mirror.
By using a mirror in github, we remove one external dependency.

Please provide feedback and comments on this proposal.  If there
are no objections, then we will proceed with a patch review for
this update.

Thanks,

Mike

^ permalink raw reply	[flat|nested] 16+ messages in thread

* 回复: [edk2-devel] [RFC] UnitTestFrameworkPkg cmocka submodule alternatives
  2020-12-16 18:45 [RFC] UnitTestFrameworkPkg cmocka submodule alternatives Michael D Kinney
@ 2020-12-17  0:58 ` gaoliming
  2020-12-17 14:48 ` [edk2-rfc] " Laszlo Ersek
  2020-12-19 18:58 ` Michael D Kinney
  2 siblings, 0 replies; 16+ messages in thread
From: gaoliming @ 2020-12-17  0:58 UTC (permalink / raw)
  To: devel, michael.d.kinney, rfc, 'Bret Barkelew'

Mike:
  I review all submodules in edk2. They are all from github except for cmocka. So, I agree to use github mirror for cmocka. 

Thanks
Liming
> -----邮件原件-----
> 发件人: bounce+27952+69010+4905953+8761045@groups.io
> <bounce+27952+69010+4905953+8761045@groups.io> 代表 Michael D
> Kinney
> 发送时间: 2020年12月17日 2:46
> 收件人: rfc@edk2.groups.io; devel@edk2.groups.io; Kinney, Michael D
> <michael.d.kinney@intel.com>; 'Bret Barkelew'
> <Bret.Barkelew@microsoft.com>
> 主题: [edk2-devel] [RFC] UnitTestFrameworkPkg cmocka submodule
> alternatives
> 
> Hello,
> 
> We have had at least three incidents in the last year where the link to the
> cmocka submodule in the UnitTestFrameworkPkg has not been available, and
> this
> impacted the EDK II CI system.  The following submodule link is the one that
> is not reliable:
> 
> 	https://git.cryptomilk.org/projects/cmocka.git
> 
> We have identified two potential mirrors of this repo:
> 
> 	https://github.com/neverware-mirrors/cmocka.git
> 	https://gitlab.com/cmocka/cmocka.git
> 
> The following patch provided a temporary fix for the EDK II CI agents, but
> does not help other consumers of the edk2 repository.
> 
> 	https://github.com/tianocore/edk2/commit/be746104d1766a8c363e74
> d6063144657820d688
> 
> I have seen one suggestion that TianoCore create its own
> mirror of cmocka.  This does require monitoring and maintenance
> by the  TianoCore community.  I would prefer to use a well
> maintained mirror in github as long as we do not observe any
> issues with the support of that mirror.
> 
> I propose we update the submodule in the UnitTestFrameworkPkg
> to use the https://github.com/neverware-mirrors/cmocka.git mirror.
> By using a mirror in github, we remove one external dependency.
> 
> Please provide feedback and comments on this proposal.  If there
> are no objections, then we will proceed with a patch review for
> this update.
> 
> Thanks,
> 
> Mike
> 
> 
> 
> 




^ permalink raw reply	[flat|nested] 16+ messages in thread

* Re: [edk2-rfc] [RFC] UnitTestFrameworkPkg cmocka submodule alternatives
  2020-12-16 18:45 [RFC] UnitTestFrameworkPkg cmocka submodule alternatives Michael D Kinney
  2020-12-17  0:58 ` 回复: [edk2-devel] " gaoliming
@ 2020-12-17 14:48 ` Laszlo Ersek
  2020-12-17 15:54   ` Laszlo Ersek
  2020-12-19 18:58 ` Michael D Kinney
  2 siblings, 1 reply; 16+ messages in thread
From: Laszlo Ersek @ 2020-12-17 14:48 UTC (permalink / raw)
  To: rfc, michael.d.kinney, devel@edk2.groups.io,
	'Bret Barkelew'

On 12/16/20 19:45, Michael D Kinney wrote:
> Hello,
> 
> We have had at least three incidents in the last year where the link to the
> cmocka submodule in the UnitTestFrameworkPkg has not been available, and this
> impacted the EDK II CI system.  The following submodule link is the one that
> is not reliable:
> 
> 	https://git.cryptomilk.org/projects/cmocka.git
> 
> We have identified two potential mirrors of this repo:
> 
> 	https://github.com/neverware-mirrors/cmocka.git
> 	https://gitlab.com/cmocka/cmocka.git
> 
> The following patch provided a temporary fix for the EDK II CI agents, but
> does not help other consumers of the edk2 repository.
> 
> 	https://github.com/tianocore/edk2/commit/be746104d1766a8c363e74d6063144657820d688
> 
> I have seen one suggestion that TianoCore create its own
> mirror of cmocka.  This does require monitoring and maintenance
> by the  TianoCore community.  I would prefer to use a well
> maintained mirror in github as long as we do not observe any
> issues with the support of that mirror.
> 
> I propose we update the submodule in the UnitTestFrameworkPkg
> to use the https://github.com/neverware-mirrors/cmocka.git mirror.
> By using a mirror in github, we remove one external dependency.
> 
> Please provide feedback and comments on this proposal.  If there
> are no objections, then we will proceed with a patch review for
> this update.

We could create our own fork under the <https://github.com/tianocore>
organization.

It does not require much extra maintenance or monitoring, in my opinion.
We only need to advance our fork to the actual master HEAD when we
intend to advance our submodule reference in edk2 as well. As long as
the submodule reference in edk2 does not move, the actual master HEAD of
the cmocka project may very well be ahead of our fork (mirror), without
causing issues.

I don't know who or what the <https://github.com/neverware-mirrors>
organization is, and I'd prefer not fetching code from them automatically.

Thanks
Laszlo


^ permalink raw reply	[flat|nested] 16+ messages in thread

* Re: [edk2-rfc] [RFC] UnitTestFrameworkPkg cmocka submodule alternatives
  2020-12-17 14:48 ` [edk2-rfc] " Laszlo Ersek
@ 2020-12-17 15:54   ` Laszlo Ersek
  2020-12-17 16:01     ` Rebecca Cran
  0 siblings, 1 reply; 16+ messages in thread
From: Laszlo Ersek @ 2020-12-17 15:54 UTC (permalink / raw)
  To: rfc, michael.d.kinney, devel@edk2.groups.io,
	'Bret Barkelew'

On 12/17/20 15:48, Laszlo Ersek wrote:

> I don't know who or what the <https://github.com/neverware-mirrors>
> organization is, and I'd prefer not fetching code from them automatically.

I'm sorry, this was silly.

The whole point of git is "addressing by content". Our submodule
reference in edk2 makes us check out the cmocka tree at a known hash, so
where that comes from is totally irrelevant.

I'm OK with the proposal as posted.

Thanks & sorry again,
Laszlo


^ permalink raw reply	[flat|nested] 16+ messages in thread

* Re: [edk2-rfc] [RFC] UnitTestFrameworkPkg cmocka submodule alternatives
  2020-12-17 15:54   ` Laszlo Ersek
@ 2020-12-17 16:01     ` Rebecca Cran
  2020-12-17 18:44       ` [EXTERNAL] " Bret Barkelew
  0 siblings, 1 reply; 16+ messages in thread
From: Rebecca Cran @ 2020-12-17 16:01 UTC (permalink / raw)
  To: rfc, lersek, michael.d.kinney, devel@edk2.groups.io,
	'Bret Barkelew'

On 12/17/20 8:54 AM, Laszlo Ersek wrote:
> On 12/17/20 15:48, Laszlo Ersek wrote:
> 
>> I don't know who or what the <https://github.com/neverware-mirrors>
>> organization is, and I'd prefer not fetching code from them automatically.
> 
> I'm sorry, this was silly.
> 
> The whole point of git is "addressing by content". Our submodule
> reference in edk2 makes us check out the cmocka tree at a known hash, so
> where that comes from is totally irrelevant.
> 
> I'm OK with the proposal as posted.

Also, apparently Neverware is part of Google:

https://cloudreadykb.neverware.com/s/article/Neverware-is-now-part-of-Google-FAQ

-- 
Rebecca Cran

^ permalink raw reply	[flat|nested] 16+ messages in thread

* Re: [EXTERNAL] Re: [edk2-rfc] [RFC] UnitTestFrameworkPkg cmocka submodule alternatives
  2020-12-17 16:01     ` Rebecca Cran
@ 2020-12-17 18:44       ` Bret Barkelew
  0 siblings, 0 replies; 16+ messages in thread
From: Bret Barkelew @ 2020-12-17 18:44 UTC (permalink / raw)
  To: Rebecca Cran, rfc@edk2.groups.io, lersek@redhat.com,
	Kinney, Michael D, devel@edk2.groups.io

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

My vote is to own a fork. I agree with Laszlo that it’s very low maintenance (may even be able to automate it with an existing DevOps pipeline to run every day) and gives us the most control of our destiny.

- Bret

From: Rebecca Cran<mailto:rebecca@nuviainc.com>
Sent: Thursday, December 17, 2020 8:01 AM
To: rfc@edk2.groups.io<mailto:rfc@edk2.groups.io>; lersek@redhat.com<mailto:lersek@redhat.com>; Kinney, Michael D<mailto:michael.d.kinney@intel.com>; devel@edk2.groups.io<mailto:devel@edk2.groups.io>; Bret Barkelew<mailto:Bret.Barkelew@microsoft.com>
Subject: [EXTERNAL] Re: [edk2-rfc] [RFC] UnitTestFrameworkPkg cmocka submodule alternatives

On 12/17/20 8:54 AM, Laszlo Ersek wrote:
> On 12/17/20 15:48, Laszlo Ersek wrote:
>
>> I don't know who or what the <https://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fneverware-mirrors&amp;data=04%7C01%7CBret.Barkelew%40microsoft.com%7C4cd7a0f805864468a2cb08d8a2a50fb8%7C72f988bf86f141af91ab2d7cd011db47%7C1%7C0%7C637438177109105172%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000&amp;sdata=oOtke9Pjf8TFVV2D0nv1xW%2B2a9sBsXgbFo1A9fodooc%3D&amp;reserved=0>
>> organization is, and I'd prefer not fetching code from them automatically.
>
> I'm sorry, this was silly.
>
> The whole point of git is "addressing by content". Our submodule
> reference in edk2 makes us check out the cmocka tree at a known hash, so
> where that comes from is totally irrelevant.
>
> I'm OK with the proposal as posted.

Also, apparently Neverware is part of Google:

https://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fcloudreadykb.neverware.com%2Fs%2Farticle%2FNeverware-is-now-part-of-Google-FAQ&amp;data=04%7C01%7CBret.Barkelew%40microsoft.com%7C4cd7a0f805864468a2cb08d8a2a50fb8%7C72f988bf86f141af91ab2d7cd011db47%7C1%7C0%7C637438177109115169%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000&amp;sdata=RssBVgGe3ryeIudu%2Fg0WEEIyyUzkNo5mUBiXw3TCqT0%3D&amp;reserved=0

--
Rebecca Cran


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

^ permalink raw reply	[flat|nested] 16+ messages in thread

* Re: [RFC] UnitTestFrameworkPkg cmocka submodule alternatives
  2020-12-16 18:45 [RFC] UnitTestFrameworkPkg cmocka submodule alternatives Michael D Kinney
  2020-12-17  0:58 ` 回复: [edk2-devel] " gaoliming
  2020-12-17 14:48 ` [edk2-rfc] " Laszlo Ersek
@ 2020-12-19 18:58 ` Michael D Kinney
  2020-12-19 19:07   ` Bret Barkelew
                     ` (2 more replies)
  2 siblings, 3 replies; 16+ messages in thread
From: Michael D Kinney @ 2020-12-19 18:58 UTC (permalink / raw)
  To: rfc@edk2.groups.io, devel@edk2.groups.io, 'Bret Barkelew',
	Laszlo Ersek <lersek@redhat.com> (lersek@redhat.com),
	Gao, Liming

Hello,

There have been a few suggestions to create a mirror of cmocka in TianoCore
org in GitHub.

I have found a GitHub action that can do a repo sync.

    https://github.com/marketplace/actions/github-repo-sync

I have created a temporary mirror of cmocka in my personal GitHub area that
uses this GitHub action to sync all branches and all tags once a day.

    https://github.com/mdkinney/mirror-cmocka

Here is the GitHub workflow file.  It must be in the default branch for the
repo using a branch name that is not present in the repo being mirrored.
In this case, I used a branch name of 'repo-sync'.

    https://github.com/mdkinney/mirror-cmocka/blob/repo-sync/.github/workflows/repo-sync.yml

Please provide feedback on this approach.  If we like this approach, then
I suggest we create a new repo in TianoCore called edk2-cmocka that is a 
mirror that is synced once a day and we update the cmocka submodule in the
edk2 repo to use edk2-cmocka.

Best regards,

Mike

> -----Original Message-----
> From: Kinney, Michael D <michael.d.kinney@intel.com>
> Sent: Wednesday, December 16, 2020 10:46 AM
> To: rfc@edk2.groups.io; devel@edk2.groups.io; Kinney, Michael D <michael.d.kinney@intel.com>; 'Bret Barkelew'
> <Bret.Barkelew@microsoft.com>
> Subject: [RFC] UnitTestFrameworkPkg cmocka submodule alternatives
> 
> Hello,
> 
> We have had at least three incidents in the last year where the link to the
> cmocka submodule in the UnitTestFrameworkPkg has not been available, and this
> impacted the EDK II CI system.  The following submodule link is the one that
> is not reliable:
> 
> 	https://git.cryptomilk.org/projects/cmocka.git
> 
> We have identified two potential mirrors of this repo:
> 
> 	https://github.com/neverware-mirrors/cmocka.git
> 	https://gitlab.com/cmocka/cmocka.git
> 
> The following patch provided a temporary fix for the EDK II CI agents, but
> does not help other consumers of the edk2 repository.
> 
> 	https://github.com/tianocore/edk2/commit/be746104d1766a8c363e74d6063144657820d688
> 
> I have seen one suggestion that TianoCore create its own
> mirror of cmocka.  This does require monitoring and maintenance
> by the  TianoCore community.  I would prefer to use a well
> maintained mirror in github as long as we do not observe any
> issues with the support of that mirror.
> 
> I propose we update the submodule in the UnitTestFrameworkPkg
> to use the https://github.com/neverware-mirrors/cmocka.git mirror.
> By using a mirror in github, we remove one external dependency.
> 
> Please provide feedback and comments on this proposal.  If there
> are no objections, then we will proceed with a patch review for
> this update.
> 
> Thanks,
> 
> Mike

^ permalink raw reply	[flat|nested] 16+ messages in thread

* Re: [RFC] UnitTestFrameworkPkg cmocka submodule alternatives
  2020-12-19 18:58 ` Michael D Kinney
@ 2020-12-19 19:07   ` Bret Barkelew
  2020-12-20  1:06   ` [edk2-rfc] " Rebecca Cran
  2021-02-15  0:43   ` [edk2-devel] " Rebecca Cran
  2 siblings, 0 replies; 16+ messages in thread
From: Bret Barkelew @ 2020-12-19 19:07 UTC (permalink / raw)
  To: Kinney, Michael D, rfc@edk2.groups.io, devel@edk2.groups.io,
	Laszlo Ersek <lersek@redhat.com> (lersek@redhat.com),
	liming.gao

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

I like it.

- Bret

From: Kinney, Michael D<mailto:michael.d.kinney@intel.com>
Sent: Saturday, December 19, 2020 10:59 AM
To: rfc@edk2.groups.io<mailto:rfc@edk2.groups.io>; devel@edk2.groups.io<mailto:devel@edk2.groups.io>; Bret Barkelew<mailto:Bret.Barkelew@microsoft.com>; Laszlo Ersek <lersek@redhat.com> (lersek@redhat.com)<mailto:lersek@redhat.com>; liming.gao<mailto:liming.gao@intel.com>
Subject: [EXTERNAL] RE: [RFC] UnitTestFrameworkPkg cmocka submodule alternatives

Hello,

There have been a few suggestions to create a mirror of cmocka in TianoCore
org in GitHub.

I have found a GitHub action that can do a repo sync.

    https://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fmarketplace%2Factions%2Fgithub-repo-sync&amp;data=04%7C01%7CBret.Barkelew%40microsoft.com%7C92da18aaec1443463b2508d8a45023a2%7C72f988bf86f141af91ab2d7cd011db47%7C1%7C0%7C637440011398666049%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C1000&amp;sdata=AX2EFVoGvtYoOZRtyFjwwTRbZkmQMgOCnjNNhWot7eo%3D&amp;reserved=0

I have created a temporary mirror of cmocka in my personal GitHub area that
uses this GitHub action to sync all branches and all tags once a day.

    https://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fmdkinney%2Fmirror-cmocka&amp;data=04%7C01%7CBret.Barkelew%40microsoft.com%7C92da18aaec1443463b2508d8a45023a2%7C72f988bf86f141af91ab2d7cd011db47%7C1%7C0%7C637440011398666049%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C1000&amp;sdata=U7ThFHC2fsgO9rVTNre3b0dI23b1Iudi1tw%2FjiFEdZc%3D&amp;reserved=0

Here is the GitHub workflow file.  It must be in the default branch for the
repo using a branch name that is not present in the repo being mirrored.
In this case, I used a branch name of 'repo-sync'.

    https://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fmdkinney%2Fmirror-cmocka%2Fblob%2Frepo-sync%2F.github%2Fworkflows%2Frepo-sync.yml&amp;data=04%7C01%7CBret.Barkelew%40microsoft.com%7C92da18aaec1443463b2508d8a45023a2%7C72f988bf86f141af91ab2d7cd011db47%7C1%7C0%7C637440011398666049%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C1000&amp;sdata=CuxE3Ljy2M7APvXfnuOqb6YPnFCX%2FUkDxsiIGEUHcvY%3D&amp;reserved=0

Please provide feedback on this approach.  If we like this approach, then
I suggest we create a new repo in TianoCore called edk2-cmocka that is a
mirror that is synced once a day and we update the cmocka submodule in the
edk2 repo to use edk2-cmocka.

Best regards,

Mike

> -----Original Message-----
> From: Kinney, Michael D <michael.d.kinney@intel.com>
> Sent: Wednesday, December 16, 2020 10:46 AM
> To: rfc@edk2.groups.io; devel@edk2.groups.io; Kinney, Michael D <michael.d.kinney@intel.com>; 'Bret Barkelew'
> <Bret.Barkelew@microsoft.com>
> Subject: [RFC] UnitTestFrameworkPkg cmocka submodule alternatives
>
> Hello,
>
> We have had at least three incidents in the last year where the link to the
> cmocka submodule in the UnitTestFrameworkPkg has not been available, and this
> impacted the EDK II CI system.  The following submodule link is the one that
> is not reliable:
>
>        https://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgit.cryptomilk.org%2Fprojects%2Fcmocka.git&amp;data=04%7C01%7CBret.Barkelew%40microsoft.com%7C92da18aaec1443463b2508d8a45023a2%7C72f988bf86f141af91ab2d7cd011db47%7C1%7C0%7C637440011398666049%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C1000&amp;sdata=m4tHEei6OQUwJu6jcldgxycoBJoajqPb9o5aKTDra%2F0%3D&amp;reserved=0
>
> We have identified two potential mirrors of this repo:
>
>        https://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fneverware-mirrors%2Fcmocka.git&amp;data=04%7C01%7CBret.Barkelew%40microsoft.com%7C92da18aaec1443463b2508d8a45023a2%7C72f988bf86f141af91ab2d7cd011db47%7C1%7C0%7C637440011398676045%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C1000&amp;sdata=19d9nV6kG4FQE3GC3ZzDTmE6%2F7EjNdMMWM%2BbSGT6bSI%3D&amp;reserved=0
>        https://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgitlab.com%2Fcmocka%2Fcmocka.git&amp;data=04%7C01%7CBret.Barkelew%40microsoft.com%7C92da18aaec1443463b2508d8a45023a2%7C72f988bf86f141af91ab2d7cd011db47%7C1%7C0%7C637440011398676045%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C1000&amp;sdata=xgErAgcMsMSKbrLZB9kgCHHl3r%2FzJM%2FJy6jhxpi0ObY%3D&amp;reserved=0
>
> The following patch provided a temporary fix for the EDK II CI agents, but
> does not help other consumers of the edk2 repository.
>
>        https://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Ftianocore%2Fedk2%2Fcommit%2Fbe746104d1766a8c363e74d6063144657820d688&amp;data=04%7C01%7CBret.Barkelew%40microsoft.com%7C92da18aaec1443463b2508d8a45023a2%7C72f988bf86f141af91ab2d7cd011db47%7C1%7C0%7C637440011398676045%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C1000&amp;sdata=Nyx8mBIwNxMEu4SdHYGiQhBGcpAPxxPHXBgI%2BM0CIU0%3D&amp;reserved=0
>
> I have seen one suggestion that TianoCore create its own
> mirror of cmocka.  This does require monitoring and maintenance
> by the  TianoCore community.  I would prefer to use a well
> maintained mirror in github as long as we do not observe any
> issues with the support of that mirror.
>
> I propose we update the submodule in the UnitTestFrameworkPkg
> to use the https://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fneverware-mirrors%2Fcmocka.git&amp;data=04%7C01%7CBret.Barkelew%40microsoft.com%7C92da18aaec1443463b2508d8a45023a2%7C72f988bf86f141af91ab2d7cd011db47%7C1%7C0%7C637440011398676045%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C1000&amp;sdata=19d9nV6kG4FQE3GC3ZzDTmE6%2F7EjNdMMWM%2BbSGT6bSI%3D&amp;reserved=0 mirror.
> By using a mirror in github, we remove one external dependency.
>
> Please provide feedback and comments on this proposal.  If there
> are no objections, then we will proceed with a patch review for
> this update.
>
> Thanks,
>
> Mike


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

^ permalink raw reply	[flat|nested] 16+ messages in thread

* Re: [edk2-rfc] [RFC] UnitTestFrameworkPkg cmocka submodule alternatives
  2020-12-19 18:58 ` Michael D Kinney
  2020-12-19 19:07   ` Bret Barkelew
@ 2020-12-20  1:06   ` Rebecca Cran
  2020-12-21  1:18     ` 回复: [edk2-devel] " gaoliming
  2021-02-15  0:43   ` [edk2-devel] " Rebecca Cran
  2 siblings, 1 reply; 16+ messages in thread
From: Rebecca Cran @ 2020-12-20  1:06 UTC (permalink / raw)
  To: rfc, michael.d.kinney, devel@edk2.groups.io,
	'Bret Barkelew', Laszlo Ersek, Gao, Liming

On 12/19/20 11:58 AM, Michael D Kinney wrote:
> There have been a few suggestions to create a mirror of cmocka in TianoCore
> org in GitHub.
> 
> I have found a GitHub action that can do a repo sync.
> 
>      https://github.com/marketplace/actions/github-repo-sync
> 
> I have created a temporary mirror of cmocka in my personal GitHub area that
> uses this GitHub action to sync all branches and all tags once a day.
> 
>      https://github.com/mdkinney/mirror-cmocka
> 
> Here is the GitHub workflow file.  It must be in the default branch for the
> repo using a branch name that is not present in the repo being mirrored.
> In this case, I used a branch name of 'repo-sync'.
> 
>      https://github.com/mdkinney/mirror-cmocka/blob/repo-sync/.github/workflows/repo-sync.yml
> 
> Please provide feedback on this approach.  If we like this approach, then
> I suggest we create a new repo in TianoCore called edk2-cmocka that is a
> mirror that is synced once a day and we update the cmocka submodule in the
> edk2 repo to use edk2-cmocka.
> 

I'd suggest just using the Gitlab mirror. Unlike cryptomilk.org, Gitlab 
should be just as reliable as Github and won't introduce another 
potential failure point.

-- 
Rebecca Cran


^ permalink raw reply	[flat|nested] 16+ messages in thread

* 回复: [edk2-devel] [edk2-rfc] [RFC] UnitTestFrameworkPkg cmocka submodule alternatives
  2020-12-20  1:06   ` [edk2-rfc] " Rebecca Cran
@ 2020-12-21  1:18     ` gaoliming
  2020-12-21 20:14       ` Michael D Kinney
  0 siblings, 1 reply; 16+ messages in thread
From: gaoliming @ 2020-12-21  1:18 UTC (permalink / raw)
  To: devel, rebecca, rfc, michael.d.kinney, 'Bret Barkelew',
	'Laszlo Ersek', 'Gao, Liming'

I also prefer to fetch all submodules from github. Then, all git repos will have the same behavior. 

Thanks
Liming
> -----邮件原件-----
> 发件人: bounce+27952+69261+4905953+8761045@groups.io
> <bounce+27952+69261+4905953+8761045@groups.io> 代表 Rebecca Cran
> 发送时间: 2020年12月20日 9:06
> 收件人: rfc@edk2.groups.io; michael.d.kinney@intel.com;
> devel@edk2.groups.io; 'Bret Barkelew' <Bret.Barkelew@microsoft.com>;
> Laszlo Ersek <lersek@redhat.com>; Gao, Liming <liming.gao@intel.com>
> 主题: Re: [edk2-devel] [edk2-rfc] [RFC] UnitTestFrameworkPkg cmocka
> submodule alternatives
> 
> On 12/19/20 11:58 AM, Michael D Kinney wrote:
> > There have been a few suggestions to create a mirror of cmocka in
> TianoCore
> > org in GitHub.
> >
> > I have found a GitHub action that can do a repo sync.
> >
> >      https://github.com/marketplace/actions/github-repo-sync
> >
> > I have created a temporary mirror of cmocka in my personal GitHub area
> that
> > uses this GitHub action to sync all branches and all tags once a day.
> >
> >      https://github.com/mdkinney/mirror-cmocka
> >
> > Here is the GitHub workflow file.  It must be in the default branch for the
> > repo using a branch name that is not present in the repo being mirrored.
> > In this case, I used a branch name of 'repo-sync'.
> >
> >
> https://github.com/mdkinney/mirror-cmocka/blob/repo-sync/.github/workflo
> ws/repo-sync.yml
> >
> > Please provide feedback on this approach.  If we like this approach, then
> > I suggest we create a new repo in TianoCore called edk2-cmocka that is a
> > mirror that is synced once a day and we update the cmocka submodule in
> the
> > edk2 repo to use edk2-cmocka.
> >
> 
> I'd suggest just using the Gitlab mirror. Unlike cryptomilk.org, Gitlab
> should be just as reliable as Github and won't introduce another
> potential failure point.
> 
> --
> Rebecca Cran
> 
> 
> 
> 
> 




^ permalink raw reply	[flat|nested] 16+ messages in thread

* Re: [edk2-devel] [edk2-rfc] [RFC] UnitTestFrameworkPkg cmocka submodule alternatives
  2020-12-21  1:18     ` 回复: [edk2-devel] " gaoliming
@ 2020-12-21 20:14       ` Michael D Kinney
  2020-12-21 20:42         ` Rebecca Cran
       [not found]         ` <1652D609A7BB8C97.24251@groups.io>
  0 siblings, 2 replies; 16+ messages in thread
From: Michael D Kinney @ 2020-12-21 20:14 UTC (permalink / raw)
  To: gaoliming, devel@edk2.groups.io, rebecca@bsdio.com,
	rfc@edk2.groups.io, 'Bret Barkelew',
	'Laszlo Ersek', 'Gao, Liming', Kinney, Michael D

Rebecca,

Are you ok with using a GitHub mirror?

Mike

> -----Original Message-----
> From: gaoliming <gaoliming@byosoft.com.cn>
> Sent: Sunday, December 20, 2020 5:18 PM
> To: devel@edk2.groups.io; rebecca@bsdio.com; rfc@edk2.groups.io; Kinney, Michael D <michael.d.kinney@intel.com>; 'Bret
> Barkelew' <Bret.Barkelew@microsoft.com>; 'Laszlo Ersek' <lersek@redhat.com>; 'Gao, Liming' <liming.gao@intel.com>
> Subject: 回复: [edk2-devel] [edk2-rfc] [RFC] UnitTestFrameworkPkg cmocka submodule alternatives
> 
> I also prefer to fetch all submodules from github. Then, all git repos will have the same behavior.
> 
> Thanks
> Liming
> > -----邮件原件-----
> > 发件人: bounce+27952+69261+4905953+8761045@groups.io
> > <bounce+27952+69261+4905953+8761045@groups.io> 代表 Rebecca Cran
> > 发送时间: 2020年12月20日 9:06
> > 收件人: rfc@edk2.groups.io; michael.d.kinney@intel.com;
> > devel@edk2.groups.io; 'Bret Barkelew' <Bret.Barkelew@microsoft.com>;
> > Laszlo Ersek <lersek@redhat.com>; Gao, Liming <liming.gao@intel.com>
> > 主题: Re: [edk2-devel] [edk2-rfc] [RFC] UnitTestFrameworkPkg cmocka
> > submodule alternatives
> >
> > On 12/19/20 11:58 AM, Michael D Kinney wrote:
> > > There have been a few suggestions to create a mirror of cmocka in
> > TianoCore
> > > org in GitHub.
> > >
> > > I have found a GitHub action that can do a repo sync.
> > >
> > >      https://github.com/marketplace/actions/github-repo-sync
> > >
> > > I have created a temporary mirror of cmocka in my personal GitHub area
> > that
> > > uses this GitHub action to sync all branches and all tags once a day.
> > >
> > >      https://github.com/mdkinney/mirror-cmocka
> > >
> > > Here is the GitHub workflow file.  It must be in the default branch for the
> > > repo using a branch name that is not present in the repo being mirrored.
> > > In this case, I used a branch name of 'repo-sync'.
> > >
> > >
> > https://github.com/mdkinney/mirror-cmocka/blob/repo-sync/.github/workflo
> > ws/repo-sync.yml
> > >
> > > Please provide feedback on this approach.  If we like this approach, then
> > > I suggest we create a new repo in TianoCore called edk2-cmocka that is a
> > > mirror that is synced once a day and we update the cmocka submodule in
> > the
> > > edk2 repo to use edk2-cmocka.
> > >
> >
> > I'd suggest just using the Gitlab mirror. Unlike cryptomilk.org, Gitlab
> > should be just as reliable as Github and won't introduce another
> > potential failure point.
> >
> > --
> > Rebecca Cran
> >
> >
> >
> > 
> >
> 
> 


^ permalink raw reply	[flat|nested] 16+ messages in thread

* Re: [edk2-devel] [edk2-rfc] [RFC] UnitTestFrameworkPkg cmocka submodule alternatives
  2020-12-21 20:14       ` Michael D Kinney
@ 2020-12-21 20:42         ` Rebecca Cran
       [not found]         ` <1652D609A7BB8C97.24251@groups.io>
  1 sibling, 0 replies; 16+ messages in thread
From: Rebecca Cran @ 2020-12-21 20:42 UTC (permalink / raw)
  To: rfc, michael.d.kinney, gaoliming, devel@edk2.groups.io,
	'Bret Barkelew', 'Laszlo Ersek',
	'Gao, Liming'

On 12/21/20 1:14 PM, Michael D Kinney wrote:
> Rebecca,
> 
> Are you ok with using a GitHub mirror?

Yes, that's fine with me.

-- 
Rebecca Cran


^ permalink raw reply	[flat|nested] 16+ messages in thread

* Re: [edk2-devel] [edk2-rfc] [RFC] UnitTestFrameworkPkg cmocka submodule alternatives
       [not found]         ` <1652D609A7BB8C97.24251@groups.io>
@ 2020-12-21 20:44           ` Rebecca Cran
  2020-12-22  0:41             ` 回复: " gaoliming
  0 siblings, 1 reply; 16+ messages in thread
From: Rebecca Cran @ 2020-12-21 20:44 UTC (permalink / raw)
  To: rfc, michael.d.kinney, gaoliming, devel@edk2.groups.io,
	'Bret Barkelew', 'Laszlo Ersek',
	'Gao, Liming'

On 12/21/20 1:42 PM, Rebecca Cran wrote:
> On 12/21/20 1:14 PM, Michael D Kinney wrote:
>> Rebecca,
>>
>> Are you ok with using a GitHub mirror?
> 
> Yes, that's fine with me.
> 

By the way, I've started getting bounces from Liming's Intel email address.

-- 
Rebecca Cran


^ permalink raw reply	[flat|nested] 16+ messages in thread

* 回复: [edk2-devel] [edk2-rfc] [RFC] UnitTestFrameworkPkg cmocka submodule alternatives
  2020-12-21 20:44           ` Rebecca Cran
@ 2020-12-22  0:41             ` gaoliming
  0 siblings, 0 replies; 16+ messages in thread
From: gaoliming @ 2020-12-22  0:41 UTC (permalink / raw)
  To: devel, rebecca, rfc, michael.d.kinney, 'Bret Barkelew',
	'Laszlo Ersek'
  Cc: 'Liming Gao'

Rebecca:
  My new mail address is Liming Gao <gaoliming@byosoft.com.cn>. It has updated in Maintainers.txt. 

Thanks
Liming
> -----邮件原件-----
> 发件人: bounce+27952+69347+4905953+8761045@groups.io
> <bounce+27952+69347+4905953+8761045@groups.io> 代表 Rebecca Cran
> 发送时间: 2020年12月22日 4:45
> 收件人: rfc@edk2.groups.io; michael.d.kinney@intel.com; gaoliming
> <gaoliming@byosoft.com.cn>; devel@edk2.groups.io; 'Bret Barkelew'
> <Bret.Barkelew@microsoft.com>; 'Laszlo Ersek' <lersek@redhat.com>; 'Gao,
> Liming' <liming.gao@intel.com>
> 主题: Re: [edk2-devel] [edk2-rfc] [RFC] UnitTestFrameworkPkg cmocka
> submodule alternatives
> 
> On 12/21/20 1:42 PM, Rebecca Cran wrote:
> > On 12/21/20 1:14 PM, Michael D Kinney wrote:
> >> Rebecca,
> >>
> >> Are you ok with using a GitHub mirror?
> >
> > Yes, that's fine with me.
> >
> 
> By the way, I've started getting bounces from Liming's Intel email address.
> 
> --
> Rebecca Cran
> 
> 
> 
> 
> 




^ permalink raw reply	[flat|nested] 16+ messages in thread

* Re: [edk2-devel] [RFC] UnitTestFrameworkPkg cmocka submodule alternatives
  2020-12-19 18:58 ` Michael D Kinney
  2020-12-19 19:07   ` Bret Barkelew
  2020-12-20  1:06   ` [edk2-rfc] " Rebecca Cran
@ 2021-02-15  0:43   ` Rebecca Cran
  2021-04-06 20:22     ` Michael D Kinney
  2 siblings, 1 reply; 16+ messages in thread
From: Rebecca Cran @ 2021-02-15  0:43 UTC (permalink / raw)
  To: devel, michael.d.kinney, rfc@edk2.groups.io,
	'Bret Barkelew', Laszlo Ersek, Gao, Liming

On 12/19/20 11:58 AM, Michael D Kinney wrote:
> Hello,
> 
> There have been a few suggestions to create a mirror of cmocka in TianoCore
> org in GitHub.
> 
> I have found a GitHub action that can do a repo sync.
> 
>      https://github.com/marketplace/actions/github-repo-sync
> 
> I have created a temporary mirror of cmocka in my personal GitHub area that
> uses this GitHub action to sync all branches and all tags once a day.
> 
>      https://github.com/mdkinney/mirror-cmocka
> 
> Here is the GitHub workflow file.  It must be in the default branch for the
> repo using a branch name that is not present in the repo being mirrored.
> In this case, I used a branch name of 'repo-sync'.
> 
>      https://github.com/mdkinney/mirror-cmocka/blob/repo-sync/.github/workflows/repo-sync.yml
> 
> Please provide feedback on this approach.  If we like this approach, then
> I suggest we create a new repo in TianoCore called edk2-cmocka that is a
> mirror that is synced once a day and we update the cmocka submodule in the
> edk2 repo to use edk2-cmocka.

I just noticed this never got committed: 
https://github.com/tianocore/edk2/blob/master/.gitmodules still shows 
the old URL.

-- 
Rebecca Cran


^ permalink raw reply	[flat|nested] 16+ messages in thread

* Re: [edk2-devel] [RFC] UnitTestFrameworkPkg cmocka submodule alternatives
  2021-02-15  0:43   ` [edk2-devel] " Rebecca Cran
@ 2021-04-06 20:22     ` Michael D Kinney
  0 siblings, 0 replies; 16+ messages in thread
From: Michael D Kinney @ 2021-04-06 20:22 UTC (permalink / raw)
  To: Rebecca Cran, devel@edk2.groups.io, rfc@edk2.groups.io,
	'Bret Barkelew', Laszlo Ersek, Gao, Liming,
	Kinney, Michael D

Hi Rebecca,

Thank you for the reminder.  I have entered a BZ and sent a patch for review

https://bugzilla.tianocore.org/show_bug.cgi?id=3301

https://edk2.groups.io/g/devel/message/73744

Mike

> -----Original Message-----
> From: Rebecca Cran <rebecca@bsdio.com>
> Sent: Sunday, February 14, 2021 4:43 PM
> To: devel@edk2.groups.io; Kinney, Michael D <michael.d.kinney@intel.com>; rfc@edk2.groups.io; 'Bret Barkelew'
> <Bret.Barkelew@microsoft.com>; Laszlo Ersek <lersek@redhat.com>; Gao, Liming <liming.gao@intel.com>
> Subject: Re: [edk2-devel] [RFC] UnitTestFrameworkPkg cmocka submodule alternatives
> 
> On 12/19/20 11:58 AM, Michael D Kinney wrote:
> > Hello,
> >
> > There have been a few suggestions to create a mirror of cmocka in TianoCore
> > org in GitHub.
> >
> > I have found a GitHub action that can do a repo sync.
> >
> >      https://github.com/marketplace/actions/github-repo-sync
> >
> > I have created a temporary mirror of cmocka in my personal GitHub area that
> > uses this GitHub action to sync all branches and all tags once a day.
> >
> >      https://github.com/mdkinney/mirror-cmocka
> >
> > Here is the GitHub workflow file.  It must be in the default branch for the
> > repo using a branch name that is not present in the repo being mirrored.
> > In this case, I used a branch name of 'repo-sync'.
> >
> >      https://github.com/mdkinney/mirror-cmocka/blob/repo-sync/.github/workflows/repo-sync.yml
> >
> > Please provide feedback on this approach.  If we like this approach, then
> > I suggest we create a new repo in TianoCore called edk2-cmocka that is a
> > mirror that is synced once a day and we update the cmocka submodule in the
> > edk2 repo to use edk2-cmocka.
> 
> I just noticed this never got committed:
> https://github.com/tianocore/edk2/blob/master/.gitmodules still shows
> the old URL.
> 
> --
> Rebecca Cran


^ permalink raw reply	[flat|nested] 16+ messages in thread

end of thread, other threads:[~2021-04-06 20:22 UTC | newest]

Thread overview: 16+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2020-12-16 18:45 [RFC] UnitTestFrameworkPkg cmocka submodule alternatives Michael D Kinney
2020-12-17  0:58 ` 回复: [edk2-devel] " gaoliming
2020-12-17 14:48 ` [edk2-rfc] " Laszlo Ersek
2020-12-17 15:54   ` Laszlo Ersek
2020-12-17 16:01     ` Rebecca Cran
2020-12-17 18:44       ` [EXTERNAL] " Bret Barkelew
2020-12-19 18:58 ` Michael D Kinney
2020-12-19 19:07   ` Bret Barkelew
2020-12-20  1:06   ` [edk2-rfc] " Rebecca Cran
2020-12-21  1:18     ` 回复: [edk2-devel] " gaoliming
2020-12-21 20:14       ` Michael D Kinney
2020-12-21 20:42         ` Rebecca Cran
     [not found]         ` <1652D609A7BB8C97.24251@groups.io>
2020-12-21 20:44           ` Rebecca Cran
2020-12-22  0:41             ` 回复: " gaoliming
2021-02-15  0:43   ` [edk2-devel] " Rebecca Cran
2021-04-06 20:22     ` Michael D Kinney

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox