)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"9f6e99728a118ee3b84610b65d30031d5d9f9d45","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"b498be69_0c68a52b","updated":"2021-11-30 19:34:55.000000000","message":"recheck","commit_id":"3692438ffae91749ce84bb2cc46bb38b5d280bf3"}],"openstack/compute/v2/volume_attachment.py":[{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"acbc9ddc25c2d151f8671dfab651a895d17284e2","unresolved":true,"context_lines":[{"line_number":27,"context_line":""},{"line_number":28,"context_line":"    _query_mapping \u003d resource.QueryParameters(\"limit\", \"offset\")"},{"line_number":29,"context_line":""},{"line_number":30,"context_line":"    #: The ID for the server."},{"line_number":31,"context_line":"    server_id \u003d resource.URI(\u0027server_id\u0027)"},{"line_number":32,"context_line":"    #: Name of the device such as, /dev/vdb."},{"line_number":33,"context_line":"    device \u003d resource.Body(\u0027device\u0027)"}],"source_content_type":"text/x-python","patch_set":1,"id":"df04300a_b4d6e8d2","line":30,"range":{"start_line":30,"start_character":11,"end_line":30,"end_character":13},"updated":"2021-11-16 09:27:26.000000000","message":"I don\u0027t understand why some comments refer to \"ID\" while some say \"UUID\". The IDs for the server, the volume, the attachment and the BDM are all UUIDs, aren\u0027t they? Would make sense to be consistent with the wording, then.","commit_id":"3692438ffae91749ce84bb2cc46bb38b5d280bf3"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"010d05c82c02ccf0d1223a8b3e7e7ab8ee173829","unresolved":false,"context_lines":[{"line_number":27,"context_line":""},{"line_number":28,"context_line":"    _query_mapping \u003d resource.QueryParameters(\"limit\", \"offset\")"},{"line_number":29,"context_line":""},{"line_number":30,"context_line":"    #: The ID for the server."},{"line_number":31,"context_line":"    server_id \u003d resource.URI(\u0027server_id\u0027)"},{"line_number":32,"context_line":"    #: Name of the device such as, /dev/vdb."},{"line_number":33,"context_line":"    device \u003d resource.Body(\u0027device\u0027)"}],"source_content_type":"text/x-python","patch_set":1,"id":"243d5e48_edd57571","line":30,"range":{"start_line":30,"start_character":11,"end_line":30,"end_character":13},"in_reply_to":"df04300a_b4d6e8d2","updated":"2021-11-17 11:45:05.000000000","message":"I assume you\u0027re referring to the comments? Yes, it would. This is pretty confusing though as we\u0027ve a policy of using \u0027_id\u0027 and not \u0027_uuid\u0027 for variables so I\u0027m not sure what type to actually expose here /o\\\n\nIn any case, this is a cherry-pick from feature/r1 and we should probably keep it identical and fix on feature/r1 first (since that\u0027ll be merged into here eventually).","commit_id":"3692438ffae91749ce84bb2cc46bb38b5d280bf3"}]}