Comment 2 for bug 1038062

Revision history for this message
Akira Yoshiyama (yosshy) wrote : Re: [Bug 1038062] Re: TgtAdm is broken

I checked that commit. I think it's not one to fix the problem.

Try to restart tgt and check TID assignment.
2012/08/24 4:56 "John Griffith" <email address hidden>:

> I suspect that problem 'a' will be addressed by
> https://review.openstack.org/#/c/11899/
>
> --
> You received this bug notification because you are subscribed to the bug
> report.
> https://bugs.launchpad.net/bugs/1038062
>
> Title:
> TgtAdm is broken
>
> Status in Cinder:
> New
>
> Bug description:
> Summary
> =======
>
> A patch for Bug #1011159 (https://review.openstack.org/10417) breaks
> ISCSIDriver with TgtAdm.
>
> a) New volumes won't be exported.
> b) Deleting volumes will be failed.
>
> Reason
> ======
>
> ISCSIDriver expects tid of a volume is specified properly when it was
> created and the driver use tid to delete it. The driver checks that tid
> exists before deleting it and doesn't free it if not found.
> But scsi-target-utils 1.0.17 doesn't have an option to specify tid in
> <target xxx> tag. Tid is numbered serially from 1. So b) happens.
>
> Currently, I can't find the reason of a). It looks exported but Nova
> can't
> mount it.
>
> I tried to revert this patch and Nova and Cinder worked perfectly.
>
> Solutions
> =========
>
> There are 2 solutions for b).
>
> 1. Use newer scsi-target-utils. It has an option "controller_tid" to
> specify tid for a target.
> 2. Add an argument for show_target() to specify iqn and use it in TgtAdm.
> There is no way for tgtadm/tgt-admin to check a target with the iqn
> but
> we can do "tgt-admin -s | grep iqn...".
>
> To manage notifications about this bug go to:
> https://bugs.launchpad.net/cinder/+bug/1038062/+subscriptions
>