)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":10607,"name":"Alexandra Settle","email":"a.settle@outlook.com","username":"asettle"},"change_message_id":"889e04873874d0b611984fc93408b431b0d9f9e9","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Greg Faust \u003cgregfaust@gmail.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2016-06-06 10:44:03 -0400"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"add note about building AIO more than once"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Change-Id: I2eed20f00b491ebfd9ead7659438534b5c11db60"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"7aa08908_5efee137","line":7,"updated":"2016-06-07 08:51:21.000000000","message":"Could you please note DOC in the commit message so the reviewer knows","commit_id":"cded3999a2f3a4531ea12951ed809e7ea4cadd68"}],"doc/source/developer-docs/quickstart-aio.rst":[{"author":{"_account_id":13995,"name":"Nate Johnston","email":"nate.johnston@redhat.com","username":"natejohnston"},"change_message_id":"6fb7b48384ebac96cb56169dbf1fa9b5c3ca23c0","unresolved":false,"context_lines":[{"line_number":156,"context_line":""},{"line_number":157,"context_line":"       # scripts/run-playbooks.sh"},{"line_number":158,"context_line":""},{"line_number":159,"context_line":"**Note:** The run-playbooks.sh script should not be run more than once.  If"},{"line_number":160,"context_line":"there is a reason that run-playbooks.sh would be needed to run a second time,"},{"line_number":161,"context_line":"it will be necessary to start over.  Alternatively, it may be possible to"},{"line_number":162,"context_line":"individually run each playbook rather than starting over.  Executing"}],"source_content_type":"text/x-rst","patch_set":1,"id":"9abb7d3a_9bb045df","line":159,"updated":"2016-06-03 19:58:05.000000000","message":"Note should be specified like so:\n\n.. note::\n\nThe block of text under it should be indented.  The below file has numerous examples.\n\nhttps://raw.githubusercontent.com/openstack/openstack-manuals/master/doc/networking-guide/source/adv-config-qos.rst","commit_id":"7d85d30284431705140b4732e5527c7536bade61"},{"author":{"_account_id":19814,"name":"Travis Truman (automagically)","email":"travis_truman@cable.comcast.com","username":"trumant"},"change_message_id":"5e4634234a9584892ec86f9aa7e668c0b2d9b547","unresolved":false,"context_lines":[{"line_number":156,"context_line":""},{"line_number":157,"context_line":"       # scripts/run-playbooks.sh"},{"line_number":158,"context_line":""},{"line_number":159,"context_line":"**Note:** The run-playbooks.sh script should not be run more than once.  If"},{"line_number":160,"context_line":"there is a reason that run-playbooks.sh would be needed to run a second time,"},{"line_number":161,"context_line":"it will be necessary to start over.  Alternatively, it may be possible to"},{"line_number":162,"context_line":"individually run each playbook rather than starting over.  Executing"}],"source_content_type":"text/x-rst","patch_set":1,"id":"9abb7d3a_3b2e79fa","line":159,"updated":"2016-06-03 19:56:40.000000000","message":"Please use the RST markup \".. note::\" here","commit_id":"7d85d30284431705140b4732e5527c7536bade61"},{"author":{"_account_id":13995,"name":"Nate Johnston","email":"nate.johnston@redhat.com","username":"natejohnston"},"change_message_id":"6fb7b48384ebac96cb56169dbf1fa9b5c3ca23c0","unresolved":false,"context_lines":[{"line_number":157,"context_line":"       # scripts/run-playbooks.sh"},{"line_number":158,"context_line":""},{"line_number":159,"context_line":"**Note:** The run-playbooks.sh script should not be run more than once.  If"},{"line_number":160,"context_line":"there is a reason that run-playbooks.sh would be needed to run a second time,"},{"line_number":161,"context_line":"it will be necessary to start over.  Alternatively, it may be possible to"},{"line_number":162,"context_line":"individually run each playbook rather than starting over.  Executing"},{"line_number":163,"context_line":"run-playbooks.sh a second time will result in container connectivity issues."}],"source_content_type":"text/x-rst","patch_set":1,"id":"9abb7d3a_3b631943","line":160,"updated":"2016-06-03 19:58:05.000000000","message":"It is confusing to say \"run run-playbooks.sh\", try a synonym like \"execute\".  Also, I think this can be reworded to be less awkward.  \"If there is a reason you need to execute run-playbooks.sh a second time then you should start over.\"","commit_id":"7d85d30284431705140b4732e5527c7536bade61"},{"author":{"_account_id":12892,"name":"Nolan Brubaker","email":"nolan.brubaker@rackspace.com","username":"nrb"},"change_message_id":"2cb7c24553d6c1f07d5adab0da01bcae84d89e59","unresolved":false,"context_lines":[{"line_number":158,"context_line":""},{"line_number":159,"context_line":"**Note:** The run-playbooks.sh script should not be run more than once.  If"},{"line_number":160,"context_line":"there is a reason that run-playbooks.sh would be needed to run a second time,"},{"line_number":161,"context_line":"it will be necessary to start over.  Alternatively, it may be possible to"},{"line_number":162,"context_line":"individually run each playbook rather than starting over.  Executing"},{"line_number":163,"context_line":"run-playbooks.sh a second time will result in container connectivity issues."},{"line_number":164,"context_line":""}],"source_content_type":"text/x-rst","patch_set":1,"id":"9abb7d3a_9be8457e","line":161,"updated":"2016-06-03 19:59:11.000000000","message":"This could probably be re-worded - if any playbooks need to be re-run after the initial deploy, they should be run from the `playbooks` directory with the openstack-ansible command.\n\nThe point about running it again is still valid, though.\n\nSee also https://review.openstack.org/#/c/308584/ for a new implementation inside of run-playbooks that provides a similar warning.","commit_id":"7d85d30284431705140b4732e5527c7536bade61"},{"author":{"_account_id":13995,"name":"Nate Johnston","email":"nate.johnston@redhat.com","username":"natejohnston"},"change_message_id":"6fb7b48384ebac96cb56169dbf1fa9b5c3ca23c0","unresolved":false,"context_lines":[{"line_number":159,"context_line":"**Note:** The run-playbooks.sh script should not be run more than once.  If"},{"line_number":160,"context_line":"there is a reason that run-playbooks.sh would be needed to run a second time,"},{"line_number":161,"context_line":"it will be necessary to start over.  Alternatively, it may be possible to"},{"line_number":162,"context_line":"individually run each playbook rather than starting over.  Executing"},{"line_number":163,"context_line":"run-playbooks.sh a second time will result in container connectivity issues."},{"line_number":164,"context_line":""},{"line_number":165,"context_line":"The installation process will take a while to complete, but here are some"}],"source_content_type":"text/x-rst","patch_set":1,"id":"9abb7d3a_bb4f09bd","line":162,"range":{"start_line":162,"start_character":43,"end_line":162,"end_character":56},"updated":"2016-06-03 19:58:05.000000000","message":"What does \"start over\" mean.  Do you mean start with a fresh VM?","commit_id":"7d85d30284431705140b4732e5527c7536bade61"},{"author":{"_account_id":13995,"name":"Nate Johnston","email":"nate.johnston@redhat.com","username":"natejohnston"},"change_message_id":"6fb7b48384ebac96cb56169dbf1fa9b5c3ca23c0","unresolved":false,"context_lines":[{"line_number":160,"context_line":"there is a reason that run-playbooks.sh would be needed to run a second time,"},{"line_number":161,"context_line":"it will be necessary to start over.  Alternatively, it may be possible to"},{"line_number":162,"context_line":"individually run each playbook rather than starting over.  Executing"},{"line_number":163,"context_line":"run-playbooks.sh a second time will result in container connectivity issues."},{"line_number":164,"context_line":""},{"line_number":165,"context_line":"The installation process will take a while to complete, but here are some"},{"line_number":166,"context_line":"general estimates:"}],"source_content_type":"text/x-rst","patch_set":1,"id":"9abb7d3a_3bf1596b","line":163,"range":{"start_line":163,"start_character":46,"end_line":163,"end_character":75},"updated":"2016-06-03 19:58:05.000000000","message":"Can you be specific about what container connectivity issues will happen?  Lets say I read this a year from now, and I want to search bugs to see if this issue has been resolved in the meantime, I have no data to investigate if this is true anymore.","commit_id":"7d85d30284431705140b4732e5527c7536bade61"},{"author":{"_account_id":10897,"name":"Joseph Robinson","email":"josephremail@gmail.com","username":"RobiJoe"},"change_message_id":"40fb2e65f2f30b881dff7cb1e94344bf7c62b95c","unresolved":false,"context_lines":[{"line_number":157,"context_line":"       # scripts/run-playbooks.sh"},{"line_number":158,"context_line":""},{"line_number":159,"context_line":".. note::"},{"line_number":160,"context_line":"   The run-playbooks.sh script should not be executed more than once.  If"},{"line_number":161,"context_line":"   something goes wrong it is necessary to start over as described below in the"},{"line_number":162,"context_line":"   `Rebuilding an AIO`_ section.  Alternatively, it may be possible to"},{"line_number":163,"context_line":"   individually run each playbook rather than starting over.   If any playbooks"}],"source_content_type":"text/x-rst","patch_set":2,"id":"7aa08908_14a0c342","line":160,"updated":"2016-06-07 05:04:43.000000000","message":"Change to ``run-playbooks.sh`` script. And on line 166.","commit_id":"cded3999a2f3a4531ea12951ed809e7ea4cadd68"},{"author":{"_account_id":10607,"name":"Alexandra Settle","email":"a.settle@outlook.com","username":"asettle"},"change_message_id":"889e04873874d0b611984fc93408b431b0d9f9e9","unresolved":false,"context_lines":[{"line_number":157,"context_line":"       # scripts/run-playbooks.sh"},{"line_number":158,"context_line":""},{"line_number":159,"context_line":".. note::"},{"line_number":160,"context_line":"   The run-playbooks.sh script should not be executed more than once.  If"},{"line_number":161,"context_line":"   something goes wrong it is necessary to start over as described below in the"},{"line_number":162,"context_line":"   `Rebuilding an AIO`_ section.  Alternatively, it may be possible to"},{"line_number":163,"context_line":"   individually run each playbook rather than starting over.   If any playbooks"}],"source_content_type":"text/x-rst","patch_set":2,"id":"7aa08908_fe5a2d4f","line":160,"range":{"start_line":160,"start_character":69,"end_line":160,"end_character":70},"updated":"2016-06-07 08:51:21.000000000","message":"Could you also please remove all double spaces after the period","commit_id":"cded3999a2f3a4531ea12951ed809e7ea4cadd68"},{"author":{"_account_id":10607,"name":"Alexandra Settle","email":"a.settle@outlook.com","username":"asettle"},"change_message_id":"889e04873874d0b611984fc93408b431b0d9f9e9","unresolved":false,"context_lines":[{"line_number":157,"context_line":"       # scripts/run-playbooks.sh"},{"line_number":158,"context_line":""},{"line_number":159,"context_line":".. note::"},{"line_number":160,"context_line":"   The run-playbooks.sh script should not be executed more than once.  If"},{"line_number":161,"context_line":"   something goes wrong it is necessary to start over as described below in the"},{"line_number":162,"context_line":"   `Rebuilding an AIO`_ section.  Alternatively, it may be possible to"},{"line_number":163,"context_line":"   individually run each playbook rather than starting over.   If any playbooks"}],"source_content_type":"text/x-rst","patch_set":2,"id":"7aa08908_3e7e75ae","line":160,"in_reply_to":"7aa08908_14a0c342","updated":"2016-06-07 08:51:21.000000000","message":"\"Do not execute the ``run-playbooks.sh`` script more than once.\" - removing passive voice.","commit_id":"cded3999a2f3a4531ea12951ed809e7ea4cadd68"},{"author":{"_account_id":10607,"name":"Alexandra Settle","email":"a.settle@outlook.com","username":"asettle"},"change_message_id":"889e04873874d0b611984fc93408b431b0d9f9e9","unresolved":false,"context_lines":[{"line_number":158,"context_line":""},{"line_number":159,"context_line":".. note::"},{"line_number":160,"context_line":"   The run-playbooks.sh script should not be executed more than once.  If"},{"line_number":161,"context_line":"   something goes wrong it is necessary to start over as described below in the"},{"line_number":162,"context_line":"   `Rebuilding an AIO`_ section.  Alternatively, it may be possible to"},{"line_number":163,"context_line":"   individually run each playbook rather than starting over.   If any playbooks"},{"line_number":164,"context_line":"   need to be re-run after the initial deploy, they should be run from the"}],"source_content_type":"text/x-rst","patch_set":2,"id":"7aa08908_be3f05d5","line":161,"updated":"2016-06-07 08:51:21.000000000","message":"wrong, *","commit_id":"cded3999a2f3a4531ea12951ed809e7ea4cadd68"},{"author":{"_account_id":10897,"name":"Joseph Robinson","email":"josephremail@gmail.com","username":"RobiJoe"},"change_message_id":"40fb2e65f2f30b881dff7cb1e94344bf7c62b95c","unresolved":false,"context_lines":[{"line_number":159,"context_line":".. note::"},{"line_number":160,"context_line":"   The run-playbooks.sh script should not be executed more than once.  If"},{"line_number":161,"context_line":"   something goes wrong it is necessary to start over as described below in the"},{"line_number":162,"context_line":"   `Rebuilding an AIO`_ section.  Alternatively, it may be possible to"},{"line_number":163,"context_line":"   individually run each playbook rather than starting over.   If any playbooks"},{"line_number":164,"context_line":"   need to be re-run after the initial deploy, they should be run from the"},{"line_number":165,"context_line":"   `playbooks` directory with the openstack-ansible command.  Executing"}],"source_content_type":"text/x-rst","patch_set":2,"id":"7aa08908_b44d6f9b","line":162,"updated":"2016-06-07 05:04:43.000000000","message":"One space after a full stop. And on line 163 and 165.","commit_id":"cded3999a2f3a4531ea12951ed809e7ea4cadd68"},{"author":{"_account_id":10607,"name":"Alexandra Settle","email":"a.settle@outlook.com","username":"asettle"},"change_message_id":"889e04873874d0b611984fc93408b431b0d9f9e9","unresolved":false,"context_lines":[{"line_number":162,"context_line":"   `Rebuilding an AIO`_ section.  Alternatively, it may be possible to"},{"line_number":163,"context_line":"   individually run each playbook rather than starting over.   If any playbooks"},{"line_number":164,"context_line":"   need to be re-run after the initial deploy, they should be run from the"},{"line_number":165,"context_line":"   `playbooks` directory with the openstack-ansible command.  Executing"},{"line_number":166,"context_line":"   run-playbooks.sh a second time will result in an inconsistent state for LXC"},{"line_number":167,"context_line":"   IPtables rules and cause network connectivity issues from within containers."},{"line_number":168,"context_line":""}],"source_content_type":"text/x-rst","patch_set":2,"id":"7aa08908_9927c3b5","line":165,"range":{"start_line":165,"start_character":34,"end_line":165,"end_character":51},"updated":"2016-06-07 08:51:21.000000000","message":"``openstack-ansible`` - as this IS a command, backticks please","commit_id":"cded3999a2f3a4531ea12951ed809e7ea4cadd68"},{"author":{"_account_id":10607,"name":"Alexandra Settle","email":"a.settle@outlook.com","username":"asettle"},"change_message_id":"889e04873874d0b611984fc93408b431b0d9f9e9","unresolved":false,"context_lines":[{"line_number":162,"context_line":"   `Rebuilding an AIO`_ section.  Alternatively, it may be possible to"},{"line_number":163,"context_line":"   individually run each playbook rather than starting over.   If any playbooks"},{"line_number":164,"context_line":"   need to be re-run after the initial deploy, they should be run from the"},{"line_number":165,"context_line":"   `playbooks` directory with the openstack-ansible command.  Executing"},{"line_number":166,"context_line":"   run-playbooks.sh a second time will result in an inconsistent state for LXC"},{"line_number":167,"context_line":"   IPtables rules and cause network connectivity issues from within containers."},{"line_number":168,"context_line":""}],"source_content_type":"text/x-rst","patch_set":2,"id":"7aa08908_f90fe72b","line":165,"range":{"start_line":165,"start_character":13,"end_line":165,"end_character":14},"updated":"2016-06-07 08:51:21.000000000","message":"remove - no need for backticks here","commit_id":"cded3999a2f3a4531ea12951ed809e7ea4cadd68"},{"author":{"_account_id":10607,"name":"Alexandra Settle","email":"a.settle@outlook.com","username":"asettle"},"change_message_id":"889e04873874d0b611984fc93408b431b0d9f9e9","unresolved":false,"context_lines":[{"line_number":163,"context_line":"   individually run each playbook rather than starting over.   If any playbooks"},{"line_number":164,"context_line":"   need to be re-run after the initial deploy, they should be run from the"},{"line_number":165,"context_line":"   `playbooks` directory with the openstack-ansible command.  Executing"},{"line_number":166,"context_line":"   run-playbooks.sh a second time will result in an inconsistent state for LXC"},{"line_number":167,"context_line":"   IPtables rules and cause network connectivity issues from within containers."},{"line_number":168,"context_line":""},{"line_number":169,"context_line":"The installation process will take a while to complete, but here are some"}],"source_content_type":"text/x-rst","patch_set":2,"id":"7aa08908_792ad77f","line":166,"range":{"start_line":166,"start_character":3,"end_line":166,"end_character":19},"updated":"2016-06-07 08:51:21.000000000","message":"ditto","commit_id":"cded3999a2f3a4531ea12951ed809e7ea4cadd68"},{"author":{"_account_id":10607,"name":"Alexandra Settle","email":"a.settle@outlook.com","username":"asettle"},"change_message_id":"889e04873874d0b611984fc93408b431b0d9f9e9","unresolved":false,"context_lines":[{"line_number":163,"context_line":"   individually run each playbook rather than starting over.   If any playbooks"},{"line_number":164,"context_line":"   need to be re-run after the initial deploy, they should be run from the"},{"line_number":165,"context_line":"   `playbooks` directory with the openstack-ansible command.  Executing"},{"line_number":166,"context_line":"   run-playbooks.sh a second time will result in an inconsistent state for LXC"},{"line_number":167,"context_line":"   IPtables rules and cause network connectivity issues from within containers."},{"line_number":168,"context_line":""},{"line_number":169,"context_line":"The installation process will take a while to complete, but here are some"}],"source_content_type":"text/x-rst","patch_set":2,"id":"7aa08908_5936fbe6","line":166,"range":{"start_line":166,"start_character":34,"end_line":166,"end_character":45},"updated":"2016-06-07 08:51:21.000000000","message":"results in* (remove the \u0027will\u0027)","commit_id":"cded3999a2f3a4531ea12951ed809e7ea4cadd68"},{"author":{"_account_id":14103,"name":"wangtianfa","email":"wang.tianfa@99cloud.net","username":"wangtianfa"},"change_message_id":"706ad38f8f8eb59e0e6935a52a3a45930e440093","unresolved":false,"context_lines":[{"line_number":157,"context_line":"       # scripts/run-playbooks.sh"},{"line_number":158,"context_line":""},{"line_number":159,"context_line":".. note::"},{"line_number":160,"context_line":"   Do not execute The ``run-playbooks.sh`` more than once. If something goes"},{"line_number":161,"context_line":"   wrong, it is necessary to start over as described below in the"},{"line_number":162,"context_line":"   `Rebuilding an AIO`_ section. Alternatively, it may be possible to"},{"line_number":163,"context_line":"   individually run each playbook rather than starting over. If any playbooks"}],"source_content_type":"text/x-rst","patch_set":3,"id":"7aa08908_870aaa1f","line":160,"range":{"start_line":160,"start_character":18,"end_line":160,"end_character":21},"updated":"2016-06-07 12:43:24.000000000","message":"Whether is \"The\" should change \"the\"?","commit_id":"ef7a3bf215840cdea2e882cdd65b3202e1116bda"}]}