From 571be25290f1d38a56fcc2f3a955e84d5b70228a Mon Sep 17 00:00:00 2001 From: David Moreau Simard Date: Mon, 19 Feb 2018 23:23:39 -0500 Subject: [PATCH] Return a proper boolean True instead of a 'True' string The type here is important, returning anything other than a boolean has unintended consequences such as breaking Ansible callbacks because they are expecting the "changed" field to be nothing other than a bool by contract. Change-Id: Ia9b4a749f64adae8969fd9ad66304d013013e056 --- roles/test-matrix/library/test_matrix.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/roles/test-matrix/library/test_matrix.py b/roles/test-matrix/library/test_matrix.py index db2c23e8..03c0bfad 100755 --- a/roles/test-matrix/library/test_matrix.py +++ b/roles/test-matrix/library/test_matrix.py @@ -172,7 +172,7 @@ def main(): LOG.debug("Services: %s " % services) if opts.ansible: - ansible_module.exit_json(changed='True', services=services) + ansible_module.exit_json(changed=True, services=services) else: if opts.mode == "services": print(",".join(services))