ISER fails to attach a volume: unrecognized option 'direct'

[Gil Meir]

Attached is the relevant error in the log.

The cause is probably a patch added on MOS side: the module /usr/lib/python2.7/dist-packages/cinder/brick/iscsi/iscsi.py has in class ISERTgtAdim an incorrent keywork for target command: "bsoflags %(bsoflags)s" instead of "%(bsoflags)s" - the first "bsoflags" is unnecessary.
The patch was done OK on other classes in this module, is broke only ISER.

A fix patch will be attached next.

[Mike Scherbakov (mihgen)]

Guys,
please take a look if we can cover this by simple unit test. Also, as we are in HCF mode for 5.1.1, we need to understand impact of this bug and possible regression impact of the fix, so to know how many of our acceptance tests we will have to re-run.

[OSCI Robot (oscirobot)]

RPM package cinder has been built for project openstack/cinder
Package version == 2014.1.3, package release == fuel5.1.1.mira2.git.599aba4.2911b36

Changeset: https://review.fuel-infra.org/1133
project: openstack/cinder
branch: openstack-ci/fuel-5.1.1/2014.1.1
author: Ivan Kolodyazhny
committer: Ivan Kolodyazhny
subject: Fix ISER to attach volume
status: patchset-created

Files placed on repository:
openstack-cinder-2014.1.3-fuel5.1.1.mira2.git.599aba4.2911b36.noarch.rpm
openstack-cinder-doc-2014.1.3-fuel5.1.1.mira2.git.599aba4.2911b36.noarch.rpm
python-cinder-2014.1.3-fuel5.1.1.mira2.git.599aba4.2911b36.noarch.rpm

NOTE: Changeset is not merged, created temporary package repository.
RPM repository URL: http://osci-obs.vm.mirantis.net:82/centos-fuel-5.1.1-stable-1133/centos

[OSCI Robot (oscirobot)]

DEB package cinder has been built for project openstack/cinder
Package version == 2014.1.3, package release == fuel5.1.1~mira2+git.599aba4.2911b36

Changeset: https://review.fuel-infra.org/1133
project: openstack/cinder
branch: openstack-ci/fuel-5.1.1/2014.1.1
author: Ivan Kolodyazhny
committer: Ivan Kolodyazhny
subject: Fix ISER to attach volume
status: patchset-created

Files placed on repository:
cinder-api_2014.1.3-fuel5.1.1~mira2+git.599aba4.2911b36_all.deb
cinder-backup_2014.1.3-fuel5.1.1~mira2+git.599aba4.2911b36_all.deb
cinder-common_2014.1.3-fuel5.1.1~mira2+git.599aba4.2911b36_all.deb
cinder-scheduler_2014.1.3-fuel5.1.1~mira2+git.599aba4.2911b36_all.deb
cinder-volume_2014.1.3-fuel5.1.1~mira2+git.599aba4.2911b36_all.deb
python-cinder_2014.1.3-fuel5.1.1~mira2+git.599aba4.2911b36_all.deb

NOTE: Changeset is not merged, created temporary package repository.
DEB repository URL: http://osci-obs.vm.mirantis.net:82/ubuntu-fuel-5.1.1-stable-1133/ubuntu

[Gil Meir (gilmeir-d)]

+1
I have problems logging into gerrit, but this fix works.

[Dmitry Borodaenko (angdraug)]

Please confirm if this problem exists in 6.0/juno.

[Ivan Kolodyazhny (e0ne)]

This issue doesn't affect 6.0

[OSCI Robot (oscirobot)]

RPM package cinder has been built for project openstack/cinder
Package version == 2014.1.3, package release == fuel5.1.1.mira2

Changeset: https://review.fuel-infra.org/1133
project: openstack/cinder
branch: openstack-ci/fuel-5.1.1/2014.1.1
author: Ivan Kolodyazhny
committer: Ivan Kolodyazhny
subject: Fix ISER to attach volume
status: change-merged

Files placed on repository:
openstack-cinder-2014.1.3-fuel5.1.1.mira2.noarch.rpm
openstack-cinder-doc-2014.1.3-fuel5.1.1.mira2.noarch.rpm
python-cinder-2014.1.3-fuel5.1.1.mira2.noarch.rpm

Changeset merged. Package placed on primary repository
RPM repository URL: http://osci-obs.vm.mirantis.net:82/centos-fuel-5.1.1-stable/centos

[OSCI Robot (oscirobot)]

DEB package cinder has been built for project openstack/cinder
Package version == 2014.1.3, package release == fuel5.1.1~mira2

Changeset: https://review.fuel-infra.org/1133
project: openstack/cinder
branch: openstack-ci/fuel-5.1.1/2014.1.1
author: Ivan Kolodyazhny
committer: Ivan Kolodyazhny
subject: Fix ISER to attach volume
status: change-merged

Files placed on repository:
cinder-api_2014.1.3-fuel5.1.1~mira2_all.deb
cinder-backup_2014.1.3-fuel5.1.1~mira2_all.deb
cinder-common_2014.1.3-fuel5.1.1~mira2_all.deb
cinder-scheduler_2014.1.3-fuel5.1.1~mira2_all.deb
cinder-volume_2014.1.3-fuel5.1.1~mira2_all.deb
python-cinder_2014.1.3-fuel5.1.1~mira2_all.deb

Changeset merged. Package placed on primary repository
DEB repository URL: http://osci-obs.vm.mirantis.net:82/ubuntu-fuel-5.1.1-stable/ubuntu

[Dmitry Borodaenko (angdraug)]

Downgraded to High in line with https://wiki.openstack.org/wiki/Fuel/How_to_contribute#Confirm_and_triage_bugs -- "specific hardware, configurations, or components are unusable and there's no workaround".

[Dmitry Borodaenko (angdraug)]

Status reset back to In Progress: it's not Fix Committed until it lands in the mirrors.

[Bogdan Dobrelya (bogdando)]

Why we have a high bug in progress for 5.1.1 target during HCF?

[Dmitry Mescheryakov (dmitrymex)]

Dmitry, Bodgan, the fix was landed into mirrors and got into ISO #46, hence it is fix committed now

[Dmitry Mescheryakov (dmitrymex)]

I've added 6.0.x series and marked bug as Invalid here to indicate that 6.0 is not affected