nova/baremetal-compute-ipmi.filters issues

[Mark McLoughlin]

From ttx, issues in nova/baremetal-compute-ipmi.filters

 * allows ipmitool, but ipmitool isn't called as root
 * allows kill, but kill is used against a process which is not run as root

These are the only two filters in the file, so we should be able to just remove the file.

We also need to remove run_as_root from:

            utils.execute('kill', '-TERM', str(console_pid),
                          run_as_root=True,
                          check_exit_code=[0, 99])

[Joe Gordon (jogo)]

Since we are in the process of deprecating nova maremetal, we should focus work on ironic instead.

[aeva black (tenbrae)]

Confirmed that this also affects Ironic.

etc/ironic/rootwrap.d/ironic-manage-ipmi.filters has duplicated the problems described in this bug report. Neither command appears to need (or even use) root privileges today. This file should be deleted.

[OpenStack Infra (hudson-openstack)]

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

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/108913
Committed: https://git.openstack.org/cgit/openstack/ironic/commit/?id=12023ad1a116be4b1b15272b2d43dce3833403a7
Submitter: Jenkins
Branch: master

commit 12023ad1a116be4b1b15272b2d43dce3833403a7
Author: Dongdong Zhou <email address hidden>
Date: Wed Jul 23 08:36:33 2014 +0100

    remove ironic-manage-ipmi.filters

    the two commands in this file doesn't need to
    be executed as root, so this file can be removed

    Closes-Bug: #1248422

    Change-Id: I8a3b8b221a260a3a04ba8bbea59c60ed2fc1c8f8