Unifying (parts of) engine and vdsm commit templates.
Doron Fediuck
dfediuck at redhat.com
Mon Aug 20 10:37:10 UTC 2012
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?
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.
More information about the Arch
mailing list