From 9fd70e3f3cc593c97d5be8305f6a7cb6d106f2d9 Mon Sep 17 00:00:00 2001 From: Dag Wieers Date: Wed, 25 Sep 2019 18:25:07 +0200 Subject: [PATCH] Fix whitespace issue in filter (backport 2.9) This fixes reported issue #60276 This is a backport of #62768 --- changelogs/fragments/aci-42-filter-whitespace.yaml | 2 ++ lib/ansible/module_utils/network/aci/aci.py | 4 ++-- 2 files changed, 4 insertions(+), 2 deletions(-) create mode 100644 changelogs/fragments/aci-42-filter-whitespace.yaml diff --git a/changelogs/fragments/aci-42-filter-whitespace.yaml b/changelogs/fragments/aci-42-filter-whitespace.yaml new file mode 100644 index 00000000000..389fbc9e2ef --- /dev/null +++ b/changelogs/fragments/aci-42-filter-whitespace.yaml @@ -0,0 +1,2 @@ +bugfixes: +- "ACI modules - Fix a whitespace issue in filters for ACI 4.2 strict validation" diff --git a/lib/ansible/module_utils/network/aci/aci.py b/lib/ansible/module_utils/network/aci/aci.py index 9c1adf5f87c..1a4bd234478 100644 --- a/lib/ansible/module_utils/network/aci/aci.py +++ b/lib/ansible/module_utils/network/aci/aci.py @@ -409,9 +409,9 @@ class ACIModule(object): ''' Build an APIC filter based on obj_class and key-value pairs ''' accepted_params = dict((k, v) for (k, v) in params.items() if v is not None) if len(accepted_params) == 1: - return ','.join('eq({0}.{1}, "{2}")'.format(obj_class, k, v) for (k, v) in accepted_params.items()) + return ','.join('eq({0}.{1},"{2}")'.format(obj_class, k, v) for (k, v) in accepted_params.items()) elif len(accepted_params) > 1: - return 'and(' + ','.join(['eq({0}.{1}, "{2}")'.format(obj_class, k, v) for (k, v) in accepted_params.items()]) + ')' + return 'and(' + ','.join(['eq({0}.{1},"{2}")'.format(obj_class, k, v) for (k, v) in accepted_params.items()]) + ')' def _deep_url_path_builder(self, obj): target_class = obj['target_class']