)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"85d73aa0f77e29fc494b588b32ae44e6b5964719","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"76cc96a7_0b97d129","updated":"2026-04-24 16:47:41.000000000","message":"I am in favor of this option","commit_id":"ff0ad98b899f96a72412f12b30a73e91625e94b6"},{"author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"change_message_id":"e50a2b20ce249e0da33c73a3ebc2d192dfb56446","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"6534778d_d70626b4","updated":"2026-04-28 14:51:37.000000000","message":"I guess my vote does not matter here so I keep it 0. I think the general direction of adding people to the core team who are SMEs in certain part make sense as I\u0027m also just like that, I\u0027m SME is some parts of nova and I\u0027m not in others. Probably the devil is in the details. \n\nI think this is can be considered a second step after option 1 had some soak time and produced good results.","commit_id":"ff0ad98b899f96a72412f12b30a73e91625e94b6"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"207cfdc6dd072cedbd4345b3fae721740fed41d4","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"289c0c37_a67b1065","updated":"2026-04-24 22:31:42.000000000","message":"Not a nova core, or even a core in any of the other projects you\u0027ve called out in the commit message.. but, playing devil\u0027s advocate here:\n\ntripleo kinda accepted a huge number of people into their core team when they existed, and unsure how well the model worked for them. The bloat is unavoidable... If I were a cinder maintainer, I wouldn\u0027t want to be the single point of failure here, so i\u0027d request adding a few more of my fellow cores. Then it becomes your problem to track my activity on cinder itself - am I continuing to remain an expert?\n\nI wish we had a module/file based \"code owners\" concept like we have in other open source projects so we can grant granular \"reviewer\" rights. BUT, not everything can really be modularized cleanly so it wouldn\u0027t be easy in practice either.  \n\nExpanding the maintainer team a lot may promote a culture where accountability slips, and a few people will be carrying all the weight - or, it becomes a trust issue where maintainers revert more patches because someone disagrees after the fact that a merge occurred. A maintainer strives to know the code base well enough to judge changes. When the team grows large, invariably this base requirement starts becoming less important and imho the project could suffer.","commit_id":"ff0ad98b899f96a72412f12b30a73e91625e94b6"},{"author":{"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},"change_message_id":"3dc65b92ac41eeac8f90f7aa8aa543d76f857af8","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"c9fb20dc_4fb4cce1","updated":"2026-04-27 21:15:40.000000000","message":"This option is too open-ended and unnecessarily complicated, in my opinion.","commit_id":"ff0ad98b899f96a72412f12b30a73e91625e94b6"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"9bf7ae926bbfad155a5d4041c1dd80a411d4b577","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"034f8d25_b6af9ab0","updated":"2026-04-26 17:46:30.000000000","message":"i fell like i should downgrade to -1 on this just becasue wheil im pretty nutasl if we do this as written i.e. addign peopel to nova-core speicifcly that add them to nova-specs and placement and all the other repos.\n\nthat why i originally proposed a new nova-reviews group just for the nova repo\nas i wanted to scope the +2 right to the repo where they would be reviwing.\n\nso im not entirly agaisnt this i just dont see the advantace for SMEs form other services vs option 1","commit_id":"ff0ad98b899f96a72412f12b30a73e91625e94b6"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"c6ea525bbaad7dbabb73052a6aa618fcd0f46bc8","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"000419a6_038fe2a6","updated":"2026-04-26 17:39:49.000000000","message":"im more or less nutal on this.\ni prefer option 1 and perhaps option 3 over this","commit_id":"ff0ad98b899f96a72412f12b30a73e91625e94b6"},{"author":{"_account_id":7166,"name":"Sylvain Bauza","email":"sbauza@redhat.com","username":"sbauza"},"change_message_id":"6770175decdeb10f433f14949c71ac7cdb284b2c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"50f4d9ab_ac0e3883","updated":"2026-04-27 13:47:52.000000000","message":"there are technical boundaries I\u0027d want us to respect as I said in the PTG. Definitely not this option.","commit_id":"ff0ad98b899f96a72412f12b30a73e91625e94b6"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"25156cf001b4cc193381f040af346ca131adc939","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"96e9afb9_75d73faf","in_reply_to":"38555420_47c65cda","updated":"2026-04-28 15:26:32.000000000","message":"\u003e i have no isue with addign new peopel to nova-core but nova-core provide rights to many repos beyond nova such as nova-spec or placement\n\nThat was a mistake, not the intention. The sole focus in all these proposals is on the group of people who can approve code for nova. This proposal specifically proposed adding specific non-nova service cores to that group. I was using `nova-core` to capture that but it is unfortunately overloaded, as you note.\n\nI won\u0027t reword this now as voting has already started, but I\u0027m just letting you know what the actual intention was.","commit_id":"ff0ad98b899f96a72412f12b30a73e91625e94b6"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"629c8ebdf9ac8266ed9c511edef207cb53c8a5e6","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"38555420_47c65cda","in_reply_to":"6534778d_d70626b4","updated":"2026-04-28 15:17:37.000000000","message":"i have no isue with addign new peopel to nova-core but nova-core provide rights to many repos beyond nova such as nova-spec or placement\nso this prospoal is overally broad since it affect repos other then opentack/nova\n\nhence my prefence for 1 since that can be scoped to the nova repo specificly.","commit_id":"ff0ad98b899f96a72412f12b30a73e91625e94b6"}]}
