)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":34860,"name":"Amit Uniyal","email":"auniyal@redhat.com","username":"auniyal"},"change_message_id":"d870166c22a082b711e5495e53cd29bb9f533d7c","unresolved":true,"context_lines":[{"line_number":7,"context_line":"LVM: Remove checks for LVM \u003c 2.02.99"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"We already require LVM 2.02.107+, no"},{"line_number":10,"context_line":"need for these version checks now."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Change-Id: Iebfbc7a8856c1169cfa29ead3fe7ff9d43f07ad4"},{"line_number":13,"context_line":"Signed-off-by: Eric Harney \u003ceharney@redhat.com\u003e"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"089701ae_39aca271","line":10,"updated":"2026-05-20 10:57:10.000000000","message":"so if the check is removed, and lvm_type \u003d thin, now  it\u0027s always thin provisioning? .\n\non systems where the kernel lacks dm-thin-pool support (e.g minimal cloud images), what happens when we try to create thin volumes? Does LVM fail gracefully with a clear error, or does cinder-volume fail to initialize?","commit_id":"ed6605c505835d3c041dedf1a48e47379da51db3"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},"change_message_id":"c4226a2dccc5b16c8593268dcbd4c54e4f349400","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"f129c9ac_99f4b9b0","updated":"2024-04-24 17:38:29.000000000","message":"recheck\n\ngrenade failed but has no logs now","commit_id":"d1220f5e106acb2c5ddfa93ac8184c5e0f707ec1"}],"cinder/volume/drivers/lvm.py":[{"author":{"_account_id":34860,"name":"Amit Uniyal","email":"auniyal@redhat.com","username":"auniyal"},"change_message_id":"d870166c22a082b711e5495e53cd29bb9f533d7c","unresolved":true,"context_lines":[{"line_number":320,"context_line":"                if lvm_type \u003d\u003d \u0027auto\u0027:"},{"line_number":321,"context_line":"                    # \u0027auto\u0027 used to use thin when possible -- now"},{"line_number":322,"context_line":"                    # we always assume that thin support is present."},{"line_number":323,"context_line":"                    lvm_type \u003d \u0027thin\u0027"},{"line_number":324,"context_line":"                self.vg \u003d lvm.LVM("},{"line_number":325,"context_line":"                    self.configuration.volume_group,"},{"line_number":326,"context_line":"                    root_helper,"}],"source_content_type":"text/x-python","patch_set":4,"id":"7750331a_3c7209e3","line":323,"updated":"2026-05-20 10:57:10.000000000","message":"for unsupported system - this fails with \"thin-pool: Required device-mapper target(s) not detected\"","commit_id":"ed6605c505835d3c041dedf1a48e47379da51db3"}],"cinder/volume/volume_utils.py":[{"author":{"_account_id":36171,"name":"jayaanand borra","display_name":"jayaanand borra","email":"jayaanand.borra@netapp.com","username":"jayaanan","status":"netapp"},"change_message_id":"903da13117480a446c8451859f3474fd8970cb9c","unresolved":true,"context_lines":[{"line_number":666,"context_line":"            value\u003dvolume_clear)"},{"line_number":667,"context_line":""},{"line_number":668,"context_line":""},{"line_number":669,"context_line":"def supports_thin_provisioning() -\u003e bool:"},{"line_number":670,"context_line":"    return brick_lvm.LVM.supports_thin_provisioning("},{"line_number":671,"context_line":"        utils.get_root_helper())"},{"line_number":672,"context_line":""}],"source_content_type":"text/x-python","patch_set":4,"id":"d22be027_384e72d0","side":"PARENT","line":669,"updated":"2026-05-23 14:59:42.000000000","message":"supports_thin_provisioning not removed from cinder/tests/unit/brick/fake_lvm.py","commit_id":"7891328b34aa0f7cd0bc1746ca5dec91cdc60520"}]}