)]}'
{"roles/run-tempest/tasks/main.yaml":[{"author":{"_account_id":5803,"name":"Attila Fazekas","email":"afazekas@redhat.com","username":"afazekas"},"change_message_id":"29bd370ed11891c0149d88fd70ac3db2ef84d343","unresolved":false,"context_lines":[{"line_number":39,"context_line":"- name: Set PATH"},{"line_number":40,"context_line":"  # we need to be able to find a tox installed with pip, but we don\u0027t pick up /etc/profile when running this in CI"},{"line_number":41,"context_line":"  set_fact:"},{"line_number":42,"context_line":"    tempest_tox_environmanet: \"{{ tempest_tox_environment | combine({\u0027PATH\u0027: ansible_env.PATH+\u0027:/usr/local/bin\u0027}) }}\""},{"line_number":43,"context_line":""},{"line_number":44,"context_line":"- when:"},{"line_number":45,"context_line":"    - tempest_test_blacklist is defined"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"1f621f24_12a2f9ae","line":42,"updated":"2020-11-05 07:54:42.000000000","message":"should it be added conditionally when it is really missing ?","commit_id":"d4d941e923594e0753c8c56b18e9b3046e36bd54"},{"author":{"_account_id":12907,"name":"Jan Zerebecki","email":"jan.openstack@zerebecki.de","username":"jzerebecki"},"change_message_id":"bd2c5a356c22ed13cef750927c69c1104f385356","unresolved":false,"context_lines":[{"line_number":39,"context_line":"- name: Set PATH"},{"line_number":40,"context_line":"  # we need to be able to find a tox installed with pip, but we don\u0027t pick up /etc/profile when running this in CI"},{"line_number":41,"context_line":"  set_fact:"},{"line_number":42,"context_line":"    tempest_tox_environmanet: \"{{ tempest_tox_environment | combine({\u0027PATH\u0027: ansible_env.PATH+\u0027:/usr/local/bin\u0027}) }}\""},{"line_number":43,"context_line":""},{"line_number":44,"context_line":"- when:"},{"line_number":45,"context_line":"    - tempest_test_blacklist is defined"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"1f621f24_230fda37","line":42,"in_reply_to":"1f621f24_12a2f9ae","updated":"2020-11-06 12:05:03.000000000","message":"I don\u0027t know of a situation in our CI where it is set at this point. It is new that we put tox in that location. Adding it twice has AFAIK no downside.\n\nIs adding this to the PATH variable only when it is not already in it something you want to see here?","commit_id":"d4d941e923594e0753c8c56b18e9b3046e36bd54"}]}