os-vif 3.7.0__py3-none-any.whl → 4.1.0__py3-none-any.whl

os_vif/internal/ip/api.py

@@ -10,21 +10,10 @@
 #    License for the specific language governing permissions and limitations
 #    under the License.
 
-import os
-import warnings
-
 from oslo_log import log as logging
 
-if os.name == 'nt':
-    warnings.warn('Support for Windows OS is deprecated.',
-                  category=DeprecationWarning)
-    from os_vif.internal.ip.windows.impl_netifaces import \
-        Netifaces as ip_lib_class
-else:
-    from os_vif.internal.ip.linux.impl_pyroute2 import \
-        PyRoute2 as ip_lib_class
-
+from os_vif.internal.ip.linux.impl_pyroute2 import PyRoute2
 
 LOG = logging.getLogger(__name__)
 
-ip = ip_lib_class()
+ip = PyRoute2()

os_vif/tests/unit/internal/ip/test_api.py

@@ -10,30 +10,12 @@
 #    License for the specific language governing permissions and limitations
 #    under the License.
 
-import importlib
-
-from unittest import mock
-
 from os_vif.internal.ip import api
+from os_vif.internal.ip.linux import impl_pyroute2
 from os_vif.tests.unit import base
 
 
 class TestIpApi(base.TestCase):
 
-    @staticmethod
-    def _reload_original_os_module():
-        importlib.reload(api)
-
-    def test_get_impl_windows(self):
-        self.addCleanup(self._reload_original_os_module)
-        with mock.patch('os.name', 'nt'):
-            importlib.reload(api)
-            from os_vif.internal.ip.windows import impl_netifaces
-            self.assertIsInstance(api.ip, impl_netifaces.Netifaces)
-
-    def test_get_impl_linux(self):
-        self.addCleanup(self._reload_original_os_module)
-        with mock.patch('os.name', 'posix'):
-            importlib.reload(api)
-            from os_vif.internal.ip.linux import impl_pyroute2
-            self.assertIsInstance(api.ip, impl_pyroute2.PyRoute2)
+    def test_get_impl(self):
+        self.assertIsInstance(api.ip, impl_pyroute2.PyRoute2)

os_vif/tests/unit/test_os_vif.py

@@ -105,10 +105,13 @@ class TestOSVIF(base.TestCase):
                                   entry_point="os-vif",
                                   plugin=DemoPlugin,
                                   obj=None)
-        with mock.patch('stevedore.extension.ExtensionManager.names',
-                        return_value=['foobar']),\
-                mock.patch('stevedore.extension.ExtensionManager.__getitem__',
-                           return_value=plg):
+        with mock.patch(
+            'stevedore.extension.ExtensionManager.names',
+            return_value=['foobar'],
+        ), mock.patch(
+            'stevedore.extension.ExtensionManager.__getitem__',
+            return_value=plg,
+        ):
             os_vif.initialize()
             info = objects.instance_info.InstanceInfo()
             vif = objects.vif.VIFBridge(
@@ -123,10 +126,13 @@ class TestOSVIF(base.TestCase):
                                   entry_point="os-vif",
                                   plugin=DemoPlugin,
                                   obj=None)
