Remove "Read vars_file" for every playbook (#447)

Signed-off-by: Winnie81 <wyue@vmware.com>

docs/TESTCASE_DEV_GUIDE.md

@@ -19,17 +19,13 @@
 - Set the playbook 'name' to the same name as the playbook file or test case name.
 2. hosts
 - Set the playbook 'hosts' to "localhost", as tasks executed on local machine, when there are tasks need to be executed on remote VM or ESXi server, use "delegate_to" keyword on the task.
-3. vars_files
-- Set the playbook 'vars_files' to the relative path to the configuration file "vars/test.yml", and/or other required variables file.
-4. tasks
+3. tasks
 - This part contains "block" and "rescue" sections, write all the tasks of test case execution and validation in "block" part, "rescue" part includes a common task "common/test_rescue.yml". It means any task fails in "block" part, the testing will jump to the "rescue" part to execute failure/error handling tasks.
 
 Below is an example of test case playbook:
 ```
 - name: new_test_case
   hosts: localhost
-  vars_files:
-    - "{{ testing_vars_file | default('../../vars/test.yml') }}"
   tasks:
     - name: "Test case block"
       block:

env_setup/env_cleanup.yml

@@ -5,8 +5,6 @@
 - name: env_cleanup
   hosts: localhost
   gather_facts: false
-  vars_files:
-    - "{{ testing_vars_file | default('../vars/test.yml') }}"
   tasks:
     - name: "Set cleanup VM to False by default"
       ansible.builtin.set_fact:

env_setup/env_setup.yml

@@ -5,8 +5,6 @@
 - name: env_setup
   hosts: localhost
   gather_facts: false
-  vars_files:
-    - "{{ testing_vars_file | default('../vars/test.yml') }}"
   tasks:
     - include_tasks: get_ansible_info.yml
     - include_tasks: set_default_variables.yml

linux/check_efi_firmware/check_efi_firmware.yml

@@ -8,8 +8,6 @@
 - name: check_efi_firmware
   hosts: localhost
   gather_facts: false
-  vars_files:
-    - "{{ testing_vars_file | default('../../vars/test.yml') }}"
   tasks:
     - name: "Test case block"
       block:

linux/check_inbox_driver/check_inbox_driver.yml

@@ -9,8 +9,6 @@
 - name: check_inbox_driver
   hosts: localhost
   gather_facts: false
-  vars_files:
-    - "{{ testing_vars_file | default('../../vars/test.yml') }}"
   vars:
   tasks:
     - name: "Initialized inbox drivers' versions dict"

linux/check_ip_address/check_ip_address.yml

@@ -9,8 +9,6 @@
 - name: check_ip_address
   hosts: localhost
   gather_facts: false
-  vars_files:
-    - "{{ testing_vars_file | default('../../vars/test.yml') }}"
   tasks:
     - name: "Test case block"
       block:

linux/check_os_fullname/check_os_fullname.yml

@@ -9,8 +9,6 @@
 - name: check_os_fullname
   hosts: localhost
   gather_facts: false
-  vars_files:
-    - "{{ testing_vars_file | default('../../vars/test.yml') }}"
   tasks:
     - name: "Test case block"
       block:

linux/check_quiesce_snapshot_custom_script/check_quiesce_snapshot_custom_script.yml

@@ -8,8 +8,6 @@
 - name: check_quiesce_snapshot_custom_script
   hosts: localhost
   gather_facts: false
-  vars_files:
-    - "{{ testing_vars_file | default('../../vars/test.yml') }}"
   tasks:
     - name: "Test case block"
       block:

linux/cpu_hot_add_basic/cpu_hot_add_basic.yml

@@ -12,8 +12,6 @@
 - name: cpu_hot_add_basic
   hosts: localhost
   gather_facts: false
-  vars_files:
-    - "{{ testing_vars_file | default('../../vars/test.yml') }}"
   tasks:
     - name: "Test case block"
       block:

linux/cpu_multicores_per_socket/cpu_multicores_per_socket.yml

@@ -12,8 +12,6 @@
 - name: cpu_multicores_per_socket
   hosts: localhost
   gather_facts: false
-  vars_files:
-    - "{{ testing_vars_file | default('../../vars/test.yml') }}"
   tasks:
     - name: "Test case block"
       block:

linux/deploy_vm/deploy_vm.yml

@@ -6,8 +6,6 @@
   gather_facts: false
   collections:
     - community.general
-  vars_files:
-    - "{{ testing_vars_file | default('../../vars/test.yml') }}"
   tasks:
     - name: "Set current test case index, name and log folder"
       include_tasks: ../../common/set_current_testcase_facts.yml

linux/device_list/device_list.yml

@@ -13,8 +13,6 @@
 - name: device_list
   hosts: localhost
   gather_facts: false
-  vars_files:
-    - "{{ testing_vars_file | default('../../vars/test.yml') }}"
   tasks:
     - name: "Test case block"
       block:

linux/export_vm/export_vm.yml

@@ -9,8 +9,6 @@
 - name: export_ovf
   hosts: localhost
   gather_facts: false
-  vars_files:
-    - "{{ testing_vars_file | default('../../vars/test.yml') }}"
   tasks:
     # Check VM power status
     - include_tasks: ../../common/vm_get_power_state.yml

linux/guest_customization/gosc_cloudinit_dhcp.yml

@@ -9,8 +9,6 @@
 - name: gosc_cloudinit_dhcp
   hosts: localhost
   gather_facts: false
-  vars_files:
-    - "{{ testing_vars_file | default('../../vars/test.yml') }}"
   vars:
     ip_assignment: "dhcp"
     gosc_workflow: "cloud-init"

linux/guest_customization/gosc_cloudinit_staticip.yml

@@ -9,8 +9,6 @@
 - name: gosc_cloudinit_staticip
   hosts: localhost
   gather_facts: false
-  vars_files:
-    - "{{ testing_vars_file | default('../../vars/test.yml') }}"
   vars:
     ip_assignment: "static"
     gosc_workflow: "cloud-init"

linux/guest_customization/gosc_perl_dhcp.yml

@@ -9,8 +9,6 @@
 - name: gosc_perl_dhcp
   hosts: localhost
   gather_facts: false
-  vars_files:
-    - "{{ testing_vars_file | default('../../vars/test.yml') }}"
   vars:
     ip_assignment: "dhcp"
     gosc_workflow: "perl"

linux/guest_customization/gosc_perl_staticip.yml

@@ -9,8 +9,6 @@
 - name: gosc_perl_staticip
   hosts: localhost
   gather_facts: false
-  vars_files:
-    - "{{ testing_vars_file | default('../../vars/test.yml') }}"
   vars:
     ip_assignment: "static"
     gosc_workflow: "perl"

linux/memory_hot_add_basic/memory_hot_add_basic.yml

@@ -11,8 +11,6 @@
 - name: memory_hot_add_basic
   hosts: localhost
   gather_facts: false
-  vars_files:
-    - "{{ testing_vars_file | default('../../vars/test.yml') }}"
   tasks:
     - name: "Test case block"
       block:

linux/network_device_ops/e1000_network_device_ops.yml

@@ -9,8 +9,6 @@
 - name: e1000_network_device_ops
   hosts: localhost
   gather_facts: false
-  vars_files:
-    - "{{ testing_vars_file | default('../../vars/test.yml') }}"
   vars:
     adapter_type: "e1000"
   tasks:

linux/network_device_ops/e1000e_network_device_ops.yml

@@ -7,8 +7,6 @@
 - name: e1000e_network_device_ops
   hosts: localhost
   gather_facts: false
-  vars_files:
-    - "{{ testing_vars_file | default('../../vars/test.yml') }}"
   vars:
     adapter_type: "e1000e"
   tasks:

linux/network_device_ops/pvrdma_network_device_ops.yml

@@ -7,8 +7,6 @@
 - name: pvrdma_network_device_ops
   hosts: localhost
   gather_facts: false
-  vars_files:
-    - "{{ testing_vars_file | default('../../vars/test.yml') }}"
   vars:
     timestamp: "{{ lookup('pipe', 'date +%Y%m%d%H%M%S') }}"
     adapter_type: "pvrdma"

linux/network_device_ops/vmxnet3_network_device_ops.yml

@@ -7,8 +7,6 @@
 - name: vmxnet3_network_device_ops
   hosts: localhost
   gather_facts: false
-  vars_files:
-    - "{{ testing_vars_file | default('../../vars/test.yml') }}"
   vars:
     adapter_type: "vmxnet3"
   tasks:

linux/nvdimm_cold_add_remove/nvdimm_cold_add_remove.yml

@@ -9,8 +9,6 @@
 - name: nvdimm_cold_add_remove
   hosts: localhost
   gather_facts: false
-  vars_files:
-    - "{{ testing_vars_file | default('../../vars/test.yml') }}"
   tasks:
     - block:
         - include_tasks: ../setup/test_setup.yml

linux/open_vm_tools/ovt_verify_install.yml

@@ -11,8 +11,6 @@
 - name: ovt_verify_install
   hosts: localhost
   gather_facts: false
-  vars_files:
-    - "{{ testing_vars_file | default('../../vars/test.yml') }}"
   tasks:
     - name: "Test case block"
       block:

linux/open_vm_tools/ovt_verify_status.yml

@@ -8,8 +8,6 @@
 - name: ovt_verify_status
   hosts: localhost
   gather_facts: false
-  vars_files:
-    - "{{ testing_vars_file | default('../../vars/test.yml') }}"
   tasks:
     - block:
         - include_tasks: ../setup/test_setup.yml

linux/open_vm_tools/ovt_verify_uninstall.yml

@@ -9,8 +9,6 @@
 - name: ovt_verify_uninstall
   hosts: localhost
   gather_facts: false
-  vars_files:
-    - "{{ testing_vars_file | default('../../vars/test.yml') }}"
   tasks:
     - name: "Test case block"
       block:

linux/secureboot_enable_disable/secureboot_enable_disable.yml

@@ -14,8 +14,6 @@
 - name: secureboot_enable_disable
   hosts: localhost
   gather_facts: false
-  vars_files:
-    - "{{ testing_vars_file | default('../../vars/test.yml') }}"
   tasks:
     - name: "Test case block"
       block:

linux/stat_balloon/stat_balloon.yml

@@ -10,8 +10,6 @@
 - name: stat_balloon
   hosts: localhost
   gather_facts: false
-  vars_files:
-    - "{{ testing_vars_file | default('../../vars/test.yml') }}"
   tasks:
     - name: "Test case block"
       block:

linux/stat_hosttime/stat_hosttime.yml

@@ -11,8 +11,6 @@
 - name: stat_hosttime
   hosts: localhost
   gather_facts: false
-  vars_files:
-    - "{{ testing_vars_file | default('../../vars/test.yml') }}"
   tasks:
     - name: "Test case block"
       block:

linux/vgauth_check_service/vgauth_check_service.yml

@@ -9,8 +9,6 @@
 - name: vgauth_check_service
   hosts: localhost
   gather_facts: false
-  vars_files:
-    - "{{ testing_vars_file | default('../../vars/test.yml') }}"
   tasks:
     - name: "Test case block"
       block:

linux/vhba_hot_add_remove/lsilogic_vhba_device_ops.yml

@@ -14,8 +14,6 @@
 - name: lsilogic_vhba_device_ops
   hosts: localhost
   gather_facts: false
-  vars_files:
-    - "{{ testing_vars_file | default('../../vars/test.yml') }}"
   vars:
     timestamp: "{{ lookup('pipe', 'date +%Y-%m-%d-%H-%M-%S') }}"
     new_disk_ctrl_type: "lsilogic"

linux/vhba_hot_add_remove/lsilogicsas_vhba_device_ops.yml

@@ -13,8 +13,6 @@
 - name: lsilogicsas_vhba_device_ops
   hosts: localhost
   gather_facts: false
-  vars_files:
-    - "{{ testing_vars_file | default('../../vars/test.yml') }}"
   vars:
     timestamp: "{{ lookup('pipe', 'date +%Y-%m-%d-%H-%M-%S') }}"
     new_disk_ctrl_type: "lsilogicsas"

linux/vhba_hot_add_remove/nvme_vhba_device_ops.yml

@@ -14,8 +14,6 @@
 - name: nvme_vhba_device_ops
   hosts: localhost
   gather_facts: false
-  vars_files:
-    - "{{ testing_vars_file | default('../../vars/test.yml') }}"
   vars:
     timestamp: "{{ lookup('pipe', 'date +%Y-%m-%d-%H-%M-%S') }}"
     new_disk_ctrl_type: "nvme"

linux/vhba_hot_add_remove/paravirtual_vhba_device_ops.yml

@@ -14,8 +14,6 @@
 - name: paravirtual_vhba_device_ops
   hosts: localhost
   gather_facts: false
-  vars_files:
-    - "{{ testing_vars_file | default('../../vars/test.yml') }}"
   vars:
     timestamp: "{{ lookup('pipe', 'date +%Y-%m-%d-%H-%M-%S') }}"
     new_disk_ctrl_type: "paravirtual"

linux/vhba_hot_add_remove/sata_vhba_device_ops.yml

@@ -14,8 +14,6 @@
 - name: sata_vhba_device_ops
   hosts: localhost
   gather_facts: false
-  vars_files:
-    - "{{ testing_vars_file | default('../../vars/test.yml') }}"
   vars:
     timestamp: "{{ lookup('pipe', 'date +%Y-%m-%d-%H-%M-%S') }}"
     new_disk_ctrl_type: "sata"

main.yml

@@ -5,6 +5,10 @@
 - name: main
   hosts: localhost
   tasks:
+    - name: "Read variables from vars file"
+      ansible.builtin.include_vars:
+        file: "{{ testing_vars_file | default('vars/test.yml') }}"
+
     - name: "Set facts of the current main playbook directory and cache directory"
       ansible.builtin.set_fact:
         main_playbook_path: "{{ playbook_dir }}"

windows/check_efi_firmware/check_efi_firmware.yml

@@ -9,8 +9,6 @@
 - name: check_efi_firmware
   hosts: localhost
   gather_facts: false
-  vars_files:
-    - "{{ testing_vars_file | default('../../vars/test.yml') }}"
   tasks:
     - block:
         - include_tasks: ../setup/test_setup.yml

windows/check_inbox_driver/check_inbox_driver.yml

@@ -9,8 +9,6 @@
 - name: check_inbox_driver
   hosts: localhost
   gather_facts: false
-  vars_files:
-    - "{{ testing_vars_file | default('../../vars/test.yml') }}"
   tasks:
     - name: "Test case block"
       block:

windows/check_ip_address/check_ip_address.yml

@@ -9,8 +9,6 @@
 - name: check_ip_address
   hosts: localhost
   gather_facts: false
-  vars_files:
-    - "{{ testing_vars_file | default('../../vars/test.yml') }}"
   tasks:
     - block:
         - include_tasks: ../setup/test_setup.yml

windows/check_os_fullname/check_os_fullname.yml

@@ -9,8 +9,6 @@
 - name: check_os_fullname
   hosts: localhost
   gather_facts: false
-  vars_files:
-    - "{{ testing_vars_file | default('../../vars/test.yml') }}"
   tasks:
     - block:
         - include_tasks: ../setup/test_setup.yml

windows/check_quiesce_snapshot/check_quiesce_snapshot.yml

@@ -8,8 +8,6 @@
 - name: check_quiesce_snapshot
   hosts: localhost
   gather_facts: false
-  vars_files:
-    - "{{ testing_vars_file | default('../../vars/test.yml') }}"
   tasks:
     - block:
         - include_tasks: ../setup/test_setup.yml

windows/cpu_hot_add_basic/cpu_hot_add_basic.yml

@@ -11,8 +11,6 @@
 - name: cpu_hot_add_basic
   hosts: localhost
   gather_facts: false
-  vars_files:
-    - "{{ testing_vars_file | default('../../vars/test.yml') }}"
   tasks:
     - block:
         - include_tasks: ../setup/test_setup.yml

windows/cpu_multicores_per_socket/cpu_multicores_per_socket.yml

@@ -8,8 +8,6 @@
 - name: cpu_multicores_per_socket
   hosts: localhost
   gather_facts: false
-  vars_files:
-    - "{{ testing_vars_file | default('../../vars/test.yml') }}"
   tasks:
     - block:
         - name: Set fact of the initial CPU number