)]}'
{"playbooks/tempest.yml":[{"author":{"_account_id":24162,"name":"Sorin Sbârnea","display_name":"zbr","email":"ssbarnea@redhat.com","username":"ssbarnea","status":"do not feed the troll 🥕"},"change_message_id":"0b2c2149158a6b4da30ae6ac57f65202e0b6c713","unresolved":false,"context_lines":[{"line_number":68,"context_line":"    debug: true"},{"line_number":69,"context_line":"    tempest_run: \u0027yes\u0027"},{"line_number":70,"context_line":"    # For python_venv_build role"},{"line_number":71,"context_line":"    venv_python_executable: \u003e-"},{"line_number":72,"context_line":"      {{ (ansible_distribution in [\u0027RedHat\u0027, \u0027CentOS\u0027] and"},{"line_number":73,"context_line":"      ansible_distribution_major_version|int \u003e\u003d 8) |"},{"line_number":74,"context_line":"      ternary(\"python3\", \"python2\") }}"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"7faddb67_7a17c847","line":71,"range":{"start_line":71,"start_character":4,"end_line":71,"end_character":26},"updated":"2019-09-01 10:19:18.000000000","message":"Do we really need to do this? My impression that this is something that should be taken care of inside os_tempest role itself (as in \"default behavior\" of the role).","commit_id":"83422785ecfb7d70cf91e0d5417cb2526bf5bd46"},{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"2a67801db19af75df0aad5176f20ec06a4680b50","unresolved":false,"context_lines":[{"line_number":68,"context_line":"    debug: true"},{"line_number":69,"context_line":"    tempest_run: \u0027yes\u0027"},{"line_number":70,"context_line":"    # For python_venv_build role"},{"line_number":71,"context_line":"    venv_python_executable: \u003e-"},{"line_number":72,"context_line":"      {{ (ansible_distribution in [\u0027RedHat\u0027, \u0027CentOS\u0027] and"},{"line_number":73,"context_line":"      ansible_distribution_major_version|int \u003e\u003d 8) |"},{"line_number":74,"context_line":"      ternary(\"python3\", \"python2\") }}"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"7faddb67_fe1dbe61","line":71,"range":{"start_line":71,"start_character":4,"end_line":71,"end_character":26},"in_reply_to":"7faddb67_641dc6bc","updated":"2019-09-04 14:11:31.000000000","message":"Moved it to https://review.opendev.org/#/c/680004/","commit_id":"83422785ecfb7d70cf91e0d5417cb2526bf5bd46"},{"author":{"_account_id":10969,"name":"Shnaidman Sagi (Sergey)","display_name":"Shnaidman Sagi","email":"sshnaidm@redhat.com","username":"sergsh"},"change_message_id":"02af8aa893472a778964eb615d9c3ead7949f6b6","unresolved":false,"context_lines":[{"line_number":68,"context_line":"    debug: true"},{"line_number":69,"context_line":"    tempest_run: \u0027yes\u0027"},{"line_number":70,"context_line":"    # For python_venv_build role"},{"line_number":71,"context_line":"    venv_python_executable: \u003e-"},{"line_number":72,"context_line":"      {{ (ansible_distribution in [\u0027RedHat\u0027, \u0027CentOS\u0027] and"},{"line_number":73,"context_line":"      ansible_distribution_major_version|int \u003e\u003d 8) |"},{"line_number":74,"context_line":"      ternary(\"python3\", \"python2\") }}"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"7faddb67_da37dce2","line":71,"range":{"start_line":71,"start_character":4,"end_line":71,"end_character":26},"in_reply_to":"7faddb67_7a17c847","updated":"2019-09-01 10:28:30.000000000","message":"Not sure, but I\u0027ll let tempest folks to decide.","commit_id":"83422785ecfb7d70cf91e0d5417cb2526bf5bd46"},{"author":{"_account_id":24162,"name":"Sorin Sbârnea","display_name":"zbr","email":"ssbarnea@redhat.com","username":"ssbarnea","status":"do not feed the troll 🥕"},"change_message_id":"44b5428de881b40bc6ead9845f172d2b016f7822","unresolved":false,"context_lines":[{"line_number":68,"context_line":"    debug: true"},{"line_number":69,"context_line":"    tempest_run: \u0027yes\u0027"},{"line_number":70,"context_line":"    # For python_venv_build role"},{"line_number":71,"context_line":"    venv_python_executable: \u003e-"},{"line_number":72,"context_line":"      {{ (ansible_distribution in [\u0027RedHat\u0027, \u0027CentOS\u0027] and"},{"line_number":73,"context_line":"      ansible_distribution_major_version|int \u003e\u003d 8) |"},{"line_number":74,"context_line":"      ternary(\"python3\", \"python2\") }}"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"7faddb67_da659ce2","line":71,"range":{"start_line":71,"start_character":4,"end_line":71,"end_character":26},"in_reply_to":"7faddb67_da37dce2","updated":"2020-01-22 11:01:49.000000000","message":"I think it will not be hard to persuade them to fix the role itself. I can help with that if we have a CR.","commit_id":"83422785ecfb7d70cf91e0d5417cb2526bf5bd46"},{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"57db7ec75c5e1f255a531b244dc45d6168e9411e","unresolved":false,"context_lines":[{"line_number":68,"context_line":"    debug: true"},{"line_number":69,"context_line":"    tempest_run: \u0027yes\u0027"},{"line_number":70,"context_line":"    # For python_venv_build role"},{"line_number":71,"context_line":"    venv_python_executable: \u003e-"},{"line_number":72,"context_line":"      {{ (ansible_distribution in [\u0027RedHat\u0027, \u0027CentOS\u0027] and"},{"line_number":73,"context_line":"      ansible_distribution_major_version|int \u003e\u003d 8) |"},{"line_number":74,"context_line":"      ternary(\"python3\", \"python2\") }}"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"7faddb67_641dc6bc","line":71,"range":{"start_line":71,"start_character":4,"end_line":71,"end_character":26},"in_reply_to":"7faddb67_da37dce2","updated":"2019-09-04 07:07:18.000000000","message":"we can customize the same venv_python_executable in python_venv_build role also in var file as well as we need to set tempest_service_setup_host_python_interpreter: \"python3\" in https://github.com/openstack/openstack-ansible-os_tempest/blob/master/vars/redhat-8.yml also.","commit_id":"83422785ecfb7d70cf91e0d5417cb2526bf5bd46"},{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"b1a249eed78f5855507b1946b31f8b57c6c9c2c0","unresolved":false,"context_lines":[{"line_number":67,"context_line":"    ansible_become: true"},{"line_number":68,"context_line":"    debug: true"},{"line_number":69,"context_line":"    tempest_run: \u0027yes\u0027"},{"line_number":70,"context_line":"    # For python_venv_build role"},{"line_number":71,"context_line":"    venv_python_executable: \u003e-"},{"line_number":72,"context_line":"      {{ (ansible_distribution in [\u0027RedHat\u0027, \u0027CentOS\u0027] and"},{"line_number":73,"context_line":"      ansible_distribution_major_version|int \u003e\u003d 8) |"},{"line_number":74,"context_line":"      ternary(\"python3\", \"python2\") }}"},{"line_number":75,"context_line":"  include_role:"},{"line_number":76,"context_line":"    name: os_tempest"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"3fa7e38b_e339d7ce","line":74,"range":{"start_line":70,"start_character":4,"end_line":74,"end_character":38},"updated":"2020-01-22 11:15:45.000000000","message":"https://opendev.org/openstack/openstack-ansible-os_tempest/src/branch/master/vars/redhat-8.yml#L28 _tempest_venv_python_executable: \"python3\" is taken care there, I think it is not needed here.","commit_id":"26b17eb6dcf0ec3398b44c78e61db6e7cf89e056"}]}