DS8K: correct the usage for ssl method(SSLContext.wrap_socket)

Bug #1797847 reported by Peng Wang
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Cinder
Fix Released
Undecided
Peng Wang

Bug Description

In patch https://review.openstack.org/#/c/591608/, the usage for SSLContext.wrap_socket() is wrong, it causes the DS8K cinder driver can not be initialized. This patch will correct it.

Peng Wang (wangpww)
Changed in cinder:
assignee: nobody → IBM Storage (ibm-storage)
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/610453

Changed in cinder:
assignee: IBM Storage (ibm-storage) → Peng Wang (wangpww)
status: New → In Progress
Revision history for this message
Alon Marx (alonma) wrote : AUTO: Alon Marx is out of the office (returning 23/10/2018)

I am out of the office until 23/10/2018.

During this time I will have occasional email access, and very limited
phone access.

For any urgent matters please contact Ran Harel.

Note: This is an automated response to your message "[Bug 1797847] [NEW]
DS8K: correct the usage for ssl method(SSLContext.wrap_socket)" sent on
10/15/18 8:01:36 AM.

This is the only notification you will receive while this person is away.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to cinder (master)

Reviewed: https://review.openstack.org/610453
Committed: https://git.openstack.org/cgit/openstack/cinder/commit/?id=b742179f332a13913c3beb90a59a5a85e417efe3
Submitter: Zuul
Branch: master

commit b742179f332a13913c3beb90a59a5a85e417efe3
Author: Peng Wang <email address hidden>
Date: Mon Oct 15 01:39:30 2018 -0700

    DS8K: correct the usage for ssl method(SSLContext.wrap_socket)

    Since Python 3.2 and 2.7.9, it is recommended to use the
    SSLContext.wrap_socket() instead of wrap_socket(). So the patch
    https://review.openstack.org/#/c/591608/ help to optimize it. But the
    usage of SSLContext.wrap_socket() is wrong, it caused DS8K cinder
    driver can not be initialized, so this patch will correct it.

    Change-Id: Ia015f3b48796dec79dd68b8d393e920a03dfa585
    Closes-Bug: 1797847

Changed in cinder:
status: In Progress → Fix Released
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/cinder 14.0.0.0rc1

This issue was fixed in the openstack/cinder 14.0.0.0rc1 release candidate.

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.