)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":32586,"name":"Elvira García Ruiz","display_name":"Elvira","email":"egarciar@redhat.com","username":"elvira"},"change_message_id":"f4fb76be204452502e925d6405659e34dbeb4be0","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"e79384b6_dc4a0b97","updated":"2025-04-24 11:56:43.000000000","message":"Not sure how to test this patch. I tried following the steps (taken from ykarel comment on 2107925) and the pg_drop ACL is still being deleted after applying this patch and restarting the service. Is there something I\u0027m missing?\n\nSteps:\n1. sudo ovn-nbctl find ACL direction\u003dto-lport action\u003ddrop\n2. openstack network log create --resource-type security_group --event DROP logmedrop\n3. sudo ovn-nbctl find ACL direction\u003dto-lport action\u003ddrop # Once network log created default port group ACL get\u0027s updated\n4. /opt/stack/data/venv/bin/neutron-ovn-db-sync-util --config-file /etc/neutron/neutron.conf --config-file /etc/neutron/plugins/ml2/ml2_conf.ini --ovn-neutron_sync_mode repair # First repair removes the above ACL","commit_id":"b39d27b31fc7930b415be62af93838c11bea30af"},{"author":{"_account_id":32586,"name":"Elvira García Ruiz","display_name":"Elvira","email":"egarciar@redhat.com","username":"elvira"},"change_message_id":"e0a1bc752b2d2c5b5ea0e3d5e469858525c9647d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"c0a07d2f_4cb520f9","updated":"2025-05-05 13:52:45.000000000","message":"-1\u0027d for the same reason as the other patch. See comment on: https://review.opendev.org/c/openstack/neutron/+/948215/4","commit_id":"fc2e611575919922fb916b12c93472cbe310adf6"},{"author":{"_account_id":1004,"name":"Mohammed Naser","email":"mnaser@vexxhost.com","username":"mnaser"},"change_message_id":"ae7acca7d1efa84fca93472c61294ea9b4f0daaa","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"15dc3ed1_6a86296e","updated":"2025-04-30 19:14:38.000000000","message":"recheck\n\n\u003e E: Failed to fetch http://archive.ubuntu.com/ubuntu/dists/focal-updates/main/binary-amd64/Packages.xz  File has unexpected size (3897232 !\u003d 3897184). Mirror sync in progress? [IP: 91.189.91.82 80]","commit_id":"fc2e611575919922fb916b12c93472cbe310adf6"},{"author":{"_account_id":1004,"name":"Mohammed Naser","email":"mnaser@vexxhost.com","username":"mnaser"},"change_message_id":"022818c4958b18d63fdd6ac7d007d75921a6a792","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"6a377e25_a3e04f4d","updated":"2025-05-15 02:24:22.000000000","message":"recheck","commit_id":"afa1aca17f25e8d1c06021b83fc05616d5db816f"},{"author":{"_account_id":12404,"name":"Rico Lin","email":"ricolin@ricolky.com","username":"rico.lin"},"change_message_id":"d2545281016d87720f7229307074062da2de911f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":13,"id":"f91196de_31b4c805","updated":"2025-05-19 00:16:02.000000000","message":"recheck","commit_id":"d00fc48ca43266ca60137ab037bde46b33ae2270"}],"neutron/plugins/ml2/drivers/ovn/mech_driver/ovsdb/ovn_db_sync.py":[{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"02bdd84fabf9f1dd173dac104dcd082809a067e9","unresolved":true,"context_lines":[{"line_number":359,"context_line":"                            log_attrs[\u0027label\u0027] \u003d oa_label"},{"line_number":360,"context_line":"                        else:"},{"line_number":361,"context_line":"                            log_attrs[\u0027label\u0027] \u003d secrets.SystemRandom("},{"line_number":362,"context_line":"                            ).randrange(1, log_driver.MAX_INT_LABEL)"},{"line_number":363,"context_line":"                        log_attrs[\u0027options\u0027] \u003d acl_options"},{"line_number":364,"context_line":""},{"line_number":365,"context_line":"                    # Update neutron acl with log related attributes"}],"source_content_type":"text/x-python","patch_set":1,"id":"6e630775_47f113ce","line":362,"updated":"2025-04-24 16:38:11.000000000","message":"I don\u0027t see the secrets library used any place else in neutron. Is it possible to use random.randint() or some equivalent from the random library?","commit_id":"b39d27b31fc7930b415be62af93838c11bea30af"},{"author":{"_account_id":12404,"name":"Rico Lin","email":"ricolin@ricolky.com","username":"rico.lin"},"change_message_id":"18948b4c9ef789eaed3752fb4b5392350f76d121","unresolved":false,"context_lines":[{"line_number":359,"context_line":"                            log_attrs[\u0027label\u0027] \u003d oa_label"},{"line_number":360,"context_line":"                        else:"},{"line_number":361,"context_line":"                            log_attrs[\u0027label\u0027] \u003d secrets.SystemRandom("},{"line_number":362,"context_line":"                            ).randrange(1, log_driver.MAX_INT_LABEL)"},{"line_number":363,"context_line":"                        log_attrs[\u0027options\u0027] \u003d acl_options"},{"line_number":364,"context_line":""},{"line_number":365,"context_line":"                    # Update neutron acl with log related attributes"}],"source_content_type":"text/x-python","patch_set":1,"id":"f0f52041_dd9892cf","line":362,"in_reply_to":"6e630775_47f113ce","updated":"2025-04-25 01:03:31.000000000","message":"We get this constraints from https://github.com/openstack/neutron/blob/master/neutron/services/logapi/drivers/ovn/driver.py#L186","commit_id":"b39d27b31fc7930b415be62af93838c11bea30af"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"02bdd84fabf9f1dd173dac104dcd082809a067e9","unresolved":true,"context_lines":[{"line_number":436,"context_line":"                            log_attrs[\u0027label\u0027] \u003d oa_label"},{"line_number":437,"context_line":"                        else:"},{"line_number":438,"context_line":"                            log_attrs[\u0027label\u0027] \u003d secrets.SystemRandom("},{"line_number":439,"context_line":"                            ).randrange(1, log_driver.MAX_INT_LABEL)"},{"line_number":440,"context_line":"                        log_attrs[\u0027options\u0027] \u003d acl_options"},{"line_number":441,"context_line":"                    # Update neutron acl with log related attributes"},{"line_number":442,"context_line":"                    na.update(log_attrs)"}],"source_content_type":"text/x-python","patch_set":1,"id":"d087adfc_eada1748","line":439,"updated":"2025-04-24 16:38:11.000000000","message":"Same comment","commit_id":"b39d27b31fc7930b415be62af93838c11bea30af"},{"author":{"_account_id":12404,"name":"Rico Lin","email":"ricolin@ricolky.com","username":"rico.lin"},"change_message_id":"18948b4c9ef789eaed3752fb4b5392350f76d121","unresolved":false,"context_lines":[{"line_number":436,"context_line":"                            log_attrs[\u0027label\u0027] \u003d oa_label"},{"line_number":437,"context_line":"                        else:"},{"line_number":438,"context_line":"                            log_attrs[\u0027label\u0027] \u003d secrets.SystemRandom("},{"line_number":439,"context_line":"                            ).randrange(1, log_driver.MAX_INT_LABEL)"},{"line_number":440,"context_line":"                        log_attrs[\u0027options\u0027] \u003d acl_options"},{"line_number":441,"context_line":"                    # Update neutron acl with log related attributes"},{"line_number":442,"context_line":"                    na.update(log_attrs)"}],"source_content_type":"text/x-python","patch_set":1,"id":"1f817f5b_18344f92","line":439,"in_reply_to":"d087adfc_eada1748","updated":"2025-04-25 01:03:31.000000000","message":"same","commit_id":"b39d27b31fc7930b415be62af93838c11bea30af"}],"releasenotes/notes/fix-network-log-with-ovn-db-sync-2fe57474079c40a1.yaml":[{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"02bdd84fabf9f1dd173dac104dcd082809a067e9","unresolved":true,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"fixes:"},{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    Fix running `neutron-ovn-db-sync-util` when network log resources exists."},{"line_number":5,"context_line":"    previously `neutron-ovn-db-sync-util` command workflow didn\u0027t consider"},{"line_number":6,"context_line":"    log resoruces. That cause acls related to `neutron_pg_drop` got deleted"},{"line_number":7,"context_line":"    during sync acls. The issue is now fixed."}],"source_content_type":"text/x-yaml","patch_set":1,"id":"5794ddc4_61a5df02","line":7,"range":{"start_line":4,"start_character":4,"end_line":7,"end_character":45},"updated":"2025-04-24 16:38:11.000000000","message":"There are a couple of typos here, I did a quick re-write adding the bug url to it:\n\nFix running ``neutron-ovn-db-sync-util`` when OVN network log resources exist.\nThe command workflow did not consider log resources, which caused ACLs related to\nthe ``neutron_pg_drop`` rule to get deleted during sync. For more information see\nbug `2107925 \u003chttps://bugs.launchpad.net/neutron/+bug/2107925\u003e`_.","commit_id":"b39d27b31fc7930b415be62af93838c11bea30af"},{"author":{"_account_id":12404,"name":"Rico Lin","email":"ricolin@ricolky.com","username":"rico.lin"},"change_message_id":"18948b4c9ef789eaed3752fb4b5392350f76d121","unresolved":false,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"fixes:"},{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    Fix running `neutron-ovn-db-sync-util` when network log resources exists."},{"line_number":5,"context_line":"    previously `neutron-ovn-db-sync-util` command workflow didn\u0027t consider"},{"line_number":6,"context_line":"    log resoruces. That cause acls related to `neutron_pg_drop` got deleted"},{"line_number":7,"context_line":"    during sync acls. The issue is now fixed."}],"source_content_type":"text/x-yaml","patch_set":1,"id":"157189fc_7ac36f1c","line":7,"range":{"start_line":4,"start_character":4,"end_line":7,"end_character":45},"in_reply_to":"5794ddc4_61a5df02","updated":"2025-04-25 01:03:31.000000000","message":"Done","commit_id":"b39d27b31fc7930b415be62af93838c11bea30af"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"eda95ed7fc2500413567d6c3d069ad33b785febf","unresolved":true,"context_lines":[{"line_number":4,"context_line":"    Fix running neutron-ovn-db-sync-util when OVN network log resources exist."},{"line_number":5,"context_line":"    The command workflow did not consider log resources, which caused ACLs related to"},{"line_number":6,"context_line":"    the neutron_pg_drop rule to get deleted during sync. For more information see"},{"line_number":7,"context_line":"    bug 2107925 \u003chttps://bugs.launchpad.net/neutron/+bug/2107925\u003e_."}],"source_content_type":"text/x-yaml","patch_set":4,"id":"30be0ae9_382b68da","line":7,"updated":"2025-04-29 15:36:16.000000000","message":"I think there was a character or two lost in my cut/paste. Can you change this last line to be:\n\nbug `2107925 \u003chttps://bugs.launchpad.net/neutron/+bug/2107925\u003e`_.","commit_id":"53babeebddb41bb0a69b16204e3ffb17af6b8f2d"},{"author":{"_account_id":12404,"name":"Rico Lin","email":"ricolin@ricolky.com","username":"rico.lin"},"change_message_id":"ed1626c7fdec238cca232fa860507d235c2c90d2","unresolved":false,"context_lines":[{"line_number":4,"context_line":"    Fix running neutron-ovn-db-sync-util when OVN network log resources exist."},{"line_number":5,"context_line":"    The command workflow did not consider log resources, which caused ACLs related to"},{"line_number":6,"context_line":"    the neutron_pg_drop rule to get deleted during sync. For more information see"},{"line_number":7,"context_line":"    bug 2107925 \u003chttps://bugs.launchpad.net/neutron/+bug/2107925\u003e_."}],"source_content_type":"text/x-yaml","patch_set":4,"id":"ed7be8aa_ba483e7f","line":7,"in_reply_to":"30be0ae9_382b68da","updated":"2025-04-30 01:01:36.000000000","message":"Done","commit_id":"53babeebddb41bb0a69b16204e3ffb17af6b8f2d"}]}