From 4dc7181e433296d2ad7b69e0cf4be2c0f14ff4c5 Mon Sep 17 00:00:00 2001 From: Cédric Ollivier Date: Wed, 14 Feb 2018 21:59:04 +0100 Subject: Compare strings instead of objects in SecurityGroupConfig MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit It could have worked thanks to interned strings. Change-Id: If56cc44fe35ca06bc9033af620ddd3c0586877fe Signed-off-by: Cédric Ollivier --- snaps/config/security_group.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/snaps/config/security_group.py b/snaps/config/security_group.py index 32a1e95..16e68c7 100644 --- a/snaps/config/security_group.py +++ b/snaps/config/security_group.py @@ -54,7 +54,7 @@ class SecurityGroupConfig(object): raise SecurityGroupConfigError('The attribute name is required') for rule_setting in self.rule_settings: - if rule_setting.sec_grp_name is not self.name: + if rule_setting.sec_grp_name != self.name: raise SecurityGroupConfigError( 'Rule settings must correspond with the name of this ' 'security group') -- cgit 1.2.3-korg