[openstack-dev] [Tempest][qa] Adding tags to commit messages
Sean Dague
sean at dague.net
Tue Dec 24 11:32:04 UTC 2013
On 12/24/2013 01:47 AM, Yair Fried wrote:
> Hi,
> Suggestion: Please consider tagging your Tempest commit messages the same way you do your mails in the mailing list
>
> Explanation: Since tempest is a single project testing multiple Openstack project we have a very diverse collection of patches as well as reviewers. Tagging our commit messages will allow us to classify patches and thus:
> 1. Allow reviewer to focus on patches related to their area of expertise
> 2. Track "trends" in patches - I think we all know that we lack in Neutron testing for example, but can we assess how many network related patches are for awaiting review
> 3. Future automation of flagging "interesting" patches
>
> You can usually tell all of this from reviewing the patch, but by then - you've spent time on a patch you might not even be qualified to review.
> I suggest we tag our patches with, to start with, the components we are looking to test, and the type of test (sceanrio, api, ...) and that reviewers should -1 untagged patches.
>
> I think the tagging should be the 2nd line in the message:
>
> ======================================
> Example commit message
>
> [Neutron][Nova][Network][Scenario]
>
> Explanation of how this scenario tests both Neutron and Nova
> Network performance
>
> Chang-id XXXXXXXXXXXXXXXXXXX
> =======================================
>
> I would like this to start immediately but what do you guys think?
>
> _______________________________________________
> OpenStack-dev mailing list
> OpenStack-dev at lists.openstack.org
> http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev
>
-2
I think this is just extra clutter, please don't.
Also, it's Holiday season so tons of people are out, policy changes are
completely on hold until January.
The commit message should be meaningful so I can read it, a bunch of
tags I find just ugly and don't want to go near. We already have this
information in the directory structure for API tests. And in service
tags for the scenario tests.
2 & 3 you can through gerrit API queries. Replicating that information
in another place is just error prone.
-Sean
--
Sean Dague
Samsung Research America
sean at dague.net / sean.dague at samsung.com
http://dague.net
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 482 bytes
Desc: OpenPGP digital signature
URL: <http://lists.openstack.org/pipermail/openstack-dev/attachments/20131224/42a09000/attachment.pgp>
More information about the OpenStack-dev
mailing list