)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"change_message_id":"9fe8cc8a9735914bee7be30dca2f1a63ec07dc5c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"fdfa0213_c307e175","updated":"2022-05-03 06:45:24.000000000","message":"I\u0027m sure Yolanda wants to forget all about this 😊  I don\u0027t see in the logs any particular reason for this other than this being a logical small shim on the LVM tools.  Per the arguments in the change, it probably makes sense to standardise it, even though it moves it away from the LVM behaviour.","commit_id":"9cd3b03c8f7bdfb40c0d9dc9d31bed55718c7a4a"},{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"2cf13105573c5597fd5a83f5edd06076f1387b40","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"1694757d_10d1574c","updated":"2022-07-20 18:23:30.000000000","message":"Aside from the failures which unrelated, this looks good to me.","commit_id":"d090126c66c1df3e8f9b507b24db94a03e2fec7c"},{"author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"change_message_id":"e2af4fec11bc918b71eafa0b2c0887dee7795be2","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"118244e0_a532fa72","updated":"2022-07-25 00:52:40.000000000","message":"recheck","commit_id":"d090126c66c1df3e8f9b507b24db94a03e2fec7c"},{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"c97fb08e7de0364542e5cb40a5b20b61f141ac0c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"cc0942ce_d9d93267","updated":"2022-07-20 18:22:31.000000000","message":"recheck unrelated opensuse and fedora failures","commit_id":"d090126c66c1df3e8f9b507b24db94a03e2fec7c"}],"diskimage_builder/block_device/level1/lvm.py":[{"author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"change_message_id":"9fe8cc8a9735914bee7be30dca2f1a63ec07dc5c","unresolved":true,"context_lines":[{"line_number":182,"context_line":"        :param state: global state pointer"},{"line_number":183,"context_line":"        :param base: the parent volume group"},{"line_number":184,"context_line":"        :param options: options passed to lvcreate"},{"line_number":185,"context_line":"        :param size: size of the LV, in MB (this or extents must be provided)"},{"line_number":186,"context_line":"        :param extents: size of the LV in extents"},{"line_number":187,"context_line":"        \"\"\""},{"line_number":188,"context_line":"        super(LvsNode, self).__init__(name, state)"}],"source_content_type":"text/x-python","patch_set":4,"id":"2018cecc_0329903b","line":185,"range":{"start_line":185,"start_character":40,"end_line":185,"end_character":42},"updated":"2022-05-03 06:45:24.000000000","message":"I guess this is now an arbitrary input","commit_id":"9cd3b03c8f7bdfb40c0d9dc9d31bed55718c7a4a"},{"author":{"_account_id":4571,"name":"Steve Baker","email":"sbaker@redhat.com","username":"steve-stevebaker"},"change_message_id":"af8f1f9857937463cb81b5fd265d5ee1691992ee","unresolved":false,"context_lines":[{"line_number":182,"context_line":"        :param state: global state pointer"},{"line_number":183,"context_line":"        :param base: the parent volume group"},{"line_number":184,"context_line":"        :param options: options passed to lvcreate"},{"line_number":185,"context_line":"        :param size: size of the LV, in MB (this or extents must be provided)"},{"line_number":186,"context_line":"        :param extents: size of the LV in extents"},{"line_number":187,"context_line":"        \"\"\""},{"line_number":188,"context_line":"        super(LvsNode, self).__init__(name, state)"}],"source_content_type":"text/x-python","patch_set":4,"id":"e8dbd471_5a9549cc","line":185,"range":{"start_line":185,"start_character":40,"end_line":185,"end_character":42},"in_reply_to":"2018cecc_0329903b","updated":"2022-06-27 22:25:51.000000000","message":"Done","commit_id":"9cd3b03c8f7bdfb40c0d9dc9d31bed55718c7a4a"}],"diskimage_builder/block_device/tests/test_lvm.py":[{"author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"change_message_id":"9fe8cc8a9735914bee7be30dca2f1a63ec07dc5c","unresolved":true,"context_lines":[{"line_number":104,"context_line":"            mock.call([\u0027vgcreate\u0027, \u0027vg\u0027,"},{"line_number":105,"context_line":"                       \u0027/dev/fake/root\u0027, \u0027/dev/fake/data\u0027, \u0027--force\u0027]),"},{"line_number":106,"context_line":"            # create a bunch of lv\u0027s on vg"},{"line_number":107,"context_line":"            mock.call([\u0027lvcreate\u0027, \u0027--name\u0027, \u0027lv_root\u0027, \u0027-L\u0027, \u00271799356416B\u0027,"},{"line_number":108,"context_line":"                       \u0027vg\u0027]),"},{"line_number":109,"context_line":"            mock.call([\u0027lvcreate\u0027, \u0027--name\u0027, \u0027lv_tmp\u0027, \u0027-L\u0027, \u002796468992B\u0027,"},{"line_number":110,"context_line":"                       \u0027vg\u0027]),"}],"source_content_type":"text/x-python","patch_set":4,"id":"6bc1a31e_c258a45a","line":107,"updated":"2022-05-03 06:45:24.000000000","message":"One thought is to calculate these with the modulo of the extent size rather than hardcode it.  Though it is unlikely to change, i guess","commit_id":"9cd3b03c8f7bdfb40c0d9dc9d31bed55718c7a4a"},{"author":{"_account_id":4571,"name":"Steve Baker","email":"sbaker@redhat.com","username":"steve-stevebaker"},"change_message_id":"6f126500974bbc5c70a8ad58c40a01729e37f426","unresolved":false,"context_lines":[{"line_number":104,"context_line":"            mock.call([\u0027vgcreate\u0027, \u0027vg\u0027,"},{"line_number":105,"context_line":"                       \u0027/dev/fake/root\u0027, \u0027/dev/fake/data\u0027, \u0027--force\u0027]),"},{"line_number":106,"context_line":"            # create a bunch of lv\u0027s on vg"},{"line_number":107,"context_line":"            mock.call([\u0027lvcreate\u0027, \u0027--name\u0027, \u0027lv_root\u0027, \u0027-L\u0027, \u00271799356416B\u0027,"},{"line_number":108,"context_line":"                       \u0027vg\u0027]),"},{"line_number":109,"context_line":"            mock.call([\u0027lvcreate\u0027, \u0027--name\u0027, \u0027lv_tmp\u0027, \u0027-L\u0027, \u002796468992B\u0027,"},{"line_number":110,"context_line":"                       \u0027vg\u0027]),"}],"source_content_type":"text/x-python","patch_set":4,"id":"5dfeb69e_0b787ad9","line":107,"in_reply_to":"247aab9c_f4654830","updated":"2022-06-27 22:53:03.000000000","message":"Done","commit_id":"9cd3b03c8f7bdfb40c0d9dc9d31bed55718c7a4a"},{"author":{"_account_id":4571,"name":"Steve Baker","email":"sbaker@redhat.com","username":"steve-stevebaker"},"change_message_id":"af8f1f9857937463cb81b5fd265d5ee1691992ee","unresolved":true,"context_lines":[{"line_number":104,"context_line":"            mock.call([\u0027vgcreate\u0027, \u0027vg\u0027,"},{"line_number":105,"context_line":"                       \u0027/dev/fake/root\u0027, \u0027/dev/fake/data\u0027, \u0027--force\u0027]),"},{"line_number":106,"context_line":"            # create a bunch of lv\u0027s on vg"},{"line_number":107,"context_line":"            mock.call([\u0027lvcreate\u0027, \u0027--name\u0027, \u0027lv_root\u0027, \u0027-L\u0027, \u00271799356416B\u0027,"},{"line_number":108,"context_line":"                       \u0027vg\u0027]),"},{"line_number":109,"context_line":"            mock.call([\u0027lvcreate\u0027, \u0027--name\u0027, \u0027lv_tmp\u0027, \u0027-L\u0027, \u002796468992B\u0027,"},{"line_number":110,"context_line":"                       \u0027vg\u0027]),"}],"source_content_type":"text/x-python","patch_set":4,"id":"247aab9c_f4654830","line":107,"in_reply_to":"6bc1a31e_c258a45a","updated":"2022-06-27 22:25:51.000000000","message":"We define the constant PHYSICAL_EXTENT_BYTES so these tests can be changed if that ever changes","commit_id":"9cd3b03c8f7bdfb40c0d9dc9d31bed55718c7a4a"}]}