From 6c654b6947cd8726991612897f794bfaadb5490a Mon Sep 17 00:00:00 2001 From: Tim Rupp Date: Mon, 18 Jun 2018 16:06:13 -0700 Subject: [PATCH] Fixes the default assignment of values (#41672) Fixes: #41640 The defaults should be inherited from the parent during initial creation. This patch fixes that. --- lib/ansible/modules/network/f5/bigip_profile_client_ssl.py | 4 +--- .../units/modules/network/f5/test_bigip_profile_client_ssl.py | 2 +- 2 files changed, 2 insertions(+), 4 deletions(-) diff --git a/lib/ansible/modules/network/f5/bigip_profile_client_ssl.py b/lib/ansible/modules/network/f5/bigip_profile_client_ssl.py index e0cbdbb14b..b4c9e3f671 100644 --- a/lib/ansible/modules/network/f5/bigip_profile_client_ssl.py +++ b/lib/ansible/modules/network/f5/bigip_profile_client_ssl.py @@ -33,7 +33,7 @@ options: ciphers: description: - Specifies the list of ciphers that the system supports. When creating a new - profile, the default cipher list is C(DEFAULT). + profile, the default cipher list is provided by the parent profile. cert_key_chain: description: - One or more certificates and keys to associate with the SSL profile. This @@ -402,8 +402,6 @@ class ModuleManager(object): def create(self): self._set_changed_options() - if self.want.ciphers is None: - self.want.update({'ciphers': 'DEFAULT'}) if self.module.check_mode: return True self.create_on_device() diff --git a/test/units/modules/network/f5/test_bigip_profile_client_ssl.py b/test/units/modules/network/f5/test_bigip_profile_client_ssl.py index e2a68b3bb0..a2c2e6d370 100644 --- a/test/units/modules/network/f5/test_bigip_profile_client_ssl.py +++ b/test/units/modules/network/f5/test_bigip_profile_client_ssl.py @@ -107,7 +107,7 @@ class TestManager(unittest.TestCase): chain='bigip_ssl_cert1' ) ], - password='passsword', + password='password', server='localhost', user='admin' ))