)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"cbedb0669c9c6715f73b69201fb050839e200add","unresolved":true,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"WIP Don\u0027t use master constraints for stable branches"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"When running tempest we run in-tree tests as well. Lots of these"},{"line_number":10,"context_line":"tests including tempest tests may not work with new client"},{"line_number":11,"context_line":"libraries. Let\u0027s use the relevant branch upper constraints when"},{"line_number":12,"context_line":"running functional tests."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Change-Id: I6a8cffdc86c895eebe4269c5cd37841325566c54"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"29f62698_d12077f8","line":12,"range":{"start_line":9,"start_character":51,"end_line":12,"end_character":25},"updated":"2021-07-30 15:54:03.000000000","message":"Tempest does not use python client as we have our own service clients and they do call v3. \n\nIt seems heat integration test (non-tempest one) are using new python-cinderclient here and so fail\n- https://zuul.opendev.org/t/openstack/build/af2c0e7c2b6143568af642704fb56631/log/job-output.txt#31496","commit_id":"dcdf9431fcd4b318d3aba157c7e7077e4e887e19"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":8833,"name":"Rabi Mishra","email":"ramishra@redhat.com","username":"rabi"},"change_message_id":"666d2a460b3da7189a63ef759779312ed41548c5","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":16,"id":"b76552e0_5ba71e7c","updated":"2022-06-15 07:09:06.000000000","message":"Well, we still don\u0027t have the change to use f36 in victoria. So grenade would anyway fail.","commit_id":"dc29afd3c13a6e91a5502d26bb19b3622202b131"},{"author":{"_account_id":8833,"name":"Rabi Mishra","email":"ramishra@redhat.com","username":"rabi"},"change_message_id":"3f863008a40ff455c9120c0c22273abd2d0f097e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":16,"id":"5a9446c2_0707609c","updated":"2022-06-15 07:06:36.000000000","message":"recheck grenade","commit_id":"dc29afd3c13a6e91a5502d26bb19b3622202b131"}],"roles/run-heat-tests/tasks/main.yaml":[{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"0e1f825f27dde8a0f017259b1be6a3a11f88ca98","unresolved":true,"context_lines":[{"line_number":8,"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":9,"context_line":""},{"line_number":10,"context_line":"- name: Run heat tests"},{"line_number":11,"context_line":"  command: tox -evenv-tempest -- stestr --test-path\u003d{{devstack_base_dir}}/heat/heat_integrationtests \\"},{"line_number":12,"context_line":"            --top-dir\u003d{{devstack_base_dir}}/heat \\"},{"line_number":13,"context_line":"            --group_regex\u003d\u0027heat_tempest_plugin\\.tests\\.api\\.test_heat_api[._]([^_]+)\u0027 run"},{"line_number":14,"context_line":"  args:"},{"line_number":15,"context_line":"    chdir: \"{{devstack_base_dir}}/tempest\""},{"line_number":16,"context_line":"  become: true"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"c54c6e85_3f7b36c8","line":13,"range":{"start_line":11,"start_character":0,"end_line":13,"end_character":89},"updated":"2021-07-30 17:12:12.000000000","message":"actually this mixup of test is now becoming the complex things.\n\n- tempest test which is branchless and as Tempest, heat-tempest-plugin master is used for stable branch testing, we needs to use master constraints.\n- heat_integrationtests need stable constraints on stable branch which might not be compatible with master constraints (cinder v2 removal unhide this issue).\n\nIf we try to fix this issue by using the wallaby constraints for master Tempest and heat-tempest-plugins then it might create issue at any time.\n\nIMO, best way is to separate these two tests run in separate jobs. one for heat_integrationtests which is basically branched can use stable constraints and second only run heat-tempest-test with master constraints (which are taken care by devstack).","commit_id":"dcdf9431fcd4b318d3aba157c7e7077e4e887e19"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"855cc47a1597d4c04723b5637fc55b9d09a34d36","unresolved":true,"context_lines":[{"line_number":8,"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":9,"context_line":""},{"line_number":10,"context_line":"- name: Run heat tests"},{"line_number":11,"context_line":"  command: tox -evenv-tempest -- stestr --test-path\u003d{{devstack_base_dir}}/heat/heat_integrationtests \\"},{"line_number":12,"context_line":"            --top-dir\u003d{{devstack_base_dir}}/heat \\"},{"line_number":13,"context_line":"            --group_regex\u003d\u0027heat_tempest_plugin\\.tests\\.api\\.test_heat_api[._]([^_]+)\u0027 run"},{"line_number":14,"context_line":"  args:"},{"line_number":15,"context_line":"    chdir: \"{{devstack_base_dir}}/tempest\""},{"line_number":16,"context_line":"  become: true"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"af79d4fe_4224d3d7","line":13,"range":{"start_line":11,"start_character":0,"end_line":13,"end_character":89},"in_reply_to":"c54c6e85_3f7b36c8","updated":"2022-06-15 04:02:28.000000000","message":"I agree we should split out our tests not to mix up venvs but at this moment it\u0027d be quite challenging to make that change in all branches. We might be able to try it in recent branches (wallaby and later ?) but that still requires some effort.\n\nSo I\u0027d prefer we fix the issue by stable constraints (and pinning tempest if required) atm and consider more appropriate solution in master first.","commit_id":"dcdf9431fcd4b318d3aba157c7e7077e4e887e19"}]}