Restructuring hooks in dist-git

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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

Attachment: signature.asc
Description: PGP signature

_______________________________________________
infrastructure mailing list -- infrastructure@xxxxxxxxxxxxxxxxxxxxxxx
To unsubscribe send an email to infrastructure-leave@xxxxxxxxxxxxxxxxxxxxxxx

[Index of Archives]     [Fedora Development]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite News]     [KDE Users]

  Powered by Linux