linux-system-roles / snapshot

Ansible role to manage storage snapshots
MIT License
2 stars 5 forks source link

fix: better error handling for all platforms and ansible versions #47

Closed richm closed 8 months ago

richm commented 8 months ago

We have to use a block/rescue/fail in order to correctly propagate errors for all combinations of ansible version and platform version. In addition, on some combinations of older ansible versions and platforms, there is a spurious locale error message written to stdout that prevents the from_json conversion. This will be better solved by converting the script to a proper Ansible module.

Signed-off-by: Rich Megginson rmeggins@redhat.com

richm commented 8 months ago

[citest]

richm commented 8 months ago

All of the failures are due to the cleanup issue which is still under investigation