From 2acfbf016d8626df445839caf522debc393f0d31 Mon Sep 17 00:00:00 2001 From: Patrik Lundin Date: Thu, 11 Dec 2014 23:01:23 +0100 Subject: [PATCH] Handle string returned by 'default' correctly. We need to handle the string returned by 'default' in the same way we handle the string returned by 'status' since the resulting flags are compared later. --- system/service.py | 9 ++++++++- 1 file changed, 8 insertions(+), 1 deletion(-) diff --git a/system/service.py b/system/service.py index 2d62edc67ac..c9ce55d1a37 100644 --- a/system/service.py +++ b/system/service.py @@ -996,7 +996,14 @@ class OpenBsdService(Service): if stderr: self.module.fail_json(msg=stderr) - default_flags = stdout.rstrip() + default_string = stdout.rstrip() + + # Depending on the service the string returned from 'default' may be + # either a set of flags or the boolean YES/NO + if default_string == "YES" or default_string == "NO": + default_flags = '' + else: + default_flags = default_string rc, stdout, stderr = self.execute_command("%s %s %s" % (self.enable_cmd, 'status', self.name))