+1 On 18 April 2018 at 13:21, Kevin Fenzi <kevin@xxxxxxxxx> wrote: > 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 > > > _______________________________________________ > infrastructure mailing list -- infrastructure@xxxxxxxxxxxxxxxxxxxxxxx > To unsubscribe send an email to infrastructure-leave@xxxxxxxxxxxxxxxxxxxxxxx > -- Stephen J Smoogen. _______________________________________________ infrastructure mailing list -- infrastructure@xxxxxxxxxxxxxxxxxxxxxxx To unsubscribe send an email to infrastructure-leave@xxxxxxxxxxxxxxxxxxxxxxx
