)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"904d29d4da62cd2832df154776bc62f2f4fc098c","unresolved":true,"context_lines":[{"line_number":10,"context_line":"even when nothing happens.  This fills up logs and makes it hard to find"},{"line_number":11,"context_line":"actual things happening."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"This was likely intended to be a debug output, so let\u0027s move this to DEBUG."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"Trivialfix"},{"line_number":16,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"501b7ef2_21b2cf58","line":13,"updated":"2025-08-25 12:06:13.000000000","message":"this patch introduced these info logs: https://review.opendev.org/c/openstack/neutron/+/881645\n\nI think the idea was that the ovs-agent for example also log on info level in every iteration.","commit_id":"1d726278d5d534d2014cae9f102f84883a582eea"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"349fadf91673ac3ae70deda7557d6b2766b9ddca","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"20cc6f8c_308ca909","updated":"2025-08-22 14:23:35.000000000","message":"Can you give a little more info on which agent you\u0027re seeing this with and an example output?\n\nI did add this code at LOG.info, probably because a lot of the OVS agent code uses the same level. Although I do see now the sriov agent (which is similar to this) uses LOG.debug.\n\nI just want to make sure we\u0027re not changing something that other users find useful.","commit_id":"1d726278d5d534d2014cae9f102f84883a582eea"},{"author":{"_account_id":34936,"name":"Baptiste Jonglez","email":"baptiste.jonglez@inria.fr","username":"bjonglez"},"change_message_id":"0e798346ecd5428e1adfa0759c7bb3890b397e4d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"7f6a25a8_69c2ed06","in_reply_to":"20cc6f8c_308ca909","updated":"2025-08-22 15:16:40.000000000","message":"This is with the linuxbridge agent (I do realize it was removed upstream recently, but we are using it currently because we had issues with OVS)\n\nHere is an example output, showing a loop every two seconds:\n\nAug 22 10:25:45 kavlan-agent-devel neutron-linuxbridge-agent[481]: 2025-08-22 10:25:45,455 - neutron.plugins.ml2.drivers.agent._common_agent - INFO - Linux bridge agent Agent loop - iteration:160226 started\nAug 22 10:25:47 kavlan-agent-devel neutron-linuxbridge-agent[481]: 2025-08-22 10:25:47,455 - neutron.plugins.ml2.drivers.agent._common_agent - INFO - Linux bridge agent Agent loop - iteration:160226 completed\nAug 22 10:25:47 kavlan-agent-devel neutron-linuxbridge-agent[481]: 2025-08-22 10:25:47,456 - neutron.plugins.ml2.drivers.agent._common_agent - INFO - Linux bridge agent Agent loop - iteration:160227 started\nAug 22 10:25:49 kavlan-agent-devel neutron-linuxbridge-agent[481]: 2025-08-22 10:25:49,459 - neutron.plugins.ml2.drivers.agent._common_agent - INFO - Linux bridge agent Agent loop - iteration:160227 completed\nAug 22 10:25:49 kavlan-agent-devel neutron-linuxbridge-agent[481]: 2025-08-22 10:25:49,460 - neutron.plugins.ml2.drivers.agent._common_agent - INFO - Linux bridge agent Agent loop - iteration:160228 started\nAug 22 10:25:51 kavlan-agent-devel neutron-linuxbridge-agent[481]: 2025-08-22 10:25:51,461 - neutron.plugins.ml2.drivers.agent._common_agent - INFO - Linux bridge agent Agent loop - iteration:160228 completed","commit_id":"1d726278d5d534d2014cae9f102f84883a582eea"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"6891df0d705782ca3bbece13af86e09352f284de","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"bf711b3c_06f355bd","in_reply_to":"7f6a25a8_69c2ed06","updated":"2025-08-22 18:26:29.000000000","message":"Thanks for the info, it helps scope the change a little. I\u0027ll add some more background.\n\n1) In https://review.opendev.org/c/openstack/neutron/+/638641 we changed to use LOG.info for the OVS agent since it seemed useful for debugging from an operator perspective. That code defines it\u0027s own daemon_loop() code though.\n\n2) The SRIOV agent uses LOG.debug.\n\n3) This common_agent code (CommonAgentLoop) was only use by two agents - Macvtap and Linuxbridge (which has been removed). So changing things here would only affect them.\n\nI\u0027ll add Liu since he made the OVS agent change and might have an opinion.\n\nSince you\u0027re using Linuxbridge, how far would you need this backported if it merged? Since if we\u0027re going to backport it we should open a bug to track things.","commit_id":"1d726278d5d534d2014cae9f102f84883a582eea"},{"author":{"_account_id":34936,"name":"Baptiste Jonglez","email":"baptiste.jonglez@inria.fr","username":"bjonglez"},"change_message_id":"60ccd2d97469893fb2cf4e13c1db2a69619fa34b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"6cd55a7e_ab46454a","in_reply_to":"bf711b3c_06f355bd","updated":"2025-08-25 08:16:52.000000000","message":"Thanks for the context. We are currently using 2024.1 Caracal.","commit_id":"1d726278d5d534d2014cae9f102f84883a582eea"}]}
