diff --git a/quantum/agent/common/config.py b/quantum/agent/common/config.py index 691ff1ecd4..05a5f01bc9 100644 --- a/quantum/agent/common/config.py +++ b/quantum/agent/common/config.py @@ -45,16 +45,18 @@ def get_log_args(conf, log_file_name): cmd_args.append('--verbose') if (conf.log_dir or conf.log_file): cmd_args.append('--log-file=%s' % log_file_name) - log_dir = None - if conf.log_dir and conf.log_file: - log_dir = os.path.dirname( - os.path.join(conf.log_dir, conf.log_file)) - elif conf.log_dir: - log_dir = conf.log_dir - elif conf.log_file: - log_dir = os.path.dirname(conf.log_file) - if log_dir: - cmd_args.append('--log-dir=%s' % log_dir) + log_dir = None + if conf.log_dir and conf.log_file: + log_dir = os.path.dirname( + os.path.join(conf.log_dir, conf.log_file)) + elif conf.log_dir: + log_dir = conf.log_dir + elif conf.log_file: + log_dir = os.path.dirname(conf.log_file) + if log_dir: + cmd_args.append('--log-dir=%s' % log_dir) + else: + cmd_args.append('--use-syslog') return cmd_args diff --git a/quantum/tests/unit/test_dhcp_agent.py b/quantum/tests/unit/test_dhcp_agent.py index 92be06c787..cf31483584 100644 --- a/quantum/tests/unit/test_dhcp_agent.py +++ b/quantum/tests/unit/test_dhcp_agent.py @@ -362,7 +362,7 @@ class TestLogArgs(base.BaseTestCase): 'log_dir': None, 'log_file': None} conf = dhcp_agent.DictModel(conf_dict) - expected_args = ['--debug'] + expected_args = ['--debug', '--use-syslog'] args = config.get_log_args(conf, 'log_file_name') self.assertEqual(expected_args, args)