)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":7488,"name":"Jeffrey Zhang","email":"zhang.lei.fly@gmail.com","username":"Jeffrey4l"},"change_message_id":"83ee60d277265288aefd9067fc783813b0f7b9f9","unresolved":false,"context_lines":[{"line_number":10,"context_line":"Updated both dependency and install sections."},{"line_number":11,"context_line":"Addressed review comments."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Trivial-Fix"},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"Change-Id: Icd2a699e26553e3c41ee58ff6081716efe77dc25"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"3ac371cc_727bcb8c","line":13,"updated":"2016-08-17 12:35:09.000000000","message":"no need add TrivialFix when changing doc","commit_id":"636a7c513636c93dc60f1e3793233d23cfd8f21c"}],"doc/quickstart.rst":[{"author":{"_account_id":16993,"name":"Hui Kang","email":"kangh@us.ibm.com","username":"huikang"},"change_message_id":"5c6a425045324e0d7a453d6dd7a3b9a61d6862bc","unresolved":false,"context_lines":[{"line_number":49,"context_line":""},{"line_number":50,"context_line":"    apt-get install linux-image-generic-lts-wily"},{"line_number":51,"context_line":""},{"line_number":52,"context_line":".. note:: Operators performing an evaluation or deployment should use a "},{"line_number":53,"context_line":"  stable branch.  Operators performing development or developers should use"},{"line_number":54,"context_line":"  master."},{"line_number":55,"context_line":""}],"source_content_type":"text/x-rst","patch_set":2,"id":"3ac371cc_0ec7ef64","line":52,"updated":"2016-08-17 02:41:53.000000000","message":"trailing space","commit_id":"28a7f957eb606f9afa37d6fbd80166c18101671d"},{"author":{"_account_id":16993,"name":"Hui Kang","email":"kangh@us.ibm.com","username":"huikang"},"change_message_id":"5c6a425045324e0d7a453d6dd7a3b9a61d6862bc","unresolved":false,"context_lines":[{"line_number":255,"context_line":""},{"line_number":256,"context_line":"..note:: If Kolla is pip installed then Kolla reads its Kolla runtime content"},{"line_number":257,"context_line":"    from the installed location.  If Kolla is operated from a git repository,"},{"line_number":258,"context_line":"    the git repositories content will not be used.  To use the git repository "},{"line_number":259,"context_line":"    content Kolla must be pip uninstalled."},{"line_number":260,"context_line":""},{"line_number":261,"context_line":" "}],"source_content_type":"text/x-rst","patch_set":2,"id":"3ac371cc_0e9ecf62","line":258,"updated":"2016-08-17 02:41:53.000000000","message":"Two spaces between . and To.","commit_id":"28a7f957eb606f9afa37d6fbd80166c18101671d"},{"author":{"_account_id":19316,"name":"Eduardo Gonzalez","email":"dabarren@gmail.com","username":"egonzalez90"},"change_message_id":"cc0f4cdd02611734110b189b9fbe7fe4972cf7d0","unresolved":false,"context_lines":[{"line_number":62,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d   \u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d  \u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d  \u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":63,"context_line":"Component               Min Version  Max Version  Comment"},{"line_number":64,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d   \u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d  \u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d  \u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":65,"context_line":"Ansible                 1.9.4        2.0.0        On deployment host"},{"line_number":66,"context_line":"Docker                  1.10.0       none         On target nodes"},{"line_number":67,"context_line":"Docker Python           1.6.0        none         On target nodes"},{"line_number":68,"context_line":"Python Jinja2           2.7.2        none         On deployment host"}],"source_content_type":"text/x-rst","patch_set":3,"id":"3ac371cc_0f724649","line":65,"range":{"start_line":65,"start_character":37,"end_line":65,"end_character":42},"updated":"2016-08-17 06:44:21.000000000","message":"From this I understand ansible 2.0.0 is included as supported version of stable branch, which is not.\nI would use something like prior to 2.0.0 or \u003c 2.0.0.","commit_id":"636a7c513636c93dc60f1e3793233d23cfd8f21c"},{"author":{"_account_id":7488,"name":"Jeffrey Zhang","email":"zhang.lei.fly@gmail.com","username":"Jeffrey4l"},"change_message_id":"83ee60d277265288aefd9067fc783813b0f7b9f9","unresolved":false,"context_lines":[{"line_number":62,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d   \u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d  \u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d  \u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":63,"context_line":"Component               Min Version  Max Version  Comment"},{"line_number":64,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d   \u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d  \u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d  \u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":65,"context_line":"Ansible                 1.9.4        2.0.0        On deployment host"},{"line_number":66,"context_line":"Docker                  1.10.0       none         On target nodes"},{"line_number":67,"context_line":"Docker Python           1.6.0        none         On target nodes"},{"line_number":68,"context_line":"Python Jinja2           2.7.2        none         On deployment host"}],"source_content_type":"text/x-rst","patch_set":3,"id":"3ac371cc_d26c3fd2","line":65,"updated":"2016-08-17 12:35:09.000000000","message":"yes, please use  \u003c2.0.0","commit_id":"636a7c513636c93dc60f1e3793233d23cfd8f21c"},{"author":{"_account_id":2834,"name":"Steven Dake","email":"steven.dake@gmail.com","username":"sdake"},"change_message_id":"1ff7cf01f333a3a5bc812463e3ec647e43e68d17","unresolved":false,"context_lines":[{"line_number":62,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d   \u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d  \u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d  \u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":63,"context_line":"Component               Min Version  Max Version  Comment"},{"line_number":64,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d   \u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d  \u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d  \u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":65,"context_line":"Ansible                 1.9.4        2.0.0        On deployment host"},{"line_number":66,"context_line":"Docker                  1.10.0       none         On target nodes"},{"line_number":67,"context_line":"Docker Python           1.6.0        none         On target nodes"},{"line_number":68,"context_line":"Python Jinja2           2.7.2        none         On deployment host"}],"source_content_type":"text/x-rst","patch_set":3,"id":"3ac371cc_d0aeff96","line":65,"range":{"start_line":65,"start_character":37,"end_line":65,"end_character":42},"in_reply_to":"3ac371cc_0f724649","updated":"2016-08-17 10:52:57.000000000","message":"The git history has \u003c 2.0.0 for this line - so I think we should match our history.","commit_id":"636a7c513636c93dc60f1e3793233d23cfd8f21c"},{"author":{"_account_id":13697,"name":"Leon Zachery","email":"lzachery@cisco.com","username":"lzachery"},"change_message_id":"c04ce5943278007d4d4253a10adff7227b8a1103","unresolved":false,"context_lines":[{"line_number":62,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d   \u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d  \u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d  \u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":63,"context_line":"Component               Min Version  Max Version  Comment"},{"line_number":64,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d   \u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d  \u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d  \u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":65,"context_line":"Ansible                 1.9.4        2.0.0        On deployment host"},{"line_number":66,"context_line":"Docker                  1.10.0       none         On target nodes"},{"line_number":67,"context_line":"Docker Python           1.6.0        none         On target nodes"},{"line_number":68,"context_line":"Python Jinja2           2.7.2        none         On deployment host"}],"source_content_type":"text/x-rst","patch_set":3,"id":"3ac371cc_71ae11e7","line":65,"in_reply_to":"3ac371cc_d26c3fd2","updated":"2016-08-17 19:02:36.000000000","message":"Done","commit_id":"636a7c513636c93dc60f1e3793233d23cfd8f21c"},{"author":{"_account_id":2834,"name":"Steven Dake","email":"steven.dake@gmail.com","username":"sdake"},"change_message_id":"1ff7cf01f333a3a5bc812463e3ec647e43e68d17","unresolved":false,"context_lines":[{"line_number":65,"context_line":"Ansible                 1.9.4        2.0.0        On deployment host"},{"line_number":66,"context_line":"Docker                  1.10.0       none         On target nodes"},{"line_number":67,"context_line":"Docker Python           1.6.0        none         On target nodes"},{"line_number":68,"context_line":"Python Jinja2           2.7.2        none         On deployment host"},{"line_number":69,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d   \u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d  \u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d  \u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":70,"context_line":""},{"line_number":71,"context_line":""}],"source_content_type":"text/x-rst","patch_set":3,"id":"3ac371cc_1095373e","line":68,"updated":"2016-08-17 10:52:57.000000000","message":"git history has 2.6.0 here as well","commit_id":"636a7c513636c93dc60f1e3793233d23cfd8f21c"},{"author":{"_account_id":13697,"name":"Leon Zachery","email":"lzachery@cisco.com","username":"lzachery"},"change_message_id":"c04ce5943278007d4d4253a10adff7227b8a1103","unresolved":false,"context_lines":[{"line_number":65,"context_line":"Ansible                 1.9.4        2.0.0        On deployment host"},{"line_number":66,"context_line":"Docker                  1.10.0       none         On target nodes"},{"line_number":67,"context_line":"Docker Python           1.6.0        none         On target nodes"},{"line_number":68,"context_line":"Python Jinja2           2.7.2        none         On deployment host"},{"line_number":69,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d   \u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d  \u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d  \u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":70,"context_line":""},{"line_number":71,"context_line":""}],"source_content_type":"text/x-rst","patch_set":3,"id":"3ac371cc_51f15500","line":68,"in_reply_to":"3ac371cc_1095373e","updated":"2016-08-17 19:02:36.000000000","message":"Done","commit_id":"636a7c513636c93dc60f1e3793233d23cfd8f21c"},{"author":{"_account_id":2834,"name":"Steven Dake","email":"steven.dake@gmail.com","username":"sdake"},"change_message_id":"296e8fa84f9e6f4d86cb949e27ed990e1b26d55e","unresolved":false,"context_lines":[{"line_number":253,"context_line":""},{"line_number":254,"context_line":"    apt-get install ansible"},{"line_number":255,"context_line":""},{"line_number":256,"context_line":"..note:: If Kolla is pip installed then Kolla reads its Kolla runtime content"},{"line_number":257,"context_line":"    from the installed location. If Kolla is operated from a git repository,"},{"line_number":258,"context_line":"    the git repositories content will not be used. To use the git repository "},{"line_number":259,"context_line":"    content Kolla must be pip uninstalled."}],"source_content_type":"text/x-rst","patch_set":3,"id":"3ac371cc_ff312d5b","line":256,"range":{"start_line":256,"start_character":2,"end_line":256,"end_character":6},"updated":"2016-08-17 18:35:55.000000000","message":"this part should be attention instead of note","commit_id":"636a7c513636c93dc60f1e3793233d23cfd8f21c"},{"author":{"_account_id":13697,"name":"Leon Zachery","email":"lzachery@cisco.com","username":"lzachery"},"change_message_id":"c04ce5943278007d4d4253a10adff7227b8a1103","unresolved":false,"context_lines":[{"line_number":253,"context_line":""},{"line_number":254,"context_line":"    apt-get install ansible"},{"line_number":255,"context_line":""},{"line_number":256,"context_line":"..note:: If Kolla is pip installed then Kolla reads its Kolla runtime content"},{"line_number":257,"context_line":"    from the installed location. If Kolla is operated from a git repository,"},{"line_number":258,"context_line":"    the git repositories content will not be used. To use the git repository "},{"line_number":259,"context_line":"    content Kolla must be pip uninstalled."}],"source_content_type":"text/x-rst","patch_set":3,"id":"3ac371cc_f164c1b9","line":256,"range":{"start_line":256,"start_character":2,"end_line":256,"end_character":6},"in_reply_to":"3ac371cc_ff312d5b","updated":"2016-08-17 19:02:36.000000000","message":"Done","commit_id":"636a7c513636c93dc60f1e3793233d23cfd8f21c"},{"author":{"_account_id":16993,"name":"Hui Kang","email":"kangh@us.ibm.com","username":"huikang"},"change_message_id":"e5c3253fb65159e4d403c4831712b2b2e35bc423","unresolved":false,"context_lines":[{"line_number":255,"context_line":""},{"line_number":256,"context_line":"..note:: If Kolla is pip installed then Kolla reads its Kolla runtime content"},{"line_number":257,"context_line":"    from the installed location. If Kolla is operated from a git repository,"},{"line_number":258,"context_line":"    the git repositories content will not be used. To use the git repository "},{"line_number":259,"context_line":"    content Kolla must be pip uninstalled."},{"line_number":260,"context_line":""},{"line_number":261,"context_line":" "}],"source_content_type":"text/x-rst","patch_set":3,"id":"3ac371cc_feb4be35","line":258,"updated":"2016-08-17 04:45:37.000000000","message":"The second if statement is confusing. Do you mean \"In this case, even if the kolla-* command is operated from ...\"","commit_id":"636a7c513636c93dc60f1e3793233d23cfd8f21c"},{"author":{"_account_id":1390,"name":"Paul Bourke","email":"pauldbourke@proton.me","username":"pauldbourke"},"change_message_id":"72882bfeaec47de960fb14c8d7d1ee95e571d5e4","unresolved":false,"context_lines":[{"line_number":253,"context_line":""},{"line_number":254,"context_line":"    apt-get install ansible"},{"line_number":255,"context_line":""},{"line_number":256,"context_line":"..note:: If Kolla is pip installed then Kolla reads its Kolla runtime content"},{"line_number":257,"context_line":"    from the installed location. If Kolla is operated from a git repository,"},{"line_number":258,"context_line":"    the git repositories content will not be used. To use the git repository "},{"line_number":259,"context_line":"    content Kolla must be pip uninstalled."},{"line_number":260,"context_line":""},{"line_number":261,"context_line":" "},{"line_number":262,"context_line":"Install Kolla for evaluation or deployment"}],"source_content_type":"text/x-rst","patch_set":3,"id":"3ac371cc_50b5ef2e","line":259,"range":{"start_line":256,"start_character":9,"end_line":259,"end_character":42},"updated":"2016-08-17 10:52:18.000000000","message":"You can actually have both. When pip installed, operators can do commands such as \u0027kolla-build\u0027 and it will use the installed location. However if you do \u0027tools/build.py\u0027 then it will the git repo.","commit_id":"636a7c513636c93dc60f1e3793233d23cfd8f21c"},{"author":{"_account_id":2834,"name":"Steven Dake","email":"steven.dake@gmail.com","username":"sdake"},"change_message_id":"296e8fa84f9e6f4d86cb949e27ed990e1b26d55e","unresolved":false,"context_lines":[{"line_number":253,"context_line":""},{"line_number":254,"context_line":"    apt-get install ansible"},{"line_number":255,"context_line":""},{"line_number":256,"context_line":"..note:: If Kolla is pip installed then Kolla reads its Kolla runtime content"},{"line_number":257,"context_line":"    from the installed location. If Kolla is operated from a git repository,"},{"line_number":258,"context_line":"    the git repositories content will not be used. To use the git repository "},{"line_number":259,"context_line":"    content Kolla must be pip uninstalled."},{"line_number":260,"context_line":""},{"line_number":261,"context_line":" "},{"line_number":262,"context_line":"Install Kolla for evaluation or deployment"}],"source_content_type":"text/x-rst","patch_set":3,"id":"3ac371cc_3fa8e5da","line":259,"range":{"start_line":256,"start_character":9,"end_line":259,"end_character":42},"in_reply_to":"3ac371cc_26291de7","updated":"2016-08-17 18:35:55.000000000","message":"This should be:\nKolla uses PBR in its implementation.  PBR provides version information to Kolla about the package in use.  This information is later used when building images to specify the Docker tag used in the image built.  When installing a package via pip, PBR will always use the PBR version information.  When obtaining a copy of the software via git, PBR will use the git version information, but **ONLY** if Kolla has not been pip installed via the pip package manager.  This is why there is an operator workflow and a developer workflow.","commit_id":"636a7c513636c93dc60f1e3793233d23cfd8f21c"},{"author":{"_account_id":13697,"name":"Leon Zachery","email":"lzachery@cisco.com","username":"lzachery"},"change_message_id":"c04ce5943278007d4d4253a10adff7227b8a1103","unresolved":false,"context_lines":[{"line_number":253,"context_line":""},{"line_number":254,"context_line":"    apt-get install ansible"},{"line_number":255,"context_line":""},{"line_number":256,"context_line":"..note:: If Kolla is pip installed then Kolla reads its Kolla runtime content"},{"line_number":257,"context_line":"    from the installed location. If Kolla is operated from a git repository,"},{"line_number":258,"context_line":"    the git repositories content will not be used. To use the git repository "},{"line_number":259,"context_line":"    content Kolla must be pip uninstalled."},{"line_number":260,"context_line":""},{"line_number":261,"context_line":" "},{"line_number":262,"context_line":"Install Kolla for evaluation or deployment"}],"source_content_type":"text/x-rst","patch_set":3,"id":"3ac371cc_d1d9c523","line":259,"range":{"start_line":256,"start_character":9,"end_line":259,"end_character":42},"in_reply_to":"3ac371cc_3fa8e5da","updated":"2016-08-17 19:02:36.000000000","message":"Done","commit_id":"636a7c513636c93dc60f1e3793233d23cfd8f21c"},{"author":{"_account_id":2834,"name":"Steven Dake","email":"steven.dake@gmail.com","username":"sdake"},"change_message_id":"39201afd3626a27c7ca387cbca8b4a700c199d5f","unresolved":false,"context_lines":[{"line_number":253,"context_line":""},{"line_number":254,"context_line":"    apt-get install ansible"},{"line_number":255,"context_line":""},{"line_number":256,"context_line":"..note:: If Kolla is pip installed then Kolla reads its Kolla runtime content"},{"line_number":257,"context_line":"    from the installed location. If Kolla is operated from a git repository,"},{"line_number":258,"context_line":"    the git repositories content will not be used. To use the git repository "},{"line_number":259,"context_line":"    content Kolla must be pip uninstalled."},{"line_number":260,"context_line":""},{"line_number":261,"context_line":" "},{"line_number":262,"context_line":"Install Kolla for evaluation or deployment"}],"source_content_type":"text/x-rst","patch_set":3,"id":"3ac371cc_c3447b1c","line":259,"range":{"start_line":256,"start_character":9,"end_line":259,"end_character":42},"in_reply_to":"3ac371cc_50b5ef2e","updated":"2016-08-17 11:11:24.000000000","message":"Paul is correct here - see paste: http://paste.openstack.org/show/558997/\n\nI think what you are seeing may be a different bug  in that the python code (where the tag is hardcoded) is being run from the installed location if pip installed, but from the git location if git installed.  I need to verify.  I\u0027ll get back to this review in ~1 hr.","commit_id":"636a7c513636c93dc60f1e3793233d23cfd8f21c"},{"author":{"_account_id":2834,"name":"Steven Dake","email":"steven.dake@gmail.com","username":"sdake"},"change_message_id":"9de8fed2d4018a2d93d8de95e22d195bd74f2e02","unresolved":false,"context_lines":[{"line_number":253,"context_line":""},{"line_number":254,"context_line":"    apt-get install ansible"},{"line_number":255,"context_line":""},{"line_number":256,"context_line":"..note:: If Kolla is pip installed then Kolla reads its Kolla runtime content"},{"line_number":257,"context_line":"    from the installed location. If Kolla is operated from a git repository,"},{"line_number":258,"context_line":"    the git repositories content will not be used. To use the git repository "},{"line_number":259,"context_line":"    content Kolla must be pip uninstalled."},{"line_number":260,"context_line":""},{"line_number":261,"context_line":" "},{"line_number":262,"context_line":"Install Kolla for evaluation or deployment"}],"source_content_type":"text/x-rst","patch_set":3,"id":"3ac371cc_26291de7","line":259,"range":{"start_line":256,"start_character":9,"end_line":259,"end_character":42},"in_reply_to":"3ac371cc_c3447b1c","updated":"2016-08-17 11:32:17.000000000","message":"further investigation shows my endless guidelines on irc were incorrect :(  The root cause of this issue is pbr is returning 3.0.0 when working from a stable branch in git, even if the master branch is not pip installed.  pbr obtains its version number from the git history.\n\nLet me have a think on how to best solve this with documentation.","commit_id":"636a7c513636c93dc60f1e3793233d23cfd8f21c"},{"author":{"_account_id":1390,"name":"Paul Bourke","email":"pauldbourke@proton.me","username":"pauldbourke"},"change_message_id":"72882bfeaec47de960fb14c8d7d1ee95e571d5e4","unresolved":false,"context_lines":[{"line_number":259,"context_line":"    content Kolla must be pip uninstalled."},{"line_number":260,"context_line":""},{"line_number":261,"context_line":" "},{"line_number":262,"context_line":"Install Kolla for evaluation or deployment"},{"line_number":263,"context_line":"------------------------------------------"},{"line_number":264,"context_line":""},{"line_number":265,"context_line":"To clone the Kolla repo:"}],"source_content_type":"text/x-rst","patch_set":3,"id":"3ac371cc_50ba0f0b","line":262,"range":{"start_line":262,"start_character":0,"end_line":262,"end_character":7},"updated":"2016-08-17 10:52:18.000000000","message":"Change to \"Installing\"","commit_id":"636a7c513636c93dc60f1e3793233d23cfd8f21c"},{"author":{"_account_id":13697,"name":"Leon Zachery","email":"lzachery@cisco.com","username":"lzachery"},"change_message_id":"c04ce5943278007d4d4253a10adff7227b8a1103","unresolved":false,"context_lines":[{"line_number":259,"context_line":"    content Kolla must be pip uninstalled."},{"line_number":260,"context_line":""},{"line_number":261,"context_line":" "},{"line_number":262,"context_line":"Install Kolla for evaluation or deployment"},{"line_number":263,"context_line":"------------------------------------------"},{"line_number":264,"context_line":""},{"line_number":265,"context_line":"To clone the Kolla repo:"}],"source_content_type":"text/x-rst","patch_set":3,"id":"3ac371cc_3101396b","line":262,"range":{"start_line":262,"start_character":0,"end_line":262,"end_character":7},"in_reply_to":"3ac371cc_50ba0f0b","updated":"2016-08-17 19:02:36.000000000","message":"Done","commit_id":"636a7c513636c93dc60f1e3793233d23cfd8f21c"},{"author":{"_account_id":2834,"name":"Steven Dake","email":"steven.dake@gmail.com","username":"sdake"},"change_message_id":"296e8fa84f9e6f4d86cb949e27ed990e1b26d55e","unresolved":false,"context_lines":[{"line_number":262,"context_line":"Install Kolla for evaluation or deployment"},{"line_number":263,"context_line":"------------------------------------------"},{"line_number":264,"context_line":""},{"line_number":265,"context_line":"To clone the Kolla repo:"},{"line_number":266,"context_line":""},{"line_number":267,"context_line":"::"},{"line_number":268,"context_line":""}],"source_content_type":"text/x-rst","patch_set":3,"id":"3ac371cc_9f7e3917","line":265,"range":{"start_line":265,"start_character":0,"end_line":265,"end_character":24},"updated":"2016-08-17 18:35:55.000000000","message":"delete this line - we dont want to recommend working for git for eval or deployment purposes","commit_id":"636a7c513636c93dc60f1e3793233d23cfd8f21c"},{"author":{"_account_id":13697,"name":"Leon Zachery","email":"lzachery@cisco.com","username":"lzachery"},"change_message_id":"c04ce5943278007d4d4253a10adff7227b8a1103","unresolved":false,"context_lines":[{"line_number":262,"context_line":"Install Kolla for evaluation or deployment"},{"line_number":263,"context_line":"------------------------------------------"},{"line_number":264,"context_line":""},{"line_number":265,"context_line":"To clone the Kolla repo:"},{"line_number":266,"context_line":""},{"line_number":267,"context_line":"::"},{"line_number":268,"context_line":""}],"source_content_type":"text/x-rst","patch_set":3,"id":"3ac371cc_b11b69c9","line":265,"range":{"start_line":265,"start_character":0,"end_line":265,"end_character":24},"in_reply_to":"3ac371cc_9f7e3917","updated":"2016-08-17 19:02:36.000000000","message":"i don\u0027t think i should delete this line - instead it should be \"install Kolla and its dependencies\".","commit_id":"636a7c513636c93dc60f1e3793233d23cfd8f21c"},{"author":{"_account_id":16993,"name":"Hui Kang","email":"kangh@us.ibm.com","username":"huikang"},"change_message_id":"e5c3253fb65159e4d403c4831712b2b2e35bc423","unresolved":false,"context_lines":[{"line_number":263,"context_line":"------------------------------------------"},{"line_number":264,"context_line":""},{"line_number":265,"context_line":"To clone the Kolla repo:"},{"line_number":266,"context_line":""},{"line_number":267,"context_line":"::"},{"line_number":268,"context_line":""},{"line_number":269,"context_line":"    pip install kolla\u003d\u003d2.0.2"}],"source_content_type":"text/x-rst","patch_set":3,"id":"3ac371cc_be952688","line":266,"updated":"2016-08-17 04:45:37.000000000","message":"missing command for cloning","commit_id":"636a7c513636c93dc60f1e3793233d23cfd8f21c"},{"author":{"_account_id":2834,"name":"Steven Dake","email":"steven.dake@gmail.com","username":"sdake"},"change_message_id":"296e8fa84f9e6f4d86cb949e27ed990e1b26d55e","unresolved":false,"context_lines":[{"line_number":263,"context_line":"------------------------------------------"},{"line_number":264,"context_line":""},{"line_number":265,"context_line":"To clone the Kolla repo:"},{"line_number":266,"context_line":""},{"line_number":267,"context_line":"::"},{"line_number":268,"context_line":""},{"line_number":269,"context_line":"    pip install kolla\u003d\u003d2.0.2"}],"source_content_type":"text/x-rst","patch_set":3,"id":"3ac371cc_7fa25d9a","line":266,"in_reply_to":"3ac371cc_70bd8b01","updated":"2016-08-17 18:35:55.000000000","message":"ignore my comment here","commit_id":"636a7c513636c93dc60f1e3793233d23cfd8f21c"},{"author":{"_account_id":2834,"name":"Steven Dake","email":"steven.dake@gmail.com","username":"sdake"},"change_message_id":"1ff7cf01f333a3a5bc812463e3ec647e43e68d17","unresolved":false,"context_lines":[{"line_number":263,"context_line":"------------------------------------------"},{"line_number":264,"context_line":""},{"line_number":265,"context_line":"To clone the Kolla repo:"},{"line_number":266,"context_line":""},{"line_number":267,"context_line":"::"},{"line_number":268,"context_line":""},{"line_number":269,"context_line":"    pip install kolla\u003d\u003d2.0.2"}],"source_content_type":"text/x-rst","patch_set":3,"id":"3ac371cc_70bd8b01","line":266,"in_reply_to":"3ac371cc_be952688","updated":"2016-08-17 10:52:57.000000000","message":"this needs fixing","commit_id":"636a7c513636c93dc60f1e3793233d23cfd8f21c"},{"author":{"_account_id":2834,"name":"Steven Dake","email":"steven.dake@gmail.com","username":"sdake"},"change_message_id":"296e8fa84f9e6f4d86cb949e27ed990e1b26d55e","unresolved":false,"context_lines":[{"line_number":264,"context_line":""},{"line_number":265,"context_line":"To clone the Kolla repo:"},{"line_number":266,"context_line":""},{"line_number":267,"context_line":"::"},{"line_number":268,"context_line":""},{"line_number":269,"context_line":"    pip install kolla\u003d\u003d2.0.2"},{"line_number":270,"context_line":""}],"source_content_type":"text/x-rst","patch_set":3,"id":"3ac371cc_ff95ad32","line":267,"updated":"2016-08-17 18:35:55.000000000","message":"and this line for same reason above.","commit_id":"636a7c513636c93dc60f1e3793233d23cfd8f21c"},{"author":{"_account_id":1390,"name":"Paul Bourke","email":"pauldbourke@proton.me","username":"pauldbourke"},"change_message_id":"72882bfeaec47de960fb14c8d7d1ee95e571d5e4","unresolved":false,"context_lines":[{"line_number":265,"context_line":"To clone the Kolla repo:"},{"line_number":266,"context_line":""},{"line_number":267,"context_line":"::"},{"line_number":268,"context_line":""},{"line_number":269,"context_line":"    pip install kolla\u003d\u003d2.0.2"},{"line_number":270,"context_line":""},{"line_number":271,"context_line":"Kolla holds configurations files in ``etc/kolla``. Copy the configuration files"}],"source_content_type":"text/x-rst","patch_set":3,"id":"3ac371cc_d0b71ff3","line":268,"updated":"2016-08-17 10:52:18.000000000","message":"Add a note about checking out the stable branch","commit_id":"636a7c513636c93dc60f1e3793233d23cfd8f21c"},{"author":{"_account_id":13697,"name":"Leon Zachery","email":"lzachery@cisco.com","username":"lzachery"},"change_message_id":"c04ce5943278007d4d4253a10adff7227b8a1103","unresolved":false,"context_lines":[{"line_number":265,"context_line":"To clone the Kolla repo:"},{"line_number":266,"context_line":""},{"line_number":267,"context_line":"::"},{"line_number":268,"context_line":""},{"line_number":269,"context_line":"    pip install kolla\u003d\u003d2.0.2"},{"line_number":270,"context_line":""},{"line_number":271,"context_line":"Kolla holds configurations files in ``etc/kolla``. Copy the configuration files"}],"source_content_type":"text/x-rst","patch_set":3,"id":"3ac371cc_91046d4a","line":268,"in_reply_to":"3ac371cc_d0b71ff3","updated":"2016-08-17 19:02:36.000000000","message":"On line 269 Steven says that we don\u0027t want operators to run from git checkout.","commit_id":"636a7c513636c93dc60f1e3793233d23cfd8f21c"},{"author":{"_account_id":1390,"name":"Paul Bourke","email":"pauldbourke@proton.me","username":"pauldbourke"},"change_message_id":"72882bfeaec47de960fb14c8d7d1ee95e571d5e4","unresolved":false,"context_lines":[{"line_number":266,"context_line":""},{"line_number":267,"context_line":"::"},{"line_number":268,"context_line":""},{"line_number":269,"context_line":"    pip install kolla\u003d\u003d2.0.2"},{"line_number":270,"context_line":""},{"line_number":271,"context_line":"Kolla holds configurations files in ``etc/kolla``. Copy the configuration files"},{"line_number":272,"context_line":"to ``/etc``:"}],"source_content_type":"text/x-rst","patch_set":3,"id":"3ac371cc_d0cd5fb8","line":269,"range":{"start_line":269,"start_character":21,"end_line":269,"end_character":28},"updated":"2016-08-17 10:52:18.000000000","message":"Is the version pin necessary?","commit_id":"636a7c513636c93dc60f1e3793233d23cfd8f21c"},{"author":{"_account_id":2834,"name":"Steven Dake","email":"steven.dake@gmail.com","username":"sdake"},"change_message_id":"296e8fa84f9e6f4d86cb949e27ed990e1b26d55e","unresolved":false,"context_lines":[{"line_number":266,"context_line":""},{"line_number":267,"context_line":"::"},{"line_number":268,"context_line":""},{"line_number":269,"context_line":"    pip install kolla\u003d\u003d2.0.2"},{"line_number":270,"context_line":""},{"line_number":271,"context_line":"Kolla holds configurations files in ``etc/kolla``. Copy the configuration files"},{"line_number":272,"context_line":"to ``/etc``:"}],"source_content_type":"text/x-rst","patch_set":3,"id":"3ac371cc_7fc37d89","line":269,"range":{"start_line":269,"start_character":21,"end_line":269,"end_character":28},"in_reply_to":"3ac371cc_a32e9f40","updated":"2016-08-17 18:35:55.000000000","message":"We don\u0027t want operators to run from git checkouts - based upon my interview with jroll and dhellman (see irc logs).  so a version pin is necessary. Might be helpful to have the version pin for liberty as well, so people know where to find both.\n\nUnfortunately we will have to hand maintain this version - along with a bunch of other stuff we already hand maintain on zstream releases.","commit_id":"636a7c513636c93dc60f1e3793233d23cfd8f21c"},{"author":{"_account_id":2834,"name":"Steven Dake","email":"steven.dake@gmail.com","username":"sdake"},"change_message_id":"39201afd3626a27c7ca387cbca8b4a700c199d5f","unresolved":false,"context_lines":[{"line_number":266,"context_line":""},{"line_number":267,"context_line":"::"},{"line_number":268,"context_line":""},{"line_number":269,"context_line":"    pip install kolla\u003d\u003d2.0.2"},{"line_number":270,"context_line":""},{"line_number":271,"context_line":"Kolla holds configurations files in ``etc/kolla``. Copy the configuration files"},{"line_number":272,"context_line":"to ``/etc``:"}],"source_content_type":"text/x-rst","patch_set":3,"id":"3ac371cc_a32e9f40","line":269,"range":{"start_line":269,"start_character":21,"end_line":269,"end_character":28},"in_reply_to":"3ac371cc_d0cd5fb8","updated":"2016-08-17 11:11:24.000000000","message":"another option here is explain how to check out the stable branch followed by customizing openstack_version to 1+ the current stable.","commit_id":"636a7c513636c93dc60f1e3793233d23cfd8f21c"},{"author":{"_account_id":16993,"name":"Hui Kang","email":"kangh@us.ibm.com","username":"huikang"},"change_message_id":"e5c3253fb65159e4d403c4831712b2b2e35bc423","unresolved":false,"context_lines":[{"line_number":269,"context_line":"    pip install kolla\u003d\u003d2.0.2"},{"line_number":270,"context_line":""},{"line_number":271,"context_line":"Kolla holds configurations files in ``etc/kolla``. Copy the configuration files"},{"line_number":272,"context_line":"to ``/etc``:"},{"line_number":273,"context_line":""},{"line_number":274,"context_line":"::"},{"line_number":275,"context_line":""}],"source_content_type":"text/x-rst","patch_set":3,"id":"3ac371cc_bedc0674","line":272,"updated":"2016-08-17 04:45:37.000000000","message":"You may add a reference to [1]\n\n[1] https://github.com/huikang/kolla/blob/master/doc/image-building.rst#generating-kolla-buildconf","commit_id":"636a7c513636c93dc60f1e3793233d23cfd8f21c"},{"author":{"_account_id":2834,"name":"Steven Dake","email":"steven.dake@gmail.com","username":"sdake"},"change_message_id":"1ff7cf01f333a3a5bc812463e3ec647e43e68d17","unresolved":false,"context_lines":[{"line_number":269,"context_line":"    pip install kolla\u003d\u003d2.0.2"},{"line_number":270,"context_line":""},{"line_number":271,"context_line":"Kolla holds configurations files in ``etc/kolla``. Copy the configuration files"},{"line_number":272,"context_line":"to ``/etc``:"},{"line_number":273,"context_line":""},{"line_number":274,"context_line":"::"},{"line_number":275,"context_line":""}],"source_content_type":"text/x-rst","patch_set":3,"id":"3ac371cc_9065a7b6","line":272,"in_reply_to":"3ac371cc_bedc0674","updated":"2016-08-17 10:52:57.000000000","message":"Hui good suggestion but I think in this case that would be overloading the documentation with too much sidetracking information resulting in extra confusion","commit_id":"636a7c513636c93dc60f1e3793233d23cfd8f21c"},{"author":{"_account_id":2834,"name":"Steven Dake","email":"steven.dake@gmail.com","username":"sdake"},"change_message_id":"296e8fa84f9e6f4d86cb949e27ed990e1b26d55e","unresolved":false,"context_lines":[{"line_number":277,"context_line":"    cp -r etc/kolla /etc/"},{"line_number":278,"context_line":""},{"line_number":279,"context_line":""},{"line_number":280,"context_line":"..note:: This does not actually install Kolla.  Many commands in this documentation are named"},{"line_number":281,"context_line":"    differently in the tools directory.  "},{"line_number":282,"context_line":""},{"line_number":283,"context_line":""}],"source_content_type":"text/x-rst","patch_set":3,"id":"3ac371cc_df4711ba","line":280,"range":{"start_line":280,"start_character":2,"end_line":280,"end_character":6},"updated":"2016-08-17 18:35:55.000000000","message":"when this is in the correct place, it should be attention, not note.","commit_id":"636a7c513636c93dc60f1e3793233d23cfd8f21c"},{"author":{"_account_id":1390,"name":"Paul Bourke","email":"pauldbourke@proton.me","username":"pauldbourke"},"change_message_id":"72882bfeaec47de960fb14c8d7d1ee95e571d5e4","unresolved":false,"context_lines":[{"line_number":277,"context_line":"    cp -r etc/kolla /etc/"},{"line_number":278,"context_line":""},{"line_number":279,"context_line":""},{"line_number":280,"context_line":"..note:: This does not actually install Kolla.  Many commands in this documentation are named"},{"line_number":281,"context_line":"    differently in the tools directory.  "},{"line_number":282,"context_line":""},{"line_number":283,"context_line":""},{"line_number":284,"context_line":"Install Kolla for development"}],"source_content_type":"text/x-rst","patch_set":3,"id":"3ac371cc_10209747","line":281,"range":{"start_line":280,"start_character":0,"end_line":281,"end_character":41},"updated":"2016-08-17 10:52:18.000000000","message":"Is this note meant for the development section below? If so can you move it under the development heading below","commit_id":"636a7c513636c93dc60f1e3793233d23cfd8f21c"},{"author":{"_account_id":2834,"name":"Steven Dake","email":"steven.dake@gmail.com","username":"sdake"},"change_message_id":"39201afd3626a27c7ca387cbca8b4a700c199d5f","unresolved":false,"context_lines":[{"line_number":277,"context_line":"    cp -r etc/kolla /etc/"},{"line_number":278,"context_line":""},{"line_number":279,"context_line":""},{"line_number":280,"context_line":"..note:: This does not actually install Kolla.  Many commands in this documentation are named"},{"line_number":281,"context_line":"    differently in the tools directory.  "},{"line_number":282,"context_line":""},{"line_number":283,"context_line":""},{"line_number":284,"context_line":"Install Kolla for development"}],"source_content_type":"text/x-rst","patch_set":3,"id":"3ac371cc_c3e0fbfc","line":281,"range":{"start_line":280,"start_character":0,"end_line":281,"end_character":41},"in_reply_to":"3ac371cc_10209747","updated":"2016-08-17 11:11:24.000000000","message":"ya note in wrong place","commit_id":"636a7c513636c93dc60f1e3793233d23cfd8f21c"},{"author":{"_account_id":13697,"name":"Leon Zachery","email":"lzachery@cisco.com","username":"lzachery"},"change_message_id":"c04ce5943278007d4d4253a10adff7227b8a1103","unresolved":false,"context_lines":[{"line_number":277,"context_line":"    cp -r etc/kolla /etc/"},{"line_number":278,"context_line":""},{"line_number":279,"context_line":""},{"line_number":280,"context_line":"..note:: This does not actually install Kolla.  Many commands in this documentation are named"},{"line_number":281,"context_line":"    differently in the tools directory.  "},{"line_number":282,"context_line":""},{"line_number":283,"context_line":""},{"line_number":284,"context_line":"Install Kolla for development"}],"source_content_type":"text/x-rst","patch_set":3,"id":"3ac371cc_317019dc","line":281,"range":{"start_line":280,"start_character":0,"end_line":281,"end_character":41},"in_reply_to":"3ac371cc_c3e0fbfc","updated":"2016-08-17 19:02:36.000000000","message":"Done","commit_id":"636a7c513636c93dc60f1e3793233d23cfd8f21c"},{"author":{"_account_id":1390,"name":"Paul Bourke","email":"pauldbourke@proton.me","username":"pauldbourke"},"change_message_id":"72882bfeaec47de960fb14c8d7d1ee95e571d5e4","unresolved":false,"context_lines":[{"line_number":281,"context_line":"    differently in the tools directory.  "},{"line_number":282,"context_line":""},{"line_number":283,"context_line":""},{"line_number":284,"context_line":"Install Kolla for development"},{"line_number":285,"context_line":"-----------------------------"},{"line_number":286,"context_line":""},{"line_number":287,"context_line":"To clone the Kolla repo:"}],"source_content_type":"text/x-rst","patch_set":3,"id":"3ac371cc_903e478a","line":284,"range":{"start_line":284,"start_character":0,"end_line":284,"end_character":7},"updated":"2016-08-17 10:52:18.000000000","message":"Change to \"Installing\"","commit_id":"636a7c513636c93dc60f1e3793233d23cfd8f21c"},{"author":{"_account_id":2834,"name":"Steven Dake","email":"steven.dake@gmail.com","username":"sdake"},"change_message_id":"296e8fa84f9e6f4d86cb949e27ed990e1b26d55e","unresolved":false,"context_lines":[{"line_number":281,"context_line":"    differently in the tools directory.  "},{"line_number":282,"context_line":""},{"line_number":283,"context_line":""},{"line_number":284,"context_line":"Install Kolla for development"},{"line_number":285,"context_line":"-----------------------------"},{"line_number":286,"context_line":""},{"line_number":287,"context_line":"To clone the Kolla repo:"}],"source_content_type":"text/x-rst","patch_set":3,"id":"3ac371cc_1fd5291b","line":284,"range":{"start_line":284,"start_character":13,"end_line":284,"end_character":17},"updated":"2016-08-17 18:35:55.000000000","message":"change \"Kolla for\" to \"Kolla dependencies for\"","commit_id":"636a7c513636c93dc60f1e3793233d23cfd8f21c"},{"author":{"_account_id":13697,"name":"Leon Zachery","email":"lzachery@cisco.com","username":"lzachery"},"change_message_id":"c04ce5943278007d4d4253a10adff7227b8a1103","unresolved":false,"context_lines":[{"line_number":281,"context_line":"    differently in the tools directory.  "},{"line_number":282,"context_line":""},{"line_number":283,"context_line":""},{"line_number":284,"context_line":"Install Kolla for development"},{"line_number":285,"context_line":"-----------------------------"},{"line_number":286,"context_line":""},{"line_number":287,"context_line":"To clone the Kolla repo:"}],"source_content_type":"text/x-rst","patch_set":3,"id":"3ac371cc_910e2d39","line":284,"range":{"start_line":284,"start_character":13,"end_line":284,"end_character":17},"in_reply_to":"3ac371cc_1fd5291b","updated":"2016-08-17 19:02:36.000000000","message":"Done","commit_id":"636a7c513636c93dc60f1e3793233d23cfd8f21c"},{"author":{"_account_id":2834,"name":"Steven Dake","email":"steven.dake@gmail.com","username":"sdake"},"change_message_id":"296e8fa84f9e6f4d86cb949e27ed990e1b26d55e","unresolved":false,"context_lines":[{"line_number":290,"context_line":""},{"line_number":291,"context_line":"    git clone https://git.openstack.org/openstack/kolla"},{"line_number":292,"context_line":""},{"line_number":293,"context_line":"To install Kolla tools and Python dependencies use:"},{"line_number":294,"context_line":""},{"line_number":295,"context_line":"::"},{"line_number":296,"context_line":""}],"source_content_type":"text/x-rst","patch_set":3,"id":"3ac371cc_7f253d07","line":293,"range":{"start_line":293,"start_character":17,"end_line":293,"end_character":27},"updated":"2016-08-17 18:35:55.000000000","message":"delete","commit_id":"636a7c513636c93dc60f1e3793233d23cfd8f21c"},{"author":{"_account_id":13697,"name":"Leon Zachery","email":"lzachery@cisco.com","username":"lzachery"},"change_message_id":"c04ce5943278007d4d4253a10adff7227b8a1103","unresolved":false,"context_lines":[{"line_number":290,"context_line":""},{"line_number":291,"context_line":"    git clone https://git.openstack.org/openstack/kolla"},{"line_number":292,"context_line":""},{"line_number":293,"context_line":"To install Kolla tools and Python dependencies use:"},{"line_number":294,"context_line":""},{"line_number":295,"context_line":"::"},{"line_number":296,"context_line":""}],"source_content_type":"text/x-rst","patch_set":3,"id":"3ac371cc_9443bb41","line":293,"range":{"start_line":293,"start_character":17,"end_line":293,"end_character":27},"in_reply_to":"3ac371cc_7f253d07","updated":"2016-08-17 19:02:36.000000000","message":"Done","commit_id":"636a7c513636c93dc60f1e3793233d23cfd8f21c"},{"author":{"_account_id":16993,"name":"Hui Kang","email":"kangh@us.ibm.com","username":"huikang"},"change_message_id":"e5c3253fb65159e4d403c4831712b2b2e35bc423","unresolved":false,"context_lines":[{"line_number":294,"context_line":""},{"line_number":295,"context_line":"::"},{"line_number":296,"context_line":""},{"line_number":297,"context_line":"    pip install -r kolla/requirements.txt kolla/test-requirements.txt"},{"line_number":298,"context_line":""},{"line_number":299,"context_line":""},{"line_number":300,"context_line":"Kolla holds configurations files in ``etc/kolla``. Copy the configuration files"}],"source_content_type":"text/x-rst","patch_set":3,"id":"3ac371cc_3ed7764c","line":297,"updated":"2016-08-17 04:45:37.000000000","message":"Could you justify the benefit of expanding the command?","commit_id":"636a7c513636c93dc60f1e3793233d23cfd8f21c"},{"author":{"_account_id":7488,"name":"Jeffrey Zhang","email":"zhang.lei.fly@gmail.com","username":"Jeffrey4l"},"change_message_id":"83ee60d277265288aefd9067fc783813b0f7b9f9","unresolved":false,"context_lines":[{"line_number":294,"context_line":""},{"line_number":295,"context_line":"::"},{"line_number":296,"context_line":""},{"line_number":297,"context_line":"    pip install -r kolla/requirements.txt kolla/test-requirements.txt"},{"line_number":298,"context_line":""},{"line_number":299,"context_line":""},{"line_number":300,"context_line":"Kolla holds configurations files in ``etc/kolla``. Copy the configuration files"}],"source_content_type":"text/x-rst","patch_set":3,"id":"3ac371cc_b271c3aa","line":297,"updated":"2016-08-17 12:35:09.000000000","message":"pip install -r kolla/requirements.txt -r kolla/test-requirements.txt\n\nleft out a `-r`","commit_id":"636a7c513636c93dc60f1e3793233d23cfd8f21c"},{"author":{"_account_id":1390,"name":"Paul Bourke","email":"pauldbourke@proton.me","username":"pauldbourke"},"change_message_id":"72882bfeaec47de960fb14c8d7d1ee95e571d5e4","unresolved":false,"context_lines":[{"line_number":294,"context_line":""},{"line_number":295,"context_line":"::"},{"line_number":296,"context_line":""},{"line_number":297,"context_line":"    pip install -r kolla/requirements.txt kolla/test-requirements.txt"},{"line_number":298,"context_line":""},{"line_number":299,"context_line":""},{"line_number":300,"context_line":"Kolla holds configurations files in ``etc/kolla``. Copy the configuration files"}],"source_content_type":"text/x-rst","patch_set":3,"id":"3ac371cc_90a8078f","line":297,"in_reply_to":"3ac371cc_3ed7764c","updated":"2016-08-17 10:52:18.000000000","message":"\u0027pip install .\u0027 is the preferred command","commit_id":"636a7c513636c93dc60f1e3793233d23cfd8f21c"},{"author":{"_account_id":2834,"name":"Steven Dake","email":"steven.dake@gmail.com","username":"sdake"},"change_message_id":"1ff7cf01f333a3a5bc812463e3ec647e43e68d17","unresolved":false,"context_lines":[{"line_number":294,"context_line":""},{"line_number":295,"context_line":"::"},{"line_number":296,"context_line":""},{"line_number":297,"context_line":"    pip install -r kolla/requirements.txt kolla/test-requirements.txt"},{"line_number":298,"context_line":""},{"line_number":299,"context_line":""},{"line_number":300,"context_line":"Kolla holds configurations files in ``etc/kolla``. Copy the configuration files"}],"source_content_type":"text/x-rst","patch_set":3,"id":"3ac371cc_b0d2635a","line":297,"in_reply_to":"3ac371cc_3ed7764c","updated":"2016-08-17 10:52:57.000000000","message":"this causes pip not to install kolla but install the requirements.  This is correct and what i\u0027ve recommended dozens of times on irc.","commit_id":"636a7c513636c93dc60f1e3793233d23cfd8f21c"},{"author":{"_account_id":2834,"name":"Steven Dake","email":"steven.dake@gmail.com","username":"sdake"},"change_message_id":"296e8fa84f9e6f4d86cb949e27ed990e1b26d55e","unresolved":false,"context_lines":[{"line_number":294,"context_line":""},{"line_number":295,"context_line":"::"},{"line_number":296,"context_line":""},{"line_number":297,"context_line":"    pip install -r kolla/requirements.txt kolla/test-requirements.txt"},{"line_number":298,"context_line":""},{"line_number":299,"context_line":""},{"line_number":300,"context_line":"Kolla holds configurations files in ``etc/kolla``. Copy the configuration files"}],"source_content_type":"text/x-rst","patch_set":3,"id":"3ac371cc_ff648db9","line":297,"in_reply_to":"3ac371cc_b271c3aa","updated":"2016-08-17 18:35:55.000000000","message":"nice catch jeffrey!","commit_id":"636a7c513636c93dc60f1e3793233d23cfd8f21c"},{"author":{"_account_id":13697,"name":"Leon Zachery","email":"lzachery@cisco.com","username":"lzachery"},"change_message_id":"c04ce5943278007d4d4253a10adff7227b8a1103","unresolved":false,"context_lines":[{"line_number":294,"context_line":""},{"line_number":295,"context_line":"::"},{"line_number":296,"context_line":""},{"line_number":297,"context_line":"    pip install -r kolla/requirements.txt kolla/test-requirements.txt"},{"line_number":298,"context_line":""},{"line_number":299,"context_line":""},{"line_number":300,"context_line":"Kolla holds configurations files in ``etc/kolla``. Copy the configuration files"}],"source_content_type":"text/x-rst","patch_set":3,"id":"3ac371cc_b45d1717","line":297,"in_reply_to":"3ac371cc_ff648db9","updated":"2016-08-17 19:02:36.000000000","message":"Done","commit_id":"636a7c513636c93dc60f1e3793233d23cfd8f21c"},{"author":{"_account_id":2834,"name":"Steven Dake","email":"steven.dake@gmail.com","username":"sdake"},"change_message_id":"dcc76bc64f03de4a6a346b08972b7b089f42d8ad","unresolved":false,"context_lines":[{"line_number":49,"context_line":""},{"line_number":50,"context_line":"    apt-get install linux-image-generic-lts-wily"},{"line_number":51,"context_line":""},{"line_number":52,"context_line":".. note:: Operators performing an evaluation or deployment should use a"},{"line_number":53,"context_line":"  stable branch.  Operators performing development or developers should use"},{"line_number":54,"context_line":"  master."},{"line_number":55,"context_line":""}],"source_content_type":"text/x-rst","patch_set":4,"id":"3ac371cc_f4c2efe7","line":52,"range":{"start_line":52,"start_character":3,"end_line":52,"end_character":7},"updated":"2016-08-17 19:23:29.000000000","message":"change note here to attention","commit_id":"b8c163566026775d06e1c2fa941b065be01dc120"},{"author":{"_account_id":2834,"name":"Steven Dake","email":"steven.dake@gmail.com","username":"sdake"},"change_message_id":"dcc76bc64f03de4a6a346b08972b7b089f42d8ad","unresolved":false,"context_lines":[{"line_number":49,"context_line":""},{"line_number":50,"context_line":"    apt-get install linux-image-generic-lts-wily"},{"line_number":51,"context_line":""},{"line_number":52,"context_line":".. note:: Operators performing an evaluation or deployment should use a"},{"line_number":53,"context_line":"  stable branch.  Operators performing development or developers should use"},{"line_number":54,"context_line":"  master."},{"line_number":55,"context_line":""}],"source_content_type":"text/x-rst","patch_set":4,"id":"3ac371cc_34a2c74b","line":52,"range":{"start_line":52,"start_character":2,"end_line":52,"end_character":3},"updated":"2016-08-17 19:23:29.000000000","message":"delete extra space","commit_id":"b8c163566026775d06e1c2fa941b065be01dc120"},{"author":{"_account_id":2834,"name":"Steven Dake","email":"steven.dake@gmail.com","username":"sdake"},"change_message_id":"dcc76bc64f03de4a6a346b08972b7b089f42d8ad","unresolved":false,"context_lines":[{"line_number":295,"context_line":"    pip install -r kolla/requirements.txt -r kolla/test-requirements.txt"},{"line_number":296,"context_line":""},{"line_number":297,"context_line":"..note:: This does not actually install Kolla.  Many commands in this documentation are named"},{"line_number":298,"context_line":"    differently in the tools directory.  "},{"line_number":299,"context_line":""},{"line_number":300,"context_line":"Kolla holds configurations files in ``etc/kolla``. Copy the configuration files"},{"line_number":301,"context_line":"to ``/etc``:"}],"source_content_type":"text/x-rst","patch_set":4,"id":"3ac371cc_9479fbca","line":298,"range":{"start_line":298,"start_character":39,"end_line":298,"end_character":40},"updated":"2016-08-17 19:23:29.000000000","message":"remove extra whitespace","commit_id":"b8c163566026775d06e1c2fa941b065be01dc120"},{"author":{"_account_id":2834,"name":"Steven Dake","email":"steven.dake@gmail.com","username":"sdake"},"change_message_id":"6e175de1140b2cb532f45528269a4ab6280845dc","unresolved":false,"context_lines":[{"line_number":253,"context_line":""},{"line_number":254,"context_line":"    apt-get install ansible"},{"line_number":255,"context_line":""},{"line_number":256,"context_line":"..attention:: Kolla uses PBR in its implementation.  PBR provides version "},{"line_number":257,"context_line":"    information to Kolla about the package in use.  This information is later "},{"line_number":258,"context_line":"    used when building images to specify the Docker tag used in the image built.  "},{"line_number":259,"context_line":"    When installing the Kolla package via pip, PBR will always use the PBR version "}],"source_content_type":"text/x-rst","patch_set":5,"id":"3ac371cc_5f0af78a","line":256,"updated":"2016-08-17 21:49:27.000000000","message":"I was incorrect before.  Syntax of notes/attentions/etc are\n.. keyword::\n\nThat is a space between the .. and keyword (which is attention).","commit_id":"44ef26341975d4918f088b5de40a09f1a0ef4350"},{"author":{"_account_id":16993,"name":"Hui Kang","email":"kangh@us.ibm.com","username":"huikang"},"change_message_id":"d540fef21912b1632016c98871365fd928b0e207","unresolved":false,"context_lines":[{"line_number":257,"context_line":"    information to Kolla about the package in use.  This information is later "},{"line_number":258,"context_line":"    used when building images to specify the Docker tag used in the image built.  "},{"line_number":259,"context_line":"    When installing the Kolla package via pip, PBR will always use the PBR version "},{"line_number":260,"context_line":"    information.  When obtaining a copy of the software via git, PBR will use the "},{"line_number":261,"context_line":"    git version information, but **ONLY** if Kolla has not been pip installed via "},{"line_number":262,"context_line":"    the pip package manager.  This is why there is an operator workflow and a "},{"line_number":263,"context_line":"    developer workflow."}],"source_content_type":"text/x-rst","patch_set":5,"id":"3ac371cc_b9a18048","line":260,"updated":"2016-08-18 02:22:27.000000000","message":"extra space between \".\" and When","commit_id":"44ef26341975d4918f088b5de40a09f1a0ef4350"},{"author":{"_account_id":16993,"name":"Hui Kang","email":"kangh@us.ibm.com","username":"huikang"},"change_message_id":"d540fef21912b1632016c98871365fd928b0e207","unresolved":false,"context_lines":[{"line_number":259,"context_line":"    When installing the Kolla package via pip, PBR will always use the PBR version "},{"line_number":260,"context_line":"    information.  When obtaining a copy of the software via git, PBR will use the "},{"line_number":261,"context_line":"    git version information, but **ONLY** if Kolla has not been pip installed via "},{"line_number":262,"context_line":"    the pip package manager.  This is why there is an operator workflow and a "},{"line_number":263,"context_line":"    developer workflow."},{"line_number":264,"context_line":""},{"line_number":265,"context_line":"Installing Kolla for evaluation or deployment"}],"source_content_type":"text/x-rst","patch_set":5,"id":"3ac371cc_398e30d0","line":262,"updated":"2016-08-18 02:22:27.000000000","message":"extra space between . and This","commit_id":"44ef26341975d4918f088b5de40a09f1a0ef4350"},{"author":{"_account_id":16993,"name":"Hui Kang","email":"kangh@us.ibm.com","username":"huikang"},"change_message_id":"d540fef21912b1632016c98871365fd928b0e207","unresolved":false,"context_lines":[{"line_number":277,"context_line":"::"},{"line_number":278,"context_line":""},{"line_number":279,"context_line":"    cd kolla"},{"line_number":280,"context_line":"    cp -r etc/kolla /etc/"},{"line_number":281,"context_line":""},{"line_number":282,"context_line":"Installing Kolla and dependencies for development"},{"line_number":283,"context_line":"-------------------------------------------------"}],"source_content_type":"text/x-rst","patch_set":5,"id":"3ac371cc_39a59046","line":280,"updated":"2016-08-18 02:22:27.000000000","message":"Another option is to run tox -e genconfig","commit_id":"44ef26341975d4918f088b5de40a09f1a0ef4350"},{"author":{"_account_id":2834,"name":"Steven Dake","email":"steven.dake@gmail.com","username":"sdake"},"change_message_id":"6e175de1140b2cb532f45528269a4ab6280845dc","unresolved":false,"context_lines":[{"line_number":294,"context_line":""},{"line_number":295,"context_line":"    pip install -r kolla/requirements.txt -r kolla/test-requirements.txt"},{"line_number":296,"context_line":""},{"line_number":297,"context_line":"..note:: This does not actually install Kolla.  Many commands in this documentation are named"},{"line_number":298,"context_line":"    differently in the tools directory."},{"line_number":299,"context_line":""},{"line_number":300,"context_line":"Kolla holds configurations files in ``etc/kolla``. Copy the configuration files"}],"source_content_type":"text/x-rst","patch_set":5,"id":"3ac371cc_dff6e78d","line":297,"updated":"2016-08-17 21:49:27.000000000","message":"need space here between .. and note","commit_id":"44ef26341975d4918f088b5de40a09f1a0ef4350"},{"author":{"_account_id":16993,"name":"Hui Kang","email":"kangh@us.ibm.com","username":"huikang"},"change_message_id":"d540fef21912b1632016c98871365fd928b0e207","unresolved":false,"context_lines":[{"line_number":294,"context_line":""},{"line_number":295,"context_line":"    pip install -r kolla/requirements.txt -r kolla/test-requirements.txt"},{"line_number":296,"context_line":""},{"line_number":297,"context_line":"..note:: This does not actually install Kolla.  Many commands in this documentation are named"},{"line_number":298,"context_line":"    differently in the tools directory."},{"line_number":299,"context_line":""},{"line_number":300,"context_line":"Kolla holds configurations files in ``etc/kolla``. Copy the configuration files"}],"source_content_type":"text/x-rst","patch_set":5,"id":"3ac371cc_59816c8e","line":297,"in_reply_to":"3ac371cc_dff6e78d","updated":"2016-08-18 02:22:27.000000000","message":"extra space between . and Many","commit_id":"44ef26341975d4918f088b5de40a09f1a0ef4350"},{"author":{"_account_id":167,"name":"Christian Berendt","email":"berendt@osism.tech","username":"berendt"},"change_message_id":"c9b31acb3522637715372dcfb477a055bca6c5a1","unresolved":false,"context_lines":[{"line_number":50,"context_line":"    apt-get install linux-image-generic-lts-wily"},{"line_number":51,"context_line":""},{"line_number":52,"context_line":".. attention:: Operators performing an evaluation or deployment should use a"},{"line_number":53,"context_line":"  stable branch.  Operators performing development or developers should use"},{"line_number":54,"context_line":"  master."},{"line_number":55,"context_line":""},{"line_number":56,"context_line":".. note:: Install is *very* sensitive about version of components. Please"},{"line_number":57,"context_line":"  review carefully because default Operating System repos are likely out of"}],"source_content_type":"text/x-rst","patch_set":7,"id":"3ac371cc_95e9d9b9","line":54,"range":{"start_line":53,"start_character":54,"end_line":54,"end_character":9},"updated":"2016-08-18 08:30:33.000000000","message":"I think developers must use the master branch. It makes no sense to develop on a stable branch.","commit_id":"ba60daf55325c0faac42b2189c838310dd0092ad"},{"author":{"_account_id":1390,"name":"Paul Bourke","email":"pauldbourke@proton.me","username":"pauldbourke"},"change_message_id":"233b36f556b03eaab6589a8b59e7bd9694bab581","unresolved":false,"context_lines":[{"line_number":50,"context_line":"    apt-get install linux-image-generic-lts-wily"},{"line_number":51,"context_line":""},{"line_number":52,"context_line":".. attention:: Operators performing an evaluation or deployment should use a"},{"line_number":53,"context_line":"  stable branch.  Operators performing development or developers should use"},{"line_number":54,"context_line":"  master."},{"line_number":55,"context_line":""},{"line_number":56,"context_line":".. note:: Install is *very* sensitive about version of components. Please"},{"line_number":57,"context_line":"  review carefully because default Operating System repos are likely out of"}],"source_content_type":"text/x-rst","patch_set":7,"id":"3ac371cc_a9b3d996","line":54,"range":{"start_line":53,"start_character":54,"end_line":54,"end_character":9},"in_reply_to":"3ac371cc_95e9d9b9","updated":"2016-08-18 11:13:05.000000000","message":"It sounds like that\u0027s what\u0027s being said here? Though I think it\u0027s best to just remove the second sentence completely, it\u0027s obvious developers should use master","commit_id":"ba60daf55325c0faac42b2189c838310dd0092ad"},{"author":{"_account_id":2834,"name":"Steven Dake","email":"steven.dake@gmail.com","username":"sdake"},"change_message_id":"cd835d348fa171037dcba5c54ca7bfdab74ef388","unresolved":false,"context_lines":[{"line_number":50,"context_line":"    apt-get install linux-image-generic-lts-wily"},{"line_number":51,"context_line":""},{"line_number":52,"context_line":".. attention:: Operators performing an evaluation or deployment should use a"},{"line_number":53,"context_line":"  stable branch.  Operators performing development or developers should use"},{"line_number":54,"context_line":"  master."},{"line_number":55,"context_line":""},{"line_number":56,"context_line":".. note:: Install is *very* sensitive about version of components. Please"},{"line_number":57,"context_line":"  review carefully because default Operating System repos are likely out of"}],"source_content_type":"text/x-rst","patch_set":7,"id":"3ac371cc_db9250f5","line":54,"range":{"start_line":53,"start_character":54,"end_line":54,"end_character":9},"in_reply_to":"3ac371cc_a9b3d996","updated":"2016-08-18 13:19:57.000000000","message":"I htink it should be left as is - to point out the delta between operating on a master branch and stable branch (one for developers one for operators).  It is obvious for developers they should be developing on master.  It isn\u0027t obvious to ops always that we use a master first approach.","commit_id":"ba60daf55325c0faac42b2189c838310dd0092ad"},{"author":{"_account_id":167,"name":"Christian Berendt","email":"berendt@osism.tech","username":"berendt"},"change_message_id":"c9b31acb3522637715372dcfb477a055bca6c5a1","unresolved":false,"context_lines":[{"line_number":74,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d   \u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d  \u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d  \u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":75,"context_line":"Component               Min Version  Max Version  Comment"},{"line_number":76,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d   \u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d  \u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d  \u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":77,"context_line":"Ansible                 2.0.0        none         On deployment host"},{"line_number":78,"context_line":"Docker                  1.10.0       none         On target nodes"},{"line_number":79,"context_line":"Docker Python           1.6.0        none         On target nodes"},{"line_number":80,"context_line":"Python Jinja2           2.8.0        none         On deployment host"}],"source_content_type":"text/x-rst","patch_set":7,"id":"3ac371cc_75b9bdcb","line":77,"range":{"start_line":77,"start_character":37,"end_line":77,"end_character":41},"updated":"2016-08-18 08:30:33.000000000","message":"Do we have to set \u003c 2.2 here?","commit_id":"ba60daf55325c0faac42b2189c838310dd0092ad"},{"author":{"_account_id":1390,"name":"Paul Bourke","email":"pauldbourke@proton.me","username":"pauldbourke"},"change_message_id":"233b36f556b03eaab6589a8b59e7bd9694bab581","unresolved":false,"context_lines":[{"line_number":74,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d   \u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d  \u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d  \u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":75,"context_line":"Component               Min Version  Max Version  Comment"},{"line_number":76,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d   \u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d  \u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d  \u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":77,"context_line":"Ansible                 2.0.0        none         On deployment host"},{"line_number":78,"context_line":"Docker                  1.10.0       none         On target nodes"},{"line_number":79,"context_line":"Docker Python           1.6.0        none         On target nodes"},{"line_number":80,"context_line":"Python Jinja2           2.8.0        none         On deployment host"}],"source_content_type":"text/x-rst","patch_set":7,"id":"3ac371cc_89753dcc","line":77,"range":{"start_line":77,"start_character":37,"end_line":77,"end_character":41},"in_reply_to":"3ac371cc_75b9bdcb","updated":"2016-08-18 11:13:05.000000000","message":"I dont see a need?","commit_id":"ba60daf55325c0faac42b2189c838310dd0092ad"},{"author":{"_account_id":2834,"name":"Steven Dake","email":"steven.dake@gmail.com","username":"sdake"},"change_message_id":"cd835d348fa171037dcba5c54ca7bfdab74ef388","unresolved":false,"context_lines":[{"line_number":74,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d   \u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d  \u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d  \u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":75,"context_line":"Component               Min Version  Max Version  Comment"},{"line_number":76,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d   \u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d  \u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d  \u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":77,"context_line":"Ansible                 2.0.0        none         On deployment host"},{"line_number":78,"context_line":"Docker                  1.10.0       none         On target nodes"},{"line_number":79,"context_line":"Docker Python           1.6.0        none         On target nodes"},{"line_number":80,"context_line":"Python Jinja2           2.8.0        none         On deployment host"}],"source_content_type":"text/x-rst","patch_set":7,"id":"3ac371cc_7b751cc9","line":77,"range":{"start_line":77,"start_character":37,"end_line":77,"end_character":41},"in_reply_to":"3ac371cc_89753dcc","updated":"2016-08-18 13:19:57.000000000","message":"if it breaks with 2.2 we will add a max version to the docs until we fix the code.","commit_id":"ba60daf55325c0faac42b2189c838310dd0092ad"},{"author":{"_account_id":167,"name":"Christian Berendt","email":"berendt@osism.tech","username":"berendt"},"change_message_id":"c9b31acb3522637715372dcfb477a055bca6c5a1","unresolved":false,"context_lines":[{"line_number":269,"context_line":""},{"line_number":270,"context_line":"::"},{"line_number":271,"context_line":""},{"line_number":272,"context_line":"    pip install kolla\u003d\u003d2.0.2"},{"line_number":273,"context_line":""},{"line_number":274,"context_line":"Kolla holds configurations files in ``etc/kolla``. Copy the configuration files"},{"line_number":275,"context_line":"to ``/etc``:"}],"source_content_type":"text/x-rst","patch_set":7,"id":"3ac371cc_7507fd81","line":272,"range":{"start_line":272,"start_character":16,"end_line":272,"end_character":28},"updated":"2016-08-18 08:30:33.000000000","message":"Why is it necessary to install 2.0.2? When we will release a 2.0.3 we will have to take care of this document.","commit_id":"ba60daf55325c0faac42b2189c838310dd0092ad"},{"author":{"_account_id":2834,"name":"Steven Dake","email":"steven.dake@gmail.com","username":"sdake"},"change_message_id":"cd835d348fa171037dcba5c54ca7bfdab74ef388","unresolved":false,"context_lines":[{"line_number":269,"context_line":""},{"line_number":270,"context_line":"::"},{"line_number":271,"context_line":""},{"line_number":272,"context_line":"    pip install kolla\u003d\u003d2.0.2"},{"line_number":273,"context_line":""},{"line_number":274,"context_line":"Kolla holds configurations files in ``etc/kolla``. Copy the configuration files"},{"line_number":275,"context_line":"to ``/etc``:"}],"source_content_type":"text/x-rst","patch_set":7,"id":"3ac371cc_565755c9","line":272,"range":{"start_line":272,"start_character":16,"end_line":272,"end_character":28},"in_reply_to":"3ac371cc_091e6d6f","updated":"2016-08-18 13:19:57.000000000","message":"I provided guidance in a previous review that the version should be pinned.  This is because I thought the milestone releases were being published to pypi.  That is not the case.  So a non-version pin here will get the job done.\n\nLeon please remove the highlighted section.","commit_id":"ba60daf55325c0faac42b2189c838310dd0092ad"},{"author":{"_account_id":1390,"name":"Paul Bourke","email":"pauldbourke@proton.me","username":"pauldbourke"},"change_message_id":"233b36f556b03eaab6589a8b59e7bd9694bab581","unresolved":false,"context_lines":[{"line_number":269,"context_line":""},{"line_number":270,"context_line":"::"},{"line_number":271,"context_line":""},{"line_number":272,"context_line":"    pip install kolla\u003d\u003d2.0.2"},{"line_number":273,"context_line":""},{"line_number":274,"context_line":"Kolla holds configurations files in ``etc/kolla``. Copy the configuration files"},{"line_number":275,"context_line":"to ``/etc``:"}],"source_content_type":"text/x-rst","patch_set":7,"id":"3ac371cc_091e6d6f","line":272,"range":{"start_line":272,"start_character":16,"end_line":272,"end_character":28},"in_reply_to":"3ac371cc_7507fd81","updated":"2016-08-18 11:13:05.000000000","message":"+1 - I see no need to pin a version here","commit_id":"ba60daf55325c0faac42b2189c838310dd0092ad"},{"author":{"_account_id":167,"name":"Christian Berendt","email":"berendt@osism.tech","username":"berendt"},"change_message_id":"c9b31acb3522637715372dcfb477a055bca6c5a1","unresolved":false,"context_lines":[{"line_number":292,"context_line":""},{"line_number":293,"context_line":"::"},{"line_number":294,"context_line":""},{"line_number":295,"context_line":"    pip install -r kolla/requirements.txt -r kolla/test-requirements.txt"},{"line_number":296,"context_line":""},{"line_number":297,"context_line":".. note:: This does not actually install Kolla. Many commands in this documentation are named"},{"line_number":298,"context_line":"    differently in the tools directory."}],"source_content_type":"text/x-rst","patch_set":7,"id":"3ac371cc_75153d9a","line":295,"updated":"2016-08-18 08:30:33.000000000","message":"Why should this replace pip install kolla/ ?","commit_id":"ba60daf55325c0faac42b2189c838310dd0092ad"},{"author":{"_account_id":1390,"name":"Paul Bourke","email":"pauldbourke@proton.me","username":"pauldbourke"},"change_message_id":"233b36f556b03eaab6589a8b59e7bd9694bab581","unresolved":false,"context_lines":[{"line_number":292,"context_line":""},{"line_number":293,"context_line":"::"},{"line_number":294,"context_line":""},{"line_number":295,"context_line":"    pip install -r kolla/requirements.txt -r kolla/test-requirements.txt"},{"line_number":296,"context_line":""},{"line_number":297,"context_line":".. note:: This does not actually install Kolla. Many commands in this documentation are named"},{"line_number":298,"context_line":"    differently in the tools directory."}],"source_content_type":"text/x-rst","patch_set":7,"id":"3ac371cc_890a5d24","line":295,"in_reply_to":"3ac371cc_75153d9a","updated":"2016-08-18 11:13:05.000000000","message":"I think because pip install kolla/ will install the package and cause version confusion (see pbr note above)","commit_id":"ba60daf55325c0faac42b2189c838310dd0092ad"},{"author":{"_account_id":2834,"name":"Steven Dake","email":"steven.dake@gmail.com","username":"sdake"},"change_message_id":"cd835d348fa171037dcba5c54ca7bfdab74ef388","unresolved":false,"context_lines":[{"line_number":292,"context_line":""},{"line_number":293,"context_line":"::"},{"line_number":294,"context_line":""},{"line_number":295,"context_line":"    pip install -r kolla/requirements.txt -r kolla/test-requirements.txt"},{"line_number":296,"context_line":""},{"line_number":297,"context_line":".. note:: This does not actually install Kolla. Many commands in this documentation are named"},{"line_number":298,"context_line":"    differently in the tools directory."}],"source_content_type":"text/x-rst","patch_set":7,"id":"3ac371cc_96fc2db9","line":295,"in_reply_to":"3ac371cc_890a5d24","updated":"2016-08-18 13:19:57.000000000","message":"pbourke - ack on that","commit_id":"ba60daf55325c0faac42b2189c838310dd0092ad"},{"author":{"_account_id":1390,"name":"Paul Bourke","email":"pauldbourke@proton.me","username":"pauldbourke"},"change_message_id":"a65b5731fa225079ffa2a4b3d0aad6c5a602edfb","unresolved":false,"context_lines":[{"line_number":269,"context_line":""},{"line_number":270,"context_line":"::"},{"line_number":271,"context_line":""},{"line_number":272,"context_line":"    pip install kolla\u003d\u003d2.0.2"},{"line_number":273,"context_line":""},{"line_number":274,"context_line":"Kolla holds configurations files in ``etc/kolla``. Copy the configuration files"},{"line_number":275,"context_line":"to ``/etc``:"}],"source_content_type":"text/x-rst","patch_set":8,"id":"3ac371cc_1177a7a0","line":272,"range":{"start_line":272,"start_character":21,"end_line":272,"end_character":28},"updated":"2016-08-18 13:28:14.000000000","message":"Please remove the pin","commit_id":"0fbcb5176c9d0d852db1da274625c67b88bb3db8"},{"author":{"_account_id":2834,"name":"Steven Dake","email":"steven.dake@gmail.com","username":"sdake"},"change_message_id":"81223583f0339ac4f3717784187e1ca2860f76f6","unresolved":false,"context_lines":[{"line_number":269,"context_line":""},{"line_number":270,"context_line":"::"},{"line_number":271,"context_line":""},{"line_number":272,"context_line":"    pip install kolla\u003d\u003d2.0.2"},{"line_number":273,"context_line":""},{"line_number":274,"context_line":"Kolla holds configurations files in ``etc/kolla``. Copy the configuration files"},{"line_number":275,"context_line":"to ``/etc``:"}],"source_content_type":"text/x-rst","patch_set":8,"id":"3ac371cc_510e3fe5","line":272,"range":{"start_line":272,"start_character":21,"end_line":272,"end_character":28},"in_reply_to":"3ac371cc_1177a7a0","updated":"2016-08-18 13:30:27.000000000","message":"ya just delete the highlighted section and this will merge this morning.\n\nthanks","commit_id":"0fbcb5176c9d0d852db1da274625c67b88bb3db8"},{"author":{"_account_id":979,"name":"Dave Walker","email":"email@daviey.com","username":"davewalker"},"change_message_id":"e926a587d0374081fa690beb8565478aadac9288","unresolved":false,"context_lines":[{"line_number":50,"context_line":"    apt-get install linux-image-generic-lts-wily"},{"line_number":51,"context_line":""},{"line_number":52,"context_line":".. attention:: Operators performing an evaluation or deployment should use a"},{"line_number":53,"context_line":"  stable branch.  Operators performing development (or developers) should use"},{"line_number":54,"context_line":"  master."},{"line_number":55,"context_line":""},{"line_number":56,"context_line":".. note:: Install is *very* sensitive about version of components. Please"}],"source_content_type":"text/x-rst","patch_set":9,"id":"3ac371cc_24fec738","line":53,"range":{"start_line":53,"start_character":9,"end_line":53,"end_character":15},"updated":"2016-08-18 20:40:37.000000000","message":"s/branch/release.  Either via their linux distribution packages, or via \"pip install kolla\"","commit_id":"8cd0d2a72ac8a995f7b07293561b76b0be04f52a"},{"author":{"_account_id":2834,"name":"Steven Dake","email":"steven.dake@gmail.com","username":"sdake"},"change_message_id":"b09cfb04aa852c1f846bb98449108c7420c87441","unresolved":false,"context_lines":[{"line_number":50,"context_line":"    apt-get install linux-image-generic-lts-wily"},{"line_number":51,"context_line":""},{"line_number":52,"context_line":".. attention:: Operators performing an evaluation or deployment should use a"},{"line_number":53,"context_line":"  stable branch.  Operators performing development (or developers) should use"},{"line_number":54,"context_line":"  master."},{"line_number":55,"context_line":""},{"line_number":56,"context_line":".. note:: Install is *very* sensitive about version of components. Please"}],"source_content_type":"text/x-rst","patch_set":9,"id":"3ac371cc_df77bc06","line":53,"range":{"start_line":53,"start_character":9,"end_line":53,"end_character":15},"in_reply_to":"3ac371cc_24fec738","updated":"2016-08-18 20:48:06.000000000","message":"cool we can tackle this in a followup patch since this one is merged.","commit_id":"8cd0d2a72ac8a995f7b07293561b76b0be04f52a"},{"author":{"_account_id":979,"name":"Dave Walker","email":"email@daviey.com","username":"davewalker"},"change_message_id":"e926a587d0374081fa690beb8565478aadac9288","unresolved":false,"context_lines":[{"line_number":57,"context_line":"  review carefully because default Operating System repos are likely out of"},{"line_number":58,"context_line":"  date."},{"line_number":59,"context_line":""},{"line_number":60,"context_line":"Dependencies for the stable branch are:"},{"line_number":61,"context_line":""},{"line_number":62,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d   \u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d  \u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d  \u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":63,"context_line":"Component               Min Version  Max Version  Comment"}],"source_content_type":"text/x-rst","patch_set":9,"id":"3ac371cc_440b032c","line":60,"updated":"2016-08-18 20:40:37.000000000","message":"Honestly, i\u0027d prefer if all of this was just tracked via \"kolla-ansible prechecks\".. This document will always be lagging behind, but running precheck should always be accurate.\n\nSomething like, \"kolla internal dependencies are satisfied during the installation but further dependencies, such as ansible, docker and the python bindings for docker can be validated as part of kolla-ansible precheck command\".","commit_id":"8cd0d2a72ac8a995f7b07293561b76b0be04f52a"},{"author":{"_account_id":2834,"name":"Steven Dake","email":"steven.dake@gmail.com","username":"sdake"},"change_message_id":"b09cfb04aa852c1f846bb98449108c7420c87441","unresolved":false,"context_lines":[{"line_number":57,"context_line":"  review carefully because default Operating System repos are likely out of"},{"line_number":58,"context_line":"  date."},{"line_number":59,"context_line":""},{"line_number":60,"context_line":"Dependencies for the stable branch are:"},{"line_number":61,"context_line":""},{"line_number":62,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d   \u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d  \u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d  \u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":63,"context_line":"Component               Min Version  Max Version  Comment"}],"source_content_type":"text/x-rst","patch_set":9,"id":"3ac371cc_9ffb8447","line":60,"in_reply_to":"3ac371cc_440b032c","updated":"2016-08-18 20:48:06.000000000","message":"great idea.  again - emergency situation - spend 3+ hrs a day debugging peoples environments and this is a simple solution that we can point to in our documentation.  Also tellling people upfront which deps to isntall is helpful, rather then seeing a precheck fail and saying \"I\u0027ve got the wrong thing installed on 100 nodes...\".","commit_id":"8cd0d2a72ac8a995f7b07293561b76b0be04f52a"},{"author":{"_account_id":979,"name":"Dave Walker","email":"email@daviey.com","username":"davewalker"},"change_message_id":"e926a587d0374081fa690beb8565478aadac9288","unresolved":false,"context_lines":[{"line_number":253,"context_line":""},{"line_number":254,"context_line":"    apt-get install ansible"},{"line_number":255,"context_line":""},{"line_number":256,"context_line":".. attention:: Kolla uses PBR in its implementation. PBR provides version "},{"line_number":257,"context_line":"    information to Kolla about the package in use. This information is later "},{"line_number":258,"context_line":"    used when building images to specify the Docker tag used in the image built.  "},{"line_number":259,"context_line":"    When installing the Kolla package via pip, PBR will always use the PBR version "}],"source_content_type":"text/x-rst","patch_set":9,"id":"3ac371cc_df721cb0","line":256,"updated":"2016-08-18 20:40:37.000000000","message":"Uhm, this is what I have been pushing for.. but I haven\u0027t done it yet.  Searching git, doesn\u0027t look like anyone else has.\n\nHave i missed something?","commit_id":"8cd0d2a72ac8a995f7b07293561b76b0be04f52a"},{"author":{"_account_id":2834,"name":"Steven Dake","email":"steven.dake@gmail.com","username":"sdake"},"change_message_id":"b09cfb04aa852c1f846bb98449108c7420c87441","unresolved":false,"context_lines":[{"line_number":253,"context_line":""},{"line_number":254,"context_line":"    apt-get install ansible"},{"line_number":255,"context_line":""},{"line_number":256,"context_line":".. attention:: Kolla uses PBR in its implementation. PBR provides version "},{"line_number":257,"context_line":"    information to Kolla about the package in use. This information is later "},{"line_number":258,"context_line":"    used when building images to specify the Docker tag used in the image built.  "},{"line_number":259,"context_line":"    When installing the Kolla package via pip, PBR will always use the PBR version "}],"source_content_type":"text/x-rst","patch_set":9,"id":"3ac371cc_bfbf605f","line":256,"in_reply_to":"3ac371cc_df721cb0","updated":"2016-08-18 20:48:06.000000000","message":"Which have you been pushing for?  To fix pbr or to have two types of workflows - one for developers and one for eval/operators?","commit_id":"8cd0d2a72ac8a995f7b07293561b76b0be04f52a"},{"author":{"_account_id":979,"name":"Dave Walker","email":"email@daviey.com","username":"davewalker"},"change_message_id":"e926a587d0374081fa690beb8565478aadac9288","unresolved":false,"context_lines":[{"line_number":292,"context_line":""},{"line_number":293,"context_line":"::"},{"line_number":294,"context_line":""},{"line_number":295,"context_line":"    pip install -r kolla/requirements.txt -r kolla/test-requirements.txt"},{"line_number":296,"context_line":""},{"line_number":297,"context_line":".. note:: This does not actually install Kolla. Many commands in this documentation are named"},{"line_number":298,"context_line":"    differently in the tools directory."}],"source_content_type":"text/x-rst","patch_set":9,"id":"3ac371cc_7fede8ee","line":295,"updated":"2016-08-18 20:40:37.000000000","message":"This is confusing.. such that test requirements are only needed to run the tests., nothing else.. and the usual requirements are satisfied as part of a normal \"pip install kolla\".  I don\u0027t think this section is needed, but if it is kept - it needs to be explicitly mentioned why you\u0027d want this.\n\nMost developers should probably be using virtualenv\u0027s for a developers environment.. So this needs to be expanded for that for a developers environment (and then this is OT for quickstart)","commit_id":"8cd0d2a72ac8a995f7b07293561b76b0be04f52a"},{"author":{"_account_id":2834,"name":"Steven Dake","email":"steven.dake@gmail.com","username":"sdake"},"change_message_id":"b09cfb04aa852c1f846bb98449108c7420c87441","unresolved":false,"context_lines":[{"line_number":292,"context_line":""},{"line_number":293,"context_line":"::"},{"line_number":294,"context_line":""},{"line_number":295,"context_line":"    pip install -r kolla/requirements.txt -r kolla/test-requirements.txt"},{"line_number":296,"context_line":""},{"line_number":297,"context_line":".. note:: This does not actually install Kolla. Many commands in this documentation are named"},{"line_number":298,"context_line":"    differently in the tools directory."}],"source_content_type":"text/x-rst","patch_set":9,"id":"3ac371cc_ff38b83f","line":295,"in_reply_to":"3ac371cc_7fede8ee","updated":"2016-08-18 20:48:06.000000000","message":"in this second workflow (Installing Kolla and dependencies for development) we don\u0027t recommend pip installing but instead working directly from the git repo.\n\ntox was suggested by dhellman but the path to enabling the tools directory via tox sounded painful - I wanted an emergency fix since this was an emergency problem.  Agree its OT for quiickstart - probably need a different guide for devs if we introduce tox for our virtualenvs.\n\nTox is not for operators/evaluators, its for developers though, so it would be a different workflow entirely.","commit_id":"8cd0d2a72ac8a995f7b07293561b76b0be04f52a"}]}