-        with mock.patch('stevedore.extension.ExtensionManager.names',
-                        return_value=['foobar']),\
-                mock.patch('stevedore.extension.ExtensionManager.__getitem__',
-                           return_value=plg):
+        with mock.patch(
+            'stevedore.extension.ExtensionManager.names',
+            return_value=['foobar']
+        ), mock.patch(
+            'stevedore.extension.ExtensionManager.__getitem__',
+            return_value=plg,
+        ):
             os_vif.initialize()
             info = objects.instance_info.InstanceInfo()
             vif = objects.vif.VIFBridge(

{os_vif-3.7.0.dist-info → os_vif-4.1.0.dist-info}/AUTHORS

@@ -2,6 +2,7 @@ Adrian Chiris <adrianc@mellanox.com>
 Alin Balutoiu <abalutoiu@cloudbasesolutions.com>
 Andreas Jaeger <aj@suse.com>
 Balazs Gibizer <gibi@redhat.com>
+Bence Romsics <bence.romsics@gmail.com>
 Brian Haley <brian.haley@hpe.com>
 Cao Xuan Hoang <hoangcx@vn.fujitsu.com>
 Carlos Goncalves <cgoncalves@redhat.com>
@@ -47,6 +48,7 @@ Sahid Orentino Ferdjaoui <sahid.ferdjaoui@redhat.com>
 Sean Dague <sean@dague.net>
 Sean M. Collins <sean@coreitpro.com>
 Sean Mooney <sean.k.mooney@intel.com>
+Sean Mooney <sean@seanmooney.info>
 Sean Mooney <work@seanmooney.info>
 Sergey Belous <sbelous@mirantis.com>
 Spencer Yu <yushb@gohighsec.com>

{os_vif-3.7.0.dist-info → os_vif-4.1.0.dist-info}/METADATA

@@ -1,12 +1,10 @@
 Metadata-Version: 2.1
-Name: os-vif
-Version: 3.7.0
+Name: os_vif
+Version: 4.1.0
 Summary: A library for plugging and unplugging virtual interfaces in OpenStack.
 Home-page: https://docs.openstack.org/os-vif/latest/
 Author: OpenStack
 Author-email: openstack-discuss@lists.openstack.org
-License: UNKNOWN
-Platform: UNKNOWN
 Classifier: Environment :: OpenStack
 Classifier: Intended Audience :: Information Technology
 Classifier: Intended Audience :: System Administrators
@@ -15,25 +13,27 @@ Classifier: Operating System :: POSIX :: Linux
 Classifier: Programming Language :: Python
 Classifier: Programming Language :: Python :: 3
 Classifier: Programming Language :: Python :: 3 :: Only
-Classifier: Programming Language :: Python :: 3.8
 Classifier: Programming Language :: Python :: 3.9
 Classifier: Programming Language :: Python :: 3.10
 Classifier: Programming Language :: Python :: 3.11
+Classifier: Programming Language :: Python :: 3.12
 Classifier: Programming Language :: Python :: Implementation :: CPython
-Requires-Python: >=3.8
-Requires-Dist: debtcollector (>=1.19.0)
-Requires-Dist: oslo.concurrency (>=3.20.0)
-Requires-Dist: oslo.config (>=5.1.0)
-Requires-Dist: oslo.i18n (>=3.15.3)
-Requires-Dist: oslo.log (>=3.30.0)
-Requires-Dist: oslo.privsep (>=1.23.0)
-Requires-Dist: oslo.serialization (>=2.20.0)
-Requires-Dist: oslo.utils (>=2.0.0)
-Requires-Dist: oslo.versionedobjects (>=1.28.0)
-Requires-Dist: ovsdbapp (>=0.12.1)
-Requires-Dist: pbr (>=3.0.0)
-Requires-Dist: stevedore (>=1.20.0)
-Requires-Dist: pyroute2 (>=0.5.2) ; (sys_platform!='win32')
+Requires-Python: >=3.9
+License-File: LICENSE
+License-File: AUTHORS
+Requires-Dist: pbr >=3.0.0
+Requires-Dist: oslo.concurrency >=3.20.0
+Requires-Dist: oslo.config >=5.1.0
+Requires-Dist: oslo.log >=3.30.0
+Requires-Dist: oslo.i18n >=3.15.3
+Requires-Dist: oslo.privsep >=1.23.0
+Requires-Dist: oslo.serialization >=2.20.0
+Requires-Dist: oslo.utils >=2.0.0
+Requires-Dist: oslo.versionedobjects >=1.28.0
+Requires-Dist: ovsdbapp >=0.12.1
+Requires-Dist: stevedore >=1.20.0
+Requires-Dist: debtcollector >=1.19.0
+Requires-Dist: pyroute2 >=0.5.2 ; (sys_platform!='win32')
 
 ========================
 Team and repository tags
@@ -64,5 +64,3 @@ A library for plugging and unplugging virtual interfaces in OpenStack.
 * Bugs: https://bugs.launchpad.net/os-vif
 * Release Notes: https://docs.openstack.org/releasenotes/os-vif
 
-
-

{os_vif-3.7.0.dist-info → os_vif-4.1.0.dist-info}/RECORD

@@ -7,12 +7,10 @@ os_vif/utils.py,sha256=bmJRam0laHAVLIHpQufzLASu_VTEqsGTuJYmoYfqOco,713
 os_vif/version.py,sha256=ilQe2xTxPDxlzJYWbq8KdHeashflTfC1OZewgFPKPRI,713
 os_vif/internal/__init__.py,sha256=VGE1e9o-7RkuTzNhZl4Wivz1tojriLYypVBObhSZfSk,967
 os_vif/internal/ip/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
-os_vif/internal/ip/api.py,sha256=AMsRxYFL9DZOzCAt-g_rzlmOYGwV5MQW_FGYdnnwzj8,1010
+os_vif/internal/ip/api.py,sha256=BlGJdRe_Yn7P23Dejo7gLK-M6UasY8sKrMCz0FPvx2s,722
 os_vif/internal/ip/ip_command.py,sha256=agySNBD4GHp87uo6HbtIg4JwtOMhfuRZN5X7ytPir5k,2787
 os_vif/internal/ip/linux/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
 os_vif/internal/ip/linux/impl_pyroute2.py,sha256=ddQ9d7fzBJEYmE1wJhMfpmHftTBFdJ2C94dCNGRsB0A,5511
-os_vif/internal/ip/windows/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
-os_vif/internal/ip/windows/impl_netifaces.py,sha256=hPgC7KaymxdD4XrYw5G5IkXiKjth-tc9Gc1_pzcsyuo,1745
 os_vif/objects/__init__.py,sha256=ePMVrOlb8ti6YmpdZ15pOXI1XLeFjHvhBvw35abZRZE,883
 os_vif/objects/base.py,sha256=kI0c1gPDN37RhGuOYg18AA7l98uUfpqs14AS9GfE4oE,1139
 os_vif/objects/fields.py,sha256=i_kVqeCOBfcNKuZiB3gBat-8qJWhLtwEQxsl-k5plgw,1761
@@ -37,19 +35,17 @@ os_vif/tests/unit/test_base.py,sha256=iqgweFRWprY98mkGL3HIbV9D8DuX_tUPH8uRiDflwD
 os_vif/tests/unit/test_exception.py,sha256=UDVJSR3CKU3rc5kmN14ilL9Fj0cgnwQbGoP4weo2P4w,1335
 os_vif/tests/unit/test_host_info.py,sha256=2PYHg4pY-YwUuSg9MyDKJbJhKVGRT6ujJATkzzIPx18,7576
 os_vif/tests/unit/test_objects.py,sha256=WflvhMvvCauXexj04tCw47HvsysHX4Q7fsgJKrPNaI8,3709
-os_vif/tests/unit/test_os_vif.py,sha256=ixMy3_RolD3K4ebWynRt7JLI63m_fb3XIO7qrXGCq60,6609
+os_vif/tests/unit/test_os_vif.py,sha256=b0h65Wa9tvxw-e7c707SoJQjfGzEALSFZUPkTtr_GkY,6612
 os_vif/tests/unit/test_vif.py,sha256=mhg2PZ3iVEPzffO5I0VT1wIUP3c-h7MAqrZgCuZdjxE,18000
 os_vif/tests/unit/internal/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
 os_vif/tests/unit/internal/ip/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
-os_vif/tests/unit/internal/ip/test_api.py,sha256=jPkByWTFZZK1Vj3s3_EkaH4mcCzBAbMznDn-VjKcH7U,1415
+os_vif/tests/unit/internal/ip/test_api.py,sha256=LnkAM0M8giNjAw_Q4-oRic3WIW3OWq-SCGlde9SCenY,820
 os_vif/tests/unit/internal/ip/linux/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
 os_vif/tests/unit/internal/ip/linux/test_impl_pyroute2.py,sha256=MbsDkFkVSmx3mqteP7GW3ruy6ehvBCiQorkdWJjE5Qk,7364
-os_vif/tests/unit/internal/ip/windows/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
-os_vif/tests/unit/internal/ip/windows/test_impl_netifaces.py,sha256=fs1DI706OXTWGr4C8_UtbBEvxwImv3klrxmuktUYol8,1992
 vif_plug_linux_bridge/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
 vif_plug_linux_bridge/constants.py,sha256=NC8n4uOMFCYrhq0sM1ZeGBKo-Gs4OMR60cYP47wMRCk,602
 vif_plug_linux_bridge/iptables.py,sha256=q4LmAMMzg-r9IgDNTu3jCLjNDM8b0JbGo-qmiA2B-JE,20641
-vif_plug_linux_bridge/linux_bridge.py,sha256=YMa_T8MfZBHmq0yQuA-fIYogZ-yRPKDzzBYGIfgUGO4,4949
+vif_plug_linux_bridge/linux_bridge.py,sha256=oNf-HuLuaixlOyAL2CgZeZ3OHSknUh-zMhmPDI6XOBc,5345
 vif_plug_linux_bridge/linux_net.py,sha256=0SuMKqMSNHRnSMs3XNVjBF0MPzhUuQIti159QkL8k1U,9481
 vif_plug_linux_bridge/privsep.py,sha256=dHAVOrY7kC3hrfDy7OAyt17XG2TpZHwgMr0Pmgx1XYM,884
 vif_plug_linux_bridge/tests/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
@@ -62,32 +58,32 @@ vif_plug_noop/tests/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSu
 vif_plug_noop/tests/unit/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
 vif_plug_noop/tests/unit/test_plugin.py,sha256=x8m9zuGtHsPoZRZP6Q6BREAFx0wiEagFtTFlB_i_Bxc,1236
 vif_plug_ovs/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
-vif_plug_ovs/constants.py,sha256=P1ngaAA2MioPrSr58JtiDA8rxkl3thttgOvabEyZGIM,944
+vif_plug_ovs/constants.py,sha256=vQKnYSeBMjV2RGQPGxk96D8FtgpvJIZEX_yZzUlsGg8,892
 vif_plug_ovs/exception.py,sha256=ZiECZVvjCFTVXhuii3uhJn4lZvozKRxziedZB3R9JNo,1328
-vif_plug_ovs/linux_net.py,sha256=nr2dNOECVZtiHSIQHFb5iRVyh7cfLPiIm4VEg3frgDE,14289
-vif_plug_ovs/ovs.py,sha256=hAkt-8CbpYuGcfUjb3NKjm41J5MniAQupVl6IsB5p7Q,24166
+vif_plug_ovs/linux_net.py,sha256=HvhfSXjeAlU2l9iZNiN8CFr-GL9psS8Hk8_KAaktH6I,13922
+vif_plug_ovs/ovs.py,sha256=tN-JxK7VoIAdlradnk6kKCChenu_LLQWMKjJ_u834QY,22946
 vif_plug_ovs/privsep.py,sha256=nPQUkYgjbSjaHMuu40fZArRnf8RBiR73l4YpDF1K1yQ,1100
 vif_plug_ovs/ovsdb/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
 vif_plug_ovs/ovsdb/api.py,sha256=_jHR8xokDAPI8sw3IEWuT3hObJuskqVzeVMDaWRbIq0,1293
 vif_plug_ovs/ovsdb/impl_idl.py,sha256=DR1aHN8riBpZyNePwpvT9PRYbf9vLk4lRJ9VHRVq_to,3028
 vif_plug_ovs/ovsdb/impl_vsctl.py,sha256=MSf8JMrMqrwt5vH8KTsThjdnsttJEYJikIcJ12CTz8c,14437
-vif_plug_ovs/ovsdb/ovsdb_lib.py,sha256=UfVJG8PtIp2ps51JXp3El9rUgtfR12Mp5dna6_i-20M,10599
+vif_plug_ovs/ovsdb/ovsdb_lib.py,sha256=SJuEnl26wGOLzEXNdgVhrKue2FTPgCpLiVLNak34Buw,10505
 vif_plug_ovs/tests/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
 vif_plug_ovs/tests/functional/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
-vif_plug_ovs/tests/functional/base.py,sha256=9EDYKUqsoqVgrYmljb6fZvGmIruL2zNJdIHlEZ9kzu8,1872
-vif_plug_ovs/tests/functional/test_plugin.py,sha256=40gbf5bvxgauHf3U3oT9xeIOcZ3r8-fFRyHOrhWbZVo,6648
+vif_plug_ovs/tests/functional/base.py,sha256=GZKyA9UR9BaEgZ0tv6xyMTQnz9FpKedV2do1ePMB-rw,2059
+vif_plug_ovs/tests/functional/test_plugin.py,sha256=JzRiEIUonC4STcV9GWiwyaJybgOVxMSHyLmqACBf8Ck,8264
 vif_plug_ovs/tests/functional/ovsdb/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
-vif_plug_ovs/tests/functional/ovsdb/test_ovsdb_lib.py,sha256=oB06rO9KgezJ5fe4kxsKANkC6b7sqKkzSyO2_M-Q8gw,13513
+vif_plug_ovs/tests/functional/ovsdb/test_ovsdb_lib.py,sha256=tXSTmjYWOmqLXYQ42_3Sv8XNNktlDxPDalJRr8KRwCU,13775
 vif_plug_ovs/tests/unit/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
 vif_plug_ovs/tests/unit/test_linux_net.py,sha256=B3buNpn9EKZQPio5rNrIma_E5MUL9vBspdg5EqZ3RHk,17532
-vif_plug_ovs/tests/unit/test_plugin.py,sha256=-RmB-IGzaFaFmg-sNLupFd0sHwdRQ6IoGpgWFNdmv8k,33303
+vif_plug_ovs/tests/unit/test_plugin.py,sha256=YfJXJirOWbZB2l2VHTiJPCMZlgBCyRXDAFgzEj6pQeY,30827
 vif_plug_ovs/tests/unit/ovsdb/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
-vif_plug_ovs/tests/unit/ovsdb/test_ovsdb_lib.py,sha256=rNPmkH_xkGCrRDeUV_xQEAJ6A7sOH-sEGXvPYju1zwk,10705
-os_vif-3.7.0.dist-info/AUTHORS,sha256=kZdfNnkbapBsRD4W-nOGD5iPcwn83A2tZvVbRJxItcA,3215
-os_vif-3.7.0.dist-info/LICENSE,sha256=XfKg2H1sVi8OoRxoisUlMqoo10TKvHmU_wU39ks7MyA,10143
-os_vif-3.7.0.dist-info/METADATA,sha256=mDS7GozmC4yApdWSNUn2jH6T7yJI1E_U7DaEIumSRKw,2356
-os_vif-3.7.0.dist-info/WHEEL,sha256=g4nMs7d-Xl9-xC9XovUrsDHGXt-FT0E17Yqo92DEfvY,92
-os_vif-3.7.0.dist-info/entry_points.txt,sha256=UhIaj_WPv49yG316SjjDuMlSqGvym-_TVYEn5JKlEaQ,207
-os_vif-3.7.0.dist-info/pbr.json,sha256=ht12tHl6tNQ2XGWgv4VLXIkGcWong6C6o_mDCjqWVF0,46
-os_vif-3.7.0.dist-info/top_level.txt,sha256=ULBxtkTk3bkfzCSYJjifWehfjJdMODVzC6SX5l_CNKo,56
-os_vif-3.7.0.dist-info/RECORD,,
+vif_plug_ovs/tests/unit/ovsdb/test_ovsdb_lib.py,sha256=-Z50UgEgg2awEgPb_r_iqQBsyYTKlSadi3S9laGFfjg,10270
+os_vif-4.1.0.dist-info/AUTHORS,sha256=Jl9BVjDjzq5vO3_b0rnuUWx-P4wsTnfH8iLVMFZrUH8,3290
+os_vif-4.1.0.dist-info/LICENSE,sha256=XfKg2H1sVi8OoRxoisUlMqoo10TKvHmU_wU39ks7MyA,10143
+os_vif-4.1.0.dist-info/METADATA,sha256=rDAbv57FXl1MAJQmQsJsLw5cK6bAEOHMwVCtVxEfOfI,2338
+os_vif-4.1.0.dist-info/WHEEL,sha256=P9jw-gEje8ByB7_hXoICnHtVCrEwMQh-630tKvQWehc,91
+os_vif-4.1.0.dist-info/entry_points.txt,sha256=BU9WplZnF_bjwT4SQP5n1mKu2f0R5xXBOznXP1GygHM,206
+os_vif-4.1.0.dist-info/pbr.json,sha256=Yy5hVFB_dwDLDjOBUX039V8KA8i0TiV456Q_kkEfqDM,46
+os_vif-4.1.0.dist-info/top_level.txt,sha256=ULBxtkTk3bkfzCSYJjifWehfjJdMODVzC6SX5l_CNKo,56
+os_vif-4.1.0.dist-info/RECORD,,

{os_vif-3.7.0.dist-info → os_vif-4.1.0.dist-info}/WHEEL

@@ -1,5 +1,5 @@
 Wheel-Version: 1.0
-Generator: bdist_wheel (0.34.2)
+Generator: setuptools (75.3.0)
 Root-Is-Purelib: true
 Tag: py3-none-any
 

{os_vif-3.7.0.dist-info → os_vif-4.1.0.dist-info}/entry_points.txt

@@ -5,4 +5,3 @@ ovs = vif_plug_ovs.ovs:OvsPlugin
 
 [oslo.config.opts]
 os_vif = os_vif.opts:list_plugins_opts
-

os_vif-4.1.0.dist-info/pbr.json

@@ -0,0 +1 @@
+{"git_version": "acdd974", "is_release": true}

vif_plug_linux_bridge/linux_bridge.py

@@ -17,6 +17,7 @@
 #    License for the specific language governing permissions and limitations
 #    under the License.
 
+import debtcollector.removals
 from os_vif import objects
 from os_vif import plugin
 from oslo_config import cfg
@@ -94,6 +95,10 @@ class LinuxBridgePlugin(plugin.PluginBase):
                     supported_port_profiles=[]),
             ])
 
