Duplicated rootwrap.d in Ironic

Bug #1268930 reported by Lucas Alvares Gomes
10
This bug affects 2 people
Affects Status Importance Assigned to Milestone
devstack
Fix Released
Undecided
Lucas Alvares Gomes

Bug Description

The Ironic setup of devstack is duplicating the rootwrap.d directory at /etc/ironic/rootwrap.d/rootwrap.d. This will cause the ironic-rootwrap command to fail:

$ ls /etc/ironic/rootwrap.d/
ironic-deploy-helper.filters ironic-images.filters ironic-manage-ipmi.filters rootwrap.d

$ sudo ironic-rootwrap /etc/ironic/rootwrap.conf ls
Traceback (most recent call last):
  File "/usr/local/bin/ironic-rootwrap", line 10, in <module>
    sys.exit(main())
  File "/opt/stack/ironic/ironic/openstack/common/rootwrap/cmd.py", line 107, in main
    filters = wrapper.load_filters(config.filters_path)
  File "/opt/stack/ironic/ironic/openstack/common/rootwrap/wrapper.py", line 113, in load_filters
    for (name, value) in filterconfig.items("Filters"):
  File "/usr/lib/python2.7/ConfigParser.py", line 347, in items
    raise NoSectionError(section)
ConfigParser.NoSectionError: No section: 'Filters'

$ sudo rm -rf /etc/ironic/rootwrap.d/rootwrap.d/

$ sudo ironic-rootwrap /etc/ironic/rootwrap.conf ls
/usr/local/bin/ironic-rootwrap: Unauthorized command: ls (no filter matched)

Changed in devstack:
assignee: nobody → Lucas Alvares Gomes (lucasagomes)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to devstack (master)

Fix proposed to branch: master
Review: https://review.openstack.org/66543

Changed in devstack:
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to devstack (master)

Reviewed: https://review.openstack.org/66543
Committed: https://git.openstack.org/cgit/openstack-dev/devstack/commit/?id=279295c72c4e7028fc6eac75412b9b5f92cd630b
Submitter: Jenkins
Branch: master

commit 279295c72c4e7028fc6eac75412b9b5f92cd630b
Author: Lucas Alvares Gomes <email address hidden>
Date: Tue Jan 14 11:37:51 2014 +0000

    Fix duplicated rootwrap.d in lib/ironic

    The Ironic setup of devstack is duplicating the rootwrap.d directory at
    /etc/ironic/rootwrap.d/rootwrap.d, this will cause the ironic-rootwrap
    command to fail to execute. This patch is removing the duplicated
    rootwrap.d directory.

    Change-Id: I24844c24620b5b33ad1a6acd0d872e9df11d6d89
    Closes-Bug: #1268930

Changed in devstack:
status: In Progress → Fix Released
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.