Merge 79335896de into 0847977d12

* warn only when zones were ignored

* add changelog 504-firewalld_info-warning

changelogs/fragments/504-firewalld_info-warning.yaml

@@ -0,0 +1,2 @@
+minor_changes:
+  - firewalld_info - Only warn about ignored zones, when there are zones ignored.

plugins/callback/profile_roles.py

@@ -26,7 +26,7 @@ DOCUMENTATION = '''
         ini:
           - section: callback_profile_roles
             key: summary_only
-        version_added: '1.5.5'
+        version_added: 1.5.0
 '''
 
 import collections

plugins/callback/profile_tasks.py

@@ -51,7 +51,7 @@ DOCUMENTATION = '''
         ini:
           - section: callback_profile_tasks
             key: summary_only
-        version_added: '1.5.5'
+        version_added: 1.5.0
 '''
 
 EXAMPLES = '''

plugins/modules/firewalld_info.py

@@ -356,8 +356,9 @@ def main():
             specified_zones = module.params['zones']
             collect_zones = list(set(specified_zones) & set(all_zones))
             ignore_zones = list(set(specified_zones) - set(collect_zones))
-            warn.append(
+            if ignore_zones:
-                'Please note: zone:(%s) have been ignored in the gathering process.' % ','.join(ignore_zones))
+                warn.append(
+                    'Please note: zone:(%s) have been ignored in the gathering process.' % ','.join(ignore_zones))
         else:
             collect_zones = get_all_zones(client)
 

plugins/modules/mount.py

@@ -831,7 +831,7 @@ def main():
         # handle mount on boot.  To avoid mount option conflicts, if 'noauto'
         # specified in 'opts',  mount module will ignore 'boot'.
         opts = args['opts'].split(',')
-        if 'noauto' in opts:
+        if module.params['boot'] and 'noauto' in opts:
             args['warnings'].append("Ignore the 'boot' due to 'opts' contains 'noauto'.")
         elif not module.params['boot']:
             args['boot'] = 'no'