)]}'
{"lib/cinder":[{"author":{"_account_id":866,"name":"Chmouel Boudjnah","email":"chmouel@chmouel.com","username":"chmouel"},"change_message_id":"e0c8599db6a0415f414ec7a2443d393823c05d5d","unresolved":false,"context_lines":[{"line_number":201,"context_line":"            # rootwrap.conf is no longer shipped in Cinder itself"},{"line_number":202,"context_line":"            echo \"filters_path\u003d\" | sudo tee $CINDER_CONF_DIR/rootwrap.conf \u003e /dev/null"},{"line_number":203,"context_line":"        fi"},{"line_number":204,"context_line":"        sudo sed -e \"s:^filters_path\u003d.*$:filters_path\u003d$CINDER_CONF_DIR/rootwrap.d:\" -i $CINDER_CONF_DIR/rootwrap.conf"},{"line_number":205,"context_line":"        sudo chown root:root $CINDER_CONF_DIR/rootwrap.conf"},{"line_number":206,"context_line":"        sudo chmod 0644 $CINDER_CONF_DIR/rootwrap.conf"},{"line_number":207,"context_line":"        # Specify rootwrap.conf as first parameter to rootwrap"}],"source_content_type":"application/octet-stream","patch_set":1,"id":"AAAATn%2F9J3I%3D","line":204,"updated":"2013-12-16 08:47:46.000000000","message":"nitpick: i guess if this line (204) was going up after 199 and 202 have a filters_path\u003d$CINDER_CONF_DIR/rootwrap.d: instead of the empty filters_path this could save a tiny shell call :)","commit_id":"0718568b1203bd11058d3cd28402f84841c01dda"},{"author":{"_account_id":308,"name":"Thierry Carrez","email":"thierry@openstack.org","username":"ttx"},"change_message_id":"780ceb80ed42e73e922b145d7c10de7b3d3ce589","unresolved":false,"context_lines":[{"line_number":201,"context_line":"            # rootwrap.conf is no longer shipped in Cinder itself"},{"line_number":202,"context_line":"            echo \"filters_path\u003d\" | sudo tee $CINDER_CONF_DIR/rootwrap.conf \u003e /dev/null"},{"line_number":203,"context_line":"        fi"},{"line_number":204,"context_line":"        sudo sed -e \"s:^filters_path\u003d.*$:filters_path\u003d$CINDER_CONF_DIR/rootwrap.d:\" -i $CINDER_CONF_DIR/rootwrap.conf"},{"line_number":205,"context_line":"        sudo chown root:root $CINDER_CONF_DIR/rootwrap.conf"},{"line_number":206,"context_line":"        sudo chmod 0644 $CINDER_CONF_DIR/rootwrap.conf"},{"line_number":207,"context_line":"        # Specify rootwrap.conf as first parameter to rootwrap"}],"source_content_type":"application/octet-stream","patch_set":1,"id":"AAAATn%2F9IgI%3D","line":204,"in_reply_to":"AAAATn%2F9J3I%3D","updated":"2013-12-16 14:09:15.000000000","message":"Fair point, although I\u0027d argue that it would duplicate information on what filters_path should contain (unless a variable is used, which sounds a bit overkill). Anyway, this will all be simplified when only one case will have to be supported, so not sure it\u0027s worth a second patchset unless something else is reported.","commit_id":"0718568b1203bd11058d3cd28402f84841c01dda"}]}