)]}'
{"docker/macros.j2":[{"author":{"_account_id":19316,"name":"Eduardo Gonzalez","email":"dabarren@gmail.com","username":"egonzalez90"},"change_message_id":"9788cecbcb046a0cba1c356d49ef420ccf5bd75b","unresolved":false,"context_lines":[{"line_number":60,"context_line":"{% macro debian_mongodb_existing_user_fix() %}"},{"line_number":61,"context_line":"{# NOTE(hrw): The postinst script breaks if the user \u0027mongodb\u0027 already exists #}"},{"line_number":62,"context_line":"{# NOTE(hrw): Version in Debian \u0027testing\u0027 checks does user exists #}"},{"line_number":63,"context_line":"RUN apt-get -y install --no-install-recommends mongodb-server \\"},{"line_number":64,"context_line":"    || sed -i \u0027/adduser --system/,+2 d\u0027 /var/lib/dpkg/info/mongodb-server.postinst \\"},{"line_number":65,"context_line":"    \u0026\u0026 dpkg --configure -a \\"},{"line_number":66,"context_line":"    \u0026\u0026 apt-get clean"}],"source_content_type":"text/x-jinja2","patch_set":1,"id":"1f485f77_aae094fb","line":63,"updated":"2017-11-15 07:07:49.000000000","message":"missing apt-get update","commit_id":"666f0f420dbb981554ea7dbba44b40e2b4027f4d"},{"author":{"_account_id":24072,"name":"Marcin Juszkiewicz","email":"mjuszkiewicz@redhat.com","username":"hrw"},"change_message_id":"37e354e49ffc948861a1b5bbec5c449ea842898e","unresolved":false,"context_lines":[{"line_number":60,"context_line":"{% macro debian_mongodb_existing_user_fix() %}"},{"line_number":61,"context_line":"{# NOTE(hrw): The postinst script breaks if the user \u0027mongodb\u0027 already exists #}"},{"line_number":62,"context_line":"{# NOTE(hrw): Version in Debian \u0027testing\u0027 checks does user exists #}"},{"line_number":63,"context_line":"RUN apt-get -y install --no-install-recommends mongodb-server \\"},{"line_number":64,"context_line":"    || sed -i \u0027/adduser --system/,+2 d\u0027 /var/lib/dpkg/info/mongodb-server.postinst \\"},{"line_number":65,"context_line":"    \u0026\u0026 dpkg --configure -a \\"},{"line_number":66,"context_line":"    \u0026\u0026 apt-get clean"}],"source_content_type":"text/x-jinja2","patch_set":1,"id":"1f485f77_cf63dab0","line":63,"in_reply_to":"1f485f77_aae094fb","updated":"2017-11-15 11:55:42.000000000","message":"Done","commit_id":"666f0f420dbb981554ea7dbba44b40e2b4027f4d"},{"author":{"_account_id":19316,"name":"Eduardo Gonzalez","email":"dabarren@gmail.com","username":"egonzalez90"},"change_message_id":"9788cecbcb046a0cba1c356d49ef420ccf5bd75b","unresolved":false,"context_lines":[{"line_number":60,"context_line":"{% macro debian_mongodb_existing_user_fix() %}"},{"line_number":61,"context_line":"{# NOTE(hrw): The postinst script breaks if the user \u0027mongodb\u0027 already exists #}"},{"line_number":62,"context_line":"{# NOTE(hrw): Version in Debian \u0027testing\u0027 checks does user exists #}"},{"line_number":63,"context_line":"RUN apt-get -y install --no-install-recommends mongodb-server \\"},{"line_number":64,"context_line":"    || sed -i \u0027/adduser --system/,+2 d\u0027 /var/lib/dpkg/info/mongodb-server.postinst \\"},{"line_number":65,"context_line":"    \u0026\u0026 dpkg --configure -a \\"},{"line_number":66,"context_line":"    \u0026\u0026 apt-get clean"},{"line_number":67,"context_line":"{% endmacro %}"},{"line_number":68,"context_line":""},{"line_number":69,"context_line":"{% macro debian_opendaylight_existing_user_fix() %}"},{"line_number":70,"context_line":"{# NOTE(egonzalez): The postinst script breaks if the user \u0027odl\u0027 already exists #}"}],"source_content_type":"text/x-jinja2","patch_set":1,"id":"1f485f77_aab9b4e9","line":67,"range":{"start_line":63,"start_character":0,"end_line":67,"end_character":14},"updated":"2017-11-15 07:07:49.000000000","message":"does not work the same macro as haproxy and odl?, why a dpkg configure rather than apt-get install -f","commit_id":"666f0f420dbb981554ea7dbba44b40e2b4027f4d"},{"author":{"_account_id":24072,"name":"Marcin Juszkiewicz","email":"mjuszkiewicz@redhat.com","username":"hrw"},"change_message_id":"37e354e49ffc948861a1b5bbec5c449ea842898e","unresolved":false,"context_lines":[{"line_number":60,"context_line":"{% macro debian_mongodb_existing_user_fix() %}"},{"line_number":61,"context_line":"{# NOTE(hrw): The postinst script breaks if the user \u0027mongodb\u0027 already exists #}"},{"line_number":62,"context_line":"{# NOTE(hrw): Version in Debian \u0027testing\u0027 checks does user exists #}"},{"line_number":63,"context_line":"RUN apt-get -y install --no-install-recommends mongodb-server \\"},{"line_number":64,"context_line":"    || sed -i \u0027/adduser --system/,+2 d\u0027 /var/lib/dpkg/info/mongodb-server.postinst \\"},{"line_number":65,"context_line":"    \u0026\u0026 dpkg --configure -a \\"},{"line_number":66,"context_line":"    \u0026\u0026 apt-get clean"},{"line_number":67,"context_line":"{% endmacro %}"},{"line_number":68,"context_line":""},{"line_number":69,"context_line":"{% macro debian_opendaylight_existing_user_fix() %}"},{"line_number":70,"context_line":"{# NOTE(egonzalez): The postinst script breaks if the user \u0027odl\u0027 already exists #}"}],"source_content_type":"text/x-jinja2","patch_set":1,"id":"1f485f77_8fea82a9","line":67,"range":{"start_line":63,"start_character":0,"end_line":67,"end_character":14},"in_reply_to":"1f485f77_aab9b4e9","updated":"2017-11-15 11:55:42.000000000","message":"\u0027apt-get install -f\u0027 ended with extra warnings which \u0027dpkg --configure -a\u0027 does not generate.\n\nAnd at that moment the only thing which needs to be done is (re)configuration of package so dpkg is enough.","commit_id":"666f0f420dbb981554ea7dbba44b40e2b4027f4d"},{"author":{"_account_id":167,"name":"Christian Berendt","email":"berendt@osism.tech","username":"berendt"},"change_message_id":"d5112cb9661221a20222d30f41ee59726871ec8b","unresolved":false,"context_lines":[{"line_number":63,"context_line":"RUN apt-get update \\"},{"line_number":64,"context_line":"    \u0026\u0026 apt-get -y install --no-install-recommends mongodb-server \\"},{"line_number":65,"context_line":"    || sed -i \u0027/adduser --system/,+2 d\u0027 /var/lib/dpkg/info/mongodb-server.postinst \\"},{"line_number":66,"context_line":"    \u0026\u0026 dpkg --configure -a \\"},{"line_number":67,"context_line":"    \u0026\u0026 apt-get clean"},{"line_number":68,"context_line":"{% endmacro %}"},{"line_number":69,"context_line":""}],"source_content_type":"text/x-jinja2","patch_set":2,"id":"1f485f77_0f55721d","line":66,"updated":"2017-11-15 12:04:19.000000000","message":"For haproxy we used apt-get -y install -f. Is there a difference?","commit_id":"be90e088944762a353e3660bf90e5818bdc42908"},{"author":{"_account_id":167,"name":"Christian Berendt","email":"berendt@osism.tech","username":"berendt"},"change_message_id":"d5112cb9661221a20222d30f41ee59726871ec8b","unresolved":false,"context_lines":[{"line_number":65,"context_line":"    || sed -i \u0027/adduser --system/,+2 d\u0027 /var/lib/dpkg/info/mongodb-server.postinst \\"},{"line_number":66,"context_line":"    \u0026\u0026 dpkg --configure -a \\"},{"line_number":67,"context_line":"    \u0026\u0026 apt-get clean"},{"line_number":68,"context_line":"{% endmacro %}"},{"line_number":69,"context_line":""},{"line_number":70,"context_line":"{% macro debian_opendaylight_existing_user_fix() %}"},{"line_number":71,"context_line":"{# NOTE(egonzalez): The postinst script breaks if the user \u0027odl\u0027 already exists #}"}],"source_content_type":"text/x-jinja2","patch_set":2,"id":"1f485f77_6f606602","line":68,"updated":"2017-11-15 12:04:19.000000000","message":"Please add     \u0026\u0026 rm -rf /var/lib/apt/lists/*","commit_id":"be90e088944762a353e3660bf90e5818bdc42908"},{"author":{"_account_id":1390,"name":"Paul Bourke","email":"pauldbourke@proton.me","username":"pauldbourke"},"change_message_id":"21189c472d47a7c779ab7a2a9b2a6f1e284db8a3","unresolved":false,"context_lines":[{"line_number":65,"context_line":""},{"line_number":66,"context_line":"{% macro debian_mongodb_existing_user_fix() %}"},{"line_number":67,"context_line":"{# NOTE(hrw): The postinst script breaks if the user \u0027mongodb\u0027 already exists #}"},{"line_number":68,"context_line":"{# NOTE(hrw): Version in Debian \u0027testing\u0027 checks does user exists #}"},{"line_number":69,"context_line":"    {{ debian_fix_existing_user(\u0027mongodb-server\u0027, 3) }}"},{"line_number":70,"context_line":"{% endmacro %}"},{"line_number":71,"context_line":""}],"source_content_type":"text/x-jinja2","patch_set":3,"id":"df87a7cf_f1460622","line":68,"range":{"start_line":68,"start_character":0,"end_line":68,"end_character":68},"updated":"2017-12-14 11:16:52.000000000","message":"Can you replace this with a TODO so it\u0027s clear this can be removed once the testing package is promoted to stable?","commit_id":"9f637dee4f6afcdb4fae50633143cbd952c6b3fa"},{"author":{"_account_id":24072,"name":"Marcin Juszkiewicz","email":"mjuszkiewicz@redhat.com","username":"hrw"},"change_message_id":"95b8c41488a797104a51b878cf01046ebabac537","unresolved":false,"context_lines":[{"line_number":65,"context_line":""},{"line_number":66,"context_line":"{% macro debian_mongodb_existing_user_fix() %}"},{"line_number":67,"context_line":"{# NOTE(hrw): The postinst script breaks if the user \u0027mongodb\u0027 already exists #}"},{"line_number":68,"context_line":"{# NOTE(hrw): Version in Debian \u0027testing\u0027 checks does user exists #}"},{"line_number":69,"context_line":"    {{ debian_fix_existing_user(\u0027mongodb-server\u0027, 3) }}"},{"line_number":70,"context_line":"{% endmacro %}"},{"line_number":71,"context_line":""}],"source_content_type":"text/x-jinja2","patch_set":3,"id":"df87a7cf_91ef7ae8","line":68,"range":{"start_line":68,"start_character":0,"end_line":68,"end_character":68},"in_reply_to":"df87a7cf_f1460622","updated":"2017-12-14 11:25:41.000000000","message":"Done","commit_id":"9f637dee4f6afcdb4fae50633143cbd952c6b3fa"}],"docker/mongodb/Dockerfile.j2":[{"author":{"_account_id":19316,"name":"Eduardo Gonzalez","email":"dabarren@gmail.com","username":"egonzalez90"},"change_message_id":"9788cecbcb046a0cba1c356d49ef420ccf5bd75b","unresolved":false,"context_lines":[{"line_number":21,"context_line":"{% if base_distro in [\u0027debian\u0027] %}"},{"line_number":22,"context_line":"    {# NOTE(hrw) Ubuntu does not need that #}"},{"line_number":23,"context_line":"    {{ macros.debian_mongodb_existing_user_fix() }}"},{"line_number":24,"context_line":"{% else %}"},{"line_number":25,"context_line":"    {{ macros.install_packages(mongodb_packages | customizable(\"packages\")) }}"},{"line_number":26,"context_line":"{% endif %}"},{"line_number":27,"context_line":""},{"line_number":28,"context_line":""},{"line_number":29,"context_line":"COPY mongodb_sudoers /etc/sudoers.d/kolla_mongodb_sudoers"}],"source_content_type":"text/x-jinja2","patch_set":1,"id":"1f485f77_6a905c4f","line":26,"range":{"start_line":24,"start_character":0,"end_line":26,"end_character":11},"updated":"2017-11-15 07:07:49.000000000","message":"else/endif not needed.\n\nIf keep the original will use the macro to install mongo-server and install any customized package in install_packages()","commit_id":"666f0f420dbb981554ea7dbba44b40e2b4027f4d"}]}