)]}'
{"neutron/db/metering/metering_rpc.py":[{"author":{"_account_id":9531,"name":"liuyulong","display_name":"LIU Yulong","email":"i@liuyulong.me","username":"LIU-Yulong"},"change_message_id":"6f7d2bb08262cc19c03896f28eb4a9c7a6a607e9","unresolved":false,"context_lines":[{"line_number":41,"context_line":"                l3_plugin, consts.L3_AGENT_SCHEDULER_EXT_ALIAS) or not host:"},{"line_number":42,"context_line":"            return metering_data"},{"line_number":43,"context_line":"        else:"},{"line_number":44,"context_line":"            router_ids \u003d l3_plugin.list_router_ids_on_host(context, host)"},{"line_number":45,"context_line":"            if not router_ids:"},{"line_number":46,"context_line":"                return"},{"line_number":47,"context_line":"            else:"}],"source_content_type":"text/x-python","patch_set":3,"id":"3fa7e38b_a224b5b3","line":44,"range":{"start_line":44,"start_character":35,"end_line":44,"end_character":58},"updated":"2019-11-07 14:25:44.000000000","message":"This will get all related router for this host, not only the scheduled routers, but also the dvr local router which is created due to some other services port, like DHCP. So is that what you really want?","commit_id":"2f47647014683cb5ab6089d340b423f2feaa8492"},{"author":{"_account_id":28655,"name":"Alexandru Sorodoc","email":"alex@privacysystems.eu","username":"bno1"},"change_message_id":"c93045d5dd3647f04c025f3af22d1460787e0d2e","unresolved":false,"context_lines":[{"line_number":41,"context_line":"                l3_plugin, consts.L3_AGENT_SCHEDULER_EXT_ALIAS) or not host:"},{"line_number":42,"context_line":"            return metering_data"},{"line_number":43,"context_line":"        else:"},{"line_number":44,"context_line":"            router_ids \u003d l3_plugin.list_router_ids_on_host(context, host)"},{"line_number":45,"context_line":"            if not router_ids:"},{"line_number":46,"context_line":"                return"},{"line_number":47,"context_line":"            else:"}],"source_content_type":"text/x-python","patch_set":3,"id":"3fa7e38b_eeae0580","line":44,"range":{"start_line":44,"start_character":35,"end_line":44,"end_character":58},"in_reply_to":"3fa7e38b_a224b5b3","updated":"2019-11-09 12:36:51.000000000","message":"I understand this is wasteful, but I don\u0027t know how to query only for scheduled routers that actually route traffic. I\u0027m open to suggestions.","commit_id":"2f47647014683cb5ab6089d340b423f2feaa8492"},{"author":{"_account_id":9531,"name":"liuyulong","display_name":"LIU Yulong","email":"i@liuyulong.me","username":"LIU-Yulong"},"change_message_id":"10fdde4ddaee1e9b17d3fd9b73ec6f7ddfb3484c","unresolved":false,"context_lines":[{"line_number":41,"context_line":"                l3_plugin, consts.L3_AGENT_SCHEDULER_EXT_ALIAS) or not host:"},{"line_number":42,"context_line":"            return metering_data"},{"line_number":43,"context_line":"        else:"},{"line_number":44,"context_line":"            router_ids \u003d l3_plugin.list_router_ids_on_host(context, host)"},{"line_number":45,"context_line":"            if not router_ids:"},{"line_number":46,"context_line":"                return"},{"line_number":47,"context_line":"            else:"}],"source_content_type":"text/x-python","patch_set":3,"id":"3fa7e38b_17284dec","line":44,"range":{"start_line":44,"start_character":35,"end_line":44,"end_character":58},"in_reply_to":"3fa7e38b_eeae0580","updated":"2019-11-11 11:34:31.000000000","message":"You may find something useful here:\nhttps://review.opendev.org/#/c/364793/\n\n1. For dvr_snat node: the \u0027l3_plugin.list_routers_on_l3_agent\u0027 is enough.\n2. For dvr_snat node with mixed compute should be\n\u0027l3_plugin.list_routers_on_l3_agent\u0027 plus \"list_router_ids_on_host\" results of dvr local routers scheduled for compute and lbaas.\n3. For compute only \"list_router_ids_on_host\"","commit_id":"2f47647014683cb5ab6089d340b423f2feaa8492"}]}
