)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":22165,"name":"caoyuan","email":"cao.yingjunz@gmail.com","username":"caoyuan"},"change_message_id":"55510a12135d1f9b8405ff4d9a21a2e099b4ca43","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Add a package for freezer"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"freezer need the trickle package  to limit the traffic speed"},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"refer to"},{"line_number":12,"context_line":"https://github.com/openstack/freezer/blob/ebc4e8a57c6b0f8670f075d646c2665d855ee10d/freezer/common/config.py#L692"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"9f6a8fd7_89ba776b","line":9,"range":{"start_line":9,"start_character":33,"end_line":9,"end_character":34},"updated":"2018-04-23 14:29:16.000000000","message":"unnecessary space","commit_id":"b0d87510fa4861f806cd973cfdab6528c8ce52ab"}],"docker/freezer/freezer-base/Dockerfile.j2":[{"author":{"_account_id":19316,"name":"Eduardo Gonzalez","email":"dabarren@gmail.com","username":"egonzalez90"},"change_message_id":"258ed2f054721b25d0e35888c1e6fd9ca190f0d4","unresolved":false,"context_lines":[{"line_number":7,"context_line":""},{"line_number":8,"context_line":"{{ macros.configure_user(name\u003d\u0027freezer\u0027) }}"},{"line_number":9,"context_line":""},{"line_number":10,"context_line":"{% set freezer_base_packages \u003d ["},{"line_number":11,"context_line":"    \u0027trickle\u0027"},{"line_number":12,"context_line":"] %}"},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"{{ macros.install_packages(freezer_base_packages | customizable(\"packages\")) }}"},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"{% if install_type \u003d\u003d \u0027binary\u0027 %}"},{"line_number":17,"context_line":""}],"source_content_type":"text/x-jinja2","patch_set":2,"id":"9f6a8fd7_20a8278a","line":14,"range":{"start_line":10,"start_character":0,"end_line":14,"end_character":79},"updated":"2018-04-23 08:55:02.000000000","message":"please, move it inside source install code block, binary does not have packages, lets not install useless packages when the image is not being  built","commit_id":"014847ca33b3b0b3defeec4f632f8f1df97f12ce"},{"author":{"_account_id":22165,"name":"caoyuan","email":"cao.yingjunz@gmail.com","username":"caoyuan"},"change_message_id":"4dcb3d92c447d19f7c8622d34fb0953a1b0c264c","unresolved":false,"context_lines":[{"line_number":7,"context_line":""},{"line_number":8,"context_line":"{{ macros.configure_user(name\u003d\u0027freezer\u0027) }}"},{"line_number":9,"context_line":""},{"line_number":10,"context_line":"{% set freezer_base_packages \u003d ["},{"line_number":11,"context_line":"    \u0027trickle\u0027"},{"line_number":12,"context_line":"] %}"},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"{{ macros.install_packages(freezer_base_packages | customizable(\"packages\")) }}"},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"{% if install_type \u003d\u003d \u0027binary\u0027 %}"},{"line_number":17,"context_line":""}],"source_content_type":"text/x-jinja2","patch_set":2,"id":"9f6a8fd7_98a16349","line":14,"range":{"start_line":10,"start_character":0,"end_line":14,"end_character":79},"in_reply_to":"9f6a8fd7_20a8278a","updated":"2018-04-23 12:39:20.000000000","message":"agree","commit_id":"014847ca33b3b0b3defeec4f632f8f1df97f12ce"},{"author":{"_account_id":22165,"name":"caoyuan","email":"cao.yingjunz@gmail.com","username":"caoyuan"},"change_message_id":"55510a12135d1f9b8405ff4d9a21a2e099b4ca43","unresolved":false,"context_lines":[{"line_number":20,"context_line":"    \u0027/freezer\u0027"},{"line_number":21,"context_line":"] %}"},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"{% set freezer_base_packages \u003d ["},{"line_number":24,"context_line":"    \u0027trickle\u0027"},{"line_number":25,"context_line":"] %}"},{"line_number":26,"context_line":""},{"line_number":27,"context_line":"{{ macros.install_packages(freezer_base_packages | customizable(\"packages\")) }}"},{"line_number":28,"context_line":""},{"line_number":29,"context_line":"RUN ln -s freezer-base-source/* freezer \\"},{"line_number":30,"context_line":"    \u0026\u0026 mkdir -p /etc/freezer \\"}],"source_content_type":"text/x-jinja2","patch_set":3,"id":"9f6a8fd7_e98f73ca","line":27,"range":{"start_line":23,"start_character":0,"end_line":27,"end_character":79},"updated":"2018-04-23 14:29:16.000000000","message":"I prepare move this top to line 16, to keep consistence with the others refer to https://github.com/openstack/kolla/blob/master/docker/cinder/cinder-base/Dockerfile.j2#L32","commit_id":"b0d87510fa4861f806cd973cfdab6528c8ce52ab"}]}