Comment 23 for bug 1609738

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

commit e4b0b9f8be165e688ee56a5063a3ccc91e823636
Author: John Schwarz <email address hidden>
Date: Thu Oct 13 16:10:07 2016 +0300

    Refactor L3 scheduler (unify code paths)

    This patch proposes a (rather major) refactor to the L3 scheduler.
    Basically, the auto_schedule_routers() code-path was split to 2
    different code-paths, each dealing with a different case (unscheduled
    routers vs underscheduled routers), in addition to the API-initiated
    schedule() logic. This patch removes the 2 code-paths in favor of moving
    most of the logic into schedule(). While the result is a slightly
    longer schedule(), the benefit is that a lot of the previous
    unmaintainable code-paths of auto_schedule_routers() are now removed.

    Yay! :D

    Related-Bug: #1609738
    Change-Id: I227ca60422545e40d3bbb8baf2b41a8ce14f4294