Rootwrap for cinder-rtstool not working on Ubuntu 14.04

Bug #1402076 reported by John Griffith
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Cinder
Fix Released
High
John Griffith

Bug Description

When attempting to attach a volume using lioadm iscsi_helper, the attach fails because the rootwrap filter isn't parsing out cinder-rtstool correctly on 14.04.

Following exception when running:
ProcessExecutionError(u"Unexpected error while running command.\nCommand: sudo cinder-rootwrap /etc/cinder/rootwrap.conf cinder-rtstool create /dev/stack-volumes-lvmdriver-1/volume-536821d6-7cbf-4cfd-a2e0-d2d1b5e984b7 iqn.2010-10.org.openstack:volume-536821d6-7cbf-4cfd-a2e0-d2d1b5e984b7 sujBf55eHQ5rY98SuG3B yZxzi3tesXcG9cXA\nExit code: 96\nStdout: u''\nStderr: u'/usr/local/bin/cinder-rootwrap: Executable not found: cinder-rtstool (filter match = cinder-rtstool)\\n'",)

Changed in cinder:
assignee: nobody → John Griffith (john-griffith)
Jay Bryant (jsbryant)
Changed in cinder:
importance: Undecided → High
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/143163

Changed in cinder:
status: New → In Progress
Mike Perez (thingee)
Changed in cinder:
milestone: none → kilo-2
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to cinder (master)

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

commit c85bd876d3ae4b6e29bb040a49e43d3ba46b6039
Author: John Griffith <email address hidden>
Date: Fri Dec 19 19:32:02 2014 +0000

    rtstool on Ubuntu installs in /usr/local/bin

    The cinder-rtstool which is required to use LIO iscsi_helper
    installs in /usr/local/bin on Ubuntu and /usr/bin on RHEL
    variants. Currently the rootwrap.conf only has /usr/bin
    configured, as a result LIO can't be used on Ubuntu without
    modifying the rootwrap file.

    This patch just adds /usr/local/bin to the rootwrap.conf; Honestly
    I'm not sure if this is good, bad or doesn't matter. May be able
    to address this via documentation, or upstream in the packaging of
    cinder-rtstool instead? Not sure what's preferred.

    Change-Id: I577e10df25181c7d4ca8189b10640a8e6e39b047
    Closes-Bug: #1402076

Changed in cinder:
status: In Progress → Fix Committed
Thierry Carrez (ttx)
Changed in cinder:
status: Fix Committed → Fix Released
Revision history for this message
Eric Harney (eharney) wrote :

As far as I can tell, the Ubuntu Cinder packages install cinder-rtstool to /usr/bin/. How were you installing this?

IMO adding /usr/local/bin/ to the default config is not appropriate and this sounds like a packaging/deployment bug.

Revision history for this message
Eric Harney (eharney) wrote :
Thierry Carrez (ttx)
Changed in cinder:
milestone: kilo-2 → 2015.1.0
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on cinder (master)

Change abandoned by Mike Perez (<email address hidden>) on branch: master
Review: https://review.openstack.org/153782
Reason: No activity for over a month.

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

Other bug subscribers

Remote bug watches

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