Fuel for OpenStack

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

  1. Series 5.1.x
  2. Bug #1397895
Bug #1397895 reported by Gil Meir
10
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Fuel for OpenStack
Invalid
High
Ivan Kolodyazhny
Fuel for OpenStack 6.0
5.1.x
Fix Committed
High
Ivan Kolodyazhny
Fuel for OpenStack 5.1.1
6.0.x
Invalid
High
Ivan Kolodyazhny
Fuel for OpenStack 6.0

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.

Revision history for this message
Gil Meir (gilmeir-d) wrote :
Revision history for this message
Gil Meir (gilmeir-d) wrote :
Evgeniya Shumakher (eshumakher)
Changed in fuel:
importance: Undecided → Critical
milestone: none → 5.1.1
assignee: nobody → MOS Cinder (mos-cinder)
Ivan Kolodyazhny (e0ne)
Changed in fuel:
assignee: MOS Cinder (mos-cinder) → Ivan Kolodyazhny (e0ne)
Revision history for this message
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.

Revision history for this message
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)
Changed in fuel:
status: New → In Progress
Revision history for this message
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

Revision history for this message
Gil Meir (gilmeir-d) wrote :

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

Revision history for this message
Dmitry Borodaenko (angdraug) wrote :

Please confirm if this problem exists in 6.0/juno.

Revision history for this message
Ivan Kolodyazhny (e0ne) wrote :

This issue doesn't affect 6.0

no longer affects: fuel/6.0.x
Revision history for this message
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

Revision history for this message
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

Revision history for this message
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
Revision history for this message
Dmitry Borodaenko (angdraug) wrote :

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

Revision history for this message
Bogdan Dobrelya (bogdando) wrote :

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

Ivan Kolodyazhny (e0ne)
tags: added: cinder
Revision history for this message
Dmitry Mescheryakov (dmitrymex) wrote :

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

Revision history for this message
Dmitry Mescheryakov (dmitrymex) wrote :

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  
Everyone can see this information.
Subscribing...

Other bug subscribers

Patches

Remote bug watches

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