)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":5689,"name":"Masayuki Igawa","email":"masayuki@igawa.io","username":"igawa"},"change_message_id":"b775cdf645db8345423a4612e22c0574aefc6a21","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Add check whether there are attachment"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Following the patch: https://review.openstack.org/#/c/417668/"},{"line_number":10,"context_line":"Add check whether there are attachment or not."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Change-Id: Ib74472b3b147f34a63ac5988c7db97230a9f4803"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"1a430d35_0ce1ced0","line":9,"updated":"2017-02-06 02:34:42.000000000","message":"nit: We prefer a Change-Id to a gerrit url here. Because we can see this patch when offline.","commit_id":"404a5db8caa52b7d1961b2f5e9f62fbfcc076844"},{"author":{"_account_id":19262,"name":"Liron Kuchlani","email":"lkuchlan@redhat.com","username":"lkuchlan"},"change_message_id":"fed7f29e19f9335adabc17c3ec3d7254cb770db7","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Add check whether there are attachment"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Following the patch: https://review.openstack.org/#/c/417668/"},{"line_number":10,"context_line":"Add check whether there are attachment or not."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Change-Id: Ib74472b3b147f34a63ac5988c7db97230a9f4803"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"1a430d35_20be8bd4","line":9,"in_reply_to":"1a430d35_0ce1ced0","updated":"2017-02-06 06:33:37.000000000","message":"Thanks np","commit_id":"404a5db8caa52b7d1961b2f5e9f62fbfcc076844"},{"author":{"_account_id":7350,"name":"Jordan Pittier","email":"jordan.pittier@gmail.com","username":"JordanP"},"change_message_id":"ee111cbab6ddb97c7cf018d358920090f37f13fa","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     lkuchlan \u003clkuchlan@redhat.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2017-02-06 08:42:29 +0200"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Add check whether there is an attachment or not"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Following the patch: If2a257b24f2d50a401d6804e983bcad55b5c89e8"},{"line_number":10,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"1a430d35_301cd469","line":7,"updated":"2017-02-07 12:41:37.000000000","message":"This is not a good commit title. The commit title should say which part of tempest code you are touching. Like \"test_volume_boot_pattern: assert dict is not empty before accessing a key\"","commit_id":"bf861c951670699147b765f4cea7c5ba17cda7fb"},{"author":{"_account_id":19262,"name":"Liron Kuchlani","email":"lkuchlan@redhat.com","username":"lkuchlan"},"change_message_id":"8f4925ed3528d21920c3dc2edfd7ad9f7e547860","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     lkuchlan \u003clkuchlan@redhat.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2017-02-06 08:42:29 +0200"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Add check whether there is an attachment or not"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Following the patch: If2a257b24f2d50a401d6804e983bcad55b5c89e8"},{"line_number":10,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"1a430d35_8ea2a097","line":7,"in_reply_to":"1a430d35_301cd469","updated":"2017-02-08 10:10:45.000000000","message":"Done","commit_id":"bf861c951670699147b765f4cea7c5ba17cda7fb"},{"author":{"_account_id":7350,"name":"Jordan Pittier","email":"jordan.pittier@gmail.com","username":"JordanP"},"change_message_id":"ee111cbab6ddb97c7cf018d358920090f37f13fa","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Add check whether there is an attachment or not"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Following the patch: If2a257b24f2d50a401d6804e983bcad55b5c89e8"},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"Change-Id: Ib74472b3b147f34a63ac5988c7db97230a9f4803"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"1a430d35_103a78cc","line":9,"updated":"2017-02-07 12:41:37.000000000","message":"That\u0027s not good. I don\"t want to have to read that whole patch and the discussion just to understand a 1 line change. Please explain the issue and the solution again briefly.","commit_id":"bf861c951670699147b765f4cea7c5ba17cda7fb"},{"author":{"_account_id":19262,"name":"Liron Kuchlani","email":"lkuchlan@redhat.com","username":"lkuchlan"},"change_message_id":"8f4925ed3528d21920c3dc2edfd7ad9f7e547860","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Add check whether there is an attachment or not"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Following the patch: If2a257b24f2d50a401d6804e983bcad55b5c89e8"},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"Change-Id: Ib74472b3b147f34a63ac5988c7db97230a9f4803"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"1a430d35_eeb7a4d5","line":9,"in_reply_to":"1a430d35_103a78cc","updated":"2017-02-08 10:10:45.000000000","message":"Done","commit_id":"bf861c951670699147b765f4cea7c5ba17cda7fb"},{"author":{"_account_id":5689,"name":"Masayuki Igawa","email":"masayuki@igawa.io","username":"igawa"},"change_message_id":"5e4129609c50899165770e17d9561b9b3cca9bed","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     lkuchlan \u003clkuchlan@redhat.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2017-02-08 12:09:12 +0200"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"test_create_server_from_volume_snapshot: assert dict is not empty before accessing a key"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Following the patch: If2a257b24f2d50a401d6804e983bcad55b5c89e8"},{"line_number":10,"context_line":"In case the dict is empty i.e. there in not an attachment, it will"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"fa31d9ce_21125263","line":7,"updated":"2017-02-10 07:30:25.000000000","message":"I think this commit title is a bit long.. (yeah, I know this was suggested by Jordan, though :-p)","commit_id":"9e22b852d09974ea2a5af85150c770792d7e3f39"},{"author":{"_account_id":5689,"name":"Masayuki Igawa","email":"masayuki@igawa.io","username":"igawa"},"change_message_id":"5e4129609c50899165770e17d9561b9b3cca9bed","unresolved":false,"context_lines":[{"line_number":9,"context_line":"Following the patch: If2a257b24f2d50a401d6804e983bcad55b5c89e8"},{"line_number":10,"context_line":"In case the dict is empty i.e. there in not an attachment, it will"},{"line_number":11,"context_line":"be helpful to check whether there is an attachment or not to prevent"},{"line_number":12,"context_line":"error in L204."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Change-Id: Ib74472b3b147f34a63ac5988c7db97230a9f4803"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"fa31d9ce_41174651","line":12,"updated":"2017-02-10 07:30:25.000000000","message":"\u0027L204\u0027 is not good because it can be changed when merging..","commit_id":"9e22b852d09974ea2a5af85150c770792d7e3f39"}]}