)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"248682d1148426b91dcdcf04f20816e2fbbf5717","unresolved":true,"context_lines":[{"line_number":13,"context_line":"  hw_virtio_packed_ring image property."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"Nova spec: https://review.opendev.org/c/openstack/nova-specs/+/868377"},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"Change-Id: Ia8caf39467a91431ca85b2a56ec93a60c41a8e7d"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"d0a41975_aef6c360","line":16,"updated":"2023-05-04 11:08:40.000000000","message":"technically this should have\n\nDepends-On: https://review.opendev.org/c/openstack/nova/+/876075\n\nsince this should only merge after the nova change has merged","commit_id":"61ee0a943a0c81090fcb0f4adcc4b84476ad3f44"},{"author":{"_account_id":35489,"name":"Danylo Vodopianov","email":"dvo-plv@napatech.com","username":"dvo-plv"},"change_message_id":"ab0951ced45fcb3c1da4864d8e6f6e1deeba3f15","unresolved":false,"context_lines":[{"line_number":13,"context_line":"  hw_virtio_packed_ring image property."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"Nova spec: https://review.opendev.org/c/openstack/nova-specs/+/868377"},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"Change-Id: Ia8caf39467a91431ca85b2a56ec93a60c41a8e7d"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"1f33f807_18d67f15","line":16,"in_reply_to":"d0a41975_aef6c360","updated":"2023-05-04 11:25:04.000000000","message":"The commit message was updated with Depends-on item","commit_id":"61ee0a943a0c81090fcb0f4adcc4b84476ad3f44"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"4131d62d977ed8bb8e78e3b596ce1878d9e71c10","unresolved":true,"context_lines":[{"line_number":15,"context_line":"Nova spec: https://review.opendev.org/c/openstack/nova-specs/+/868377"},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"Depends-On: https://review.opendev.org/c/openstack/nova/+/876075"},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"Change-Id: Ia8caf39467a91431ca85b2a56ec93a60c41a8e7d"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"6827f12e_a08c96b4","line":18,"updated":"2023-12-05 08:44:33.000000000","message":"both the nova spec and code change have now been merged so we can proceed\nwith the glance update.","commit_id":"eaefeb09bcc27bd1b81fc8497b4e5984d6bad4e2"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"b0750045070e896dcdce7ac4e9c8ecf6b83d0b87","unresolved":false,"context_lines":[{"line_number":15,"context_line":"Nova spec: https://review.opendev.org/c/openstack/nova-specs/+/868377"},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"Depends-On: https://review.opendev.org/c/openstack/nova/+/876075"},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"Change-Id: Ia8caf39467a91431ca85b2a56ec93a60c41a8e7d"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"f36eabd2_bce2dd40","line":18,"in_reply_to":"6827f12e_a08c96b4","updated":"2023-12-05 11:26:03.000000000","message":"Acknowledged","commit_id":"eaefeb09bcc27bd1b81fc8497b4e5984d6bad4e2"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"248682d1148426b91dcdcf04f20816e2fbbf5717","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"5086132c_349a0c65","updated":"2023-05-04 11:08:40.000000000","message":"+1 but this should wait for the nova patch to land before its meged so that we are not reporting properties that have not been added yet","commit_id":"61ee0a943a0c81090fcb0f4adcc4b84476ad3f44"},{"author":{"_account_id":8122,"name":"Cyril Roelandt","email":"cyril@redhat.com","username":"cyril.roelandt.enovance"},"change_message_id":"c8b912384e0112b944bc406e81f53e64377d66ef","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"1b332a58_5b41c4f5","updated":"2023-05-03 13:17:42.000000000","message":"Thanks for the patch, I just fixed the pep8 issue so we can try and merge this in the review party tomorrow.","commit_id":"61ee0a943a0c81090fcb0f4adcc4b84476ad3f44"},{"author":{"_account_id":35489,"name":"Danylo Vodopianov","email":"dvo-plv@napatech.com","username":"dvo-plv"},"change_message_id":"a9a64255fbe3176eec3a717eb406dbac25d7324f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"fb3dbd27_67aa2c73","updated":"2023-11-30 12:18:01.000000000","message":"recheck","commit_id":"cb5ec1e692af20037d85a81963eb6728b132e5bf"},{"author":{"_account_id":35489,"name":"Danylo Vodopianov","email":"dvo-plv@napatech.com","username":"dvo-plv"},"change_message_id":"c1c61c3f78d2b6697a0e257e1eded12efb0ae4bd","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"eee81430_cd4d4292","updated":"2023-11-30 12:17:22.000000000","message":"rechek","commit_id":"cb5ec1e692af20037d85a81963eb6728b132e5bf"},{"author":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"change_message_id":"be7e7df5a4700bbfe774995dc9a321a3e0fb97df","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"8fc0b0a3_d8ddffd4","updated":"2023-12-01 04:28:42.000000000","message":"Looks good to me!!","commit_id":"eaefeb09bcc27bd1b81fc8497b4e5984d6bad4e2"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"4131d62d977ed8bb8e78e3b596ce1878d9e71c10","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"34ed31c8_eaf2306b","updated":"2023-12-05 08:44:33.000000000","message":"this is ok but i would presonally prefer if we made the description a litlle more detailed i have left a suggestion inline","commit_id":"eaefeb09bcc27bd1b81fc8497b4e5984d6bad4e2"},{"author":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"change_message_id":"483ec2e7ed59fff8799c955521dd0d8413a73d98","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"73967a1b_62ae1d0d","updated":"2023-12-21 09:25:38.000000000","message":"Looks good, ninja approving it since cyril already voted on it earlier before some changes in description.\n\nThank you for your patience.","commit_id":"66566c3286890eab2927ece085e250cbe032119b"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"80e911465a5e853f5b1c497c16783186905c7da2","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"cb73a9ef_25286aed","updated":"2023-12-21 12:26:58.000000000","message":"mysql was OOM killed in tempest-integrated-storage","commit_id":"66566c3286890eab2927ece085e250cbe032119b"},{"author":{"_account_id":35489,"name":"Danylo Vodopianov","email":"dvo-plv@napatech.com","username":"dvo-plv"},"change_message_id":"509edb71384b6e8405c694a60b0dcac82bae33a9","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"15b09898_8373e959","updated":"2023-12-21 11:47:41.000000000","message":"recheck","commit_id":"66566c3286890eab2927ece085e250cbe032119b"},{"author":{"_account_id":35489,"name":"Danylo Vodopianov","email":"dvo-plv@napatech.com","username":"dvo-plv"},"change_message_id":"626af92fbcc0e635488c8fff8a895d97b919837e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"75b953d0_cb17e57c","updated":"2023-12-21 11:40:26.000000000","message":"rechek","commit_id":"66566c3286890eab2927ece085e250cbe032119b"}],"doc/source/admin/useful-image-properties.rst":[{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"4131d62d977ed8bb8e78e3b596ce1878d9e71c10","unresolved":true,"context_lines":[{"line_number":449,"context_line":"``hw_virtio_packed_ring``"},{"line_number":450,"context_line":"  :Type: bool"},{"line_number":451,"context_line":""},{"line_number":452,"context_line":"  Enables Packed Virtqueue feature. If there are compute hosts available"},{"line_number":453,"context_line":"  which support this, instance will have additional performance impact."},{"line_number":454,"context_line":""},{"line_number":455,"context_line":"  Only supported by the libvirt driver."},{"line_number":456,"context_line":""}],"source_content_type":"text/x-rst","patch_set":4,"id":"b72fa86c_6764a427","line":453,"range":{"start_line":452,"start_character":2,"end_line":453,"end_character":70},"updated":"2023-12-05 08:44:33.000000000","message":"This is a little spares on detail, i would add more explainaition like this.\n\nEnables Packed VIRT-IO Queue feature. When set to true, instance will be\nscheduled to hosts that support negotiating the packed virt queue format.\nThis feature may or may not be enabled depending on the guest driver.\nWhen used it will improve the small packet performance of network io.","commit_id":"eaefeb09bcc27bd1b81fc8497b4e5984d6bad4e2"},{"author":{"_account_id":35489,"name":"Danylo Vodopianov","email":"dvo-plv@napatech.com","username":"dvo-plv"},"change_message_id":"0e8ba2de3df2fe4daa1d46d8a9859914f04faa57","unresolved":false,"context_lines":[{"line_number":449,"context_line":"``hw_virtio_packed_ring``"},{"line_number":450,"context_line":"  :Type: bool"},{"line_number":451,"context_line":""},{"line_number":452,"context_line":"  Enables Packed Virtqueue feature. If there are compute hosts available"},{"line_number":453,"context_line":"  which support this, instance will have additional performance impact."},{"line_number":454,"context_line":""},{"line_number":455,"context_line":"  Only supported by the libvirt driver."},{"line_number":456,"context_line":""}],"source_content_type":"text/x-rst","patch_set":4,"id":"2b204bf3_1f370023","line":453,"range":{"start_line":452,"start_character":2,"end_line":453,"end_character":70},"in_reply_to":"b72fa86c_6764a427","updated":"2023-12-05 09:00:24.000000000","message":"Acknowledged","commit_id":"eaefeb09bcc27bd1b81fc8497b4e5984d6bad4e2"}],"etc/metadefs/compute-libvirt.json":[{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"248682d1148426b91dcdcf04f20816e2fbbf5717","unresolved":true,"context_lines":[{"line_number":34,"context_line":"            \"enum\": [\"true\", \"false\"]"},{"line_number":35,"context_line":"        }"},{"line_number":36,"context_line":"        ,"},{"line_number":37,"context_line":"        \"virtio_packed_ring\": {"},{"line_number":38,"context_line":"            \"title\": \"Virtio Packed Ring\","},{"line_number":39,"context_line":"            \"description\": \"Enables Packed Virtqueues. If there are compute hosts available which support this, instance will have additional performance impact.\","},{"line_number":40,"context_line":"            \"type\": \"string\","}],"source_content_type":"application/json","patch_set":2,"id":"9cfaae63_8cbd5f4b","line":37,"range":{"start_line":37,"start_character":9,"end_line":37,"end_character":29},"updated":"2023-05-04 11:08:40.000000000","message":"+1 this will be both an image property and flavor extra spec so by adding it here it will be correctly reported for both","commit_id":"61ee0a943a0c81090fcb0f4adcc4b84476ad3f44"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"b0750045070e896dcdce7ac4e9c8ecf6b83d0b87","unresolved":false,"context_lines":[{"line_number":34,"context_line":"            \"enum\": [\"true\", \"false\"]"},{"line_number":35,"context_line":"        }"},{"line_number":36,"context_line":"        ,"},{"line_number":37,"context_line":"        \"virtio_packed_ring\": {"},{"line_number":38,"context_line":"            \"title\": \"Virtio Packed Ring\","},{"line_number":39,"context_line":"            \"description\": \"Enables Packed Virtqueues. If there are compute hosts available which support this, instance will have additional performance impact.\","},{"line_number":40,"context_line":"            \"type\": \"string\","}],"source_content_type":"application/json","patch_set":2,"id":"027b356b_60267927","line":37,"range":{"start_line":37,"start_character":9,"end_line":37,"end_character":29},"in_reply_to":"9cfaae63_8cbd5f4b","updated":"2023-12-05 11:26:03.000000000","message":"Done","commit_id":"61ee0a943a0c81090fcb0f4adcc4b84476ad3f44"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"4131d62d977ed8bb8e78e3b596ce1878d9e71c10","unresolved":true,"context_lines":[{"line_number":36,"context_line":"        ,"},{"line_number":37,"context_line":"        \"virtio_packed_ring\": {"},{"line_number":38,"context_line":"            \"title\": \"Virtio Packed Ring\","},{"line_number":39,"context_line":"            \"description\": \"Enables Packed Virtqueues. If there are compute hosts available which support this, instance will have additional performance impact.\","},{"line_number":40,"context_line":"            \"type\": \"string\","},{"line_number":41,"context_line":"            \"enum\": [\"true\", \"false\"]"},{"line_number":42,"context_line":"        }"}],"source_content_type":"application/json","patch_set":4,"id":"7b80e78c_9177aa94","line":39,"updated":"2023-12-05 08:44:33.000000000","message":"this is now wrong but i think we can impvoe the descript as i suggested here\nhttps://review.opendev.org/c/openstack/glance/+/882133/4/doc/source/admin/useful-image-properties.rst#453","commit_id":"eaefeb09bcc27bd1b81fc8497b4e5984d6bad4e2"},{"author":{"_account_id":35489,"name":"Danylo Vodopianov","email":"dvo-plv@napatech.com","username":"dvo-plv"},"change_message_id":"0e8ba2de3df2fe4daa1d46d8a9859914f04faa57","unresolved":false,"context_lines":[{"line_number":36,"context_line":"        ,"},{"line_number":37,"context_line":"        \"virtio_packed_ring\": {"},{"line_number":38,"context_line":"            \"title\": \"Virtio Packed Ring\","},{"line_number":39,"context_line":"            \"description\": \"Enables Packed Virtqueues. If there are compute hosts available which support this, instance will have additional performance impact.\","},{"line_number":40,"context_line":"            \"type\": \"string\","},{"line_number":41,"context_line":"            \"enum\": [\"true\", \"false\"]"},{"line_number":42,"context_line":"        }"}],"source_content_type":"application/json","patch_set":4,"id":"60f67086_a1bfd9e2","line":39,"in_reply_to":"7b80e78c_9177aa94","updated":"2023-12-05 09:00:24.000000000","message":"Acknowledged","commit_id":"eaefeb09bcc27bd1b81fc8497b4e5984d6bad4e2"}]}