Merge PR #351 by @Nemtal to 2.5 as well

Author: BrutalBirdie

Diff for: tasks/bbb_secret.yml

@@ -1,16 +1,27 @@
 ---
-- name: Set bbb_secret as fact
+- name: Set bbb_secret as fact if bbb-conf is present
   when: bbb_secret is not defined
   block:
-    - name: Register bbb secret
+    - name: Check if bbb-conf is present
       become: true
-      command: bbb-conf --secret
+      command: which bbb-conf
       changed_when: false
+      failed_when: which_bbb_conf.rc not in [0, 1]
       check_mode: false
-      register: result
+      register: which_bbb_conf
 
-    - name: Parse bbb secret
-      set_fact:
-        bbb_secret: "{{ result.stdout | regex_search('Secret: ([a-zA-Z0-9]*)', multiline=True) | regex_replace('Secret: ') }}"
-        cacheable: true
-      when: result.rc == 0
+    - name: Set bbb_secret as fact
+      when: which_bbb_conf.rc == 0
+      block:
+        - name: Register bbb secret
+          become: true
+          command: bbb-conf --secret
+          changed_when: false
+          check_mode: false
+          register: result
+
+        - name: Parse bbb secret
+          set_fact:
+            bbb_secret: "{{ result.stdout | regex_search('Secret: ([a-zA-Z0-9]*)', multiline=True) | regex_replace('Secret: ') }}"
+            cacheable: true
+          when: result.rc == 0