)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"35d7f5acd9af64ffc9b43cde57f43a61012ba523","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"214e3389_d4690aa2","updated":"2023-04-17 09:30:50.000000000","message":"Validated in https://review.opendev.org/c/openstack/neutron/+/880578","commit_id":"2364327bd1e4d26214e11c2f3f4627959a6a3e88"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"a5f89a906f7333d88d70c08a1970bc1418893caf","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"8cc96ef5_42c35fc8","updated":"2023-04-14 16:06:08.000000000","message":"check experimental","commit_id":"2364327bd1e4d26214e11c2f3f4627959a6a3e88"}],"zuul.d/job-templates.yaml":[{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"7b87c599b1c3abbfce6ad815c7e56b963c51e0e3","unresolved":true,"context_lines":[{"line_number":106,"context_line":"        - neutron-ovs-tempest-with-oslo-master"},{"line_number":107,"context_line":"        - neutron-ovn-tempest-ovs-release-with-oslo-master"},{"line_number":108,"context_line":"    experimental:"},{"line_number":109,"context_line":"      jobs: *neutron-periodic-jobs"},{"line_number":110,"context_line":""},{"line_number":111,"context_line":"- project-template:"},{"line_number":112,"context_line":"    name: neutron-skip-level-jobs"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"377d8c5c_63538836","line":109,"range":{"start_line":109,"start_character":6,"end_line":109,"end_character":34},"updated":"2023-04-14 16:21:12.000000000","message":"It is not merging this list and what is defined in \"neutron-experimental-jobs:experimental:jobs\" list.\n\nShouldn\u0027t we need to define only once \"experimental\" queue? And add there the reference to *neutron-periodic-jobs (just asking, I don\u0027t see a clear example in the zuul doc).\n\nOr maybe [1] needs to be enabled.\n\n[1]https://zuul-ci.org/docs/zuul/latest/config/queue.html#attr-queue.allow-circular-dependencies","commit_id":"2364327bd1e4d26214e11c2f3f4627959a6a3e88"},{"author":{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},"change_message_id":"1b87b07bf128585b8f1f4b89e51ed8bd9beb56f8","unresolved":true,"context_lines":[{"line_number":106,"context_line":"        - neutron-ovs-tempest-with-oslo-master"},{"line_number":107,"context_line":"        - neutron-ovn-tempest-ovs-release-with-oslo-master"},{"line_number":108,"context_line":"    experimental:"},{"line_number":109,"context_line":"      jobs: *neutron-periodic-jobs"},{"line_number":110,"context_line":""},{"line_number":111,"context_line":"- project-template:"},{"line_number":112,"context_line":"    name: neutron-skip-level-jobs"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"a28e8eb0_b62e5927","line":109,"range":{"start_line":109,"start_character":6,"end_line":109,"end_character":34},"in_reply_to":"377d8c5c_63538836","updated":"2023-04-17 06:27:38.000000000","message":"\u003c\u003c It is not merging this list and what is defined in \"neutron-experimental-jobs:experimental:jobs\" list.\nActually it is working but due to irrelevant-files defined in the jobs not all jobs got triggered in this patch. Pushed test patch to confirm it https://review.opendev.org/c/openstack/neutron/+/880578, results in https://zuul.openstack.org/buildset/5dbe78bbecf34b2b9171b550d2c11954\n\n\u003c\u003c Shouldn\u0027t we need to define only once \"experimental\" queue? And add there the reference to *neutron-periodic-jobs (just asking, I don\u0027t see a clear example in the zuul doc).\n\nWe can define the queue multiple times in multiple templates like we already doing in templates neutron-skip-level-jobs and neutron-experimental-jobs.\nyaml aliases don\u0027t work with list merge(with dictionaries it works) so defining just once and adding reference for *neutron-periodic-jobs will not work.\n\n\u003c\u003c Or maybe [1] needs to be enabled.\n/me seeing this first time and reading it it\u0027s related to allowing circular dependencies across cross projects.","commit_id":"2364327bd1e4d26214e11c2f3f4627959a6a3e88"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"35d7f5acd9af64ffc9b43cde57f43a61012ba523","unresolved":false,"context_lines":[{"line_number":106,"context_line":"        - neutron-ovs-tempest-with-oslo-master"},{"line_number":107,"context_line":"        - neutron-ovn-tempest-ovs-release-with-oslo-master"},{"line_number":108,"context_line":"    experimental:"},{"line_number":109,"context_line":"      jobs: *neutron-periodic-jobs"},{"line_number":110,"context_line":""},{"line_number":111,"context_line":"- project-template:"},{"line_number":112,"context_line":"    name: neutron-skip-level-jobs"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"ae98c887_464f4e3c","line":109,"range":{"start_line":109,"start_character":6,"end_line":109,"end_character":34},"in_reply_to":"a28e8eb0_b62e5927","updated":"2023-04-17 09:30:50.000000000","message":"Ack","commit_id":"2364327bd1e4d26214e11c2f3f4627959a6a3e88"}]}
