diff options
| author | Dag Wieers <dag@wieers.com> | 2019-09-25 18:25:07 +0200 |
|---|---|---|
| committer | Toshio Kuratomi <a.badger@gmail.com> | 2019-10-12 09:17:09 -0700 |
| commit | 076988e8a6aeb02208dfca55872e6c98cc613fa8 (patch) | |
| tree | bab3ba4db03e3350d8bee59c7c5c52db7071b8b1 | |
| parent | 36a96a628a53c143ff13cd9a9eb4ac90c600c9ce (diff) | |
| download | ansible-076988e8a6aeb02208dfca55872e6c98cc613fa8.tar.gz | |
Fix whitespace issue in filter (backport 2.9)
This fixes reported issue #60276
This is a backport of #62768
| -rw-r--r-- | changelogs/fragments/aci-42-filter-whitespace.yaml | 2 | ||||
| -rw-r--r-- | lib/ansible/module_utils/network/aci/aci.py | 4 |
2 files changed, 4 insertions, 2 deletions
diff --git a/changelogs/fragments/aci-42-filter-whitespace.yaml b/changelogs/fragments/aci-42-filter-whitespace.yaml new file mode 100644 index 0000000000..389fbc9e2e --- /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 9c1adf5f87..1a4bd23447 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'] |