)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":19300,"name":"Eric Lemoine","email":"elemoine@mirantis.com","username":"elemoine"},"change_message_id":"a07a1fbdc9027eb97a62a96cdb98edb36f7e2246","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Benedikt Trefzer \u003cbenedikt.trefzer@cirrax.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2016-03-05 22:04:37 +0100"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Use debian repos for debian base docker image."},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Change-Id: Ic5c1d20fa9f7f1cd9fd20e658b4bd1abb235f3b6"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"7af24918_9cba616d","line":7,"updated":"2016-03-06 09:34:11.000000000","message":"Please remove the final dot.\n\nAlso, I think it would make sense to create a blueprint in Launchpad for this, and tag this change with \"Implements: blueprint\" here in the commit message.","commit_id":"f3abc836fdb001606e05dffc24df15f0cac7e053"},{"author":{"_account_id":19300,"name":"Eric Lemoine","email":"elemoine@mirantis.com","username":"elemoine"},"change_message_id":"cefb0bb917aff1a0309a8119713283270e9a1acb","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Benedikt Trefzer \u003cbenedikt.trefzer@cirrax.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2016-03-06 09:51:05 +0100"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Use debian repos for debian base docker image."},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Change-Id: Ic5c1d20fa9f7f1cd9fd20e658b4bd1abb235f3b6"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"7af24918_9ca3814f","line":7,"updated":"2016-03-06 09:34:27.000000000","message":"Please remove the final dot.\n\nAlso, there is no \"Implements: blueprint\", \"ClosesBug:\" or \"TrivialFix\" tag.","commit_id":"31ae9a31af9d164da13fea0f8cbc415b7e538ae7"},{"author":{"_account_id":16233,"name":"Mauricio Lima","email":"mauriciolimab@gmail.com","username":"limamauricio"},"change_message_id":"49148ead486854b534ffc05f7db97e3c8bb7ce0e","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Benedikt Trefzer \u003cbenedikt.trefzer@cirrax.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2016-03-22 17:41:12 +0100"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Use debian repos for debian base docker image."},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"bp:build-debian"},{"line_number":10,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"fa0719c6_8ac80c24","line":7,"range":{"start_line":7,"start_character":45,"end_line":7,"end_character":46},"updated":"2016-03-22 19:09:51.000000000","message":"remove","commit_id":"eb8d1d1d03a9073aa579523a6f6e1077094f397c"},{"author":{"_account_id":7888,"name":"Benedikt Trefzer","email":"benedikt.trefzer@cirrax.com","username":"trefzer"},"change_message_id":"475c54cd89ba569812e8cab93d627cd08dc052de","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Benedikt Trefzer \u003cbenedikt.trefzer@cirrax.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2016-03-22 17:41:12 +0100"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Use debian repos for debian base docker image."},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"bp:build-debian"},{"line_number":10,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"fa0719c6_5535f4f4","line":7,"range":{"start_line":7,"start_character":45,"end_line":7,"end_character":46},"in_reply_to":"fa0719c6_8ac80c24","updated":"2016-03-22 21:12:04.000000000","message":"Done","commit_id":"eb8d1d1d03a9073aa579523a6f6e1077094f397c"},{"author":{"_account_id":13642,"name":"Vikram Hosakote","email":"vikram.ganesh@gmail.com","username":"vhosakot"},"change_message_id":"3d277f15ff922deddc6fc734ec29231e1a40ef2b","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Use debian repos for debian base docker image."},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"bp:build-debian"},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"Change-Id: Ic5c1d20fa9f7f1cd9fd20e658b4bd1abb235f3b6"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"fa0719c6_39c1642f","line":9,"range":{"start_line":9,"start_character":0,"end_line":9,"end_character":15},"updated":"2016-03-22 18:17:26.000000000","message":"Please use \"Implements\" or \"Partially-implements\" here to be consistent.\n\nImplements: blueprint build-debian\nPartially-implements: build-delian\n\nhttps://wiki.openstack.org/wiki/GitCommitMessages","commit_id":"eb8d1d1d03a9073aa579523a6f6e1077094f397c"},{"author":{"_account_id":7888,"name":"Benedikt Trefzer","email":"benedikt.trefzer@cirrax.com","username":"trefzer"},"change_message_id":"475c54cd89ba569812e8cab93d627cd08dc052de","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Use debian repos for debian base docker image."},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"bp:build-debian"},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"Change-Id: Ic5c1d20fa9f7f1cd9fd20e658b4bd1abb235f3b6"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"fa0719c6_95166c53","line":9,"range":{"start_line":9,"start_character":0,"end_line":9,"end_character":15},"in_reply_to":"fa0719c6_39c1642f","updated":"2016-03-22 21:12:04.000000000","message":"Done","commit_id":"eb8d1d1d03a9073aa579523a6f6e1077094f397c"},{"author":{"_account_id":13642,"name":"Vikram Hosakote","email":"vikram.ganesh@gmail.com","username":"vhosakot"},"change_message_id":"0e561e0d72cc3171e064b3a13960db678251403e","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Use debian repos for debian base docker image"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Partially-implements: build-debian"},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"Change-Id: Ic5c1d20fa9f7f1cd9fd20e658b4bd1abb235f3b6"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":5,"id":"fa0719c6_1b08cd2b","line":9,"range":{"start_line":9,"start_character":0,"end_line":9,"end_character":34},"updated":"2016-03-22 21:43:13.000000000","message":"For consistency, change this to:\n\n  Partially-implements: blueprint build-debian","commit_id":"0a1b06b360fcb90854fc359f46307b7e56eafcc7"},{"author":{"_account_id":7888,"name":"Benedikt Trefzer","email":"benedikt.trefzer@cirrax.com","username":"trefzer"},"change_message_id":"7909120e15eed660c994b27bccf25c2cc266fdbe","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Use debian repos for debian base docker image"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Partially-implements: build-debian"},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"Change-Id: Ic5c1d20fa9f7f1cd9fd20e658b4bd1abb235f3b6"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":5,"id":"fa0719c6_e669fcbf","line":9,"range":{"start_line":9,"start_character":0,"end_line":9,"end_character":34},"in_reply_to":"fa0719c6_1b08cd2b","updated":"2016-03-22 21:50:35.000000000","message":"Done","commit_id":"0a1b06b360fcb90854fc359f46307b7e56eafcc7"}],"docker/base/Dockerfile.j2":[{"author":{"_account_id":13998,"name":"Daniel Gonzalez Nothnagel","email":"daniel.gonzalez.nothnagel@sap.com","username":"dgonzalez"},"change_message_id":"359663d17bd7be64227cfc329a5ab54c7b551c82","unresolved":false,"context_lines":[{"line_number":152,"context_line":"COPY sources.list.{{ base_distro }} /etc/apt/sources.list"},{"line_number":153,"context_line":"COPY apt_preferences.{{ base_distro }} /etc/apt/preferences"},{"line_number":154,"context_line":""},{"line_number":155,"context_line":"{% if base_distro in [\u0027ubuntu\u0027] %}"},{"line_number":156,"context_line":""},{"line_number":157,"context_line":"RUN apt-key adv --recv-keys --keyserver hkp://keyserver.ubuntu.com:80 199369E5404BD5FC7D2FE43BCBCB082A1BB943DB \\"},{"line_number":158,"context_line":"    \u0026\u0026 apt-key adv --recv-keys --keyserver hkp://keyserver.ubuntu.com:80 391A9AA2147192839E9DB0315EDB1B62EC4926EA \\"}],"source_content_type":"text/x-jinja2","patch_set":1,"id":"7af24918_a5fd8470","line":155,"range":{"start_line":155,"start_character":3,"end_line":155,"end_character":31},"updated":"2016-03-05 21:02:09.000000000","message":"if base_distro \u003d\u003d \u0027ubuntu\u0027","commit_id":"a41e1225157707744949092991e210875ad82c2e"},{"author":{"_account_id":13998,"name":"Daniel Gonzalez Nothnagel","email":"daniel.gonzalez.nothnagel@sap.com","username":"dgonzalez"},"change_message_id":"359663d17bd7be64227cfc329a5ab54c7b551c82","unresolved":false,"context_lines":[{"line_number":174,"context_line":"    \u0026\u0026 rm -rf /usr/lib/python2.7/site-packages \\"},{"line_number":175,"context_line":"    \u0026\u0026 ln -s dist-packages /usr/lib/python2.7/site-packages"},{"line_number":176,"context_line":""},{"line_number":177,"context_line":"{% elif base_distro in [\u0027debian\u0027] %}"},{"line_number":178,"context_line":""},{"line_number":179,"context_line":"RUN apt-key adv --recv-keys --keyserver hkp://keyserver.ubuntu.com:80 08B73419AC32B4E966C1A330E84AC2C0460F3994 \\"},{"line_number":180,"context_line":"    \u0026\u0026 apt-get update \\"}],"source_content_type":"text/x-jinja2","patch_set":1,"id":"7af24918_c5f41089","line":177,"range":{"start_line":177,"start_character":3,"end_line":177,"end_character":33},"updated":"2016-03-05 21:02:09.000000000","message":"elif base_distro \u003d\u003d \u0027debian\u0027","commit_id":"a41e1225157707744949092991e210875ad82c2e"},{"author":{"_account_id":13642,"name":"Vikram Hosakote","email":"vikram.ganesh@gmail.com","username":"vhosakot"},"change_message_id":"585f5e0d57b122a65e4b7c57dd5b5842b6c84263","unresolved":false,"context_lines":[{"line_number":149,"context_line":"# This will prevent questions from being asked during the install"},{"line_number":150,"context_line":"ENV DEBIAN_FRONTEND noninteractive"},{"line_number":151,"context_line":""},{"line_number":152,"context_line":"COPY sources.list.{{ base_distro }} /etc/apt/sources.list"},{"line_number":153,"context_line":"COPY apt_preferences.{{ base_distro }} /etc/apt/preferences"},{"line_number":154,"context_line":""},{"line_number":155,"context_line":"{% if base_distro \u003d\u003d \u0027ubuntu\u0027 %}"}],"source_content_type":"text/x-jinja2","patch_set":2,"id":"7af24918_36aa4c1a","line":152,"range":{"start_line":152,"start_character":0,"end_line":152,"end_character":57},"updated":"2016-03-06 04:45:42.000000000","message":"Could you test this with both Debian and Ubuntu.","commit_id":"f3abc836fdb001606e05dffc24df15f0cac7e053"},{"author":{"_account_id":7888,"name":"Benedikt Trefzer","email":"benedikt.trefzer@cirrax.com","username":"trefzer"},"change_message_id":"224f83152b3bc3d31d4ca00024e93c2d77d58f80","unresolved":false,"context_lines":[{"line_number":149,"context_line":"# This will prevent questions from being asked during the install"},{"line_number":150,"context_line":"ENV DEBIAN_FRONTEND noninteractive"},{"line_number":151,"context_line":""},{"line_number":152,"context_line":"COPY sources.list.{{ base_distro }} /etc/apt/sources.list"},{"line_number":153,"context_line":"COPY apt_preferences.{{ base_distro }} /etc/apt/preferences"},{"line_number":154,"context_line":""},{"line_number":155,"context_line":"{% if base_distro \u003d\u003d \u0027ubuntu\u0027 %}"}],"source_content_type":"text/x-jinja2","patch_set":2,"id":"7af24918_5ca579b3","line":152,"range":{"start_line":152,"start_character":0,"end_line":152,"end_character":57},"in_reply_to":"7af24918_36aa4c1a","updated":"2016-03-06 08:47:22.000000000","message":"Yes I build glance and keystone container successfully on Debian (both from source (with modification on glance/keystone Dockerfile) and binary. It builds the container without error. I do not yet have the infrastructur to do any functional tests but build seems ok.","commit_id":"f3abc836fdb001606e05dffc24df15f0cac7e053"},{"author":{"_account_id":13642,"name":"Vikram Hosakote","email":"vikram.ganesh@gmail.com","username":"vhosakot"},"change_message_id":"0ca556cd50137a5d7268f0a9bd89e5be87b0335e","unresolved":false,"context_lines":[{"line_number":149,"context_line":"# This will prevent questions from being asked during the install"},{"line_number":150,"context_line":"ENV DEBIAN_FRONTEND noninteractive"},{"line_number":151,"context_line":""},{"line_number":152,"context_line":"COPY sources.list.{{ base_distro }} /etc/apt/sources.list"},{"line_number":153,"context_line":"COPY apt_preferences.{{ base_distro }} /etc/apt/preferences"},{"line_number":154,"context_line":""},{"line_number":155,"context_line":"{% if base_distro \u003d\u003d \u0027ubuntu\u0027 %}"}],"source_content_type":"text/x-jinja2","patch_set":2,"id":"7af24918_7d5e6c77","line":152,"range":{"start_line":152,"start_character":0,"end_line":152,"end_character":57},"in_reply_to":"7af24918_5ca579b3","updated":"2016-03-06 22:36:34.000000000","message":"Instead of testing just glance and keystone containers, I think it is good to build and test all the core OpenStack services on Debian (like glance, haproxy, keystone, memcached, mariadb, neutron, nova, rabbitmq, heat, horizon, etc) so that an operator can get a working OpenStack cloud on Debian. Please refer ansible/group_vars/all.yml about how services can be enabled/disabled in kolla.","commit_id":"f3abc836fdb001606e05dffc24df15f0cac7e053"},{"author":{"_account_id":2834,"name":"Steven Dake","email":"steven.dake@gmail.com","username":"sdake"},"change_message_id":"8d859d111b2373dbd9c8abed19f7db5fe1120a4e","unresolved":false,"context_lines":[{"line_number":172,"context_line":"    \u0026\u0026 apt-get clean \\"},{"line_number":173,"context_line":"    \u0026\u0026 sed -i \"s|\u0027purelib\u0027: \u0027\\$base/local/lib/python\\$py_version_short/dist-packages\u0027,|\u0027purelib\u0027: \u0027\\$base/lib/python\\$py_version_short/dist-packages\u0027,|;s|\u0027platlib\u0027: \u0027\\$platbase/local/lib/python\\$py_version_short/dist-packages\u0027,|\u0027platlib\u0027: \u0027\\$platbase/lib/python\\$py_version_short/dist-packages\u0027,|;s|\u0027headers\u0027: \u0027\\$base/local/include/python\\$py_version_short/\\$dist_name\u0027,|\u0027headers\u0027: \u0027\\$base/include/python\\$py_version_short/\\$dist_name\u0027,|;s|\u0027scripts\u0027: \u0027\\$base/local/bin\u0027,|\u0027scripts\u0027: \u0027\\$base/bin\u0027,|;s|\u0027data\u0027   : \u0027\\$base/local\u0027,|\u0027data\u0027   : \u0027\\$base\u0027,|\" /usr/lib/python2.7/distutils/command/install.py \\"},{"line_number":174,"context_line":"    \u0026\u0026 rm -rf /usr/lib/python2.7/site-packages \\"},{"line_number":175,"context_line":"    \u0026\u0026 ln -s dist-packages /usr/lib/python2.7/site-packages"},{"line_number":176,"context_line":""},{"line_number":177,"context_line":"{% elif base_distro \u003d\u003d \u0027debian\u0027 %}"},{"line_number":178,"context_line":""}],"source_content_type":"text/x-jinja2","patch_set":2,"id":"7af24918_abb5557f","line":175,"updated":"2016-03-06 02:30:38.000000000","message":"you will need something like this for debian, unless debian shares a similar python layout as centos.  We sort of hack our python container layout to match on all distros.","commit_id":"f3abc836fdb001606e05dffc24df15f0cac7e053"},{"author":{"_account_id":7888,"name":"Benedikt Trefzer","email":"benedikt.trefzer@cirrax.com","username":"trefzer"},"change_message_id":"224f83152b3bc3d31d4ca00024e93c2d77d58f80","unresolved":false,"context_lines":[{"line_number":172,"context_line":"    \u0026\u0026 apt-get clean \\"},{"line_number":173,"context_line":"    \u0026\u0026 sed -i \"s|\u0027purelib\u0027: \u0027\\$base/local/lib/python\\$py_version_short/dist-packages\u0027,|\u0027purelib\u0027: \u0027\\$base/lib/python\\$py_version_short/dist-packages\u0027,|;s|\u0027platlib\u0027: \u0027\\$platbase/local/lib/python\\$py_version_short/dist-packages\u0027,|\u0027platlib\u0027: \u0027\\$platbase/lib/python\\$py_version_short/dist-packages\u0027,|;s|\u0027headers\u0027: \u0027\\$base/local/include/python\\$py_version_short/\\$dist_name\u0027,|\u0027headers\u0027: \u0027\\$base/include/python\\$py_version_short/\\$dist_name\u0027,|;s|\u0027scripts\u0027: \u0027\\$base/local/bin\u0027,|\u0027scripts\u0027: \u0027\\$base/bin\u0027,|;s|\u0027data\u0027   : \u0027\\$base/local\u0027,|\u0027data\u0027   : \u0027\\$base\u0027,|\" /usr/lib/python2.7/distutils/command/install.py \\"},{"line_number":174,"context_line":"    \u0026\u0026 rm -rf /usr/lib/python2.7/site-packages \\"},{"line_number":175,"context_line":"    \u0026\u0026 ln -s dist-packages /usr/lib/python2.7/site-packages"},{"line_number":176,"context_line":""},{"line_number":177,"context_line":"{% elif base_distro \u003d\u003d \u0027debian\u0027 %}"},{"line_number":178,"context_line":""}],"source_content_type":"text/x-jinja2","patch_set":2,"id":"7af24918_7c879512","line":175,"in_reply_to":"7af24918_abb5557f","updated":"2016-03-06 08:47:22.000000000","message":"removing files installed from packeges seems to be a very hackish way for me. Since I could build from source, I expect this to be not necessary.","commit_id":"f3abc836fdb001606e05dffc24df15f0cac7e053"},{"author":{"_account_id":2834,"name":"Steven Dake","email":"steven.dake@gmail.com","username":"sdake"},"change_message_id":"8d859d111b2373dbd9c8abed19f7db5fe1120a4e","unresolved":false,"context_lines":[{"line_number":189,"context_line":"        python-six \\"},{"line_number":190,"context_line":"        curl \\"},{"line_number":191,"context_line":"    \u0026\u0026 apt-get clean"},{"line_number":192,"context_line":"{% endif %}"},{"line_number":193,"context_line":""},{"line_number":194,"context_line":"{# endif for base_distro ubuntu, debian #}"},{"line_number":195,"context_line":"{% endif %}"}],"source_content_type":"text/x-jinja2","patch_set":2,"id":"7af24918_4bc011e1","line":192,"updated":"2016-03-06 02:30:38.000000000","message":"I\u0027m pretty sure this  won\u0027t build source packages on debian.  Thanks for the contribution, but could you test ubuntu source?  We don\u0027t have any gating for that at present.","commit_id":"f3abc836fdb001606e05dffc24df15f0cac7e053"},{"author":{"_account_id":7888,"name":"Benedikt Trefzer","email":"benedikt.trefzer@cirrax.com","username":"trefzer"},"change_message_id":"224f83152b3bc3d31d4ca00024e93c2d77d58f80","unresolved":false,"context_lines":[{"line_number":189,"context_line":"        python-six \\"},{"line_number":190,"context_line":"        curl \\"},{"line_number":191,"context_line":"    \u0026\u0026 apt-get clean"},{"line_number":192,"context_line":"{% endif %}"},{"line_number":193,"context_line":""},{"line_number":194,"context_line":"{# endif for base_distro ubuntu, debian #}"},{"line_number":195,"context_line":"{% endif %}"}],"source_content_type":"text/x-jinja2","patch_set":2,"id":"7af24918_dc78692e","line":192,"in_reply_to":"7af24918_4bc011e1","updated":"2016-03-06 08:47:22.000000000","message":"you mean \u0027debian source\u0027 ?\nI build debian source keystone and glance which both build error free. I did not yet test all the others.","commit_id":"f3abc836fdb001606e05dffc24df15f0cac7e053"},{"author":{"_account_id":2834,"name":"Steven Dake","email":"steven.dake@gmail.com","username":"sdake"},"change_message_id":"b6f3521a019b27c6883d8a042355ffb2c6f476b2","unresolved":false,"context_lines":[{"line_number":191,"context_line":"    \u0026\u0026 apt-get clean"},{"line_number":192,"context_line":"{% endif %}"},{"line_number":193,"context_line":""},{"line_number":194,"context_line":"{# endif for base_distro ubuntu, debian #}"},{"line_number":195,"context_line":"{% endif %}"},{"line_number":196,"context_line":""},{"line_number":197,"context_line":"COPY set_configs.py /usr/local/bin/kolla_set_configs"}],"source_content_type":"text/x-jinja2","patch_set":2,"id":"7af24918_6b622d18","line":194,"updated":"2016-03-06 02:32:14.000000000","message":"I\u0027m not certain, but I don\u0027t think this is correct jinja2 syntax.  Can you test that this builds?","commit_id":"f3abc836fdb001606e05dffc24df15f0cac7e053"},{"author":{"_account_id":7888,"name":"Benedikt Trefzer","email":"benedikt.trefzer@cirrax.com","username":"trefzer"},"change_message_id":"224f83152b3bc3d31d4ca00024e93c2d77d58f80","unresolved":false,"context_lines":[{"line_number":191,"context_line":"    \u0026\u0026 apt-get clean"},{"line_number":192,"context_line":"{% endif %}"},{"line_number":193,"context_line":""},{"line_number":194,"context_line":"{# endif for base_distro ubuntu, debian #}"},{"line_number":195,"context_line":"{% endif %}"},{"line_number":196,"context_line":""},{"line_number":197,"context_line":"COPY set_configs.py /usr/local/bin/kolla_set_configs"}],"source_content_type":"text/x-jinja2","patch_set":2,"id":"7af24918_7c9af5c6","line":194,"in_reply_to":"7af24918_6b622d18","updated":"2016-03-06 08:47:22.000000000","message":"This is copy paste from line 146 or 104.","commit_id":"f3abc836fdb001606e05dffc24df15f0cac7e053"},{"author":{"_account_id":13642,"name":"Vikram Hosakote","email":"vikram.ganesh@gmail.com","username":"vhosakot"},"change_message_id":"3d277f15ff922deddc6fc734ec29231e1a40ef2b","unresolved":false,"context_lines":[{"line_number":182,"context_line":""},{"line_number":183,"context_line":"{% elif base_distro \u003d\u003d \u0027debian\u0027 %}"},{"line_number":184,"context_line":""},{"line_number":185,"context_line":"RUN apt-key adv --recv-keys --keyserver hkp://keyserver.ubuntu.com:80 08B73419AC32B4E966C1A330E84AC2C0460F3994 \\"},{"line_number":186,"context_line":"    \u0026\u0026 apt-key adv --recv-keys --keyserver hkp://keyserver.ubuntu.com 0xcbcb082a1bb943db \\"},{"line_number":187,"context_line":"    \u0026\u0026 apt-get update \\"},{"line_number":188,"context_line":"    \u0026\u0026 apt-get upgrade -y \\"}],"source_content_type":"text/x-jinja2","patch_set":4,"id":"fa0719c6_b4a9f9e1","line":185,"range":{"start_line":185,"start_character":70,"end_line":185,"end_character":110},"updated":"2016-03-22 18:17:26.000000000","message":"I see the key 08B73419AC32B4E966C1A330E84AC2C0460F3994 already downloaded for debian distro in line 164 in this file. Can\u0027t it be used ?\n\nhttps://github.com/openstack/kolla/blob/master/docker/base/Dockerfile.j2#L153-L164","commit_id":"eb8d1d1d03a9073aa579523a6f6e1077094f397c"},{"author":{"_account_id":14119,"name":"Sam Yaple","email":"sam+git@yaple.net","username":"SamYaple"},"change_message_id":"6b44756ff8a8d2e5feb7bf18a2d65008afdae3ef","unresolved":false,"context_lines":[{"line_number":182,"context_line":""},{"line_number":183,"context_line":"{% elif base_distro \u003d\u003d \u0027debian\u0027 %}"},{"line_number":184,"context_line":""},{"line_number":185,"context_line":"RUN apt-key adv --recv-keys --keyserver hkp://keyserver.ubuntu.com:80 08B73419AC32B4E966C1A330E84AC2C0460F3994 \\"},{"line_number":186,"context_line":"    \u0026\u0026 apt-key adv --recv-keys --keyserver hkp://keyserver.ubuntu.com 0xcbcb082a1bb943db \\"},{"line_number":187,"context_line":"    \u0026\u0026 apt-get update \\"},{"line_number":188,"context_line":"    \u0026\u0026 apt-get upgrade -y \\"}],"source_content_type":"text/x-jinja2","patch_set":4,"id":"fa0719c6_084d2c3e","line":185,"updated":"2016-03-22 17:11:27.000000000","message":"this section need only have teh specifics needed for debian or ubuntu, where code is shared please share code (like the updates, shared keystone, upgrades, packages etc)\n\nUse your best judgement","commit_id":"eb8d1d1d03a9073aa579523a6f6e1077094f397c"},{"author":{"_account_id":7888,"name":"Benedikt Trefzer","email":"benedikt.trefzer@cirrax.com","username":"trefzer"},"change_message_id":"475c54cd89ba569812e8cab93d627cd08dc052de","unresolved":false,"context_lines":[{"line_number":182,"context_line":""},{"line_number":183,"context_line":"{% elif base_distro \u003d\u003d \u0027debian\u0027 %}"},{"line_number":184,"context_line":""},{"line_number":185,"context_line":"RUN apt-key adv --recv-keys --keyserver hkp://keyserver.ubuntu.com:80 08B73419AC32B4E966C1A330E84AC2C0460F3994 \\"},{"line_number":186,"context_line":"    \u0026\u0026 apt-key adv --recv-keys --keyserver hkp://keyserver.ubuntu.com 0xcbcb082a1bb943db \\"},{"line_number":187,"context_line":"    \u0026\u0026 apt-get update \\"},{"line_number":188,"context_line":"    \u0026\u0026 apt-get upgrade -y \\"}],"source_content_type":"text/x-jinja2","patch_set":4,"id":"fa0719c6_f57888c6","line":185,"in_reply_to":"fa0719c6_084d2c3e","updated":"2016-03-22 21:12:04.000000000","message":"ok, reworked that part, using jinja2 variables.\ndo not now if you like that ;)","commit_id":"eb8d1d1d03a9073aa579523a6f6e1077094f397c"},{"author":{"_account_id":7888,"name":"Benedikt Trefzer","email":"benedikt.trefzer@cirrax.com","username":"trefzer"},"change_message_id":"475c54cd89ba569812e8cab93d627cd08dc052de","unresolved":false,"context_lines":[{"line_number":182,"context_line":""},{"line_number":183,"context_line":"{% elif base_distro \u003d\u003d \u0027debian\u0027 %}"},{"line_number":184,"context_line":""},{"line_number":185,"context_line":"RUN apt-key adv --recv-keys --keyserver hkp://keyserver.ubuntu.com:80 08B73419AC32B4E966C1A330E84AC2C0460F3994 \\"},{"line_number":186,"context_line":"    \u0026\u0026 apt-key adv --recv-keys --keyserver hkp://keyserver.ubuntu.com 0xcbcb082a1bb943db \\"},{"line_number":187,"context_line":"    \u0026\u0026 apt-get update \\"},{"line_number":188,"context_line":"    \u0026\u0026 apt-get upgrade -y \\"}],"source_content_type":"text/x-jinja2","patch_set":4,"id":"fa0719c6_754bd887","line":185,"range":{"start_line":185,"start_character":70,"end_line":185,"end_character":110},"in_reply_to":"fa0719c6_b4a9f9e1","updated":"2016-03-22 21:12:04.000000000","message":"that was only for ubuntu, it was not run for debian.\nbut I reworked that part now (but since I cannot cancat two arrays in plain jinja2, the key is still doubled).","commit_id":"eb8d1d1d03a9073aa579523a6f6e1077094f397c"},{"author":{"_account_id":13642,"name":"Vikram Hosakote","email":"vikram.ganesh@gmail.com","username":"vhosakot"},"change_message_id":"0e561e0d72cc3171e064b3a13960db678251403e","unresolved":false,"context_lines":[{"line_number":204,"context_line":"{% endif %}"},{"line_number":205,"context_line":""},{"line_number":206,"context_line":"{# endif for base_distro ubuntu, debian #}"},{"line_number":207,"context_line":"{% endif %}"},{"line_number":208,"context_line":""},{"line_number":209,"context_line":"COPY set_configs.py /usr/local/bin/kolla_set_configs"},{"line_number":210,"context_line":"COPY start.sh /usr/local/bin/kolla_start"}],"source_content_type":"text/x-jinja2","patch_set":5,"id":"fa0719c6_bb21e1b6","line":207,"range":{"start_line":207,"start_character":0,"end_line":207,"end_character":11},"updated":"2016-03-22 21:43:13.000000000","message":"This line failed in gate.\n\nhttp://logs.openstack.org/36/288936/5/check/gate-kolla-dsvm-build-centos-binary/a20a7fe/console.html#_2016-03-22_21_19_40_073","commit_id":"0a1b06b360fcb90854fc359f46307b7e56eafcc7"},{"author":{"_account_id":7888,"name":"Benedikt Trefzer","email":"benedikt.trefzer@cirrax.com","username":"trefzer"},"change_message_id":"7909120e15eed660c994b27bccf25c2cc266fdbe","unresolved":false,"context_lines":[{"line_number":204,"context_line":"{% endif %}"},{"line_number":205,"context_line":""},{"line_number":206,"context_line":"{# endif for base_distro ubuntu, debian #}"},{"line_number":207,"context_line":"{% endif %}"},{"line_number":208,"context_line":""},{"line_number":209,"context_line":"COPY set_configs.py /usr/local/bin/kolla_set_configs"},{"line_number":210,"context_line":"COPY start.sh /usr/local/bin/kolla_start"}],"source_content_type":"text/x-jinja2","patch_set":5,"id":"fa0719c6_466268ae","line":207,"range":{"start_line":207,"start_character":0,"end_line":207,"end_character":11},"in_reply_to":"fa0719c6_bb21e1b6","updated":"2016-03-22 21:50:35.000000000","message":"thanks, the endif in line 178 was to much !","commit_id":"0a1b06b360fcb90854fc359f46307b7e56eafcc7"},{"author":{"_account_id":13642,"name":"Vikram Hosakote","email":"vikram.ganesh@gmail.com","username":"vhosakot"},"change_message_id":"cd781ae51af8b1e60d785dac3d94401b4b43aff8","unresolved":false,"context_lines":[{"line_number":185,"context_line":"    -%}"},{"line_number":186,"context_line":"{% endif %}"},{"line_number":187,"context_line":""},{"line_number":188,"context_line":"RUN"},{"line_number":189,"context_line":"    {% for key in keys -%}"},{"line_number":190,"context_line":"    \u0026\u0026 apt-key adv --recv-keys --keyserver hkp://keyserver.ubuntu.com:80 {{ key }}"},{"line_number":191,"context_line":"    {% endfor -%}"}],"source_content_type":"text/x-jinja2","patch_set":6,"id":"fa0719c6_0257b5ce","line":188,"range":{"start_line":188,"start_character":0,"end_line":188,"end_character":3},"updated":"2016-03-23 02:33:19.000000000","message":"Looks like this is not correct Dockerfile syntax.\n\nhttp://logs.openstack.org/36/288936/6/check/gate-kolla-dsvm-build-ubuntu-source/2a68bbe/console.html#_2016-03-22_21_59_39_423\n\nHere is an example how to use the \"RUN\" syntax to run multiple commands in multiple lines in Dockerfile.\n\nhttps://github.com/openstack/kolla/blob/master/docker/base/Dockerfile.j2#L34-L37","commit_id":"bc5cfb03cce1c24f5622457b9c540e0b1e6ce91f"},{"author":{"_account_id":16620,"name":"Alicja Kwasniewska","email":"alicja.kwasniewska@intel.com","username":"akwasnie"},"change_message_id":"bc6b4dacb974aaa9bdb378aa7db967e888e4ac68","unresolved":false,"context_lines":[{"line_number":159,"context_line":"COPY apt_preferences.{{ base_distro }} /etc/apt/preferences"},{"line_number":160,"context_line":""},{"line_number":161,"context_line":"{% set packages \u003d \u0027\\"},{"line_number":162,"context_line":"   ca-certificates \\"},{"line_number":163,"context_line":"   python \\"},{"line_number":164,"context_line":"   python-jinja2 \\"},{"line_number":165,"context_line":"   python-kazoo \\"},{"line_number":166,"context_line":"   python-six \\"},{"line_number":167,"context_line":"   curl\u0027"},{"line_number":168,"context_line":"-%}"},{"line_number":169,"context_line":""},{"line_number":170,"context_line":"{% if base_distro \u003d\u003d \u0027ubuntu\u0027 %}"}],"source_content_type":"text/x-jinja2","patch_set":8,"id":"da0c15f0_8d3411df","line":167,"range":{"start_line":162,"start_character":3,"end_line":167,"end_character":8},"updated":"2016-03-24 18:10:37.000000000","message":"I know that these names were not ordered alphabetically, but could you order them in this way?","commit_id":"fc17ed80d593cb9ee8887e7a7ad69ed33fbb61de"},{"author":{"_account_id":7888,"name":"Benedikt Trefzer","email":"benedikt.trefzer@cirrax.com","username":"trefzer"},"change_message_id":"25295a2c0be48b5cc5ef4a68f1d37b29fc9518ac","unresolved":false,"context_lines":[{"line_number":159,"context_line":"COPY apt_preferences.{{ base_distro }} /etc/apt/preferences"},{"line_number":160,"context_line":""},{"line_number":161,"context_line":"{% set packages \u003d \u0027\\"},{"line_number":162,"context_line":"   ca-certificates \\"},{"line_number":163,"context_line":"   python \\"},{"line_number":164,"context_line":"   python-jinja2 \\"},{"line_number":165,"context_line":"   python-kazoo \\"},{"line_number":166,"context_line":"   python-six \\"},{"line_number":167,"context_line":"   curl\u0027"},{"line_number":168,"context_line":"-%}"},{"line_number":169,"context_line":""},{"line_number":170,"context_line":"{% if base_distro \u003d\u003d \u0027ubuntu\u0027 %}"}],"source_content_type":"text/x-jinja2","patch_set":8,"id":"da0c15f0_f8baa86d","line":167,"range":{"start_line":162,"start_character":3,"end_line":167,"end_character":8},"in_reply_to":"da0c15f0_8d3411df","updated":"2016-03-24 20:29:50.000000000","message":"Done","commit_id":"fc17ed80d593cb9ee8887e7a7ad69ed33fbb61de"},{"author":{"_account_id":16620,"name":"Alicja Kwasniewska","email":"alicja.kwasniewska@intel.com","username":"akwasnie"},"change_message_id":"374a5e039e443107dadcce2d21ee602d1f555a79","unresolved":false,"context_lines":[{"line_number":165,"context_line":"   python-jinja2 \\"},{"line_number":166,"context_line":"   python-kazoo \\"},{"line_number":167,"context_line":"   python-six\u0027"},{"line_number":168,"context_line":"-%}"},{"line_number":169,"context_line":""},{"line_number":170,"context_line":"{% if base_distro \u003d\u003d \u0027ubuntu\u0027 %}"},{"line_number":171,"context_line":"    {% set keys \u003d ["}],"source_content_type":"text/x-jinja2","patch_set":9,"id":"da0c15f0_66ab721e","line":168,"range":{"start_line":168,"start_character":0,"end_line":168,"end_character":3},"updated":"2016-03-25 16:09:55.000000000","message":"we don\u0027t use this convention in any other template so I would prefer to change it to %}","commit_id":"9a1f1251641b3a03e60455adca197c0a31c9501a"},{"author":{"_account_id":7888,"name":"Benedikt Trefzer","email":"benedikt.trefzer@cirrax.com","username":"trefzer"},"change_message_id":"89a08384d3e3f60f2bd099df4476442b68e4b9b1","unresolved":false,"context_lines":[{"line_number":165,"context_line":"   python-jinja2 \\"},{"line_number":166,"context_line":"   python-kazoo \\"},{"line_number":167,"context_line":"   python-six\u0027"},{"line_number":168,"context_line":"-%}"},{"line_number":169,"context_line":""},{"line_number":170,"context_line":"{% if base_distro \u003d\u003d \u0027ubuntu\u0027 %}"},{"line_number":171,"context_line":"    {% set keys \u003d ["}],"source_content_type":"text/x-jinja2","patch_set":9,"id":"da0c15f0_a979dbfe","line":168,"range":{"start_line":168,"start_character":0,"end_line":168,"end_character":3},"in_reply_to":"da0c15f0_66ab721e","updated":"2016-03-25 16:25:27.000000000","message":"Done","commit_id":"9a1f1251641b3a03e60455adca197c0a31c9501a"},{"author":{"_account_id":16620,"name":"Alicja Kwasniewska","email":"alicja.kwasniewska@intel.com","username":"akwasnie"},"change_message_id":"374a5e039e443107dadcce2d21ee602d1f555a79","unresolved":false,"context_lines":[{"line_number":174,"context_line":"      \u0027430BDF5C56E7C94E848EE60C1C4CBDCDCD2EFD2A\u0027,"},{"line_number":175,"context_line":"      \u002708B73419AC32B4E966C1A330E84AC2C0460F3994\u0027,"},{"line_number":176,"context_line":"      \u002746095ACC8548582C1A2699A9D27D666CD88E42B4\u0027,"},{"line_number":177,"context_line":"    ] -%}"},{"line_number":178,"context_line":"{% elif base_distro \u003d\u003d \u0027debian\u0027 %}"},{"line_number":179,"context_line":"    {% set keys \u003d ["},{"line_number":180,"context_line":"      \u002708B73419AC32B4E966C1A330E84AC2C0460F3994\u0027,"}],"source_content_type":"text/x-jinja2","patch_set":9,"id":"da0c15f0_e66dc205","line":177,"range":{"start_line":177,"start_character":6,"end_line":177,"end_character":9},"updated":"2016-03-25 16:09:55.000000000","message":"ditto","commit_id":"9a1f1251641b3a03e60455adca197c0a31c9501a"},{"author":{"_account_id":7888,"name":"Benedikt Trefzer","email":"benedikt.trefzer@cirrax.com","username":"trefzer"},"change_message_id":"89a08384d3e3f60f2bd099df4476442b68e4b9b1","unresolved":false,"context_lines":[{"line_number":174,"context_line":"      \u0027430BDF5C56E7C94E848EE60C1C4CBDCDCD2EFD2A\u0027,"},{"line_number":175,"context_line":"      \u002708B73419AC32B4E966C1A330E84AC2C0460F3994\u0027,"},{"line_number":176,"context_line":"      \u002746095ACC8548582C1A2699A9D27D666CD88E42B4\u0027,"},{"line_number":177,"context_line":"    ] -%}"},{"line_number":178,"context_line":"{% elif base_distro \u003d\u003d \u0027debian\u0027 %}"},{"line_number":179,"context_line":"    {% set keys \u003d ["},{"line_number":180,"context_line":"      \u002708B73419AC32B4E966C1A330E84AC2C0460F3994\u0027,"}],"source_content_type":"text/x-jinja2","patch_set":9,"id":"da0c15f0_a9c71b11","line":177,"range":{"start_line":177,"start_character":6,"end_line":177,"end_character":9},"in_reply_to":"da0c15f0_e66dc205","updated":"2016-03-25 16:25:27.000000000","message":"Done","commit_id":"9a1f1251641b3a03e60455adca197c0a31c9501a"},{"author":{"_account_id":16620,"name":"Alicja Kwasniewska","email":"alicja.kwasniewska@intel.com","username":"akwasnie"},"change_message_id":"374a5e039e443107dadcce2d21ee602d1f555a79","unresolved":false,"context_lines":[{"line_number":179,"context_line":"    {% set keys \u003d ["},{"line_number":180,"context_line":"      \u002708B73419AC32B4E966C1A330E84AC2C0460F3994\u0027,"},{"line_number":181,"context_line":"      \u00270xcbcb082a1bb943db\u0027,"},{"line_number":182,"context_line":"    ] -%}"},{"line_number":183,"context_line":"    {% set packages \u003d packages +"},{"line_number":184,"context_line":"      \u0027 sudo\u0027"},{"line_number":185,"context_line":"    -%}"}],"source_content_type":"text/x-jinja2","patch_set":9,"id":"da0c15f0_66825296","line":182,"range":{"start_line":182,"start_character":6,"end_line":182,"end_character":9},"updated":"2016-03-25 16:09:55.000000000","message":"ditto","commit_id":"9a1f1251641b3a03e60455adca197c0a31c9501a"},{"author":{"_account_id":7888,"name":"Benedikt Trefzer","email":"benedikt.trefzer@cirrax.com","username":"trefzer"},"change_message_id":"89a08384d3e3f60f2bd099df4476442b68e4b9b1","unresolved":false,"context_lines":[{"line_number":179,"context_line":"    {% set keys \u003d ["},{"line_number":180,"context_line":"      \u002708B73419AC32B4E966C1A330E84AC2C0460F3994\u0027,"},{"line_number":181,"context_line":"      \u00270xcbcb082a1bb943db\u0027,"},{"line_number":182,"context_line":"    ] -%}"},{"line_number":183,"context_line":"    {% set packages \u003d packages +"},{"line_number":184,"context_line":"      \u0027 sudo\u0027"},{"line_number":185,"context_line":"    -%}"}],"source_content_type":"text/x-jinja2","patch_set":9,"id":"da0c15f0_89ccd736","line":182,"range":{"start_line":182,"start_character":6,"end_line":182,"end_character":9},"in_reply_to":"da0c15f0_66825296","updated":"2016-03-25 16:25:27.000000000","message":"Done","commit_id":"9a1f1251641b3a03e60455adca197c0a31c9501a"},{"author":{"_account_id":16620,"name":"Alicja Kwasniewska","email":"alicja.kwasniewska@intel.com","username":"akwasnie"},"change_message_id":"374a5e039e443107dadcce2d21ee602d1f555a79","unresolved":false,"context_lines":[{"line_number":182,"context_line":"    ] -%}"},{"line_number":183,"context_line":"    {% set packages \u003d packages +"},{"line_number":184,"context_line":"      \u0027 sudo\u0027"},{"line_number":185,"context_line":"    -%}"},{"line_number":186,"context_line":"{% endif %}"},{"line_number":187,"context_line":""},{"line_number":188,"context_line":"RUN true \\"}],"source_content_type":"text/x-jinja2","patch_set":9,"id":"da0c15f0_a64aaa81","line":185,"range":{"start_line":185,"start_character":4,"end_line":185,"end_character":7},"updated":"2016-03-25 16:09:55.000000000","message":"ditto","commit_id":"9a1f1251641b3a03e60455adca197c0a31c9501a"},{"author":{"_account_id":7888,"name":"Benedikt Trefzer","email":"benedikt.trefzer@cirrax.com","username":"trefzer"},"change_message_id":"89a08384d3e3f60f2bd099df4476442b68e4b9b1","unresolved":false,"context_lines":[{"line_number":182,"context_line":"    ] -%}"},{"line_number":183,"context_line":"    {% set packages \u003d packages +"},{"line_number":184,"context_line":"      \u0027 sudo\u0027"},{"line_number":185,"context_line":"    -%}"},{"line_number":186,"context_line":"{% endif %}"},{"line_number":187,"context_line":""},{"line_number":188,"context_line":"RUN true \\"}],"source_content_type":"text/x-jinja2","patch_set":9,"id":"da0c15f0_e9d113cf","line":185,"range":{"start_line":185,"start_character":4,"end_line":185,"end_character":7},"in_reply_to":"da0c15f0_a64aaa81","updated":"2016-03-25 16:25:27.000000000","message":"Done","commit_id":"9a1f1251641b3a03e60455adca197c0a31c9501a"},{"author":{"_account_id":16620,"name":"Alicja Kwasniewska","email":"alicja.kwasniewska@intel.com","username":"akwasnie"},"change_message_id":"275134af564748a56dcfca1335abd5ef76bc2603","unresolved":false,"context_lines":[{"line_number":196,"context_line":"    \u0026\u0026 apt-get clean"},{"line_number":197,"context_line":""},{"line_number":198,"context_line":"{% if base_distro \u003d\u003d \u0027ubuntu\u0027 %}"},{"line_number":199,"context_line":"RUN     sed -i \"s|\u0027purelib\u0027: \u0027\\$base/local/lib/python\\$py_version_short/dist-packages\u0027,|\u0027purelib\u0027: \u0027\\$base/lib/python\\$py_version_short/dist-packages\u0027,|;s|\u0027platlib\u0027: \u0027\\$platbase/local/lib/python\\$py_version_short/dist-packages\u0027,|\u0027platlib\u0027: \u0027\\$platbase/lib/python\\$py_version_short/dist-packages\u0027,|;s|\u0027headers\u0027: \u0027\\$base/local/include/python\\$py_version_short/\\$dist_name\u0027,|\u0027headers\u0027: \u0027\\$base/include/python\\$py_version_short/\\$dist_name\u0027,|;s|\u0027scripts\u0027: \u0027\\$base/local/bin\u0027,|\u0027scripts\u0027: \u0027\\$base/bin\u0027,|;s|\u0027data\u0027   : \u0027\\$base/local\u0027,|\u0027data\u0027   : \u0027\\$base\u0027,|\" /usr/lib/python2.7/distutils/command/install.py \\"},{"line_number":200,"context_line":"    \u0026\u0026 rm -rf /usr/lib/python2.7/site-packages \\"},{"line_number":201,"context_line":"    \u0026\u0026 ln -s dist-packages /usr/lib/python2.7/site-packages"},{"line_number":202,"context_line":"{% endif %}"}],"source_content_type":"text/x-jinja2","patch_set":12,"id":"ba0121b8_dd6cdb38","line":199,"range":{"start_line":199,"start_character":3,"end_line":199,"end_character":8},"updated":"2016-04-04 06:36:30.000000000","message":"sorry for not pointing it out previously, please remove unnecessary spaces","commit_id":"400c2f1cdd327bb3180ba3218af7f25e9bc29459"},{"author":{"_account_id":7888,"name":"Benedikt Trefzer","email":"benedikt.trefzer@cirrax.com","username":"trefzer"},"change_message_id":"15807ee9f8350ca9bd69c2a29719da184c340383","unresolved":false,"context_lines":[{"line_number":196,"context_line":"    \u0026\u0026 apt-get clean"},{"line_number":197,"context_line":""},{"line_number":198,"context_line":"{% if base_distro \u003d\u003d \u0027ubuntu\u0027 %}"},{"line_number":199,"context_line":"RUN     sed -i \"s|\u0027purelib\u0027: \u0027\\$base/local/lib/python\\$py_version_short/dist-packages\u0027,|\u0027purelib\u0027: \u0027\\$base/lib/python\\$py_version_short/dist-packages\u0027,|;s|\u0027platlib\u0027: \u0027\\$platbase/local/lib/python\\$py_version_short/dist-packages\u0027,|\u0027platlib\u0027: \u0027\\$platbase/lib/python\\$py_version_short/dist-packages\u0027,|;s|\u0027headers\u0027: \u0027\\$base/local/include/python\\$py_version_short/\\$dist_name\u0027,|\u0027headers\u0027: \u0027\\$base/include/python\\$py_version_short/\\$dist_name\u0027,|;s|\u0027scripts\u0027: \u0027\\$base/local/bin\u0027,|\u0027scripts\u0027: \u0027\\$base/bin\u0027,|;s|\u0027data\u0027   : \u0027\\$base/local\u0027,|\u0027data\u0027   : \u0027\\$base\u0027,|\" /usr/lib/python2.7/distutils/command/install.py \\"},{"line_number":200,"context_line":"    \u0026\u0026 rm -rf /usr/lib/python2.7/site-packages \\"},{"line_number":201,"context_line":"    \u0026\u0026 ln -s dist-packages /usr/lib/python2.7/site-packages"},{"line_number":202,"context_line":"{% endif %}"}],"source_content_type":"text/x-jinja2","patch_set":12,"id":"9a061dce_43b0fcdd","line":199,"range":{"start_line":199,"start_character":3,"end_line":199,"end_character":8},"in_reply_to":"ba0121b8_dd6cdb38","updated":"2016-04-04 14:31:20.000000000","message":"Done","commit_id":"400c2f1cdd327bb3180ba3218af7f25e9bc29459"},{"author":{"_account_id":11105,"name":"Swapnil Kulkarni","email":"coolsvap@gmail.com","username":"coolsvap"},"change_message_id":"143e77338236451a9242c5d86fb5690505773e7c","unresolved":false,"context_lines":[{"line_number":199,"context_line":"%}"},{"line_number":200,"context_line":""},{"line_number":201,"context_line":"{% if base_distro \u003d\u003d \u0027ubuntu\u0027 %}"},{"line_number":202,"context_line":"    {% set keys \u003d ["},{"line_number":203,"context_line":"      \u0027199369E5404BD5FC7D2FE43BCBCB082A1BB943DB\u0027,"},{"line_number":204,"context_line":"      \u0027391A9AA2147192839E9DB0315EDB1B62EC4926EA\u0027,"},{"line_number":205,"context_line":"      \u0027430BDF5C56E7C94E848EE60C1C4CBDCDCD2EFD2A\u0027,"}],"source_content_type":"text/x-jinja2","patch_set":19,"id":"dab17558_50898a25","line":202,"range":{"start_line":202,"start_character":10,"end_line":202,"end_character":11},"updated":"2016-05-16 03:25:42.000000000","message":"Can you make it configurable like its currently done for centos in I1fc6732ce9af7db63e15c89b437d0bce0ff0b121","commit_id":"3df80d87e2dcb0986e4457854a00a303477f565d"},{"author":{"_account_id":7888,"name":"Benedikt Trefzer","email":"benedikt.trefzer@cirrax.com","username":"trefzer"},"change_message_id":"0953bbd3991b01d56413925d589bbaad76debea9","unresolved":false,"context_lines":[{"line_number":199,"context_line":"%}"},{"line_number":200,"context_line":""},{"line_number":201,"context_line":"{% if base_distro \u003d\u003d \u0027ubuntu\u0027 %}"},{"line_number":202,"context_line":"    {% set keys \u003d ["},{"line_number":203,"context_line":"      \u0027199369E5404BD5FC7D2FE43BCBCB082A1BB943DB\u0027,"},{"line_number":204,"context_line":"      \u0027391A9AA2147192839E9DB0315EDB1B62EC4926EA\u0027,"},{"line_number":205,"context_line":"      \u0027430BDF5C56E7C94E848EE60C1C4CBDCDCD2EFD2A\u0027,"}],"source_content_type":"text/x-jinja2","patch_set":19,"id":"dab17558_de490433","line":202,"range":{"start_line":202,"start_character":10,"end_line":202,"end_character":11},"in_reply_to":"dab17558_50898a25","updated":"2016-05-16 08:03:18.000000000","message":"NO.\nI will not mix the build-debian blueprint with the commit you mentioned. This change is about making debian running. the other one about repo cleanup. It does not make any sence to mix these two (and make dependencies upon each other (or worse implement the same config stuff a second time !).","commit_id":"3df80d87e2dcb0986e4457854a00a303477f565d"},{"author":{"_account_id":11105,"name":"Swapnil Kulkarni","email":"coolsvap@gmail.com","username":"coolsvap"},"change_message_id":"9f9b3aa87787aefececc86dd26dcc6f6151fc7be","unresolved":false,"context_lines":[{"line_number":199,"context_line":"%}"},{"line_number":200,"context_line":""},{"line_number":201,"context_line":"{% if base_distro \u003d\u003d \u0027ubuntu\u0027 %}"},{"line_number":202,"context_line":"    {% set keys \u003d ["},{"line_number":203,"context_line":"      \u0027199369E5404BD5FC7D2FE43BCBCB082A1BB943DB\u0027,"},{"line_number":204,"context_line":"      \u0027391A9AA2147192839E9DB0315EDB1B62EC4926EA\u0027,"},{"line_number":205,"context_line":"      \u0027430BDF5C56E7C94E848EE60C1C4CBDCDCD2EFD2A\u0027,"}],"source_content_type":"text/x-jinja2","patch_set":19,"id":"dab17558_c9414365","line":202,"range":{"start_line":202,"start_character":10,"end_line":202,"end_character":11},"in_reply_to":"dab17558_de490433","updated":"2016-05-16 11:11:09.000000000","message":"I agree. I think depending on the change set which is merged first it will create merge conflicts.\n\nI suggested only because you\u0027re almost half way down the road. Np.","commit_id":"3df80d87e2dcb0986e4457854a00a303477f565d"},{"author":{"_account_id":1390,"name":"Paul Bourke","email":"pauldbourke@proton.me","username":"pauldbourke"},"change_message_id":"7e20b2b73a61dc52e5015cdbb5805b28dc842b3b","unresolved":false,"context_lines":[{"line_number":219,"context_line":"    %}"},{"line_number":220,"context_line":"{% endif %}"},{"line_number":221,"context_line":""},{"line_number":222,"context_line":"RUN true \\"},{"line_number":223,"context_line":"    {% for key in keys %}"},{"line_number":224,"context_line":"    \u0026\u0026 apt-key adv --recv-keys --keyserver hkp://keyserver.ubuntu.com:80 {{ key }} \\"},{"line_number":225,"context_line":"    {% endfor %}"}],"source_content_type":"text/x-jinja2","patch_set":20,"id":"9abb7d3a_ae67b7e7","line":222,"updated":"2016-05-31 10:33:25.000000000","message":"why this line?","commit_id":"97a416d131cf1677ead940512bf70f55a2c58b0e"},{"author":{"_account_id":7888,"name":"Benedikt Trefzer","email":"benedikt.trefzer@cirrax.com","username":"trefzer"},"change_message_id":"dbfdba3c42019689c29fadc5a559d43f5297ae63","unresolved":false,"context_lines":[{"line_number":219,"context_line":"    %}"},{"line_number":220,"context_line":"{% endif %}"},{"line_number":221,"context_line":""},{"line_number":222,"context_line":"RUN true \\"},{"line_number":223,"context_line":"    {% for key in keys %}"},{"line_number":224,"context_line":"    \u0026\u0026 apt-key adv --recv-keys --keyserver hkp://keyserver.ubuntu.com:80 {{ key }} \\"},{"line_number":225,"context_line":"    {% endfor %}"}],"source_content_type":"text/x-jinja2","patch_set":20,"id":"9abb7d3a_f2c6b998","line":222,"in_reply_to":"9abb7d3a_ae67b7e7","updated":"2016-05-31 12:36:29.000000000","message":"It\u0027s not possible to have RUN alone on a line, so I had to add a command in front of the jinja text.\nAdding the RUN command in the loop should be avoid.","commit_id":"97a416d131cf1677ead940512bf70f55a2c58b0e"},{"author":{"_account_id":10787,"name":"Michal Jastrzebski (inc0)","email":"michal.jastrzebski@intel.com","username":"inc0"},"change_message_id":"322539acd15b71c42ac54c0d51abf1ccf252d6e0","unresolved":false,"context_lines":[{"line_number":206,"context_line":"%}"},{"line_number":207,"context_line":""},{"line_number":208,"context_line":"{% if base_distro \u003d\u003d \u0027ubuntu\u0027 %}"},{"line_number":209,"context_line":"    {% set keys \u003d ["},{"line_number":210,"context_line":"      \u0027199369E5404BD5FC7D2FE43BCBCB082A1BB943DB\u0027,"},{"line_number":211,"context_line":"      \u0027391A9AA2147192839E9DB0315EDB1B62EC4926EA\u0027,"},{"line_number":212,"context_line":"      \u0027430BDF5C56E7C94E848EE60C1C4CBDCDCD2EFD2A\u0027,"}],"source_content_type":"text/x-jinja2","patch_set":23,"id":"9abb7d3a_bb1a807a","line":209,"updated":"2016-06-03 13:55:41.000000000","message":"I like this one!","commit_id":"30c805596f34fe8652875afdce57c4db31c0e032"}],"docker/base/sources.list.debian":[{"author":{"_account_id":13642,"name":"Vikram Hosakote","email":"vikram.ganesh@gmail.com","username":"vhosakot"},"change_message_id":"585f5e0d57b122a65e4b7c57dd5b5842b6c84263","unresolved":false,"context_lines":[{"line_number":9,"context_line":"deb http://security.debian.org jessie/updates main"},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"# Ceph repo"},{"line_number":12,"context_line":"deb http://download.ceph.com/debian-hammer/ jessie main"}],"source_content_type":"application/octet-stream","patch_set":2,"id":"7af24918_56671825","line":12,"range":{"start_line":12,"start_character":11,"end_line":12,"end_character":19},"updated":"2016-03-06 04:45:42.000000000","message":"Aren\u0027t the MariaDB, Percona and Elastic repos needed here as well for debian ?\n\nRefer docker/base/sources.list.","commit_id":"f3abc836fdb001606e05dffc24df15f0cac7e053"},{"author":{"_account_id":7888,"name":"Benedikt Trefzer","email":"benedikt.trefzer@cirrax.com","username":"trefzer"},"change_message_id":"224f83152b3bc3d31d4ca00024e93c2d77d58f80","unresolved":false,"context_lines":[{"line_number":9,"context_line":"deb http://security.debian.org jessie/updates main"},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"# Ceph repo"},{"line_number":12,"context_line":"deb http://download.ceph.com/debian-hammer/ jessie main"}],"source_content_type":"application/octet-stream","patch_set":2,"id":"7af24918_dc9fc94e","line":12,"range":{"start_line":12,"start_character":11,"end_line":12,"end_character":19},"in_reply_to":"7af24918_56671825","updated":"2016-03-06 08:47:22.000000000","message":"MariaDB yes (will add in next patch).\nI do not know yet for the others, could be.\nMy intention was to add as less repositories as necessary. So as times go by and I build more containers (till know I successfully build keystone and glance), more needs to be added.\n\nI also thought about adding debian-backports which has more recent openstack packages.","commit_id":"f3abc836fdb001606e05dffc24df15f0cac7e053"},{"author":{"_account_id":19300,"name":"Eric Lemoine","email":"elemoine@mirantis.com","username":"elemoine"},"change_message_id":"a07a1fbdc9027eb97a62a96cdb98edb36f7e2246","unresolved":false,"context_lines":[{"line_number":9,"context_line":"deb http://security.debian.org jessie/updates main"},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"# Ceph repo"},{"line_number":12,"context_line":"deb http://download.ceph.com/debian-hammer/ jessie main"}],"source_content_type":"application/octet-stream","patch_set":2,"id":"7af24918_7c62f517","line":12,"in_reply_to":"7af24918_dc9fc94e","updated":"2016-03-06 09:34:11.000000000","message":"The Elasticsearch repo is probably required for Elasticsearch 2.x.","commit_id":"f3abc836fdb001606e05dffc24df15f0cac7e053"},{"author":{"_account_id":16233,"name":"Mauricio Lima","email":"mauriciolimab@gmail.com","username":"limamauricio"},"change_message_id":"49148ead486854b534ffc05f7db97e3c8bb7ce0e","unresolved":false,"context_lines":[{"line_number":1,"context_line":""},{"line_number":2,"context_line":"# Default repos"},{"line_number":3,"context_line":"deb http://httpredir.debian.org/debian jessie main"},{"line_number":4,"context_line":""}],"source_content_type":"application/octet-stream","patch_set":4,"id":"fa0719c6_cae96482","line":1,"updated":"2016-03-22 19:09:51.000000000","message":"remove this blank line","commit_id":"eb8d1d1d03a9073aa579523a6f6e1077094f397c"},{"author":{"_account_id":7888,"name":"Benedikt Trefzer","email":"benedikt.trefzer@cirrax.com","username":"trefzer"},"change_message_id":"475c54cd89ba569812e8cab93d627cd08dc052de","unresolved":false,"context_lines":[{"line_number":1,"context_line":""},{"line_number":2,"context_line":"# Default repos"},{"line_number":3,"context_line":"deb http://httpredir.debian.org/debian jessie main"},{"line_number":4,"context_line":""}],"source_content_type":"application/octet-stream","patch_set":4,"id":"fa0719c6_604e7470","line":1,"in_reply_to":"fa0719c6_cae96482","updated":"2016-03-22 21:12:04.000000000","message":"Done","commit_id":"eb8d1d1d03a9073aa579523a6f6e1077094f397c"},{"author":{"_account_id":14119,"name":"Sam Yaple","email":"sam+git@yaple.net","username":"SamYaple"},"change_message_id":"6b44756ff8a8d2e5feb7bf18a2d65008afdae3ef","unresolved":false,"context_lines":[{"line_number":12,"context_line":"deb http://download.ceph.com/debian-hammer/ jessie main"},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"# Maria DB repo"},{"line_number":15,"context_line":"deb [arch\u003damd64,i386] http://nyc2.mirrors.digitalocean.com/mariadb/repo/10.0/debian jessie main"}],"source_content_type":"application/octet-stream","patch_set":4,"id":"fa0719c6_88065ccc","line":15,"updated":"2016-03-22 17:11:27.000000000","message":"is i386 needed?","commit_id":"eb8d1d1d03a9073aa579523a6f6e1077094f397c"},{"author":{"_account_id":7888,"name":"Benedikt Trefzer","email":"benedikt.trefzer@cirrax.com","username":"trefzer"},"change_message_id":"475c54cd89ba569812e8cab93d627cd08dc052de","unresolved":false,"context_lines":[{"line_number":12,"context_line":"deb http://download.ceph.com/debian-hammer/ jessie main"},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"# Maria DB repo"},{"line_number":15,"context_line":"deb [arch\u003damd64,i386] http://nyc2.mirrors.digitalocean.com/mariadb/repo/10.0/debian jessie main"}],"source_content_type":"application/octet-stream","patch_set":4,"id":"fa0719c6_20af5c82","line":15,"in_reply_to":"fa0719c6_88065ccc","updated":"2016-03-22 21:12:04.000000000","message":"this does not mean i386 packages are installed, but that they are available. Since this archive is only available for amd64 and i386 this part ensures, that the repository is ignored for other architectures, and no error is produced.\n\nalso see https://wiki.debian.org/Multiarch/HOWTO:","commit_id":"eb8d1d1d03a9073aa579523a6f6e1077094f397c"}],"docker/nova/nova-libvirt/Dockerfile.j2":[{"author":{"_account_id":16620,"name":"Alicja Kwasniewska","email":"alicja.kwasniewska@intel.com","username":"akwasnie"},"change_message_id":"c1a18509ecc2da08aa1735d1e17273bf68c843c5","unresolved":false,"context_lines":[{"line_number":20,"context_line":"{% if base_distro \u003d\u003d \u0027ubuntu\u0027 %}"},{"line_number":21,"context_line":"{% set packages \u003d"},{"line_number":22,"context_line":"      \u0027 xen-utils-4.6\u0027"},{"line_number":23,"context_line":"-%}"},{"line_number":24,"context_line":"{% elif base_distro \u003d\u003d \u0027debian\u0027 %}"},{"line_number":25,"context_line":"{% set packages \u003d"},{"line_number":26,"context_line":"      \u0027 xen-utils-4.4\u0027"}],"source_content_type":"text/x-jinja2","patch_set":16,"id":"9a061dce_e722d700","line":23,"range":{"start_line":23,"start_character":0,"end_line":23,"end_character":3},"updated":"2016-04-09 19:02:58.000000000","message":"please do not use \u0027-\u0027, as it\u0027s not used anywhere by us","commit_id":"4ea116f0b3940c4a07d74cc979a4e3277477e9b7"},{"author":{"_account_id":7888,"name":"Benedikt Trefzer","email":"benedikt.trefzer@cirrax.com","username":"trefzer"},"change_message_id":"b2d10b84b31368b445d6da08c29938c91fd2af38","unresolved":false,"context_lines":[{"line_number":20,"context_line":"{% if base_distro \u003d\u003d \u0027ubuntu\u0027 %}"},{"line_number":21,"context_line":"{% set packages \u003d"},{"line_number":22,"context_line":"      \u0027 xen-utils-4.6\u0027"},{"line_number":23,"context_line":"-%}"},{"line_number":24,"context_line":"{% elif base_distro \u003d\u003d \u0027debian\u0027 %}"},{"line_number":25,"context_line":"{% set packages \u003d"},{"line_number":26,"context_line":"      \u0027 xen-utils-4.4\u0027"}],"source_content_type":"text/x-jinja2","patch_set":16,"id":"9a061dce_274b2fb2","line":23,"range":{"start_line":23,"start_character":0,"end_line":23,"end_character":3},"in_reply_to":"9a061dce_e722d700","updated":"2016-04-09 19:17:12.000000000","message":"Done","commit_id":"4ea116f0b3940c4a07d74cc979a4e3277477e9b7"},{"author":{"_account_id":16620,"name":"Alicja Kwasniewska","email":"alicja.kwasniewska@intel.com","username":"akwasnie"},"change_message_id":"c1a18509ecc2da08aa1735d1e17273bf68c843c5","unresolved":false,"context_lines":[{"line_number":24,"context_line":"{% elif base_distro \u003d\u003d \u0027debian\u0027 %}"},{"line_number":25,"context_line":"{% set packages \u003d"},{"line_number":26,"context_line":"      \u0027 xen-utils-4.4\u0027"},{"line_number":27,"context_line":"-%}"},{"line_number":28,"context_line":"{% endif %}"},{"line_number":29,"context_line":""},{"line_number":30,"context_line":"RUN apt-get install -y --no-install-recommends \\"}],"source_content_type":"text/x-jinja2","patch_set":16,"id":"9a061dce_4720e3f7","line":27,"range":{"start_line":27,"start_character":0,"end_line":27,"end_character":3},"updated":"2016-04-09 19:02:58.000000000","message":"ditto","commit_id":"4ea116f0b3940c4a07d74cc979a4e3277477e9b7"},{"author":{"_account_id":7888,"name":"Benedikt Trefzer","email":"benedikt.trefzer@cirrax.com","username":"trefzer"},"change_message_id":"b2d10b84b31368b445d6da08c29938c91fd2af38","unresolved":false,"context_lines":[{"line_number":24,"context_line":"{% elif base_distro \u003d\u003d \u0027debian\u0027 %}"},{"line_number":25,"context_line":"{% set packages \u003d"},{"line_number":26,"context_line":"      \u0027 xen-utils-4.4\u0027"},{"line_number":27,"context_line":"-%}"},{"line_number":28,"context_line":"{% endif %}"},{"line_number":29,"context_line":""},{"line_number":30,"context_line":"RUN apt-get install -y --no-install-recommends \\"}],"source_content_type":"text/x-jinja2","patch_set":16,"id":"9a061dce_87c67b14","line":27,"range":{"start_line":27,"start_character":0,"end_line":27,"end_character":3},"in_reply_to":"9a061dce_4720e3f7","updated":"2016-04-09 19:17:12.000000000","message":"Done","commit_id":"4ea116f0b3940c4a07d74cc979a4e3277477e9b7"}]}