FBR: 🎶if you liked it then you should've put a lock on it...🎶

Mon, 25 Sep 2017 13:04:41 -0700 

... or even if you didn't like it, it still probably needs a lock.

/usr/local/bin/pagure-sync-bugzilla.py takes more than 24 hours to
run, and it doesn't have a lock wrapper on it.  It needs one.

    diff --git a/roles/distgit/pagure/tasks/main.yml 
b/roles/distgit/pagure/tasks/main.yml
    index 8cdf0b3..9079f06 100644
    --- a/roles/distgit/pagure/tasks/main.yml
    +++ b/roles/distgit/pagure/tasks/main.yml
    @@ -228,7 +228,7 @@
         user: root
         minute: 0
         hour: 18
    -    job: /usr/local/bin/pagure-sync-bugzilla.py
    +    job: /usr/bin/lock-wrapper pagure-sync-bugzilla 
"/usr/local/bin/pagure-sync-bugzilla.py"
         cron_file: pagure-sync-bugzilla
         state: present
       when: env != 'staging'

Anyone for it?  Against it?

Attachment: signature.asc
Description: PGP signature

_______________________________________________
infrastructure mailing list -- infrastructure@lists.fedoraproject.org
To unsubscribe send an email to infrastructure-le...@lists.fedoraproject.org

Reply via email to