Merge branch '2-univention_config_registry-doesn-t-properly-fail' into 'master'

Resolve "univention_config_registry doesn't properly fail"

Closes #2

See merge request univention/internal/ansible-modules!2
This commit is contained in:
Lukas Zumvorde 2021-02-11 14:22:10 +00:00
commit 976df5d659

View File

@ -123,7 +123,7 @@ def _set_keys(keys, result, module):
result['rc'] = rc
result['message'] = "These keys were set: {}".format(" ".join(to_set))
result['meta']['changed_keys'] = to_set
result['failed'] = rc != 0
result['failed'] = rc != 0 or len(err) > 0
if rc != 0:
module.fail_json(msg='non-zero return code', **result)
@ -160,7 +160,7 @@ def _unset_keys(keys, result, module):
result['rc'] = rc
result['message'] = "These keys were unset: {}".format(" ".join(to_unset))
result['meta']['changed_keys'] = to_unset
result['failed'] = rc != 0
result['failed'] = rc != 0
if rc != 0:
module.fail_json(msg='non-zero return code', **result)