)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":35125,"name":"Mauricio Harley","email":"mharley@redhat.com","username":"mharley-rh"},"change_message_id":"758e68093a5489a87a2a2d126ab700821a597fef","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"93378d89_361f3736","updated":"2026-04-24 11:20:20.000000000","message":"Fixing my previous approval.  Need some attention here.","commit_id":"ca85d4f02760339723cca81f68c8d29dadd6f142"},{"author":{"_account_id":35125,"name":"Mauricio Harley","email":"mharley@redhat.com","username":"mharley-rh"},"change_message_id":"ec737a995bacd8ef95888ee7295802abb78f09bf","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"40895595_05f5d1d3","updated":"2026-04-24 11:24:59.000000000","message":"Good job, here.  Just fix these two small issues.","commit_id":"ca85d4f02760339723cca81f68c8d29dadd6f142"}],"barbicanclient/v1/acls.py":[{"author":{"_account_id":35125,"name":"Mauricio Harley","email":"mharley@redhat.com","username":"mharley-rh"},"change_message_id":"ec737a995bacd8ef95888ee7295802abb78f09bf","unresolved":true,"context_lines":[{"line_number":404,"context_line":""},{"line_number":405,"context_line":"        if ref_type:"},{"line_number":406,"context_line":"            if ref_type not in [\u0027secret\u0027, \u0027container\u0027]:"},{"line_number":407,"context_line":"                raise ValueError(\u0027Invalid ref_type: %s\u0027)"},{"line_number":408,"context_line":"            return ref_type"},{"line_number":409,"context_line":""},{"line_number":410,"context_line":"        elems \u003d entity_ref.split(\u0027/\u0027)"}],"source_content_type":"text/x-python","patch_set":2,"id":"3d55e468_70260dd3","line":407,"updated":"2026-04-24 11:24:59.000000000","message":"Bug: %s is not interpolated here. The error message will literally show %s instead of the actual value. Should be raise ValueError(\u0027Invalid ref_type: %s\u0027 % ref_type).","commit_id":"ca85d4f02760339723cca81f68c8d29dadd6f142"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"0f06a3b2c886e8e843da1c4fd92a081f453b5c10","unresolved":false,"context_lines":[{"line_number":404,"context_line":""},{"line_number":405,"context_line":"        if ref_type:"},{"line_number":406,"context_line":"            if ref_type not in [\u0027secret\u0027, \u0027container\u0027]:"},{"line_number":407,"context_line":"                raise ValueError(\u0027Invalid ref_type: %s\u0027)"},{"line_number":408,"context_line":"            return ref_type"},{"line_number":409,"context_line":""},{"line_number":410,"context_line":"        elems \u003d entity_ref.split(\u0027/\u0027)"}],"source_content_type":"text/x-python","patch_set":2,"id":"8f30ebf7_686cc23c","line":407,"in_reply_to":"3d55e468_70260dd3","updated":"2026-04-24 15:25:01.000000000","message":"Yes. Good catch !","commit_id":"ca85d4f02760339723cca81f68c8d29dadd6f142"},{"author":{"_account_id":35125,"name":"Mauricio Harley","email":"mharley@redhat.com","username":"mharley-rh"},"change_message_id":"ec737a995bacd8ef95888ee7295802abb78f09bf","unresolved":true,"context_lines":[{"line_number":467,"context_line":""},{"line_number":468,"context_line":"        :param str entity_ref: Full HATEOAS reference to a secret or container,"},{"line_number":469,"context_line":"            or a UUID."},{"line_number":470,"context_line":"        :param users: List of Keystone userid(s) to be used in ACL."},{"line_number":471,"context_line":"        :param users: List of Keystone userid(s) to be used in ACL."},{"line_number":472,"context_line":"        :type users: List or None"},{"line_number":473,"context_line":"        :param bool project_access: Flag indicating project access behavior"}],"source_content_type":"text/x-python","patch_set":2,"id":"e97d2603_86f3493f","line":470,"updated":"2026-04-24 11:24:59.000000000","message":"Nit: :param users: is documented twice.","commit_id":"ca85d4f02760339723cca81f68c8d29dadd6f142"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"0f06a3b2c886e8e843da1c4fd92a081f453b5c10","unresolved":false,"context_lines":[{"line_number":467,"context_line":""},{"line_number":468,"context_line":"        :param str entity_ref: Full HATEOAS reference to a secret or container,"},{"line_number":469,"context_line":"            or a UUID."},{"line_number":470,"context_line":"        :param users: List of Keystone userid(s) to be used in ACL."},{"line_number":471,"context_line":"        :param users: List of Keystone userid(s) to be used in ACL."},{"line_number":472,"context_line":"        :type users: List or None"},{"line_number":473,"context_line":"        :param bool project_access: Flag indicating project access behavior"}],"source_content_type":"text/x-python","patch_set":2,"id":"8df98e72_86a84710","line":470,"in_reply_to":"e97d2603_86f3493f","updated":"2026-04-24 15:25:01.000000000","message":"Done","commit_id":"ca85d4f02760339723cca81f68c8d29dadd6f142"}]}