From c3eb9ec9c9247f8abf235489adda60d2314d4cad Mon Sep 17 00:00:00 2001 From: Kevin Carter Date: Thu, 10 Sep 2020 13:18:59 -0500 Subject: [PATCH] Add guard to ensure a mapping has a valid key This change will ensure our list comprehension doesn't fault when attempting to perform a "containers" lookup against a given resultant. While the result object should always have a "containers" we've seen intermitent failures in CI pointing at this being an issue. Change-Id: I8f22888ba473c78dd56988546ae402ca41c8b89f Closes-Bug: #1886636 Signed-off-by: Kevin Carter (cherry picked from commit a446dd4fcea1a6b23438956f310f7a51cd15189c) --- tripleo_ansible/ansible_plugins/action/container_status.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tripleo_ansible/ansible_plugins/action/container_status.py b/tripleo_ansible/ansible_plugins/action/container_status.py index 9f8f14da6..b30daf507 100644 --- a/tripleo_ansible/ansible_plugins/action/container_status.py +++ b/tripleo_ansible/ansible_plugins/action/container_status.py @@ -206,7 +206,7 @@ class ActionModule(ActionBase): module_args=dict(name=containers), task_vars=tvars ) - return [c for c in result["containers"]] + return [c for c in result["containers"] if "containers" in result] @tenacity.retry( reraise=True,