emc scaleio uses drv_cfg exe which isn't in volume.filters

Bug #1479530 reported by Matt Riedemann
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Cinder
Fix Released
Medium
Xing Yang

Bug Description

This came up here:

https://review.openstack.org/#/c/194454/13/etc/nova/rootwrap.d/compute.filters

The emc scaleio volume driver is running a command as root which isn't in cinder's default volume.filters:

https://github.com/openstack/cinder/blob/master/cinder/volume/drivers/emc/scaleio.py#L989

I have no idea how the emc scaleio third party CI is passing, they probably just hack it in as part of their setup, but this was the link to a passing job - but there are no logs:

http://publiclogs.emc.com/sio_ostack/EMC_SCALEIO/7

Matt Riedemann (mriedem)
tags: added: emc rootwrap scaleio
Xing Yang (xing-yang)
Changed in cinder:
assignee: nobody → Xing Yang (xing-yang)
milestone: none → liberty-3
status: New → In Progress
Revision history for this message
Matt Riedemann (mriedem) wrote :
Changed in cinder:
importance: Undecided → Medium
Revision history for this message
Xing Yang (xing-yang) wrote :

This one will be fixed once the following bugs are fixed:

os-brick needs to provide it's own rootwrap filters file:
https://bugs.launchpad.net/nova/+bug/1479842

ScaleIO driver should use os-brick connector:
https://bugs.launchpad.net/cinder/+bug/1486315

Thierry Carrez (ttx)
Changed in cinder:
milestone: liberty-3 → liberty-rc1
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/222342

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to cinder (master)

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

commit ed0a11ba0b395c2c5972568695c5118a44fed963
Author: Xing Yang <email address hidden>
Date: Thu Sep 10 15:31:39 2015 -0400

    Add ScaleIO Cinder driver commands

    ScaleIO Cinder driver commands are currently missing from
    Cinder's rootwrap filters. Since we are not using os-brick's
    rootwrap filters in Liberty, these missing commands need to be
    added to Cinder's rootwrap filters.

    Change-Id: Ia466a722a17730fa2104fa9a4b1898c6373d10ec
    Closes-Bug: #1479530

Changed in cinder:
status: In Progress → Fix Committed
Thierry Carrez (ttx)
Changed in cinder:
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in cinder:
milestone: liberty-rc1 → 7.0.0
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.