]> review.fuel-infra Code Review - openstack-build/cinder-build.git/commit
VMware: Fix initialization of datastore selector
authorVipin Balachandran <vbala@vmware.com>
Mon, 13 Oct 2014 14:04:36 +0000 (19:34 +0530)
committerThomas Goirand <thomas@goirand.fr>
Sun, 14 Dec 2014 09:18:30 +0000 (09:18 +0000)
commit1b49739a3905d6394586750e72afd1b88d0b98a5
tree62199b389e4385b45ac868ed2ea4351a68124948
parentcd1c0cccbb6a525a5558ef0c354c19cbf6232af4
VMware: Fix initialization of datastore selector

The WSDL URL of storage policy service is determined and a session is
created using it in do_setup(). This session is later used to initialize
the datastore selector property (ds_sel), which uses the session for all
storage policy related API calls.

After commit a8fa3ceb1e72bac2ab67f569a2ca009f995f59fd (Integrate
OSprofiler and Cinder), the properties defined in vmdk module are called
before do_setup(). As a result, the ds_sel (datastore selector) property
is initialized with a session instance containing a 'None' PBM (storage
policy service) WSDL URL. This results in failures of all storage policy
related APIs invoked using datastore selector. This patch fixes the
problem by re-initializing the property in do_setup().

Change-Id: Ibdf8b23f9e215000cf9053b81d374066fabd6851
Closes-Bug: #1380675
(cherry picked from commit 6ac6225e72bde92f66da8e92c563c140471b949b)
cinder/tests/test_vmware_vmdk.py
cinder/volume/drivers/vmware/vmdk.py