)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":35263,"name":"Matt Crees","email":"mattc@stackhpc.com","username":"mattcrees"},"change_message_id":"281c41d532c2c3c6479fd444e1f3bb38af38c967","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"2b6f0ae9_1f8eb64f","updated":"2024-03-08 16:00:31.000000000","message":"Have tested, this works. The unit test changed look sensible too.\n\nWill +2 once the pep8 formatting is resolved. :)","commit_id":"819bf9d76d09ef79eccc1364583ce86f6ad371e1"}],"blazar/api/v1/leases/service.py":[{"author":{"_account_id":35263,"name":"Matt Crees","email":"mattc@stackhpc.com","username":"mattcrees"},"change_message_id":"281c41d532c2c3c6479fd444e1f3bb38af38c967","unresolved":true,"context_lines":[{"line_number":51,"context_line":"        \"\"\""},{"line_number":52,"context_line":"        # TODO(priteau): If possible, extend the context object used in the"},{"line_number":53,"context_line":"        # manager to keep track of the trustor and if they are admin"},{"line_number":54,"context_line":"        # instead of using the following three lines"},{"line_number":55,"context_line":"        ctx \u003d context.current()"},{"line_number":56,"context_line":"        data[\u0027user_id\u0027] \u003d ctx.user_id"},{"line_number":57,"context_line":"        data[\u0027is_admin_user\u0027] \u003d policy.enforce("}],"source_content_type":"text/x-python","patch_set":3,"id":"89e0885d_9c93a8f6","line":54,"range":{"start_line":54,"start_character":27,"end_line":54,"end_character":52},"updated":"2024-03-08 16:00:31.000000000","message":"nit: I count five lines ;). I\u0027d say just drop the number entirely here, \"the following lines\" is sufficient.","commit_id":"819bf9d76d09ef79eccc1364583ce86f6ad371e1"},{"author":{"_account_id":36611,"name":"Anish Reddy","display_name":"Anish Ravula","email":"2anishreddy@gmail.com","username":"aravula"},"change_message_id":"9ccfcfa3762b8f4779c6c4f3a677ecaa6805d0de","unresolved":false,"context_lines":[{"line_number":51,"context_line":"        \"\"\""},{"line_number":52,"context_line":"        # TODO(priteau): If possible, extend the context object used in the"},{"line_number":53,"context_line":"        # manager to keep track of the trustor and if they are admin"},{"line_number":54,"context_line":"        # instead of using the following three lines"},{"line_number":55,"context_line":"        ctx \u003d context.current()"},{"line_number":56,"context_line":"        data[\u0027user_id\u0027] \u003d ctx.user_id"},{"line_number":57,"context_line":"        data[\u0027is_admin_user\u0027] \u003d policy.enforce("}],"source_content_type":"text/x-python","patch_set":3,"id":"b49a46ef_7ef751ad","line":54,"range":{"start_line":54,"start_character":27,"end_line":54,"end_character":52},"in_reply_to":"89e0885d_9c93a8f6","updated":"2024-03-11 15:48:35.000000000","message":"Acknowledged","commit_id":"819bf9d76d09ef79eccc1364583ce86f6ad371e1"}]}
