)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},"change_message_id":"f850266b93f687b9afdc901c244dce1c7c251418","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Reset-state\u0027s default parameter needs correction"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Changed reset-state default parameter to be \u0027active\u0027"},{"line_number":10,"context_line":"instead of \u0027error\u0027."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Change-Id: I1995b13e707e38a8a8f4de3fd6cfa8497e21ecdf"},{"line_number":13,"context_line":"Closes-Bug: #1674247"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"3f044301_d6bf0bce","line":10,"updated":"2017-05-09 21:54:57.000000000","message":"Why?  I know you have a \"Closes-Bug\" footer in here, but the commit message needs to be able to stand up on its own; we want contributors to be able to determine why a change was merged even if they happen to be without network at the time they\u0027re looking.  (This also helps us if the bug repository ever has a catastrophe.)","commit_id":"003cad9a34eded4a0a0dbddf43b5be1173cac113"}],"novaclient/v2/shell.py":[{"author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"change_message_id":"5ca819afd09a255037ac57f60552c9cbd48bce00","unresolved":false,"context_lines":[{"line_number":3282,"context_line":"@utils.arg("},{"line_number":3283,"context_line":"    \u0027--active\u0027, action\u003d\u0027store_const\u0027, dest\u003d\u0027state\u0027,"},{"line_number":3284,"context_line":"    default\u003d\u0027error\u0027, const\u003d\u0027active\u0027,"},{"line_number":3285,"context_line":"    help\u003d_(\u0027Request the server be reset to \"active\" state instead \u0027"},{"line_number":3286,"context_line":"           \u0027of \"error\" state (the default).\u0027))"},{"line_number":3287,"context_line":"def do_reset_state(cs, args):"},{"line_number":3288,"context_line":"    \"\"\"Reset the state of a server.\"\"\""},{"line_number":3289,"context_line":"    failure_flag \u003d False"}],"source_content_type":"text/x-python","patch_set":2,"id":"df140735_7f4ac783","side":"PARENT","line":3286,"range":{"start_line":3285,"start_character":58,"end_line":3286,"end_character":42},"updated":"2017-06-06 17:58:26.000000000","message":"This clearly says the default status is \"error\". According to the bug report, this change is just the reporter\u0027s thinking.\nI think it is hard to change the default value with such thing only..","commit_id":"ff1eff18d9815ffa2588034769c842615e76507d"},{"author":{"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},"change_message_id":"52da15d2ca5c0063dcad04c70e16bb7ea1e96d5e","unresolved":false,"context_lines":[{"line_number":3282,"context_line":"@utils.arg("},{"line_number":3283,"context_line":"    \u0027--active\u0027, action\u003d\u0027store_const\u0027, dest\u003d\u0027state\u0027,"},{"line_number":3284,"context_line":"    default\u003d\u0027active\u0027, const\u003d\u0027active\u0027,"},{"line_number":3285,"context_line":"    help\u003d_(\u0027Request the server be reset to \"active\" state.\u0027))"},{"line_number":3286,"context_line":"def do_reset_state(cs, args):"},{"line_number":3287,"context_line":"    \"\"\"Reset the state of a server.\"\"\""},{"line_number":3288,"context_line":"    failure_flag \u003d False"}],"source_content_type":"text/x-python","patch_set":2,"id":"df140735_0ecd3b65","line":3285,"updated":"2017-06-02 15:20:38.000000000","message":"So, this means that --active no longer does anything.  That means we should deprecate the option, so we can remove it in a future release.","commit_id":"720d5a787b22e3644744db72115eb6da18d12c11"}]}