)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":6951,"name":"Brandon Logan","email":"brandon.logan@rackspace.com","username":"brandon-logan"},"change_message_id":"0659868e50b2a140a7a5b014c24a95081d114a91","unresolved":false,"context_lines":[{"line_number":10,"context_line":"Add alembic migration to modify the vip table for subnet_id"},{"line_number":11,"context_line":"Update allowed address pairs methods to use subnet_id to retrieve network_id"},{"line_number":12,"context_line":"Update network and database tasks"},{"line_number":13,"context_line":"Update tests accordingly"},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"Change-Id: Ie2876ce8c93bfb5cce8ee8606028ec91fb6c1db4"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"ba3cc151_ec0ab489","line":13,"updated":"2015-07-02 02:09:41.000000000","message":"We should get away from bullet pointing the things this commit did and just provide a paragraph as to why this commit is needed and if there are any interesting caveats to it.  I\u0027m guilty of bullet pointing as well but bullet pointing doesn\u0027t really give any additional information.\n\nJust for future reference.","commit_id":"f23f59167902904e90e9ed91c5cc036c41cdd733"},{"author":{"_account_id":10273,"name":"Adam Harwell","email":"flux.adam@gmail.com","username":"rm_you"},"change_message_id":"685eb80c170b400718200c1f0ab36e92b428b79a","unresolved":false,"context_lines":[{"line_number":10,"context_line":"Add alembic migration to modify the vip table for subnet_id"},{"line_number":11,"context_line":"Update allowed address pairs methods to use subnet_id to retrieve network_id"},{"line_number":12,"context_line":"Update network and database tasks"},{"line_number":13,"context_line":"Update tests accordingly"},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"Change-Id: Ie2876ce8c93bfb5cce8ee8606028ec91fb6c1db4"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"ba3cc151_2dc8f373","line":13,"in_reply_to":"ba3cc151_ec0ab489","updated":"2015-07-07 07:31:10.000000000","message":"I guess? Personally I find this kind of thing useful, but I suppose people should really look and see what is happening for themselves during the review process... Maybe just providing reasoning is fine.","commit_id":"f23f59167902904e90e9ed91c5cc036c41cdd733"}],"octavia/network/drivers/neutron/allowed_address_pairs.py":[{"author":{"_account_id":10273,"name":"Adam Harwell","email":"flux.adam@gmail.com","username":"rm_you"},"change_message_id":"685eb80c170b400718200c1f0ab36e92b428b79a","unresolved":false,"context_lines":[{"line_number":61,"context_line":"    def _port_to_vip(self, port, load_balancer_id\u003dNone):"},{"line_number":62,"context_line":"        port \u003d port[\u0027port\u0027]"},{"line_number":63,"context_line":"        ip_address \u003d port[\u0027fixed_ips\u0027][0][\u0027ip_address\u0027]"},{"line_number":64,"context_line":"        subnet_id \u003d port[\u0027fixed_ips\u0027][0][\u0027subnet_id\u0027]"},{"line_number":65,"context_line":"        port_id \u003d port[\u0027id\u0027]"},{"line_number":66,"context_line":"        return data_models.Vip(ip_address\u003dip_address,"},{"line_number":67,"context_line":"                               subnet_id\u003dsubnet_id,"}],"source_content_type":"text/x-python","patch_set":4,"id":"ba3cc151_8d77bf3a","line":64,"updated":"2015-07-07 07:31:10.000000000","message":"I assume that ports can only exist on a single subnet, so just choosing the subnet from the first fixed_Ip is fine?","commit_id":"f23f59167902904e90e9ed91c5cc036c41cdd733"}]}