)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"change_message_id":"195912fab3eebd2e8490eefedd019462ae89710b","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     adi \u003caditi.s@nectechnologies.in\u003e"},{"line_number":5,"context_line":"CommitDate: 2016-01-19 16:10:58 +0530"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Fixes bug \"No message to user after successful snapshot delete"},{"line_number":8,"context_line":"operation\""},{"line_number":9,"context_line":""},{"line_number":10,"context_line":"Currently theres is no message given by python cinderclient after"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"3a57f1b5_7d1682a8","line":7,"updated":"2016-02-11 15:44:00.000000000","message":"Please reword to be within 50 characters just stating what is done, not the bug it is addressing.","commit_id":"1adc5a994efe0f2bb48ca8f12d3df42bb0c58069"},{"author":{"_account_id":10485,"name":"Ankit Agrawal","email":"ankitagrawal8188@gmail.com","username":"ankitagrawal"},"change_message_id":"86c01944faa809295dd41da1c81157de7154c055","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     aditi sharma \u003caditi.s@nectechnologies.in\u003e"},{"line_number":5,"context_line":"CommitDate: 2016-03-22 06:12:55 +0000"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Adds missing user message"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Currently theres is no message given by python cinderclient after"},{"line_number":10,"context_line":"successfully accepting the request of delete snapshot and force delete"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"fa0719c6_58ec328f","line":7,"range":{"start_line":7,"start_character":0,"end_line":7,"end_character":4},"updated":"2016-03-22 10:09:52.000000000","message":"s/Adds/Add","commit_id":"9ec7a2aa00613d015365fed8fcc17ee4940999e1"},{"author":{"_account_id":10485,"name":"Ankit Agrawal","email":"ankitagrawal8188@gmail.com","username":"ankitagrawal"},"change_message_id":"86c01944faa809295dd41da1c81157de7154c055","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Adds missing user message"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Currently theres is no message given by python cinderclient after"},{"line_number":10,"context_line":"successfully accepting the request of delete snapshot and force delete"},{"line_number":11,"context_line":"volume."},{"line_number":12,"context_line":"This Fix adds the message."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"fa0719c6_98feca36","line":9,"range":{"start_line":9,"start_character":10,"end_line":9,"end_character":16},"updated":"2016-03-22 10:09:52.000000000","message":"s/theres/there","commit_id":"9ec7a2aa00613d015365fed8fcc17ee4940999e1"},{"author":{"_account_id":16708,"name":"Kendall Nelson","display_name":"Kendall (diablo_rojo)","email":"kennelson11@gmail.com","username":"kjnelson"},"change_message_id":"e1ae46abe9f952d1fe641bcd260fa151e4d9f3c9","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Currently there is no message given by python cinderclient after"},{"line_number":10,"context_line":"successfully accepting the request of delete snapshot and force delete"},{"line_number":11,"context_line":"volume."},{"line_number":12,"context_line":"This Fix adds the message."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Change-Id: Ia6933dd45f445fa312c5c64ca741dc2868aa4fa2"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"ba0121b8_4c131f5d","line":11,"updated":"2016-03-28 16:35:36.000000000","message":"Might be good to add something related to Sean\u0027s comment about how this is making these commands more consistent with others.","commit_id":"525da5600b4519169bcf05eeb269c7fdc4d110d1"},{"author":{"_account_id":16708,"name":"Kendall Nelson","display_name":"Kendall (diablo_rojo)","email":"kennelson11@gmail.com","username":"kjnelson"},"change_message_id":"e1ae46abe9f952d1fe641bcd260fa151e4d9f3c9","unresolved":false,"context_lines":[{"line_number":9,"context_line":"Currently there is no message given by python cinderclient after"},{"line_number":10,"context_line":"successfully accepting the request of delete snapshot and force delete"},{"line_number":11,"context_line":"volume."},{"line_number":12,"context_line":"This Fix adds the message."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Change-Id: Ia6933dd45f445fa312c5c64ca741dc2868aa4fa2"},{"line_number":15,"context_line":"Closes-Bug: #1534060"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"ba0121b8_21099432","line":12,"range":{"start_line":12,"start_character":5,"end_line":12,"end_character":8},"updated":"2016-03-28 16:35:36.000000000","message":"Fix-\u003e fix","commit_id":"525da5600b4519169bcf05eeb269c7fdc4d110d1"}],"cinderclient/v1/shell.py":[{"author":{"_account_id":10485,"name":"Ankit Agrawal","email":"ankitagrawal8188@gmail.com","username":"ankitagrawal"},"change_message_id":"86c01944faa809295dd41da1c81157de7154c055","unresolved":false,"context_lines":[{"line_number":335,"context_line":"        try:"},{"line_number":336,"context_line":"            utils.find_volume(cs, volume).force_delete()"},{"line_number":337,"context_line":"            print(\"Request to force delete volume %s has been accepted.\""},{"line_number":338,"context_line":"                 % (volume))"},{"line_number":339,"context_line":"        except Exception as e:"},{"line_number":340,"context_line":"            failure_count +\u003d 1"},{"line_number":341,"context_line":"            print(\"Delete for volume %s failed: %s\" % (volume, e))"}],"source_content_type":"text/x-python","patch_set":3,"id":"fa0719c6_98cd2a60","line":338,"range":{"start_line":338,"start_character":26,"end_line":338,"end_character":28},"updated":"2016-03-22 10:09:52.000000000","message":"Below exception is not supposed to be raised by this print statement so it should not be in try block. You can put this in \"else:\" block instead.","commit_id":"9ec7a2aa00613d015365fed8fcc17ee4940999e1"},{"author":{"_account_id":10485,"name":"Ankit Agrawal","email":"ankitagrawal8188@gmail.com","username":"ankitagrawal"},"change_message_id":"86c01944faa809295dd41da1c81157de7154c055","unresolved":false,"context_lines":[{"line_number":527,"context_line":"    for snapshot in args.snapshot:"},{"line_number":528,"context_line":"        try:"},{"line_number":529,"context_line":"            _find_volume_snapshot(cs, snapshot).delete()"},{"line_number":530,"context_line":"            print(\"Request to delete snapshot %s has been accepted.\""},{"line_number":531,"context_line":"                % (snapshot))"},{"line_number":532,"context_line":"        except Exception as e:"},{"line_number":533,"context_line":"            failure_count +\u003d 1"}],"source_content_type":"text/x-python","patch_set":3,"id":"fa0719c6_38133eba","line":530,"range":{"start_line":530,"start_character":12,"end_line":530,"end_character":17},"updated":"2016-03-22 10:09:52.000000000","message":"Same as above.","commit_id":"9ec7a2aa00613d015365fed8fcc17ee4940999e1"},{"author":{"_account_id":10485,"name":"Ankit Agrawal","email":"ankitagrawal8188@gmail.com","username":"ankitagrawal"},"change_message_id":"42bb37a1f196d37bf9befde0352c72f00ee8c115","unresolved":false,"context_lines":[{"line_number":335,"context_line":"        try:"},{"line_number":336,"context_line":"            utils.find_volume(cs, volume).force_delete()"},{"line_number":337,"context_line":"            print(\"Request to force delete volume %s has been accepted.\""},{"line_number":338,"context_line":"                 % (volume))"},{"line_number":339,"context_line":"        except Exception as e:"},{"line_number":340,"context_line":"            failure_count +\u003d 1"},{"line_number":341,"context_line":"            print(\"Delete for volume %s failed: %s\" % (volume, e))"}],"source_content_type":"text/x-python","patch_set":4,"id":"fa0719c6_0e3550f6","line":338,"range":{"start_line":338,"start_character":26,"end_line":338,"end_character":28},"updated":"2016-03-22 10:52:12.000000000","message":"We should move this print statement in \u0027else\u0027 block but its OK to keep it here in favor of maintaining the consistency with other APIs ex. do_delete","commit_id":"525da5600b4519169bcf05eeb269c7fdc4d110d1"}],"cinderclient/v2/shell.py":[{"author":{"_account_id":10485,"name":"Ankit Agrawal","email":"ankitagrawal8188@gmail.com","username":"ankitagrawal"},"change_message_id":"86c01944faa809295dd41da1c81157de7154c055","unresolved":false,"context_lines":[{"line_number":468,"context_line":"        try:"},{"line_number":469,"context_line":"            utils.find_volume(cs, volume).force_delete()"},{"line_number":470,"context_line":"            print(\"Request to force delete volume %s has been accepted.\""},{"line_number":471,"context_line":"                 % (volume))"},{"line_number":472,"context_line":"        except Exception as e:"},{"line_number":473,"context_line":"            failure_count +\u003d 1"},{"line_number":474,"context_line":"            print(\"Delete for volume %s failed: %s\" % (volume, e))"}],"source_content_type":"text/x-python","patch_set":3,"id":"fa0719c6_788496d3","line":471,"range":{"start_line":471,"start_character":20,"end_line":471,"end_character":26},"updated":"2016-03-22 10:09:52.000000000","message":"Below exception is not supposed to be raised by this print statement so it should not be in try block. You can put this in \"else:\" block instead.","commit_id":"9ec7a2aa00613d015365fed8fcc17ee4940999e1"},{"author":{"_account_id":10485,"name":"Ankit Agrawal","email":"ankitagrawal8188@gmail.com","username":"ankitagrawal"},"change_message_id":"86c01944faa809295dd41da1c81157de7154c055","unresolved":false,"context_lines":[{"line_number":762,"context_line":"        try:"},{"line_number":763,"context_line":"            _find_volume_snapshot(cs, snapshot).delete()"},{"line_number":764,"context_line":"            print(\"Request to delete snapshot %s has been accepted.\""},{"line_number":765,"context_line":"                % (snapshot))"},{"line_number":766,"context_line":"        except Exception as e:"},{"line_number":767,"context_line":"            failure_count +\u003d 1"},{"line_number":768,"context_line":"            print(\"Delete for snapshot %s failed: %s\" % (snapshot, e))"}],"source_content_type":"text/x-python","patch_set":3,"id":"fa0719c6_b8558e51","line":765,"range":{"start_line":765,"start_character":27,"end_line":765,"end_character":29},"updated":"2016-03-22 10:09:52.000000000","message":"same as above.","commit_id":"9ec7a2aa00613d015365fed8fcc17ee4940999e1"}]}