Greetings. Just before freeze in ansible commit dc86b22f9054bdcefd51e46db90433c03d54410c pingou disabled the fedmsg hook because there was a permissions problem reading the fedmsg key. In ansible commit 9cbf120b54a79ecb07931f3a3a1b059525d4d766 patrick fixed this perm issue via a facl. So, I would like to re-enable the hook on normal projects and remove it from forks. (I think we decided forks should only get the pagure hook, not the main fedmsg one). So, I'd like to apply: > diff --git a/roles/git/hooks/files/post-receive-chained b/roles/git/hooks/files/post-receive-chained > index 99b5a2a..5c74da4 100755 > --- a/roles/git/hooks/files/post-receive-chained > +++ b/roles/git/hooks/files/post-receive-chained > @@ -8,7 +8,7 @@ pee \ > /usr/share/git-core/post-receive-alternativearch \ > /usr/lib/python2.7/site-packages/pagure/hooks/files/default_hook.py \ > /usr/lib/python2.7/site-packages/pagure/hooks/files/pagure_hook.py \ > - /usr/lib/python2.7/site-packages/pagure/hooks/files/fedmsg_hook.py \ > +# /usr/lib/python2.7/site-packages/pagure/hooks/files/fedmsg_hook.py \ > "/usr/bin/grok-manifest -m /srv/git/grokmirror/manifest.js.gz -t /srv/git/repositories/ -n `pwd`" > > # We used to send emails directly from the git hook here, but now we send to > diff --git a/roles/git/hooks/files/post-receive-chained-forks b/roles/git/hooks/files/post-receive-chained-forks > index 5e0056d..67565e6 100755 > --- a/roles/git/hooks/files/post-receive-chained-forks > +++ b/roles/git/hooks/files/post-receive-chained-forks > @@ -6,7 +6,7 @@ > pee \ > /usr/lib/python2.7/site-packages/pagure/hooks/files/default_hook.py \ > /usr/lib/python2.7/site-packages/pagure/hooks/files/pagure_hook.py \ > - /usr/lib/python2.7/site-packages/pagure/hooks/files/fedmsg_hook.py \ > +# /usr/lib/python2.7/site-packages/pagure/hooks/files/fedmsg_hook.py \ > "/usr/bin/grok-manifest -m /srv/git/grokmirror/manifest.js.gz -t /srv/git/repositories/ -n `pwd`" > > # We used to send emails directly from the git hook here, but now we send to Run the ansible playbook against pkgs, And then run the script to make sure all projects are set right. +1s? kevin
Attachment:
signature.asc
Description: OpenPGP digital signature
_______________________________________________ infrastructure mailing list -- infrastructure@xxxxxxxxxxxxxxxxxxxxxxx To unsubscribe send an email to infrastructure-leave@xxxxxxxxxxxxxxxxxxxxxxx