Comment 4 for bug 1218142

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

Reviewed: https://review.openstack.org/44198
Committed: http://github.com/openstack/neutron/commit/89e38929d2bc626b639df9912a98d24b25a7e964
Submitter: Jenkins
Branch: master

commit 89e38929d2bc626b639df9912a98d24b25a7e964
Author: Brian Haley <email address hidden>
Date: Tue Aug 27 17:36:43 2013 -0400

    Change daemon Pidfile class to not use root_helper

    Some users of the Pidfile class don't specify root_helper,
    which then defaults to 'sudo', which will generate an error.
    Remove root_helper altogether since we actually don't need
    root priveleges to read /proc/$pid/cmdline.

    Changed code to use open.readline() instead of a shell, and
    tweaked tests accordingly.

    Also cleaned-up the rootwrap filters that allow it as they
    are not used anymore.

    Fixes bug 1218142

    Change-Id: I6691feb1c9f7bfa261a7ec464fd8f3f92168c302