)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":9236,"name":"Jon Bernard","email":"jobernar@redhat.com","username":"jbernard"},"change_message_id":"d56f2135aa92f803f483a682f2f08fedbb40a9c4","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"8098c074_dd8f135f","updated":"2023-08-31 19:44:18.000000000","message":"I perfer this version (as opposed to\nhttps://review.opendev.org/c/openstack/cinder/+/893429).","commit_id":"785f23b3bc219c554698fe54fbd478ef04fc9229"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"ad0f76d6add95b93cf4eaa25db25da846d8dcf54","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"aca8c598_c1eb2fdb","updated":"2023-08-31 16:04:59.000000000","message":"Makes sense to me.","commit_id":"785f23b3bc219c554698fe54fbd478ef04fc9229"},{"author":{"_account_id":10342,"name":"Jay Faulkner","display_name":"JayF","email":"jay@jvf.cc","username":"JayF","status":"youtube.com/@oss-gr / podcast.gr-oss.io"},"change_message_id":"64aa99ab79b6ba528531a356208bd1dc1f67b0b5","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"5e703301_eb091319","updated":"2023-08-31 18:54:26.000000000","message":"See https://review.opendev.org/c/openstack/cinder/+/893429 for alternative implementation suggested by Vasyl","commit_id":"785f23b3bc219c554698fe54fbd478ef04fc9229"},{"author":{"_account_id":10342,"name":"Jay Faulkner","display_name":"JayF","email":"jay@jvf.cc","username":"JayF","status":"youtube.com/@oss-gr / podcast.gr-oss.io"},"change_message_id":"75641634fdd41965160b05a50f010c7a3daddaf1","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"9be95713_7cadf0d3","updated":"2023-08-30 15:52:43.000000000","message":"Sorry for breaking you in oslo :(","commit_id":"785f23b3bc219c554698fe54fbd478ef04fc9229"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"881b1a778510beb22804eb2f63169d36bee31b3a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"98706ab8_17bf859f","updated":"2023-08-31 16:18:24.000000000","message":"The commit message is very clear, but see comment #42 on the bug for more details:\nhttps://bugs.launchpad.net/ironic/+bug/2030976/comments/42","commit_id":"785f23b3bc219c554698fe54fbd478ef04fc9229"},{"author":{"_account_id":10342,"name":"Jay Faulkner","display_name":"JayF","email":"jay@jvf.cc","username":"JayF","status":"youtube.com/@oss-gr / podcast.gr-oss.io"},"change_message_id":"87d8e79cf137abd47740af7652e47337bf3c3dbe","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"4e5c4320_23a9451e","updated":"2023-08-31 14:31:27.000000000","message":"This change needs to go back to xena.","commit_id":"785f23b3bc219c554698fe54fbd478ef04fc9229"},{"author":{"_account_id":10342,"name":"Jay Faulkner","display_name":"JayF","email":"jay@jvf.cc","username":"JayF","status":"youtube.com/@oss-gr / podcast.gr-oss.io"},"change_message_id":"220010824b2b4f046d2bcca16b11f907182d76dd","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"ec3d831c_1ae005a9","updated":"2023-08-31 20:23:11.000000000","message":"recheck cinder-tempest-plugin-lvm-lio-barbican timeout","commit_id":"785f23b3bc219c554698fe54fbd478ef04fc9229"},{"author":{"_account_id":10342,"name":"Jay Faulkner","display_name":"JayF","email":"jay@jvf.cc","username":"JayF","status":"youtube.com/@oss-gr / podcast.gr-oss.io"},"change_message_id":"11625a6262b958bd7b661fe971989549449a356d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"08caccb6_75502d7b","updated":"2023-09-01 21:02:39.000000000","message":"recheck cinder-tempest-plugin-lvm-lio-barbican timeout unrelated to change","commit_id":"785f23b3bc219c554698fe54fbd478ef04fc9229"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"1352fd2cf873c5910da0b2294fd0eb4809f140be","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"cb0cfc01_0cd606d0","updated":"2023-09-09 00:55:40.000000000","message":"recheck https://review.opendev.org/c/openstack/tempest/+/894269 has merged","commit_id":"785f23b3bc219c554698fe54fbd478ef04fc9229"},{"author":{"_account_id":10342,"name":"Jay Faulkner","display_name":"JayF","email":"jay@jvf.cc","username":"JayF","status":"youtube.com/@oss-gr / podcast.gr-oss.io"},"change_message_id":"cc0d0e23cbb39157b0c107ef32fb574562ab60ed","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"13116a72_c634e403","updated":"2023-08-31 17:58:18.000000000","message":"recheck some zuul error uploading logs\n\nzuul@172.99.67.80: Permission denied (publickey,gssapi-keyex,gssapi-with-mic,password).","commit_id":"785f23b3bc219c554698fe54fbd478ef04fc9229"},{"author":{"_account_id":10342,"name":"Jay Faulkner","display_name":"JayF","email":"jay@jvf.cc","username":"JayF","status":"youtube.com/@oss-gr / podcast.gr-oss.io"},"change_message_id":"af34c6952fff240e2b3ebc93698303a7ec327073","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"87716695_e74da31a","updated":"2023-08-31 14:30:49.000000000","message":"recheck unit test failure is transitory","commit_id":"785f23b3bc219c554698fe54fbd478ef04fc9229"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"5153298c86bdac00d8536e5d51818af8d1eae8df","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"597f3f3c_60c3dafc","updated":"2023-09-08 21:08:04.000000000","message":"waiting for https://review.opendev.org/c/openstack/tempest/+/894269 to merge before rechecking","commit_id":"785f23b3bc219c554698fe54fbd478ef04fc9229"}],"cinder/context.py":[{"author":{"_account_id":14525,"name":"Vasyl Saienko","email":"vsaienko@mirantis.com","username":"vsaienko"},"change_message_id":"9271efd9045b6845b195b9d4d99f9400d9a02d9d","unresolved":true,"context_lines":[{"line_number":190,"context_line":"                   project_id\u003dvalues.get(\u0027project_id\u0027),"},{"line_number":191,"context_line":"                   project_name\u003dvalues.get(\u0027project_name\u0027),"},{"line_number":192,"context_line":"                   domain_id\u003dvalues.get(\u0027domain_id\u0027),"},{"line_number":193,"context_line":"                   read_deleted\u003dvalues.get(\u0027read_deleted\u0027, \u0027no\u0027),"},{"line_number":194,"context_line":"                   remote_address\u003dvalues.get(\u0027remote_address\u0027),"},{"line_number":195,"context_line":"                   timestamp\u003dvalues.get(\u0027timestamp\u0027),"},{"line_number":196,"context_line":"                   quota_class\u003dvalues.get(\u0027quota_class\u0027),"}],"source_content_type":"text/x-python","patch_set":2,"id":"ecc79c32_85de22ac","line":193,"range":{"start_line":193,"start_character":60,"end_line":193,"end_character":62},"updated":"2023-08-31 18:45:29.000000000","message":"why we need to duplicate defaults for optional options? maybe better to change at L89\n\nread_deleted: Optional[str] \u003d None,\n\nand later in method\n\nif read_deleted is None:\n    read_deleted \u003d \"no\"","commit_id":"785f23b3bc219c554698fe54fbd478ef04fc9229"},{"author":{"_account_id":10342,"name":"Jay Faulkner","display_name":"JayF","email":"jay@jvf.cc","username":"JayF","status":"youtube.com/@oss-gr / podcast.gr-oss.io"},"change_message_id":"64aa99ab79b6ba528531a356208bd1dc1f67b0b5","unresolved":true,"context_lines":[{"line_number":190,"context_line":"                   project_id\u003dvalues.get(\u0027project_id\u0027),"},{"line_number":191,"context_line":"                   project_name\u003dvalues.get(\u0027project_name\u0027),"},{"line_number":192,"context_line":"                   domain_id\u003dvalues.get(\u0027domain_id\u0027),"},{"line_number":193,"context_line":"                   read_deleted\u003dvalues.get(\u0027read_deleted\u0027, \u0027no\u0027),"},{"line_number":194,"context_line":"                   remote_address\u003dvalues.get(\u0027remote_address\u0027),"},{"line_number":195,"context_line":"                   timestamp\u003dvalues.get(\u0027timestamp\u0027),"},{"line_number":196,"context_line":"                   quota_class\u003dvalues.get(\u0027quota_class\u0027),"}],"source_content_type":"text/x-python","patch_set":2,"id":"ba596777_c2f1dd62","line":193,"range":{"start_line":193,"start_character":60,"end_line":193,"end_character":62},"in_reply_to":"ecc79c32_85de22ac","updated":"2023-08-31 18:54:26.000000000","message":"I opened https://review.opendev.org/c/openstack/cinder/+/893429 implementing this suggestion. I think I still prefer this method, as it\u0027s the way we solved this in Manila, but my main goal is to unbreak Cinder.","commit_id":"785f23b3bc219c554698fe54fbd478ef04fc9229"}]}
