)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"ab57a13bd0f9aafbeca6e7403f1e6b6026d3cdbf","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"d0b136c8_c7f927e2","updated":"2022-04-26 02:42:25.000000000","message":"It would be really nice to have this documented.  I have a quibble about your phrasing in the checklist item, otherwise this looks great.","commit_id":"f15bb884da89974b0b6cce4f157461f383b75e00"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"8761ae9cc677854fd2cc0ef05a5ee7e769c796e7","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"b86250ea_c0cef17a","updated":"2022-05-12 20:34:43.000000000","message":"LGTM.  Ninja-approving a docs change.","commit_id":"270763214776596b8a17b726e41a3719378a23af"}],"doc/source/contributor/drivers.rst":[{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"ab57a13bd0f9aafbeca6e7403f1e6b6026d3cdbf","unresolved":true,"context_lines":[{"line_number":156,"context_line":"Volume ID"},{"line_number":157,"context_line":"`````````"},{"line_number":158,"context_line":""},{"line_number":159,"context_line":"Driver\u0027s should always get a volume\u0027s ID using the ``name_id`` attribute"},{"line_number":160,"context_line":"instead of the ``id`` attribute."},{"line_number":161,"context_line":""},{"line_number":162,"context_line":"A Cinder volume may have two different UUIDs, a user facing one, and one the"}],"source_content_type":"text/x-rst","patch_set":2,"id":"bc0e648f_0ec8c997","line":159,"range":{"start_line":159,"start_character":0,"end_line":159,"end_character":8},"updated":"2022-04-26 02:42:25.000000000","message":"nit: \"Drivers\" (no apostrophe)","commit_id":"f15bb884da89974b0b6cce4f157461f383b75e00"},{"author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"change_message_id":"4eb40befa447d2031c7608e63fd4deb77e872e68","unresolved":false,"context_lines":[{"line_number":156,"context_line":"Volume ID"},{"line_number":157,"context_line":"`````````"},{"line_number":158,"context_line":""},{"line_number":159,"context_line":"Driver\u0027s should always get a volume\u0027s ID using the ``name_id`` attribute"},{"line_number":160,"context_line":"instead of the ``id`` attribute."},{"line_number":161,"context_line":""},{"line_number":162,"context_line":"A Cinder volume may have two different UUIDs, a user facing one, and one the"}],"source_content_type":"text/x-rst","patch_set":2,"id":"9928e8d3_7a270bb7","line":159,"range":{"start_line":159,"start_character":0,"end_line":159,"end_character":8},"in_reply_to":"bc0e648f_0ec8c997","updated":"2022-05-12 17:53:59.000000000","message":"Done","commit_id":"f15bb884da89974b0b6cce4f157461f383b75e00"}],"doc/source/contributor/new_driver_checklist.rst":[{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"ab57a13bd0f9aafbeca6e7403f1e6b6026d3cdbf","unresolved":true,"context_lines":[{"line_number":24,"context_line":""},{"line_number":25,"context_line":"* Common gotchas"},{"line_number":26,"context_line":""},{"line_number":27,"context_line":"  * Code uses ``volume.name_id`` instead of ``volume.id``."},{"line_number":28,"context_line":"  * Handles detach where ``connector \u003d\u003d None`` for force detach"},{"line_number":29,"context_line":"  * Create from snapshot and clone properly account for new volume size being"},{"line_number":30,"context_line":"    larger than original volume size"}],"source_content_type":"text/x-rst","patch_set":2,"id":"0e124cb7_56c9a1f9","line":27,"range":{"start_line":27,"start_character":4,"end_line":27,"end_character":58},"updated":"2022-04-26 02:42:25.000000000","message":"I think it would be more clear if you s/uses/should use/ here, otherwise it\u0027s not obvious what the desired behavior is.","commit_id":"f15bb884da89974b0b6cce4f157461f383b75e00"},{"author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"change_message_id":"4eb40befa447d2031c7608e63fd4deb77e872e68","unresolved":false,"context_lines":[{"line_number":24,"context_line":""},{"line_number":25,"context_line":"* Common gotchas"},{"line_number":26,"context_line":""},{"line_number":27,"context_line":"  * Code uses ``volume.name_id`` instead of ``volume.id``."},{"line_number":28,"context_line":"  * Handles detach where ``connector \u003d\u003d None`` for force detach"},{"line_number":29,"context_line":"  * Create from snapshot and clone properly account for new volume size being"},{"line_number":30,"context_line":"    larger than original volume size"}],"source_content_type":"text/x-rst","patch_set":2,"id":"d91d484f_005c5d47","line":27,"range":{"start_line":27,"start_character":4,"end_line":27,"end_character":58},"in_reply_to":"0e124cb7_56c9a1f9","updated":"2022-05-12 17:53:59.000000000","message":"Done","commit_id":"f15bb884da89974b0b6cce4f157461f383b75e00"}]}