LinstorBaseDriver use the wrong variable when getting the device path.

Bug #1811792 reported by RIES Alexis
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Cinder
Fix Released
Undecided
RIES Alexis

Bug Description

In linstordrv driver, use the wrong variable when getting the device path.

The "backingDevice" key is used instead of "devicePath".

This error causes an error while mounting the resource on the diskless compute node, and destroys the drbd metadata on backing device when converting an image, restore backup, etc...

RIES Alexis (riesalexis)
Changed in cinder:
assignee: nobody → RIES Alexis (riesalexis)
status: New → In Progress
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/630930

RIES Alexis (riesalexis)
Changed in cinder:
status: In Progress → Fix Committed
Revision history for this message
Woojay Poynter (w00jay) wrote :

Thank you Alexis for the patch. It looks valid, but I'm awaiting on the LINSTOR CI to finish. I will post review shortly after that.

Revision history for this message
Woojay Poynter (w00jay) wrote :

LINSTOR CI now shows test results passing and confirms the patch working properly.
( http://us.linbit.com:8080/CI-LINSTOR/30/763/ )
Thank you for the patch.

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

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

commit 64d4f717f3e898674fd248cac8876334d25dace5
Author: Alexis RIES <email address hidden>
Date: Tue Jan 15 14:37:43 2019 +0100

    LinstorDrv : Fix wrong key using when getting device path.

    Using the "devicePath" key instead of "backingDevice" when getting
    the device path corresponding to the volume

    Change-Id: Id9f9c2acf0cf7ef8ebe58eeea75dc611cdce9dd1
    Closes-Bug: #1811792

Changed in cinder:
status: Fix Committed → 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.