]> review.fuel-infra Code Review - openstack-build/cinder-build.git/commit
VMware: Usng RetrvProprtisEx & does multi ESX scan
authorKartik Bommepally <kbommepally@vmware.com>
Mon, 16 Sep 2013 10:23:23 +0000 (03:23 -0700)
committerKartik Bommepally <kbommepally@vmware.com>
Mon, 30 Sep 2013 05:05:43 +0000 (22:05 -0700)
commit010fe685e50a3680d8c8b9b6602677df90e815b3
tree98a389febf5b6778635a4bd315202b5a8e5fe00f
parent3680773c6dcee21da13e5e76e6796309581f97fb
VMware: Usng RetrvProprtisEx & does multi ESX scan

The existing code uses deprecated API RetrieveProperties for querying through
property collector. The public doc asks to use RetrievePropertiesEx instead.

RetrievePropertiesEx works by retrieving results in serveral batches rather than
retrieval in a single API call. This works well in environments containing
large number of VMs or ESX hosts.

There are three steps involved 1. Get first batch 2. Continue retrieving using
a token 3. Cancel retrieval when not needed

Batch size is user configurable, default is 100 results per batch.

The review also fixes a careless programming error leading to scanning only
1 ESX host under the vCenter server for datastores.

Fixes bug: 1227078
Fixes bug: 1229654
DocImpact

Change-Id: I85a6f6a7e2d764b9843efbd0dd347fba3ef19310
cinder/tests/test_vmware_vmdk.py
cinder/volume/drivers/vmware/vim_util.py
cinder/volume/drivers/vmware/vmdk.py
cinder/volume/drivers/vmware/volumeops.py
etc/cinder/cinder.conf.sample