[PATCH 4/5] gen_hosts: blktests: fix hosts names generation

[Daniel Gomez <da.gomez@kernel.org>]

From: Daniel Gomez <da.gomez@samsung.com>

The user of vars in set_fact tasks makes blktests_enabled_test_type to
expand to a list of chars instead of a list of strings, resulting in a
wrong hosts file generation with every char of a blktests enabled
section (BLKTESTS_SECTION_) a different host.

Pipe the variable generation into a block without using intermediate
variables that generates a list of strings.

Before:
[all]
dgc-[
dgc-'
dgc-b
dgc-l
dgc-o
dgc-c
dgc-k
dgc-'
dgc-,
dgc-
dgc-'
dgc-l
dgc-o
dgc-o
dgc-p
dgc-'
dgc-,
dgc-
dgc-'
dgc-n
dgc-b
dgc-d
dgc-'
{..}

After:
[all]
dgc-block
dgc-loop
{..}

Signed-off-by: Daniel Gomez <da.gomez@samsung.com>
---
 playbooks/roles/gen_hosts/tasks/main.yml | 13 +++++++------
 1 file changed, 7 insertions(+), 6 deletions(-)

diff --git a/playbooks/roles/gen_hosts/tasks/main.yml b/playbooks/roles/gen_hosts/tasks/main.yml
index a50355f721606a7da961b54ec29b16875391eca9..aeec3e934c19ab0f16a21bea76956e4c9e937985 100644
--- a/playbooks/roles/gen_hosts/tasks/main.yml
+++ b/playbooks/roles/gen_hosts/tasks/main.yml
@@ -203,13 +203,14 @@
     - ansible_hosts_template.stat.exists
 
 - name: Infer enabled blktests test section types
-  vars:
-    kdevops_config_data: "{{ lookup('file', topdir_path + '/.config') }}"
-    config_val: "CONFIG_BLKTESTS_SECTION_"
-    config_block_test_types: "{{ kdevops_config_data | regex_findall('^' + config_val + '(.*)=y$', multiline=True) }}"
-    config_block_test_type_names: "{{ config_block_test_types | lower }}"
   set_fact:
-    blktests_enabled_test_types: "{{ config_block_test_type_names }}"
+    blktests_enabled_test_types: >-
+      {{
+        lookup('file', topdir_path + '/.config')
+        | regex_findall('^CONFIG_BLKTESTS_SECTION_(.*)=y$', multiline=True)
+        | map('lower')
+        | list
+      }}
   when:
     - kdevops_workflows_dedicated_workflow
     - kdevops_workflow_enable_blktests

-- 
2.49.0