From e96dbc48d434823affcd8e5514fe6cac2189e502 Mon Sep 17 00:00:00 2001
From: "Chandan Kumar (raukadah)" <chkumar@redhat.com>
Date: Mon, 6 Jun 2022 17:58:28 +0530
Subject: [PATCH] Run dstat on overcloud nodes

It's make sure, all the overcloud nodes are known and then
run dstat before running tempest on it.

Signed-off-by: Chandan Kumar (raukadah) <chkumar@redhat.com>
Change-Id: Ia0e5218a4ff294db27b758caaa453d47945e82ec
---
 .../baremetal-full-overcloud-validate.yml      | 18 +++++++++++++++++-
 roles/validate-perf/tasks/main.yml             |  2 +-
 2 files changed, 18 insertions(+), 2 deletions(-)

diff --git a/playbooks/baremetal-full-overcloud-validate.yml b/playbooks/baremetal-full-overcloud-validate.yml
index eba51f9f5..065320048 100644
--- a/playbooks/baremetal-full-overcloud-validate.yml
+++ b/playbooks/baremetal-full-overcloud-validate.yml
@@ -1,7 +1,23 @@
 ---
+- name: Run dump vars on all!localhost
+  hosts: all:!localhost:!127.0.0.2
+  roles:
+    - dump_vars
+
+- name: Ensure the overcloud nodes are added to the generated inventory
+  hosts: undercloud
+  gather_facts: true
+  tags:
+    - overcloud-validate
+  vars:
+    inventory: all
+  roles:
+    - tripleo-inventory
+  ignore_errors: true
+
 # Execute sanity checks agsinst the overcloud deployment
 - name: setup dstat performance monitoring
-  hosts: overcloud
+  hosts: all:!localhost:!127.0.0.2!undercloud
   tags:
     - overcloud-validate
   roles:
diff --git a/roles/validate-perf/tasks/main.yml b/roles/validate-perf/tasks/main.yml
index aafabe325..eae2794d6 100644
--- a/roles/validate-perf/tasks/main.yml
+++ b/roles/validate-perf/tasks/main.yml
@@ -20,7 +20,7 @@
         --nocolor
         --output /var/log/extra/{{ dstat_log_name }}
         {{ dstat_delay }}
-        > /dev/null &
+        >> /var/log/extra/dstat.txt &
     args:
       executable: /bin/bash
     async: 1