)]}'
{".zuul.d/project.yaml":[{"author":{"_account_id":841,"name":"Akihiro Motoki","email":"amotoki@gmail.com","username":"amotoki"},"change_message_id":"ee6f60cc83c1e42eb46d48a0ddec2b816249111e","unresolved":true,"context_lines":[{"line_number":13,"context_line":"      jobs:"},{"line_number":14,"context_line":"        - horizon-selenium-headless"},{"line_number":15,"context_line":"        - horizon-integration-tests"},{"line_number":16,"context_line":"        - horizon-integration-pytest-tests"},{"line_number":17,"context_line":"        - horizon-dsvm-tempest-plugin"},{"line_number":18,"context_line":"        - horizon-tox-bandit-baseline"},{"line_number":19,"context_line":"        - horizon-tempest-plugin-ipv6"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"7984e7fd_c9694d6c","line":16,"updated":"2023-10-24 13:09:36.000000000","message":"In addition, how about starting it with non-voting.","commit_id":"ab4be9f5218d550ce490806a6e314926eab6278a"}],".zuul.d/tempest-and-integrated.yaml":[{"author":{"_account_id":841,"name":"Akihiro Motoki","email":"amotoki@gmail.com","username":"amotoki"},"change_message_id":"0859a4f7268eacc89315f967befa066bf834bb8e","unresolved":true,"context_lines":[{"line_number":32,"context_line":""},{"line_number":33,"context_line":"- job:"},{"line_number":34,"context_line":"    name: horizon-integration-pytest-tests"},{"line_number":35,"context_line":"    parent: devstack"},{"line_number":36,"context_line":"    nodeset: devstack-single-node-debian-bullseye"},{"line_number":37,"context_line":"    pre-run: playbooks/horizon-devstack-integration/pre.yaml"},{"line_number":38,"context_line":"    run: playbooks/horizon-devstack-integration/run.yaml"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"76d3983d_b3f6a820","line":35,"updated":"2023-10-24 13:04:21.000000000","message":"Can\u0027t we use horizon-integration-tests as parent?\nAt a quick look, the only difference is tox_envlist.","commit_id":"ab4be9f5218d550ce490806a6e314926eab6278a"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":35153,"name":"Ashish Gupta","email":"ashigupt@redhat.com","username":"ashigupt","status":"Redhat"},"change_message_id":"40a7f01770748453cce1b841c5daadedc00bfd27","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"b86d555c_46bf1bd9","updated":"2023-10-24 14:13:47.000000000","message":"recheck","commit_id":"104234ad230bf083cfc3e4fd1cd02efeecac962a"}],"openstack_dashboard/test/selenium/conftest.py":[{"author":{"_account_id":841,"name":"Akihiro Motoki","email":"amotoki@gmail.com","username":"amotoki"},"change_message_id":"0859a4f7268eacc89315f967befa066bf834bb8e","unresolved":true,"context_lines":[{"line_number":132,"context_line":"    ]"},{"line_number":133,"context_line":"    fnull \u003d open(os.devnull, \u0027w\u0027)"},{"line_number":134,"context_line":"    print(fnull)"},{"line_number":135,"context_line":"    print(command)"},{"line_number":136,"context_line":"    popen \u003d subprocess.Popen(command, stdout\u003dfnull, stderr\u003dfnull)"},{"line_number":137,"context_line":"    yield None"},{"line_number":138,"context_line":"    popen.send_signal(signal.SIGINT)"}],"source_content_type":"text/x-python","patch_set":3,"id":"267ee97b_5790a098","line":135,"updated":"2023-10-24 13:04:21.000000000","message":"Looks unnecessary.","commit_id":"ab4be9f5218d550ce490806a6e314926eab6278a"},{"author":{"_account_id":35133,"name":"Jan Jasek","email":"jjasek@redhat.com","username":"janjasek"},"change_message_id":"ab2e4e033c2b41c7d3da73a1a36e30d7521a92cc","unresolved":true,"context_lines":[{"line_number":132,"context_line":"    ]"},{"line_number":133,"context_line":"    fnull \u003d open(os.devnull, \u0027w\u0027)"},{"line_number":134,"context_line":"    print(fnull)"},{"line_number":135,"context_line":"    print(command)"},{"line_number":136,"context_line":"    popen \u003d subprocess.Popen(command, stdout\u003dfnull, stderr\u003dfnull)"},{"line_number":137,"context_line":"    yield None"},{"line_number":138,"context_line":"    popen.send_signal(signal.SIGINT)"}],"source_content_type":"text/x-python","patch_set":3,"id":"7b3fd5a2_012c1670","line":135,"in_reply_to":"267ee97b_5790a098","updated":"2023-10-24 13:17:38.000000000","message":"This patch is duplicit of https://review.opendev.org/c/openstack/horizon/+/899029\nSo we decided to use this patch for debugging issue with video recording that we do not facing in our local environments. Then this patch will be abandoned.","commit_id":"ab4be9f5218d550ce490806a6e314926eab6278a"}],"openstack_dashboard/test/selenium/integration/test_instances.py":[{"author":{"_account_id":35133,"name":"Jan Jasek","email":"jjasek@redhat.com","username":"janjasek"},"change_message_id":"3f18f26c40752212751ce77b26a00c63bb944a08","unresolved":true,"context_lines":[{"line_number":32,"context_line":""},{"line_number":33,"context_line":""},{"line_number":34,"context_line":"@pytest.fixture"},{"line_number":35,"context_line":"def update_default_network(openstack_admin):"},{"line_number":36,"context_line":"    openstack_admin.update_network("},{"line_number":37,"context_line":"        default_network,"},{"line_number":38,"context_line":"        shared\u003dTrue,"}],"source_content_type":"text/x-python","patch_set":7,"id":"22947393_939d0cbb","line":35,"range":{"start_line":35,"start_character":4,"end_line":35,"end_character":26},"updated":"2023-10-25 09:58:22.000000000","message":"I think that it is not a good way to solve our issues. It could be a good idea if we are facing the issue in one test but not when we are talking about setting the whole environment.\nI am afraid that if we will fix all the environment issues this way then the code will be a crazy mess.\nFrom my point of view the code has to be as clear as possible (ideally without fixtures like this) and executable without any issues in local environment (in clean devstack with default settings).\nSo I would recommend fixing all those issues in the Zuul environment level. We have to find what is the reason that the Zuul environment is different than clear devstack (with default setting) and then suppress the changes or fix them using some pre.yaml before tests will be executed.\n\nThis is probably the part that causes that somewhere there is a \"shared\" network and somewhere not. (parent for our job is \"devstack\")\nhttps://github.com/openstack/devstack/blob/4975fad23efeb97e5b8fd799c93ad9367face1ba/lib/tempest#L365\n\nI will continue with investigation.","commit_id":"e8c7a08d5fc2f7563de0ce5e70f21edb0989c547"},{"author":{"_account_id":35153,"name":"Ashish Gupta","email":"ashigupt@redhat.com","username":"ashigupt","status":"Redhat"},"change_message_id":"95d2734a16e005d650ca3519aa9ba3c99135bbdc","unresolved":false,"context_lines":[{"line_number":32,"context_line":""},{"line_number":33,"context_line":""},{"line_number":34,"context_line":"@pytest.fixture"},{"line_number":35,"context_line":"def update_default_network(openstack_admin):"},{"line_number":36,"context_line":"    openstack_admin.update_network("},{"line_number":37,"context_line":"        default_network,"},{"line_number":38,"context_line":"        shared\u003dTrue,"}],"source_content_type":"text/x-python","patch_set":7,"id":"48bfd0e8_46136b2f","line":35,"range":{"start_line":35,"start_character":4,"end_line":35,"end_character":26},"in_reply_to":"22947393_939d0cbb","updated":"2023-10-25 15:42:08.000000000","message":"Yes this is not the fix here and will be removed , this was just for testing purpose","commit_id":"e8c7a08d5fc2f7563de0ce5e70f21edb0989c547"}]}