+    @debtcollector.removals.remove(
+        message="LinuxBridgePlugin is deprecated and will be removed in "
+                "a future release",
+        category=DeprecationWarning)
     def plug(self, vif, instance_info):
         """Ensure that the bridge exists, and add VIF to it."""
         network = vif.network
@@ -111,6 +116,10 @@ class LinuxBridgePlugin(plugin.PluginBase):
                 linux_net.ensure_bridge(bridge_name, iface,
                                         filtering=install_filters, mtu=mtu)
 
+    @debtcollector.removals.remove(
+        message="LinuxBridgePlugin is deprecated and will be removed in "
+                "a future release",
+        category=DeprecationWarning)
     def unplug(self, vif, instance_info):
         # Nothing required to unplug a port for a VIF using standard
         # Linux bridge device...

vif_plug_ovs/constants.py

@@ -19,9 +19,6 @@ OVS_VHOSTUSER_PREFIX = 'vhu'
 OVS_DATAPATH_SYSTEM = 'system'
 OVS_DATAPATH_NETDEV = 'netdev'
 
-PLATFORM_LINUX = 'linux2'
-PLATFORM_WIN32 = 'win32'
-
 OVS_DPDK_INTERFACE_TYPE = 'dpdk'
 
 # Neutron dead VLAN.

