)]}'
{"docker/cinder/cinder-volume/Dockerfile.j2":[{"author":{"_account_id":16282,"name":"YaZug","email":"jschluet@redhat.com","username":"yazug"},"change_message_id":"b00e727921535387669e18fbffa7ca10b367b74c","unresolved":false,"context_lines":[{"line_number":23,"context_line":"    ] %}"},{"line_number":24,"context_line":"{{ macros.install_packages(cinder_volume_packages | customizable(\"packages\")) }}"},{"line_number":25,"context_line":""},{"line_number":26,"context_line":"{% block cinder_volume_ubuntu_setup %}"},{"line_number":27,"context_line":"RUN sed -i \u00271 i include /var/lib/cinder/volumes/*\u0027 /etc/tgt/targets.conf"},{"line_number":28,"context_line":"{% endblock %}"},{"line_number":29,"context_line":""}],"source_content_type":"text/x-jinja2","patch_set":1,"id":"7f231b9d_89bfe2a8","side":"PARENT","line":26,"updated":"2017-06-16 15:14:20.000000000","message":"someone mentioned this is still in ubuntu so this may be too broad sweeping","commit_id":"49e45698da71cbea87ede2f2f053fa1e8e9e2817"},{"author":{"_account_id":4162,"name":"Paul Belanger","email":"pabelanger@redhat.com","username":"pabelanger"},"change_message_id":"ff9726046ec7c8de758bab380fbda92d82cfa5ba","unresolved":false,"context_lines":[{"line_number":8,"context_line":"{% if base_distro in [\u0027centos\u0027, \u0027oraclelinux\u0027, \u0027rhel\u0027] %}"},{"line_number":9,"context_line":"    {% set cinder_volume_packages \u003d ["},{"line_number":10,"context_line":"            \u0027nfs-utils\u0027,"},{"line_number":11,"context_line":"            \u0027scsi-target-utils\u0027"},{"line_number":12,"context_line":"    ] %}"},{"line_number":13,"context_line":"{{ macros.install_packages(cinder_volume_packages | customizable(\"packages\")) }}"},{"line_number":14,"context_line":""}],"source_content_type":"text/x-jinja2","patch_set":2,"id":"7f231b9d_5ff7398c","side":"PARENT","line":11,"updated":"2017-06-16 17:54:51.000000000","message":"FWIW: scsi-target-utils is a dependency for devstack today, on centos. Maybe just update the patch to remove it just for rhel.\n\nOr, if scsi-target-utils isn\u0027t even used, maybe drop it directly from http://git.openstack.org/cgit/openstack-dev/devstack/tree/lib/cinder#n435","commit_id":"49e45698da71cbea87ede2f2f053fa1e8e9e2817"},{"author":{"_account_id":16282,"name":"YaZug","email":"jschluet@redhat.com","username":"yazug"},"change_message_id":"5003ec9051da7e59d087816708dfb78390f814de","unresolved":false,"context_lines":[{"line_number":8,"context_line":"{% if base_distro in [\u0027centos\u0027, \u0027oraclelinux\u0027, \u0027rhel\u0027] %}"},{"line_number":9,"context_line":"    {% set cinder_volume_packages \u003d ["},{"line_number":10,"context_line":"            \u0027nfs-utils\u0027,"},{"line_number":11,"context_line":"            \u0027scsi-target-utils\u0027"},{"line_number":12,"context_line":"    ] %}"},{"line_number":13,"context_line":"{{ macros.install_packages(cinder_volume_packages | customizable(\"packages\")) }}"},{"line_number":14,"context_line":""}],"source_content_type":"text/x-jinja2","patch_set":2,"id":"7f231b9d_7c9f7165","side":"PARENT","line":11,"in_reply_to":"7f231b9d_5ff7398c","updated":"2017-06-19 19:09:48.000000000","message":"Should I consider splitting this out into RHEL section only for nfs-utils and exclude the sed for rhel only? or coordinate with the cinder devstack to conditional things to not use scsi-target-utils for centos rhel","commit_id":"49e45698da71cbea87ede2f2f053fa1e8e9e2817"}]}