xiv_ds8k driver fails chap support due to missing parameter

Bug #1288816 reported by Alon Marx
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Cinder
Fix Released
High
Alon Marx

Bug Description

CHAP encryption is supported in the xiv_ds8k cinder driver, but the value specified in the cinder conf file is not passed and the feature does not work.

The fix is minimal (patch attached).

Revision history for this message
Alon Marx (alonma) wrote :
Alon Marx (alonma)
Changed in cinder:
assignee: nobody → Alon Marx (alonma)
Revision history for this message
Avishay Traeger (avishay-il) wrote :

will you submit the patch?

Jay Bryant (jsbryant)
Changed in cinder:
importance: Undecided → High
milestone: none → icehouse-rc1
Revision history for this message
Alon Marx (alonma) wrote :

We're taking this patch back - changing some text - and should submit it in a day or two. 10x

Revision history for this message
Jay Bryant (jsbryant) wrote :

I chatted with Alon and he will submit the next patch via the gerrit workflow process.

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/81878

Changed in cinder:
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to cinder (master)

Reviewed: https://review.openstack.org/81878
Committed: https://git.openstack.org/cgit/openstack/cinder/commit/?id=5119d323650f97f12e12fdb6a2a80e46739629fc
Submitter: Jenkins
Branch: master

commit 5119d323650f97f12e12fdb6a2a80e46739629fc
Author: alonma <email address hidden>
Date: Thu Mar 20 19:41:56 2014 +0200

    Adds xiv_chap to xiv/ds8k driver configuration

    In order to support CHAP, xiv_chap was added to
    the driver configuration. Valid values are disabled
    and enabled (default: disabled)

    Change-Id: I0c68f581ad78a87a506b00ab308806f4a8fa78a8
    DocImpact
    Closes-Bug: #1288816

Changed in cinder:
status: In Progress → Fix Committed
Thierry Carrez (ttx)
Changed in cinder:
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in cinder:
milestone: icehouse-rc1 → 2014.1
Revision history for this message
Alon Marx (alonma) wrote : AUTO: Alon Marx is out of the office (returning 04/20/2014)

I am out of the office until 04/20/2014.

I will be out of the office August 9 to August 12. I will have no email
connectivity and no phone connectivity.
For any urgent matters please contact Ohad Atia.

Note: This is an automated response to your message "[Bug 1288816] Re:
xiv_ds8k driver fails chap support due to missing parameter" sent on
17/04/2014 12:35:15.

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

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.