)]}'
{"doc/source/process.rst":[{"author":{"_account_id":10670,"name":"Michael McCune","email":"elmiko@redhat.com","username":"mimccune"},"change_message_id":"c6f70ed41e40f278e569b931b0f12fe06a75aa27","unresolved":false,"context_lines":[{"line_number":64,"context_line":"   responsible for freezing the guideline should +2 it. All other core"},{"line_number":65,"context_line":"   reviewers should vote with at most a +1 when reviewing."},{"line_number":66,"context_line":""},{"line_number":67,"context_line":"5. Once a changeset meets the requirements of #4, the CPLs have 1 week to "},{"line_number":68,"context_line":"   review it. If there is no review by a CPL, lazy consensus is assumed."},{"line_number":69,"context_line":"   If there is a -1 review by a CPL that requires an update to the changeset,"},{"line_number":70,"context_line":"   it does not reset the 1 week the CPLs have to review it."}],"source_content_type":"text/x-rst","patch_set":1,"id":"fa1b9901_14ed338b","line":67,"updated":"2015-08-17 22:06:12.000000000","message":"whitespace","commit_id":"a9db995bc666da1f440027f6255b00eed7990017"},{"author":{"_account_id":13997,"name":"SHIGEMATSU Mitsuhiro","email":"shigematsu.mitsuhiro@lab.ntt.co.jp","username":"pshige"},"change_message_id":"cdc1d5edd10f81992ec153979e942932f11b62ec","unresolved":false,"context_lines":[{"line_number":72,"context_line":"6. Once a changeset meets the requirements of #5, it can be merged."},{"line_number":73,"context_line":""},{"line_number":74,"context_line":"   An email should be sent to the openstack-dev mailing list containing the"},{"line_number":75,"context_line":"   links to all of the guidelines that have have been merged. This could"},{"line_number":76,"context_line":"   simply be a reply to the original email in #3.2. The finalized"},{"line_number":77,"context_line":"   guidelines should be buffered such that a maximum of one announcement"},{"line_number":78,"context_line":"   email is sent per week."}],"source_content_type":"text/x-rst","patch_set":2,"id":"fa1b9901_2aa2e4fc","line":75,"updated":"2015-08-25 16:34:09.000000000","message":"s/have have/have/","commit_id":"69becf1fbb4432cec3be29afee0d9a6ef5b84b3c"},{"author":{"_account_id":1112,"name":"Everett Toews","email":"everett.toews@rackspace.com","username":"everett-toews"},"change_message_id":"3dee73752efbd00c55b2e8ce4d86fdcab8d38000","unresolved":false,"context_lines":[{"line_number":72,"context_line":"6. Once a changeset meets the requirements of #5, it can be merged."},{"line_number":73,"context_line":""},{"line_number":74,"context_line":"   An email should be sent to the openstack-dev mailing list containing the"},{"line_number":75,"context_line":"   links to all of the guidelines that have have been merged. This could"},{"line_number":76,"context_line":"   simply be a reply to the original email in #3.2. The finalized"},{"line_number":77,"context_line":"   guidelines should be buffered such that a maximum of one announcement"},{"line_number":78,"context_line":"   email is sent per week."}],"source_content_type":"text/x-rst","patch_set":2,"id":"fa1b9901_87a2feac","line":75,"in_reply_to":"fa1b9901_2aa2e4fc","updated":"2015-08-25 20:24:01.000000000","message":"Done","commit_id":"69becf1fbb4432cec3be29afee0d9a6ef5b84b3c"},{"author":{"_account_id":261,"name":"Salvatore Orlando","email":"salv.orlando@gmail.com","username":"salvatore-orlando"},"change_message_id":"a13afde462300fc5cb7a858a6f64b063ebeafaac","unresolved":false,"context_lines":[{"line_number":62,"context_line":"4. Once a changeset meets the requirements of #3, it should be frozen by"},{"line_number":63,"context_line":"   exactly one core reviewer by setting Code-Review +2. Only the core reviewer"},{"line_number":64,"context_line":"   responsible for freezing the guideline should +2 it. All other core"},{"line_number":65,"context_line":"   reviewers should vote with at most a +1 when reviewing."},{"line_number":66,"context_line":""},{"line_number":67,"context_line":"5. Once a changeset meets the requirements of #4, the CPLs have 1 week to"},{"line_number":68,"context_line":"   review it. If there is no review by a CPL, lazy consensus is assumed."}],"source_content_type":"text/x-rst","patch_set":3,"id":"fa1b9901_1b5f0ae6","line":65,"updated":"2015-08-26 12:36:46.000000000","message":"What if another core reviewer disagrees with the frozen status? This is a very rare circumstance, because I think a small core team like the api-wg one should be able to coordinate easily.\n\nNevertheless, is it fair to say that a review is frozen if there is a +2 from a core review _and_ neither a -1 or a -2 from another core reviewer (perhaps the -2 will be better as liasions and other reviewers might not know who are the core team members)","commit_id":"b77c68c6c4972d7ad0e6951671df3ab6216c9fa9"},{"author":{"_account_id":10670,"name":"Michael McCune","email":"elmiko@redhat.com","username":"mimccune"},"change_message_id":"ec86cb3a97f79370a992bb22fae1088c27ed4bce","unresolved":false,"context_lines":[{"line_number":62,"context_line":"4. Once a changeset meets the requirements of #3, it should be frozen by"},{"line_number":63,"context_line":"   exactly one core reviewer by setting Code-Review +2. Only the core reviewer"},{"line_number":64,"context_line":"   responsible for freezing the guideline should +2 it. All other core"},{"line_number":65,"context_line":"   reviewers should vote with at most a +1 when reviewing."},{"line_number":66,"context_line":""},{"line_number":67,"context_line":"5. Once a changeset meets the requirements of #4, the CPLs have 1 week to"},{"line_number":68,"context_line":"   review it. If there is no review by a CPL, lazy consensus is assumed."}],"source_content_type":"text/x-rst","patch_set":3,"id":"fa1b9901_f4b583ec","line":65,"in_reply_to":"fa1b9901_1b5f0ae6","updated":"2015-08-26 13:28:13.000000000","message":"a fair point, as for the core reviewer -1, i think core reviewers should be treated like any other reviewer at that point. so if there is a -1 from a core it shouldn\u0027t count more than anyone else\u0027s -1.\n\nas for the -2 from a core, i would think that is self explanatory. the guideline wouldn\u0027t be mergeable until the -2 is removed.","commit_id":"b77c68c6c4972d7ad0e6951671df3ab6216c9fa9"},{"author":{"_account_id":7,"name":"Jay Pipes","email":"jaypipes@gmail.com","username":"jaypipes"},"change_message_id":"6c5aa3b80db895aee2636d2396d962a7661535d0","unresolved":false,"context_lines":[{"line_number":62,"context_line":"4. Once a changeset meets the requirements of #3, it should be frozen by"},{"line_number":63,"context_line":"   exactly one core reviewer by setting Code-Review +2. Only the core reviewer"},{"line_number":64,"context_line":"   responsible for freezing the guideline should +2 it. All other core"},{"line_number":65,"context_line":"   reviewers should vote with at most a +1 when reviewing."},{"line_number":66,"context_line":""},{"line_number":67,"context_line":"5. Once a changeset meets the requirements of #4, the CPLs have 1 week to"},{"line_number":68,"context_line":"   review it. If there is no review by a CPL, lazy consensus is assumed."}],"source_content_type":"text/x-rst","patch_set":3,"id":"fa1b9901_7ba07b42","line":65,"in_reply_to":"fa1b9901_550a267a","updated":"2015-08-26 15:36:31.000000000","message":"Yeah, good points, but I agree with Mike and Everett on this.","commit_id":"b77c68c6c4972d7ad0e6951671df3ab6216c9fa9"},{"author":{"_account_id":1112,"name":"Everett Toews","email":"everett.toews@rackspace.com","username":"everett-toews"},"change_message_id":"8ec464cec89659b2a03d98f3d27d7861f4970c34","unresolved":false,"context_lines":[{"line_number":62,"context_line":"4. Once a changeset meets the requirements of #3, it should be frozen by"},{"line_number":63,"context_line":"   exactly one core reviewer by setting Code-Review +2. Only the core reviewer"},{"line_number":64,"context_line":"   responsible for freezing the guideline should +2 it. All other core"},{"line_number":65,"context_line":"   reviewers should vote with at most a +1 when reviewing."},{"line_number":66,"context_line":""},{"line_number":67,"context_line":"5. Once a changeset meets the requirements of #4, the CPLs have 1 week to"},{"line_number":68,"context_line":"   review it. If there is no review by a CPL, lazy consensus is assumed."}],"source_content_type":"text/x-rst","patch_set":3,"id":"fa1b9901_550a267a","line":65,"in_reply_to":"fa1b9901_f4b583ec","updated":"2015-08-26 15:18:08.000000000","message":"Agreed with Michael here. I don\u0027t think we need to be explicit about it because it\u0027s just the normal way of reviewing things.","commit_id":"b77c68c6c4972d7ad0e6951671df3ab6216c9fa9"},{"author":{"_account_id":261,"name":"Salvatore Orlando","email":"salv.orlando@gmail.com","username":"salvatore-orlando"},"change_message_id":"a13afde462300fc5cb7a858a6f64b063ebeafaac","unresolved":false,"context_lines":[{"line_number":79,"context_line":""},{"line_number":80,"context_line":"7. Anyone who is listed on the `API Working Group Wiki"},{"line_number":81,"context_line":"   \u003chttps://wiki.openstack.org/wiki/API_Working_Group\u003e`_ by"},{"line_number":82,"context_line":"   25/5/2015 (L1) can vote during the Liberty development cycle."},{"line_number":83,"context_line":""},{"line_number":84,"context_line":"8. Before an official version of the guidelines can be released the"},{"line_number":85,"context_line":"   following has to occur:"}],"source_content_type":"text/x-rst","patch_set":3,"id":"fa1b9901_fbac96a0","line":82,"updated":"2015-08-26 12:36:46.000000000","message":"Unrelated: I always wondered if we\u0027re not breaking any rule by restricting voting rights also to a few people who manifested interested. Also, since anyone can add himself/herself to that list, I honestly feel this rule is a bit pointless now.","commit_id":"b77c68c6c4972d7ad0e6951671df3ab6216c9fa9"},{"author":{"_account_id":1112,"name":"Everett Toews","email":"everett.toews@rackspace.com","username":"everett-toews"},"change_message_id":"8ec464cec89659b2a03d98f3d27d7861f4970c34","unresolved":false,"context_lines":[{"line_number":79,"context_line":""},{"line_number":80,"context_line":"7. Anyone who is listed on the `API Working Group Wiki"},{"line_number":81,"context_line":"   \u003chttps://wiki.openstack.org/wiki/API_Working_Group\u003e`_ by"},{"line_number":82,"context_line":"   25/5/2015 (L1) can vote during the Liberty development cycle."},{"line_number":83,"context_line":""},{"line_number":84,"context_line":"8. Before an official version of the guidelines can be released the"},{"line_number":85,"context_line":"   following has to occur:"}],"source_content_type":"text/x-rst","patch_set":3,"id":"fa1b9901_95b8bec9","line":82,"in_reply_to":"fa1b9901_549c6f69","updated":"2015-08-26 15:18:08.000000000","message":"Thanks for calling this out Salvatore. I\u0027m in full agreement. It\u0027s a hold over from a Summit discussion when we didn\u0027t really know how everything would work out. I don\u0027t think it\u0027s necessary at this point.\n\nBest to leave it for a follow up patch though.","commit_id":"b77c68c6c4972d7ad0e6951671df3ab6216c9fa9"},{"author":{"_account_id":7,"name":"Jay Pipes","email":"jaypipes@gmail.com","username":"jaypipes"},"change_message_id":"6c5aa3b80db895aee2636d2396d962a7661535d0","unresolved":false,"context_lines":[{"line_number":79,"context_line":""},{"line_number":80,"context_line":"7. Anyone who is listed on the `API Working Group Wiki"},{"line_number":81,"context_line":"   \u003chttps://wiki.openstack.org/wiki/API_Working_Group\u003e`_ by"},{"line_number":82,"context_line":"   25/5/2015 (L1) can vote during the Liberty development cycle."},{"line_number":83,"context_line":""},{"line_number":84,"context_line":"8. Before an official version of the guidelines can be released the"},{"line_number":85,"context_line":"   following has to occur:"}],"source_content_type":"text/x-rst","patch_set":3,"id":"fa1b9901_9bc9cf28","line":82,"in_reply_to":"fa1b9901_95b8bec9","updated":"2015-08-26 15:36:31.000000000","message":"Agreed. Remove it in a followup patch, IMHO.","commit_id":"b77c68c6c4972d7ad0e6951671df3ab6216c9fa9"},{"author":{"_account_id":10670,"name":"Michael McCune","email":"elmiko@redhat.com","username":"mimccune"},"change_message_id":"ec86cb3a97f79370a992bb22fae1088c27ed4bce","unresolved":false,"context_lines":[{"line_number":79,"context_line":""},{"line_number":80,"context_line":"7. Anyone who is listed on the `API Working Group Wiki"},{"line_number":81,"context_line":"   \u003chttps://wiki.openstack.org/wiki/API_Working_Group\u003e`_ by"},{"line_number":82,"context_line":"   25/5/2015 (L1) can vote during the Liberty development cycle."},{"line_number":83,"context_line":""},{"line_number":84,"context_line":"8. Before an official version of the guidelines can be released the"},{"line_number":85,"context_line":"   following has to occur:"}],"source_content_type":"text/x-rst","patch_set":3,"id":"fa1b9901_549c6f69","line":82,"in_reply_to":"fa1b9901_fbac96a0","updated":"2015-08-26 13:28:13.000000000","message":"yea, might be time to open that door a little wider.","commit_id":"b77c68c6c4972d7ad0e6951671df3ab6216c9fa9"}]}