)]}'
{"doc/source/contributor/process.rst":[{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"b93ba08a81698566323909923952c90f751198bb","unresolved":true,"context_lines":[{"line_number":693,"context_line":"   date in the etherpad. So it needed a central authority to manage the"},{"line_number":694,"context_line":"   process. This did not scale well."},{"line_number":695,"context_line":""},{"line_number":696,"context_line":"2) It was as effective as we, cores, are kept it honest and allocated our"},{"line_number":697,"context_line":"   review time on the patches in the slots. Such commitment is hard to get or"},{"line_number":698,"context_line":"   follow up on without being aggressive."},{"line_number":699,"context_line":""}],"source_content_type":"text/x-rst","patch_set":1,"id":"3dd698a2_bd3c6497","line":696,"range":{"start_line":696,"start_character":26,"end_line":696,"end_character":45},"updated":"2021-06-01 14:23:41.000000000","message":"we where, cores tried to keep ..","commit_id":"b28579a7b0dd744e83d1b01c777eef7654066694"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"b93ba08a81698566323909923952c90f751198bb","unresolved":true,"context_lines":[{"line_number":695,"context_line":""},{"line_number":696,"context_line":"2) It was as effective as we, cores, are kept it honest and allocated our"},{"line_number":697,"context_line":"   review time on the patches in the slots. Such commitment is hard to get or"},{"line_number":698,"context_line":"   follow up on without being aggressive."},{"line_number":699,"context_line":""},{"line_number":700,"context_line":"So the aim of the new review priority process is to be as decentralized amongst"},{"line_number":701,"context_line":"cores as possible. We trust cores that when they mark something as priority"}],"source_content_type":"text/x-rst","patch_set":1,"id":"f213214d_71948fa4","line":698,"range":{"start_line":698,"start_character":30,"end_line":698,"end_character":40},"updated":"2021-06-01 14:23:41.000000000","message":"nit: disciplined","commit_id":"b28579a7b0dd744e83d1b01c777eef7654066694"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"b93ba08a81698566323909923952c90f751198bb","unresolved":true,"context_lines":[{"line_number":704,"context_line":"second reviewer when needed."},{"line_number":705,"context_line":""},{"line_number":706,"context_line":"Note that this process does not want to change how a patch is discovered to be"},{"line_number":707,"context_line":"ready for review. The patch authors free to you any existing forums and ways to"},{"line_number":708,"context_line":"get review attention."},{"line_number":709,"context_line":""},{"line_number":710,"context_line":"Therefore we use the Review-Priority label in Gerrit in the following way:"}],"source_content_type":"text/x-rst","patch_set":1,"id":"a85fab89_269c8222","line":707,"range":{"start_line":707,"start_character":44,"end_line":707,"end_character":47},"updated":"2021-06-01 14:23:41.000000000","message":"nit use","commit_id":"b28579a7b0dd744e83d1b01c777eef7654066694"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"a55bc716b59b230287b2a55688e048c3abdc2497","unresolved":true,"context_lines":[{"line_number":718,"context_line":"* We expect that the cores will limit the number of patches marked with +1"},{"line_number":719,"context_line":"  Review-Priority based on their actual review bandwidth"},{"line_number":720,"context_line":""},{"line_number":721,"context_line":"* We expect that cores will check the list of reviews already having"},{"line_number":722,"context_line":"  Review-Priority +1 set by other cores before they mark a new one as such to"},{"line_number":723,"context_line":"  see where they can help first by being the second core."},{"line_number":724,"context_line":""},{"line_number":725,"context_line":"* There will be a regular agenda point on the weekly meeting where the team"},{"line_number":726,"context_line":"  look at the list of patches with +1 mark to keep an overall view what is"}],"source_content_type":"text/x-rst","patch_set":1,"id":"aaf2fa67_f26e683d","line":723,"range":{"start_line":721,"start_character":0,"end_line":723,"end_character":57},"updated":"2021-05-20 14:57:25.000000000","message":"is there any limit you want to set for number of RP +1 at a time? or we all can just take best decision on these number and try to finish the already RP +1 one ?","commit_id":"b28579a7b0dd744e83d1b01c777eef7654066694"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"9f8c706c3bf9daf1d6cd656b0d1d1c8d4f9fd4ca","unresolved":true,"context_lines":[{"line_number":718,"context_line":"* We expect that the cores will limit the number of patches marked with +1"},{"line_number":719,"context_line":"  Review-Priority based on their actual review bandwidth"},{"line_number":720,"context_line":""},{"line_number":721,"context_line":"* We expect that cores will check the list of reviews already having"},{"line_number":722,"context_line":"  Review-Priority +1 set by other cores before they mark a new one as such to"},{"line_number":723,"context_line":"  see where they can help first by being the second core."},{"line_number":724,"context_line":""},{"line_number":725,"context_line":"* There will be a regular agenda point on the weekly meeting where the team"},{"line_number":726,"context_line":"  look at the list of patches with +1 mark to keep an overall view what is"}],"source_content_type":"text/x-rst","patch_set":1,"id":"9a8dac44_c8ef3422","line":723,"range":{"start_line":721,"start_character":0,"end_line":723,"end_character":57},"in_reply_to":"47ef15dd_3f53937c","updated":"2021-05-21 14:13:05.000000000","message":"+1. ok","commit_id":"b28579a7b0dd744e83d1b01c777eef7654066694"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"b93ba08a81698566323909923952c90f751198bb","unresolved":true,"context_lines":[{"line_number":718,"context_line":"* We expect that the cores will limit the number of patches marked with +1"},{"line_number":719,"context_line":"  Review-Priority based on their actual review bandwidth"},{"line_number":720,"context_line":""},{"line_number":721,"context_line":"* We expect that cores will check the list of reviews already having"},{"line_number":722,"context_line":"  Review-Priority +1 set by other cores before they mark a new one as such to"},{"line_number":723,"context_line":"  see where they can help first by being the second core."},{"line_number":724,"context_line":""},{"line_number":725,"context_line":"* There will be a regular agenda point on the weekly meeting where the team"},{"line_number":726,"context_line":"  look at the list of patches with +1 mark to keep an overall view what is"}],"source_content_type":"text/x-rst","patch_set":1,"id":"41dce2b6_751903a5","line":723,"range":{"start_line":721,"start_character":0,"end_line":723,"end_character":57},"in_reply_to":"9a8dac44_c8ef3422","updated":"2021-06-01 14:23:41.000000000","message":"Ack","commit_id":"b28579a7b0dd744e83d1b01c777eef7654066694"},{"author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"change_message_id":"755dbc2745d45aaea031a566a51a8b03359ee8cd","unresolved":true,"context_lines":[{"line_number":718,"context_line":"* We expect that the cores will limit the number of patches marked with +1"},{"line_number":719,"context_line":"  Review-Priority based on their actual review bandwidth"},{"line_number":720,"context_line":""},{"line_number":721,"context_line":"* We expect that cores will check the list of reviews already having"},{"line_number":722,"context_line":"  Review-Priority +1 set by other cores before they mark a new one as such to"},{"line_number":723,"context_line":"  see where they can help first by being the second core."},{"line_number":724,"context_line":""},{"line_number":725,"context_line":"* There will be a regular agenda point on the weekly meeting where the team"},{"line_number":726,"context_line":"  look at the list of patches with +1 mark to keep an overall view what is"}],"source_content_type":"text/x-rst","patch_set":1,"id":"47ef15dd_3f53937c","line":723,"range":{"start_line":721,"start_character":0,"end_line":723,"end_character":57},"in_reply_to":"aaf2fa67_f26e683d","updated":"2021-05-21 06:49:35.000000000","message":"I trust the cores to judge their own review bandwidth. If everybody only set +1 to those they commit to review in the foreseeable future then the number of +1 will be limited anyhow.","commit_id":"b28579a7b0dd744e83d1b01c777eef7654066694"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"b93ba08a81698566323909923952c90f751198bb","unresolved":true,"context_lines":[{"line_number":736,"context_line":""},{"line_number":737,"context_line":"Cons:"},{"line_number":738,"context_line":""},{"line_number":739,"context_line":"* No externally enforced time limit on patches sitting idle with +1"},{"line_number":740,"context_line":"  Review-Priority"},{"line_number":741,"context_line":""},{"line_number":742,"context_line":"* No externally enforced limit on how many things can be a priority at any"},{"line_number":743,"context_line":"  given time."}],"source_content_type":"text/x-rst","patch_set":1,"id":"225cec44_cc4132af","line":740,"range":{"start_line":739,"start_character":2,"end_line":740,"end_character":17},"updated":"2021-06-01 14:23:41.000000000","message":"cores can always remove the flag too possible after discussing in the meeting so i think that will be self regulating.","commit_id":"b28579a7b0dd744e83d1b01c777eef7654066694"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"b93ba08a81698566323909923952c90f751198bb","unresolved":true,"context_lines":[{"line_number":743,"context_line":"  given time."},{"line_number":744,"context_line":""},{"line_number":745,"context_line":"* Does not (want to) solve the problem of discovering reviews that are ready to"},{"line_number":746,"context_line":"  core review"},{"line_number":747,"context_line":""},{"line_number":748,"context_line":"Process Evolution Ideas"},{"line_number":749,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-rst","patch_set":1,"id":"3f38f585_a5e1f13f","line":746,"updated":"2021-06-01 14:23:41.000000000","message":"to adress this we can encurage peopel to use workflow -1 or add [WIP] to the commit message and assume the rest should be in a good state.\nat least for now and we can codify that later in the docs as a sperate patch if we want too.\nthat is covered in the general contibutors guide already\nhttps://docs.openstack.org/project-team-guide/review-the-openstack-way.html#workflow-1","commit_id":"b28579a7b0dd744e83d1b01c777eef7654066694"},{"author":{"_account_id":7166,"name":"Sylvain Bauza","email":"sbauza@redhat.com","username":"sbauza"},"change_message_id":"a94c3a0079e9acbe11c1546668426595dbd13929","unresolved":true,"context_lines":[{"line_number":744,"context_line":""},{"line_number":745,"context_line":"* Does not (want to) solve the problem of discovering reviews that are ready to"},{"line_number":746,"context_line":"  core review"},{"line_number":747,"context_line":""},{"line_number":748,"context_line":"Process Evolution Ideas"},{"line_number":749,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":750,"context_line":""}],"source_content_type":"text/x-rst","patch_set":1,"id":"9421efbc_af2ead3e","line":747,"updated":"2021-06-08 13:22:45.000000000","message":"I honestly don\u0027t really see how this will fix the problem that contributors would want to have reviews.\nIf it\u0027s for knowing whether some core wants to review some change but didn\u0027t had time yet to do this, I understand it but again, I don\u0027t really see the difference with adding for example my name to the Gerrit reviewers.","commit_id":"b28579a7b0dd744e83d1b01c777eef7654066694"},{"author":{"_account_id":7166,"name":"Sylvain Bauza","email":"sbauza@redhat.com","username":"sbauza"},"change_message_id":"164c49f7e9eff6983b899392278a5430e0c53718","unresolved":true,"context_lines":[{"line_number":744,"context_line":""},{"line_number":745,"context_line":"* Does not (want to) solve the problem of discovering reviews that are ready to"},{"line_number":746,"context_line":"  core review"},{"line_number":747,"context_line":""},{"line_number":748,"context_line":"Process Evolution Ideas"},{"line_number":749,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":750,"context_line":""}],"source_content_type":"text/x-rst","patch_set":1,"id":"66ad118d_c8c9d513","line":747,"in_reply_to":"9421efbc_af2ead3e","updated":"2021-09-10 09:34:15.000000000","message":"OK, given we now have the label in Gerrit and some core folks being to use it, I\u0027ll accept this change and merge it directly, but then I\u0027ll try to provide another change for changing what I\u0027d like to have for this label (about my concerns)","commit_id":"b28579a7b0dd744e83d1b01c777eef7654066694"}]}