Use "!=" instead of "is not" when comparing two values

Change-Id: I2cd9575585dde7c44b528077a7bffa16379b1759
Closes-Bug: #1283019

diff --git a/neutron/agent/common/config.py b/neutron/agent/common/config.py
index 9d5f5b1c48ef08cd2b03e864c1871adf60992644..4bf3710eeb6f5ea63e0f06cab05a6cfd92b19c61 100644 (file)
--- a/neutron/agent/common/config.py
+++ b/neutron/agent/common/config.py
@@ -78,11 +78,11 @@ def register_agent_state_opts_helper(conf):
 
 def get_root_helper(conf):
     root_helper = conf.AGENT.root_helper
-    if root_helper is not 'sudo':
+    if root_helper != 'sudo':
         return root_helper
 
     root_helper = conf.root_helper
-    if root_helper is not 'sudo':
+    if root_helper != 'sudo':
         LOG.deprecated(_('DEFAULT.root_helper is deprecated! Please move '
                          'root_helper configuration to [AGENT] section.'))
         return root_helper

diff --git a/neutron/plugins/bigswitch/plugin.py b/neutron/plugins/bigswitch/plugin.py
index d0bc7d46e4b53fcd31a06a05d660337a72de1b50..e1f9f32e9e673f2c76ff4f9935f1a96143ff7712 100644 (file)
--- a/neutron/plugins/bigswitch/plugin.py
+++ b/neutron/plugins/bigswitch/plugin.py
@@ -262,7 +262,7 @@ class NeutronRestProxyV2Base(db_base_plugin_v2.NeutronDbPluginV2,
                           "resource: %s"), resource)
 
         if 'status' in resource:
-            if resource['status'] is not const.NET_STATUS_ACTIVE:
+            if resource['status'] != const.NET_STATUS_ACTIVE:
                 LOG.warning(_("Operational status is internally set by the "
                               "plugin. Ignoring setting status=%s."),
                             resource['status'])

diff --git a/neutron/plugins/linuxbridge/agent/linuxbridge_neutron_agent.py b/neutron/plugins/linuxbridge/agent/linuxbridge_neutron_agent.py
index 3b5ec31a4c1584b6dda7e10ddb2d3b6ce42bae3b..6bf1e05d23b36101037fafc8944dd417c490ad27 100755 (executable)
--- a/neutron/plugins/linuxbridge/agent/linuxbridge_neutron_agent.py
+++ b/neutron/plugins/linuxbridge/agent/linuxbridge_neutron_agent.py
@@ -781,7 +781,7 @@ class LinuxBridgeNeutronAgentRPC(sg_rpc.SecurityGroupAgentRpcMixin):
         self.root_helper = root_helper
         self.setup_linux_bridge(interface_mappings)
         configurations = {'interface_mappings': interface_mappings}
-        if self.br_mgr.vxlan_mode is not lconst.VXLAN_NONE:
+        if self.br_mgr.vxlan_mode != lconst.VXLAN_NONE:
             configurations['tunneling_ip'] = self.br_mgr.local_ip
             configurations['tunnel_types'] = [p_const.TYPE_VXLAN]
             configurations['l2_population'] = cfg.CONF.VXLAN.l2_population

diff --git a/neutron/plugins/nec/nec_plugin.py b/neutron/plugins/nec/nec_plugin.py
index 41aaf61f7c96e473a6f298d148fa4a6dd142a0bf..b57a7ec08e39d251d4d046a335b4ce2b23bb5814 100644 (file)
--- a/neutron/plugins/nec/nec_plugin.py
+++ b/neutron/plugins/nec/nec_plugin.py
@@ -227,7 +227,7 @@ class NECPluginV2(db_base_plugin_v2.NeutronDbPluginV2,
             LOG.error(_("create_ofc_port() failed due to %s"), exc)
             port_status = const.PORT_STATUS_ERROR
 
-        if port_status is not port['status']:
+        if port_status != port['status']:
             self._update_resource_status(context, "port", port['id'],
                                          port_status)
             port['status'] = port_status
@@ -248,7 +248,7 @@ class NECPluginV2(db_base_plugin_v2.NeutronDbPluginV2,
             LOG.error(_("delete_ofc_port() failed due to %s"), exc)
             port_status = const.PORT_STATUS_ERROR
 
-        if port_status is not port['status']:
+        if port_status != port['status']:
             self._update_resource_status(context, "port", port['id'],
                                          port_status)
             port['status'] = port_status
@@ -313,7 +313,7 @@ class NECPluginV2(db_base_plugin_v2.NeutronDbPluginV2,
                                                               network)
             self._process_l3_update(context, new_net, network['network'])
 
-        changed = (old_net['admin_state_up'] is not new_net['admin_state_up'])
+        changed = (old_net['admin_state_up'] != new_net['admin_state_up'])
         if changed and not new_net['admin_state_up']:
             # disable all active ports of the network
             filters = dict(network_id=[id], status=[const.PORT_STATUS_ACTIVE])

diff --git a/neutron/tests/unit/test_extension_portsecurity.py b/neutron/tests/unit/test_extension_portsecurity.py
index a90e7a2cba9cd8cc277251b288c8435b5ce78b8d..c336db23a853ee052a1911d50ef2a9e9f0c7d531 100644 (file)
--- a/neutron/tests/unit/test_extension_portsecurity.py
+++ b/neutron/tests/unit/test_extension_portsecurity.py
@@ -136,7 +136,7 @@ class PortSecurityTestPlugin(db_base_plugin_v2.NeutronDbPluginV2,
 
             # Port security/IP was updated off. Need to check that no security
             # groups are on port.
-            if (ret_port[psec.PORTSECURITY] is not True or not has_ip):
+            if (ret_port[psec.PORTSECURITY] != True or not has_ip):
                 if has_security_groups:
                     raise psec.PortSecurityAndIPRequiredForSecurityGroups()