<div dir="ltr">Hi,<div><br></div><div style>I've filed bug/1175745 to perform the following file tree change in quantum/lbaas:</div><div style>1) quantum/db/loadbalancer/loadbalancer_db.py -> quantum/plugins/services/agent_loadbalancer/loadbalancer_db.py</div>
<div style>2) quantum/tests/unit/db/loadbalancer/test_db_loadbalancer.py -> quantum/tests/unit/services/agent_loadbalancer/test_db_loadbalancer.py<br></div><div style>3) quantum/tests/unit/test_loadbalancer_plugin.py -> quantum/tests/unit/services/agent_loadbalancer/test_loadbalancer_extension.py<br>
</div><div style><br></div><div style>__init__.py left in db/loadbalancer and tests/unit/db/loadbalancer/ are removed.</div><div style>Loadbalancer extension remains in extensions directory</div><div style><br></div><div style>
<br></div><div style>As a second step, which will be separate bug, I suggest the following name changes in services:</div><div style>quantum/plugins/services/agent_loadbalancer ->  quantum/plugins/services/loadbalancer</div>
<div style>and corresponding renaming of unit tests subdir<br></div><div style><br></div><div style>The intent of this renaming is to show that this implementation is in fact multivendor. </div><div style>It's not at this moment, but multivendor support will be implemented using current plugin, so</div>
<div style>I'd also rename agent->agents and driver->drivers.</div><div style>Under agents I would move current files under namespace_agent subdir.</div><div style>Under drivers I'd move current files under haproxy_on_host (or something similar)</div>
<div style><br></div><div style>All these changes target the following file tree pattern for services:</div><div style>plugins</div><div style>  services</div><div style>     service_type (loadbalancer, fw)</div><div style>
        agents</div><div style>            vendor_specific_agent (vendor or implementation-specific)</div><div style>        drivers</div><div style>            vendor_specific_agent</div><div style>        this_service_type_db_plugin.py</div>
<div style>        alt_service_type_implementation</div><div style>            whatever (including possible alternative DB plugin)</div><div style><br></div><div style>I'd suggest the same pattern for the unit tests.</div>
<div style><br></div><div style>Thanks,</div><div style>Eugene.</div></div>