Browse Source

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
changes/29/546029/1
David Moreau Simard 3 years ago
parent
commit
571be25290
No known key found for this signature in database GPG Key ID: 33A07694CBB71ECC
1 changed files with 1 additions and 1 deletions
  1. +1
    -1
      roles/test-matrix/library/test_matrix.py

+ 1
- 1
roles/test-matrix/library/test_matrix.py View File

@ -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))


Loading…
Cancel
Save