diff --git a/virtualbmc/cmd/vbmc.py b/virtualbmc/cmd/vbmc.py index 304177c..c984734 100644 --- a/virtualbmc/cmd/vbmc.py +++ b/virtualbmc/cmd/vbmc.py @@ -52,10 +52,14 @@ class ZmqClient(object): SERVER_TIMEOUT = 5000 # milliseconds + @staticmethod + def to_dict(obj): + return {attr: getattr(obj, attr) + for attr in dir(obj) if not attr.startswith('_')} + def communicate(self, command, args, no_daemon=False): - data_out = {attr: getattr(args, attr) - for attr in dir(args) if not attr.startswith('_')} + data_out = self.to_dict(args) data_out.update(command=command) diff --git a/virtualbmc/control.py b/virtualbmc/control.py index 13ca418..7f22bb0 100644 --- a/virtualbmc/control.py +++ b/virtualbmc/control.py @@ -143,14 +143,14 @@ def command_dispatcher(vbmc_manager, data_in): data_out = [vbmc_manager.delete(domain_name) for domain_name in data_in['domain_names']] return { - 'rc': max([rc for rc, msg in data_out]), + 'rc': max(rc for rc, msg in data_out), 'msg': [msg for rc, msg in data_out if msg], } elif command == 'start': data_out = [vbmc_manager.start(data_in['domain_name'])] return { - 'rc': max([rc for rc, msg in data_out]), + 'rc': max(rc for rc, msg in data_out), 'msg': [msg for rc, msg in data_out if msg], } @@ -158,7 +158,7 @@ def command_dispatcher(vbmc_manager, data_in): data_out = [vbmc_manager.stop(domain_name) for domain_name in data_in['domain_names']] return { - 'rc': max([rc for rc, msg in data_out]), + 'rc': max(rc for rc, msg in data_out), 'msg': [msg for rc, msg in data_out if msg], } diff --git a/virtualbmc/manager.py b/virtualbmc/manager.py index b65e02d..412b142 100644 --- a/virtualbmc/manager.py +++ b/virtualbmc/manager.py @@ -76,7 +76,7 @@ class VirtualBMCManager(object): config = configparser.ConfigParser() config.add_section(DEFAULT_SECTION) - for option, value in sorted(options.items()): + for option, value in options.items(): if value is not None: config.set(DEFAULT_SECTION, option, six.text_type(value)) @@ -170,7 +170,7 @@ class VirtualBMCManager(object): if not instance: instance = multiprocessing.Process( - name='xxx', + name='vbmcd-managing-domain-%s' % domain_name, target=vbmc_runner, args=(bmc_config,) )