From 0847977d1216cbe2b09a22900210343812591d6c Mon Sep 17 00:00:00 2001 From: Michael <35783820+mib1185@users.noreply.github.com> Date: Tue, 9 Jan 2024 23:07:58 +0100 Subject: [PATCH] Warn only when zones were ignored in firewalld_info (#504) * warn only when zones were ignored * add changelog 504-firewalld_info-warning --- changelogs/fragments/504-firewalld_info-warning.yaml | 2 ++ plugins/modules/firewalld_info.py | 5 +++-- 2 files changed, 5 insertions(+), 2 deletions(-) create mode 100644 changelogs/fragments/504-firewalld_info-warning.yaml diff --git a/changelogs/fragments/504-firewalld_info-warning.yaml b/changelogs/fragments/504-firewalld_info-warning.yaml new file mode 100644 index 0000000..73e00aa --- /dev/null +++ b/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. diff --git a/plugins/modules/firewalld_info.py b/plugins/modules/firewalld_info.py index 8b5c80c..c83437d 100644 --- a/plugins/modules/firewalld_info.py +++ b/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( - 'Please note: zone:(%s) have been ignored in the gathering process.' % ','.join(ignore_zones)) + if 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)