)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":7198,"name":"Jay Bryant","email":"jungleboyj@electronicjungle.net","username":"jsbryant"},"change_message_id":"81d70e6a2342680f6abe4c57b7365721a24615ad","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Cinder client reset-state improvements"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Now we can use \u0027cinder reset-state\u0027 command for \u0027volume\u0027,"},{"line_number":10,"context_line":"\u0027snapshot\u0027, \u0027bakcup\u0027, \u0027group\u0027 and \u0027group-snapshot\u0027."},{"line_number":11,"context_line":"Also change volume\u0027s default status from \u0027available\u0027 to"},{"line_number":12,"context_line":"\u0027None\u0027 when not any status is specified."},{"line_number":13,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":5,"id":"3f044301_7f886e1e","line":10,"range":{"start_line":10,"start_character":13,"end_line":10,"end_character":19},"updated":"2017-05-04 15:00:46.000000000","message":"backup","commit_id":"476a340d55088927023763ca26501a770e8fdb12"},{"author":{"_account_id":23083,"name":"TommyLike","email":"tommylikehu@gmail.com","username":"TommyLike"},"change_message_id":"2420422b7e321e5bff24e3ed4b2acde352a3ae6f","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Cinder client reset-state improvements"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Now we can use \u0027cinder reset-state\u0027 command for \u0027volume\u0027,"},{"line_number":10,"context_line":"\u0027snapshot\u0027, \u0027bakcup\u0027, \u0027group\u0027 and \u0027group-snapshot\u0027."},{"line_number":11,"context_line":"Also change volume\u0027s default status from \u0027available\u0027 to"},{"line_number":12,"context_line":"\u0027None\u0027 when not any status is specified."},{"line_number":13,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":5,"id":"3f044301_7fd50eb9","line":10,"range":{"start_line":10,"start_character":13,"end_line":10,"end_character":19},"in_reply_to":"3f044301_7f886e1e","updated":"2017-05-04 15:19:09.000000000","message":"Done","commit_id":"476a340d55088927023763ca26501a770e8fdb12"}],"cinderclient/v2/shell.py":[{"author":{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},"change_message_id":"2f8938171847ae45cf0aaf6f6a9dc707df1aa93a","unresolved":false,"context_lines":[{"line_number":738,"context_line":"        \"``cinder snapshot-reset-state`` is deprecated, use ``cinder \""},{"line_number":739,"context_line":"        \"reset-state`` instead. The ``cinder snapshot-reset-state`` command \""},{"line_number":740,"context_line":"        \"may be removed in the Q release or next major release of \""},{"line_number":741,"context_line":"        \"cinderclient.\")"},{"line_number":742,"context_line":"    failure_count \u003d 0"},{"line_number":743,"context_line":""},{"line_number":744,"context_line":"    single \u003d (len(args.snapshot) \u003d\u003d 1)"}],"source_content_type":"text/x-python","patch_set":4,"id":"3f044301_9c724511","line":741,"updated":"2017-05-03 19:16:19.000000000","message":"I\u0027m not sure this is true for v2.  It may be better to handle deprecations in a separate patch and focus here on just landing the new code.","commit_id":"4e5bef20f86b6f8b7601320ca831dc481b81cb15"}],"cinderclient/v3/group_snapshots.py":[{"author":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"change_message_id":"87c491674b06a0cfbc5b20be07401954171832ec","unresolved":false,"context_lines":[{"line_number":36,"context_line":"        return self.manager.update(self, **kwargs)"},{"line_number":37,"context_line":""},{"line_number":38,"context_line":"    def reset_state(self, state):"},{"line_number":39,"context_line":"        \"\"\"Reset the group snapshot\u0027s state with specified one\"\"\""},{"line_number":40,"context_line":"        return self.manager.reset_state(self, state)"},{"line_number":41,"context_line":""},{"line_number":42,"context_line":""}],"source_content_type":"text/x-python","patch_set":6,"id":"ff0f0b1f_1ed251d3","line":39,"range":{"start_line":39,"start_character":61,"end_line":39,"end_character":63},"updated":"2017-05-19 21:11:59.000000000","message":"Extremely minor nit: One line docstrings should end with a period.","commit_id":"96af504cb4fbb8b60be83007dcb0f8a980c77352"}],"cinderclient/v3/shell.py":[{"author":{"_account_id":7198,"name":"Jay Bryant","email":"jungleboyj@electronicjungle.net","username":"jsbryant"},"change_message_id":"81d70e6a2342680f6abe4c57b7365721a24615ad","unresolved":false,"context_lines":[{"line_number":226,"context_line":"           help\u003d(\u0027This only used in volume entity. Clears the migration \u0027"},{"line_number":227,"context_line":"                 \u0027status of the volume in the DataBase that indicates the \u0027"},{"line_number":228,"context_line":"                 \u0027volume is source or destination of volume migration, \u0027"},{"line_number":229,"context_line":"                 \u0027with no regard to the actual status.\u0027))"},{"line_number":230,"context_line":"def do_reset_state(cs, args):"},{"line_number":231,"context_line":"    \"\"\"Explicitly updates the entity state in the Cinder database."},{"line_number":232,"context_line":""}],"source_content_type":"text/x-python","patch_set":5,"id":"3f044301_df2d4247","line":229,"range":{"start_line":229,"start_character":18,"end_line":229,"end_character":54},"updated":"2017-05-04 15:00:46.000000000","message":"Feel like this is one of those drug commercials.  reset-state may cause itching, burning, redness and tingling.  Do not use reset state if you do not know the actual state of your volume or how to find the actual status.  Ask your administrator if reset state is right for you.  :-)","commit_id":"476a340d55088927023763ca26501a770e8fdb12"},{"author":{"_account_id":23083,"name":"TommyLike","email":"tommylikehu@gmail.com","username":"TommyLike"},"change_message_id":"2420422b7e321e5bff24e3ed4b2acde352a3ae6f","unresolved":false,"context_lines":[{"line_number":226,"context_line":"           help\u003d(\u0027This only used in volume entity. Clears the migration \u0027"},{"line_number":227,"context_line":"                 \u0027status of the volume in the DataBase that indicates the \u0027"},{"line_number":228,"context_line":"                 \u0027volume is source or destination of volume migration, \u0027"},{"line_number":229,"context_line":"                 \u0027with no regard to the actual status.\u0027))"},{"line_number":230,"context_line":"def do_reset_state(cs, args):"},{"line_number":231,"context_line":"    \"\"\"Explicitly updates the entity state in the Cinder database."},{"line_number":232,"context_line":""}],"source_content_type":"text/x-python","patch_set":5,"id":"3f044301_9f3a1ae5","line":229,"range":{"start_line":229,"start_character":18,"end_line":229,"end_character":54},"in_reply_to":"3f044301_df2d4247","updated":"2017-05-04 15:19:09.000000000","message":"Yes, you are right! This commercial is version two. and our first version is in v2/shell.py","commit_id":"476a340d55088927023763ca26501a770e8fdb12"},{"author":{"_account_id":7198,"name":"Jay Bryant","email":"jungleboyj@electronicjungle.net","username":"jsbryant"},"change_message_id":"81d70e6a2342680f6abe4c57b7365721a24615ad","unresolved":false,"context_lines":[{"line_number":247,"context_line":"        try:"},{"line_number":248,"context_line":"            collector(cs, entity).reset_state(*argument)"},{"line_number":249,"context_line":"        except Exception as e:"},{"line_number":250,"context_line":"            print(e)"},{"line_number":251,"context_line":"            failure_count +\u003d 1"},{"line_number":252,"context_line":"            msg \u003d \"Reset state for entity %s failed: %s\" % (entity, e)"},{"line_number":253,"context_line":"            if not single:"}],"source_content_type":"text/x-python","patch_set":5,"id":"3f044301_3fe30608","line":250,"range":{"start_line":250,"start_character":12,"end_line":250,"end_character":20},"updated":"2017-05-04 15:00:46.000000000","message":"I don\u0027t think we need this here since we have it printed in the message below.  Right?","commit_id":"476a340d55088927023763ca26501a770e8fdb12"}],"releasenotes/notes/add-generic-reset-state-command-d83v1f3accbf5807.yaml":[{"author":{"_account_id":7198,"name":"Jay Bryant","email":"jungleboyj@electronicjungle.net","username":"jsbryant"},"change_message_id":"81d70e6a2342680f6abe4c57b7365721a24615ad","unresolved":false,"context_lines":[{"line_number":2,"context_line":"features:"},{"line_number":3,"context_line":"  - Use \u0027cinder reset-state\u0027 as generic resource reset"},{"line_number":4,"context_line":"    state command for resource \u0027volume\u0027, \u0027snapshot\u0027, \u0027backup\u0027"},{"line_number":5,"context_line":"    \u0027group\u0027 and \u0027group-snapshot\u0027, mark \u0027cinder snapshot-reset-state\u0027"},{"line_number":6,"context_line":"    and \u0027cinder backup-reset-state\u0027 commands deprecated."},{"line_number":7,"context_line":"    Also change volume\u0027s default status from \u0027available\u0027"},{"line_number":8,"context_line":"    to none when not any status is specified."}],"source_content_type":"text/x-yaml","patch_set":5,"id":"3f044301_3f91c653","line":6,"range":{"start_line":5,"start_character":34,"end_line":6,"end_character":56},"updated":"2017-05-04 15:00:46.000000000","message":"I think this part should be moved to a \u0027Deprecation Notes\u0027 section.","commit_id":"476a340d55088927023763ca26501a770e8fdb12"},{"author":{"_account_id":23083,"name":"TommyLike","email":"tommylikehu@gmail.com","username":"TommyLike"},"change_message_id":"2420422b7e321e5bff24e3ed4b2acde352a3ae6f","unresolved":false,"context_lines":[{"line_number":2,"context_line":"features:"},{"line_number":3,"context_line":"  - Use \u0027cinder reset-state\u0027 as generic resource reset"},{"line_number":4,"context_line":"    state command for resource \u0027volume\u0027, \u0027snapshot\u0027, \u0027backup\u0027"},{"line_number":5,"context_line":"    \u0027group\u0027 and \u0027group-snapshot\u0027, mark \u0027cinder snapshot-reset-state\u0027"},{"line_number":6,"context_line":"    and \u0027cinder backup-reset-state\u0027 commands deprecated."},{"line_number":7,"context_line":"    Also change volume\u0027s default status from \u0027available\u0027"},{"line_number":8,"context_line":"    to none when not any status is specified."}],"source_content_type":"text/x-yaml","patch_set":5,"id":"3f044301_dfef8285","line":6,"range":{"start_line":5,"start_character":34,"end_line":6,"end_character":56},"in_reply_to":"3f044301_3f91c653","updated":"2017-05-04 15:19:09.000000000","message":"Thanks, I would remove this part, and deprecate the xx-reset-state command with another patch:).","commit_id":"476a340d55088927023763ca26501a770e8fdb12"},{"author":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"change_message_id":"87c491674b06a0cfbc5b20be07401954171832ec","unresolved":false,"context_lines":[{"line_number":3,"context_line":"  - Use \u0027cinder reset-state\u0027 as generic resource reset"},{"line_number":4,"context_line":"    state command for resource \u0027volume\u0027, \u0027snapshot\u0027, \u0027backup\u0027"},{"line_number":5,"context_line":"    \u0027group\u0027 and \u0027group-snapshot\u0027. Also change volume\u0027s"},{"line_number":6,"context_line":"    default status from \u0027available\u0027 to none when not any"},{"line_number":7,"context_line":"    status is specified."}],"source_content_type":"text/x-yaml","patch_set":6,"id":"ff0f0b1f_3ef4f55a","line":6,"range":{"start_line":6,"start_character":44,"end_line":6,"end_character":56},"updated":"2017-05-19 21:11:59.000000000","message":"when no","commit_id":"96af504cb4fbb8b60be83007dcb0f8a980c77352"}]}
