Comment 3 for bug 1336566

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

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

commit fc01ddaaa5f1170b746f47837104352bfe36eaa5
Author: armando-migliaccio <email address hidden>
Date: Tue Jul 1 15:08:11 2014 -0700

    Fix UnboundLocalError raised during L3 router sync task

    This can be fixed in a number of ways: a) consolidating the
    two except clauses into one; b) adding a 'return' after the
    last except clause c) by calling the cleanup method only on
    success; d) initializing 'routers' before usage.

    Approach c) has the benefit of stating the developer's intent
    more explicitly and minimize chances of regression.

    Closes-bug: #1336566

    Change-Id: Ib91ac5bb07869cbec6825b60d7c4c5b23c4c4d3a