The L3 agent tries to catch an exception from processutils when processutils is not used

Bug #1401626 reported by Terry Wilson
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
neutron
Fix Released
Undecided
Terry Wilson

Bug Description

L3 agent imports the processutils module to catch exceptions that wouldn't ever be thrown because the underlying execute() being called is the one from neutron.agent.linux.utils which raises a RuntimeError on failure.

Changed in neutron:
assignee: nobody → Terry Wilson (otherwiseguy)
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to neutron (master)

Reviewed: https://review.openstack.org/140507
Committed: https://git.openstack.org/cgit/openstack/neutron/commit/?id=4cfe2fc98ae273034ffef7264f21b1bd5dff414d
Submitter: Jenkins
Branch: master

commit 4cfe2fc98ae273034ffef7264f21b1bd5dff414d
Author: Terry Wilson <email address hidden>
Date: Tue Dec 9 15:24:37 2014 -0700

    Have L3 agent catch the correct exception

    L3 agent imports the processutils module to catch exceptions that
    wouldn't ever be thrown because the underlying execute() being
    called is the one from neutron.agent.linux.utils which raises a
    RuntimeError on failure.

    Also, processutils is now part of oslo.concurrency. So when we
    actually start using it, we'll use it from there.

    Closes-Bug: 1401626
    Change-Id: I43874e1b63a0ba7b01415cafe0538f4343057066

Changed in neutron:
status: In Progress → Fix Committed
Thierry Carrez (ttx)
Changed in neutron:
milestone: none → kilo-1
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in neutron:
milestone: kilo-1 → 2015.1.0
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.