It seems that check-mirroring was reporting a lot of spurious failures due to it sometimes taking more than 3 seconds for the setup phase of check-mirroring's receive path to run. So fix this: (1) Wait a bit more than 3 seconds for the receiver to subscribe to messages (2) Subscribe to Humbug messages before forking (we can't do this with zephyr.init() because python-zephyr gets totally messed up if you use it from multiple processes with a shared initialization) (3) Get rid of the old time.sleep(0.x) values that were intended to make messages arrive in order -- since we're now checking that messages correctly arrived using set(), they aren't needed. (4) Use a single request to subscribe to both zephyr classes we need to subscribe to (saves 1 RTT). (imported from commit d96aef05405ce43e9a4a549de189da9a2e393875) |
||
|---|---|---|
| .. | ||
| __init__.py | ||
| check-mirroring | ||
| feedback-bot | ||
| gcal-bot | ||
| humbug_trac.py | ||
| mit_subs_list.py | ||
| send-nagios-notification | ||
| zephyr_mirror.py | ||
| zephyr_mirror_backend.py | ||