)]}'
{"nova/network/neutronv2/api.py":[{"author":{"_account_id":5441,"name":"Andrew Laski","email":"andrew@lascii.com","username":"alaski"},"change_message_id":"f5b871750fc00e444d213804b12e05b1ccd4208d","unresolved":false,"context_lines":[{"line_number":479,"context_line":"                        network_uuid\u003dnet[\u0027id\u0027])"},{"line_number":480,"context_line":""},{"line_number":481,"context_line":"    def _unbind_ports(self, context, ports,"},{"line_number":482,"context_line":"                      neutron, port_client\u003dNone, instance\u003dNone):"},{"line_number":483,"context_line":"        \"\"\"Unbind the given ports by clearing their device_id and"},{"line_number":484,"context_line":"        device_owner."},{"line_number":485,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"9a89bdaa_36f6e0d7","line":482,"updated":"2016-09-09 15:32:23.000000000","message":"A small opinionated nit, but I dislike having to pass in an instance just for logging. I think the automated inclusion of the request-id and the port_id is enough.","commit_id":"3aee20587091dbc9dc51b33811e9fb6f2ebd0e4a"},{"author":{"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},"change_message_id":"6b4dee0efeea7fac59b3274d132a0e882174fcad","unresolved":false,"context_lines":[{"line_number":479,"context_line":"                        network_uuid\u003dnet[\u0027id\u0027])"},{"line_number":480,"context_line":""},{"line_number":481,"context_line":"    def _unbind_ports(self, context, ports,"},{"line_number":482,"context_line":"                      neutron, port_client\u003dNone, instance\u003dNone):"},{"line_number":483,"context_line":"        \"\"\"Unbind the given ports by clearing their device_id and"},{"line_number":484,"context_line":"        device_owner."},{"line_number":485,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"9a89bdaa_2b574903","line":482,"in_reply_to":"9a89bdaa_36f6e0d7","updated":"2016-09-09 21:42:32.000000000","message":"Agreed, considering we have request-id and port_id","commit_id":"3aee20587091dbc9dc51b33811e9fb6f2ebd0e4a"},{"author":{"_account_id":11803,"name":"Ryan Rossiter","email":"rlrossit@us.ibm.com","username":"rlrossiter"},"change_message_id":"c90e75fbe2812d79a32a6b0bc7cf35d11375ad74","unresolved":false,"context_lines":[{"line_number":479,"context_line":"                        network_uuid\u003dnet[\u0027id\u0027])"},{"line_number":480,"context_line":""},{"line_number":481,"context_line":"    def _unbind_ports(self, context, ports,"},{"line_number":482,"context_line":"                      neutron, port_client\u003dNone, instance\u003dNone):"},{"line_number":483,"context_line":"        \"\"\"Unbind the given ports by clearing their device_id and"},{"line_number":484,"context_line":"        device_owner."},{"line_number":485,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"9a89bdaa_ea400438","line":482,"in_reply_to":"9a89bdaa_36f6e0d7","updated":"2016-09-09 18:12:37.000000000","message":"I don\u0027t disagree, but seeing that we always have the instance on hand (so we don\u0027t have to go look it up), it\u0027s not bad.","commit_id":"3aee20587091dbc9dc51b33811e9fb6f2ebd0e4a"}]}