)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":5689,"name":"Masayuki Igawa","email":"masayuki@igawa.io","username":"igawa"},"change_message_id":"e6d53b8838fd9238790a24152a39709e784b6e68","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"To avoid to introduce a dependency to \"websocket-client\", we re-use"},{"line_number":10,"context_line":"the internal \"_Websocket\" class which encapsulates the sockets."},{"line_number":11,"context_line":"This class got introduced with commit 1f87a5611 for the VNC console"},{"line_number":12,"context_line":"test cases."},{"line_number":13,"context_line":"This change moves this class to a common place, so that it can be"},{"line_number":14,"context_line":"used by VNC and (in a later change) \"serial console\" test cases"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"1a1ced50_26c8fda7","line":11,"updated":"2017-03-22 10:03:12.000000000","message":"nit, I think it is better to change this commit id to the Change-Id Ic7e149f79705cbb910c5b6524dac5cdb73d69710. It can be followed on both of the gerrit and git.","commit_id":"14b452307032cfe6ab4daa385c0f9b9b62fc17ec"},{"author":{"_account_id":1921,"name":"Andrea Frittoli","email":"andrea.frittoli@gmail.com","username":"andrea-frittoli"},"change_message_id":"611db272681eab5e34dabc861ec85040f281af8b","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Markus Zoeller \u003cmzoeller@de.ibm.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2017-03-23 08:50:26 +0100"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Nova: Move _Websocket class to a common place (2/3)"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"To avoid to introduce a dependency to \"websocket-client\", we re-use"},{"line_number":10,"context_line":"the internal \"_Websocket\" class which encapsulates the sockets."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"1a1ced50_6e8fec74","line":7,"range":{"start_line":7,"start_character":46,"end_line":7,"end_character":51},"updated":"2017-03-23 13:52:48.000000000","message":"NIT: if you have to rearrange patches or add something in the middle, this means that you\u0027ll have to change all the commit messages to stay consistent.\nGiven that git / gerrit already handle topic / order for you, I think you\u0027d be better off without.","commit_id":"c48d3eff85d48a73e16b33023ed626750dd99a8a"},{"author":{"_account_id":11303,"name":"Markus Zoeller (markus_z)","email":"mzoeller@linux.vnet.ibm.com","username":"mzoeller"},"change_message_id":"a4e89766f62d4f82131402376f8e46aedafb7ed1","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Markus Zoeller \u003cmzoeller@de.ibm.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2017-03-23 08:50:26 +0100"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Nova: Move _Websocket class to a common place (2/3)"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"To avoid to introduce a dependency to \"websocket-client\", we re-use"},{"line_number":10,"context_line":"the internal \"_Websocket\" class which encapsulates the sockets."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"ffe62b97_d9832956","line":7,"range":{"start_line":7,"start_character":46,"end_line":7,"end_character":51},"in_reply_to":"1a1ced50_6e8fec74","updated":"2017-03-24 07:48:30.000000000","message":"OK, will remove those.","commit_id":"c48d3eff85d48a73e16b33023ed626750dd99a8a"}],"tempest/api/compute/servers/test_novnc.py":[{"author":{"_account_id":1921,"name":"Andrea Frittoli","email":"andrea.frittoli@gmail.com","username":"andrea-frittoli"},"change_message_id":"256b65532a6cdb445f7d42350fddf1bbfc96b2d7","unresolved":false,"context_lines":[{"line_number":19,"context_line":"import urllib3"},{"line_number":20,"context_line":""},{"line_number":21,"context_line":"from tempest.api.compute import base"},{"line_number":22,"context_line":"from tempest.common.compute import create_websocket"},{"line_number":23,"context_line":"from tempest import config"},{"line_number":24,"context_line":"from tempest.lib import decorators"},{"line_number":25,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"1a1ced50_4e3870d3","line":22,"range":{"start_line":22,"start_character":0,"end_line":22,"end_character":51},"updated":"2017-03-23 13:51:26.000000000","message":"I thought this would be covered by pep8... we always import modules, not method directly.","commit_id":"c48d3eff85d48a73e16b33023ed626750dd99a8a"},{"author":{"_account_id":11303,"name":"Markus Zoeller (markus_z)","email":"mzoeller@linux.vnet.ibm.com","username":"mzoeller"},"change_message_id":"a4e89766f62d4f82131402376f8e46aedafb7ed1","unresolved":false,"context_lines":[{"line_number":19,"context_line":"import urllib3"},{"line_number":20,"context_line":""},{"line_number":21,"context_line":"from tempest.api.compute import base"},{"line_number":22,"context_line":"from tempest.common.compute import create_websocket"},{"line_number":23,"context_line":"from tempest import config"},{"line_number":24,"context_line":"from tempest.lib import decorators"},{"line_number":25,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"ffe62b97_f9314dee","line":22,"range":{"start_line":22,"start_character":0,"end_line":22,"end_character":51},"in_reply_to":"1a1ced50_4e3870d3","updated":"2017-03-24 07:48:30.000000000","message":"Done","commit_id":"c48d3eff85d48a73e16b33023ed626750dd99a8a"}]}