)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":6282,"name":"Wu Wenxiang","email":"wu.wenxiang@algoblu.com","username":"wu-wenxiang"},"change_message_id":"bfc4fc9b355086763f1c1499138a861730525b37","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"d452b53a_f54a49ec","updated":"2026-09-23 01:52:28.000000000","message":"Regenerate Zaqar policies from upstream instead of hand-writing them.\n\nSkyline policies must come from the service’s `oslo.policy` rules via `skyline-policy-manager generate-rule \u003cservice\u003e` (same as zun/masakari). The proposed `zaqar.py` invents its own rules (`messaging:queues:get`, member/reader/system-scope checks, partial coverage, wrong paths) and does not match Zaqar’s real policies (https://opendev.org/openstack/zaqar/src/branch/master/zaqar/common/policies). That desyncs UI RBAC from Zaqar and breaks policy generate/compare.\n\nSteps：\n\n1. Install zaqar so entry point `oslo.policy.policies:zaqar` is available.  \n2. Replace hand-written `zaqar.py` with:\n\n```bash\nskyline-policy-manager generate-rule zaqar \u003e skyline_apiserver/policy/manager/zaqar.py\n```\n\n3. Drop (or greatly simplify) the custom unit tests that only assert the invented rule names.\n\nTo skyine-console，use upstream rule names and prefixes, e.g.:\n\n- `policyMap`: `zaqar: [\u0027queues:\u0027, \u0027messages:\u0027, \u0027claims:\u0027, \u0027subscription:\u0027, ...]` (not `messaging:`)\n- actions: `queues:get_all`, `queues:create`, `queues:purge`, `messages:create`, `messages:delete`, `claims:create`, `claims:delete`, `subscription:*`, etc. — not `messaging:queues:update` for everything","commit_id":"b6e76f94a0a0b773e0afc371d393df124dc24782"},{"author":{"_account_id":37598,"name":"Ivan Anfimov","display_name":"Ivan Anfimov","email":"lazekteam@gmail.com","username":"anfimovir"},"change_message_id":"df16b009009165da73ed1fb531e8e783d6d0b2e4","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"3233e43b_3828f841","updated":"2026-09-22 16:06:54.000000000","message":"lgtm, thanks you.","commit_id":"b6e76f94a0a0b773e0afc371d393df124dc24782"},{"author":{"_account_id":39376,"name":"Jitendra Mahari","display_name":"jitendra","email":"jitendra.mahari@rackspace.com","username":"jitenubuntu1988"},"change_message_id":"87af750198a5a6e5eceaf276f3cfca7d614c4765","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"b0e10bf4_9fad9ab3","in_reply_to":"d452b53a_f54a49ec","updated":"2026-09-23 14:21:14.000000000","message":"Thanks for the detailed review. I\u0027ve regenerated zaqar.py from Zaqar\u0027s oslo.policy rules using skyline-policy-manager generate-rule zaqar, consistent with the upstream pattern.\n\nI also updated the unit tests to validate Zaqar registration, service mapping, rule discovery, and upstream prefixes.\n\nPatchset 2 has been pushed.","commit_id":"b6e76f94a0a0b773e0afc371d393df124dc24782"},{"author":{"_account_id":6282,"name":"Wu Wenxiang","email":"wu.wenxiang@algoblu.com","username":"wu-wenxiang"},"change_message_id":"fa52504abc3ed4bdab87f72296510fa554f1a299","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"7c903f5b_0afd0b43","updated":"2026-09-25 02:25:20.000000000","message":"Thanks for your patch!","commit_id":"11462a7d6539564eec206d264cc676f2d086cf64"}]}