On Wed, Jun 24, 2026 at 03:19:15PM +0200, Philippe Mathieu-Daudé wrote:
> On 24/6/26 14:29, Daniel P. Berrangé wrote:
> > When triaging GitLab issues, whether regular bugs or security
> > disclosures, we need to be able to assign suitable accounts.
> > 
> > We can only assign Git Lab handles for accounts that are members
> > of the QEMU group, or QEMU repository with the "Reporter" role
> > or higher.
> > 
> > Currently one needs to run 'get_maintainer.pl -f <file>' to get
> > the maintainer / reviewer names & emails, and then look in the
> > GitLab members page to see if there is a match.
> > 
> > This simplifies that by including the GitLab handle alongside
> > the email addresses:
> > 
> > The GitLab account handle -> real name mapping is stored in
> > separate files .gitlab-map-auto & .gitlab-map-manual to avoid
> > redundancy in the MAINTAINERS file. The former file is
> > auto-generated and catches 99% of the mappings, the latter is
> > for manual fixups.
> > 
> > With this series applied as is, we get the following stats
> > 
> > $ ./scripts/gitlab-map-check
> > Missing GitLab handle: Akihiko Odaki

...

> > Missing GitLab handle: zhenwei pi
> > Missing GitLab handle: Zhenzhong Duan
> > Missing GitLab handle: Zhuoying Cai
> > GitLab handles missing: 161 / present: 57
> > 
> > Plenty of those will have GitLab accounts, but are simply not members
> > of QEMU. Plenty more probably aren't registered with GitLab at all.
> > 
> > Short of emailing all of those people directly, I don't see a good
> > way to identify their GitLab accounts (if any), in order to invite
> > them to the QEMU project. I don't want to blindly invite accounts
> > which merely have the same name on GitLab as there's too much risk
> > of getting a mis-match.

...

> No clue why Hanna is listed here...
> 
> Applying: gitlab: introduce files mapping GitLab accounts to real names
> Applying: get_maintainer: add ability to report Git Lab handle
> .git/rebase-apply/patch:121: trailing whitespace.
> hreitz        Hanna Reitz
> warning: 1 line adds whitespace errors.

There is trailing whitespace after 'Reitz' that crept in.

> 
> Anyhow,
> Tested-by: Philippe Mathieu-Daudé <[email protected]>
> 

With regards,
Daniel
-- 
|: https://berrange.com       ~~        https://hachyderm.io/@berrange :|
|: https://libvirt.org          ~~          https://entangle-photo.org :|
|: https://pixelfed.art/berrange   ~~    https://fstop138.berrange.com :|


Reply via email to