)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":11105,"name":"Swapnil Kulkarni","email":"coolsvap@gmail.com","username":"coolsvap"},"change_message_id":"d5eadd339a8990dda81c6bf4cac64826d1a9d84a","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Jeffrey Zhang \u003cjeffrey.zhang@99cloud.net\u003e"},{"line_number":5,"context_line":"CommitDate: 2016-06-07 23:29:34 +0800"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Implement Rally docker file"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Partially Implements: blueprint rally-container"},{"line_number":10,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"7aa08908_1caec897","line":7,"updated":"2016-06-08 11:36:20.000000000","message":"Dockerfile","commit_id":"17625b424b5227b9af3f7778f63476dc3d4bf2e6"}],"docker/rally/Dockerfile.j2":[{"author":{"_account_id":11105,"name":"Swapnil Kulkarni","email":"coolsvap@gmail.com","username":"coolsvap"},"change_message_id":"d5eadd339a8990dda81c6bf4cac64826d1a9d84a","unresolved":false,"context_lines":[{"line_number":11,"context_line":"    {% elif base_distro in [\u0027ubuntu\u0027] %}"},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"RUN apt-get -y install --no-install-recommends \\"},{"line_number":14,"context_line":"        tempest rally \\"},{"line_number":15,"context_line":"    \u0026\u0026 apt-get clean"},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"    {% endif %}"}],"source_content_type":"text/x-jinja2","patch_set":2,"id":"7aa08908_7cd0ac1f","line":14,"updated":"2016-06-08 11:36:20.000000000","message":"please split it on individual lines and arrange alphabetically.","commit_id":"17625b424b5227b9af3f7778f63476dc3d4bf2e6"},{"author":{"_account_id":11105,"name":"Swapnil Kulkarni","email":"coolsvap@gmail.com","username":"coolsvap"},"change_message_id":"62231578068ef74f7199379ab2580a5f6ba383a2","unresolved":false,"context_lines":[{"line_number":28,"context_line":""},{"line_number":29,"context_line":"{{ include_footer }}"},{"line_number":30,"context_line":""},{"line_number":31,"context_line":"USER rally"}],"source_content_type":"text/x-jinja2","patch_set":4,"id":"7aa08908_cdfbd777","line":31,"updated":"2016-06-08 15:28:20.000000000","message":"I think this will not work for binary containers. user rally will not be available.","commit_id":"865ac80937bfff8e9bc3859bd5a8ae7c3dc340a7"},{"author":{"_account_id":16233,"name":"Mauricio Lima","email":"mauriciolimab@gmail.com","username":"limamauricio"},"change_message_id":"683f974cf20861bdd1837e5199579c7d124a5042","unresolved":false,"context_lines":[{"line_number":7,"context_line":"RUN yum -y install \\"},{"line_number":8,"context_line":"        openstack-rally \\"},{"line_number":9,"context_line":"    \u0026\u0026 yum clean all \\"},{"line_number":10,"context_line":"    \u0026\u0026 useradd --user-group rally \\"},{"line_number":11,"context_line":"    \u0026\u0026 usermod -a -G kolla rally"},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"    {% elif base_distro in [\u0027ubuntu\u0027] %}"},{"line_number":14,"context_line":""}],"source_content_type":"text/x-jinja2","patch_set":5,"id":"7aa08908_550805da","line":11,"range":{"start_line":10,"start_character":4,"end_line":11,"end_character":32},"updated":"2016-06-08 18:19:53.000000000","message":"Move it to and of file.","commit_id":"56be6148b03abf6e72b31b788432af90b8e7f85f"},{"author":{"_account_id":16233,"name":"Mauricio Lima","email":"mauriciolimab@gmail.com","username":"limamauricio"},"change_message_id":"683f974cf20861bdd1837e5199579c7d124a5042","unresolved":false,"context_lines":[{"line_number":15,"context_line":"RUN apt-get -y install --no-install-recommends \\"},{"line_number":16,"context_line":"        rally \\"},{"line_number":17,"context_line":"    \u0026\u0026 apt-get clean \\"},{"line_number":18,"context_line":"    \u0026\u0026 usermod -a -G kolla rally"},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"    {% endif %}"},{"line_number":21,"context_line":"{% elif install_type \u003d\u003d \u0027source\u0027 %}"}],"source_content_type":"text/x-jinja2","patch_set":5,"id":"7aa08908_151afd16","line":18,"range":{"start_line":18,"start_character":4,"end_line":18,"end_character":32},"updated":"2016-06-08 18:19:53.000000000","message":"if you move it to end of file, isn\u0027t needed have it here","commit_id":"56be6148b03abf6e72b31b788432af90b8e7f85f"},{"author":{"_account_id":7488,"name":"Jeffrey Zhang","email":"zhang.lei.fly@gmail.com","username":"Jeffrey4l"},"change_message_id":"225f33a1f1a7fefea0f4df96616de066ef20e530","unresolved":false,"context_lines":[{"line_number":15,"context_line":"RUN apt-get -y install --no-install-recommends \\"},{"line_number":16,"context_line":"        rally \\"},{"line_number":17,"context_line":"    \u0026\u0026 apt-get clean \\"},{"line_number":18,"context_line":"    \u0026\u0026 usermod -a -G kolla rally"},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"    {% endif %}"},{"line_number":21,"context_line":"{% elif install_type \u003d\u003d \u0027source\u0027 %}"}],"source_content_type":"text/x-jinja2","patch_set":5,"id":"7aa08908_3a827ac8","line":18,"updated":"2016-06-08 22:51:40.000000000","message":"in this way, we can reduce 1 layer of the docker image. So I put them\nhere.","commit_id":"56be6148b03abf6e72b31b788432af90b8e7f85f"},{"author":{"_account_id":11105,"name":"Swapnil Kulkarni","email":"coolsvap@gmail.com","username":"coolsvap"},"change_message_id":"046a967c326f5c7db5e46885c4925359adf00328","unresolved":false,"context_lines":[{"line_number":24,"context_line":"RUN ln -s rally-source/* rally \\"},{"line_number":25,"context_line":"    \u0026\u0026 /var/lib/kolla/venv/bin/pip --no-cache-dir install --upgrade -c requirements/upper-constraints.txt /rally \\"},{"line_number":26,"context_line":"    \u0026\u0026 mkdir -p /etc/rally /var/log/rally \\"},{"line_number":27,"context_line":"    \u0026\u0026 useradd --user-group rally \\"},{"line_number":28,"context_line":"    \u0026\u0026 usermod -a -G kolla rally"},{"line_number":29,"context_line":""},{"line_number":30,"context_line":"{% endif %}"},{"line_number":31,"context_line":""}],"source_content_type":"text/x-jinja2","patch_set":5,"id":"7aa08908_136980f9","line":28,"range":{"start_line":27,"start_character":1,"end_line":28,"end_character":32},"updated":"2016-06-08 15:56:13.000000000","message":"Can we just move useradd and usermod steps to end of file?","commit_id":"56be6148b03abf6e72b31b788432af90b8e7f85f"}]}