+1 On 18 April 2018 at 13:21, Kevin Fenzi <[email protected]> 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 -- [email protected] > To unsubscribe send an email to [email protected] >
-- Stephen J Smoogen. _______________________________________________ infrastructure mailing list -- [email protected] To unsubscribe send an email to [email protected]
