)]}'
{"id":"openstack%2Fneutron~994875","triplet_id":"openstack%2Fneutron~master~I769b64f5f4303e54568e766455d43364d77f498d","project":"openstack/neutron","branch":"master","topic":"fix_call_read_initial_chassis_conf","attention_set":{"16688":{"account":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"last_update":"2026-06-28 17:31:35.000000000","reason":"\u003cGERRIT_ACCOUNT_33341\u003e replied on the change","reason_account":{"_account_id":33341,"name":"Maor Blaustein","email":"mblue@redhat.com","username":"blue"}}},"removed_from_attention_set":{"33341":{"account":{"_account_id":33341,"name":"Maor Blaustein","email":"mblue@redhat.com","username":"blue"},"last_update":"2026-06-28 17:31:35.000000000","reason":"\u003cGERRIT_ACCOUNT_33341\u003e replied on the change","reason_account":{"_account_id":33341,"name":"Maor Blaustein","email":"mblue@redhat.com","username":"blue"}}},"hashtags":[],"change_id":"I769b64f5f4303e54568e766455d43364d77f498d","subject":"[OVN] Sync chassis placement config on SB DB startup","status":"NEW","created":"2026-06-25 10:39:54.000000000","updated":"2026-06-28 17:42:44.000000000","submit_type":"MERGE_IF_NECESSARY","mergeable":true,"submittable":false,"total_comment_count":3,"unresolved_comment_count":1,"has_review_started":true,"meta_rev_id":"0109102a8800a60d80e2018376c1eb701ad0c81b","_number":994875,"virtual_id_number":994875,"owner":{"_account_id":33341,"name":"Maor Blaustein","email":"mblue@redhat.com","username":"blue"},"actions":{},"labels":{"Verified":{"recommended":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"all":[{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},{"tag":"autogenerated:zuul:check","value":1,"date":"2026-06-25 12:39:38.000000000","permitted_voting_range":{"min":-2,"max":2},"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}],"values":{"-2":"Fails","-1":"Doesn\u0027t seem to work"," 0":"No score","+1":"Works for me","+2":"Verified"},"description":"","value":1,"default_value":0,"optional":true},"Code-Review":{"disliked":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"all":[{"value":-1,"date":"2026-06-26 09:29:54.000000000","permitted_voting_range":{"min":-2,"max":2},"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}],"values":{"-2":"Do not merge","-1":"This patch needs further work before it can be merged"," 0":"No score","+1":"Looks good to me, but someone else must approve","+2":"Looks good to me (core reviewer)"},"description":"","value":-1,"default_value":0,"optional":true},"Workflow":{"all":[{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true},"Review-Priority":{"all":[{"value":0,"permitted_voting_range":{"min":-1,"max":2},"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}],"values":{"-1":"Branch Freeze"," 0":"No Priority","+1":"Important Change","+2":"Gate Blocker Fix / Urgent Change"},"description":"","default_value":0,"optional":true}},"removable_reviewers":[],"reviewers":{"REVIEWER":[{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2026-06-25 11:41:59.000000000","updated_by":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"reviewer":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"state":"CC"},{"updated":"2026-06-25 12:39:38.000000000","updated_by":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"reviewer":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"state":"REVIEWER"},{"updated":"2026-06-25 13:09:48.000000000","updated_by":{"_account_id":33341,"name":"Maor Blaustein","email":"mblue@redhat.com","username":"blue"},"reviewer":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"state":"REVIEWER"}],"messages":[{"id":"82452c9dd605e7487753577653c4d0c38323c25d","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":33341,"name":"Maor Blaustein","email":"mblue@redhat.com","username":"blue"},"date":"2026-06-25 10:39:54.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"465304aab7bae8d179c5ae4608bb3ad469696747","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":33341,"name":"Maor Blaustein","email":"mblue@redhat.com","username":"blue"},"date":"2026-06-25 10:55:49.000000000","message":"Uploaded patch set 2.","accounts_in_message":[],"_revision_number":2},{"id":"6c8b0d63fa7daba97590da5ba290c3e7474a345e","tag":"autogenerated:zuul:check-arm64","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2026-06-25 11:41:59.000000000","message":"Patch Set 2:\n\nBuild succeeded (ARM64 pipeline).\nhttps://zuul.opendev.org/t/openstack/buildset/927b1ddc63104d2c94c73b4e432e0e34\n\n- openstack-tox-py311-arm64 https://zuul.opendev.org/t/openstack/build/bafcf55a0de946c583c168de6dd543b2 : SUCCESS in 25m 57s (non-voting)\n- openstack-tox-py313-arm64 https://zuul.opendev.org/t/openstack/build/59960dafa111460b8c1fa2156711d420 : SUCCESS in 17m 07s (non-voting)\n- openstack-tox-py314-arm64 https://zuul.opendev.org/t/openstack/build/8ff8faeeead246f597abd57cb32e345f : SUCCESS in 44m 07s (non-voting)","accounts_in_message":[],"_revision_number":2},{"id":"950b85df99fff715209f128021917386d6c95374","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2026-06-25 12:39:38.000000000","message":"Patch Set 2: Verified+1\n\nBuild succeeded (check pipeline).\nhttps://zuul.opendev.org/t/openstack/buildset/fcc0c59ac846448ca419373dede0ff80\n\n- tempest-integrated-networking https://zuul.opendev.org/t/openstack/build/da0fe7d7e6fe4591bc06eecb2e602b61 : SUCCESS in 1h 35m 40s\n- openstacksdk-functional-devstack https://zuul.opendev.org/t/openstack/build/b059181dd9e64ac986e20aabb20f6df2 : SUCCESS in 1h 04m 55s\n- neutron-tempest-plugin-ovn https://zuul.opendev.org/t/openstack/build/8fe06c0c75e64793bcf2ff5cdd5a08ef : SUCCESS in 1h 42m 39s\n- openstack-tox-cover https://zuul.opendev.org/t/openstack/build/0f6a1e7a53b64cd18b13adc57c1a9f1e : SUCCESS in 39m 38s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/1ed65aa3d3214638a1d2cdd6a04fc237 : SUCCESS in 11m 09s\n- openstack-tox-py311 https://zuul.opendev.org/t/openstack/build/a37fb24799d44773bf9fcb1a15b25d0c : SUCCESS in 28m 36s\n- openstack-tox-py313 https://zuul.opendev.org/t/openstack/build/5098f545d09d48bba258128b1a6315e6 : SUCCESS in 16m 10s\n- openstack-tox-py314 https://zuul.opendev.org/t/openstack/build/8b3e835a52234e42a768432354d962ac : SUCCESS in 25m 42s (non-voting)\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/8d02af8665ff458c993aa17a2c7f1677 : SUCCESS in 12m 16s\n- build-openstack-releasenotes https://zuul.opendev.org/t/openstack/build/0057356e9470450e9632ab0280472d31 : SUCCESS in 6m 54s\n- neutron-functional https://zuul.opendev.org/t/openstack/build/1aa9d353dc464091ae991ab8e687c8d0 : SUCCESS in 1h 31m 43s\n- neutron-ovn-grenade-multinode https://zuul.opendev.org/t/openstack/build/a0e0703965a64a4187f03975d265c11e : SUCCESS in 58m 33s\n- neutron-ovn-tempest-ipv6-only-ovs-release https://zuul.opendev.org/t/openstack/build/f2000a1eb0ce47538d0eade252842c8d : SUCCESS in 57m 19s\n- neutron-ovn-rally-task https://zuul.opendev.org/t/openstack/build/d1882b3ceefa4d8d98d4d2bab1885b3c : SUCCESS in 1h 29m 10s\n\nWarning:\n  Comments left for invalid file neutron/agent/l3/router_info.py","accounts_in_message":[],"_revision_number":2},{"id":"1fe16bae410005dd74f404875849b557c698a9ca","author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"date":"2026-06-26 09:29:54.000000000","message":"Patch Set 2: Code-Review-1\n\n(1 comment)","accounts_in_message":[],"_revision_number":2},{"id":"52900fefcbfd5840464631cd912adce005eb2073","author":{"_account_id":33341,"name":"Maor Blaustein","email":"mblue@redhat.com","username":"blue"},"date":"2026-06-28 17:31:35.000000000","message":"Patch Set 2:\n\n(1 comment)","accounts_in_message":[],"_revision_number":2},{"id":"0109102a8800a60d80e2018376c1eb701ad0c81b","author":{"_account_id":33341,"name":"Maor Blaustein","email":"mblue@redhat.com","username":"blue"},"date":"2026-06-28 17:42:44.000000000","message":"Patch Set 2:\n\n(1 comment)","accounts_in_message":[],"_revision_number":2}],"current_revision_number":2,"current_revision":"46afbff1888768eea2583a8d3cdf04953cc535b9","revisions":{"59593201c65fa013d45eda5c22d4c21c84348792":{"kind":"REWORK","_number":1,"created":"2026-06-25 10:39:54.000000000","uploader":{"_account_id":33341,"name":"Maor Blaustein","email":"mblue@redhat.com","username":"blue"},"ref":"refs/changes/75/994875/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/neutron","ref":"refs/changes/75/994875/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/neutron refs/changes/75/994875/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/neutron refs/changes/75/994875/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/neutron refs/changes/75/994875/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/neutron refs/changes/75/994875/1"}}},"commit":{"parents":[{"commit":"1cab59daa0aee66d23150f7d1f92c82e0a960ab6","subject":"Merge \"Add configurable HAProxy timeouts for metadata proxy\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/neutron/commit/1cab59daa0aee66d23150f7d1f92c82e0a960ab6"}]}],"author":{"name":"Maor Blaustein","email":"mblue@redhat.com","date":"2026-06-25 10:33:27.000000000","tz":180},"committer":{"name":"Maor Blaustein","email":"mblue@redhat.com","date":"2026-06-25 10:39:05.000000000","tz":180},"subject":"[OVN] Sync chassis placement config on SB DB startup","message":"[OVN] Sync chassis placement config on SB DB startup\n\nThe OVNClientPlacementExtension.read_initial_chassis_config() method\nwas designed to sync pre-existing chassis bandwidth configuration to\nthe Placement API during neutron-server startup. However, this method\nwas never called from production code, only from unit tests.\n\nWhen OVN controller registers a chassis in the SB DB before\nneutron-server starts, the OVSDB IDL initial sync does not generate\nROW_CREATE events for pre-existing rows. The ChassisBandwidthConfigEvent\nhandler only processes ROW_CREATE and ROW_UPDATE events, so chassis\nthat already exist with resource_provider_bandwidths in their CMS\noptions are silently ignored. This results in no bandwidth resource\nproviders being created in the Placement API.\n\nThis change adds a call to read_initial_chassis_config() in the\nOvnSbSynchronizer.do_sync() method.\n\nA unit test is added to verify all SB sync methods including the\nnew placement sync are called during do_sync().\n\nCloses-Bug: #2158246\nAssisted-By: Claude Opus 4.6\nChange-Id: I769b64f5f4303e54568e766455d43364d77f498d\nSigned-off-by: Maor Blaustein \u003cmblue@redhat.com\u003e\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/neutron/commit/59593201c65fa013d45eda5c22d4c21c84348792"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/neutron/commit/59593201c65fa013d45eda5c22d4c21c84348792"}]},"branch":"refs/heads/master"},"46afbff1888768eea2583a8d3cdf04953cc535b9":{"kind":"REWORK","_number":2,"created":"2026-06-25 10:55:49.000000000","uploader":{"_account_id":33341,"name":"Maor Blaustein","email":"mblue@redhat.com","username":"blue"},"ref":"refs/changes/75/994875/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/neutron","ref":"refs/changes/75/994875/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/neutron refs/changes/75/994875/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/neutron refs/changes/75/994875/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/neutron refs/changes/75/994875/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/neutron refs/changes/75/994875/2"}}},"commit":{"parents":[{"commit":"1cab59daa0aee66d23150f7d1f92c82e0a960ab6","subject":"Merge \"Add configurable HAProxy timeouts for metadata proxy\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/neutron/commit/1cab59daa0aee66d23150f7d1f92c82e0a960ab6"}]}],"author":{"name":"Maor Blaustein","email":"mblue@redhat.com","date":"2026-06-25 10:33:27.000000000","tz":180},"committer":{"name":"Maor Blaustein","email":"mblue@redhat.com","date":"2026-06-25 10:55:41.000000000","tz":180},"subject":"[OVN] Sync chassis placement config on SB DB startup","message":"[OVN] Sync chassis placement config on SB DB startup\n\nThe OVNClientPlacementExtension.read_initial_chassis_config() method\nwas designed to sync pre-existing chassis bandwidth configuration to\nthe Placement API during neutron-server startup. However, this method\nwas never called from production code, only from unit tests.\n\nWhen OVN controller registers a chassis in the SB DB before\nneutron-server starts, the OVSDB IDL initial sync does not generate\nROW_CREATE events for pre-existing rows. The ChassisBandwidthConfigEvent\nhandler only processes ROW_CREATE and ROW_UPDATE events, so chassis\nthat already exist with resource_provider_bandwidths in their CMS\noptions are silently ignored. This results in no bandwidth resource\nproviders being created in the Placement API.\n\nThis change adds a call to read_initial_chassis_config() in the\nOvnSbSynchronizer.do_sync() method.\n\nA unit test is added to verify all SB sync methods including the\nnew placement sync are called during do_sync().\n\nCloses-Bug: #2158246\nAssisted-By: Claude Opus 4.6\nChange-Id: I769b64f5f4303e54568e766455d43364d77f498d\nSigned-off-by: Maor Blaustein \u003cmblue@redhat.com\u003e\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/neutron/commit/46afbff1888768eea2583a8d3cdf04953cc535b9"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/neutron/commit/46afbff1888768eea2583a8d3cdf04953cc535b9"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[{"rule_name":"gerrit~DefaultSubmitRule","status":"OK","labels":[{"label":"Verified","status":"MAY","applied_by":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}},{"label":"Code-Review","status":"MAY","applied_by":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"}},{"label":"Workflow","status":"MAY"},{"label":"Review-Priority","status":"MAY"}]}],"submit_requirements":[{"name":"Verified","description":"Verified in gate by CI","status":"UNSATISFIED","is_legacy":false,"submittability_expression_result":{"expression":"label:Verified\u003dMAX AND -label:Verified\u003dMIN","fulfilled":false,"status":"FAIL","passing_atoms":[],"failing_atoms":["label:Verified\u003dMAX","label:Verified\u003dMIN"],"atom_explanations":{"label:Verified\u003dMAX":"","label:Verified\u003dMIN":""}}},{"name":"Code-Review","description":"Code reviewed by core reviewer","status":"UNSATISFIED","is_legacy":false,"submittability_expression_result":{"expression":"label:Code-Review\u003dMAX AND -label:Code-Review\u003dMIN","fulfilled":false,"status":"FAIL","passing_atoms":[],"failing_atoms":["label:Code-Review\u003dMAX","label:Code-Review\u003dMIN"],"atom_explanations":{"label:Code-Review\u003dMAX":"","label:Code-Review\u003dMIN":""}}},{"name":"Review-Priority","description":"Review priority","status":"SATISFIED","is_legacy":false,"submittability_expression_result":{"expression":"-label:Review-Priority\u003dMIN","fulfilled":true,"status":"PASS","passing_atoms":[],"failing_atoms":["label:Review-Priority\u003dMIN"],"atom_explanations":{"label:Review-Priority\u003dMIN":""}}},{"name":"Workflow","description":"Approved for gate by core reviewer","status":"UNSATISFIED","is_legacy":false,"submittability_expression_result":{"expression":"label:Workflow\u003dMAX AND -label:Workflow\u003dMIN","fulfilled":false,"status":"FAIL","passing_atoms":[],"failing_atoms":["label:Workflow\u003dMAX","label:Workflow\u003dMIN"],"atom_explanations":{"label:Workflow\u003dMAX":"","label:Workflow\u003dMIN":""}}}]}