)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":9712,"name":"Dougal Matthews","email":"dougal.matthews@canonical.com","username":"dougal"},"change_message_id":"d02a9d2b650ce40cb6df4dfff27e73829bfcaeb5","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Andras Kovi \u003cakovi@nokia.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2017-06-13 17:09:04 +0200"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"update docker build"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Change-Id: I803d69ee17e7f5ebc95ec2c81887c4f580d73715"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"7f231b9d_4de9b6e3","line":7,"updated":"2017-06-19 08:21:22.000000000","message":"I agree with Renat - please tell us more details about what you changed and why you changed them.","commit_id":"83684913aa3c8bddcbc47ee0f9adcaced4ae4cb9"}],"tools/docker/Dockerfile":[{"author":{"_account_id":8731,"name":"Renat Akhmerov","email":"renat.akhmerov@gmail.com","username":"rakhmerov"},"change_message_id":"c3d1520016f2d3e8354d5bcda7b62f937db4bca8","unresolved":false,"context_lines":[{"line_number":1,"context_line":"FROM krallin/ubuntu-tini:16.04"},{"line_number":2,"context_line":""},{"line_number":3,"context_line":"MAINTAINER hardik.parekh@nectechnologies.in"},{"line_number":4,"context_line":""},{"line_number":5,"context_line":"RUN export DEBIAN_FRONTEND\u003dnoninteractive \u0026\u0026 \\"},{"line_number":6,"context_line":"    apt-get -qq update \u0026\u0026 \\"}],"source_content_type":"text/x-dockerfile","patch_set":2,"id":"3f1d235d_3fb8a428","line":3,"range":{"start_line":3,"start_character":11,"end_line":3,"end_character":43},"updated":"2017-06-30 07:35:05.000000000","message":"Maybe it\u0027s worth changing maintainer\u0027s name to Andras Kovi? )","commit_id":"8f0eb1b6e1bb1844ee8ae16355b727bf05ec30ca"},{"author":{"_account_id":21970,"name":"Andras Kovi","email":"akovi@nokia.com","username":"andras.kovi"},"change_message_id":"8a97d39cc45a60c8f0ddeffc9b1c15cb77cad8b8","unresolved":false,"context_lines":[{"line_number":1,"context_line":"FROM krallin/ubuntu-tini:16.04"},{"line_number":2,"context_line":""},{"line_number":3,"context_line":"MAINTAINER hardik.parekh@nectechnologies.in"},{"line_number":4,"context_line":""},{"line_number":5,"context_line":"RUN export DEBIAN_FRONTEND\u003dnoninteractive \u0026\u0026 \\"},{"line_number":6,"context_line":"    apt-get -qq update \u0026\u0026 \\"}],"source_content_type":"text/x-dockerfile","patch_set":2,"id":"3f1d235d_d36f67c3","line":3,"range":{"start_line":3,"start_character":11,"end_line":3,"end_character":43},"in_reply_to":"3f1d235d_3fb8a428","updated":"2017-06-30 07:42:38.000000000","message":"Done","commit_id":"8f0eb1b6e1bb1844ee8ae16355b727bf05ec30ca"},{"author":{"_account_id":8731,"name":"Renat Akhmerov","email":"renat.akhmerov@gmail.com","username":"rakhmerov"},"change_message_id":"c3d1520016f2d3e8354d5bcda7b62f937db4bca8","unresolved":false,"context_lines":[{"line_number":48,"context_line":"    $INI_SET oslo_policy policy_file /opt/stack/mistral/etc/policy.json \u0026\u0026 \\"},{"line_number":49,"context_line":"    $INI_SET pecan auth_enable false"},{"line_number":50,"context_line":""},{"line_number":51,"context_line":"# Install MySQL libs"},{"line_number":52,"context_line":""},{"line_number":53,"context_line":""},{"line_number":54,"context_line":"EXPOSE 8989"}],"source_content_type":"text/x-dockerfile","patch_set":2,"id":"3f1d235d_7f973c92","line":51,"range":{"start_line":51,"start_character":0,"end_line":51,"end_character":20},"updated":"2017-06-30 07:35:05.000000000","message":"Is it left by mistake?","commit_id":"8f0eb1b6e1bb1844ee8ae16355b727bf05ec30ca"},{"author":{"_account_id":21970,"name":"Andras Kovi","email":"akovi@nokia.com","username":"andras.kovi"},"change_message_id":"8a97d39cc45a60c8f0ddeffc9b1c15cb77cad8b8","unresolved":false,"context_lines":[{"line_number":48,"context_line":"    $INI_SET oslo_policy policy_file /opt/stack/mistral/etc/policy.json \u0026\u0026 \\"},{"line_number":49,"context_line":"    $INI_SET pecan auth_enable false"},{"line_number":50,"context_line":""},{"line_number":51,"context_line":"# Install MySQL libs"},{"line_number":52,"context_line":""},{"line_number":53,"context_line":""},{"line_number":54,"context_line":"EXPOSE 8989"}],"source_content_type":"text/x-dockerfile","patch_set":2,"id":"3f1d235d_736653f0","line":51,"range":{"start_line":51,"start_character":0,"end_line":51,"end_character":20},"in_reply_to":"3f1d235d_7f973c92","updated":"2017-06-30 07:42:38.000000000","message":"Done","commit_id":"8f0eb1b6e1bb1844ee8ae16355b727bf05ec30ca"},{"author":{"_account_id":7065,"name":"Brad P. Crochet","email":"brad@redhat.com","username":"bcrochet"},"change_message_id":"1c698b91601b77ef34fb7c1ca5ed2820e2e9b703","unresolved":false,"context_lines":[{"line_number":7,"context_line":""},{"line_number":8,"context_line":"EXPOSE 8989"},{"line_number":9,"context_line":""},{"line_number":10,"context_line":"VOLUME [\"/opt/stack/mistral\"]"},{"line_number":11,"context_line":"VOLUME [\"/home/mistral\"]"},{"line_number":12,"context_line":"WORKDIR /home/mistral"},{"line_number":13,"context_line":"CMD mistral-server --server all --config-file /home/mistral/mistral.conf"}],"source_content_type":"text/x-dockerfile","patch_set":3,"id":"3f1d235d_e5a3ab4e","side":"PARENT","line":10,"updated":"2017-06-30 16:16:59.000000000","message":"Having these volumes available allows to use a local code base without having to necessarily rebuild the container. Is there a problem with keeping the volumes?","commit_id":"fe1a1e15189221154b9ca1da3345ec449a37830e"},{"author":{"_account_id":8731,"name":"Renat Akhmerov","email":"renat.akhmerov@gmail.com","username":"rakhmerov"},"change_message_id":"e9bd46a1eeed8ce7262b6db9dbfd263c6b9d8fa2","unresolved":false,"context_lines":[{"line_number":7,"context_line":""},{"line_number":8,"context_line":"EXPOSE 8989"},{"line_number":9,"context_line":""},{"line_number":10,"context_line":"VOLUME [\"/opt/stack/mistral\"]"},{"line_number":11,"context_line":"VOLUME [\"/home/mistral\"]"},{"line_number":12,"context_line":"WORKDIR /home/mistral"},{"line_number":13,"context_line":"CMD mistral-server --server all --config-file /home/mistral/mistral.conf"}],"source_content_type":"text/x-dockerfile","patch_set":3,"id":"3f1d235d_72ccbf60","side":"PARENT","line":10,"in_reply_to":"3f1d235d_5109ec29","updated":"2017-07-04 07:08:06.000000000","message":"So, is this OK with you, Brad?\n\nI don\u0027t have a strong opinion on this.","commit_id":"fe1a1e15189221154b9ca1da3345ec449a37830e"},{"author":{"_account_id":21970,"name":"Andras Kovi","email":"akovi@nokia.com","username":"andras.kovi"},"change_message_id":"f37fe454bfe3171b74b05627ffcd2846f4414b52","unresolved":false,"context_lines":[{"line_number":7,"context_line":""},{"line_number":8,"context_line":"EXPOSE 8989"},{"line_number":9,"context_line":""},{"line_number":10,"context_line":"VOLUME [\"/opt/stack/mistral\"]"},{"line_number":11,"context_line":"VOLUME [\"/home/mistral\"]"},{"line_number":12,"context_line":"WORKDIR /home/mistral"},{"line_number":13,"context_line":"CMD mistral-server --server all --config-file /home/mistral/mistral.conf"}],"source_content_type":"text/x-dockerfile","patch_set":3,"id":"3f1d235d_5109ec29","side":"PARENT","line":10,"in_reply_to":"3f1d235d_e5a3ab4e","updated":"2017-07-02 14:33:50.000000000","message":"Brad, nothing prevents the users from attaching the local code to the same path. Declaring the volume here is not necessary. I removed these because running Mistral this way didn\u0027t really prove to be useful in my case. Additionally, as a the user inside is \u0027root\u0027, this can possibly interfere with ongoing development.","commit_id":"fe1a1e15189221154b9ca1da3345ec449a37830e"}],"tools/docker/start_mistral_rabbit_mysql.sh":[{"author":{"_account_id":21970,"name":"Andras Kovi","email":"akovi@nokia.com","username":"andras.kovi"},"change_message_id":"f37fe454bfe3171b74b05627ffcd2846f4414b52","unresolved":false,"context_lines":[{"line_number":66,"context_line":""},{"line_number":67,"context_line":"multi)"},{"line_number":68,"context_line":"    # Multinode setup"},{"line_number":69,"context_line":"    MORE_OPTS\u003d\"${EXTRA_OPTS} -p 8989:8989\" run_mistral \"mistral-api\" mistral-server --server api"},{"line_number":70,"context_line":"    run_mistral \"mistral-engine\" mistral-server --server engine"},{"line_number":71,"context_line":"    run_mistral \"mistral-executor-1\" mistral-server --server executor"},{"line_number":72,"context_line":"    run_mistral \"mistral-executor-2\" mistral-server --server executor"}],"source_content_type":"text/x-sh","patch_set":3,"id":"3f1d235d_f10d9835","line":69,"updated":"2017-07-02 14:33:50.000000000","message":"Thhe MORE_OPTS handling should be made more generic, I think.","commit_id":"3c0558c4ba180eb19aaef09c3ae5ab733c4d4ce4"}]}