+1

On Wed, Apr 18, 2018 at 7:21 PM, Kevin Fenzi <kevin@scrye.com> 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@lists.fedoraproject.org
To unsubscribe send an email to infrastructure-leave@lists.fedoraproject.org