Cinder requires unrelease oslo.rootwrap

Bug #1340877 reported by Dan Prince
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Cinder
Fix Released
Critical
Dan Prince

Bug Description

As of c9d4bd7c04ad3ca664906c4a528ab5da8d8c0b6f I'm seeing the following rootwrap warnings in my Cinder log file:

Jul 11 14:44:49 overcloud-controller0-aneizzovgipg cinder-volume[6655]: Exit code: 96
Jul 11 14:44:49 overcloud-controller0-aneizzovgipg cinder-volume[6655]: Stdout: ''
Jul 11 14:44:49 overcloud-controller0-aneizzovgipg cinder-volume[6655]: Stderr: 'WARNING:root:Skipping unknown filter class (ChainingRegExpFilter) specified in filter definitions

Normally commands appear to run fine but if you do get a failed command you'll notice this silent error from rootwrap.

Revision history for this message
Dan Prince (dan-prince) wrote :

Hmm. So there are actually two problematic commits here:

c9d4bd7c04ad3ca664906c4a528ab5da8d8c0b6f (Add cgroups related commands to rootwrap filters)

12548cc311d43d704d25c25a31478941ede27a6b (I/O rate limit for volume copy with qemu-img convert)

They appear to have landed in reverse order...

Given that the oslo.rootwrap feature we require has not yet been released we really should disable the cgroups function for now. Normally I'd submit reverts for these but in this case I think simply commenting out the code until we get a proper fix in place might be acceptable.

Changed in cinder:
assignee: nobody → Dan Prince (dan-prince)
importance: Undecided → Critical
status: New → In Progress
Revision history for this message
Dan Prince (dan-prince) wrote :

Talked to dhellmann on IRC and we may be able to cut an alpha release for oslo-rootwrap early next week. Once we do that we could bump cinder's requirements.txt file to use it...

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/106920

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

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

commit c9c807092fd04f4c777830084e9399b8c124a857
Author: Dan Prince <email address hidden>
Date: Mon Jul 14 22:51:32 2014 -0400

    Bump oslo.rootwrap to 1.3.0.0a1 for Cinder

    Cinder already requires oslo.rootwrap 1.3.0 filters so we need to
    bump requirements accordingly.

    This will fix a variety of rootwrap warnings that can occur when using
    the new cgroups features.

    Change-Id: Ia72444d054a35c3dbb34c24dbefe8f05cdebbd33
    Closes-bug: #1340877

Changed in cinder:
status: In Progress → Fix Committed
Thierry Carrez (ttx)
Changed in cinder:
milestone: none → juno-3
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in cinder:
milestone: juno-3 → 2014.2
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.