We went from `ignore_errors: true` to `failed_when: false` to prevent
Ansible printing errors which aren't really errors. When using
`succeeded` filter on a registered variable from such task, we will
always get true, which is not what we want. Check for `.rc == 0`
instead to set the facts as they were intended.
Reviewed: https:/ /review. opendev. org/675629 /git.openstack. org/cgit/ openstack/ tripleo- heat-templates/ commit/ ?id=e26009fa9c5 7d42a0c82ebc221 69e10741005946
Committed: https:/
Submitter: Zuul
Branch: master
commit e26009fa9c57d42 a0c82ebc22169e1 0741005946
Author: Jiri Stransky <email address hidden>
Date: Fri Aug 9 15:52:29 2019 +0200
Check for rc instead of |succeeded
We went from `ignore_errors: true` to `failed_when: false` to prevent
Ansible printing errors which aren't really errors. When using
`succeeded` filter on a registered variable from such task, we will
always get true, which is not what we want. Check for `.rc == 0`
instead to set the facts as they were intended.
Change-Id: Ib8dea68720fda8 6cf4fd879cdf194 d206968f6fb
Partial-Bug: #1831690