ci: use vars directly instead of thru env

test/automated/ansible/install-requirements.yml

@@ -31,7 +31,7 @@
 
 - hosts: testing_hosts
   become: true
-  gather_facts: no
+  gather_facts: yes
   tasks:
     # Crowdstrike reqs
     - name: install cloudstrike-falconpy (non-windows)
@@ -45,16 +45,19 @@
       become_method: runas
       when: ansible_facts['os_family'] == "Windows"
 
-- hosts: testing_hosts_linux
-  become: true
+- hosts: testing_hosts
+  become: false
   gather_facts: yes
   vars:
-    falcon_client_id: "{{ lookup('ansible.builtin.env', 'crowdstrike_client_id') | mandatory }}"
-    falcon_client_secret: "{{ lookup('ansible.builtin.env', 'crowdstrike_client_secret') | mandatory }}"
+    falcon_client_id: "{{ crowdstrike_client_id }}"
+    falcon_client_secret: "{{ crowdstrike_client_secret }}"
   roles:
     - role: crowdstrike.falcon.falcon_install
     - role: crowdstrike.falcon.falcon_configure
 
+- hosts: testing_hosts_linux
+  become: true
+  gather_facts: yes
   tasks:
     - name: Configure logrotate
       include_role:
@@ -67,13 +70,6 @@
 - hosts: testing_hosts_macos
   become: false
   gather_facts: yes
-  vars:
-    falcon_client_id: "{{ lookup('ansible.builtin.env', 'crowdstrike_client_id') | mandatory }}"
-    falcon_client_secret: "{{ lookup('ansible.builtin.env', 'crowdstrike_client_secret') | mandatory }}"
-  roles:
-    - role: crowdstrike.falcon.falcon_install
-    - role: crowdstrike.falcon.falcon_configure
-
   tasks:
     - name: install brew
       include_role:
@@ -88,13 +84,6 @@
 
 - hosts: testing_hosts_windows
   gather_facts: yes
-  vars:
-    falcon_client_id: "{{ lookup('ansible.builtin.env', 'crowdstrike_client_id') | mandatory }}"
-    falcon_client_secret: "{{ lookup('ansible.builtin.env', 'crowdstrike_client_secret') | mandatory }}"
-  roles:
-    - role: crowdstrike.falcon.falcon_install
-    - role: crowdstrike.falcon.falcon_configure
-
   tasks:
     - name: Configure hostname
       include_role: