Freeze Break Request: Add more fedmsg endpoints for fedimg

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

 



The fedimg uploader daemon that uploads cloud images to AWS is
complaining that it doesn't have enough fedmsg endpoints.

It needs an endpoint for each worker thread, one for the parent thread, and
then twice that number so that the daemon can do work while an admin can run
the commands by hand at the same time (otherwise, the daemon claims all the
ports and the manual commands fail).

Can I get two +1s to apply this and push it out to all our hosts?


diff --git a/roles/fedmsg/base/templates/endpoints-fedimg.py.j2 b/roles/fedmsg/base/templates/endpoints-fedimg.py.j2
index b13f3a7..5a4fb9d 100644
--- a/roles/fedmsg/base/templates/endpoints-fedimg.py.j2
+++ b/roles/fedmsg/base/templates/endpoints-fedimg.py.j2
@@ -4,11 +4,15 @@ suffix  = 'stg.phx2.fedoraproject.org'
 suffix = 'phx2.fedoraproject.org'
 {% endif %}

+primary_threads = 4
+atomic_threads = 2
+NUM_FEDIMG_PORTS = 2 * ((primary_threads + atomic_threads) + 1)
+
 config = dict(
     endpoints={
         "fedimg.fedimg01": [
             "tcp://fedimg01.%s:30%0.2i" % (suffix, i)
-            for i in range(4)
+            for i in range(NUM_FEDIMG_PORTS)
         ],
     },
 )

Attachment: signature.asc
Description: PGP signature

_______________________________________________
infrastructure mailing list
infrastructure@xxxxxxxxxxxxxxxxxxxxxxx
https://admin.fedoraproject.org/mailman/listinfo/infrastructure

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

  Powered by Linux