Freeze Break Request: re-enable fedmsg hook on src

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

 



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

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

  Powered by Linux