)]}'
{"docker/sahara/sahara-base/Dockerfile.j2":[{"author":{"_account_id":19316,"name":"Eduardo Gonzalez","email":"dabarren@gmail.com","username":"egonzalez90"},"change_message_id":"5a879f47e8a195e8c9b151f0d252c3a0e49ccbdc","unresolved":false,"context_lines":[{"line_number":10,"context_line":"{% if install_type \u003d\u003d \u0027binary\u0027 %}"},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"{% if base_package_type \u003d\u003d \u0027rpm\u0027 %}"},{"line_number":13,"context_line":"    {% set sahara_base_packages \u003d [\u0027openstack-sahara-common\u0027] %}"},{"line_number":14,"context_line":"    {% if distro_python_version.startswith(\u00273\u0027) %}"},{"line_number":15,"context_line":"    {% set sahara_base_plugins_packages \u003d ["},{"line_number":16,"context_line":"         \u0027python3-sahara-plugin-ambari\u0027,"},{"line_number":17,"context_line":"         \u0027python3-sahara-plugin-cdh\u0027,"},{"line_number":18,"context_line":"         \u0027python3-sahara-plugin-mapr\u0027,"},{"line_number":19,"context_line":"         \u0027python3-sahara-plugin-spark\u0027,"},{"line_number":20,"context_line":"         \u0027python3-sahara-plugin-storm\u0027,"},{"line_number":21,"context_line":"         \u0027python3-sahara-plugin-vanilla\u0027"},{"line_number":22,"context_line":"     ] %}"},{"line_number":23,"context_line":"    {% else %}"},{"line_number":24,"context_line":"    {% set sahara_base_plugins_packages \u003d ["},{"line_number":25,"context_line":"         \u0027python2-sahara-plugin-ambari\u0027,"},{"line_number":26,"context_line":"         \u0027python2-sahara-plugin-cdh\u0027,"},{"line_number":27,"context_line":"         \u0027python2-sahara-plugin-mapr\u0027,"},{"line_number":28,"context_line":"         \u0027python2-sahara-plugin-spark\u0027,"},{"line_number":29,"context_line":"         \u0027python2-sahara-plugin-storm\u0027,"},{"line_number":30,"context_line":"         \u0027python2-sahara-plugin-vanilla\u0027"},{"line_number":31,"context_line":"     ] %}"},{"line_number":32,"context_line":"    {% endif %}"},{"line_number":33,"context_line":"{% elif base_package_type \u003d\u003d \u0027deb\u0027 %}"},{"line_number":34,"context_line":"    {% set sahara_base_packages \u003d [\u0027sahara-common\u0027] %}"},{"line_number":35,"context_line":"{% endif %}"}],"source_content_type":"text/x-jinja2","patch_set":2,"id":"5fc1f717_bbc3aeec","line":32,"range":{"start_line":13,"start_character":0,"end_line":32,"end_character":15},"updated":"2019-03-18 10:01:04.000000000","message":"why not append those plugins to the main sahara_base_pacakges variable, wont create 2 image layers merging them in a single RUN statement\n\n{% set sahara_base_packages \u003d sahara_base_packages + [\n\u0027python3-sahara-plugin-ambari\u0027,\n         \u0027python3-sahara-plugin-cdh\u0027,\n         \u0027python3-sahara-plugin-mapr\u0027,\n         \u0027python3-sahara-plugin-spark\u0027,\n         \u0027python3-sahara-plugin-storm\u0027,\n         \u0027python3-sahara-plugin-vanilla\u0027\n     ] %}","commit_id":"cd2211a868939c303669c14d3352057ce744bfe1"},{"author":{"_account_id":10459,"name":"Luigi Toscano","email":"ltoscano@redhat.com","username":"ltoscano"},"change_message_id":"3390014819e8d240a91fade79c3d078706c33003","unresolved":false,"context_lines":[{"line_number":10,"context_line":"{% if install_type \u003d\u003d \u0027binary\u0027 %}"},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"{% if base_package_type \u003d\u003d \u0027rpm\u0027 %}"},{"line_number":13,"context_line":"    {% set sahara_base_packages \u003d [\u0027openstack-sahara-common\u0027] %}"},{"line_number":14,"context_line":"    {% if distro_python_version.startswith(\u00273\u0027) %}"},{"line_number":15,"context_line":"    {% set sahara_base_plugins_packages \u003d ["},{"line_number":16,"context_line":"         \u0027python3-sahara-plugin-ambari\u0027,"},{"line_number":17,"context_line":"         \u0027python3-sahara-plugin-cdh\u0027,"},{"line_number":18,"context_line":"         \u0027python3-sahara-plugin-mapr\u0027,"},{"line_number":19,"context_line":"         \u0027python3-sahara-plugin-spark\u0027,"},{"line_number":20,"context_line":"         \u0027python3-sahara-plugin-storm\u0027,"},{"line_number":21,"context_line":"         \u0027python3-sahara-plugin-vanilla\u0027"},{"line_number":22,"context_line":"     ] %}"},{"line_number":23,"context_line":"    {% else %}"},{"line_number":24,"context_line":"    {% set sahara_base_plugins_packages \u003d ["},{"line_number":25,"context_line":"         \u0027python2-sahara-plugin-ambari\u0027,"},{"line_number":26,"context_line":"         \u0027python2-sahara-plugin-cdh\u0027,"},{"line_number":27,"context_line":"         \u0027python2-sahara-plugin-mapr\u0027,"},{"line_number":28,"context_line":"         \u0027python2-sahara-plugin-spark\u0027,"},{"line_number":29,"context_line":"         \u0027python2-sahara-plugin-storm\u0027,"},{"line_number":30,"context_line":"         \u0027python2-sahara-plugin-vanilla\u0027"},{"line_number":31,"context_line":"     ] %}"},{"line_number":32,"context_line":"    {% endif %}"},{"line_number":33,"context_line":"{% elif base_package_type \u003d\u003d \u0027deb\u0027 %}"},{"line_number":34,"context_line":"    {% set sahara_base_packages \u003d [\u0027sahara-common\u0027] %}"},{"line_number":35,"context_line":"{% endif %}"}],"source_content_type":"text/x-jinja2","patch_set":2,"id":"5fc1f717_fb529634","line":32,"range":{"start_line":13,"start_character":0,"end_line":32,"end_character":15},"in_reply_to":"5fc1f717_bbc3aeec","updated":"2019-03-18 10:09:54.000000000","message":"I thought that it would have made it easier to override the list of packages separately, following the same pattern of the \"source\" section, which uses the plugin-archive trick.\n\nBut now I notice that other container definitions also use the same _base_packages variable for plugins, not a separate plugin_packages variable.\n\nI\u0027m going to change it.","commit_id":"cd2211a868939c303669c14d3352057ce744bfe1"},{"author":{"_account_id":14826,"name":"Mark Goddard","email":"markgoddard86@gmail.com","username":"mgoddard"},"change_message_id":"1b858f81ede5a29adab382cf7471e22aa5e233a5","unresolved":false,"context_lines":[{"line_number":31,"context_line":"     ] %}"},{"line_number":32,"context_line":"    {% endif %}"},{"line_number":33,"context_line":"{% elif base_package_type \u003d\u003d \u0027deb\u0027 %}"},{"line_number":34,"context_line":"    {% set sahara_base_packages \u003d [\u0027sahara-common\u0027] %}"},{"line_number":35,"context_line":"{% endif %}"},{"line_number":36,"context_line":"{{ macros.install_packages(sahara_base_packages | customizable(\"packages\")) }}"},{"line_number":37,"context_line":"{{ macros.install_packages(sahara_base_plugins_packages | customizable(\"plugins_packages\")) }}"}],"source_content_type":"text/x-jinja2","patch_set":2,"id":"5fc1f717_16a659ac","line":34,"updated":"2019-03-18 10:40:23.000000000","message":"Are plugin packages available for debian/ubuntu?","commit_id":"cd2211a868939c303669c14d3352057ce744bfe1"},{"author":{"_account_id":10459,"name":"Luigi Toscano","email":"ltoscano@redhat.com","username":"ltoscano"},"change_message_id":"bf85cfdb2f31a8d0380ff2a4792338d3a62efcb2","unresolved":false,"context_lines":[{"line_number":31,"context_line":"     ] %}"},{"line_number":32,"context_line":"    {% endif %}"},{"line_number":33,"context_line":"{% elif base_package_type \u003d\u003d \u0027deb\u0027 %}"},{"line_number":34,"context_line":"    {% set sahara_base_packages \u003d [\u0027sahara-common\u0027] %}"},{"line_number":35,"context_line":"{% endif %}"},{"line_number":36,"context_line":"{{ macros.install_packages(sahara_base_packages | customizable(\"packages\")) }}"},{"line_number":37,"context_line":"{{ macros.install_packages(sahara_base_plugins_packages | customizable(\"plugins_packages\")) }}"}],"source_content_type":"text/x-jinja2","patch_set":2,"id":"5fc1f717_7681ddc2","line":34,"in_reply_to":"5fc1f717_16a659ac","updated":"2019-03-18 23:56:54.000000000","message":"As far as I know, the packages are not available right now.\n\nI filed this https://bugs.launchpad.net/ubuntu/+source/sahara/+bug/1820334 for ubuntu and we (sahara) sent an email long time ago to packagers to inform them about the upcoming split.","commit_id":"cd2211a868939c303669c14d3352057ce744bfe1"}]}
