[multiple] Update no_log to use cifmw_nolog variable

This change enhances the debugging experience and ensures conistent
usage of the no_log variable throughout the CI Framework repository.

Signed-off-by: Roberto Alfieri <ralfieri@redhat.com>

Author: rebtoor

ci/playbooks/collect-logs.yml

@@ -177,6 +177,6 @@
       become: true
       ansible.builtin.shell: |
           journalctl -u kubelet > kubelet.log
-      no_log: true
+      no_log: "{{ cifmw_nolog | default(true) | bool }}"
       args:
         chdir: "{{ ansible_user_dir }}/zuul-output/logs/"

ci/playbooks/e2e-collect-logs.yml

@@ -47,6 +47,6 @@
       become: true
       ansible.builtin.shell: |
         journalctl -u kubelet > kubelet.log
-      no_log: true
+      no_log: "{{ cifmw_nolog | default(true) | bool }}"
       args:
         chdir: "{{ ansible_user_dir }}/zuul-output/logs/"

hooks/playbooks/ceph.yml

@@ -47,7 +47,7 @@
     - name: Create private key if it does not exist
       ansible.builtin.command:
         cmd: "ssh-keygen -t rsa -q -N '' -f {{ private_key }}"
-      no_log: true
+      no_log: "{{ cifmw_nolog | default(true) | bool }}"
       when:
         - not private_key_stat.stat.exists
 
@@ -83,7 +83,7 @@
         src: "{{ lookup('env', 'HOME') }}/.ssh/{{ cifmw_admin_user }}-id_rsa"
       register: private_key_get
       delegate_to: localhost
-      no_log: true
+      no_log: "{{ cifmw_nolog | default(true) | bool }}"
 
     - name: Get local public key
       ansible.builtin.slurp:
@@ -96,7 +96,7 @@
       cifmw_admin_pubkey: "{{ public_key_get['content'] | b64decode }}"
       cifmw_admin_prikey: "{{ private_key_get['content'] | b64decode }}"
       cifmw_admin_distribute_private_key: true
-      no_log: true
+      no_log: "{{ cifmw_nolog | default(true) | bool }}"
 
 - name: Create Block Device on target nodes
   tags: block
@@ -332,7 +332,7 @@
     - name: Generate a cephx key
       cephx_key:
       register: cephx
-      no_log: true
+      no_log: "{{ cifmw_nolog | default(true) | bool }}"
 
     - name: Set cifmw_cephadm_keys with the cephx key and cifmw_cephadm_pools
       ansible.builtin.set_fact:
@@ -347,7 +347,7 @@
                                    'profile rbd pool=\\1') | join(', ') }}"
       vars:
         pools: "{{ cifmw_cephadm_pools | map(attribute='name') | list }}"
-      no_log: true
+      no_log: "{{ cifmw_nolog | default(true) | bool }}"
 
     # for deploying external ceph for 17.1 using cifmw, we need this playbook to create keyring
     # for manila client and manila_data pool
@@ -370,7 +370,7 @@
           pg_autoscale_mode: true
           application: cephfs
       when: adoption_deploy_ceph_for_tripleo | default (false)
-      no_log: true
+      no_log: "{{ cifmw_nolog | default(true) | bool }}"
 
     # public network always exist because is provided by the ceph_spec role
     - name: Get Storage network range

roles/adoption_osp_deploy/tasks/login_registries.yml

@@ -21,7 +21,7 @@
   block:
     - name: Make redhat subscription
       become: true
-      no_log: true
+      no_log: "{{ cifmw_nolog | default(true) | bool }}"
       community.general.redhat_subscription:
         activationkey: "{{ cifmw_adoption_osp_deploy_rhsm_key }}"
         org_id: "{{ cifmw_adoption_osp_deploy_rhsm_org }}"
@@ -58,7 +58,7 @@
 
     - name: Login to container registry both zuul and root users
       become: "{{ item == 'root' }}"
-      no_log: true
+      no_log: "{{ cifmw_nolog | default(true) | bool }}"
       ansible.builtin.command: >
         podman login
         --username "{{ _container_user }}"

