LioAdm object has no attribute 'update_iscsi_target'

Bug #1273480 reported by Eric Harney
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Cinder
Fix Released
Undecided
Eric Harney

Bug Description

update_iscsi_target was implemented for TgtAdm but does not exist for the IetAdm or LioAdm classes. Needs to at least have a stub added to the TargetAdmin class.

When using iscsi_helper="lioadm":

2014-01-27 18:24:57.884 ERROR cinder.volume.manager [req-52b8e9f2-d862-4054-b029-bcb374485b24 c7cfb70e637e40d3
bb91b7dacbcf26fa dfc65160e685427385cb3c22478b56ef] Unable to fetch connection information from backend: 'LioAd
m' object has no attribute 'update_iscsi_target'
2014-01-27 18:24:57.885 ERROR cinder.openstack.common.rpc.amqp [req-52b8e9f2-d862-4054-b029-bcb374485b24 c7cfb70e637e40d3bb91b7dacbcf26fa dfc65160e685427385cb3c22478b56ef] Exception during message handling
2014-01-27 18:24:57.885 TRACE cinder.openstack.common.rpc.amqp Traceback (most recent call last):
2014-01-27 18:24:57.885 TRACE cinder.openstack.common.rpc.amqp File "/opt/stack/cinder/cinder/openstack/common/rpc/amqp.py", line 441, in _process_data
2014-01-27 18:24:57.885 TRACE cinder.openstack.common.rpc.amqp **args)
2014-01-27 18:24:57.885 TRACE cinder.openstack.common.rpc.amqp File "/opt/stack/cinder/cinder/openstack/common/rpc/dispatcher.py", line 148, in dispatch
2014-01-27 18:24:57.885 TRACE cinder.openstack.common.rpc.amqp return getattr(proxyobj, method)(ctxt, **kwargs)
2014-01-27 18:24:57.885 TRACE cinder.openstack.common.rpc.amqp File "/opt/stack/cinder/cinder/volume/manager.py", line 755, in initialize_connection
2014-01-27 18:24:57.885 TRACE cinder.openstack.common.rpc.amqp raise exception.VolumeBackendAPIException(data=err_msg)
2014-01-27 18:24:57.885 TRACE cinder.openstack.common.rpc.amqp VolumeBackendAPIException: Bad or unexpected response from the storage volume backend API: Unable to fetch connection information from backend: 'LioAdm' object has no attribute 'update_iscsi_target'
2014-01-27 18:24:57.885 TRACE cinder.openstack.common.rpc.amqp

Tags: brick
Eric Harney (eharney)
tags: added: brick
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/69506

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

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

commit a56ace019b2ed24181d8d8658d67f6557352cfeb
Author: Eric Harney <email address hidden>
Date: Mon Jan 27 18:46:26 2014 -0500

    Add create_iscsi_target stub to TargetAdmin

    This should be defined in TargetAdmin, as it is assumed to be
    present in all subclasses by callers.

    IetAdm and LioAdm may need additional work to behave as desired
    in all circumstances (such as volume extend), but this should
    at least stabilize things in the meantime.

    Closes-Bug: 1273480

    Change-Id: Iac1381286ea07982e6db7f7c7abf3f9100e4be50

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

Other bug subscribers

Related blueprints

Remote bug watches

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