)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":5689,"name":"Masayuki Igawa","email":"masayuki@igawa.io","username":"igawa"},"change_message_id":"1c16c110a2692cca9c6503a7c76cbf5b3aabe15c","unresolved":false,"context_lines":[{"line_number":9,"context_line":"Uses the fixed_network_name option to select which network"},{"line_number":10,"context_line":"to boot test instances on.  This allows Tempest to be run"},{"line_number":11,"context_line":"in environments where there are multiple networks available."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Change-Id: I64ad6ce6bee3fe33537ac4f8d8d913856fe46554"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"AAAAUH%2F%2BDuo%3D","line":12,"updated":"2014-01-30 15:50:13.000000000","message":"I think this patch fixes a bug similar to bug 1250866 . If so, I think it\u0027s good to have\n Related-Bug: #1250866\nin this commit message.\nOr it\u0027s better to report a bug about this, and then\n Closes-Bug: #xxxxxxxx\nin this commit message.","commit_id":"2cf2400434b3a852004b027ad6e630a534f4c64b"}],"tempest/scenario/manager.py":[{"author":{"_account_id":6928,"name":"Ben Nemec","email":"openstack@nemebean.com","username":"bnemec"},"change_message_id":"8a4ee1c73e044ebc2ae1e4588097b041746d2803","unresolved":false,"context_lines":[{"line_number":465,"context_line":"                    if network.label \u003d\u003d fixed_network_name:"},{"line_number":466,"context_line":"                        create_kwargs[\u0027nics\u0027] \u003d [{\u0027net-id\u0027: network.id}]"},{"line_number":467,"context_line":"                        break"},{"line_number":468,"context_line":"                else:"},{"line_number":469,"context_line":"                    msg \u003d (\"The network on which the NIC of the server must \""},{"line_number":470,"context_line":"                           \"be connected can not be found : \""},{"line_number":471,"context_line":"                           \"fixed_network_name\u003d%s. Starting instance without \""}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAUH%2F%2BUBw%3D","line":468,"updated":"2014-01-28 03:52:06.000000000","message":"I think this needs to be another level of indentation to the left.","commit_id":"215a022c8b0e44d125f7a5f066d1cd8aff41c7a5"},{"author":{"_account_id":5689,"name":"Masayuki Igawa","email":"masayuki@igawa.io","username":"igawa"},"change_message_id":"079c4fdfe4237f4597377cf736e3d0c2b4895af1","unresolved":false,"context_lines":[{"line_number":465,"context_line":"                    if network.label \u003d\u003d fixed_network_name:"},{"line_number":466,"context_line":"                        create_kwargs[\u0027nics\u0027] \u003d [{\u0027net-id\u0027: network.id}]"},{"line_number":467,"context_line":"                        break"},{"line_number":468,"context_line":"                else:"},{"line_number":469,"context_line":"                    msg \u003d (\"The network on which the NIC of the server must \""},{"line_number":470,"context_line":"                           \"be connected can not be found : \""},{"line_number":471,"context_line":"                           \"fixed_network_name\u003d%s. Starting instance without \""}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAUH%2F%2BMNY%3D","line":468,"in_reply_to":"AAAAUH%2F%2BTJE%3D","updated":"2014-01-29 08:54:02.000000000","message":"I feel it\u0027s good to have a comment what does it  or split into another function here because it\u0027s a little complex looping and condition.","commit_id":"215a022c8b0e44d125f7a5f066d1cd8aff41c7a5"},{"author":{"_account_id":7350,"name":"Jordan Pittier","email":"jordan.pittier@gmail.com","username":"JordanP"},"change_message_id":"355869c21eb987bb3d9bf3abc1b55cf9731526dd","unresolved":false,"context_lines":[{"line_number":465,"context_line":"                    if network.label \u003d\u003d fixed_network_name:"},{"line_number":466,"context_line":"                        create_kwargs[\u0027nics\u0027] \u003d [{\u0027net-id\u0027: network.id}]"},{"line_number":467,"context_line":"                        break"},{"line_number":468,"context_line":"                else:"},{"line_number":469,"context_line":"                    msg \u003d (\"The network on which the NIC of the server must \""},{"line_number":470,"context_line":"                           \"be connected can not be found : \""},{"line_number":471,"context_line":"                           \"fixed_network_name\u003d%s. Starting instance without \""}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAUH%2F%2BTJE%3D","line":468,"in_reply_to":"AAAAUH%2F%2BUBw%3D","updated":"2014-01-28 08:51:27.000000000","message":"Nope, this is a valid Python construct. The code in the else bloc is executed only if the for loop doesn\u0027t \"break\".","commit_id":"215a022c8b0e44d125f7a5f066d1cd8aff41c7a5"}]}