)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":29244,"name":"Gregory Thiemonge","email":"gthiemon@redhat.com","username":"gthiemonge"},"change_message_id":"5dbbb13c48b41238d0dee79efc0d1baa471e94db","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"0bf968cc_e5c6ac4f","updated":"2025-04-11 10:03:16.000000000","message":"-1 to raise awareness of my question","commit_id":"6693956d27a8f48b766873bfcb87628667fa1264"},{"author":{"_account_id":11628,"name":"Michael Johnson","email":"johnsomor@gmail.com","username":"johnsom"},"change_message_id":"519db9a0efb7544331a0224358f5773d2a7bb7f4","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"287b2555_f800d263","updated":"2025-04-10 17:15:19.000000000","message":"LGTM","commit_id":"6693956d27a8f48b766873bfcb87628667fa1264"},{"author":{"_account_id":11628,"name":"Michael Johnson","email":"johnsomor@gmail.com","username":"johnsom"},"change_message_id":"1ac4a23a89321c2d2eee456dce72ea55f2ce8a22","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"3a80013e_89df334a","updated":"2025-04-15 15:48:00.000000000","message":"LGTM","commit_id":"32eca8ae06436003e59ded82f1cf5f99d62cae8f"},{"author":{"_account_id":29244,"name":"Gregory Thiemonge","email":"gthiemon@redhat.com","username":"gthiemonge"},"change_message_id":"6de4d0172b560e4e916002dfaf4ef19ca0951f5c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"b1b97fe0_8dfaa140","updated":"2025-04-14 09:20:45.000000000","message":"LGTM","commit_id":"32eca8ae06436003e59ded82f1cf5f99d62cae8f"}],"octavia/policies/member.py":[{"author":{"_account_id":29244,"name":"Gregory Thiemonge","email":"gthiemon@redhat.com","username":"gthiemonge"},"change_message_id":"5dbbb13c48b41238d0dee79efc0d1baa471e94db","unresolved":true,"context_lines":[{"line_number":30,"context_line":"    ),"},{"line_number":31,"context_line":"    policy.DocumentedRuleDefault("},{"line_number":32,"context_line":"        f\u0027{constants.RBAC_MEMBER}{constants.RBAC_GET_ONE}\u0027,"},{"line_number":33,"context_line":"        constants.RULE_MEMBER_API_READ,"},{"line_number":34,"context_line":"        \"Show Member details\","},{"line_number":35,"context_line":"        [{\u0027method\u0027: \u0027GET\u0027,"},{"line_number":36,"context_line":"          \u0027path\u0027: \u0027/v2/lbaas/pools/{pool_id}/members/{member_id}\u0027}]"}],"source_content_type":"text/x-python","patch_set":3,"id":"df9104e2_c0849c2e","line":33,"range":{"start_line":33,"start_character":8,"end_line":33,"end_character":38},"updated":"2025-04-11 10:03:16.000000000","message":"I\u0027m wondering if it\u0027s really what we want to do with the \"get one\" call, shouldn\u0027t this rule only be used on line 21 (get all the members of a pool)?","commit_id":"6693956d27a8f48b766873bfcb87628667fa1264"},{"author":{"_account_id":16137,"name":"Tobias Urdin","email":"tobias.urdin@binero.com","username":"tobasco"},"change_message_id":"1f2432d58753ae8ea935c0d45b139842c7cda744","unresolved":true,"context_lines":[{"line_number":30,"context_line":"    ),"},{"line_number":31,"context_line":"    policy.DocumentedRuleDefault("},{"line_number":32,"context_line":"        f\u0027{constants.RBAC_MEMBER}{constants.RBAC_GET_ONE}\u0027,"},{"line_number":33,"context_line":"        constants.RULE_MEMBER_API_READ,"},{"line_number":34,"context_line":"        \"Show Member details\","},{"line_number":35,"context_line":"        [{\u0027method\u0027: \u0027GET\u0027,"},{"line_number":36,"context_line":"          \u0027path\u0027: \u0027/v2/lbaas/pools/{pool_id}/members/{member_id}\u0027}]"}],"source_content_type":"text/x-python","patch_set":3,"id":"e2595f0e_77069b99","line":33,"range":{"start_line":33,"start_character":8,"end_line":33,"end_character":38},"in_reply_to":"df9104e2_c0849c2e","updated":"2025-04-11 14:19:00.000000000","message":"Thanks Gregory, this is really a miss on my part, this is where the API call is made in Aodh [1] using the client from python-octaviaclient here [2] and it calls member_list() which indeed is the ``/pools/\u003cpool_id\u003e/members`` URL path and not a single member.\n\nLet me fix that up asap, thanks!\n\n[1] https://github.com/openstack/aodh/blob/master/aodh/evaluator/loadbalancer.py#L72\n[2] https://github.com/openstack/python-octaviaclient/blob/master/octaviaclient/api/v2/octavia.py#L386","commit_id":"6693956d27a8f48b766873bfcb87628667fa1264"},{"author":{"_account_id":16137,"name":"Tobias Urdin","email":"tobias.urdin@binero.com","username":"tobasco"},"change_message_id":"e407127e2c196a7b5596886a2fb646140a2ca3f4","unresolved":false,"context_lines":[{"line_number":30,"context_line":"    ),"},{"line_number":31,"context_line":"    policy.DocumentedRuleDefault("},{"line_number":32,"context_line":"        f\u0027{constants.RBAC_MEMBER}{constants.RBAC_GET_ONE}\u0027,"},{"line_number":33,"context_line":"        constants.RULE_MEMBER_API_READ,"},{"line_number":34,"context_line":"        \"Show Member details\","},{"line_number":35,"context_line":"        [{\u0027method\u0027: \u0027GET\u0027,"},{"line_number":36,"context_line":"          \u0027path\u0027: \u0027/v2/lbaas/pools/{pool_id}/members/{member_id}\u0027}]"}],"source_content_type":"text/x-python","patch_set":3,"id":"9929b157_3f40b4d7","line":33,"range":{"start_line":33,"start_character":8,"end_line":33,"end_character":38},"in_reply_to":"e2595f0e_77069b99","updated":"2025-04-14 06:25:58.000000000","message":"Done","commit_id":"6693956d27a8f48b766873bfcb87628667fa1264"}]}
