)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":23567,"name":"Luis Tomas Bolivar","email":"ltomasbo@redhat.com","username":"ltomasbo"},"change_message_id":"7bc6ad01aea1c3d3c65f17bfb0aa61b65624923a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"ffe7252c_9b59488e","updated":"2022-12-01 06:54:20.000000000","message":"the patch looks good to me, though there is some tests failing","commit_id":"a49f4d9beb4871bd475398724dd3544df73832ae"},{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"21b762c4221302abea5909d1015402259186a3dd","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"f5436145_99560f33","updated":"2022-12-01 13:03:01.000000000","message":"recheck\nLet\u0027s see together with  https://review.opendev.org/c/openstack/neutron/+/866307","commit_id":"ae045c27bfb81e596152b4f54bbd79fc9325f928"},{"author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"change_message_id":"ee64d8dbaeff68716f364d9446eef41d5c9d6494","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"f2be4227_26fbf2db","updated":"2022-12-06 10:20:25.000000000","message":"Are You sure those tempest failures aren\u0027t related to the patch? In both jobs same test failed. Please check it before rechecking that patch again.","commit_id":"dae344c801899a86877ad19b87d40c3a6fbcac6e"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"338c0e5d919450666ac2cf8c1a592bff4d036bb0","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"021a0c4a_e2c348a4","updated":"2022-12-05 17:28:07.000000000","message":"recheck tempest","commit_id":"dae344c801899a86877ad19b87d40c3a6fbcac6e"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"a9272b30055da47cba37bf8c3dec30c2b8f2fbc8","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"7d149fec_d1ce458e","in_reply_to":"f2be4227_26fbf2db","updated":"2022-12-07 11:47:44.000000000","message":"Right, this is not the correct way of implementing this feature. This patch is preventing the compute node from loading the OVS trunk extension. But the problem is in the compute node configuration: the service_plugin list is empty (not defined):\nDec 05 17:53:35.903922 ubuntu-jammy-inmotion-iad3-0032415088 neutron-openvswitch-agent[26931]: DEBUG neutron.plugins.ml2.drivers.openvswitch.agent.ovs_neutron_agent [-] service_plugins                \u003d [] {{(pid\u003d26931) log_opt_values /usr/local/lib/python3.10/dist-packages/oslo_config/cfg.py:2602}}\n\n\nThe question here is if we should enforce the definition of the service_plugins in a compute node. This is the perfect example of what could happen in an environment if upgraded with this patch; if the compute node doesn\u0027t have the service_plugins list with \"trunk\" populated, the trunk extension in OVS won\u0027t be loaded.\n\nI\u0027ll raise this issue in the driver\u0027s meeting next time.","commit_id":"dae344c801899a86877ad19b87d40c3a6fbcac6e"}]}