)]}'
{"docker/cinder/cinder-backup/Dockerfile.j2":[{"author":{"_account_id":19316,"name":"Eduardo Gonzalez","email":"dabarren@gmail.com","username":"egonzalez90"},"change_message_id":"05aa12f62a28708b47c672cc9df5c99d02371540","unresolved":false,"context_lines":[{"line_number":19,"context_line":""},{"line_number":20,"context_line":"    {% endif %}"},{"line_number":21,"context_line":""},{"line_number":22,"context_line":"{{ macros.install_packages(cinder_backup_packages | customizable(\"packages\")) }}"},{"line_number":23,"context_line":""},{"line_number":24,"context_line":"{% elif install_type \u003d\u003d \u0027source\u0027 %}"},{"line_number":25,"context_line":"    {% if base_distro in [\u0027centos\u0027, \u0027fedora\u0027, \u0027oraclelinux\u0027, \u0027rhel\u0027] %}"}],"source_content_type":"text/x-jinja2","patch_set":1,"id":"9a89bdaa_bc79c6fe","line":22,"range":{"start_line":22,"start_character":0,"end_line":22,"end_character":80},"updated":"2016-09-06 11:19:58.000000000","message":"This macro is not necessary here, you can close all if and then add macro install in line 42 for both source and binary","commit_id":"e502963dbed78eda95bf3730f2f0a530696e3b38"},{"author":{"_account_id":13671,"name":"Vladislav Belogrudov","email":"v.belogrudov@yadro.com","username":"vb"},"change_message_id":"8728e7d88fa77f6abb6d541da23cbb9639c291f4","unresolved":false,"context_lines":[{"line_number":19,"context_line":""},{"line_number":20,"context_line":"    {% endif %}"},{"line_number":21,"context_line":""},{"line_number":22,"context_line":"{{ macros.install_packages(cinder_backup_packages | customizable(\"packages\")) }}"},{"line_number":23,"context_line":""},{"line_number":24,"context_line":"{% elif install_type \u003d\u003d \u0027source\u0027 %}"},{"line_number":25,"context_line":"    {% if base_distro in [\u0027centos\u0027, \u0027fedora\u0027, \u0027oraclelinux\u0027, \u0027rhel\u0027] %}"}],"source_content_type":"text/x-jinja2","patch_set":1,"id":"9a89bdaa_97ba1d09","line":22,"range":{"start_line":22,"start_character":0,"end_line":22,"end_character":80},"in_reply_to":"9a89bdaa_bc79c6fe","updated":"2016-09-06 11:28:47.000000000","message":"this has been done so in all other Dockerfiles","commit_id":"e502963dbed78eda95bf3730f2f0a530696e3b38"},{"author":{"_account_id":19316,"name":"Eduardo Gonzalez","email":"dabarren@gmail.com","username":"egonzalez90"},"change_message_id":"05aa12f62a28708b47c672cc9df5c99d02371540","unresolved":false,"context_lines":[{"line_number":36,"context_line":""},{"line_number":37,"context_line":"    {% endif %}"},{"line_number":38,"context_line":""},{"line_number":39,"context_line":"{{ macros.install_packages(cinder_backup_packages | customizable(\"packages\")) }}"},{"line_number":40,"context_line":""},{"line_number":41,"context_line":"{% endif %}"},{"line_number":42,"context_line":""}],"source_content_type":"text/x-jinja2","patch_set":1,"id":"9a89bdaa_fc834eca","line":39,"range":{"start_line":39,"start_character":0,"end_line":39,"end_character":80},"updated":"2016-09-06 11:19:58.000000000","message":"Ditto","commit_id":"e502963dbed78eda95bf3730f2f0a530696e3b38"},{"author":{"_account_id":1390,"name":"Paul Bourke","email":"pauldbourke@proton.me","username":"pauldbourke"},"change_message_id":"56121f0b389363dabf093444889c30433a92031b","unresolved":false,"context_lines":[{"line_number":12,"context_line":""},{"line_number":13,"context_line":"    {% elif base_distro in [\u0027ubuntu\u0027] %}"},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"        {% set cinder_base_packages \u003d ["},{"line_number":16,"context_line":"                \u0027nfs-common\u0027,"},{"line_number":17,"context_line":"                \u0027cinder-backup\u0027"},{"line_number":18,"context_line":"        ] %}"}],"source_content_type":"text/x-jinja2","patch_set":2,"id":"9a89bdaa_3e247cc3","line":15,"range":{"start_line":15,"start_character":15,"end_line":15,"end_character":35},"updated":"2016-09-07 14:56:01.000000000","message":"This should be cinder_backup_packages","commit_id":"b41820d291223119b86129896750b5fd97676a89"},{"author":{"_account_id":1390,"name":"Paul Bourke","email":"pauldbourke@proton.me","username":"pauldbourke"},"change_message_id":"56121f0b389363dabf093444889c30433a92031b","unresolved":false,"context_lines":[{"line_number":19,"context_line":""},{"line_number":20,"context_line":"    {% endif %}"},{"line_number":21,"context_line":""},{"line_number":22,"context_line":"{{ macros.install_packages(cinder_backup_packages | customizable(\"packages\")) }}"},{"line_number":23,"context_line":""},{"line_number":24,"context_line":"{% elif install_type \u003d\u003d \u0027source\u0027 %}"},{"line_number":25,"context_line":"    {% if base_distro in [\u0027centos\u0027, \u0027fedora\u0027, \u0027oraclelinux\u0027, \u0027rhel\u0027] %}"}],"source_content_type":"text/x-jinja2","patch_set":2,"id":"9a89bdaa_1e0ed83a","line":22,"updated":"2016-09-07 14:56:01.000000000","message":"This line can be removed I think","commit_id":"b41820d291223119b86129896750b5fd97676a89"}]}