)]}'
{"docker/collectd/Dockerfile.j2":[{"author":{"_account_id":30491,"name":"Radosław Piliszek","display_name":"Radek","email":"radek@piliszek.it","username":"yoctozepto","status":"self-employed techologist, collaborating mostly with 7bulls.com"},"change_message_id":"8bfd24b2961bc1db0bac04e3c6724f4ee10b5df0","unresolved":false,"context_lines":[{"line_number":82,"context_line":"    {% else %}"},{"line_number":83,"context_line":"      {% set collectd_packages \u003d collectd_packages + ["},{"line_number":84,"context_line":"        \u0027collectd-hugepages\u0027,"},{"line_number":85,"context_line":"        \u0027collectd-intel_pmu\u0027,"},{"line_number":86,"context_line":"        \u0027collectd-rdt\u0027,"},{"line_number":87,"context_line":"        \u0027collectd-turbostat\u0027"},{"line_number":88,"context_line":"      ] %}"}],"source_content_type":"text/x-jinja2","patch_set":1,"id":"3fa7e38b_a6b3a698","line":85,"updated":"2020-01-28 13:13:03.000000000","message":"soon irrelevant due to branch above taking control","commit_id":"1cae846b00daa9b918eb72799165e214722e72e1"},{"author":{"_account_id":4264,"name":"Matthias Runge","email":"mrunge@redhat.com","username":"mrunge"},"change_message_id":"2e8305e48ac885516aff2d396a6d6a6d48afa2e9","unresolved":false,"context_lines":[{"line_number":82,"context_line":"    {% else %}"},{"line_number":83,"context_line":"      {% set collectd_packages \u003d collectd_packages + ["},{"line_number":84,"context_line":"        \u0027collectd-hugepages\u0027,"},{"line_number":85,"context_line":"        \u0027collectd-intel_pmu\u0027,"},{"line_number":86,"context_line":"        \u0027collectd-rdt\u0027,"},{"line_number":87,"context_line":"        \u0027collectd-turbostat\u0027"},{"line_number":88,"context_line":"      ] %}"}],"source_content_type":"text/x-jinja2","patch_set":1,"id":"3fa7e38b_f8e36b01","line":85,"in_reply_to":"3fa7e38b_092e4d8c","updated":"2020-01-29 08:37:49.000000000","message":"iirc this was/is due to different package sources. centos7 does not have python3, for example.\n\nFor centos based packages, I would prefer to make sure we pull packages from centos-opstools repository. However, packages on centos 8 are not available yet (see openstack-dev mailing list for that).\nThe alternative source EPEL uses different package names and also builds differents subpackages.","commit_id":"1cae846b00daa9b918eb72799165e214722e72e1"},{"author":{"_account_id":30491,"name":"Radosław Piliszek","display_name":"Radek","email":"radek@piliszek.it","username":"yoctozepto","status":"self-employed techologist, collaborating mostly with 7bulls.com"},"change_message_id":"0f315453816143d57b5e61a7a1a11f73d1ff6229","unresolved":false,"context_lines":[{"line_number":82,"context_line":"    {% else %}"},{"line_number":83,"context_line":"      {% set collectd_packages \u003d collectd_packages + ["},{"line_number":84,"context_line":"        \u0027collectd-hugepages\u0027,"},{"line_number":85,"context_line":"        \u0027collectd-intel_pmu\u0027,"},{"line_number":86,"context_line":"        \u0027collectd-rdt\u0027,"},{"line_number":87,"context_line":"        \u0027collectd-turbostat\u0027"},{"line_number":88,"context_line":"      ] %}"}],"source_content_type":"text/x-jinja2","patch_set":1,"id":"3fa7e38b_092e4d8c","line":85,"in_reply_to":"3fa7e38b_86330a12","updated":"2020-01-28 13:25:34.000000000","message":"if it\u0027s available - I am not sure about the reason for this branching","commit_id":"1cae846b00daa9b918eb72799165e214722e72e1"},{"author":{"_account_id":31539,"name":"Piotr Rabiega","email":"piotrx.rabiega@intel.com","username":"prabiegx"},"change_message_id":"f82c5c6b6a019d87e054b1c701c4b23e16e28c04","unresolved":false,"context_lines":[{"line_number":82,"context_line":"    {% else %}"},{"line_number":83,"context_line":"      {% set collectd_packages \u003d collectd_packages + ["},{"line_number":84,"context_line":"        \u0027collectd-hugepages\u0027,"},{"line_number":85,"context_line":"        \u0027collectd-intel_pmu\u0027,"},{"line_number":86,"context_line":"        \u0027collectd-rdt\u0027,"},{"line_number":87,"context_line":"        \u0027collectd-turbostat\u0027"},{"line_number":88,"context_line":"      ] %}"}],"source_content_type":"text/x-jinja2","patch_set":1,"id":"3fa7e38b_86330a12","line":85,"in_reply_to":"3fa7e38b_a6b3a698","updated":"2020-01-28 13:18:58.000000000","message":"Should it also be added above to branch with Python 3.x?","commit_id":"1cae846b00daa9b918eb72799165e214722e72e1"},{"author":{"_account_id":4264,"name":"Matthias Runge","email":"mrunge@redhat.com","username":"mrunge"},"change_message_id":"c9e5414bba6e6ca79101fd022803f0be11f6acc7","unresolved":false,"context_lines":[{"line_number":78,"context_line":"      {# NOTE(aschultz): there are some missing packages here #}"},{"line_number":79,"context_line":"      {% set collectd_packages \u003d collectd_packages + ["},{"line_number":80,"context_line":"        \u0027collectd-hugepages\u0027,"},{"line_number":81,"context_line":"        \u0027collectd-pmu\u0027"},{"line_number":82,"context_line":"      ] %}"},{"line_number":83,"context_line":"    {% else %}"},{"line_number":84,"context_line":"      {% set collectd_packages \u003d collectd_packages + ["}],"source_content_type":"text/x-jinja2","patch_set":2,"id":"3fa7e38b_6de51b3d","line":81,"updated":"2020-02-05 11:32:22.000000000","message":"this here is the centos/rhel8 version. I suspect leaving this out here will cause the build to succeed.","commit_id":"4ce116d6fdd62c4922ce7f0af00a162883a89539"},{"author":{"_account_id":31539,"name":"Piotr Rabiega","email":"piotrx.rabiega@intel.com","username":"prabiegx"},"change_message_id":"dd40ce68e3a12d15cb26c585ae75c46eebcd5a5c","unresolved":false,"context_lines":[{"line_number":78,"context_line":"      {# NOTE(aschultz): there are some missing packages here #}"},{"line_number":79,"context_line":"      {% set collectd_packages \u003d collectd_packages + ["},{"line_number":80,"context_line":"        \u0027collectd-hugepages\u0027,"},{"line_number":81,"context_line":"        \u0027collectd-pmu\u0027"},{"line_number":82,"context_line":"      ] %}"},{"line_number":83,"context_line":"    {% else %}"},{"line_number":84,"context_line":"      {% set collectd_packages \u003d collectd_packages + ["}],"source_content_type":"text/x-jinja2","patch_set":2,"id":"3fa7e38b_ed158be8","line":81,"in_reply_to":"3fa7e38b_6de51b3d","updated":"2020-02-05 11:39:43.000000000","message":"Could you please clarify? Do you mean that removing this line should make tripleo-build-containers-rhel-8-master build succeed? If so, is it OK to remove it, or some work should be done to make it work without removing this line?","commit_id":"4ce116d6fdd62c4922ce7f0af00a162883a89539"}]}