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

Bug #1397895 reported by Gil Meir on 2014-12-01
10
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Fuel for OpenStack
High
Ivan Kolodyazhny
5.1.x
High
Ivan Kolodyazhny
6.0.x
High
Ivan Kolodyazhny

Bug Description

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.

Gil Meir (gilmeir-d) wrote :
Gil Meir (gilmeir-d) wrote :
Changed in fuel:
importance: Undecided → Critical
milestone: none → 5.1.1
assignee: nobody → MOS Cinder (mos-cinder)
Ivan Kolodyazhny (e0ne) on 2014-12-01
Changed in fuel:
assignee: MOS Cinder (mos-cinder) → Ivan Kolodyazhny (e0ne)
Mike Scherbakov (mihgen) wrote :

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) wrote :

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

Ivan Kolodyazhny (e0ne) on 2014-12-01
Changed in fuel:
status: New → In Progress
OSCI Robot (oscirobot) wrote :

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) wrote :

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

Dmitry Borodaenko (angdraug) wrote :

Please confirm if this problem exists in 6.0/juno.

Ivan Kolodyazhny (e0ne) wrote :

This issue doesn't affect 6.0

no longer affects: fuel/6.0.x
OSCI Robot (oscirobot) wrote :

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) wrote :

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) wrote :

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".

Changed in fuel:
importance: Critical → High
Dmitry Borodaenko (angdraug) wrote :

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

Bogdan Dobrelya (bogdando) wrote :

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

Ivan Kolodyazhny (e0ne) on 2014-12-02
tags: added: cinder

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

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

To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers

Patches