)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":6618,"name":"Ruby Loo","email":"opensrloo@gmail.com","username":"rloo"},"change_message_id":"ad8548077346d80b60b204f898ec9c7ed158711e","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Kyrylo Romanenko \u003ckromanenko@mirantis.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2017-02-16 16:48:52 +0000"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Change condition for deletion of node in functional tests"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Allow to cleanup node when it is in provision state \"active\"."},{"line_number":10,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"ba2be162_ee67d0e6","line":7,"updated":"2017-03-02 20:38:33.000000000","message":"it looks like you only changed one test.\n\nI don\u0027t see how this line is associated with the description below.","commit_id":"3bc6281b6be426295691daf88703d904f2bad893"},{"author":{"_account_id":6618,"name":"Ruby Loo","email":"opensrloo@gmail.com","username":"rloo"},"change_message_id":"ad8548077346d80b60b204f898ec9c7ed158711e","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Change condition for deletion of node in functional tests"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Allow to cleanup node when it is in provision state \"active\"."},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"Closes-Bug: #1665403"},{"line_number":12,"context_line":"Change-Id: I26f7eda195304a8e44e52e3907d14dd04ad7d61f"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"ba2be162_8e6c94d1","line":9,"updated":"2017-03-02 20:38:33.000000000","message":"It should be straightforward for me as a reviewer, to review this patch. however, the description in the bug report \u0027test_node_set_provision_state can not delete...\u0027 does not make sense. I don\u0027t see any \u0027test_node_set_provision_state\u0027 in the code here.\n\nAlso, I don\u0027t see any \u0027cleanup\u0027 code being done here. I realize that English isn\u0027t your native language, but please try to be clear.","commit_id":"3bc6281b6be426295691daf88703d904f2bad893"},{"author":{"_account_id":14614,"name":"Kyrylo Romanenko","display_name":"K.Romanenko","email":"romanenko-kv@hotmail.com","username":"kromanenko"},"change_message_id":"8627ddb200f4f813a42ba3de47d5277648b9e054","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Change condition for deletion of node in functional tests"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Allow to cleanup node when it is in provision state \"active\"."},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"Closes-Bug: #1665403"},{"line_number":12,"context_line":"Change-Id: I26f7eda195304a8e44e52e3907d14dd04ad7d61f"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"9a30ddce_ecf0bf93","line":9,"in_reply_to":"ba2be162_8e6c94d1","updated":"2017-03-09 14:51:02.000000000","message":"Thank you for review, Ruby. I\u0027m going to put aside this patch while issue will be investigated better. \nInstead of this i\u0027ll add another patch to update conditions for node deletion according to states in which node deletion is allowed.","commit_id":"3bc6281b6be426295691daf88703d904f2bad893"}]}