[libvirt] [PATCH v2] esx: Simplify VI (vSphere) API and VMware product version handling

Matthias Bolte matthias.bolte at googlemail.com
Sat Nov 8 15:29:32 UTC 2014


2014-10-24 20:09 GMT+02:00 Eric Blake <eblake at redhat.com>:
> On 10/18/2014 04:08 PM, Matthias Bolte wrote:
>> Store version numbers in this format
>>
>> version = 1000000 * major + 1000 * minor + micro
>>
>> produced by virParseVersionString instead of dedicated enums.
>>
>> Split the complex esxVI_ProductVersion enum into a simpler
>> esxVI_ProductLine enum and a product version number.
>>
>> Relax API and product version number checks to accept everything that
>> is equal or greater than the supported minimum version. VMware ESX
>> went through 3 major versions and the vSphere API always stayed
>> backward compatible. This commit assumes that this will also be true
>> for future VMware ESX versions.
>>
>> Also reword error messages in esxConnectTo* to say what was expected
>> and what was found instead (suggested by Richard W.M. Jones).
>> ---
>>
>> v2: - Fix copy&paste accident that replaced hostname with conn->uri->server
>>       in esxConnectToVCenter
>>     - Fix and improve commit message
>>
>>  src/esx/esx_driver.c |  59 +++++---------
>>  src/esx/esx_vi.c     | 215 ++++++++++++++++++++++++++-------------------------
>>  src/esx/esx_vi.h     |  55 +++----------
>>  3 files changed, 140 insertions(+), 189 deletions(-)
>>
>
> ACK

Thanks, pushed.

-- 
Matthias Bolte
http://photron.blogspot.com




More information about the libvir-list mailing list