)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"fb8e6276a21754cba6aff90ef2c98cbb47c4b372","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"8a1c3255_e7e878c7","updated":"2025-11-24 08:25:44.000000000","message":"good idea","commit_id":"8da340842c1eba46be83b486b2ea908a3a1f6591"},{"author":{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},"change_message_id":"f5028da4f4aa4d3b5b3fc03fb5b026dbeace286d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"02607633_bb5c9b97","updated":"2025-11-24 09:00:50.000000000","message":"why adding only to gate queue? it should be either check or both check/gate","commit_id":"8da340842c1eba46be83b486b2ea908a3a1f6591"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"ff9532d0fcc9613c8818cdac8c4b42fb45711147","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"7ef57214_81c64a52","in_reply_to":"02607633_bb5c9b97","updated":"2025-11-24 09:27:52.000000000","message":"I think that adding it to the gate queue is to reduce the CI execution time. It is unlikely to have these errors in the Neutron CI and this is just a late check.\n\nWe could also have, in Neutron, a periodic `neutron-functional-with-ovsdbapp-master` too.","commit_id":"8da340842c1eba46be83b486b2ea908a3a1f6591"},{"author":{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},"change_message_id":"30f4a383114cbc9022130804b5420251ee9b8787","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"a1800cbb_980816ed","in_reply_to":"23f60781_20344900","updated":"2025-11-26 04:56:31.000000000","message":"yes goal wise it\u0027s totally fine to avoid merging if impacts neutron and for that running a voting job in check queue here is enough. Only concern as raised above was constant loop in feedback that should be avoided, so moving this job to check queue should be enough. Also now instead of adding neutron-functional in check you can utilize the new job neutron-functional-with-ovsdbapp-master or else have to add ovsdbapp to required-projects.","commit_id":"8da340842c1eba46be83b486b2ea908a3a1f6591"},{"author":{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},"change_message_id":"f9a76b8aa35d75643cb9485a40a9a4344c555b15","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"bc8f47db_c8bfcea3","in_reply_to":"312245d4_936a1304","updated":"2025-11-24 15:44:18.000000000","message":"The problem with running in \"only gate\" is that you get very late feedback from contributor/reviewer perspective and require multiple approvers, like you keep working to get all check jobs working, then you get a surprise(which is good as we caught an issue) a new job failed. Issue is fixed in next iteration, we see again green in check, then until +W and gate job run issue is fixed, if it fails same set of iterations are needed until it get\u0027s fixed.\nAnd this is not a normal workflow and is not ideal from contributor/reviewer perspective but if we want to get into this good to get quoram with wider audience, as may be just me missing on this.\n\nAlso adding such new jobs only in gate we don\u0027t know(atleast i have a doubt based on recalling previous similar things) if this new job works as expected(i.e it tests the patch in question without adding it in required-projects/LIBS_FROM_GIT) until it merges.\nWhat rodolfo pointed may be better too to have a job in periodic , again not an ideal one as we catch the issue only after merge but quite normal as we can\u0027t have full coverages in all projects.","commit_id":"8da340842c1eba46be83b486b2ea908a3a1f6591"},{"author":{"_account_id":8655,"name":"Jakub Libosvar","email":"libosvar@redhat.com","username":"jlibosva"},"change_message_id":"96afe55d9a8b967e42925efed16e6dc8e1f09ca8","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"23f60781_20344900","in_reply_to":"6973ef0c_8a1db10a","updated":"2025-11-25 21:03:46.000000000","message":"I see, I\u0027d be more trusting that people can run functional tests on their own and do such testing even before pushing the patch and trust people if they say they fixed what we wanted. But it\u0027s probably just because don\u0027t like how we created so many jobs in the check queue in Neutron nowadays. I wanted to have a way how to prevent merging code that breaks Neutron, not really give a feedback on a test.","commit_id":"8da340842c1eba46be83b486b2ea908a3a1f6591"},{"author":{"_account_id":8655,"name":"Jakub Libosvar","email":"libosvar@redhat.com","username":"jlibosva"},"change_message_id":"d781d17a213b178efd2e12b99951f176de4d8c46","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"312245d4_936a1304","in_reply_to":"7ef57214_81c64a52","updated":"2025-11-24 14:58:11.000000000","message":"I thought of the gate queue just to avoid merging patches that break Neutron and rather get feedback on ovsdbapp patches quicker. I think it makes sense to have it in gate only and skip the check as the check is mostly to validate ovsdbapp and not 3rd parties. Neutron being the biggest consumer would make sense to have a safeguard imho.","commit_id":"8da340842c1eba46be83b486b2ea908a3a1f6591"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"790d0f7886fd4b326647bdc6aa134c245bfb221f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"fa6f9ff5_cbb4b14f","in_reply_to":"bc8f47db_c8bfcea3","updated":"2025-11-25 12:14:19.000000000","message":"I\u0027ll propose the patch of the FT job for Neutron","commit_id":"8da340842c1eba46be83b486b2ea908a3a1f6591"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"6227c90aa4279b0d3f3d86814cb9f7520437f4da","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"6973ef0c_8a1db10a","in_reply_to":"fa6f9ff5_cbb4b14f","updated":"2025-11-25 12:20:00.000000000","message":"https://review.opendev.org/c/openstack/neutron/+/968317","commit_id":"8da340842c1eba46be83b486b2ea908a3a1f6591"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"7866e4f70fb3820ac5811e1dbe79d025e3c97cb2","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"c0c7977e_008b1e31","updated":"2026-01-15 07:00:47.000000000","message":"recheck neutron-functional-with-ovsdbapp-master","commit_id":"92cdd3d987083e92dba839e06ceeea8abb6ff683"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"fa6b596fd9e34e4e37b26d85e4b9c32d48a44005","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"31eb8301_7d059b51","in_reply_to":"aba1841b_9c94b1f1","updated":"2026-01-16 07:17:07.000000000","message":"Done. Thanks for the tip.","commit_id":"92cdd3d987083e92dba839e06ceeea8abb6ff683"},{"author":{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},"change_message_id":"6b280a0f12f154d0b1dd11e4482bc468601076c4","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"aba1841b_9c94b1f1","in_reply_to":"c0c7977e_008b1e31","updated":"2026-01-15 07:14:29.000000000","message":"failure looks real and will require test_setup_skip: true to be set in job override","commit_id":"92cdd3d987083e92dba839e06ceeea8abb6ff683"},{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"5112902b85a6946ab488c31f5b66fdd75671e4b9","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"432e5297_7a8e23ea","updated":"2026-02-18 11:30:44.000000000","message":"perhaps tox.ini must be changed also, at least the error msg says: \"ERROR: unknown environment \u0027dsvm-functional-gate\u0027\"","commit_id":"6c07877a442eee11859c97b52907de79da5cd14f"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"b19d8dbc158b3fb56377e11ae1be83f17a529a0b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"77954a63_c68c3abe","updated":"2026-02-17 19:42:44.000000000","message":"recheck get new logs","commit_id":"6c07877a442eee11859c97b52907de79da5cd14f"},{"author":{"_account_id":5756,"name":"Terry Wilson","email":"twilson@redhat.com","username":"otherwiseguy"},"change_message_id":"b6a356e3a6e6f7cd318a8209a3b1a15e0d76dfc6","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"2f816df6_d6d33185","updated":"2026-04-16 13:09:57.000000000","message":"Can we define a minimal set of functional tests to run? Maybe just ovn tests? Seems like there would be a ton of irrelevant tests in the standard neutron functional job. Verrry tentative -1. Maybe it doesn\u0027t matter that much.","commit_id":"f6a527c3fc1b53e094bebae326eb949f4828c815"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"648bea4328a84887e898dde66c12c49077e58f4d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"3252b222_8edba4a1","in_reply_to":"2f816df6_d6d33185","updated":"2026-04-23 15:06:06.000000000","message":"You are right. We can try initially running only ML2/OVN ones. Let me check how to do this.","commit_id":"f6a527c3fc1b53e094bebae326eb949f4828c815"},{"author":{"_account_id":8655,"name":"Jakub Libosvar","email":"libosvar@redhat.com","username":"jlibosva"},"change_message_id":"bfc851351c1638ed2fdd40fb9897d6b28bf67db9","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"3c227018_8de3f37d","in_reply_to":"3252b222_8edba4a1","updated":"2026-04-23 15:08:18.000000000","message":"I have it on my plate too, if you don\u0027t want to deal with it.","commit_id":"f6a527c3fc1b53e094bebae326eb949f4828c815"},{"author":{"_account_id":8655,"name":"Jakub Libosvar","email":"libosvar@redhat.com","username":"jlibosva"},"change_message_id":"6d0342cdd2e780835500ab663f4d3b3bf5aba824","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"db10566d_6ac451d9","updated":"2026-04-23 15:33:38.000000000","message":"Thanks :)","commit_id":"2a41c7526db6d2a8c32aa118ba9c45917e452aff"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"e55092fe53072f7cd4db8f8f40c9e5b5b4cc8233","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"1f386625_7a5b1b3b","updated":"2026-04-28 09:28:00.000000000","message":"I think the patch is now ready for review","commit_id":"e5fe03382e59c8b1df0442f4432d1b2bf22bb53d"}],"zuul.d/ovsdbapp-jobs.yaml":[{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"bc1520863d0110a812a019d09fed36c0db0f285d","unresolved":true,"context_lines":[{"line_number":8,"context_line":"        OVN_BUILD_MODULES: False"},{"line_number":9,"context_line":""},{"line_number":10,"context_line":"- job:"},{"line_number":11,"context_line":"    name: ovsdbapp-neutron-functional-with-ovsdbapp-master"},{"line_number":12,"context_line":"    parent: neutron-functional-with-ovsdbapp-master"},{"line_number":13,"context_line":"    vars:"},{"line_number":14,"context_line":"      test_setup_skip: true"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"5fb575ce_0a5754ff","line":11,"updated":"2026-03-12 11:13:04.000000000","message":"I think this is going to be more complicated. We would need to execute the setup and add the tox environment copying the neutron `dsvm-functional-gate` env. Usually the functional tests use the tests from the same repo.\n\nWe already have `ovsdbapp-neutron-ovn-tempest-ovs-release` and we can rely on the periodic execution of `neutron-functional-with-ovsdbapp-master`.","commit_id":"6c07877a442eee11859c97b52907de79da5cd14f"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"6679e43605064b63a667622e30aefa93375a239a","unresolved":false,"context_lines":[{"line_number":8,"context_line":"        OVN_BUILD_MODULES: False"},{"line_number":9,"context_line":""},{"line_number":10,"context_line":"- job:"},{"line_number":11,"context_line":"    name: ovsdbapp-neutron-functional-with-ovsdbapp-master"},{"line_number":12,"context_line":"    parent: neutron-functional-with-ovsdbapp-master"},{"line_number":13,"context_line":"    vars:"},{"line_number":14,"context_line":"      test_setup_skip: true"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"2576b816_0fab5d02","line":11,"in_reply_to":"5fb575ce_0a5754ff","updated":"2026-04-15 11:13:47.000000000","message":"Done","commit_id":"6c07877a442eee11859c97b52907de79da5cd14f"}],"zuul.d/project.yaml":[{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"6d1400c09c8e2346a0c820bdb42479e41fd84dc0","unresolved":true,"context_lines":[{"line_number":11,"context_line":"      jobs:"},{"line_number":12,"context_line":"        - openstack-tox-functional"},{"line_number":13,"context_line":"        - ovsdbapp-neutron-ovn-tempest-ovs-release"},{"line_number":14,"context_line":"        - neutron-functional"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"d5c16a90_ae9828bf","line":14,"updated":"2026-01-13 14:14:15.000000000","message":"https://review.opendev.org/c/openstack/neutron/+/968317 merged. This should be `neutron-functional-with-ovsdbapp-master`.\n\nAlso it is worth considering Yatin\u0027s proposal to also add it to `check` queue","commit_id":"8da340842c1eba46be83b486b2ea908a3a1f6591"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"f85c21a75a6bac36660b0f8229610ac8e77f7206","unresolved":false,"context_lines":[{"line_number":11,"context_line":"      jobs:"},{"line_number":12,"context_line":"        - openstack-tox-functional"},{"line_number":13,"context_line":"        - ovsdbapp-neutron-ovn-tempest-ovs-release"},{"line_number":14,"context_line":"        - neutron-functional"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"7432ccb4_9af33424","line":14,"in_reply_to":"d5c16a90_ae9828bf","updated":"2026-01-14 22:35:15.000000000","message":"Done","commit_id":"8da340842c1eba46be83b486b2ea908a3a1f6591"},{"author":{"_account_id":8655,"name":"Jakub Libosvar","email":"libosvar@redhat.com","username":"jlibosva"},"change_message_id":"6d0342cdd2e780835500ab663f4d3b3bf5aba824","unresolved":true,"context_lines":[{"line_number":24,"context_line":"            vars:"},{"line_number":25,"context_line":"              zuul_work_dir: src/opendev.org/openstack/neutron"},{"line_number":26,"context_line":"              tox_envlist: dsvm-functional"},{"line_number":27,"context_line":"              tox_extra_args: \"neutron.tests.functional.plugins.ml2.drivers.ovn.*\""},{"line_number":28,"context_line":""},{"line_number":29,"context_line":"    gate:"},{"line_number":30,"context_line":"      jobs:"}],"source_content_type":"text/x-yaml","patch_set":7,"id":"2b57cc94_6195a5f2","line":27,"updated":"2026-04-23 15:33:38.000000000","message":"`neutron.tests.functional.services.ovn_l3` too? Also I think you do not need the asterisk as `neutron.tests.functional.plugins.ml2.drivers.ovn` already matches the regex","commit_id":"2a41c7526db6d2a8c32aa118ba9c45917e452aff"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"e4b12918a1793397023408b3f9d56adf4d986530","unresolved":false,"context_lines":[{"line_number":24,"context_line":"            vars:"},{"line_number":25,"context_line":"              zuul_work_dir: src/opendev.org/openstack/neutron"},{"line_number":26,"context_line":"              tox_envlist: dsvm-functional"},{"line_number":27,"context_line":"              tox_extra_args: \"neutron.tests.functional.plugins.ml2.drivers.ovn.*\""},{"line_number":28,"context_line":""},{"line_number":29,"context_line":"    gate:"},{"line_number":30,"context_line":"      jobs:"}],"source_content_type":"text/x-yaml","patch_set":7,"id":"b226d5f4_4145f6d7","line":27,"in_reply_to":"2b57cc94_6195a5f2","updated":"2026-04-28 08:56:05.000000000","message":"Done","commit_id":"2a41c7526db6d2a8c32aa118ba9c45917e452aff"},{"author":{"_account_id":8655,"name":"Jakub Libosvar","email":"libosvar@redhat.com","username":"jlibosva"},"change_message_id":"02267d15f067a2bb8a73ed5c998195a3e8728475","unresolved":true,"context_lines":[{"line_number":19,"context_line":"              - ^LICENSE$"},{"line_number":20,"context_line":"              - ^releasenotes/.*$"},{"line_number":21,"context_line":"              - ^tox.ini$"},{"line_number":22,"context_line":"            required-projects:"},{"line_number":23,"context_line":"              - openstack/ovsdbapp"},{"line_number":24,"context_line":"            vars:"},{"line_number":25,"context_line":"              zuul_work_dir: src/opendev.org/openstack/neutron"},{"line_number":26,"context_line":"              tox_envlist: dsvm-functional"}],"source_content_type":"text/x-yaml","patch_set":8,"id":"042658e9_943db76f","line":23,"range":{"start_line":22,"start_character":0,"end_line":23,"end_character":34},"updated":"2026-04-28 12:09:06.000000000","message":"Is this needed?","commit_id":"e5fe03382e59c8b1df0442f4432d1b2bf22bb53d"},{"author":{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},"change_message_id":"acfb9918a9a1e15a98fd0a9201dab28e5e393684","unresolved":false,"context_lines":[{"line_number":19,"context_line":"              - ^LICENSE$"},{"line_number":20,"context_line":"              - ^releasenotes/.*$"},{"line_number":21,"context_line":"              - ^tox.ini$"},{"line_number":22,"context_line":"            required-projects:"},{"line_number":23,"context_line":"              - openstack/ovsdbapp"},{"line_number":24,"context_line":"            vars:"},{"line_number":25,"context_line":"              zuul_work_dir: src/opendev.org/openstack/neutron"},{"line_number":26,"context_line":"              tox_envlist: dsvm-functional"}],"source_content_type":"text/x-yaml","patch_set":8,"id":"2e2f397c_f15e0760","line":23,"range":{"start_line":22,"start_character":0,"end_line":23,"end_character":34},"in_reply_to":"042658e9_943db76f","updated":"2026-04-28 12:18:22.000000000","message":"yes, without it job uses u-c version of ovsdbapp.","commit_id":"e5fe03382e59c8b1df0442f4432d1b2bf22bb53d"},{"author":{"_account_id":8655,"name":"Jakub Libosvar","email":"libosvar@redhat.com","username":"jlibosva"},"change_message_id":"cc2af24b88f8d6705293e94947e9af66473c348e","unresolved":false,"context_lines":[{"line_number":19,"context_line":"              - ^LICENSE$"},{"line_number":20,"context_line":"              - ^releasenotes/.*$"},{"line_number":21,"context_line":"              - ^tox.ini$"},{"line_number":22,"context_line":"            required-projects:"},{"line_number":23,"context_line":"              - openstack/ovsdbapp"},{"line_number":24,"context_line":"            vars:"},{"line_number":25,"context_line":"              zuul_work_dir: src/opendev.org/openstack/neutron"},{"line_number":26,"context_line":"              tox_envlist: dsvm-functional"}],"source_content_type":"text/x-yaml","patch_set":8,"id":"d842f6d5_e9198e20","line":23,"range":{"start_line":22,"start_character":0,"end_line":23,"end_character":34},"in_reply_to":"2e2f397c_f15e0760","updated":"2026-04-28 12:19:46.000000000","message":"thanks!","commit_id":"e5fe03382e59c8b1df0442f4432d1b2bf22bb53d"}]}