Henry,<div><br></div><div>Yes, it looks like the Gerrit tests didn't pick this one up. There must be some passwordless sudo set somewhere where it shouldn't. I'll file a fix shortly.</div><div><br></div><div>Thanks,</div>

<div>Armando</div><div><br><div class="gmail_quote">On Fri, May 10, 2013 at 8:34 PM, Henry Gessau <span dir="ltr"><<a href="mailto:gessau@cisco.com" target="_blank">gessau@cisco.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

Not sure if I am the only one affected by this. Can any one else confirm?<br>
<br>
I cloned quantum today and kicked off a full-suite unit test. The<br>
test run stopped mid-stride with prompts for my sudo password. I have<br>
confirmed that by reversing out the following change, the tests run fine:<br>
<br>
><br>
> Change subject: Log a warning if dnsmasq version is below the minimum required<br>
> ......................................................................<br>
><br>
><br>
> Log a warning if dnsmasq version is below the minimum required<br>
><br>
> It has been noted that older versions of dnsmasq may have unexpected<br>
> behavior, so this patch introduces a check on dnsmasq version; if<br>
> the minimum version is not met, a warning is logged. This is less<br>
> invasive than raising an Exception and abort the dhcp agent, even<br>
> though this also seems like a perfectly acceptable behavior.<br>
><br>
> Fixes bug 1170793<br>
><br>
> Change-Id: Idd91c3753fb05f223ed0e3c53ad692ee65ea4905<br>
> ---<br>
> M etc/quantum/rootwrap.d/dhcp.filters<br>
> M quantum/agent/dhcp_agent.py<br>
> M quantum/agent/linux/dhcp.py<br>
> M quantum/rootwrap/filters.py<br>
> M quantum/tests/unit/test_linux_dhcp.py<br>
> 5 files changed, 57 insertions(+), 0 deletions(-)<br>
><br>
> Approvals:<br>
>   Alex Xu: Looks good to me, but someone else must approve<br>
>   Jenkins: Verified<br>
>   Salvatore Orlando: Looks good to me (core reviewer); Approved<br>
>   enikanorov: Looks good to me, but someone else must approve<br>
>   garyk: Looks good to me (core reviewer)<br>
><br>
><br>
> --<br>
> To view, visit <a href="https://review.openstack.org/28489" target="_blank">https://review.openstack.org/28489</a><br>
> To unsubscribe, visit <a href="https://review.openstack.org/settings" target="_blank">https://review.openstack.org/settings</a><br>
><br>
> Gerrit-MessageType: merged<br>
> Gerrit-Change-Id: Idd91c3753fb05f223ed0e3c53ad692ee65ea4905<br>
> Gerrit-PatchSet: 2<br>
> Gerrit-Project: openstack/quantum<br>
> Gerrit-Branch: master<br>
> Gerrit-Owner: Armando Migliaccio <<a href="mailto:amigliaccio@nicira.com">amigliaccio@nicira.com</a>><br>
> Gerrit-Reviewer: Alex Xu <<a href="mailto:xuhj@linux.vnet.ibm.com">xuhj@linux.vnet.ibm.com</a>><br>
> Gerrit-Reviewer: Armando Migliaccio <<a href="mailto:amigliaccio@nicira.com">amigliaccio@nicira.com</a>><br>
> Gerrit-Reviewer: Dirk Mueller <<a href="mailto:dirk@dmllr.de">dirk@dmllr.de</a>><br>
> Gerrit-Reviewer: Jenkins<br>
> Gerrit-Reviewer: Nachi Ueno <<a href="mailto:nachi@ntti3.com">nachi@ntti3.com</a>><br>
> Gerrit-Reviewer: Paul Michali <<a href="mailto:pcm@cisco.com">pcm@cisco.com</a>><br>
> Gerrit-Reviewer: Salvatore Orlando <<a href="mailto:salv.orlando@gmail.com">salv.orlando@gmail.com</a>><br>
> Gerrit-Reviewer: enikanorov <<a href="mailto:enikanorov@mirantis.com">enikanorov@mirantis.com</a>><br>
> Gerrit-Reviewer: garyk <<a href="mailto:gkotton@redhat.com">gkotton@redhat.com</a>><br>
><br>
</blockquote></div><br></div>