roles/ci_dcn_site/tasks/remove_node.yml

@@ -228,4 +228,4 @@
         content: "{{ item.content | b64decode | from_yaml | ansible.utils.remove_keys(target=[_node_to_remove]) | to_nice_yaml }}"
         mode: '0644'
       with_items: "{{ vars_files.results }}"
-      no_log: true
+      no_log: "{{ cifmw_nolog | default(true) | bool }}"

roles/ci_dcn_site/tasks/scaledown_site.yml

@@ -192,7 +192,7 @@
     cinder_prefix: "{{ (item.metadata.name | regex_search('^(cinder-[a-z0-9]+)')) }}"
   loop: "{{ pod_list.resources }}"
   when: item.metadata.name is match('^cinder-.*-scheduler-0$')
-  no_log: true
+  no_log: "{{ cifmw_nolog | default(true) | bool }}"
 
 - name: Delete the cinder-volume service
   kubernetes.core.k8s_exec:

roles/ci_setup/tasks/packages.yml

@@ -73,7 +73,7 @@
           {{ cifmw_ci_setup_oc_install_path }}/oc completion bash |
           tee -a ~/.oc_completion
         creates: "{{ ansible_user_dir }}/.oc_completion"
-      no_log: true
+      no_log: "{{ cifmw_nolog | default(true) | bool }}"
 
     - name: Source completion from within .bashrc
       ansible.builtin.blockinfile:

roles/cifmw_ceph_client/tasks/main.yml

@@ -60,15 +60,15 @@
     src: "{{ cifmw_ceph_client_fetch_dir }}/{{ cifmw_ceph_client_cluster | default('ceph') }}.conf"
   register: cifmw_ceph_client_conf_file_b64
   delegate_to: localhost
-  no_log: true
+  no_log: "{{ cifmw_nolog | default(true) | bool }}"
 
 - name: Base64 encode cephx key file
   tags: k8s
   ansible.builtin.slurp:
     src: "{{ cifmw_ceph_client_fetch_dir }}/{{ cifmw_ceph_client_cluster | default('ceph') }}.client.openstack.keyring"
   register: cifmw_ceph_client_key_file_b64
   delegate_to: localhost
-  no_log: true
+  no_log: "{{ cifmw_nolog | default(true) | bool }}"
 
 - name: Create k8s_ceph_secret.yml with base64 encoded files
   tags: k8s

roles/cifmw_external_dns/tasks/cleanup.yml

@@ -15,7 +15,7 @@
 # under the License.
 
 - name: Remove key and certificate files on target host
-  no_log: true
+  no_log: "{{ cifmw_nolog | default(true) | bool }}"
   become: true
   ansible.builtin.file:
     state: absent

roles/devscripts/tasks/320_restore_pull_secret.yml

@@ -20,13 +20,13 @@
 # pulling images from external registries for operators and other workloads.
 
 - name: Get original pull-secret content
-  no_log: true
+  no_log: "{{ cifmw_nolog | default(true) | bool }}"
   ansible.builtin.slurp:
     src: "{{ cifmw_devscripts_repo_dir }}/pull_secret.json"
   register: _original_pull_secret
 
 - name: Get current cluster pull-secret
-  no_log: true
+  no_log: "{{ cifmw_nolog | default(true) | bool }}"
   kubernetes.core.k8s_info:
     kubeconfig: "{{ cifmw_openshift_kubeconfig }}"
     api_key: "{{ cifmw_openshift_token | default(omit) }}"
@@ -37,7 +37,7 @@
   register: _cluster_pull_secret_raw
 
 - name: Update cluster pull-secret
-  no_log: true
+  no_log: "{{ cifmw_nolog | default(true) | bool }}"
   vars:
     _original_auths: "{{ (_original_pull_secret.content | b64decode | from_json).auths }}"
     _cluster_auths: "{{ (_cluster_pull_secret_raw.resources[0].data['.dockerconfigjson'] | b64decode | from_json).auths }}"