VMware: expose boolean insecure for certificate verification

Bug #1451633 reported by Eric Brown
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Cinder
Fix Released
Low
Eric Brown

Bug Description

Commit Ida730db66b154a4d445f7a91bccb9ca5b5a26f5e introduced certificate verification but did not expose the option to turn
on/off verification using the insecure boolean.

 It's actually more beneficial to have trusted certificates loaded into the operating systems ca-certificate store and verify using that instead of specifying a location to a cert file.

Therefore, the 'insecure' option of oslo_vmware should be exposed in the cinder vmware driver.

Tags: drivers vmware
Eric Brown (ericwb)
Changed in cinder:
assignee: nobody → Eric Brown (ericwb)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to cinder (master)

Fix proposed to branch: master
Review: https://review.openstack.org/179991

Changed in cinder:
status: New → In Progress
Mike Perez (thingee)
tags: added: drivers vmware
Changed in cinder:
importance: Undecided → Low
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to cinder (master)

Reviewed: https://review.openstack.org/179991
Committed: https://git.openstack.org/cgit/openstack/cinder/commit/?id=9240fef1b29fefd125a3e5c0066fe9a89519437c
Submitter: Jenkins
Branch: master

commit 9240fef1b29fefd125a3e5c0066fe9a89519437c
Author: Eric Brown <email address hidden>
Date: Mon May 4 18:42:48 2015 -0700

    VMware: insecure option should be exposed

    Commit Ida730db66b154a4d445f7a91bccb9ca5b5a26f5e introduced
    certificate verification but did not expose the option to turn
    on/off verification using the insecure boolean of oslo.vmware.

    DocImpact

    Change-Id: I834c9fb407c8790dab14db0308f7e7f1d551669e
    Closes-Bug: #1451633

Changed in cinder:
status: In Progress → Fix Committed
Thierry Carrez (ttx)
Changed in cinder:
milestone: none → liberty-1
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in cinder:
milestone: liberty-1 → 7.0.0
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.