Activity log for bug #1703954

Date Who What changed Old value New value Message
2017-07-12 18:43:04 Gorka Eguileor bug added bug
2017-07-12 18:43:43 Gorka Eguileor bug task added cinder
2017-07-12 18:44:04 Gorka Eguileor bug task added nova
2017-07-12 18:46:40 Gorka Eguileor description OS-Brick on 1.14 and 1.15 returns real paths instead of returning symbolic links, which results in the encryption attach_volume call replacing the real device with a link to the crypt dm. The issue comes from the Nova flow when attaching an encrypted volume: 1- Attach volume 2- Generate libvirt configuration with path from step 1 3- Encrypt attach volume Since step 2 has already generated the config with the path from step 1 then step 3 must preserve this path. When step 1 returns a symbolic link we just forcefully replace it with a link to the crypt dm and everything is OK, but when we return a real path it does the same thing. If flow order was changed to be 1, 3, 2 then the encrypt attach volume could give a different path to be used for the libvirt config generation. OS-Brick on 1.14 and 1.15 returns real paths instead of returning symbolic links, which results in the encryption attach_volume call replacing the real device with a link to the crypt dm. The issue comes from the Nova flow when attaching an encrypted volume: 1- Attach volume 2- Generate libvirt configuration with path from step 1 3- Encrypt attach volume Since step 2 has already generated the config with the path from step 1 then step 3 must preserve this path. When step 1 returns a symbolic link we just forcefully replace it with a link to the crypt dm and everything is OK, but when we return a real path it does the same thing, which means we'll be replacing for example /dev/sda with a symlink, which will then break the detach process, and all future attachments. If flow order was changed to be 1, 3, 2 then the encrypt attach volume could give a different path to be used for the libvirt config generation.
2017-07-12 21:52:04 Eric Harney os-brick: importance Undecided High
2017-07-27 14:11:30 Sean Dague nova: status New Incomplete
2017-12-15 11:09:15 Gorka Eguileor os-brick: status New Fix Released
2017-12-15 11:09:28 Gorka Eguileor cinder: assignee Gorka Eguileor (gorka)
2017-12-18 11:40:54 OpenStack Infra cinder: status New In Progress
2017-12-19 01:08:14 OpenStack Infra cinder: status In Progress Fix Released
2018-01-12 17:54:25 Jonathan Mills bug added subscriber Jonathan Mills
2018-01-16 19:55:17 OpenStack Infra tags in-stable-pike