Comment 6 for bug 1370226

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

So, as I am looking at this more closely. It seems that we do have a bug in the process here right now. Given where we are in development, it seems like changing _get_host_device for LibvertISCSIVolumeDriver to do something like this:

    def _get_host_device(self, iser_properties):
        time.sleep(1)
        host_device = None
        device = ("ip-%s-iscsi-%s-lun-%s" %
                  (iser_properties['target_portal'],
                   iser_properties['target_iqn'],
                   iser_properties.get('target_lun', 0)))
        look_for_device = glob.glob('/dev/disk/by-path/*%s' % device)
        if look_for_device:
            host_device = look_for_device[0]
        return host_device

Is a low risk change that will cover more configurations.

As for your suggestion it looks like the path in sys/class/scsi_disk is <host>:<channel>:<id>:<lun> . Figuring out a way to put that together for Kilo, perhaps, sounds like a better long term solution.