)]}'
{"etc/cinder/rootwrap.d/volume.filters":[{"author":{"_account_id":11600,"name":"Michał Dulko","email":"michal.dulko@gmail.com","username":"dulek"},"change_message_id":"888686ffe5cb6c6dd9dcb1ed1c4bebd50dd56919","unresolved":false,"context_lines":[{"line_number":39,"context_line":"# cinder/brick/local_dev/lvm.py: \u0027lvcreate\u0027, \u0027-L\u0027, sizestr, \u0027-n\u0027, volume_name,.."},{"line_number":40,"context_line":"# cinder/brick/local_dev/lvm.py: \u0027lvcreate\u0027, \u0027-L\u0027, ..."},{"line_number":41,"context_line":"lvcreate: EnvFilter, env, root, LC_ALL\u003dC, lvcreate"},{"line_number":42,"context_line":"lvcreate_lvmconf: EnvFilter, env, root, LVM_SYSTEM_DIR\u003d, LC_ALL\u003dC, lvcreate"},{"line_number":43,"context_line":""},{"line_number":44,"context_line":"# cinder/volume/driver.py: \u0027dd\u0027, \u0027if\u003d%s\u0027 % srcstr, \u0027of\u003d%s\u0027 % deststr,..."},{"line_number":45,"context_line":"dd: CommandFilter, dd, root"}],"source_content_type":"application/octet-stream","patch_set":1,"id":"fa1b9901_4ce0a6fa","line":42,"updated":"2015-08-25 12:25:24.000000000","message":"I\u0027m not sure here so I\u0027ll better to point it out myself - I\u0027ve followed the convention from lvdisplay and others - just added lvm.LVM.LVM_CMD_PREFIX to the commands. It\u0027s sometimes just LC_ALL but if you have lvm_conf set, then it adds LVM_SYSTEM_DIR pointing to it. It is fine to make it included here? Or maybe all lv*, vg*, pv* commands should have it included?\n\nCan anyone more experienced help on that?","commit_id":"1156db86bcbd1873e6848ac97805b35ff72a97bf"}]}
