Skip to content
This repository was archived by the owner on Dec 13, 2018. It is now read-only.

Commit 5d08f6b

Browse files
author
Feng Honglin
authored
Merge pull request #89 from docker/staging
CLOUD-1189 fix the exception on user reload, fix #50
2 parents 47fbc7b + c77417c commit 5d08f6b

File tree

2 files changed

+8
-8
lines changed

2 files changed

+8
-8
lines changed

haproxy/eventhandler.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -46,7 +46,7 @@ def on_websocket_close():
4646
logger.info("Websocket close")
4747

4848

49-
def on_user_reload(signum, frame):
49+
def on_user_reload():
5050
Haproxy.cls_cfg = None
5151
if config.LINK_MODE == "legacy":
5252
logger.info("User reload is not supported in legacy link mode")

haproxy/main.py

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,11 @@
1+
from gevent import monkey
2+
monkey.patch_all()
3+
14
import logging
25
import os
3-
import signal
46
import sys
7+
import signal
8+
import gevent
59

610
import dockercloud
711
from compose.cli.docker_client import docker_client
@@ -13,10 +17,6 @@
1317
from haproxycfg import run_haproxy
1418
from utils import save_to_file
1519

16-
17-
from gevent import monkey
18-
monkey.patch_all()
19-
2020
dockercloud.user_agent = "dockercloud-haproxy/%s" % __version__
2121

2222
logger = logging.getLogger("haproxy")
@@ -35,8 +35,8 @@ def main():
3535
logging.getLogger("python-dockercloud").setLevel(logging.DEBUG)
3636

3737
config.LINK_MODE = check_link_mode(HAPROXY_CONTAINER_URI, HAPROXY_SERVICE_URI, API_AUTH)
38-
signal.signal(signal.SIGUSR1, on_user_reload)
39-
signal.signal(signal.SIGTERM, sys.exit)
38+
gevent.signal(signal.SIGUSR1, on_user_reload)
39+
gevent.signal(signal.SIGTERM, sys.exit)
4040

4141
pid = create_pid_file()
4242
logger.info("dockercloud/haproxy PID: %s" % pid)

0 commit comments

Comments
 (0)