)]}'
{"docker/ceilometer/ceilometer-ipmi/Dockerfile.j2":[{"author":{"_account_id":19316,"name":"Eduardo Gonzalez","email":"dabarren@gmail.com","username":"egonzalez90"},"change_message_id":"56bc624db247cb5d41973543669e3bbe5b3fa0c0","unresolved":false,"context_lines":[{"line_number":1,"context_line":"FROM {{ namespace }}/{{ image_prefix }}ceilometer-base:{{ tag }}"},{"line_number":2,"context_line":"MAINTAINER {{ maintainer }}"},{"line_number":3,"context_line":""},{"line_number":4,"context_line":"{% block ceilometer_ipmi_header %}{% endblock %}"},{"line_number":5,"context_line":""}],"source_content_type":"text/x-jinja2","patch_set":4,"id":"3f044301_33152c9d","line":2,"range":{"start_line":2,"start_character":0,"end_line":2,"end_character":27},"updated":"2017-05-09 09:35:42.000000000","message":"new maintainer syntax is https://github.com/openstack/kolla/blob/master/docker/aodh/aodh-api/Dockerfile.j2#L2\n\nPlease, append whole line here","commit_id":"f1a12975f6652a0133795b266df190cdd85fd425"},{"author":{"_account_id":6924,"name":"Pradeep Kilambi","email":"pkilambi@redhat.com","username":"pkilambi"},"change_message_id":"3a664cd04b64d332a91045f2be7988568b6bddb2","unresolved":false,"context_lines":[{"line_number":1,"context_line":"FROM {{ namespace }}/{{ image_prefix }}ceilometer-base:{{ tag }}"},{"line_number":2,"context_line":"MAINTAINER {{ maintainer }}"},{"line_number":3,"context_line":""},{"line_number":4,"context_line":"{% block ceilometer_ipmi_header %}{% endblock %}"},{"line_number":5,"context_line":""}],"source_content_type":"text/x-jinja2","patch_set":4,"id":"3f044301_64f1accd","line":2,"range":{"start_line":2,"start_character":0,"end_line":2,"end_character":27},"in_reply_to":"3f044301_33152c9d","updated":"2017-05-09 12:29:54.000000000","message":"Done","commit_id":"f1a12975f6652a0133795b266df190cdd85fd425"},{"author":{"_account_id":7488,"name":"Jeffrey Zhang","email":"zhang.lei.fly@gmail.com","username":"Jeffrey4l"},"change_message_id":"edf3d5add8e4b8792dbe854d28368e23efff3397","unresolved":false,"context_lines":[{"line_number":1,"context_line":"FROM {{ namespace }}/{{ image_prefix }}ceilometer-base:{{ tag }}"},{"line_number":2,"context_line":"LABEL maintainer\u003d\"{{ maintainer }}\" name\u003d\"{{ image_name }}\" build-date\u003d\"{{ build_date }}\""},{"line_number":3,"context_line":""},{"line_number":4,"context_line":"{% block ceilometer_ipmi_header %}{% endblock %}"},{"line_number":5,"context_line":""}],"source_content_type":"text/x-jinja2","patch_set":5,"id":"3f044301_eb854f5b","line":2,"updated":"2017-05-12 09:04:33.000000000","message":"we are not use this in kolla right now. This may should be in another patch.","commit_id":"cd04dc84d0f82dfe01f10df9a2b1cdeee03ed169"},{"author":{"_account_id":19316,"name":"Eduardo Gonzalez","email":"dabarren@gmail.com","username":"egonzalez90"},"change_message_id":"3aac6155803bf0acc22dcc2c1d79870b15974103","unresolved":false,"context_lines":[{"line_number":1,"context_line":"FROM {{ namespace }}/{{ image_prefix }}ceilometer-base:{{ tag }}"},{"line_number":2,"context_line":"LABEL maintainer\u003d\"{{ maintainer }}\" name\u003d\"{{ image_name }}\" build-date\u003d\"{{ build_date }}\""},{"line_number":3,"context_line":""},{"line_number":4,"context_line":"{% block ceilometer_ipmi_header %}{% endblock %}"},{"line_number":5,"context_line":""}],"source_content_type":"text/x-jinja2","patch_set":5,"id":"1f013ff3_fd9c435e","line":2,"in_reply_to":"1f013ff3_21bc2a68","updated":"2017-05-17 09:03:31.000000000","message":"Yep, the new labels were introduced last week iirc","commit_id":"cd04dc84d0f82dfe01f10df9a2b1cdeee03ed169"},{"author":{"_account_id":6924,"name":"Pradeep Kilambi","email":"pkilambi@redhat.com","username":"pkilambi"},"change_message_id":"8d0b08e268b2057d5612a291d89e547d3dd9eaa1","unresolved":false,"context_lines":[{"line_number":1,"context_line":"FROM {{ namespace }}/{{ image_prefix }}ceilometer-base:{{ tag }}"},{"line_number":2,"context_line":"LABEL maintainer\u003d\"{{ maintainer }}\" name\u003d\"{{ image_name }}\" build-date\u003d\"{{ build_date }}\""},{"line_number":3,"context_line":""},{"line_number":4,"context_line":"{% block ceilometer_ipmi_header %}{% endblock %}"},{"line_number":5,"context_line":""}],"source_content_type":"text/x-jinja2","patch_set":5,"id":"1f013ff3_21bc2a68","line":2,"in_reply_to":"3f044301_eb854f5b","updated":"2017-05-12 16:18:34.000000000","message":"I don\u0027t follow what you\u0027re trying to say here. Originally this was MAINTAINER {{ maintainer }} and Eduardo asked to chenge this to ^^. What do you mean by should be another path? why?","commit_id":"cd04dc84d0f82dfe01f10df9a2b1cdeee03ed169"},{"author":{"_account_id":19316,"name":"Eduardo Gonzalez","email":"dabarren@gmail.com","username":"egonzalez90"},"change_message_id":"3aac6155803bf0acc22dcc2c1d79870b15974103","unresolved":false,"context_lines":[{"line_number":5,"context_line":""},{"line_number":6,"context_line":"{% import \"macros.j2\" as macros with context %}"},{"line_number":7,"context_line":""},{"line_number":8,"context_line":"{% if install_type \u003d\u003d \u0027binary\u0027 %}"},{"line_number":9,"context_line":"    {% if base_distro in [\u0027centos\u0027, \u0027oraclelinux\u0027, \u0027rhel\u0027] %}"},{"line_number":10,"context_line":"        {% set ceilometer_ipmi_packages \u003d [\u0027openstack-ceilometer-ipmi\u0027] %}"},{"line_number":11,"context_line":"    {% elif base_distro in [\u0027debian\u0027, \u0027ubuntu\u0027] %}"}],"source_content_type":"text/x-jinja2","patch_set":5,"id":"1f013ff3_1de80fb2","line":8,"range":{"start_line":8,"start_character":0,"end_line":8,"end_character":33},"updated":"2017-05-17 09:03:31.000000000","message":"is OK to have this binary, source images are expected to be mostly empty (only USER, LABELS, etc). All ceilometer stuff for source is installed in ceilometer-base","commit_id":"cd04dc84d0f82dfe01f10df9a2b1cdeee03ed169"}]}