[Mesa-dev] [PATCH v2] docs: Document GitLab merge request process (email alternative)

Rob Clark robdclark at gmail.com
Wed Dec 12 01:15:19 UTC 2018


On Tue, Dec 11, 2018 at 7:06 PM Jason Ekstrand <jason at jlekstrand.net> wrote:
>
> Ping?
>
> I see about 5 acks/reviews, 3 of which are from Intel which doesn't exactly seem like overwhelming consensus.  However, we also haven't had any debate in a while.
>
> I know some people are somewhat skeptical as to how well it will work but they won't be able to see until we actually start experimenting with it which we can't do until we allow MRs.  Personally, I say we should just turn on the option in GitLab and people can start playing with it and see how it goes.  We can always decide later that it was a terrible idea and move all active MRs back to the list.
>

I think some way to get cover-letters or similar (pref w/ link to mr)
to list would be nice, but I'm defn ok w/ 'lets try it and then fine
tune' approach.  I think worst case is I overlook mr notification spam
and someone has to poke me on irc, which is more or less the current
worst case...

Acked-by: Rob Clark <robdclark at gmail.com>


> --Jason
>
> On Fri, Dec 7, 2018 at 2:24 PM Daniel Stone <daniel at fooishbar.org> wrote:
>>
>> Hi,
>>
>> On Sat, 8 Dec 2018 at 05:15, Eric Engestrom <eric.engestrom at intel.com> wrote:
>> > On Friday, 2018-12-07 10:19:23 +0100, Erik Faye-Lund wrote:
>> > > Automated emails (and perhaps IRC bot) would be really nice.
>> >
>> > Agreed. Email would be great to help with the transition.
>> > There's work currently being done on GitLab to allow for mailing lists
>> > to be notified; this should cover 'new MR' as well.
>> > If we need this feature before GitLab is done, it should be possible to
>> > write a bot using the webhooks, just needs someone to take the time to
>> > do it :)
>> >
>> > For IRC, there's already some integration, but it's limited to notifying
>> > about git pushes for now:
>> > https://docs.gitlab.com/ee/user/project/integrations/irker.html
>> >
>> > There's an open issue about adding more events, but it hasn't seen much
>> > activity:
>> > https://gitlab.com/gitlab-org/gitlab-ce/issues/7965
>>
>> Wayland uses a couple of eventd plugins chained together:
>> https://github.com/sardemff7/git-eventc
>>
>> That notifies the channel when issues and MRs are opened or closed and
>> on push as well, including things like the labels. It's been pretty
>> useful so far.
>>
>> > > Even better if it could be hooked up to scripts/get_reviewer.pl, and
>> > > automatically CC "the right people".
>> >
>> > Side note, I've been rewriting that script, although I need to send v2
>> > out at some point:
>> > https://patchwork.freedesktop.org/patch/226256/
>> >
>> > I would be trivial to hook that into a bot we'd write, but I don't think
>> > GitLab has support for something like this. I just opened an issue about
>> > adding support directly in GitLab:
>> > https://gitlab.com/gitlab-org/gitlab-ce/issues/55035
>>
>> This already exists, as an EE-only feature called 'code owners':
>> https://docs.gitlab.com/ee/user/project/code_owners.html
>> https://gitlab.com/gitlab-org/gitlab-ee/issues/1012
>>
>> Cheers,
>> Daniel
>> _______________________________________________
>> mesa-dev mailing list
>> mesa-dev at lists.freedesktop.org
>> https://lists.freedesktop.org/mailman/listinfo/mesa-dev
>
> _______________________________________________
> mesa-dev mailing list
> mesa-dev at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/mesa-dev


More information about the mesa-dev mailing list