ironic-rootwrap: No section: 'Filters'

Bug #1268674 reported by Lucas Alvares Gomes
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Ironic
Fix Released
Medium
Lucas Alvares Gomes

Bug Description

When testing the deployment ironic was failing to run the iscsiadm command:

2014-01-13 17:00:11.777 18464 DEBUG ironic.openstack.common.processutils [req-2f2d57bc-6dc3-4986-a4e5-629635c599d8 admin admin] Running cmd (subprocess): sudo ironic-rootwrap /etc/ironic/rootwrap.conf iscsiadm -m discovery -t st -p 192.0.2.36:3260 execute /opt/stack/ironic/ironic/openstack/common/processutils.py:142
2014-01-13 17:00:11.809 18464 DEBUG ironic.openstack.common.processutils [req-2f2d57bc-6dc3-4986-a4e5-629635c599d8 admin admin] Result was 1 execute /opt/stack/ironic/ironic/openstack/common/processutils.py:167

Turns out the problem is in the ironic-rootwrap script:

stack@stack-virtual-machine:~/devstack$ sudo ironic-rootwrap /etc/ironic/rootwrap.conf iscsiadm -m discovery -t st -p 192.0.2.36:3260
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 101, in main
    filters = wrapper.load_filters(config.filters_path)
  File "/opt/stack/ironic/ironic/openstack/common/rootwrap/wrapper.py", line 114, 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'
stack@stack-virtual-machine:~/devstack$ echo $?
1

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

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

Changed in ironic:
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

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

Changed in ironic:
milestone: none → icehouse-2
importance: Undecided → Medium
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to ironic (master)

Reviewed: https://review.openstack.org/66389
Committed: https://git.openstack.org/cgit/openstack/ironic/commit/?id=60ca7410ba88ee782675a243a4a81f4d92647f29
Submitter: Jenkins
Branch: master

commit 60ca7410ba88ee782675a243a4a81f4d92647f29
Author: Lucas Alvares Gomes <email address hidden>
Date: Mon Jan 13 18:10:24 2014 +0000

    Use oslo.rootwrap library instead of local copy

    Remove rootwrap code copied from oslo-incubator, make the
    ironic-rootwrap console_script entrypoint point to oslo.rootwrap
    code instead.

    Patch based on: https://review.openstack.org/#/c/67422

    Change-Id: I3dd2c4cab6652dea5bd755b6048cbeea9edc81cf
    Partial-Bug: #1268674

Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

Reviewed: https://review.openstack.org/66390
Committed: https://git.openstack.org/cgit/openstack/ironic/commit/?id=8c847344f21b221757e7d945441362e6470e6e20
Submitter: Jenkins
Branch: master

commit 8c847344f21b221757e7d945441362e6470e6e20
Author: Lucas Alvares Gomes <email address hidden>
Date: Mon Jan 13 18:38:54 2014 +0000

    Add missing "Filters" section to the ironic-images.filters

    The ironic-images.filters file is missing the [Filters] section. This
    will cause the ironic-rootwrap command to fail.

    Change-Id: I3780fc89e0144093a40035e6b91fcf946bbdf6a3
    Partial-Bug: #1268674

Revision history for this message
aeva black (tenbrae) wrote :

Lucas - what's left in this bug to fix, now that the two related patchsets have merged?

Revision history for this message
Lucas Alvares Gomes (lucasagomes) wrote :

Oh nothing really, the #66390 was suppose to Closes-Bug: ... I think I put the wrong tag there.

I'll close it. Thanks for pointing it out.

ps*: There's a related but, but it's a devstack problem https://bugs.launchpad.net/devstack/+bug/1268930

Changed in ironic:
status: In Progress → Fix Committed
Thierry Carrez (ttx)
Changed in ironic:
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in ironic:
milestone: icehouse-2 → 2014.1
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.