)]}'
{"tempest/services/network/json/network_client.py":[{"author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"change_message_id":"df2752240c55036136921dbdbe6cac82e2bb2695","unresolved":false,"context_lines":[{"line_number":344,"context_line":""},{"line_number":345,"context_line":"        Available params: see http://developer.openstack.org/"},{"line_number":346,"context_line":"                              api-ref-networking-v2-ext.html"},{"line_number":347,"context_line":"                              #Extraroutes"},{"line_number":348,"context_line":"        \"\"\""},{"line_number":349,"context_line":"        uri \u003d \u0027/routers/%s\u0027 % router_id"},{"line_number":350,"context_line":"        put_body \u003d {\u0027router\u0027: kwargs}"}],"source_content_type":"text/x-python","patch_set":1,"id":"7a740942_7994b436","line":347,"updated":"2015-12-04 06:24:28.000000000","message":"We can put line346-347 on the same line because the length is 72.\nAfter that, we can check the link more easily on the review :)","commit_id":"f73ce53037f682abc2a0556c0bfdd2cb79c5bb3c"},{"author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"change_message_id":"66dda01da723f5b6c51d8a2f05e61a88b6659286","unresolved":false,"context_lines":[{"line_number":344,"context_line":""},{"line_number":345,"context_line":"        Available params: see http://developer.openstack.org/"},{"line_number":346,"context_line":"                              api-ref-networking-v2-ext.html#Extraroutes"},{"line_number":347,"context_line":"        \"\"\""},{"line_number":348,"context_line":"        uri \u003d \u0027/routers/%s\u0027 % router_id"},{"line_number":349,"context_line":"        put_body \u003d {\u0027router\u0027: kwargs}"},{"line_number":350,"context_line":"        return self.update_resource(uri, put_body)"}],"source_content_type":"text/x-python","patch_set":2,"id":"7a740942_e8a3e89c","line":347,"updated":"2015-12-04 09:12:10.000000000","message":"Thanks for updating, but the above link doesn\u0027t exist.\n\nThe link is http://developer.openstack.org/api-ref-networking-v2-ext.html#updateRouter now.\nHowever, the link is wrong because duplicated to the other link.\nand https://review.openstack.org/#/c/253340/ fixes it.\n\nAt this time\n\n Available params: see http://developer.openstack.org/\n                       api-ref-networking-v2-ext.html#updateRouter\n\nis fine because the above patch is not merged yet.\nand please add TODO like\nhttps://review.openstack.org/#/c/252152/5/tempest_lib/services/compute/security_group_default_rules_client.py","commit_id":"a4ffc56952230d3e659eb5a0955508422cfae66d"},{"author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"change_message_id":"d2b59995e34d13856e562bd62c11868896c135d6","unresolved":false,"context_lines":[{"line_number":343,"context_line":"        \"\"\"Update Extra routes."},{"line_number":344,"context_line":""},{"line_number":345,"context_line":"        Available params: see http://developer.openstack.org/"},{"line_number":346,"context_line":"                              api-ref-networking-v2-ext.html#Extraroutes"},{"line_number":347,"context_line":"        \"\"\""},{"line_number":348,"context_line":"        # TODO(piyush): We can see the API doc of the above method with the"},{"line_number":349,"context_line":"        # above link, but the link is wrong because The links of \"Configures"}],"source_content_type":"text/x-python","patch_set":3,"id":"7a740942_30454045","line":346,"updated":"2015-12-08 13:16:36.000000000","message":"Now the problem is solved.\n\nso we can put here like\n\n api-ref-networking-v2-ext.html#updateExtraRoutes\n\nand we can remove the following TODO.","commit_id":"677ee77aef0ea2aadbba7b20951a2a6a5908a6b7"}]}