)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"0191bca0e486acd7f524d89367789ce2bd2a393a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"c6483958_88e7dd7d","updated":"2025-12-07 18:37:45.000000000","message":"What\u0027s crazy is the unit test doesn\u0027t catch this, I tried printing both ways and it works 😞","commit_id":"ef6c09736887ee26c2f130333cb91c83fd4dae81"},{"author":{"_account_id":33341,"name":"Maor Blaustein","email":"mblue@redhat.com","username":"blue"},"change_message_id":"eff024ea5a37b92a8730fc58b228ce49175eae09","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"e354178d_4f638524","in_reply_to":"c6483958_88e7dd7d","updated":"2025-12-07 21:51:02.000000000","message":"Thank you for checking that!\n\nI did look initially if can add a check on related tests [1], but to my understanding it seems to already test what is needed.\nWhen looked more carefully now, it seems FakeOvsdbRow also sets the keys as attributes in parent class, iiuc that is meant to test db_find_rows correctly where objects allow attribute access, so I will try to avoid that when testing db_find by using dict directly in test\u0027s mocks instead.\n\nTesting if it would fail here [2], fingers crossed, if so then I\u0027ll add similar change here.\n\n[1]\nhttps://review.opendev.org/c/openstack/neutron/+/940445/48/neutron/tests/unit/plugins/ml2/drivers/ovn/mech_driver/ovsdb/test_maintenance.py\n[2]\n970092: [DNM] Test dict instead of FakeOvsdbRow | https://review.opendev.org/c/openstack/neutron/+/970092","commit_id":"ef6c09736887ee26c2f130333cb91c83fd4dae81"},{"author":{"_account_id":33341,"name":"Maor Blaustein","email":"mblue@redhat.com","username":"blue"},"change_message_id":"95bd9ae4a7c95d71b62d64e52a2a4e16cdf4381b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"ff87ceb4_40a7fe02","in_reply_to":"e354178d_4f638524","updated":"2025-12-07 22:40:50.000000000","message":"It catches the issue in LP :) \nhttps://zuul.opendev.org/t/openstack/build/3d43e0f64b3349cd9970ed5bc52200ee","commit_id":"ef6c09736887ee26c2f130333cb91c83fd4dae81"},{"author":{"_account_id":33341,"name":"Maor Blaustein","email":"mblue@redhat.com","username":"blue"},"change_message_id":"8d7eed8391eb791d40448fc474782d20b985b8a8","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"6b169d45_ec2029cf","updated":"2025-12-08 12:20:16.000000000","message":"recheck docs gate unrelated failure","commit_id":"a991dd1d6b0d38dd34a2f2a01cc191e496f32bb5"},{"author":{"_account_id":33341,"name":"Maor Blaustein","email":"mblue@redhat.com","username":"blue"},"change_message_id":"5431d3c483fbbc2edb7b35d0d50c2800d59c27ae","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"80b2e832_209cd7ec","updated":"2025-12-08 07:35:01.000000000","message":"recheck pep8 post failure no logs","commit_id":"a991dd1d6b0d38dd34a2f2a01cc191e496f32bb5"},{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"fee85c2b78b3b08e2b27cba1bc5af921449b7931","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"66d52bfa_e857f4c3","updated":"2025-12-08 09:55:27.000000000","message":"thanks, good catch","commit_id":"a991dd1d6b0d38dd34a2f2a01cc191e496f32bb5"},{"author":{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},"change_message_id":"c075b35d52883fc780d8019973f666ce6df4bce4","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"42ea7ae3_961101ca","updated":"2025-12-08 10:20:05.000000000","message":"would also need to backported to 2025.2","commit_id":"a991dd1d6b0d38dd34a2f2a01cc191e496f32bb5"},{"author":{"_account_id":33341,"name":"Maor Blaustein","email":"mblue@redhat.com","username":"blue"},"change_message_id":"669ce6a097c1a8b1e31652be4785605fdd988ba8","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"9df6b0f1_743047ac","in_reply_to":"42ea7ae3_961101ca","updated":"2025-12-08 10:41:17.000000000","message":"thanks, made backport patch:\n970110: Fix maintenance LRP prefix adding fails over \u0027name\u0027 attribute missing | https://review.opendev.org/c/openstack/neutron/+/970110","commit_id":"a991dd1d6b0d38dd34a2f2a01cc191e496f32bb5"}]}