)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":5174,"name":"Mauro Sergio Martins Rodrigues","email":"maurosr@linux.vnet.ibm.com","username":"maurosr"},"change_message_id":"39ba3d1dfa7078d75c68c8a01f18694ae86af3e4","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Lock is unused here, as the test is not adding"},{"line_number":10,"context_line":"a host to an aggregate.Previoulsy this lock was"},{"line_number":11,"context_line":"added to ensure the faliure will not occur where"},{"line_number":12,"context_line":"a host is added to an availability zone."},{"line_number":13,"context_line":"Previous patchlink:-"},{"line_number":14,"context_line":"https://github.com/openstack/tempest/commit/a80c334c9abacf6b395f740be313983bd35c9dd3"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"AAAAUH%2F%2BNy4%3D","line":11,"updated":"2014-01-29 00:28:00.000000000","message":"typo here: failure","commit_id":"d04be171599acb84b2958e293768607859215641"},{"author":{"_account_id":5174,"name":"Mauro Sergio Martins Rodrigues","email":"maurosr@linux.vnet.ibm.com","username":"maurosr"},"change_message_id":"39ba3d1dfa7078d75c68c8a01f18694ae86af3e4","unresolved":false,"context_lines":[{"line_number":11,"context_line":"added to ensure the faliure will not occur where"},{"line_number":12,"context_line":"a host is added to an availability zone."},{"line_number":13,"context_line":"Previous patchlink:-"},{"line_number":14,"context_line":"https://github.com/openstack/tempest/commit/a80c334c9abacf6b395f740be313983bd35c9dd3"},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"Change-Id: I9d6d610302406d5aa7bee9135c27dbc83896e005"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"AAAAUH%2F%2BNyc%3D","line":14,"updated":"2014-01-29 00:28:00.000000000","message":"instead of this long link, you can refer to the change id Iaf8a903d92c88c7c3092eaa00a9f7a96e6348ddc\n\nSuggestion: Anyway it wasn\u0027t this patch which inserted this specific lock, but I1624c366faa9a97461b68c7408e309a658c4cb83, so I would rewrite this part as:\n\nAs reference, this change Iaf8a903d92c88c7c3092eaa00a9f7a96e6348ddc clarifies the need to use a Lock mechanism on aggregates tests which add a host to an aggregate.","commit_id":"d04be171599acb84b2958e293768607859215641"},{"author":{"_account_id":8085,"name":"anju Tiwari","email":"anjutiwari5@gmail.com","username":"Anju"},"change_message_id":"cf354a56ea9ee86aaf34e920cd7b97e5e7fed81e","unresolved":false,"context_lines":[{"line_number":11,"context_line":"added to ensure the faliure will not occur where"},{"line_number":12,"context_line":"a host is added to an availability zone."},{"line_number":13,"context_line":"Previous patchlink:-"},{"line_number":14,"context_line":"https://github.com/openstack/tempest/commit/a80c334c9abacf6b395f740be313983bd35c9dd3"},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"Change-Id: I9d6d610302406d5aa7bee9135c27dbc83896e005"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"AAAAUH%2F%2BNJQ%3D","line":14,"in_reply_to":"AAAAUH%2F%2BNyc%3D","updated":"2014-01-29 04:17:18.000000000","message":"Thanks Mauro, Will cover it soon.","commit_id":"d04be171599acb84b2958e293768607859215641"},{"author":{"_account_id":5689,"name":"Masayuki Igawa","email":"masayuki@igawa.io","username":"igawa"},"change_message_id":"72c10bd94a8d228eb0c18d9f6af9012427452dd2","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Anju Tiwari \u003canju.tiwari@nectechnologies.in\u003e"},{"line_number":5,"context_line":"CommitDate: 2014-01-29 09:57:58 +0530"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Removed unused lock from a aggregate test"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Lock is unused here, as the test is not adding"},{"line_number":10,"context_line":"a host to an aggregate.Previoulsy this lock was"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"AAAAUH%2F%2BMcQ%3D","line":7,"updated":"2014-01-29 07:22:02.000000000","message":"I think the lock is used but unnecessary. I was confused when I read this commit message first.","commit_id":"001347103b484ed1f5d9f1b480a1d5adcbfbe09a"}]}