Good Morning Everyone, Looking into fixing the current git hook situation on dist-git I came accross the following architecture and I would like to discuss it with you.
Each repo contains:
- post-receive pointing to /usr/share/git-core/post-receive-chained
- a folder post-receive-chained.d containing:
- post-receive-alternativearch ->
/usr/share/git-core/post-receive-alternativearch
- post-receive-email ->
/usr/share/git-core/mail-hooks/gnome-post-receive-email
- post-receive-fedmsg -> /usr/share/git-core/post-receive-fedmsg
And /usr/share/git-core/post-receive-chained contains:
pee \
$GIT_DIR/hooks/post-receive-chained.d/post-receive-fedmsg \
$GIT_DIR/hooks/post-receive-chained.d/post-receive-alternativearch
So each git repo contains symlinks to the different git hooks, but there is no
need for them.
I would propose the following changes:
- Drop post-receive-chained.d entirely from each and every dist-git repo
- Replace /usr/share/git-core/post-receive-chained to be:
pee \
/usr/share/git-core/post-receive-fedmsg \
/usr/share/git-core/post-receive-alternativearch
Adjust the script in ansible in roles/git/checks/files/check-perms.py to
implement these changes.
Bonus, adding/removing hooks would then just become a matter of updating
roles/git/hooks/files/post-receive-chained in ansible.
So we could just add the pagure hooks:
/usr/lib/python2.7/site-packages/pagure/hooks/files/default_hook.py
/usr/lib/python2.7/site-packages/pagure/hooks/files/pagure_hook.py
to it to solve both the lack of logging of commits activity and to add support
for Merges/Fixes/Relates keywords in commit messages.
Finally it looks like the check-perms.py script isn't being run, that may be my
fault (I might have remove the script/cron when cleaning up after deploying
pagure).
What do you think? I know we are in freeze but this shouldn't impact directly
anything release-related. Worth trying now or should it wait some more?
Thanks,
Pierre
signature.asc
Description: PGP signature
_______________________________________________ infrastructure mailing list -- [email protected] To unsubscribe send an email to [email protected]
