)]}'
{"dragonflow/neutron/services/l3_router_plugin.py":[{"author":{"_account_id":11159,"name":"Hong Hui Xiao","email":"honghui_xiao@yeah.net","username":"HongHuiXiao"},"change_message_id":"d63371ab6bb7cb70f1800c817ba0d42df55b1e1f","unresolved":false,"context_lines":[{"line_number":46,"context_line":""},{"line_number":47,"context_line":"class DFL3RouterPlugin(service_base.ServicePluginBase,"},{"line_number":48,"context_line":"                       common_db_mixin.CommonDbMixin,"},{"line_number":49,"context_line":"                       extraroute_db.ExtraRoute_dbonly_mixin,"},{"line_number":50,"context_line":"                       l3_gwmode_db.L3_NAT_db_mixin,"},{"line_number":51,"context_line":"                       l3_agentschedulers_db.L3AgentSchedulerDbMixin):"},{"line_number":52,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"da36d5c6_672ea044","line":49,"range":{"start_line":49,"start_character":23,"end_line":49,"end_character":60},"updated":"2017-02-23 01:53:00.000000000","message":"Maybe we should keep it until extra route is fully supported. The l3 app doesn\u0027t have distributed extra route now, and still counts on falling back to network node for extra route.","commit_id":"851aae7a595bbd3d23756aefb657ea61dbd8243c"},{"author":{"_account_id":11159,"name":"Hong Hui Xiao","email":"honghui_xiao@yeah.net","username":"HongHuiXiao"},"change_message_id":"3c6d01cd77be898522eafece352f876aced225cf","unresolved":false,"context_lines":[{"line_number":46,"context_line":""},{"line_number":47,"context_line":"class DFL3RouterPlugin(service_base.ServicePluginBase,"},{"line_number":48,"context_line":"                       common_db_mixin.CommonDbMixin,"},{"line_number":49,"context_line":"                       extraroute_db.ExtraRoute_dbonly_mixin,"},{"line_number":50,"context_line":"                       l3_gwmode_db.L3_NAT_db_mixin,"},{"line_number":51,"context_line":"                       l3_agentschedulers_db.L3AgentSchedulerDbMixin):"},{"line_number":52,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"da36d5c6_520bc67c","line":49,"range":{"start_line":49,"start_character":23,"end_line":49,"end_character":60},"in_reply_to":"da36d5c6_635c827d","updated":"2017-02-25 22:32:25.000000000","message":"I think it is feasible way :)","commit_id":"851aae7a595bbd3d23756aefb657ea61dbd8243c"},{"author":{"_account_id":7805,"name":"Li Ma","email":"skywalker.nick@gmail.com","username":"Nick"},"change_message_id":"2d76a73a49ee4dfd7488c2a5cf9174d5e5c58e36","unresolved":false,"context_lines":[{"line_number":46,"context_line":""},{"line_number":47,"context_line":"class DFL3RouterPlugin(service_base.ServicePluginBase,"},{"line_number":48,"context_line":"                       common_db_mixin.CommonDbMixin,"},{"line_number":49,"context_line":"                       extraroute_db.ExtraRoute_dbonly_mixin,"},{"line_number":50,"context_line":"                       l3_gwmode_db.L3_NAT_db_mixin,"},{"line_number":51,"context_line":"                       l3_agentschedulers_db.L3AgentSchedulerDbMixin):"},{"line_number":52,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"da36d5c6_80d16c25","line":49,"range":{"start_line":49,"start_character":23,"end_line":49,"end_character":60},"in_reply_to":"da36d5c6_672ea044","updated":"2017-02-25 07:08:25.000000000","message":"I checked the code:\nhttps://github.com/openstack/dragonflow/blob/master/dragonflow/controller/l3_proactive_app.py#L318\nIt seems the l3 app processes extra routes for distributed router. Am I missing something?","commit_id":"851aae7a595bbd3d23756aefb657ea61dbd8243c"},{"author":{"_account_id":11159,"name":"Hong Hui Xiao","email":"honghui_xiao@yeah.net","username":"HongHuiXiao"},"change_message_id":"7f5ba2baddc76d089d31eb05ed02b0ea55496ddc","unresolved":false,"context_lines":[{"line_number":46,"context_line":""},{"line_number":47,"context_line":"class DFL3RouterPlugin(service_base.ServicePluginBase,"},{"line_number":48,"context_line":"                       common_db_mixin.CommonDbMixin,"},{"line_number":49,"context_line":"                       extraroute_db.ExtraRoute_dbonly_mixin,"},{"line_number":50,"context_line":"                       l3_gwmode_db.L3_NAT_db_mixin,"},{"line_number":51,"context_line":"                       l3_agentschedulers_db.L3AgentSchedulerDbMixin):"},{"line_number":52,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"da36d5c6_a0f6708c","line":49,"range":{"start_line":49,"start_character":23,"end_line":49,"end_character":60},"in_reply_to":"da36d5c6_80d16c25","updated":"2017-02-25 07:36:12.000000000","message":"l3_proactive_app processes the extra route, but l3_app doesn\u0027t.","commit_id":"851aae7a595bbd3d23756aefb657ea61dbd8243c"},{"author":{"_account_id":7805,"name":"Li Ma","email":"skywalker.nick@gmail.com","username":"Nick"},"change_message_id":"cee693f16142b99e2c2d9c7e57c57147757d3b50","unresolved":false,"context_lines":[{"line_number":46,"context_line":""},{"line_number":47,"context_line":"class DFL3RouterPlugin(service_base.ServicePluginBase,"},{"line_number":48,"context_line":"                       common_db_mixin.CommonDbMixin,"},{"line_number":49,"context_line":"                       extraroute_db.ExtraRoute_dbonly_mixin,"},{"line_number":50,"context_line":"                       l3_gwmode_db.L3_NAT_db_mixin,"},{"line_number":51,"context_line":"                       l3_agentschedulers_db.L3AgentSchedulerDbMixin):"},{"line_number":52,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"da36d5c6_635c827d","line":49,"range":{"start_line":49,"start_character":23,"end_line":49,"end_character":60},"in_reply_to":"da36d5c6_a0f6708c","updated":"2017-02-25 10:17:18.000000000","message":"l3_proactive_app is the default application in devstack. I think we can submit a bug for l3_app and let this pass the review procedure? what do you think?","commit_id":"851aae7a595bbd3d23756aefb657ea61dbd8243c"}]}