Do not require sudo/rootwrap to check for dnsmasq version
The dnsmasq version check does not need root privileges
to run as with low privs it works just fine. As a side
effect, the use of the rootwrapper was causing unit tests
to hung because the execute call in check_version was not
being stubbed out. Weirdly enough this wasn't caught in
a previous Gerrit run; there must be a passwordless sudo
lurking around somewhere in the Gerrit infrastructure.
Reviewed: https:/ /review. openstack. org/28836 github. com/openstack/ quantum/ commit/ 91b56e41eecf2cd 8b4c848b4e773a1 be307b40c3
Committed: http://
Submitter: Jenkins
Branch: master
commit 91b56e41eecf2cd 8b4c848b4e773a1 be307b40c3
Author: armando-migliaccio <email address hidden>
Date: Fri May 10 13:10:02 2013 -0700
Do not require sudo/rootwrap to check for dnsmasq version
The dnsmasq version check does not need root privileges
to run as with low privs it works just fine. As a side
effect, the use of the rootwrapper was causing unit tests
to hung because the execute call in check_version was not
being stubbed out. Weirdly enough this wasn't caught in
a previous Gerrit run; there must be a passwordless sudo
lurking around somewhere in the Gerrit infrastructure.
Fixes bug #1178800
Change-Id: I4d0bd218768eec 2439d3907587682 ff35884a262