)]}'
{"id":"openstack%2Fovsdbapp~995276","triplet_id":"openstack%2Fovsdbapp~master~Iedc5a1f4b3c2a8f0d9e7b6a5c4d3e2f1a0b9c8d7","project":"openstack/ovsdbapp","branch":"master","topic":"bug/2158253","attention_set":{},"removed_from_attention_set":{"16688":{"account":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"last_update":"2026-07-01 07:28:31.000000000","reason":"\u003cGERRIT_ACCOUNT_16688\u003e replied on the change","reason_account":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"}},"5756":{"account":{"_account_id":5756,"name":"Terry Wilson","email":"twilson@redhat.com","username":"otherwiseguy"},"last_update":"2026-07-01 07:28:34.000000000","reason":"Change was abandoned"}},"hashtags":[],"change_id":"Iedc5a1f4b3c2a8f0d9e7b6a5c4d3e2f1a0b9c8d7","subject":"Add ``hostname`` index to SB ``Chassis`` table","status":"ABANDONED","created":"2026-06-29 09:54:26.000000000","updated":"2026-07-01 07:28:34.000000000","total_comment_count":2,"unresolved_comment_count":1,"has_review_started":true,"meta_rev_id":"5ce804035bfa6c84d3b7d163bda4bd1bd91828f0","_number":995276,"virtual_id_number":995276,"owner":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"actions":{},"labels":{"Verified":{"recommended":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"all":[{"_account_id":5756,"name":"Terry Wilson","email":"twilson@redhat.com","username":"otherwiseguy"},{"tag":"autogenerated:zuul:check","value":1,"date":"2026-06-30 15:04:42.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":5756,"name":"Terry Wilson","email":"twilson@redhat.com","username":"otherwiseguy"},"all":[{"value":-1,"date":"2026-06-30 20:18:15.000000000","permitted_voting_range":{"min":-2,"max":2},"_account_id":5756,"name":"Terry Wilson","email":"twilson@redhat.com","username":"otherwiseguy"},{"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":5756,"name":"Terry Wilson","email":"twilson@redhat.com","username":"otherwiseguy"},{"_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":5756,"name":"Terry Wilson","email":"twilson@redhat.com","username":"otherwiseguy"},{"_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":5756,"name":"Terry Wilson","email":"twilson@redhat.com","username":"otherwiseguy"},{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}],"CC":[{"_account_id":8655,"name":"Jakub Libosvar","email":"libosvar@redhat.com","username":"jlibosva"},{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2026-06-29 09:54:46.000000000","updated_by":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"reviewer":{"_account_id":8655,"name":"Jakub Libosvar","email":"libosvar@redhat.com","username":"jlibosva"},"state":"CC"},{"updated":"2026-06-29 09:54:46.000000000","updated_by":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"reviewer":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"state":"CC"},{"updated":"2026-06-29 09:54:46.000000000","updated_by":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"reviewer":{"_account_id":5756,"name":"Terry Wilson","email":"twilson@redhat.com","username":"otherwiseguy"},"state":"CC"},{"updated":"2026-06-29 11:25:50.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-30 20:18:15.000000000","updated_by":{"_account_id":5756,"name":"Terry Wilson","email":"twilson@redhat.com","username":"otherwiseguy"},"reviewer":{"_account_id":5756,"name":"Terry Wilson","email":"twilson@redhat.com","username":"otherwiseguy"},"state":"REVIEWER"}],"messages":[{"id":"c3b8e5bcd185c7502a1ff0d66afd1e6cb47cf1e4","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"date":"2026-06-29 09:54:26.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"4c5120258b46aef25995b67362b3249b98d8f356","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2026-06-29 11:25:50.000000000","message":"Patch Set 1: Verified-1\n\nBuild failed (check pipeline).  For information on how to proceed, see\nhttps://docs.opendev.org/opendev/infra-manual/latest/developers.html#automated-testing\nand https://docs.openstack.org/project-team-guide/testing.html#how-to-handle-test-failures\n\nhttps://zuul.opendev.org/t/openstack/buildset/c3dad9408c2849a79dac04a0f804ea08\n\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/fb1f03f347d64a2382c670ec3abaee03 : SUCCESS in 3m 10s\n- openstack-tox-py311 https://zuul.opendev.org/t/openstack/build/99bd5e88400a432b8318565aaa2bba96 : SUCCESS in 2m 28s\n- openstack-tox-py313 https://zuul.opendev.org/t/openstack/build/98581e05f6f24b1f90b8e2e7c2297cfb : SUCCESS in 2m 56s\n- openstack-tox-py314 https://zuul.opendev.org/t/openstack/build/784972adae024d16a6a21c392d6d809b : SUCCESS in 5m 21s (non-voting)\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/9cddc68cba1740fd8f5a4efe2ef8be06 : SUCCESS in 4m 02s\n- openstack-tox-functional https://zuul.opendev.org/t/openstack/build/29d85755cf0645cf81eb91dd87484cb8 : FAILURE in 7m 25s\n- ovsdbapp-neutron-ovn-tempest-ovs-release https://zuul.opendev.org/t/openstack/build/c83b158c3f6e41bb98681d24d09d8564 : SUCCESS in 1h 25m 36s\n- neutron-functional https://zuul.opendev.org/t/openstack/build/8c2241eac061459fb08999fb28f0f47f : SUCCESS in 26m 15s","accounts_in_message":[],"_revision_number":1},{"id":"43c11a338db2f34711fef29bcf803f17b2d699a6","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"date":"2026-06-30 14:00:18.000000000","message":"Uploaded patch set 2.\n\nOutdated Votes:\n* Verified-1 (copy condition: \"NEVER\")\n","accounts_in_message":[],"_revision_number":2},{"id":"5ba665d7a986ba1a93b54a93870f9893b52a90e1","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2026-06-30 15:04:42.000000000","message":"Patch Set 2: Verified+1\n\nBuild succeeded (check pipeline).\nhttps://zuul.opendev.org/t/openstack/buildset/fff465f113c84925ae7a15a628ea6a14\n\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/5329878083a54ba2b548029da6c37231 : SUCCESS in 3m 47s\n- openstack-tox-py311 https://zuul.opendev.org/t/openstack/build/9e8bc9baee854f3e980da8c22d67b125 : SUCCESS in 3m 42s\n- openstack-tox-py313 https://zuul.opendev.org/t/openstack/build/2f0910486fea464f9f9431adf7c5b150 : SUCCESS in 2m 46s\n- openstack-tox-py314 https://zuul.opendev.org/t/openstack/build/bb9a7a6b5fcd45e78c1b178a3d35acc5 : SUCCESS in 6m 01s (non-voting)\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/2fd6c357b40642b0869c8b1bfefa154c : SUCCESS in 7m 07s\n- openstack-tox-functional https://zuul.opendev.org/t/openstack/build/381f3f42456e46e0a058464ed8203a15 : SUCCESS in 7m 15s\n- ovsdbapp-neutron-ovn-tempest-ovs-release https://zuul.opendev.org/t/openstack/build/c1d62a7630884946a0222bde36041dfd : SUCCESS in 1h 01m 47s\n- neutron-functional https://zuul.opendev.org/t/openstack/build/ad1ee2702dde487b908df7d9877c1ed2 : SUCCESS in 51m 48s","accounts_in_message":[],"_revision_number":2},{"id":"8ab92c12ab80e70bd8313fbca5dee1d0695e7727","author":{"_account_id":5756,"name":"Terry Wilson","email":"twilson@redhat.com","username":"otherwiseguy"},"date":"2026-06-30 20:18:15.000000000","message":"Patch Set 2: Code-Review-1\n\n(1 comment)","accounts_in_message":[],"_revision_number":2},{"id":"50ae398429063c9426088f97cd2042b367a2c3f9","author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"date":"2026-07-01 07:28:31.000000000","message":"Patch Set 2:\n\n(1 comment)","accounts_in_message":[],"_revision_number":2},{"id":"5ce804035bfa6c84d3b7d163bda4bd1bd91828f0","tag":"autogenerated:gerrit:abandon","author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"date":"2026-07-01 07:28:34.000000000","message":"Abandoned","accounts_in_message":[],"_revision_number":2}],"current_revision_number":2,"current_revision":"e731e1e985f1cd7e4875daf4758312ab8c6d2d0c","revisions":{"fed4274b82ab5f1d7e82d6e5cb01b2c34a12bf1b":{"kind":"REWORK","_number":1,"created":"2026-06-29 09:54:26.000000000","uploader":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"ref":"refs/changes/76/995276/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/ovsdbapp","ref":"refs/changes/76/995276/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/ovsdbapp refs/changes/76/995276/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/ovsdbapp refs/changes/76/995276/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/ovsdbapp refs/changes/76/995276/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/ovsdbapp refs/changes/76/995276/1"}}},"commit":{"parents":[{"commit":"b51e6cb04122541598af4e2cad766ac05ae32b46","subject":"Don\u0027t call frozen_row if there are no matching events","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/ovsdbapp/commit/b51e6cb04122541598af4e2cad766ac05ae32b46"}]}],"author":{"name":"Rodolfo Alonso Hernandez","email":"ralonsoh@redhat.com","date":"2026-06-29 09:46:48.000000000","tz":120},"committer":{"name":"Rodolfo Alonso Hernandez","email":"ralonsoh@redhat.com","date":"2026-06-29 09:54:02.000000000","tz":120},"subject":"Add ``hostname`` index to SB ``Chassis`` table","message":"Add ``hostname`` index to SB ``Chassis`` table\n\nThe ``Chassis`` table only had a client-side index on ``name``.\nCallers that query by ``hostname`` (e.g. Neutron\u0027s\n``get_datapath_type``) fell back to a linear scan over all rows.\n\nOverride ``autocreate_indices`` in ``OvnSbApiIdlImpl`` to create\nan additional index on the ``hostname`` column so that ``db_find``\nconditions on ``hostname`` use an O(log n) lookup.\n\nRelated-Bug: #2158253\nAssisted-By: Claude Opus 4.6\nSigned-off-by: Rodolfo Alonso Hernandez \u003cralonsoh@redhat.com\u003e\nChange-Id: Iedc5a1f4b3c2a8f0d9e7b6a5c4d3e2f1a0b9c8d7\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/ovsdbapp/commit/fed4274b82ab5f1d7e82d6e5cb01b2c34a12bf1b"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/ovsdbapp/commit/fed4274b82ab5f1d7e82d6e5cb01b2c34a12bf1b"}]},"branch":"refs/heads/master"},"e731e1e985f1cd7e4875daf4758312ab8c6d2d0c":{"kind":"REWORK","_number":2,"created":"2026-06-30 14:00:18.000000000","uploader":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"ref":"refs/changes/76/995276/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/ovsdbapp","ref":"refs/changes/76/995276/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/ovsdbapp refs/changes/76/995276/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/ovsdbapp refs/changes/76/995276/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/ovsdbapp refs/changes/76/995276/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/ovsdbapp refs/changes/76/995276/2"}}},"commit":{"parents":[{"commit":"b51e6cb04122541598af4e2cad766ac05ae32b46","subject":"Don\u0027t call frozen_row if there are no matching events","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/ovsdbapp/commit/b51e6cb04122541598af4e2cad766ac05ae32b46"}]}],"author":{"name":"Rodolfo Alonso Hernandez","email":"ralonsoh@redhat.com","date":"2026-06-29 09:46:48.000000000","tz":120},"committer":{"name":"Rodolfo Alonso Hernandez","email":"ralonsoh@redhat.com","date":"2026-06-30 14:00:11.000000000","tz":120},"subject":"Add ``hostname`` index to SB ``Chassis`` table","message":"Add ``hostname`` index to SB ``Chassis`` table\n\nThe ``Chassis`` table only had a client-side index on ``name``.\nCallers that query by ``hostname`` (e.g. Neutron\u0027s\n``get_datapath_type``) fell back to a linear scan over all rows.\n\nOverride ``autocreate_indices`` in ``OvnSbApiIdlImpl`` to create\nan additional index on the ``hostname`` column so that ``db_find``\nconditions on ``hostname`` use an O(log n) lookup.\n\nThe index creation is guarded by a column existence check because\nthe OVS Python IDL ``custom_index`` implementation requires that\nevery indexed column is explicitly set on newly inserted rows.\nWhen ``ChassisAddCommand`` inserts a row via ``txn.insert()``, the\nRow is created with ``_data\u003dNone``; only columns written through\n``Row.__setattr__`` are added to client-side indexes (via\n``MultiColumnIndex.add``).  During transaction ``__disassemble``,\nthe IDL removes the temporary row from ``table.rows``, which\ntriggers ``MultiColumnIndex.remove`` for every index.  Unlike\n``add``, ``remove`` does not guard against missing attributes, so\nit crashes with ``AttributeError`` when the indexed column was\nnever set.  The guard ensures the index is only created when the\nschema exposes ``hostname``, keeping backward compatibility with\nany test environment whose ``ovs`` Python library triggers this\npath.\n\nRelated-Bug: #2158253\nAssisted-By: Claude Opus 4.6\nSigned-off-by: Rodolfo Alonso Hernandez \u003cralonsoh@redhat.com\u003e\nChange-Id: Iedc5a1f4b3c2a8f0d9e7b6a5c4d3e2f1a0b9c8d7\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/ovsdbapp/commit/e731e1e985f1cd7e4875daf4758312ab8c6d2d0c"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/ovsdbapp/commit/e731e1e985f1cd7e4875daf4758312ab8c6d2d0c"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"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":""}}}]}