From 6535d6eafa651a526451a3bd31215be8c44115a7 Mon Sep 17 00:00:00 2001
From: "Jesse Pretorius (odyssey4me)" <jesse@odyssey4.me>
Date: Wed, 25 Mar 2020 20:06:51 +0000
Subject: [PATCH] [tests] Improve devstack/post playbook efficiency

By adjusting the syntax to the newer format, the fetch-subunit
role will be skipped entirely, rather than run with all its
tasks skipped. This gives us results sooner and burns less
electricity, making the world a better place. :)

Change-Id: I6ae5a63051d512eecb2a603af9d74ec139eb0877
---
 playbooks/devstack/post.yaml | 11 +++++++----
 1 file changed, 7 insertions(+), 4 deletions(-)

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