Pin flake8 version in tox.ini to 6.1.0 47/14147/1 v12.0.7
authorgarciadeblas <gerardo.garciadeblas@telefonica.com>
Thu, 21 Dec 2023 15:28:15 +0000 (16:28 +0100)
committergarciadeblas <gerardo.garciadeblas@telefonica.com>
Thu, 21 Dec 2023 15:28:15 +0000 (16:28 +0100)
Change-Id: I8b16a5b5bb7f037bc627613303344d51ba6b5672
Signed-off-by: garciadeblas <gerardo.garciadeblas@telefonica.com>
RO-SDN-floodlight_openflow/osm_rosdn_floodlightof/floodlight_of.py
RO-SDN-onos_openflow/osm_rosdn_onosof/onos_of.py
RO-VIM-vmware/osm_rovim_vmware/vimconn_vmware.py
releasenotes/notes/pin_flake8_version-6816c0e2c7231066.yaml [new file with mode: 0644]
tox.ini

index c5fec1a..43b3a30 100644 (file)
@@ -241,7 +241,7 @@ class OfConnFloodLight(OpenflowConn):
             self.logger.debug("get_of_rules " + error_text)
             info = of_response.json()
 
-            if type(info) != dict:
+            if not info.isinstance(dict):
                 self.logger.error(
                     "get_of_rules. Unexpected response not a dict %s", str(type(info))
                 )
@@ -445,7 +445,7 @@ class OfConnFloodLight(OpenflowConn):
                     self.logger.debug("obtain_port_correspondence " + error_text)
                     info = of_response.json()
 
-                    if type(info) != dict:
+                    if not info.isinstance(dict):
                         raise OpenflowConnUnexpectedResponse(
                             "unexpected openflow port-desc response, "
                             "not a dict. Wrong version?"
@@ -457,10 +457,9 @@ class OfConnFloodLight(OpenflowConn):
                             "'portDesc' not found. Wrong version?"
                         )
 
-                    if (
-                        type(info["portDesc"]) != list
-                        and type(info["portDesc"]) != tuple
-                    ):
+                    if not info["portDesc"].isinstance(list) and not info[
+                        "portDesc"
+                    ].isinstance(tuple):
                         raise OpenflowConnUnexpectedResponse(
                             "unexpected openflow port-desc response at "
                             "'portDesc', not a list. Wrong version?"
index 34359ae..be8fa7d 100644 (file)
@@ -128,7 +128,7 @@ class OfConnOnos(OpenflowConn):
             self.logger.debug("get_of_switches " + error_text)
             info = of_response.json()
 
-            if type(info) != dict:
+            if not info.isinstance(dict):
                 self.logger.error(
                     "get_of_switches. Unexpected response, not a dict: %s", str(info)
                 )
@@ -312,7 +312,7 @@ class OfConnOnos(OpenflowConn):
 
             info = of_response.json()
 
-            if type(info) != dict:
+            if not info.isinstance(dict):
                 self.logger.error(
                     "get_of_rules. Unexpected response, not a dict: %s",
                     str(info),
index 9ad628b..6852747 100644 (file)
@@ -6330,7 +6330,7 @@ class vimconnector(vimconn.VimConnector):
                         ].spec.ip.ipAddress
 
                         for device in vm_obj.config.hardware.device:
-                            if type(device) == vim.vm.device.VirtualPCIPassthrough:
+                            if device.isinstance(vim.vm.device.VirtualPCIPassthrough):
                                 device_details = {
                                     "devide_id": device.backing.id,
                                     "pciSlotNumber": device.slotInfo.pciSlotNumber,
diff --git a/releasenotes/notes/pin_flake8_version-6816c0e2c7231066.yaml b/releasenotes/notes/pin_flake8_version-6816c0e2c7231066.yaml
new file mode 100644 (file)
index 0000000..ea239df
--- /dev/null
@@ -0,0 +1,23 @@
+#######################################################################################
+# Copyright ETSI Contributors and Others.
+#
+# Licensed under the Apache License, Version 2.0 (the "License");
+# you may not use this file except in compliance with the License.
+# You may obtain a copy of the License at
+#
+#    http://www.apache.org/licenses/LICENSE-2.0
+#
+# Unless required by applicable law or agreed to in writing, software
+# distributed under the License is distributed on an "AS IS" BASIS,
+# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or
+# implied.
+# See the License for the specific language governing permissions and
+# limitations under the License.
+#######################################################################################
+---
+other:
+  - |
+    This change pins flake8 version in tox.ini to 6.1.0. In addition, all the flake8
+    issues have been properly fixed. All fixes are related to comparisons done with
+    == or != instead of using the keyword isinstance.
+
diff --git a/tox.ini b/tox.ini
index f2d9692..41a0999 100644 (file)
--- a/tox.ini
+++ b/tox.ini
@@ -138,7 +138,7 @@ commands =
 #######################################################################################
 [testenv:flake8]
 deps =
-        flake8
+        flake8==6.1.0
         flake8-import-order
 skip_install = true
 commands =