)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":13642,"name":"Vikram Hosakote","email":"vikram.ganesh@gmail.com","username":"vhosakot"},"change_message_id":"0e92247699a4423874c29d90cbb49c387881801b","unresolved":false,"context_lines":[{"line_number":5,"context_line":"CommitDate: 2016-03-25 16:47:56 +0100"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"use silent option for curl download in heka container"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"TrivialFix"},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"Change-Id: I54e14d6bb4966de9c1651712e52652fee7cc4855"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"da0c15f0_0938c7d2","line":8,"updated":"2016-03-25 16:12:01.000000000","message":"Could you please add the reason / error / bug ID here so that reviewers know why this is done.","commit_id":"5001c65d7023b1756d11cb7386cd17f5749ae315"},{"author":{"_account_id":7888,"name":"Benedikt Trefzer","email":"benedikt.trefzer@cirrax.com","username":"trefzer"},"change_message_id":"944dc1f46fcf8ba8480e781957ecca489415800e","unresolved":false,"context_lines":[{"line_number":5,"context_line":"CommitDate: 2016-03-25 16:47:56 +0100"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"use silent option for curl download in heka container"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"TrivialFix"},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"Change-Id: I54e14d6bb4966de9c1651712e52652fee7cc4855"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"da0c15f0_898df733","line":8,"in_reply_to":"da0c15f0_0938c7d2","updated":"2016-03-25 16:18:52.000000000","message":"Done","commit_id":"5001c65d7023b1756d11cb7386cd17f5749ae315"}],"docker/heka/Dockerfile.j2":[{"author":{"_account_id":7488,"name":"Jeffrey Zhang","email":"zhang.lei.fly@gmail.com","username":"Jeffrey4l"},"change_message_id":"14f74062d0888938099c3c9eb56b2c4ebef551b9","unresolved":false,"context_lines":[{"line_number":9,"context_line":""},{"line_number":10,"context_line":"{% elif base_distro in [\u0027ubuntu\u0027] %}"},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"RUN curl -s --location https://github.com/mozilla-services/heka/releases/download/v0.10.0/heka_0.10.0_amd64.deb -o heka_0.10.0_amd64.deb \\"},{"line_number":13,"context_line":"    \u0026\u0026 dpkg -i heka_0.10.0_amd64.deb \\"},{"line_number":14,"context_line":"    \u0026\u0026 rm -f heka_0.10.0_amd64.deb"},{"line_number":15,"context_line":""}],"source_content_type":"text/x-jinja2","patch_set":3,"id":"9a061dce_99b6e5da","line":12,"updated":"2016-04-05 03:02:16.000000000","message":"when use -s, please add -S too. moreover, please use long parameter rathen than short. It will be more easy to understand, especically to the new reviewer.\n\ncurl --show-error --silent --location http://\n\nI am always think should we add this feature to all the curl command in the Dockerfile?","commit_id":"96d64685ed67c814bc7c6d4d27d8ce289d3b567e"},{"author":{"_account_id":7888,"name":"Benedikt Trefzer","email":"benedikt.trefzer@cirrax.com","username":"trefzer"},"change_message_id":"2f2851850e4a3ede614a67f4a22a5076b407867d","unresolved":false,"context_lines":[{"line_number":9,"context_line":""},{"line_number":10,"context_line":"{% elif base_distro in [\u0027ubuntu\u0027] %}"},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"RUN curl -s --location https://github.com/mozilla-services/heka/releases/download/v0.10.0/heka_0.10.0_amd64.deb -o heka_0.10.0_amd64.deb \\"},{"line_number":13,"context_line":"    \u0026\u0026 dpkg -i heka_0.10.0_amd64.deb \\"},{"line_number":14,"context_line":"    \u0026\u0026 rm -f heka_0.10.0_amd64.deb"},{"line_number":15,"context_line":""}],"source_content_type":"text/x-jinja2","patch_set":3,"id":"9a061dce_6e798493","line":12,"in_reply_to":"9a061dce_99b6e5da","updated":"2016-04-05 06:10:16.000000000","message":"I fully agree, I will add separate patches for the other curl statements.","commit_id":"96d64685ed67c814bc7c6d4d27d8ce289d3b567e"}]}
