)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":38261,"name":"Jeongwon Been","display_name":"jbeen","email":"jeongwon.been@gmail.com","username":"jbeen"},"change_message_id":"f8446c01bbf5e819819ed8c8ec78d38bd444a950","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"f473e909_c6d1ee20","updated":"2025-09-06 12:39:09.000000000","message":"I’m aware that the project quota list API exists, but it is not included here since it will be implemented by @platanus.kr@gmail.com","commit_id":"970b64ec9e8bdd127526513267cd61a3bc54d63d"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"4e7b98cdcefb71e93e3e4767baff2db53e4cf5d2","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"5c72b3a8_5e65eaed","updated":"2025-09-09 21:14:35.000000000","message":"There\u0027s a small issue with the docs that is causing the failure of that job (the failure in the ironic job is unrelated and won\u0027t happen anymore).\n\nCould you fix that? Could you also add functional test to prove that this works? You will need to create a new project as part of `setUp`. You can see an example of us doing this in another test such as `openstack/tests/functional/identity/v3/test_limit.py`.","commit_id":"970b64ec9e8bdd127526513267cd61a3bc54d63d"},{"author":{"_account_id":38261,"name":"Jeongwon Been","display_name":"jbeen","email":"jeongwon.been@gmail.com","username":"jbeen"},"change_message_id":"f197acb6b67feefa8e63d469386cfd7b3bd87b35","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"72c7efcf_b7834e7d","in_reply_to":"5c72b3a8_5e65eaed","updated":"2025-09-15 11:15:13.000000000","message":"Thanks for the detailed and thoughtful review!  \nI have updated the docs, added the requested functional test (creating a new project in setUp as suggested), and also renamed the parameter from `project_quota` to `project_id`.  \nReally appreciate your careful feedback.","commit_id":"970b64ec9e8bdd127526513267cd61a3bc54d63d"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"5498b2b25f6aae3ae3b82df3c292e70a34826138","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"eda5feee_d0f89211","updated":"2025-09-25 10:09:34.000000000","message":"I have one remaining question inline. Also, I forgot to ask for a release note. Could you please add one (hint: you can use the `reno` tool to generate the initial note file and find examples of release notes by doing e.g. `git log -- releasenotes` in the repo). The -1 is for the release note","commit_id":"3f13da0b9c3724090595cb508d6f7eda223176d5"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"81822f0f1c781eabc9349a7ccbb2441d5ae1c4ec","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"ed93520c_cd538532","updated":"2025-11-13 10:31:23.000000000","message":"recheck","commit_id":"3ba50349249ad4466f48fb11f9ed3afaf312cfd2"}],"doc/source/user/resources/key_manager/index.rst":[{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"4e7b98cdcefb71e93e3e4767baff2db53e4cf5d2","unresolved":true,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"   v1/container"},{"line_number":8,"context_line":"   v1/order"},{"line_number":9,"context_line":"   v1/project-quota"},{"line_number":10,"context_line":"   v1/secret"}],"source_content_type":"text/x-rst","patch_set":1,"id":"58a06b81_1154dfbb","line":9,"updated":"2025-09-09 21:14:35.000000000","message":"```suggestion\n   v1/project_quota\n```","commit_id":"970b64ec9e8bdd127526513267cd61a3bc54d63d"},{"author":{"_account_id":38261,"name":"Jeongwon Been","display_name":"jbeen","email":"jeongwon.been@gmail.com","username":"jbeen"},"change_message_id":"f197acb6b67feefa8e63d469386cfd7b3bd87b35","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"   v1/container"},{"line_number":8,"context_line":"   v1/order"},{"line_number":9,"context_line":"   v1/project-quota"},{"line_number":10,"context_line":"   v1/secret"}],"source_content_type":"text/x-rst","patch_set":1,"id":"ac251736_84a25c08","line":9,"in_reply_to":"58a06b81_1154dfbb","updated":"2025-09-15 11:15:13.000000000","message":"Done","commit_id":"970b64ec9e8bdd127526513267cd61a3bc54d63d"}],"openstack/tests/functional/key_manager/v1/test_project_quota.py":[{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"5498b2b25f6aae3ae3b82df3c292e70a34826138","unresolved":true,"context_lines":[{"line_number":30,"context_line":"            self.system_admin_cloud.identity.delete_project, self.project"},{"line_number":31,"context_line":"        )"},{"line_number":32,"context_line":""},{"line_number":33,"context_line":"        self.role \u003d self.system_admin_cloud.identity.create_role("},{"line_number":34,"context_line":"            name\u003dADMIN_ROLE_NAME"},{"line_number":35,"context_line":"        )"},{"line_number":36,"context_line":"        self.addCleanup("}],"source_content_type":"text/x-python","patch_set":5,"id":"dad8c630_6addc032","line":33,"updated":"2025-09-25 10:09:34.000000000","message":"I\u0027m not sure why we need to create a role. Could you add a comment explaining why?","commit_id":"3f13da0b9c3724090595cb508d6f7eda223176d5"},{"author":{"_account_id":38261,"name":"Jeongwon Been","display_name":"jbeen","email":"jeongwon.been@gmail.com","username":"jbeen"},"change_message_id":"7709d38dfaff71e4b1b1f0e6f34fe4eede321173","unresolved":false,"context_lines":[{"line_number":30,"context_line":"            self.system_admin_cloud.identity.delete_project, self.project"},{"line_number":31,"context_line":"        )"},{"line_number":32,"context_line":""},{"line_number":33,"context_line":"        self.role \u003d self.system_admin_cloud.identity.create_role("},{"line_number":34,"context_line":"            name\u003dADMIN_ROLE_NAME"},{"line_number":35,"context_line":"        )"},{"line_number":36,"context_line":"        self.addCleanup("}],"source_content_type":"text/x-python","patch_set":5,"id":"b66adcb8_f1dc34d8","line":33,"in_reply_to":"dad8c630_6addc032","updated":"2025-09-28 15:50:36.000000000","message":"Added a release note and a comment explaining why we create the role. Thanks for the helpful guidance.","commit_id":"3f13da0b9c3724090595cb508d6f7eda223176d5"}]}