vif_plug_ovs/linux_net.py

@@ -22,14 +22,12 @@
 import glob
 import os
 import re
-import sys
 
 from os_vif.internal.ip.api import ip as ip_lib
 from oslo_concurrency import processutils
 from oslo_log import log as logging
 from oslo_utils import excutils
 
-from vif_plug_ovs import constants
 from vif_plug_ovs import exception
 from vif_plug_ovs import privsep
 
@@ -55,12 +53,7 @@ NIC_NAME_LEN = 14
 def _update_device_mtu(dev, mtu):
     if not mtu:
         return
-    if sys.platform != constants.PLATFORM_WIN32:
-        # Hyper-V with OVS does not support external programming of
-        # virtual interface MTUs via netsh or other Windows tools.
-        # When plugging an interface on Windows, we therefore skip
-        # programming the MTU and fallback to DHCP advertisement.
-        set_device_mtu(dev, mtu)
+    set_device_mtu(dev, mtu)
 
 
 @privsep.vif_plug.entrypoint

vif_plug_ovs/ovs.py

@@ -17,8 +17,6 @@
 #    License for the specific language governing permissions and limitations
 #    under the License.
 
-import sys
-
 from oslo_config import cfg
 from oslo_log import log as logging
 
@@ -27,7 +25,6 @@ from os_vif.internal.ip.api import ip as ip_lib
 from os_vif import objects
 from os_vif import plugin
 
