[PATCH 00/11] Add labels support

Michael Ellerman mpe at ellerman.id.au
Tue Sep 25 13:30:10 AEST 2018


Don Zickus <dzickus at redhat.com> writes:

> On Tue, Sep 11, 2018 at 12:19:02PM -0600, Stephen Finucane wrote:
>> > > Personally I would *really* like labels to land. They unlock a lot of
>> > > potential improvements to workflow for maintainers, eg. automated
>> > > tagging, tagging based on test results etc. As well as finer grained
>> > > reporting of status to submitters, eg. instead of "new" -> "under
>> > > review" -> "accepted", I can mark a patch as "under review" and
>> > > "applied-to-some-branch", then "under review" and
>> > > "in-testing" etc. etc.
>> > 
>> > Hi,
>> > 
>> > I missed the labels discussion patch, but at Red Hat we implemented
>> > something called 'tags' on our internal patchwork version which is probably
>> > the same idea.  It allows us to add arbitrary data like 'bugzilla' and
>> > 'needinfo' (and various CI tags) to our patches.  We have been trying to
>> > push parts of it to this mailing list
>> > 
>> > https://lists.ozlabs.org/pipermail/patchwork/2018-April/005098.html
>> > 
>> > with little success.  We would be interested in promoting either concept to
>> > patchwork.  Anything that allows us to attach random data to and is
>> > searchable/filterable.
>> 
>> Yeah, sorry about the delay reviewing that series. I've jumped on it
>> now and will do my best to push it to completion over the next few
>> weeks.
>> 
>> I see labels as different to tags. Tags are key:value pairs, generally
>> extracted from a message body, while Labels are simply values initially
>> stripped from the subject. We could build a unified model that supports
>> both, perhaps by making the value aspect of Tags nullable, but I'm not
>> sure if that's something we'd want nor how this would be handled in the
>> UI/APIs. What are your thoughts here?
>
> Hi Stephen,
>
> Our internal workflow is a mixture.  We called everything 'tags' for
> consistency but really our data is either:
>
> *  key:value pairs extracted from message Body (bugzilla, acked-by, backported
>    commit id)
>
> *  arbitrary key:value pairs from automation tools (build id, test job id,
>    status of comment), nothing attributed to an email message body or easily
>    re-created by re-reading emails.
>
> Now I know you were not excited about arbitrary tags/labels when we first
> spoke, so we have been focusing on the first type of data.
>
> But ideally we would like both (or actually the second type covers the first
> type :-) ).  And I think the second type of data covers what Michael E. was
> describing for his needs (he called them labels).

Yeah I don't claim to have the terminology correct :)

But certainly "tags", ie. key/value string pairs, would work for me.

cheers


More information about the Patchwork mailing list