Re: [PATCH 1/6] .github: point Repo Lockdown bot to GitLab repo

2021-01-12 Thread Stefan Hajnoczi
On Tue, Jan 12, 2021 at 08:49:55AM +0100, Thomas Huth wrote:
> On 11/01/2021 12.51, Daniel P. Berrangé wrote:
> > On Mon, Jan 11, 2021 at 11:50:12AM +, Stefan Hajnoczi wrote:
> > > Use the GitLab repo URL as the main repo location in order to reduce
> > > load on qemu.org.
> > > 
> > > Signed-off-by: Stefan Hajnoczi 
> > > ---
> > >   .github/lockdown.yml | 8 
> > >   1 file changed, 4 insertions(+), 4 deletions(-)
> > > 
> > > diff --git a/.github/lockdown.yml b/.github/lockdown.yml
> > > index 9acc393f1c..07fc2f31ee 100644
> > > --- a/.github/lockdown.yml
> > > +++ b/.github/lockdown.yml
> > > @@ -10,8 +10,8 @@ issues:
> > > comment: |
> > >   Thank you for your interest in the QEMU project.
> > > -This repository is a read-only mirror of the project's master
> > > -repostories hosted on https://git.qemu.org/git/qemu.git.
> > > +This repository is a read-only mirror of the project's repostories 
> > > hosted
> > > +at https://gitlab.com/qemu-project/qemu.git.
> > 
> > NB this URL triggers a redirect to strip the .git suffix
> > 
> >https://gitlab.com/qemu-project/qemu
> 
> If you try to do:
> 
>  git clone https://gitlab.com/qemu-project/qemu
> 
> You get this warning message instead:
> 
>  warning: redirecting to https://gitlab.com/qemu-project/qemu.git/
> 
> So I think the patch is fine.
> 
> Reviewed-by: Thomas Huth 

Same here. https://gitlab.com/qemu-project/qemu.git clones without
warnings here. https://gitlab.com/qemu-project/qemu clones with a
warning, so I think we should leave the patch unchanged.

Stefan


signature.asc
Description: PGP signature


Re: [PATCH 1/6] .github: point Repo Lockdown bot to GitLab repo

2021-01-11 Thread Thomas Huth

On 11/01/2021 12.51, Daniel P. Berrangé wrote:

On Mon, Jan 11, 2021 at 11:50:12AM +, Stefan Hajnoczi wrote:

Use the GitLab repo URL as the main repo location in order to reduce
load on qemu.org.

Signed-off-by: Stefan Hajnoczi 
---
  .github/lockdown.yml | 8 
  1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/.github/lockdown.yml b/.github/lockdown.yml
index 9acc393f1c..07fc2f31ee 100644
--- a/.github/lockdown.yml
+++ b/.github/lockdown.yml
@@ -10,8 +10,8 @@ issues:
comment: |
  Thank you for your interest in the QEMU project.
  
-This repository is a read-only mirror of the project's master

-repostories hosted on https://git.qemu.org/git/qemu.git.
+This repository is a read-only mirror of the project's repostories hosted
+at https://gitlab.com/qemu-project/qemu.git.


NB this URL triggers a redirect to strip the .git suffix

   https://gitlab.com/qemu-project/qemu


If you try to do:

 git clone https://gitlab.com/qemu-project/qemu

You get this warning message instead:

 warning: redirecting to https://gitlab.com/qemu-project/qemu.git/

So I think the patch is fine.

Reviewed-by: Thomas Huth 




Re: [PATCH 1/6] .github: point Repo Lockdown bot to GitLab repo

2021-01-11 Thread Daniel P . Berrangé
On Mon, Jan 11, 2021 at 11:50:12AM +, Stefan Hajnoczi wrote:
> Use the GitLab repo URL as the main repo location in order to reduce
> load on qemu.org.
> 
> Signed-off-by: Stefan Hajnoczi 
> ---
>  .github/lockdown.yml | 8 
>  1 file changed, 4 insertions(+), 4 deletions(-)
> 
> diff --git a/.github/lockdown.yml b/.github/lockdown.yml
> index 9acc393f1c..07fc2f31ee 100644
> --- a/.github/lockdown.yml
> +++ b/.github/lockdown.yml
> @@ -10,8 +10,8 @@ issues:
>comment: |
>  Thank you for your interest in the QEMU project.
>  
> -This repository is a read-only mirror of the project's master
> -repostories hosted on https://git.qemu.org/git/qemu.git.
> +This repository is a read-only mirror of the project's repostories hosted
> +at https://gitlab.com/qemu-project/qemu.git.

NB this URL triggers a redirect to strip the .git suffix

  https://gitlab.com/qemu-project/qemu

>  The project does not process issues filed on GitHub.
>  
>  The project issues are tracked on Launchpad:
> @@ -24,8 +24,8 @@ pulls:
>comment: |
>  Thank you for your interest in the QEMU project.
>  
> -This repository is a read-only mirror of the project's master
> -repostories hosted on https://git.qemu.org/git/qemu.git.
> +This repository is a read-only mirror of the project's repostories hosted
> +on https://gitlab.com/qemu-project/qemu.git.

Likewise

>  The project does not process merge requests filed on GitHub.
>  
>  QEMU welcomes contributions of code (either fixing bugs or adding new

Regards,
Daniel
-- 
|: https://berrange.com  -o-https://www.flickr.com/photos/dberrange :|
|: https://libvirt.org -o-https://fstop138.berrange.com :|
|: https://entangle-photo.org-o-https://www.instagram.com/dberrange :|




[PATCH 1/6] .github: point Repo Lockdown bot to GitLab repo

2021-01-11 Thread Stefan Hajnoczi
Use the GitLab repo URL as the main repo location in order to reduce
load on qemu.org.

Signed-off-by: Stefan Hajnoczi 
---
 .github/lockdown.yml | 8 
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/.github/lockdown.yml b/.github/lockdown.yml
index 9acc393f1c..07fc2f31ee 100644
--- a/.github/lockdown.yml
+++ b/.github/lockdown.yml
@@ -10,8 +10,8 @@ issues:
   comment: |
 Thank you for your interest in the QEMU project.
 
-This repository is a read-only mirror of the project's master
-repostories hosted on https://git.qemu.org/git/qemu.git.
+This repository is a read-only mirror of the project's repostories hosted
+at https://gitlab.com/qemu-project/qemu.git.
 The project does not process issues filed on GitHub.
 
 The project issues are tracked on Launchpad:
@@ -24,8 +24,8 @@ pulls:
   comment: |
 Thank you for your interest in the QEMU project.
 
-This repository is a read-only mirror of the project's master
-repostories hosted on https://git.qemu.org/git/qemu.git.
+This repository is a read-only mirror of the project's repostories hosted
+on https://gitlab.com/qemu-project/qemu.git.
 The project does not process merge requests filed on GitHub.
 
 QEMU welcomes contributions of code (either fixing bugs or adding new
-- 
2.29.2