)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"change_message_id":"b4bcbe171cf8a9429651c84cc776549b22bc2c42","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"* Fix volume transfer request accept to actually not crash when"},{"line_number":10,"context_line":"  trying to call Volume API."},{"line_number":11,"context_line":"* Fix volume transfer request accept syntax to have only one"},{"line_number":12,"context_line":"  positional argument, which is the ID of the resource in the command"},{"line_number":13,"context_line":"* Change the output column order in volume transfer request list to"},{"line_number":14,"context_line":"  have ID followed by Name then the remaining columns."},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"Closes-bug: 1633582"},{"line_number":17,"context_line":"Change-Id: I5cc005f039d171cc70859f60e7fe649b09ead229"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"fa6399be_d61ee30a","line":14,"range":{"start_line":11,"start_character":0,"end_line":14,"end_character":54},"updated":"2016-10-15 11:30:38.000000000","message":"i agree with these two changes ++","commit_id":"74f4829b531d8341b23b503583dbd28b705223d5"}],"doc/source/command-objects/volume-transfer-request.rst":[{"author":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"change_message_id":"b4bcbe171cf8a9429651c84cc776549b22bc2c42","unresolved":false,"context_lines":[{"line_number":18,"context_line":""},{"line_number":19,"context_line":".. option:: --auth-key \u003ckey\u003e"},{"line_number":20,"context_line":""},{"line_number":21,"context_line":"    Volume transfer request uthentication key"},{"line_number":22,"context_line":""},{"line_number":23,"context_line":".. _volume_transfer_request_accept:"},{"line_number":24,"context_line":".. describe:: \u003ctransfer-request\u003e"}],"source_content_type":"text/x-rst","patch_set":1,"id":"fa6399be_4964ffb7","line":21,"range":{"start_line":21,"start_character":28,"end_line":21,"end_character":41},"updated":"2016-10-15 11:30:38.000000000","message":"typo here","commit_id":"74f4829b531d8341b23b503583dbd28b705223d5"},{"author":{"_account_id":170,"name":"Mike Perez","email":"thingee@gmail.com","username":"thingee"},"change_message_id":"ea04a4c9989218a33ca545f26c5c57c7ba0d0c30","unresolved":false,"context_lines":[{"line_number":18,"context_line":""},{"line_number":19,"context_line":".. option:: --auth-key \u003ckey\u003e"},{"line_number":20,"context_line":""},{"line_number":21,"context_line":"    Volume transfer request uthentication key"},{"line_number":22,"context_line":""},{"line_number":23,"context_line":".. _volume_transfer_request_accept:"},{"line_number":24,"context_line":".. describe:: \u003ctransfer-request\u003e"}],"source_content_type":"text/x-rst","patch_set":1,"id":"9a30ddce_a0518c10","line":21,"range":{"start_line":21,"start_character":28,"end_line":21,"end_character":41},"in_reply_to":"fa6399be_4964ffb7","updated":"2017-03-08 19:52:32.000000000","message":"Done","commit_id":"74f4829b531d8341b23b503583dbd28b705223d5"}],"openstackclient/volume/v2/volume_transfer_request.py":[{"author":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"change_message_id":"b4bcbe171cf8a9429651c84cc776549b22bc2c42","unresolved":false,"context_lines":[{"line_number":46,"context_line":""},{"line_number":47,"context_line":"    def take_action(self, parsed_args):"},{"line_number":48,"context_line":"        volume_client \u003d self.app.client_manager.volume"},{"line_number":49,"context_line":"        transfer_request_id \u003d utils.find_resource("},{"line_number":50,"context_line":"            volume_client.transfers, parsed_args.transfer_request).id"},{"line_number":51,"context_line":"        transfer_accept \u003d volume_client.transfers.accept("},{"line_number":52,"context_line":"            transfer_request_id, parsed_args.auth_key)"},{"line_number":53,"context_line":"        transfer_accept._info.pop(\"links\", None)"}],"source_content_type":"text/x-python","patch_set":1,"id":"fa6399be_1615bbe3","side":"PARENT","line":50,"range":{"start_line":49,"start_character":8,"end_line":50,"end_character":69},"updated":"2016-10-15 11:30:38.000000000","message":"why not wrap this in a try/except and should any error come back, then resort to using just an ID?","commit_id":"3a2716362820f47e9d9ff0ba7dcb50c191092192"},{"author":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"change_message_id":"ddf2707791fbdcf16aa0964b9ab71ba74ba2c942","unresolved":false,"context_lines":[{"line_number":47,"context_line":"    def take_action(self, parsed_args):"},{"line_number":48,"context_line":"        volume_client \u003d self.app.client_manager.volume"},{"line_number":49,"context_line":"        transfer_request_id \u003d utils.find_resource("},{"line_number":50,"context_line":"            volume_client.transfers, parsed_args.transfer_request).id"},{"line_number":51,"context_line":"        transfer_accept \u003d volume_client.transfers.accept("},{"line_number":52,"context_line":"            transfer_request_id, parsed_args.auth_key)"},{"line_number":53,"context_line":"        transfer_accept._info.pop(\"links\", None)"}],"source_content_type":"text/x-python","patch_set":1,"id":"fa6399be_8eae0f2c","side":"PARENT","line":50,"in_reply_to":"fa6399be_1615bbe3","updated":"2016-10-15 22:47:45.000000000","message":"Because until Cinder is changed, an error will ALWAYS come back.  Without a proveleged service doing the cross-project lookup, there is no way to fix this for a non-admin.\n\nShould we comment the code and leave it with a note as a reminder?  That\u0027s why I left the comment below, which is not too clear... since I have to rev this anyway I\u0027ll clean that up.","commit_id":"3a2716362820f47e9d9ff0ba7dcb50c191092192"},{"author":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"change_message_id":"b4bcbe171cf8a9429651c84cc776549b22bc2c42","unresolved":false,"context_lines":[{"line_number":47,"context_line":"        parser.add_argument("},{"line_number":48,"context_line":"            \u0027--auth-key\u0027,"},{"line_number":49,"context_line":"            metavar\u003d\"\u003ckey\u003e\","},{"line_number":50,"context_line":"            help\u003d_(\u0027Volume transfer request uthentication key\u0027),"},{"line_number":51,"context_line":"        )"},{"line_number":52,"context_line":"        return parser"},{"line_number":53,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"fa6399be_095047d3","line":50,"range":{"start_line":50,"start_character":44,"end_line":50,"end_character":57},"updated":"2016-10-15 11:30:38.000000000","message":"typo here","commit_id":"74f4829b531d8341b23b503583dbd28b705223d5"},{"author":{"_account_id":170,"name":"Mike Perez","email":"thingee@gmail.com","username":"thingee"},"change_message_id":"ea04a4c9989218a33ca545f26c5c57c7ba0d0c30","unresolved":false,"context_lines":[{"line_number":47,"context_line":"        parser.add_argument("},{"line_number":48,"context_line":"            \u0027--auth-key\u0027,"},{"line_number":49,"context_line":"            metavar\u003d\"\u003ckey\u003e\","},{"line_number":50,"context_line":"            help\u003d_(\u0027Volume transfer request uthentication key\u0027),"},{"line_number":51,"context_line":"        )"},{"line_number":52,"context_line":"        return parser"},{"line_number":53,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"9a30ddce_c06f60c3","line":50,"range":{"start_line":50,"start_character":44,"end_line":50,"end_character":57},"in_reply_to":"fa6399be_095047d3","updated":"2017-03-08 19:52:32.000000000","message":"Done","commit_id":"74f4829b531d8341b23b503583dbd28b705223d5"}]}
