On 04/23/2018 02:40 AM, Pierre-Yves Chibon wrote: > Good Morning Everyone, > > I re-wrote last week the script fixing the git hooks in dist-git, but its work > is getting overridden by the old script running in a cron (that Kevin fixed this > week-end). > > So I would like to replace the cron running the old script with a cron running > the new script: > > Here is the diff: > > diff --git a/ roles/gitolite/check_fedmsg_hooks/tasks/main.yml b/ roles/gitolite/check_fedmsg_hooks/tasks/main.yml > index d357a7d83..bea193af1 100644 > --- a/ roles/gitolite/check_fedmsg_hooks/tasks/main.yml > +++ b/ roles/gitolite/check_fedmsg_hooks/tasks/main.yml > @@ -8,7 +8,7 @@ > minute=10 > hour="0,12" > user=pagure > - job="/usr/local/bin/git-check-perms /srv/git/repositories --check=fedmsg-hook -f" > + job="/usr/local/bin/distgit_check_hook.py" > tags: > - git > - gitolite > > > Note, we could make this run more often as it runs in a few minutes (the forks > being the slowest namespace, the other namespace run under a minute). > > Thoughts? +1 to this, but also as we discussed on irc we should get pagure to make the repos the way we want them to start with. ;) But that can come in with the next pagure version and this fix will make the script much nicer. Perhaps we should move it to every 15min until we get the pagure change in and then back it off to 1 or 2 times a day. kevin
Attachment:
signature.asc
Description: OpenPGP digital signature
_______________________________________________ infrastructure mailing list -- infrastructure@xxxxxxxxxxxxxxxxxxxxxxx To unsubscribe send an email to infrastructure-leave@xxxxxxxxxxxxxxxxxxxxxxx
