)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"change_message_id":"5b084ba8a984d995292d4eb9e898535c9f7a06ca","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"936a6f8f_4bc44b74","updated":"2024-11-07 18:26:36.000000000","message":"I think this is a good idea so we can have people like gmann as core on the tempest plugin and help spread the review responsibility around.","commit_id":"e3a19abd2b5a0e3918cba17afdffeab275910c5c"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"631fccb863cba772ff1ff5d778ea6e15a5c5703b","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":1,"id":"49265bda_6565ea47","updated":"2024-11-07 18:50:46.000000000","message":"i think this makes sense to do.\n\ni would consider seeding it with the tempest-core group and\nadd chenker, gmann and perhaps mkopec as direct members.\n\nwhile i have some tempest experience its pretty indirect via the whitebox-tempest-plugin which is why i did not suggest my own name.\n\nadding the tempest-core group is perhaps something that could be done later after talking to the qa team and getting feedback form @chen.ke14@zte.com.cn\nThe intent of adding them would be to have a fallback in the event that there is no active review.\n\nthe watcher-core group could be added for a similar reason but that partly detracts form why this is being suggested in the first place which is why i am not suggesting that.","commit_id":"e3a19abd2b5a0e3918cba17afdffeab275910c5c"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"eb9992be9b0c9d08f1cb9a7db4f04fd44dd4cf25","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":1,"id":"6cbb2be1_79b16510","in_reply_to":"49265bda_6565ea47","updated":"2024-11-07 23:32:24.000000000","message":"yeah adding tempest-core group also good idea. I agree to do that separately after Schenker feedback.","commit_id":"e3a19abd2b5a0e3918cba17afdffeab275910c5c"}],"gerrit/acls/openstack/watcher-tempest.config":[{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"d16c4322ae80d2a61a1e6bdb0fc92ab6123f0359","unresolved":true,"context_lines":[{"line_number":1,"context_line":"[access]"},{"line_number":2,"context_line":"\tinheritFrom \u003d openstack/meta-config"},{"line_number":3,"context_line":""},{"line_number":4,"context_line":"[access \"refs/heads/*\"]"},{"line_number":5,"context_line":"\tabandon \u003d group watcher-tempest-core"}],"source_content_type":"text/x-ini","patch_set":1,"id":"283a6480_07292d61","line":2,"updated":"2024-11-07 18:52:46.000000000","message":"we might want to respine to remove all these tabs howvers\n\nif you respin for the file name mind fixing this too.","commit_id":"e3a19abd2b5a0e3918cba17afdffeab275910c5c"},{"author":{"_account_id":5263,"name":"Jeremy Stanley","display_name":"fungi","email":"fungi@yuggoth.org","username":"fungi","status":"missing, presumed fed"},"change_message_id":"36441b9fcf886ffa031564d8d1e39d18b0a99e22","unresolved":false,"context_lines":[{"line_number":1,"context_line":"[access]"},{"line_number":2,"context_line":"\tinheritFrom \u003d openstack/meta-config"},{"line_number":3,"context_line":""},{"line_number":4,"context_line":"[access \"refs/heads/*\"]"},{"line_number":5,"context_line":"\tabandon \u003d group watcher-tempest-core"}],"source_content_type":"text/x-ini","patch_set":1,"id":"fda07ac0_6a33d481","line":2,"in_reply_to":"283a6480_07292d61","updated":"2024-11-07 18:58:18.000000000","message":"For unfortunate reasons, Gerrit ACL files require hard tab indentation instead of space indentation. I think it has something to do with trying to mimic Git\u0027s config writing behavior, but basically it will end up rewriting to tabs on the backend once this is imported, and then we end up with a formatting mismatch.","commit_id":"e3a19abd2b5a0e3918cba17afdffeab275910c5c"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"3444f70d1568e2ca3d92243d911327dc0c2490be","unresolved":false,"context_lines":[{"line_number":1,"context_line":"[access]"},{"line_number":2,"context_line":"\tinheritFrom \u003d openstack/meta-config"},{"line_number":3,"context_line":""},{"line_number":4,"context_line":"[access \"refs/heads/*\"]"},{"line_number":5,"context_line":"\tabandon \u003d group watcher-tempest-core"}],"source_content_type":"text/x-ini","patch_set":1,"id":"cc5876b7_eac6f9ad","line":2,"in_reply_to":"fda07ac0_6a33d481","updated":"2024-11-07 19:33:00.000000000","message":"oh ok I never noticed that before good to know.","commit_id":"e3a19abd2b5a0e3918cba17afdffeab275910c5c"}],"gerrit/projects.yaml":[{"author":{"_account_id":5263,"name":"Jeremy Stanley","display_name":"fungi","email":"fungi@yuggoth.org","username":"fungi","status":"missing, presumed fed"},"change_message_id":"92aac2c6a4f2ad128c0aa78d085d5f2732c7cadd","unresolved":false,"context_lines":[{"line_number":5920,"context_line":"  description: Watcher Specifications"},{"line_number":5921,"context_line":"- project: openstack/watcher-tempest-plugin"},{"line_number":5922,"context_line":"  description: Tempest plugin for the watcher project."},{"line_number":5923,"context_line":"  acl-config: /home/gerrit2/acls/openstack/watcher-tempest.config"},{"line_number":5924,"context_line":"- project: openstack/whereto"},{"line_number":5925,"context_line":"  description: Test Apache redirect rules."},{"line_number":5926,"context_line":"  acl-config: /home/gerrit2/acls/openstack/oslo-independent.config"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"6bc70eca_1342da64","line":5923,"updated":"2024-11-07 18:32:08.000000000","message":"Note that if you named the file watcher-tempest-plugin then you could just get rid of this acl-config override entry entirely.","commit_id":"e3a19abd2b5a0e3918cba17afdffeab275910c5c"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"d16c4322ae80d2a61a1e6bdb0fc92ab6123f0359","unresolved":false,"context_lines":[{"line_number":5920,"context_line":"  description: Watcher Specifications"},{"line_number":5921,"context_line":"- project: openstack/watcher-tempest-plugin"},{"line_number":5922,"context_line":"  description: Tempest plugin for the watcher project."},{"line_number":5923,"context_line":"  acl-config: /home/gerrit2/acls/openstack/watcher-tempest.config"},{"line_number":5924,"context_line":"- project: openstack/whereto"},{"line_number":5925,"context_line":"  description: Test Apache redirect rules."},{"line_number":5926,"context_line":"  acl-config: /home/gerrit2/acls/openstack/oslo-independent.config"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"f8465853_8eb6ebe4","line":5923,"in_reply_to":"6bc70eca_1342da64","updated":"2024-11-07 18:52:46.000000000","message":"oh ya that would be nice to do but probably not worth a respin on its own","commit_id":"e3a19abd2b5a0e3918cba17afdffeab275910c5c"}]}