Revert "mark oslo.vmware as optional dependency"

This reverts commit e67adc385c.

Adding optional dependency tracking would be nice,
but Cinder is not yet in a place to do this.

- Cinder supports tox 1.6 (minversion in tox.ini)
- This functionality does not work on 1.6, requires 1.7.  Tox
  fails to build a test environment if using tox 1.6.
- We have decided we can't move to tox 1.7 for now.
  ( https://review.openstack.org/#/c/211614/ )

This reverts commit 478e8e970e.
- Need to remove oslo.vmware from test-requirements.txt now, as
well.

Closes-Bug: #1484035

Change-Id: Icff52508ff1cd67e8c17964c840d3a5cdcfbbce3

cinder/volume/drivers/vmware/vmdk.py

@@ -32,16 +32,11 @@ from oslo_log import log as logging
 from oslo_utils import excutils
 from oslo_utils import units
 from oslo_utils import uuidutils
-try:
-    import oslo_vmware
-except ImportError:
-    oslo_vmware = None
-else:
-    from oslo_vmware import api
-    from oslo_vmware import exceptions
-    from oslo_vmware import image_transfer
-    from oslo_vmware import pbm
-    from oslo_vmware import vim_util
+from oslo_vmware import api
+from oslo_vmware import exceptions
+from oslo_vmware import image_transfer
+from oslo_vmware import pbm
+from oslo_vmware import vim_util
 import six
 
 from cinder import exception
@@ -278,11 +273,6 @@ class VMwareEsxVmdkDriver(driver.VolumeDriver):
         :param context: Context information
         """
 
-        if oslo_vmware is None:
-            msg = _("Missing 'oslo_vmware' python module, ensure the library"
-                    " is installed and available.")
-            raise exception.VolumeDriverException(message=msg)
-
         # Throw error if required parameters are not set.
         required_params = ['vmware_host_ip',
                            'vmware_host_username',

requirements.txt

@@ -50,4 +50,5 @@ stevedore>=1.5.0 # Apache-2.0
 suds-jurko>=0.6
 WebOb>=1.2.3
 oslo.i18n>=1.5.0 # Apache-2.0
+oslo.vmware>=1.16.0 # Apache-2.0
 os-brick>=0.3.2 # Apache-2.0

setup.cfg

@@ -25,10 +25,6 @@ setup-hooks =
 packages =
     cinder
 
-[extras]
-vmdk =
-  oslo.vmware>=1.16.0 # Apache-2.0
-
 [entry_points]
 cinder.scheduler.filters =
     AvailabilityZoneFilter = cinder.openstack.common.scheduler.filters.availability_zone_filter:AvailabilityZoneFilter

test-requirements.txt

@@ -11,7 +11,6 @@ discover
 fixtures>=1.3.1
 mock>=1.2
 mox3>=0.7.0
-oslo.vmware>=1.16.0 # Apache-2.0
 PyMySQL>=0.6.2 # MIT License
 psycopg2
 oslotest>=1.10.0 # Apache-2.0

tox.ini

@@ -13,7 +13,6 @@ install_command = pip install {opts} {packages}
 
 deps = -r{toxinidir}/requirements.txt
        -r{toxinidir}/test-requirements.txt
-       .[vmdk]
 
 # By default ostestr will set concurrency
 # to ncpu, to specify something else use