On Fri, Jun 25, 2021 at 10:38 PM Nir Soffer <nsoffer@redhat.com> wrote:

In ovirt-imageio developer docs, we ask to install the commit-msg hook from gerrit:

wget -P .git/hooks https://gerrit.ovirt.org/tools/hooks/commit-msg

This downloads now a broken script that does not add a ChangeId header.

I found the issue, our instructions were missing: chmod +x .git/hooks/commit-msg After adding it, the new script works fine.

In the past I think it returned something like the attached file, without the code for adding signed-off-by.

I suggest to replace the current file at: https://gerrit.ovirt.org/tools/hooks/commit-msg

Or if we want to keep the default file for some reason, add: https://gerrit.ovirt.org/tools/ovirt/hooks/commit-msg

This file contains the code for adding the signed-off-by header, so so developers do not need to add this manually:

# Add Signed-off-by trailer. sob=$(git var GIT_AUTHOR_IDENT | sed -n 's/^\(.*>\).*$/Signed-off-by: \1/p') git interpret-trailers --in-place --trailer "$sob" "$1"

But developers still need to add the code above. Attaching a version with this code to replace the current file.