)]}'
{"roles/run-tempest-26/defaults/main.yaml":[{"author":{"_account_id":22873,"name":"Martin Kopec","email":"mkopec@redhat.com","username":"mkopec"},"change_message_id":"56ec2319cdf19a7631109f29b2b8c78b76915121","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":7,"id":"ddba0bc7_ddb958d3","line":10,"updated":"2021-06-18 07:50:14.000000000","message":"run-tempest role contains one more default value:\n\n```\n# NOTE(gmann): external_bridge_mtu shows as undefined for run-tempest role\n# defining default value here to avoid that error.\nexternal_bridge_mtu: 0\n```\n\nIs it not needed in run-tempest-26?","commit_id":"b0cde4b9530e70343b65af5c7e52e5b0b8d8f0c5"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"54cedea916547c4dc71fa05400aafe8dc4532823","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":7,"id":"a66cb39b_f750c380","line":10,"in_reply_to":"ddba0bc7_ddb958d3","updated":"2021-06-24 21:51:38.000000000","message":"not sure if that is needed for old tempest and I do not remember the failure (which job and which branch) so safer side I will add it.\n\nThanks","commit_id":"b0cde4b9530e70343b65af5c7e52e5b0b8d8f0c5"}],"roles/run-tempest-26/tasks/main.yaml":[{"author":{"_account_id":22873,"name":"Martin Kopec","email":"mkopec@redhat.com","username":"mkopec"},"change_message_id":"56ec2319cdf19a7631109f29b2b8c78b76915121","unresolved":true,"context_lines":[{"line_number":25,"context_line":"    target_branch: \"{{ zuul.override_checkout }}\""},{"line_number":26,"context_line":"  when: zuul.override_checkout is defined"},{"line_number":27,"context_line":""},{"line_number":28,"context_line":"- name: Use stable branch upper-constraints till stable/rocky"},{"line_number":29,"context_line":"  set_fact:"},{"line_number":30,"context_line":"    # TOX_CONSTRAINTS_FILE is new name, UPPER_CONSTRAINTS_FILE is old one, best to set both"},{"line_number":31,"context_line":"    tempest_tox_environment: \"{{ tempest_tox_environment | combine({\u0027UPPER_CONSTRAINTS_FILE\u0027: stable_constraints_file}) | combine({\u0027TOX_CONSTRAINTS_FILE\u0027: stable_constraints_file}) }}\""}],"source_content_type":"text/x-yaml","patch_set":7,"id":"72948651_9b81e5f8","line":28,"range":{"start_line":28,"start_character":56,"end_line":28,"end_character":61},"updated":"2021-06-18 07:50:14.000000000","message":"just rocky? not until stein as it\u0027s in run-tempest as well?","commit_id":"b0cde4b9530e70343b65af5c7e52e5b0b8d8f0c5"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"54cedea916547c4dc71fa05400aafe8dc4532823","unresolved":true,"context_lines":[{"line_number":25,"context_line":"    target_branch: \"{{ zuul.override_checkout }}\""},{"line_number":26,"context_line":"  when: zuul.override_checkout is defined"},{"line_number":27,"context_line":""},{"line_number":28,"context_line":"- name: Use stable branch upper-constraints till stable/rocky"},{"line_number":29,"context_line":"  set_fact:"},{"line_number":30,"context_line":"    # TOX_CONSTRAINTS_FILE is new name, UPPER_CONSTRAINTS_FILE is old one, best to set both"},{"line_number":31,"context_line":"    tempest_tox_environment: \"{{ tempest_tox_environment | combine({\u0027UPPER_CONSTRAINTS_FILE\u0027: stable_constraints_file}) | combine({\u0027TOX_CONSTRAINTS_FILE\u0027: stable_constraints_file}) }}\""}],"source_content_type":"text/x-yaml","patch_set":7,"id":"14e376a5_c5f368f7","line":28,"range":{"start_line":28,"start_character":56,"end_line":28,"end_character":61},"in_reply_to":"72948651_9b81e5f8","updated":"2021-06-24 21:51:38.000000000","message":"yes, we need stein also here","commit_id":"b0cde4b9530e70343b65af5c7e52e5b0b8d8f0c5"},{"author":{"_account_id":22873,"name":"Martin Kopec","email":"mkopec@redhat.com","username":"mkopec"},"change_message_id":"56ec2319cdf19a7631109f29b2b8c78b76915121","unresolved":true,"context_lines":[{"line_number":30,"context_line":"    # TOX_CONSTRAINTS_FILE is new name, UPPER_CONSTRAINTS_FILE is old one, best to set both"},{"line_number":31,"context_line":"    tempest_tox_environment: \"{{ tempest_tox_environment | combine({\u0027UPPER_CONSTRAINTS_FILE\u0027: stable_constraints_file}) | combine({\u0027TOX_CONSTRAINTS_FILE\u0027: stable_constraints_file}) }}\""},{"line_number":32,"context_line":"  when: target_branch in [\"stable/ocata\", \"stable/pike\", \"stable/queens\", \"stable/rocky\"]"},{"line_number":33,"context_line":""},{"line_number":34,"context_line":"- name: Set OS_TEST_TIMEOUT if requested"},{"line_number":35,"context_line":"  set_fact:"},{"line_number":36,"context_line":"    tempest_tox_environment: \"{{ tempest_tox_environment | combine({\u0027OS_TEST_TIMEOUT\u0027: tempest_test_timeout}) }}\""}],"source_content_type":"text/x-yaml","patch_set":7,"id":"9328169a_62209ecc","line":33,"updated":"2021-06-18 07:50:14.000000000","message":"run-tempest contains also this task:\n```\n- name: Use Configured upper-constraints for non-master Tempest\n  set_fact:\n    # TOX_CONSTRAINTS_FILE is new name, UPPER_CONSTRAINTS_FILE is old one, best to set both\n    tempest_tox_environment: \"{{ tempest_tox_environment | combine({\u0027UPPER_CONSTRAINTS_FILE\u0027: devstack_localrc[\u0027TEMPEST_VENV_UPPER_CONSTRAINTS\u0027]}) | combine({\u0027TOX_CONSTRAINTS_FILE\u0027: devstack_localrc[\u0027TEMPEST_VENV_UPPER_CONSTRAINTS\u0027]}) }}\"\n  when:\n    - devstack_localrc is defined\n    - \"\u0027TEMPEST_BRANCH\u0027 in devstack_localrc\"\n    - \"\u0027TEMPEST_VENV_UPPER_CONSTRAINTS\u0027 in devstack_localrc\"\n    - devstack_localrc[\u0027TEMPEST_BRANCH\u0027] !\u003d \u0027master\u0027\n    - devstack_localrc[\u0027TEMPEST_VENV_UPPER_CONSTRAINTS\u0027] !\u003d \u0027master\u0027\n```\n\nIs it not needed in run-tempest-26?","commit_id":"b0cde4b9530e70343b65af5c7e52e5b0b8d8f0c5"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"54cedea916547c4dc71fa05400aafe8dc4532823","unresolved":true,"context_lines":[{"line_number":30,"context_line":"    # TOX_CONSTRAINTS_FILE is new name, UPPER_CONSTRAINTS_FILE is old one, best to set both"},{"line_number":31,"context_line":"    tempest_tox_environment: \"{{ tempest_tox_environment | combine({\u0027UPPER_CONSTRAINTS_FILE\u0027: stable_constraints_file}) | combine({\u0027TOX_CONSTRAINTS_FILE\u0027: stable_constraints_file}) }}\""},{"line_number":32,"context_line":"  when: target_branch in [\"stable/ocata\", \"stable/pike\", \"stable/queens\", \"stable/rocky\"]"},{"line_number":33,"context_line":""},{"line_number":34,"context_line":"- name: Set OS_TEST_TIMEOUT if requested"},{"line_number":35,"context_line":"  set_fact:"},{"line_number":36,"context_line":"    tempest_tox_environment: \"{{ tempest_tox_environment | combine({\u0027OS_TEST_TIMEOUT\u0027: tempest_test_timeout}) }}\""}],"source_content_type":"text/x-yaml","patch_set":7,"id":"ecfab1db_c30d32a3","line":33,"in_reply_to":"9328169a_62209ecc","updated":"2021-06-24 21:51:38.000000000","message":"let me check how long I backported this TEMPEST_VENV_UPPER_CONSTRAINTS and accordingly I can add that.","commit_id":"b0cde4b9530e70343b65af5c7e52e5b0b8d8f0c5"}]}
