)]}'
{"tasks/main.yml":[{"author":{"_account_id":8367,"name":"Arx Cruz","email":"arxcruz@redhat.com","username":"arxcruz"},"change_message_id":"de2b753a44abebe04690f45c6c259b8e375d7710","unresolved":false,"context_lines":[{"line_number":35,"context_line":"- name: Gather variables for source installs"},{"line_number":36,"context_line":"  include_vars: source_install.yml"},{"line_number":37,"context_line":"  when:"},{"line_number":38,"context_line":"    - ((tempest_install_method \u003d\u003d \u0027source\u0027) or (tempest_install_method \u003d\u003d \u0027distro\u0027 and ansible_distribution \u003d\u003d \u0027Ubuntu\u0027))"},{"line_number":39,"context_line":"  tags:"},{"line_number":40,"context_line":"    - always"},{"line_number":41,"context_line":""}],"source_content_type":"text/x-yaml","patch_set":5,"id":"3fa7e38b_15746c03","line":38,"updated":"2020-02-10 09:10:35.000000000","message":"I would sugest to have the and logic in a new line in the list for readability but I won\u0027t block the change","commit_id":"ffad7ca0a7fcb4961625e655947be89a9619d6eb"},{"author":{"_account_id":28619,"name":"Dmitriy Rabotyagov","email":"noonedeadpunk@gmail.com","username":"noonedeadpunk"},"change_message_id":"47ed0373e4462f32f06678dafd7af33c187f1495","unresolved":false,"context_lines":[{"line_number":35,"context_line":"- name: Gather variables for source installs"},{"line_number":36,"context_line":"  include_vars: source_install.yml"},{"line_number":37,"context_line":"  when:"},{"line_number":38,"context_line":"    - ((tempest_install_method \u003d\u003d \u0027source\u0027) or (tempest_install_method \u003d\u003d \u0027distro\u0027 and ansible_distribution \u003d\u003d \u0027Ubuntu\u0027))"},{"line_number":39,"context_line":"  tags:"},{"line_number":40,"context_line":"    - always"},{"line_number":41,"context_line":""}],"source_content_type":"text/x-yaml","patch_set":5,"id":"3fa7e38b_5da9bfdb","line":38,"range":{"start_line":38,"start_character":87,"end_line":38,"end_character":119},"updated":"2020-02-07 15:50:33.000000000","message":"ditto","commit_id":"ffad7ca0a7fcb4961625e655947be89a9619d6eb"}],"tasks/tempest_install.yml":[{"author":{"_account_id":28619,"name":"Dmitriy Rabotyagov","email":"noonedeadpunk@gmail.com","username":"noonedeadpunk"},"change_message_id":"f1845e2dee600aa6de771d5403eacf9fde5fcd8b","unresolved":false,"context_lines":[{"line_number":31,"context_line":"# in defaults as in case of non-integrated tests meta handlers from other roles might fail."},{"line_number":32,"context_line":"- name: Install Tempest packages from PIP"},{"line_number":33,"context_line":"  import_tasks: tempest_install_source.yml"},{"line_number":34,"context_line":"  when: (tempest_install_method | default(\u0027source\u0027)) \u003d\u003d \u0027source\u0027 or (tempest_install_method \u003d\u003d \u0027distro\u0027 and ansible_distribution \u003d\u003d \u0027Ubuntu\u0027)"},{"line_number":35,"context_line":""},{"line_number":36,"context_line":"- name: Install stackviz on venv"},{"line_number":37,"context_line":"  import_role:"}],"source_content_type":"text/x-yaml","patch_set":5,"id":"3fa7e38b_5d645fe6","line":34,"range":{"start_line":34,"start_character":108,"end_line":34,"end_character":140},"updated":"2020-02-07 15:49:48.000000000","message":"I just have a thought, that Debian actually do not ship tempest packages as well (at least according to [1])? So maybe we should use ansible_os_family isntead?\n\n[1] http://osbpo.debian.net/osbpo/dists/buster-train-backports/main/binary-amd64/Packages","commit_id":"ffad7ca0a7fcb4961625e655947be89a9619d6eb"}]}