)]}'
{"tests/bin/setup_config.sh":[{"author":{"_account_id":9237,"name":"Kevin Fox","email":"kevin@efox.cc","username":"kfox1111"},"change_message_id":"5683aa1a14a6eb2113255a1c5f70016ab0f03841","unresolved":false,"context_lines":[{"line_number":8,"context_line":"echo \"kolla_base_distro: $1\" | sudo tee -a /etc/kolla/globals.yml"},{"line_number":9,"context_line":""},{"line_number":10,"context_line":"IP\u003d172.18.0.1"},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"sudo -E sed -i \"s/^\\(kolla_external_vip_address:\\).*/\\1 \u0027$IP\u0027/\" /etc/kolla/globals.yml"},{"line_number":13,"context_line":"sudo -E sed -i \"s/^\\(kolla_kubernetes_external_vip:\\).*/\\1 \u0027$IP\u0027/\" /etc/kolla-kubernetes/kolla-kubernetes.yml"},{"line_number":14,"context_line":"sudo -E sed -i \"s/initial_mon:.*/initial_mon: $NODE/\" /etc/kolla-kubernetes/kolla-kubernetes.yml"}],"source_content_type":"text/x-sh","patch_set":39,"id":"bff0334d_c0ea7ad6","line":11,"updated":"2017-04-10 18:21:43.000000000","message":"this is now doing an awful lot of sudo\u0027s. This seems like a step backwards, not a step forwards.\n\nCan you change the permissions on /etc/kolla to be owned by the user?","commit_id":"6c84a0d422878e911dfb567137e92dd26d63ddb2"}],"tools/setup_gate_common.sh":[{"author":{"_account_id":22525,"name":"Qin Wang (qwang)","email":"qin.wang@intel.com","username":"qwang"},"change_message_id":"2e772a461463fca8b3d6ef6fdd358d278a5e6e3e","unresolved":false,"context_lines":[{"line_number":80,"context_line":"pip install ansible"},{"line_number":81,"context_line":"pip install kolla-ansible/"},{"line_number":82,"context_line":"pip install -r kolla-ansible/requirements.txt"},{"line_number":83,"context_line":"pip install pyyaml"},{"line_number":84,"context_line":"sudo mkdir -p /etc/kolla/config"},{"line_number":85,"context_line":"# NOTE (sbezverk) Added as a workaround since kolla-ansible master had"},{"line_number":86,"context_line":"# use_neutron config option removed. Next 4 lines can be removed after"}],"source_content_type":"text/x-sh","patch_set":2,"id":"ffe62b97_efd60250","line":83,"updated":"2017-03-24 17:13:48.000000000","message":"kolla-kubernetes command pip installation are removed but the command is used in a later script.","commit_id":"7407ad3e5e44af85ab5273aa8cb930b7f820988a"},{"author":{"_account_id":9237,"name":"Kevin Fox","email":"kevin@efox.cc","username":"kfox1111"},"change_message_id":"f319ee7b4397aab59141d9ceea6c32823f294827","unresolved":false,"context_lines":[{"line_number":88,"context_line":"pip install pyyaml"},{"line_number":89,"context_line":"popd"},{"line_number":90,"context_line":"pip install -r requirements.txt"},{"line_number":91,"context_line":"pip install ."},{"line_number":92,"context_line":"sudo mkdir -p /etc/kolla/config"},{"line_number":93,"context_line":"# NOTE (sbezverk) Added as a workaround since kolla-ansible master had"},{"line_number":94,"context_line":"# use_neutron config option removed. Next 4 lines can be removed after"}],"source_content_type":"text/x-sh","patch_set":18,"id":"ffe62b97_38a86f75","side":"PARENT","line":91,"updated":"2017-03-27 16:02:29.000000000","message":"A lot of the stuff removed here has nothing to do with kolla-ansible. I think you need to add most of this back.","commit_id":"2953dbd94964d4d9b41ae909ed3f4921409b3609"},{"author":{"_account_id":9237,"name":"Kevin Fox","email":"kevin@efox.cc","username":"kfox1111"},"change_message_id":"f319ee7b4397aab59141d9ceea6c32823f294827","unresolved":false,"context_lines":[{"line_number":79,"context_line":"pip install pip --upgrade"},{"line_number":80,"context_line":"pip install ansible"},{"line_number":81,"context_line":"pip install kolla-ansible/"},{"line_number":82,"context_line":"pip install -r kolla-ansible/requirements.txt"},{"line_number":83,"context_line":"pip install pyyaml"},{"line_number":84,"context_line":"sudo mkdir -p /etc/kolla/config"},{"line_number":85,"context_line":"# NOTE (sbezverk) Added as a workaround since kolla-ansible master had"}],"source_content_type":"text/x-sh","patch_set":18,"id":"ffe62b97_38eceff1","line":82,"updated":"2017-03-27 16:02:29.000000000","message":"should requirements come before the main package install?","commit_id":"33df5f3f817cdf8b41bac293e1a02e1feb854260"},{"author":{"_account_id":22959,"name":"Zeyu Zhu","email":"zhuzeyu0409@gmail.com","username":"zhuzeyu"},"change_message_id":"530ad3216e3bf284e4bc2179326e884e79936454","unresolved":false,"context_lines":[{"line_number":66,"context_line":"fi"},{"line_number":67,"context_line":""},{"line_number":68,"context_line":"pip install pip --upgrade"},{"line_number":69,"context_line":"pip install ansible"},{"line_number":70,"context_line":"pip install -r kolla-ansible/requirements.txt"},{"line_number":71,"context_line":"pip install kolla-ansible/"},{"line_number":72,"context_line":"pip install pyyaml"}],"source_content_type":"text/x-sh","patch_set":21,"id":"dfeb2761_572b3dea","line":69,"range":{"start_line":69,"start_character":12,"end_line":69,"end_character":19},"updated":"2017-04-01 09:13:20.000000000","message":"why not restrict the version of ansible \"ansible\u003e\u003d2.2.0\"","commit_id":"993c7d8792233dbc5d9f4997c6a2b30795f276dc"},{"author":{"_account_id":9237,"name":"Kevin Fox","email":"kevin@efox.cc","username":"kfox1111"},"change_message_id":"50c3482f99398a70cf6ff4bce6bcc43c5c1842ac","unresolved":false,"context_lines":[{"line_number":76,"context_line":"pip install -r requirements.txt"},{"line_number":77,"context_line":"pip install pyyaml"},{"line_number":78,"context_line":"popd"},{"line_number":79,"context_line":"pip install -r requirements.txt"},{"line_number":80,"context_line":"pip install ."},{"line_number":81,"context_line":"sudo mkdir -p /etc/kolla/config"},{"line_number":82,"context_line":"# NOTE (sbezverk) Added as a workaround since kolla-ansible master had"}],"source_content_type":"text/x-sh","patch_set":22,"id":"dfeb2761_ebd1ea24","side":"PARENT","line":79,"updated":"2017-04-04 23:39:36.000000000","message":"why remove requirements.txt?","commit_id":"941623eaf1561416e7d51875ac9bd7d1f70dc54d"},{"author":{"_account_id":9237,"name":"Kevin Fox","email":"kevin@efox.cc","username":"kfox1111"},"change_message_id":"d519ddf6c092594a38fe035064fdc2b870d77924","unresolved":false,"context_lines":[{"line_number":77,"context_line":"pip install pyyaml"},{"line_number":78,"context_line":"popd"},{"line_number":79,"context_line":"pip install -r requirements.txt"},{"line_number":80,"context_line":"pip install ."},{"line_number":81,"context_line":"sudo mkdir -p /etc/kolla/config"},{"line_number":82,"context_line":"# NOTE (sbezverk) Added as a workaround since kolla-ansible master had"},{"line_number":83,"context_line":"# use_neutron config option removed. Next 4 lines can be removed after"}],"source_content_type":"text/x-sh","patch_set":22,"id":"bff0334d_1d3bc4d1","side":"PARENT","line":80,"updated":"2017-04-05 16:08:19.000000000","message":"it broke because the install of kolla-kubernetes has been removed.","commit_id":"941623eaf1561416e7d51875ac9bd7d1f70dc54d"},{"author":{"_account_id":9237,"name":"Kevin Fox","email":"kevin@efox.cc","username":"kfox1111"},"change_message_id":"5683aa1a14a6eb2113255a1c5f70016ab0f03841","unresolved":false,"context_lines":[{"line_number":41,"context_line":"}"},{"line_number":42,"context_line":""},{"line_number":43,"context_line":"function setup_kolla {"},{"line_number":44,"context_line":"#I don\u0027t gather why this is needed"},{"line_number":45,"context_line":"#virtualenv .venv"},{"line_number":46,"context_line":"#. .venv/bin/activate"},{"line_number":47,"context_line":""}],"source_content_type":"text/x-sh","patch_set":39,"id":"bff0334d_405c2aec","line":44,"updated":"2017-04-10 18:21:43.000000000","message":"installing random junk all over your system is bad.","commit_id":"6c84a0d422878e911dfb567137e92dd26d63ddb2"},{"author":{"_account_id":9237,"name":"Kevin Fox","email":"kevin@efox.cc","username":"kfox1111"},"change_message_id":"5683aa1a14a6eb2113255a1c5f70016ab0f03841","unresolved":false,"context_lines":[{"line_number":83,"context_line":"sudo pip install python-openstackclient"},{"line_number":84,"context_line":""},{"line_number":85,"context_line":"sudo cp -aR /usr/share/kolla-ansible/etc_examples/kolla /etc"},{"line_number":86,"context_line":"sudo cp -aR etc/kolla-kubernetes /etc"},{"line_number":87,"context_line":"sudo mkdir -p /etc/kolla/config"},{"line_number":88,"context_line":""},{"line_number":89,"context_line":"# NOTE (sbezverk) Added as a workaround since kolla-ansible master had"}],"source_content_type":"text/x-sh","patch_set":39,"id":"bff0334d_e018de1e","line":86,"updated":"2017-04-10 18:21:43.000000000","message":"still removing deps here. why?","commit_id":"6c84a0d422878e911dfb567137e92dd26d63ddb2"},{"author":{"_account_id":9237,"name":"Kevin Fox","email":"kevin@efox.cc","username":"kfox1111"},"change_message_id":"f69fb4cf4444e380bd6a933958d08976479e0e93","unresolved":false,"context_lines":[{"line_number":83,"context_line":"sudo pip install python-openstackclient"},{"line_number":84,"context_line":""},{"line_number":85,"context_line":"sudo cp -aR /usr/share/kolla-ansible/etc_examples/kolla /etc"},{"line_number":86,"context_line":"sudo cp -aR etc/kolla-kubernetes /etc"},{"line_number":87,"context_line":"sudo mkdir -p /etc/kolla/config"},{"line_number":88,"context_line":""},{"line_number":89,"context_line":"# NOTE (sbezverk) Added as a workaround since kolla-ansible master had"}],"source_content_type":"text/x-sh","patch_set":39,"id":"7ffa3b31_5409d0da","line":86,"in_reply_to":"7ffa3b31_73f4425c","updated":"2017-04-18 20:22:13.000000000","message":"hmm. ok.\n\nthe gate job was still failing due to \u0027pip install .\u0027 getting cut out I think.","commit_id":"6c84a0d422878e911dfb567137e92dd26d63ddb2"},{"author":{"_account_id":2834,"name":"Steven Dake","email":"steven.dake@gmail.com","username":"sdake"},"change_message_id":"845679a793019082588a255dcf09d1e615eeb8b9","unresolved":false,"context_lines":[{"line_number":83,"context_line":"sudo pip install python-openstackclient"},{"line_number":84,"context_line":""},{"line_number":85,"context_line":"sudo cp -aR /usr/share/kolla-ansible/etc_examples/kolla /etc"},{"line_number":86,"context_line":"sudo cp -aR etc/kolla-kubernetes /etc"},{"line_number":87,"context_line":"sudo mkdir -p /etc/kolla/config"},{"line_number":88,"context_line":""},{"line_number":89,"context_line":"# NOTE (sbezverk) Added as a workaround since kolla-ansible master had"}],"source_content_type":"text/x-sh","patch_set":39,"id":"7ffa3b31_73f4425c","line":86,"in_reply_to":"bff0334d_e018de1e","updated":"2017-04-14 02:21:35.000000000","message":"if you mean pip install -r reuirements.txt, I am certain that is not needed.  pip install kolla-ansible/ kolla-kubernetes/ installs requirements.txt naturally.","commit_id":"6c84a0d422878e911dfb567137e92dd26d63ddb2"}]}