-
 from vif_plug_ovs import constants
 from vif_plug_ovs import exception
 from vif_plug_ovs import linux_net
@@ -181,9 +178,6 @@ class OvsPlugin(plugin.PluginBase):
     def supports_tc_qdisc(self, vif) -> bool:
         if self._get_vif_datapath_type(vif) != constants.OVS_DATAPATH_SYSTEM:
             return False
-        if sys.platform == constants.PLATFORM_WIN32:
-            return False
-
         return True
 
     def _isolate_vif(self, vif_name, bridge):
@@ -208,8 +202,12 @@ class OvsPlugin(plugin.PluginBase):
         # bound the interface in the vif binding details so isolation
         # can be enabled automatically in the future.
         bridge = kwargs.pop('bridge', vif.network.bridge)
-        if self._isolate_vif(vif_name, bridge):
+        # See bug #2069543.
+        if (self._isolate_vif(vif_name, bridge) and
+                not self._is_trunk_bridge(bridge)):
             kwargs['tag'] = constants.DEAD_VLAN
+            kwargs['vlan_mode'] = 'trunk'
+            kwargs['trunks'] = constants.DEAD_VLAN
         qos_type = self._get_qos_type(vif)
         if qos_type is not None:
             # NOTE(sean-k-mooney): If the port is not already created
@@ -283,14 +281,6 @@ class OvsPlugin(plugin.PluginBase):
             linux_net.update_veth_pair(v1_name, v2_name, mtu)
             self._update_vif_port(vif, v2_name)
 
-    def _plug_vif_windows(self, vif, instance_info):
-        """Create a per-VIF OVS port."""
-
-        if not ip_lib.exists(vif.id):
-            self.ovsdb.ensure_ovs_bridge(vif.network.bridge,
-                                         self._get_vif_datapath_type(vif))
-            self._create_vif_port(vif, vif.id, instance_info)
-
     def _plug_port_bridge(self, vif, instance_info):
         """Create a per-VIF OVS bridge and patch pair."""
 
@@ -317,8 +307,8 @@ class OvsPlugin(plugin.PluginBase):
         instance_id = instance_info.uuid
         LOG.debug(
             'creating patch port pair \n'
-            f'{port_bridge_name}:({port_bridge_patch}) -> '
-            f'{int_bridge_name}:({int_bridge_patch})'
+            f'{port_bridge_name}: ({port_bridge_patch}) -> '
+            f'{int_bridge_name}: ({int_bridge_patch})'
         )
         self.ovsdb.create_patch_port_pair(
             port_bridge_name, port_bridge_patch, int_bridge_name,
@@ -381,15 +371,7 @@ class OvsPlugin(plugin.PluginBase):
             raise exception.WrongPortProfile(
                 profile=vif.port_profile.__class__.__name__)
 
-        if sys.platform == constants.PLATFORM_WIN32:
-            if type(vif) not in (
-                objects.vif.VIFOpenVSwitch, objects.vif.VIFBridge
-            ):
-                raise osv_exception.PlugException(
-                    vif=vif, err="This vif type is not supported on Windows")
-
-            self._plug_vif_windows(vif, instance_info)
-        elif isinstance(vif, objects.vif.VIFOpenVSwitch):
+        if isinstance(vif, objects.vif.VIFOpenVSwitch):
             if self.config.per_port_bridge:
                 self._plug_port_bridge(vif, instance_info)
             else:
@@ -443,12 +425,6 @@ class OvsPlugin(plugin.PluginBase):
             qos_type = self.config.default_qos_type
         return qos_type
 
-    def _unplug_vif_windows(self, vif, instance_info):
-        """Remove port from OVS."""
-        self.ovsdb.delete_ovs_vif_port(vif.network.bridge, vif.id,
-                                       delete_netdev=False)
-        self._delete_bridge_if_trunk(vif)
-
     def _unplug_port_bridge(self, vif, instance_info):
         """Create a per-VIF OVS bridge and patch pair."""
         # NOTE(sean-k-mooney): the port name prefix should not be
@@ -508,14 +484,7 @@ class OvsPlugin(plugin.PluginBase):
                           objects.vif.VIFPortProfileOpenVSwitch):
             raise exception.WrongPortProfile(
                 profile=vif.port_profile.__class__.__name__)
-        if sys.platform == constants.PLATFORM_WIN32:
-            if type(vif) not in (
-                objects.vif.VIFOpenVSwitch, objects.vif.VIFBridge
-            ):
-                raise osv_exception.UnplugException(
-                    vif=vif, err="This vif type is not supported on windows.")
-            self._unplug_vif_windows(vif, instance_info)
-        elif isinstance(vif, objects.vif.VIFOpenVSwitch):
+        if isinstance(vif, objects.vif.VIFOpenVSwitch):
             if self.config.per_port_bridge:
                 self._unplug_port_bridge(vif, instance_info)
             else:

vif_plug_ovs/ovsdb/ovsdb_lib.py

@@ -10,7 +10,6 @@
 #    License for the specific language governing permissions and limitations
 #    under the License.
 
-import sys
 import uuid
 
 from oslo_log import log as logging
@@ -59,12 +58,7 @@ class BaseOVS(object):
         if interface_type not in [
             constants.OVS_VHOSTUSER_INTERFACE_TYPE,
             constants.OVS_VHOSTUSER_CLIENT_INTERFACE_TYPE]:
-            if sys.platform != constants.PLATFORM_WIN32:
-                # Hyper-V with OVS does not support external programming of
-                # virtual interface MTUs via netsh or other Windows tools.
-                # When plugging an interface on Windows, we therefore skip
-                # programming the MTU and fallback to DHCP advertisement.
-                linux_net.set_device_mtu(dev, mtu)
+            linux_net.set_device_mtu(dev, mtu)
         elif self._ovs_supports_mtu_requests():
             self._set_mtu_request(txn, dev, mtu)
         else:
@@ -145,7 +139,7 @@ class BaseOVS(object):
         self, bridge, dev, iface_id, mac, instance_id,
         mtu=None, interface_type=None, vhost_server_path=None,
         tag=None, pf_pci=None, vf_num=None, set_ids=True, datapath_type=None,
