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.
Reviewed: https:/ /review. openstack. org/44198 github. com/openstack/ neutron/ commit/ 89e38929d2bc626 b639df9912a98d2 4b25a7e964
Committed: http://
Submitter: Jenkins
Branch: master
commit 89e38929d2bc626 b639df9912a98d2 4b25a7e964
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: I6691feb1c9f7bf a261a7ec464fd8f 3f92168c302