)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":20813,"name":"Sofia Enriquez","email":"lsofia.enriquez@gmail.com","username":"enriquetaso"},"change_message_id":"e8cc66f98d77460dffde59ef7872fece9240faa1","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"5221a979_93bf6c9c","updated":"2023-06-21 09:10:31.000000000","message":"LGTM, please reply to my questions in the last review 😊","commit_id":"bdcf3106db8259bc0c8d81224392d3a69dd9ec6f"},{"author":{"_account_id":20813,"name":"Sofia Enriquez","email":"lsofia.enriquez@gmail.com","username":"enriquetaso"},"change_message_id":"a9a2a2d374380536bafbc6ee56107a9f7fddca69","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":2,"id":"d06049a5_42132993","updated":"2023-06-20 12:40:44.000000000","message":"LGTM. I just have a few questions for my own clarification:\n\n(1) Should the file api-ref/source/v3/samples/attachment-create-response.json be deleted?\n(2) Should the create response be placed inside a directory for the microversion, like api-ref/source/v3/samples/attachments/v3.27/attachment-create-response.json?","commit_id":"bdcf3106db8259bc0c8d81224392d3a69dd9ec6f"}],"api-ref/source/v3/samples/attachments/attachment-create-response.json":[{"author":{"_account_id":20813,"name":"Sofia Enriquez","email":"lsofia.enriquez@gmail.com","username":"enriquetaso"},"change_message_id":"062f354a280aa9e4b17c087e8ebb56c912f4cbbc","unresolved":true,"context_lines":[{"line_number":11,"context_line":"            \"qos_specs\": null"},{"line_number":12,"context_line":"        },"},{"line_number":13,"context_line":"        \"detached_at\": \"\","},{"line_number":14,"context_line":"        \"id\": \"4be95312-9d57-4610-9e3f-1d374a09a21b\","},{"line_number":15,"context_line":"        \"instance\": \"fa617131-cdbc-45dc-afff-f21f17ae054e\","},{"line_number":16,"context_line":"        \"status\": \"reserved\","},{"line_number":17,"context_line":"        \"volume_id\": \"5e9fd8c2-33b4-44c4-b88e-1daeeb82cf76\""}],"source_content_type":"application/json","patch_set":2,"id":"8cb7778c_9f1630e8","line":14,"range":{"start_line":14,"start_character":9,"end_line":14,"end_character":11},"updated":"2023-06-21 09:52:21.000000000","message":"same as the other comment, this should be `attachment_id`","commit_id":"bdcf3106db8259bc0c8d81224392d3a69dd9ec6f"}],"cinder/tests/functional/api_sample_tests/samples/attachments/attachment-create-response.json.tpl":[{"author":{"_account_id":20813,"name":"Sofia Enriquez","email":"lsofia.enriquez@gmail.com","username":"enriquetaso"},"change_message_id":"a9a2a2d374380536bafbc6ee56107a9f7fddca69","unresolved":true,"context_lines":[{"line_number":1,"context_line":"{"},{"line_number":2,"context_line":"    \"attachment\": {"},{"line_number":3,"context_line":"        \"attach_mode\": \"null\","},{"line_number":4,"context_line":"        \"attached_at\": \"\","},{"line_number":5,"context_line":"        \"connection_info\": {"},{"line_number":6,"context_line":"            \"access_mode\": \"rw\","}],"source_content_type":"text/x-smarty","patch_set":2,"id":"ecdc95a3_57737e08","line":3,"range":{"start_line":3,"start_character":24,"end_line":3,"end_character":28},"updated":"2023-06-20 12:40:44.000000000","message":"I\u0027m not sure if this is an actual issue or not. Should this particular item be differentiated from \u0027null\u0027?","commit_id":"bdcf3106db8259bc0c8d81224392d3a69dd9ec6f"},{"author":{"_account_id":20813,"name":"Sofia Enriquez","email":"lsofia.enriquez@gmail.com","username":"enriquetaso"},"change_message_id":"062f354a280aa9e4b17c087e8ebb56c912f4cbbc","unresolved":true,"context_lines":[{"line_number":11,"context_line":"            \"qos_specs\": null"},{"line_number":12,"context_line":"        },"},{"line_number":13,"context_line":"        \"detached_at\": \"\","},{"line_number":14,"context_line":"        \"id\": \"%(uuid)s\","},{"line_number":15,"context_line":"        \"instance\": \"%(instance_uuid)s\","},{"line_number":16,"context_line":"        \"status\": \"reserved\","},{"line_number":17,"context_line":"        \"volume_id\": \"%(uuid)s\""}],"source_content_type":"text/x-smarty","patch_set":2,"id":"d402e7df_fdc3ef82","line":14,"range":{"start_line":14,"start_character":9,"end_line":14,"end_character":11},"updated":"2023-06-21 09:52:21.000000000","message":"this should be `attachment_id` \n\nResponse:\n`{\u0027attachment\u0027: {\u0027id\u0027: \u002772232409-e7a0-4312-a895-13c81e3ac4cc\u0027, \u0027status\u0027: \u0027reserved\u0027, \u0027instance\u0027: \u0027fa617131-cdbc-45dc-afff-f21f17ae054e\u0027, \u0027volume_id\u0027: \u00279c814136-a90e-49a7-8062-7463fe84866b\u0027, \u0027\nattached_at\u0027: \u0027\u0027, \u0027detached_at\u0027: \u0027\u0027, \u0027attach_mode\u0027: \u0027null\u0027, \u0027connection_info\u0027: {\u0027access_mode\u0027: \u0027rw\u0027, \u0027qos_specs\u0027: None, \u0027encrypted\u0027: False, \u0027cacheable\u0027: False, \u0027driver_volume_type\u0027: \u0027iscsi\u0027,\n \u0027attachment_id\u0027: \u002772232409-e7a0-4312-a895-13c81e3ac4cc\u0027}}}\n`","commit_id":"bdcf3106db8259bc0c8d81224392d3a69dd9ec6f"}]}