-        qos_type=None
+        qos_type=None, vlan_mode=None, trunks=None
     ):
         """Create OVS port
 
@@ -210,6 +204,11 @@ class BaseOVS(object):
             txn.add(self.ovsdb.add_port(bridge, dev))
             if tag:
                 txn.add(self.ovsdb.db_set('Port', dev, ('tag', tag)))
+            if vlan_mode:
+                txn.add(self.ovsdb.db_set('Port', dev,
+                                          ('vlan_mode', vlan_mode)))
+            if trunks:
+                txn.add(self.ovsdb.db_set('Port', dev, ('trunks', trunks)))
             if qid:
                 txn.add(self.ovsdb.db_set('Port', dev, ('qos', qid)))
             if col_values:

vif_plug_ovs/tests/functional/base.py

@@ -9,6 +9,8 @@
 #    WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. See the
 #    License for the specific language governing permissions and limitations
 #    under the License.
+import functools
+import os
 
 from os_vif.tests.functional import base as os_vif_base
 
@@ -28,6 +30,10 @@ class VifPlugOvsBaseFunctionalTestCase(os_vif_base.BaseFunctionalTestCase):
     def _check_port(self, name, bridge):
         return self.ovs.port_exists(name, bridge)
 
+    @functools.cache
+    def _get_timeout(self):
+        return int(os.environ.get('OS_VIF_CHECK_PARAMETER_TIMEOUT', '10'))
+
     def _check_parameter(self, table, port, parameter, expected_value):
         def get_value():
             return self._ovsdb.db_get(table, port, parameter).execute()
@@ -36,7 +42,8 @@ class VifPlugOvsBaseFunctionalTestCase(os_vif_base.BaseFunctionalTestCase):
             val = get_value()
             return val == expected_value
         self.assertTrue(
-            wait_until_true(check_value, timeout=2, sleep=0.5),
+            wait_until_true(
+                check_value, timeout=self._get_timeout(), sleep=0.5),
             f"Parameter {parameter} of {table} {port} is {get_value()} "
             f"not {expected_value}"
         )

vif_plug_ovs/tests/functional/ovsdb/test_ovsdb_lib.py

@@ -183,6 +183,11 @@ class TestOVSDBLib(testscenarios.WithScenarios,
             'Interface', port_bridge_port, 'options', port_opts)
 
     def test_create_ovs_vif_port_with_default_qos(self):
+        if self.interface == 'native':
+            self.skipTest(
+                'test_create_ovs_vif_port_with_default_qos is unstable '
+                'when run with the native driver, see: '
+                'https://bugs.launchpad.net/os-vif/+bug/2087982')
         port_name = 'qos-port-' + self.interface
         iface_id = 'iface_id'
         mac = 'ca:fe:ca:fe:ca:fe'

vif_plug_ovs/tests/functional/test_plugin.py

@@ -12,6 +12,7 @@
 
 import testscenarios
 import time
+from unittest import mock
 
 from oslo_concurrency import processutils
 from oslo_config import cfg
@@ -183,3 +184,37 @@ class TestOVSPlugin(testscenarios.WithScenarios,
         self._check_parameter(
             'QoS', str(qos_uuid), 'type', None
         )
+
+    def test_plug_br_int_isolate_vif_dead_vlan(self):
+        with mock.patch.object(self.plugin.config, 'isolate_vif', True):
+            network = objects.network.Network(
+                id='5449523c-3a08-11ef-86d6-17149687aa4d',
+                bridge='br-5449523c',
+                subnets=self.subnets,
+                vlan=99)
+            vif = objects.vif.VIFOpenVSwitch(
+                id='85cb9bc6-3a08-11ef-b2d4-9b7c38edd677',
+                address='ca:fe:de:ad:be:ef',
+                network=network,
+                port_profile=self.profile_ovs_system,
+                vif_name="port-85cb9bc6")
+            self.plugin.plug(vif, self.instance)
+            self.addCleanup(self._del_bridge, 'br-5449523c')
+            self._check_parameter('Port', vif.vif_name, 'tag', 4095)
+
+    def test_plug_trunk_bridge_ignores_isolate_vif(self):
+        with mock.patch.object(self.plugin.config, 'isolate_vif', True):
+            network = objects.network.Network(
+                id='ef98b384-3a0f-11ef-9009-47345fca266f',
+                bridge='tbr-ef98b384',
+                subnets=self.subnets,
+                vlan=99)
+            vif = objects.vif.VIFOpenVSwitch(
+                id='631f52bc-3a07-11ef-a006-1319ef9d6edd',
+                address='ca:fe:de:ad:be:ef',
+                network=network,
+                port_profile=self.profile_ovs_system,
+                vif_name='port-631f52bc')
+            self.plugin.plug(vif, self.instance)
+            self.addCleanup(self._del_bridge, 'tbr-ef98b384')
+            self._check_parameter('Port', vif.vif_name, 'tag', [])

vif_plug_ovs/tests/unit/ovsdb/test_ovsdb_lib.py

@@ -53,24 +53,14 @@ class BaseOVSTest(testtools.TestCase):
         calls = [mock.call('Interface', 'device', ('mtu_request', 1500))]
         self.mock_db_set.assert_has_calls(calls)
 
-    @mock.patch('sys.platform', constants.PLATFORM_LINUX)
     @mock.patch.object(linux_net, 'set_device_mtu')
-    def test__update_device_mtu_interface_not_vhostuser_linux(self,
+    def test__update_device_mtu_interface_not_vhostuser(self,
             mock_set_device_mtu):
         self.br.update_device_mtu(
             self.mock_transaction, 'device', 1500, 'not_vhost'
         )
         mock_set_device_mtu.assert_has_calls([mock.call('device', 1500)])
 
-    @mock.patch('sys.platform', constants.PLATFORM_WIN32)
-    @mock.patch.object(linux_net, 'set_device_mtu')
-    def test__update_device_mtu_interface_not_vhostuser_windows(self,
-            mock_set_device_mtu):
-        self.br.update_device_mtu(
-            self.mock_transaction, 'device', 1500, 'not_vhost'
-        )
-        mock_set_device_mtu.assert_not_called()
-
     def test__update_device_mtu_interface_vhostuser_supports_mtu_req(self):
         with mock.patch.object(self.br, '_ovs_supports_mtu_requests',
                 return_value=True), \

vif_plug_ovs/tests/unit/test_plugin.py

@@ -219,7 +219,10 @@ class PluginTest(testtools.TestCase):
                 self.vif_ovs.address, self.instance.uuid,
                 mtu=plugin.config.network_device_mtu,
                 interface_type=constants.OVS_VHOSTUSER_INTERFACE_TYPE,
-                tag=constants.DEAD_VLAN)
+                tag=constants.DEAD_VLAN,
+                vlan_mode='trunk',
+                trunks=constants.DEAD_VLAN
+                )
 
     @mock.patch.object(ovsdb_lib.BaseOVS, 'create_ovs_vif_port')
     @mock.patch.object(ovsdb_lib.BaseOVS, 'port_exists')
@@ -238,20 +241,16 @@ class PluginTest(testtools.TestCase):
                 mtu=plugin.config.network_device_mtu,
                 interface_type=constants.OVS_VHOSTUSER_INTERFACE_TYPE)
 
-    @mock.patch.object(ovs, 'sys')
     @mock.patch.object(ovs.OvsPlugin, '_plug_vif_generic')
-    def test_plug_ovs_port_bridge_false(self, plug_vif_generic, mock_sys):
-        mock_sys.platform = 'linux'
+    def test_plug_ovs_port_bridge_false(self, plug_vif_generic):
         plugin = ovs.OvsPlugin.load(constants.PLUGIN_NAME)
         with mock.patch.object(plugin.config, 'per_port_bridge', False):
             plugin.plug(self.vif_ovs, self.instance)
             plug_vif_generic.assert_called_once_with(
                 self.vif_ovs, self.instance)
 
-    @mock.patch.object(ovs, 'sys')
     @mock.patch.object(ovs.OvsPlugin, '_plug_port_bridge')
-    def test_plug_ovs_port_bridge_true(self, plug_vif, mock_sys):
-        mock_sys.platform = 'linux'
+    def test_plug_ovs_port_bridge_true(self, plug_vif):
         plugin = ovs.OvsPlugin.load(constants.PLUGIN_NAME)
         with mock.patch.object(plugin.config, 'per_port_bridge', True):
             plugin.plug(self.vif_ovs, self.instance)
@@ -276,8 +275,7 @@ class PluginTest(testtools.TestCase):
     @mock.patch.object(linux_net, 'create_veth_pair')
     @mock.patch.object(ip_lib, 'exists')
     @mock.patch.object(linux_net, 'ensure_bridge')
-    @mock.patch.object(ovs, 'sys')
-    def test_plug_ovs_bridge(self, mock_sys, ensure_bridge, device_exists,
+    def test_plug_ovs_bridge(self, ensure_bridge, device_exists,
                              create_veth_pair, update_veth_pair,
                              add_bridge_port, _create_vif_port,
                              _update_vif_port, ensure_ovs_bridge,
@@ -307,7 +305,6 @@ class PluginTest(testtools.TestCase):
         # plugging new devices should result in devices being created
 
         device_exists.return_value = False
-        mock_sys.platform = 'linux'
         plugin = ovs.OvsPlugin.load(constants.PLUGIN_NAME)
         plugin.plug(self.vif_ovs_hybrid, self.instance)
         ensure_bridge.assert_has_calls(calls['ensure_bridge'])
@@ -333,38 +330,10 @@ class PluginTest(testtools.TestCase):
         update_veth_pair.assert_has_calls(calls['update_veth_pair'])
         _update_vif_port.assert_has_calls(calls['_update_vif_port'])
 
-    @mock.patch.object(ovsdb_lib.BaseOVS, 'ensure_ovs_bridge')
-    @mock.patch.object(ovs.OvsPlugin, '_create_vif_port')
-    @mock.patch.object(ip_lib, 'exists', return_value=False)
-    @mock.patch.object(ovs, 'sys')
-    def _check_plug_ovs_windows(self, vif, mock_sys, mock_exists,
-                                _create_vif_port, ensure_ovs_bridge):
-        dp_type = ovs.OvsPlugin._get_vif_datapath_type(vif)
-        calls = {
-            'exists': [mock.call(vif.id)],
-            '_create_vif_port': [mock.call(vif, vif.id, self.instance)],
-            'ensure_ovs_bridge': [mock.call('br0', dp_type)]
-        }
-
-        mock_sys.platform = constants.PLATFORM_WIN32
-        plugin = ovs.OvsPlugin.load(constants.PLUGIN_NAME)
-        plugin.plug(vif, self.instance)
-        mock_exists.assert_has_calls(calls['exists'])
-        _create_vif_port.assert_has_calls(calls['_create_vif_port'])
-        ensure_ovs_bridge.assert_has_calls(calls['ensure_ovs_bridge'])
-
-    def test_plug_ovs_windows(self):
-        self._check_plug_ovs_windows(self.vif_ovs)
-
-    def test_plug_ovs_bridge_windows(self):
-        self._check_plug_ovs_windows(self.vif_ovs_hybrid)
-
     @mock.patch.object(ovsdb_lib.BaseOVS, 'delete_ovs_bridge')
-    @mock.patch.object(ovs, 'sys')
     @mock.patch.object(ovs.OvsPlugin, '_unplug_vif_generic')
-    def test_unplug_ovs_port_bridge_false(self, unplug, mock_sys,
+    def test_unplug_ovs_port_bridge_false(self, unplug,
                                           delete_ovs_bridge):
-        mock_sys.platform = 'linux'
         plugin = ovs.OvsPlugin.load(constants.PLUGIN_NAME)
         with mock.patch.object(plugin.config, 'per_port_bridge', False):
             plugin.unplug(self.vif_ovs, self.instance)
@@ -372,11 +341,9 @@ class PluginTest(testtools.TestCase):
         delete_ovs_bridge.assert_not_called()
 
     @mock.patch.object(ovsdb_lib.BaseOVS, 'delete_ovs_bridge')
-    @mock.patch.object(ovs, 'sys')
     @mock.patch.object(ovs.OvsPlugin, '_unplug_port_bridge')
-    def test_unplug_ovs_port_bridge_true(self, unplug, mock_sys,
+    def test_unplug_ovs_port_bridge_true(self, unplug,
                                          delete_ovs_bridge):
-        mock_sys.platform = 'linux'
         plugin = ovs.OvsPlugin.load(constants.PLUGIN_NAME)
         with mock.patch.object(plugin.config, 'per_port_bridge', True):
             plugin.unplug(self.vif_ovs, self.instance)
@@ -394,8 +361,7 @@ class PluginTest(testtools.TestCase):
     @mock.patch.object(ovsdb_lib.BaseOVS, 'delete_ovs_bridge')
     @mock.patch.object(ovsdb_lib.BaseOVS, 'delete_ovs_vif_port')
     @mock.patch.object(linux_net, 'delete_bridge')
-    @mock.patch.object(ovs, 'sys')
-    def test_unplug_ovs_bridge(self, mock_sys, delete_bridge,
+    def test_unplug_ovs_bridge(self, delete_bridge,
                                delete_ovs_vif_port, delete_ovs_bridge):
         calls = {
             'delete_bridge': [mock.call('qbrvif-xxx-yyy', 'qvbb679325f-ca')],
@@ -403,32 +369,12 @@ class PluginTest(testtools.TestCase):
                 'br0', 'qvob679325f-ca', qos_type='linux-noop'
             )]
         }
-        mock_sys.platform = 'linux'
         plugin = ovs.OvsPlugin.load(constants.PLUGIN_NAME)
         plugin.unplug(self.vif_ovs_hybrid, self.instance)
         delete_bridge.assert_has_calls(calls['delete_bridge'])
         delete_ovs_vif_port.assert_has_calls(calls['delete_ovs_vif_port'])
         delete_ovs_bridge.assert_not_called()
 
-    @mock.patch.object(ovsdb_lib.BaseOVS, 'delete_ovs_vif_port')
-    @mock.patch.object(ovs, 'sys')
-    def _check_unplug_ovs_windows(self, vif, mock_sys, delete_ovs_vif_port):
-        mock_sys.platform = constants.PLATFORM_WIN32
-        plugin = ovs.OvsPlugin.load(constants.PLUGIN_NAME)
-        plugin.unplug(vif, self.instance)
-        delete_ovs_vif_port.assert_called_once_with('br0', vif.id,
-                                                    delete_netdev=False)
-
-    @mock.patch.object(ovsdb_lib.BaseOVS, 'delete_ovs_bridge')
-    def test_unplug_ovs_windows(self, delete_ovs_bridge):
-        self._check_unplug_ovs_windows(self.vif_ovs)
-        delete_ovs_bridge.assert_not_called()
-
-    @mock.patch.object(ovsdb_lib.BaseOVS, 'delete_ovs_bridge')
-    def test_unplug_ovs_bridge_windows(self, delete_ovs_bridge):
-        self._check_unplug_ovs_windows(self.vif_ovs_hybrid)
-        delete_ovs_bridge.assert_not_called()
-
     @mock.patch.object(ovs.OvsPlugin, '_create_vif_port')
     def test_plug_ovs_vhostuser(self, _create_vif_port):
         dp_type = ovs.OvsPlugin._get_vif_datapath_type(self.vif_vhostuser)

os_vif/internal/ip/windows/impl_netifaces.py

@@ -1,47 +0,0 @@
-#    Derived from: neutron/agent/windows/ip_lib.py
-#
-#    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.
-
-import netifaces
-
-from oslo_log import log as logging
-
-from os_vif import exception
-from os_vif.internal.ip import ip_command
-
-LOG = logging.getLogger(__name__)
-
-
-class Netifaces(ip_command.IpCommand):
-
-    def exists(self, device):
-        """Return True if the device exists in the namespace."""
-        try:
-            return bool(netifaces.ifaddresses(device))
-        except ValueError:
-            LOG.warning("The device does not exist on the system: %s", device)
-            return False
-        except OSError:
-            LOG.error("Failed to get interface addresses: %s", device)
-            return False
-
-    def set(self, device, check_exit_code=None, state=None, mtu=None,
-            address=None, promisc=None, master=None):
-        exception.NotImplementedForOS(function='ip.set', os='Windows')
-
-    def add(self, device, dev_type, check_exit_code=None, peer=None, link=None,
-            vlan_id=None):
-        exception.NotImplementedForOS(function='ip.add', os='Windows')
-
-    def delete(self, device, check_exit_code=None):
-        exception.NotImplementedForOS(function='ip.delete', os='Windows')

os_vif/tests/unit/internal/ip/windows/test_impl_netifaces.py

@@ -1,46 +0,0 @@
-#    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.
-
-from unittest import mock
-
-import netifaces
-
-from os_vif.internal.ip.windows import impl_netifaces as ip_lib
-from os_vif.tests.unit import base
-
-
-class TestIPDevice(base.TestCase):
-
-    def setUp(self):
-        super(TestIPDevice, self).setUp()
-        self.device_name = 'test_device'
-        self.mock_log = mock.patch.object(ip_lib, "LOG").start()
-        self.ip_lib = ip_lib.Netifaces()
-
-    @mock.patch.object(netifaces, 'ifaddresses', return_value=True)
-    def test_exists(self, mock_ifaddresses):
-        self.assertTrue(self.ip_lib.exists(self.device_name))
-        mock_ifaddresses.assert_called_once_with(self.device_name)
-
-    @mock.patch.object(netifaces, 'ifaddresses', side_effect=ValueError())
-    def test_exists_not_found(self, mock_ifaddresses):
-        self.assertFalse(self.ip_lib.exists(self.device_name))
-        mock_ifaddresses.assert_called_once_with(self.device_name)
-        self.mock_log.warning.assert_called_once_with(
-            "The device does not exist on the system: %s", self.device_name)
-
-    @mock.patch.object(netifaces, 'ifaddresses', side_effect=OSError())
-    def test_exists_os_error_exception(self, mock_ifaddresses):
-        self.assertFalse(self.ip_lib.exists(self.device_name))
-        mock_ifaddresses.assert_called_once_with(self.device_name)
-        self.mock_log.error.assert_called_once_with(
-            "Failed to get interface addresses: %s", self.device_name)

os_vif-3.7.0.dist-info/pbr.json

@@ -1 +0,0 @@
-{"git_version": "bb3e3da", "is_release": true}