)]}'
{"docker/ceilometer/ceilometer-base/Dockerfile.j2":[{"author":{"_account_id":19316,"name":"Eduardo Gonzalez","email":"dabarren@gmail.com","username":"egonzalez90"},"change_message_id":"1cd7cb8bb5cea54d3f8fa9c16b242d2580a756bf","unresolved":false,"context_lines":[{"line_number":14,"context_line":"            \u0027python-gnocchiclient\u0027,"},{"line_number":15,"context_line":"            \u0027python-tooz\u0027,"},{"line_number":16,"context_line":"            \u0027python-oslo-db\u0027,"},{"line_number":17,"context_line":"            \u0027python-pecan\u0027"},{"line_number":18,"context_line":"         ] %}"},{"line_number":19,"context_line":"    {% elif base_distro in [\u0027ubuntu\u0027] %}"},{"line_number":20,"context_line":"        {% set ceilometer_base_packages \u003d ["}],"source_content_type":"text/x-jinja2","patch_set":1,"id":"da36d5c6_5a671d25","line":17,"updated":"2017-02-28 10:03:42.000000000","message":"when I tested this last week, after installing pecan was complaing about python-wsme package too.\nAnyway, maybe is worth add ceilometer-api in ceilometer-base, as it contains all dependencies.","commit_id":"441d8a420b3c0ba80bab2091ffacddf92084a23d"},{"author":{"_account_id":22582,"name":"Duong Ha-Quang","email":"quangduongha@gmail.com","username":"duonghq"},"change_message_id":"adcd3508cc154d0e899548e81e1bd23359216d7e","unresolved":false,"context_lines":[{"line_number":14,"context_line":"            \u0027python-gnocchiclient\u0027,"},{"line_number":15,"context_line":"            \u0027python-tooz\u0027,"},{"line_number":16,"context_line":"            \u0027python-oslo-db\u0027,"},{"line_number":17,"context_line":"            \u0027python-pecan\u0027"},{"line_number":18,"context_line":"         ] %}"},{"line_number":19,"context_line":"    {% elif base_distro in [\u0027ubuntu\u0027] %}"},{"line_number":20,"context_line":"        {% set ceilometer_base_packages \u003d ["}],"source_content_type":"text/x-jinja2","patch_set":1,"id":"ba2be162_15302156","line":17,"in_reply_to":"da36d5c6_5a671d25","updated":"2017-03-01 02:07:29.000000000","message":"Done","commit_id":"441d8a420b3c0ba80bab2091ffacddf92084a23d"},{"author":{"_account_id":1390,"name":"Paul Bourke","email":"pauldbourke@proton.me","username":"pauldbourke"},"change_message_id":"dc41250bb2a08c1837d9f3fe8ec0388629a1f4eb","unresolved":false,"context_lines":[{"line_number":11,"context_line":"    {% if base_distro in [\u0027centos\u0027, \u0027oraclelinux\u0027, \u0027rhel\u0027] %}"},{"line_number":12,"context_line":"        {% set ceilometer_base_packages \u003d ["},{"line_number":13,"context_line":"            \u0027openstack-ceilometer-common\u0027,"},{"line_number":14,"context_line":"            \u0027python-gnocchiclient\u0027,"},{"line_number":15,"context_line":"            \u0027python-tooz\u0027,"},{"line_number":16,"context_line":"            \u0027python-oslo-db\u0027,"},{"line_number":17,"context_line":"            \u0027python-pecan\u0027,"},{"line_number":18,"context_line":"            \u0027python-wsme\u0027"},{"line_number":19,"context_line":"         ] %}"},{"line_number":20,"context_line":"    {% elif base_distro in [\u0027ubuntu\u0027] %}"},{"line_number":21,"context_line":"        {% set ceilometer_base_packages \u003d ["}],"source_content_type":"text/x-jinja2","patch_set":3,"id":"ba2be162_f6ec502b","line":18,"range":{"start_line":14,"start_character":13,"end_line":18,"end_character":25},"updated":"2017-03-01 17:37:27.000000000","message":"I think these should go in openstack-base, unless there\u0027s some explicit reason for breaking consistency with the rest of the images","commit_id":"dfe6c7d4693f6010585652274e2649dc8e8a41ae"},{"author":{"_account_id":7488,"name":"Jeffrey Zhang","email":"zhang.lei.fly@gmail.com","username":"Jeffrey4l"},"change_message_id":"dc255713ae95b6cdbce97a53acf51b01c1d531eb","unresolved":false,"context_lines":[{"line_number":15,"context_line":"            \u0027python-tooz\u0027,"},{"line_number":16,"context_line":"            \u0027python-oslo-db\u0027,"},{"line_number":17,"context_line":"            \u0027python-pecan\u0027,"},{"line_number":18,"context_line":"            \u0027python-wsme\u0027"},{"line_number":19,"context_line":"         ] %}"},{"line_number":20,"context_line":"    {% elif base_distro in [\u0027ubuntu\u0027] %}"},{"line_number":21,"context_line":"        {% set ceilometer_base_packages \u003d ["}],"source_content_type":"text/x-jinja2","patch_set":3,"id":"ba2be162_7fa79aa6","line":18,"updated":"2017-03-06 15:37:52.000000000","message":"yes, add TODO is better.","commit_id":"dfe6c7d4693f6010585652274e2649dc8e8a41ae"},{"author":{"_account_id":7488,"name":"Jeffrey Zhang","email":"zhang.lei.fly@gmail.com","username":"Jeffrey4l"},"change_message_id":"9ec4db1a95e3a79951b19feb4a378a3a7d6afc69","unresolved":false,"context_lines":[{"line_number":11,"context_line":"    {% if base_distro in [\u0027centos\u0027, \u0027oraclelinux\u0027, \u0027rhel\u0027] %}"},{"line_number":12,"context_line":"        {% set ceilometer_base_packages \u003d ["},{"line_number":13,"context_line":"            \u0027openstack-ceilometer-common\u0027,"},{"line_number":14,"context_line":"            \u0027python-gnocchiclient\u0027,"},{"line_number":15,"context_line":"            \u0027python-tooz\u0027,"},{"line_number":16,"context_line":"            \u0027python-oslo-db\u0027,"},{"line_number":17,"context_line":"            \u0027python-pecan\u0027,"},{"line_number":18,"context_line":"            \u0027python-wsme\u0027"},{"line_number":19,"context_line":"         ] %}"},{"line_number":20,"context_line":"    {% elif base_distro in [\u0027ubuntu\u0027] %}"},{"line_number":21,"context_line":"        {% set ceilometer_base_packages \u003d ["}],"source_content_type":"text/x-jinja2","patch_set":3,"id":"ba2be162_9c570e25","line":18,"range":{"start_line":14,"start_character":13,"end_line":18,"end_character":25},"in_reply_to":"ba2be162_20867eb9","updated":"2017-03-06 09:10:51.000000000","message":"This is a bug in RDO actually. So add them here is correct.","commit_id":"dfe6c7d4693f6010585652274e2649dc8e8a41ae"},{"author":{"_account_id":1390,"name":"Paul Bourke","email":"pauldbourke@proton.me","username":"pauldbourke"},"change_message_id":"b6584a37d7d23f7194288a985bfc91f067f4ccd0","unresolved":false,"context_lines":[{"line_number":11,"context_line":"    {% if base_distro in [\u0027centos\u0027, \u0027oraclelinux\u0027, \u0027rhel\u0027] %}"},{"line_number":12,"context_line":"        {% set ceilometer_base_packages \u003d ["},{"line_number":13,"context_line":"            \u0027openstack-ceilometer-common\u0027,"},{"line_number":14,"context_line":"            \u0027python-gnocchiclient\u0027,"},{"line_number":15,"context_line":"            \u0027python-tooz\u0027,"},{"line_number":16,"context_line":"            \u0027python-oslo-db\u0027,"},{"line_number":17,"context_line":"            \u0027python-pecan\u0027,"},{"line_number":18,"context_line":"            \u0027python-wsme\u0027"},{"line_number":19,"context_line":"         ] %}"},{"line_number":20,"context_line":"    {% elif base_distro in [\u0027ubuntu\u0027] %}"},{"line_number":21,"context_line":"        {% set ceilometer_base_packages \u003d ["}],"source_content_type":"text/x-jinja2","patch_set":3,"id":"ba2be162_bf0af2cf","line":18,"range":{"start_line":14,"start_character":13,"end_line":18,"end_character":25},"in_reply_to":"ba2be162_9c570e25","updated":"2017-03-06 15:34:34.000000000","message":"Please add a TODO then with a reference to the RDO bug if possible","commit_id":"dfe6c7d4693f6010585652274e2649dc8e8a41ae"},{"author":{"_account_id":19316,"name":"Eduardo Gonzalez","email":"dabarren@gmail.com","username":"egonzalez90"},"change_message_id":"9c6bf37d0939b827e034011ee49591379b83a506","unresolved":false,"context_lines":[{"line_number":11,"context_line":"    {% if base_distro in [\u0027centos\u0027, \u0027oraclelinux\u0027, \u0027rhel\u0027] %}"},{"line_number":12,"context_line":"        {% set ceilometer_base_packages \u003d ["},{"line_number":13,"context_line":"            \u0027openstack-ceilometer-common\u0027,"},{"line_number":14,"context_line":"            \u0027python-gnocchiclient\u0027,"},{"line_number":15,"context_line":"            \u0027python-tooz\u0027,"},{"line_number":16,"context_line":"            \u0027python-oslo-db\u0027,"},{"line_number":17,"context_line":"            \u0027python-pecan\u0027,"},{"line_number":18,"context_line":"            \u0027python-wsme\u0027"},{"line_number":19,"context_line":"         ] %}"},{"line_number":20,"context_line":"    {% elif base_distro in [\u0027ubuntu\u0027] %}"},{"line_number":21,"context_line":"        {% set ceilometer_base_packages \u003d ["}],"source_content_type":"text/x-jinja2","patch_set":3,"id":"ba2be162_20867eb9","line":18,"range":{"start_line":14,"start_character":13,"end_line":18,"end_character":25},"in_reply_to":"ba2be162_e0b466d2","updated":"2017-03-02 09:50:47.000000000","message":"Yes this is only ceilometer specific, also this change must be fixed in RDO community side as is a dependency issue between services, but this is a workaround for kolla for now.\nI\u0027m wondering what changed in the packages to miss this dependencies as in newton was working fine.","commit_id":"dfe6c7d4693f6010585652274e2649dc8e8a41ae"},{"author":{"_account_id":22582,"name":"Duong Ha-Quang","email":"quangduongha@gmail.com","username":"duonghq"},"change_message_id":"036fad4943ea36189fdc368b92a14a927397e522","unresolved":false,"context_lines":[{"line_number":11,"context_line":"    {% if base_distro in [\u0027centos\u0027, \u0027oraclelinux\u0027, \u0027rhel\u0027] %}"},{"line_number":12,"context_line":"        {% set ceilometer_base_packages \u003d ["},{"line_number":13,"context_line":"            \u0027openstack-ceilometer-common\u0027,"},{"line_number":14,"context_line":"            \u0027python-gnocchiclient\u0027,"},{"line_number":15,"context_line":"            \u0027python-tooz\u0027,"},{"line_number":16,"context_line":"            \u0027python-oslo-db\u0027,"},{"line_number":17,"context_line":"            \u0027python-pecan\u0027,"},{"line_number":18,"context_line":"            \u0027python-wsme\u0027"},{"line_number":19,"context_line":"         ] %}"},{"line_number":20,"context_line":"    {% elif base_distro in [\u0027ubuntu\u0027] %}"},{"line_number":21,"context_line":"        {% set ceilometer_base_packages \u003d ["}],"source_content_type":"text/x-jinja2","patch_set":3,"id":"ba2be162_e0b466d2","line":18,"range":{"start_line":14,"start_character":13,"end_line":18,"end_character":25},"in_reply_to":"ba2be162_f6ec502b","updated":"2017-03-02 09:46:31.000000000","message":"These packages are specific to ceilometer, I always do not like to put the burden to openstack-base (although we was do this all the time).\n\nEspecially, last 2 packages added in this ps is due to fault of package maintainer.","commit_id":"dfe6c7d4693f6010585652274e2649dc8e8a41ae"},{"author":{"_account_id":1390,"name":"Paul Bourke","email":"pauldbourke@proton.me","username":"pauldbourke"},"change_message_id":"dc41250bb2a08c1837d9f3fe8ec0388629a1f4eb","unresolved":false,"context_lines":[{"line_number":20,"context_line":"    {% elif base_distro in [\u0027ubuntu\u0027] %}"},{"line_number":21,"context_line":"        {% set ceilometer_base_packages \u003d ["},{"line_number":22,"context_line":"            \u0027ceilometer-common\u0027,"},{"line_number":23,"context_line":"            \u0027python-gnocchiclient\u0027,"},{"line_number":24,"context_line":"            \u0027python-pymongo\u0027,"},{"line_number":25,"context_line":"            \u0027python-pecan\u0027,"},{"line_number":26,"context_line":"            \u0027python-wsme\u0027"},{"line_number":27,"context_line":"         ] %}"},{"line_number":28,"context_line":"    {% endif %}"},{"line_number":29,"context_line":""}],"source_content_type":"text/x-jinja2","patch_set":3,"id":"ba2be162_36f7c8f6","line":26,"range":{"start_line":23,"start_character":12,"end_line":26,"end_character":25},"updated":"2017-03-01 17:37:27.000000000","message":"same","commit_id":"dfe6c7d4693f6010585652274e2649dc8e8a41ae"},{"author":{"_account_id":7488,"name":"Jeffrey Zhang","email":"zhang.lei.fly@gmail.com","username":"Jeffrey4l"},"change_message_id":"d0164a7bc906010beb3e7b942407e55ac3565d54","unresolved":false,"context_lines":[{"line_number":20,"context_line":"    {% elif base_distro in [\u0027ubuntu\u0027] %}"},{"line_number":21,"context_line":"        {% set ceilometer_base_packages \u003d ["},{"line_number":22,"context_line":"            \u0027ceilometer-common\u0027,"},{"line_number":23,"context_line":"            \u0027python-gnocchiclient\u0027,"},{"line_number":24,"context_line":"            \u0027python-pymongo\u0027,"},{"line_number":25,"context_line":"            \u0027python-pecan\u0027,"},{"line_number":26,"context_line":"            \u0027python-wsme\u0027"},{"line_number":27,"context_line":"         ] %}"},{"line_number":28,"context_line":"    {% endif %}"},{"line_number":29,"context_line":""}],"source_content_type":"text/x-jinja2","patch_set":3,"id":"9a30ddce_109e8556","line":26,"range":{"start_line":23,"start_character":12,"end_line":26,"end_character":25},"in_reply_to":"ba2be162_36f7c8f6","updated":"2017-03-07 02:03:33.000000000","message":"Ubuntu is correct here. when installing ceilometer-common, pecan and wsme is installed.\n\ncheck http://paste.openstack.org/show/601682/","commit_id":"dfe6c7d4693f6010585652274e2649dc8e8a41ae"},{"author":{"_account_id":7488,"name":"Jeffrey Zhang","email":"zhang.lei.fly@gmail.com","username":"Jeffrey4l"},"change_message_id":"f6753c4e88515b3a455b9f912672a4739961dbe4","unresolved":false,"context_lines":[{"line_number":15,"context_line":"            \u0027python-tooz\u0027,"},{"line_number":16,"context_line":"            \u0027python-oslo-db\u0027,"},{"line_number":17,"context_line":"# TODO(duonghq) remove 2 following packages when RDO fix is merged"},{"line_number":18,"context_line":"# https://review.rdoproject.org/r/#/c/5633/"},{"line_number":19,"context_line":"            \u0027python-pecan\u0027,"},{"line_number":20,"context_line":"            \u0027python-wsme\u0027"},{"line_number":21,"context_line":"         ] %}"}],"source_content_type":"text/x-jinja2","patch_set":4,"id":"9a30ddce_3002a93f","line":18,"updated":"2017-03-07 02:12:33.000000000","message":"hrm just curiosity whether this inline comment works.","commit_id":"cf06f422c058207eb64054ceb99b0ea52781146e"},{"author":{"_account_id":7488,"name":"Jeffrey Zhang","email":"zhang.lei.fly@gmail.com","username":"Jeffrey4l"},"change_message_id":"f6753c4e88515b3a455b9f912672a4739961dbe4","unresolved":false,"context_lines":[{"line_number":27,"context_line":"# TODO(duonghq) remove 2 following packages when RDO fix is merged"},{"line_number":28,"context_line":"# https://review.rdoproject.org/r/#/c/5633/"},{"line_number":29,"context_line":"            \u0027python-pecan\u0027,"},{"line_number":30,"context_line":"            \u0027python-wsme\u0027"},{"line_number":31,"context_line":"         ] %}"},{"line_number":32,"context_line":"    {% endif %}"},{"line_number":33,"context_line":""}],"source_content_type":"text/x-jinja2","patch_set":4,"id":"9a30ddce_d0f22d6f","line":30,"updated":"2017-03-07 02:12:33.000000000","message":"ubuntu packages are correct. check comments here in last patch.","commit_id":"cf06f422c058207eb64054ceb99b0ea52781146e"}]}