)]}'
{"playbooks/roles/bifrost-ironic-install/tasks/get_ipxe.yml":[{"author":{"_account_id":6133,"name":"yolanda.robla","email":"yroblamo@redhat.com","username":"yolanda.robla"},"change_message_id":"c431d7a90148856a2881b833aa2162b2ae906a47","unresolved":false,"context_lines":[{"line_number":14,"context_line":"# limitations under the License."},{"line_number":15,"context_line":"---"},{"line_number":16,"context_line":"- name: \"If VENV is set in the environment\""},{"line_number":17,"context_line":"  set_fact:"},{"line_number":18,"context_line":"    enable_venv: true"},{"line_number":19,"context_line":"  when: lookup(\u0027env\u0027, \u0027VENV\u0027) | length \u003e 0"},{"line_number":20,"context_line":""}],"source_content_type":"text/x-yaml","patch_set":9,"id":"3f79a3b5_e83986f0","line":17,"range":{"start_line":17,"start_character":2,"end_line":17,"end_character":10},"updated":"2018-09-20 10:50:40.000000000","message":"this is repeating everywhere, can you add into a common file, then include it?","commit_id":"038cec337ea4b72aef33fb72a416113839f6843a"},{"author":{"_account_id":19578,"name":"Olivier Bourdon","email":"olivierbourdon38@gmail.com","username":"olivierbourdon38"},"change_message_id":"1760d150c36f4f68e5df358a6fd2c09a9fab792c","unresolved":false,"context_lines":[{"line_number":14,"context_line":"# limitations under the License."},{"line_number":15,"context_line":"---"},{"line_number":16,"context_line":"- name: \"If VENV is set in the environment\""},{"line_number":17,"context_line":"  set_fact:"},{"line_number":18,"context_line":"    enable_venv: true"},{"line_number":19,"context_line":"  when: lookup(\u0027env\u0027, \u0027VENV\u0027) | length \u003e 0"},{"line_number":20,"context_line":""}],"source_content_type":"text/x-yaml","patch_set":9,"id":"3f79a3b5_8b08a127","line":17,"range":{"start_line":17,"start_character":2,"end_line":17,"end_character":10},"in_reply_to":"3f79a3b5_12fb8015","updated":"2018-09-26 08:10:58.000000000","message":"Not sure either but found alternative way which seems to work across roles","commit_id":"038cec337ea4b72aef33fb72a416113839f6843a"},{"author":{"_account_id":19578,"name":"Olivier Bourdon","email":"olivierbourdon38@gmail.com","username":"olivierbourdon38"},"change_message_id":"1760d150c36f4f68e5df358a6fd2c09a9fab792c","unresolved":false,"context_lines":[{"line_number":14,"context_line":"# limitations under the License."},{"line_number":15,"context_line":"---"},{"line_number":16,"context_line":"- name: \"If VENV is set in the environment\""},{"line_number":17,"context_line":"  set_fact:"},{"line_number":18,"context_line":"    enable_venv: true"},{"line_number":19,"context_line":"  when: lookup(\u0027env\u0027, \u0027VENV\u0027) | length \u003e 0"},{"line_number":20,"context_line":""}],"source_content_type":"text/x-yaml","patch_set":9,"id":"3f79a3b5_abfb1d13","line":17,"range":{"start_line":17,"start_character":2,"end_line":17,"end_character":10},"in_reply_to":"3f79a3b5_e83986f0","updated":"2018-09-26 08:10:58.000000000","message":"Done","commit_id":"038cec337ea4b72aef33fb72a416113839f6843a"},{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"b6fe88be385a9a0e00db3ccb9595183714364de0","unresolved":false,"context_lines":[{"line_number":14,"context_line":"# limitations under the License."},{"line_number":15,"context_line":"---"},{"line_number":16,"context_line":"- name: \"If VENV is set in the environment\""},{"line_number":17,"context_line":"  set_fact:"},{"line_number":18,"context_line":"    enable_venv: true"},{"line_number":19,"context_line":"  when: lookup(\u0027env\u0027, \u0027VENV\u0027) | length \u003e 0"},{"line_number":20,"context_line":""}],"source_content_type":"text/x-yaml","patch_set":9,"id":"3f79a3b5_12fb8015","line":17,"range":{"start_line":17,"start_character":2,"end_line":17,"end_character":10},"in_reply_to":"3f79a3b5_e83986f0","updated":"2018-09-25 03:03:47.000000000","message":"I guess I\u0027m curious why this can\u0027t just be in main.yml.","commit_id":"038cec337ea4b72aef33fb72a416113839f6843a"}],"playbooks/roles/bifrost-ironic-install/tasks/keystone_setup.yml":[{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"4107fbe37698705a46d18b8347c61808e030f82e","unresolved":false,"context_lines":[{"line_number":45,"context_line":"      ironic.keystone.default_password is undefined"},{"line_number":46,"context_line":""},{"line_number":47,"context_line":"- name: \"Retrieve venv python path\""},{"line_number":48,"context_line":"  shell: python -c \"exec(\\\"import sys\\\\nprint \u0027:\u0027.join(sys.path)\\\")\""},{"line_number":49,"context_line":"  environment: \"{{ bifrost_venv_env | default({}) }}\""},{"line_number":50,"context_line":"  register: venv_pythonpath_result"},{"line_number":51,"context_line":"  when: enable_venv"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"3f79a3b5_1bd89c7d","line":48,"updated":"2018-08-16 20:12:49.000000000","message":"Creative, although how do we know we\u0027re executing the correct python, or even what is the correct python?  Since there is python3.  I\u0027ve been kind of heading into this issue recently.\n\nYou might glean some ideas from\nhttps://review.openstack.org/#/c/588325/","commit_id":"6e8ffe29855afdc718c5e5fecc1e2f0518ff410b"},{"author":{"_account_id":19578,"name":"Olivier Bourdon","email":"olivierbourdon38@gmail.com","username":"olivierbourdon38"},"change_message_id":"b41f3dd5f550c9efe1680550755d0571c52c6d64","unresolved":false,"context_lines":[{"line_number":45,"context_line":"      ironic.keystone.default_password is undefined"},{"line_number":46,"context_line":""},{"line_number":47,"context_line":"- name: \"Retrieve venv python path\""},{"line_number":48,"context_line":"  shell: python -c \"exec(\\\"import sys\\\\nprint \u0027:\u0027.join(sys.path)\\\")\""},{"line_number":49,"context_line":"  environment: \"{{ bifrost_venv_env | default({}) }}\""},{"line_number":50,"context_line":"  register: venv_pythonpath_result"},{"line_number":51,"context_line":"  when: enable_venv"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"3f79a3b5_bad63727","line":48,"in_reply_to":"3f79a3b5_1bd89c7d","updated":"2018-08-17 06:35:25.000000000","message":"Agreed and understood, I already had the idea of looking into your patch but did not think of integrating stuff from it so soon ;-) Wrong guess from my end. Will fix","commit_id":"6e8ffe29855afdc718c5e5fecc1e2f0518ff410b"},{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"1ccc2973835e0cf6d52ad58da43eaea8faec7bb2","unresolved":false,"context_lines":[{"line_number":45,"context_line":"      ironic.keystone.default_password is undefined"},{"line_number":46,"context_line":""},{"line_number":47,"context_line":"- name: \"Retrieve venv python path\""},{"line_number":48,"context_line":"  shell: python -c \"exec(\\\"import sys\\\\nprint \u0027:\u0027.join(sys.path)\\\")\""},{"line_number":49,"context_line":"  environment: \"{{ bifrost_venv_env | default({}) }}\""},{"line_number":50,"context_line":"  register: venv_pythonpath_result"},{"line_number":51,"context_line":"  when: enable_venv"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"3f79a3b5_9d276013","line":48,"in_reply_to":"3f79a3b5_bad63727","updated":"2018-08-21 19:41:47.000000000","message":"Well, best to try and be ahead of the curve as it were :)","commit_id":"6e8ffe29855afdc718c5e5fecc1e2f0518ff410b"},{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"b6fe88be385a9a0e00db3ccb9595183714364de0","unresolved":false,"context_lines":[{"line_number":19,"context_line":"- name: \"Initialize default venv\""},{"line_number":20,"context_line":"  set_fact:"},{"line_number":21,"context_line":"    venv:"},{"line_number":22,"context_line":"      OS_IDENTITY_API_VERSION: \"3\""},{"line_number":23,"context_line":""},{"line_number":24,"context_line":"- name: \"If VENV is set in the environment, enable installation into venv\""},{"line_number":25,"context_line":"  set_fact:"}],"source_content_type":"text/x-yaml","patch_set":9,"id":"3f79a3b5_3200fc00","line":22,"updated":"2018-09-25 03:03:47.000000000","message":"Ugh, this is a necessary evil :(","commit_id":"038cec337ea4b72aef33fb72a416113839f6843a"}],"playbooks/roles/bifrost-ironic-install/tasks/keystone_setup_inspector.yml":[{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"4107fbe37698705a46d18b8347c61808e030f82e","unresolved":false,"context_lines":[{"line_number":19,"context_line":"- name: \"Initialize default venv\""},{"line_number":20,"context_line":"  set_fact:"},{"line_number":21,"context_line":"    venv:"},{"line_number":22,"context_line":"      OS_IDENTITY_API_VERSION: \"3\""},{"line_number":23,"context_line":""},{"line_number":24,"context_line":"- name: \"If VENV is set in the environment, enable installation into venv\""},{"line_number":25,"context_line":"  set_fact:"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"3f79a3b5_3bf1d802","line":22,"updated":"2018-08-16 20:12:49.000000000","message":":( but I understand why.","commit_id":"6e8ffe29855afdc718c5e5fecc1e2f0518ff410b"},{"author":{"_account_id":19578,"name":"Olivier Bourdon","email":"olivierbourdon38@gmail.com","username":"olivierbourdon38"},"change_message_id":"b41f3dd5f550c9efe1680550755d0571c52c6d64","unresolved":false,"context_lines":[{"line_number":19,"context_line":"- name: \"Initialize default venv\""},{"line_number":20,"context_line":"  set_fact:"},{"line_number":21,"context_line":"    venv:"},{"line_number":22,"context_line":"      OS_IDENTITY_API_VERSION: \"3\""},{"line_number":23,"context_line":""},{"line_number":24,"context_line":"- name: \"If VENV is set in the environment, enable installation into venv\""},{"line_number":25,"context_line":"  set_fact:"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"3f79a3b5_3a9307a6","line":22,"in_reply_to":"3f79a3b5_3bf1d802","updated":"2018-08-17 06:35:25.000000000","message":"Anything for me to do on this ?","commit_id":"6e8ffe29855afdc718c5e5fecc1e2f0518ff410b"}],"playbooks/roles/common/venv_python_path.yml":[{"author":{"_account_id":14826,"name":"Mark Goddard","email":"markgoddard86@gmail.com","username":"mgoddard"},"change_message_id":"d9829bcec3f5192fef2d1aaaceaccfc1a74925e9","unresolved":false,"context_lines":[{"line_number":16,"context_line":"    enable_venv: true"},{"line_number":17,"context_line":"  when: lookup(\u0027env\u0027, \u0027VENV\u0027) | length \u003e 0"},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"- name: \"Retrieve venv python path\""},{"line_number":20,"context_line":"  shell: \"echo \\\"import sys\\\\nprint(\u0027:\u0027.join(sys.path))\\\" | {{ ansible_python.get(\u0027executable\u0027, \u0027/usr/bin/python\u0027).split(\u0027/\u0027)[-1] }}\""},{"line_number":21,"context_line":"  environment: \"{{ bifrost_venv_env | default({}) }}\""},{"line_number":22,"context_line":"  register: venv_pythonpath_result"}],"source_content_type":"text/x-yaml","patch_set":10,"id":"3f79a3b5_0f78a1f5","line":19,"updated":"2018-10-02 10:53:34.000000000","message":"It looks like this change may have caused an issue with the kolla bifrost image: http://logs.openstack.org/44/606644/1/gate/kolla-build-centos-source/7ff6e67/logs/build/000_FAILED_bifrost-base.txt.gz\n\n\"stderr_lines\": [\n  \"  File \\\"\u003cstdin\u003e\\\", line 1\",\n  \"    import sys\\\\nprint(\u0027:\u0027.join(sys.path))\",\n  \"                                        ^\",\n  \"SyntaxError: unexpected character after line continuation character\"\n],","commit_id":"1e19c45199e4fa2bf89cc6487f820b6915b926e1"}]}