Unifying (parts of) engine and vdsm commit templates.

[Eli Mesika]

----- Original Message -----
> From: "Doron Fediuck" <dfediuck at redhat.com>
> To: arch at ovirt.org
> Sent: Monday, August 20, 2012 1:37:10 PM
> Subject: Unifying (parts of) engine and vdsm commit templates.
> 
> Hi All,
> 
> Recently, we've been doing some thinking on commit message templates.
> It seems that for commit subjects, vdsm is using a general concept
> of-
> BZ#??????? some message
> 
> Whilst for the engine, we have something like:
> <core | restapi | tools | history | engine | userportal | webadmin>:
> short summary under 50 chars (#xxxxxx)
> 
> Since several people are looking at writing some scripts to extract
> BZ numbers
> from commit messages, I'd like to propose a minor change both for
> vdsm and engine,
> which should benefit both projects.
> 
> The change, originally suggested by Alon Bar-Lev, is to remove BZ
> reference from
> commit subject, and add it to the message body as "Bug-Id: BZ#??????
> description".
> 
> * For engine, we'll have-
> =====================================================================================
> <core | restapi | tools | history | engine | userportal | webadmin>:
> short summary under 50 chars
> 
> ...
>  (message body)
> ...
> 
>     Bug-Id: BZ#888888 dummy bz1
>     Bug-Id: BZ#888889 dummy bz2
>     Bug-Id: BZ#888890 dummy bz2
>     Change-Id: Ixxxxx
>     Signed-off-by: Igor Lvovsky <ilvovsky at redhat.com>
>     Reviewed-on: http://gerrit.ovirt.org/8888
>     Reviewed-by: Dan Kenigsberg <danken at redhat.com>
> 
> =====================================================================================
> 
> * and for vdsm-
> =====================================================================================
>     networking: Fix some issue
> 
> ...
>  (message body)
> ...
> 
>     Bug-Id: BZ#888888 dummy bz1
>     Bug-Id: BZ#888889 dummy bz2
>     Bug-Id: BZ#888890 dummy bz2
>     Change-Id: Ixxxxx
>     Signed-off-by: Igor Lvovsky <ilvovsky at redhat.com>
>     Reviewed-on: http://gerrit.ovirt.org/8888
>     Reviewed-by: Dan Kenigsberg <danken at redhat.com>
> 
> =====================================================================================
> 
> As you can see, with this change:
>  
> 1. We have more space for commit text;
> 
> 2. We can document easily several bugs;
> 
> 3. It is clear what bugs we solve as the description is embedded;
> 
> 4. It is easy to automate fetching bug ids from commit message;
> 
> 5. A commit msg hooks can be easily implemented to fetch the bug
> description and embed description.
> 
> 
> Any objections?
+1
> 
> Doron.
> 
> 
> P.S.
> Currently this is suggested for engine and vdsm projects.
> Other projects are invited to adopt this proposal, but it's
> completely optional.
> _______________________________________________
> Arch mailing list
> Arch at ovirt.org
> http://lists.ovirt.org/mailman/listinfo/arch
>