stackforge/libra
Code Issues Proposed changes

Worker now supports multiple servers specified in config file.

Browse Source
This commit is contained in:
David Shrewsbury
2012-09-25 16:05:36 -04:00
parent a4faba5b7f
commit 4e92a9128b
1 changed files with 11 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

14
libra/worker/worker.py
View File

@@ -163,6 +163,17 @@ def main():
logger = setup_logging('libra_worker', args) logger = setup_logging('libra_worker', args)
if not args.server:
# NOTE(shrews): Can't set a default in argparse method because the
# value is appended to the specified default.
args.server.append('localhost:4730')
elif not isinstance(args.server, list):
# NOTE(shrews): The Options object cannot intelligently handle
# creating a list from an option that may have multiple values.
# We convert it to the expected type here.
svr_list = args.server.split()
args.server = svr_list
# Import the device driver we are going to use. This will be sent # Import the device driver we are going to use. This will be sent
# along to the Gearman task that will use it to communicate with # along to the Gearman task that will use it to communicate with
# the device. # the device.
@@ -171,9 +182,6 @@ def main():
driver_class = import_class(known_drivers[args.driver]) driver_class = import_class(known_drivers[args.driver])
driver = driver_class() driver = driver_class()
if not args.server:
args.server.append('localhost:4730')
logger.debug("Job server list: %s" % args.server) logger.debug("Job server list: %s" % args.server)
server = Server(args.server, args.reconnect_sleep) server = Server(args.server, args.reconnect_sleep)
server.logger = logger server.logger = logger