From 994c97e0bfbc1ac693e54ff14d4121f68c620b0d Mon Sep 17 00:00:00 2001
From: Sagi Shnaidman <sshnaidm@redhat.com>
Date: Sun, 22 Mar 2020 16:03:37 +0200
Subject: [PATCH] Run fetch-subunit-output role conditionally

Jobs that inherit from base job don't always run unit tests or
use subunit. This post playbook will fail for them, so make subinit
role optional and set by default for true.
For example disable it for ansible collection functional jobs:
https://review.opendev.org/#/c/711471/
Change-Id: I50b1020c896d7d8a7d58e62de778bb8c2b6e970e
---
 playbooks/devstack/post.yaml | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/playbooks/devstack/post.yaml b/playbooks/devstack/post.yaml
index 0b18f2357..14b50edab 100644
--- a/playbooks/devstack/post.yaml
+++ b/playbooks/devstack/post.yaml
@@ -1,5 +1,6 @@
 - hosts: all
   roles:
     - fetch-tox-output
-    - fetch-subunit-output
+    - role: fetch-subunit-output
+      when: fetch_subunit|default(true)|bool
     - process-stackviz