)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":14611,"name":"dasm","display_name":"dasm","email":"dsmigiel@redhat.com","username":"dasm"},"change_message_id":"ff87325dd2f9d43829297e6a4d6dbb8fd02843e1","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Sindhu Devale \u003csindhu.devale@intel.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2016-10-03 14:01:50 -0500"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"[WIP] Integration of L3HARouterAgentPortBinding"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This patch integrates the object created for"},{"line_number":10,"context_line":"L3HARouterAgentPortBinding into rest of the code base."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"fa6399be_5a22caba","line":7,"range":{"start_line":7,"start_character":0,"end_line":7,"end_character":6},"updated":"2016-10-11 02:05:38.000000000","message":"If this patch is ready, probably would be good to drop `[WIP]`","commit_id":"63d13354ce0602fefd2cf537d7e04bf5d730cc32"},{"author":{"_account_id":14611,"name":"dasm","display_name":"dasm","email":"dsmigiel@redhat.com","username":"dasm"},"change_message_id":"9247fbf3fa0c35a718fd86b412b5c0a8b03e0894","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Sindhu Devale \u003csindhu.devale@intel.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2016-10-03 14:01:50 -0500"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"[WIP] Integration of L3HARouterAgentPortBinding"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This patch integrates the object created for"},{"line_number":10,"context_line":"L3HARouterAgentPortBinding into rest of the code base."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"fa6399be_e14bc329","line":7,"range":{"start_line":7,"start_character":0,"end_line":7,"end_character":6},"in_reply_to":"fa6399be_2b5ce624","updated":"2016-10-11 22:09:02.000000000","message":"Ack, you can always add TODO over there, and then allow to merge this change. When Agent OVO will be done, you can get back to this place and update it.","commit_id":"63d13354ce0602fefd2cf537d7e04bf5d730cc32"},{"author":{"_account_id":8726,"name":"Victor Morales","email":"chipahuac@hotmail.com","username":"electrocucaracha"},"change_message_id":"5ecf823ce20d9c9f4e2746a111fd565591da558d","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Sindhu Devale \u003csindhu.devale@intel.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2016-10-03 14:01:50 -0500"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"[WIP] Integration of L3HARouterAgentPortBinding"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This patch integrates the object created for"},{"line_number":10,"context_line":"L3HARouterAgentPortBinding into rest of the code base."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"fa6399be_ff55c24f","line":7,"range":{"start_line":7,"start_character":0,"end_line":7,"end_character":6},"in_reply_to":"fa6399be_2b5ce624","updated":"2016-10-20 16:31:04.000000000","message":"What about the UTs located under the following folders:\n\nneutron/tests/unit/plugins/ml2/drivers/l2pop\nneutron/tests/unit/scheduler\n\nCan you also consider the option to have the creation and integration in a single patch, if that\u0027s not possible could you list the reasons of that into the commit message.","commit_id":"63d13354ce0602fefd2cf537d7e04bf5d730cc32"},{"author":{"_account_id":22218,"name":"Sindhu Devale","email":"sindhu.devale@intel.com","username":"sindhudevale"},"change_message_id":"d36cefb646037fb690c9655543f58f793daf5702","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Sindhu Devale \u003csindhu.devale@intel.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2016-10-03 14:01:50 -0500"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"[WIP] Integration of L3HARouterAgentPortBinding"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This patch integrates the object created for"},{"line_number":10,"context_line":"L3HARouterAgentPortBinding into rest of the code base."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"fa6399be_2b5ce624","line":7,"range":{"start_line":7,"start_character":0,"end_line":7,"end_character":6},"in_reply_to":"fa6399be_5a22caba","updated":"2016-10-11 03:33:04.000000000","message":"Hey Darek, there is one query that depends on the Agent OVO [1]\n\n[1]https://review.openstack.org/#/c/381239/2/neutron/db/l3_hamode_db.py     #596\n\nso waiting for Agent OVO work to be done.","commit_id":"63d13354ce0602fefd2cf537d7e04bf5d730cc32"},{"author":{"_account_id":22218,"name":"Sindhu Devale","email":"sindhu.devale@intel.com","username":"sindhudevale"},"change_message_id":"3146000068900f68f106b055079f91e783f393af","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Sindhu Devale \u003csindhu.devale@intel.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2016-10-03 14:01:50 -0500"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"[WIP] Integration of L3HARouterAgentPortBinding"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This patch integrates the object created for"},{"line_number":10,"context_line":"L3HARouterAgentPortBinding into rest of the code base."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"da6895a0_fe7e14ad","line":7,"range":{"start_line":7,"start_character":0,"end_line":7,"end_character":6},"in_reply_to":"fa6399be_e14bc329","updated":"2016-10-21 16:57:57.000000000","message":"Done","commit_id":"63d13354ce0602fefd2cf537d7e04bf5d730cc32"},{"author":{"_account_id":22218,"name":"Sindhu Devale","email":"sindhu.devale@intel.com","username":"sindhudevale"},"change_message_id":"3146000068900f68f106b055079f91e783f393af","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Sindhu Devale \u003csindhu.devale@intel.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2016-10-03 14:01:50 -0500"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"[WIP] Integration of L3HARouterAgentPortBinding"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This patch integrates the object created for"},{"line_number":10,"context_line":"L3HARouterAgentPortBinding into rest of the code base."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"da6895a0_3e4fac93","line":7,"range":{"start_line":7,"start_character":0,"end_line":7,"end_character":6},"in_reply_to":"fa6399be_ff55c24f","updated":"2016-10-21 16:57:57.000000000","message":"Done","commit_id":"63d13354ce0602fefd2cf537d7e04bf5d730cc32"}],"neutron/db/l3_hamode_db.py":[{"author":{"_account_id":8726,"name":"Victor Morales","email":"chipahuac@hotmail.com","username":"electrocucaracha"},"change_message_id":"5ecf823ce20d9c9f4e2746a111fd565591da558d","unresolved":false,"context_lines":[{"line_number":607,"context_line":"            context, router_id\u003drouter_id, l3_agent_id\u003dagent_id)"},{"line_number":608,"context_line":"        if obj:"},{"line_number":609,"context_line":"            return obj.pop()"},{"line_number":610,"context_line":"        else:"},{"line_number":611,"context_line":"            return None"},{"line_number":612,"context_line":""},{"line_number":613,"context_line":"    def _get_bindings_and_update_router_state_for_dead_agents(self, context,"},{"line_number":614,"context_line":"                                                              router_id):"}],"source_content_type":"text/x-python","patch_set":2,"id":"fa6399be_1f4a7626","line":611,"range":{"start_line":610,"start_character":8,"end_line":611,"end_character":23},"updated":"2016-10-20 16:31:04.000000000","message":"nit: the else statement is not needed. You can return None at the end.","commit_id":"63d13354ce0602fefd2cf537d7e04bf5d730cc32"},{"author":{"_account_id":22218,"name":"Sindhu Devale","email":"sindhu.devale@intel.com","username":"sindhudevale"},"change_message_id":"3146000068900f68f106b055079f91e783f393af","unresolved":false,"context_lines":[{"line_number":607,"context_line":"            context, router_id\u003drouter_id, l3_agent_id\u003dagent_id)"},{"line_number":608,"context_line":"        if obj:"},{"line_number":609,"context_line":"            return obj.pop()"},{"line_number":610,"context_line":"        else:"},{"line_number":611,"context_line":"            return None"},{"line_number":612,"context_line":""},{"line_number":613,"context_line":"    def _get_bindings_and_update_router_state_for_dead_agents(self, context,"},{"line_number":614,"context_line":"                                                              router_id):"}],"source_content_type":"text/x-python","patch_set":2,"id":"da6895a0_5e446073","line":611,"range":{"start_line":610,"start_character":8,"end_line":611,"end_character":23},"in_reply_to":"fa6399be_1f4a7626","updated":"2016-10-21 16:57:57.000000000","message":"Done","commit_id":"63d13354ce0602fefd2cf537d7e04bf5d730cc32"}]}