)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":15100,"name":"zhongjun","email":"jun.zhongjun2@gmail.com","username":"zhongjun2"},"change_message_id":"4b43db139fce483c8a1c733c41ea74a4ad9f4e6c","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Fixing cinder-status value when the service is disabled"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":" When a service is  in \u0027down\u0027 state and a service-disable command is "},{"line_number":10,"context_line":"launced  by the user ,the state is shown as \u0027up\u0027 if a user launches a"},{"line_number":11,"context_line":"service-list command before threshold time(CONF.service_down_time)."},{"line_number":12,"context_line":"This fix will set the state of any cinder service to \u0027down\u0027 state if"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":7,"id":"da9b358b_7d46bcc0","line":9,"updated":"2015-03-30 09:51:40.000000000","message":"Should have only one space between \u0027is  in\u0027.\nDelete space before \u0027When\u0027.","commit_id":"85be9a3d0ef5bcd0371a9f3f7b9051ee17a22f75"},{"author":{"_account_id":13900,"name":"Accela Zhao","email":"accelazh@gmail.com","username":"accelazh"},"change_message_id":"8a790de761d224cced98df9f2cafc9327e7d09c7","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Fixing cinder-status value when the service is disabled"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":" When a service is  in \u0027down\u0027 state and a service-disable command is "},{"line_number":10,"context_line":"launced  by the user ,the state is shown as \u0027up\u0027 if a user launches a"},{"line_number":11,"context_line":"service-list command before threshold time(CONF.service_down_time)."},{"line_number":12,"context_line":"This fix will set the state of any cinder service to \u0027down\u0027 state if"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":7,"id":"da86d52c_35e605b1","line":9,"updated":"2015-02-11 14:27:49.000000000","message":"Tailing space.","commit_id":"85be9a3d0ef5bcd0371a9f3f7b9051ee17a22f75"},{"author":{"_account_id":1736,"name":"Ivan Kolodyazhny","email":"e0ne@e0ne.info","username":"e0ne"},"change_message_id":"cdec1c23732697cc5ee8c8015fd6394193ebbf91","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Fixing cinder-status value when the service is disabled"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":" When a service is  in \u0027down\u0027 state and a service-disable command is "},{"line_number":10,"context_line":"launced  by the user ,the state is shown as \u0027up\u0027 if a user launches a"},{"line_number":11,"context_line":"service-list command before threshold time(CONF.service_down_time)."},{"line_number":12,"context_line":"This fix will set the state of any cinder service to \u0027down\u0027 state if"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":7,"id":"da86d52c_ef255ec2","line":9,"in_reply_to":"da86d52c_35e605b1","updated":"2015-02-11 22:33:13.000000000","message":"The same in the start of the lime","commit_id":"85be9a3d0ef5bcd0371a9f3f7b9051ee17a22f75"},{"author":{"_account_id":15100,"name":"zhongjun","email":"jun.zhongjun2@gmail.com","username":"zhongjun2"},"change_message_id":"4b43db139fce483c8a1c733c41ea74a4ad9f4e6c","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Fixing cinder-status value when the service is disabled"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":" When a service is  in \u0027down\u0027 state and a service-disable command is "},{"line_number":10,"context_line":"launced  by the user ,the state is shown as \u0027up\u0027 if a user launches a"},{"line_number":11,"context_line":"service-list command before threshold time(CONF.service_down_time)."},{"line_number":12,"context_line":"This fix will set the state of any cinder service to \u0027down\u0027 state if"},{"line_number":13,"context_line":"the disabled field of the service is  set ,without considering the "}],"source_content_type":"text/x-gerrit-commit-message","patch_set":7,"id":"da9b358b_1d8330ce","line":10,"updated":"2015-03-30 09:51:40.000000000","message":"Should have only one space between \u0027launced  by\u0027.","commit_id":"85be9a3d0ef5bcd0371a9f3f7b9051ee17a22f75"},{"author":{"_account_id":13900,"name":"Accela Zhao","email":"accelazh@gmail.com","username":"accelazh"},"change_message_id":"8a790de761d224cced98df9f2cafc9327e7d09c7","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Fixing cinder-status value when the service is disabled"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":" When a service is  in \u0027down\u0027 state and a service-disable command is "},{"line_number":10,"context_line":"launced  by the user ,the state is shown as \u0027up\u0027 if a user launches a"},{"line_number":11,"context_line":"service-list command before threshold time(CONF.service_down_time)."},{"line_number":12,"context_line":"This fix will set the state of any cinder service to \u0027down\u0027 state if"},{"line_number":13,"context_line":"the disabled field of the service is  set ,without considering the "}],"source_content_type":"text/x-gerrit-commit-message","patch_set":7,"id":"da86d52c_d571290d","line":10,"updated":"2015-02-11 14:27:49.000000000","message":"Typo \u0027launched\u0027? Extra space before \u0027by\u0027? Wrong position for \u0027,\u0027 after \u0027by the user\u0027?","commit_id":"85be9a3d0ef5bcd0371a9f3f7b9051ee17a22f75"},{"author":{"_account_id":15100,"name":"zhongjun","email":"jun.zhongjun2@gmail.com","username":"zhongjun2"},"change_message_id":"4b43db139fce483c8a1c733c41ea74a4ad9f4e6c","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":" When a service is  in \u0027down\u0027 state and a service-disable command is "},{"line_number":10,"context_line":"launced  by the user ,the state is shown as \u0027up\u0027 if a user launches a"},{"line_number":11,"context_line":"service-list command before threshold time(CONF.service_down_time)."},{"line_number":12,"context_line":"This fix will set the state of any cinder service to \u0027down\u0027 state if"},{"line_number":13,"context_line":"the disabled field of the service is  set ,without considering the "},{"line_number":14,"context_line":"threshold time."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":7,"id":"da9b358b_9d774002","line":11,"updated":"2015-03-30 09:51:40.000000000","message":"Insert space before (CONF.service_down_time).","commit_id":"85be9a3d0ef5bcd0371a9f3f7b9051ee17a22f75"},{"author":{"_account_id":15100,"name":"zhongjun","email":"jun.zhongjun2@gmail.com","username":"zhongjun2"},"change_message_id":"4b43db139fce483c8a1c733c41ea74a4ad9f4e6c","unresolved":false,"context_lines":[{"line_number":10,"context_line":"launced  by the user ,the state is shown as \u0027up\u0027 if a user launches a"},{"line_number":11,"context_line":"service-list command before threshold time(CONF.service_down_time)."},{"line_number":12,"context_line":"This fix will set the state of any cinder service to \u0027down\u0027 state if"},{"line_number":13,"context_line":"the disabled field of the service is  set ,without considering the "},{"line_number":14,"context_line":"threshold time."},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"Change-Id: If9534d7ebbde104e8ff00fc3bc7888aabb497ba5"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":7,"id":"da9b358b_fdd52cb9","line":13,"updated":"2015-03-30 09:51:40.000000000","message":"Delete space between \u0027is  set\u0027.","commit_id":"85be9a3d0ef5bcd0371a9f3f7b9051ee17a22f75"},{"author":{"_account_id":13900,"name":"Accela Zhao","email":"accelazh@gmail.com","username":"accelazh"},"change_message_id":"8a790de761d224cced98df9f2cafc9327e7d09c7","unresolved":false,"context_lines":[{"line_number":10,"context_line":"launced  by the user ,the state is shown as \u0027up\u0027 if a user launches a"},{"line_number":11,"context_line":"service-list command before threshold time(CONF.service_down_time)."},{"line_number":12,"context_line":"This fix will set the state of any cinder service to \u0027down\u0027 state if"},{"line_number":13,"context_line":"the disabled field of the service is  set ,without considering the "},{"line_number":14,"context_line":"threshold time."},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"Change-Id: If9534d7ebbde104e8ff00fc3bc7888aabb497ba5"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":7,"id":"da86d52c_359d250b","line":13,"updated":"2015-02-11 14:27:49.000000000","message":"Same as the above.","commit_id":"85be9a3d0ef5bcd0371a9f3f7b9051ee17a22f75"}],"cinder/api/contrib/services.py":[{"author":{"_account_id":7198,"name":"Jay Bryant","email":"jungleboyj@electronicjungle.net","username":"jsbryant"},"change_message_id":"1597716b5a93379ba9125c0a6527e247b70f280d","unresolved":false,"context_lines":[{"line_number":106,"context_line":"        for svc in services:"},{"line_number":107,"context_line":"            delta \u003d now - (svc[\u0027updated_at\u0027] or svc[\u0027created_at\u0027])"},{"line_number":108,"context_line":"            alive \u003d abs(delta.total_seconds()) \u003c\u003d CONF.service_down_time"},{"line_number":109,"context_line":"\t    art \u003d (svc[\u0027disabled\u0027] and \"down\") or (alive and \"up\") or \"down\"             "},{"line_number":110,"context_line":"            active \u003d \u0027enabled\u0027"},{"line_number":111,"context_line":"            if svc[\u0027disabled\u0027]:"},{"line_number":112,"context_line":"                active \u003d \u0027disabled\u0027"}],"source_content_type":"text/x-python","patch_set":1,"id":"da86d52c_3f6c729c","line":109,"updated":"2015-02-09 22:56:13.000000000","message":"You have extra spaces at the end of this line.","commit_id":"02bdee02bb35bbce97e09e7b35a780ccfa500d05"},{"author":{"_account_id":13900,"name":"Accela Zhao","email":"accelazh@gmail.com","username":"accelazh"},"change_message_id":"8a790de761d224cced98df9f2cafc9327e7d09c7","unresolved":false,"context_lines":[{"line_number":106,"context_line":"        for svc in services:"},{"line_number":107,"context_line":"            delta \u003d now - (svc[\u0027updated_at\u0027] or svc[\u0027created_at\u0027])"},{"line_number":108,"context_line":"            alive \u003d abs(delta.total_seconds()) \u003c\u003d CONF.service_down_time"},{"line_number":109,"context_line":"            art \u003d (svc[\u0027disabled\u0027] and \"down\") or (alive and \"up\") or \"down\""},{"line_number":110,"context_line":"            active \u003d \u0027enabled\u0027"},{"line_number":111,"context_line":"            if svc[\u0027disabled\u0027]:"},{"line_number":112,"context_line":"                active \u003d \u0027disabled\u0027"}],"source_content_type":"text/x-python","patch_set":7,"id":"da86d52c_15a4a049","line":109,"updated":"2015-02-11 14:27:49.000000000","message":"Nice combo expression but you\u0027d better break it up in case it\u0027s too complex.","commit_id":"85be9a3d0ef5bcd0371a9f3f7b9051ee17a22f75"}],"cinder/tests/api/contrib/test_services.py":[{"author":{"_account_id":7198,"name":"Jay Bryant","email":"jungleboyj@electronicjungle.net","username":"jsbryant"},"change_message_id":"1597716b5a93379ba9125c0a6527e247b70f280d","unresolved":false,"context_lines":[{"line_number":61,"context_line":"                       \u0027updated_at\u0027: datetime(2012, 9, 18, 8, 3, 38),"},{"line_number":62,"context_line":"                       \u0027created_at\u0027: datetime(2012, 9, 18, 2, 46, 28),"},{"line_number":63,"context_line":"                       \u0027disabled_reason\u0027: \u0027test4\u0027},"},{"line_number":64,"context_line":"\t\t      {\u0027binary\u0027: \u0027cinder-volume\u0027,"},{"line_number":65,"context_line":"                       \u0027host\u0027: \u0027host2\u0027,"},{"line_number":66,"context_line":"                       \u0027availability_zone\u0027: \u0027cinder\u0027,"},{"line_number":67,"context_line":"                       \u0027id\u0027: 5,"}],"source_content_type":"text/x-python","patch_set":1,"id":"da86d52c_1ff40ece","line":64,"updated":"2015-02-09 22:56:13.000000000","message":"Don\u0027t use Tabs.","commit_id":"02bdee02bb35bbce97e09e7b35a780ccfa500d05"},{"author":{"_account_id":7198,"name":"Jay Bryant","email":"jungleboyj@electronicjungle.net","username":"jsbryant"},"change_message_id":"1597716b5a93379ba9125c0a6527e247b70f280d","unresolved":false,"context_lines":[{"line_number":68,"context_line":"                       \u0027disabled\u0027: True,"},{"line_number":69,"context_line":"                       \u0027updated_at\u0027: datetime(2012, 9, 18, 2, 46,15 ),"},{"line_number":70,"context_line":"                       \u0027created_at\u0027: datetime(2012, 9, 18, 2, 46, 1),"},{"line_number":71,"context_line":"                       \u0027disabled_reason\u0027: \u0027\u0027}, \t"},{"line_number":72,"context_line":"                      ]"},{"line_number":73,"context_line":""},{"line_number":74,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"da86d52c_df127608","line":71,"updated":"2015-02-09 22:56:13.000000000","message":"Extra spaces here.","commit_id":"02bdee02bb35bbce97e09e7b35a780ccfa500d05"},{"author":{"_account_id":7198,"name":"Jay Bryant","email":"jungleboyj@electronicjungle.net","username":"jsbryant"},"change_message_id":"1597716b5a93379ba9125c0a6527e247b70f280d","unresolved":false,"context_lines":[{"line_number":184,"context_line":"                                  \u0027status\u0027: \u0027disabled\u0027, \u0027state\u0027: \u0027down\u0027,"},{"line_number":185,"context_line":"                                  \u0027updated_at\u0027: datetime("},{"line_number":186,"context_line":"                                      2012, 9, 18, 8, 3, 38)},"},{"line_number":187,"context_line":"   \t\t\t\t {\u0027binary\u0027: \u0027cinder-volume\u0027,"},{"line_number":188,"context_line":"                                  \u0027host\u0027: \u0027host2\u0027,"},{"line_number":189,"context_line":"                                  \u0027zone\u0027: \u0027cinder\u0027,"},{"line_number":190,"context_line":"                                  \u0027status\u0027: \u0027disabled\u0027, \u0027state\u0027: \u0027down\u0027,"}],"source_content_type":"text/x-python","patch_set":1,"id":"da86d52c_3f0392b9","line":187,"updated":"2015-02-09 22:56:13.000000000","message":"Tabs","commit_id":"02bdee02bb35bbce97e09e7b35a780ccfa500d05"},{"author":{"_account_id":13900,"name":"Accela Zhao","email":"accelazh@gmail.com","username":"accelazh"},"change_message_id":"8a790de761d224cced98df9f2cafc9327e7d09c7","unresolved":false,"context_lines":[{"line_number":156,"context_line":""},{"line_number":157,"context_line":"        response \u003d {\u0027services\u0027: [{\u0027binary\u0027: \u0027cinder-scheduler\u0027,"},{"line_number":158,"context_line":"                                  \u0027host\u0027: \u0027host1\u0027, \u0027zone\u0027: \u0027cinder\u0027,"},{"line_number":159,"context_line":"                                  \u0027status\u0027: \u0027disabled\u0027, \u0027state\u0027: \u0027down\u0027,"},{"line_number":160,"context_line":"                                  \u0027updated_at\u0027: datetime("},{"line_number":161,"context_line":"                                      2012, 10, 29, 13, 44, 2)},"},{"line_number":162,"context_line":"                                 {\u0027binary\u0027: \u0027cinder-volume\u0027,"}],"source_content_type":"text/x-python","patch_set":7,"id":"da86d52c_3800b900","line":159,"updated":"2015-02-11 14:27:49.000000000","message":"If you set all cases to state \u0027down\u0027. There will be no test cases for the \u0027up\u0027 state.","commit_id":"85be9a3d0ef5bcd0371a9f3f7b9051ee17a22f75"}]}