Revert "Use HasStandardAttributes as parent class for Tags DB model"
This reverts commit b92fc1ad6c87c02537d790e63dd9c8a391682ae0.
Reason for revert:
The original change was made as a “cheap win” to optimize the number
of queries the neutron server makes during testing. This did
improve the number of queries made but introduced regression in
real world deployments where some customers(through automation)
would define hundreds of tags per port across a large deployment.
I am proposing to revert this change in favor of the old “subquery”
relation in order to fix this regression. In addition, I filed the
Related-Bug #2069061 to investigate using `selectin` as the more
appropriate long term solution.
Reviewed: https:/ /review. opendev. org/c/openstack /neutron/ +/921569 /opendev. org/openstack/ neutron/ commit/ 4f5c023279a0f9d 0db8005848c8b35 6749ff4c8a
Committed: https:/
Submitter: "Zuul (22348)"
Branch: stable/2023.1
commit 4f5c023279a0f9d 0db8005848c8b35 6749ff4c8a
Author: Miro Tomaska <email address hidden>
Date: Fri Jun 7 19:31:43 2024 +0000
Revert "Use HasStandardAttr ibutes as parent class for Tags DB model"
This reverts commit b92fc1ad6c87c02 537d790e63dd9c8 a391682ae0.
Reason for revert:
The original change was made as a “cheap win” to optimize the number
of queries the neutron server makes during testing. This did
improve the number of queries made but introduced regression in
real world deployments where some customers(through automation)
would define hundreds of tags per port across a large deployment.
I am proposing to revert this change in favor of the old “subquery”
relation in order to fix this regression. In addition, I filed the
Related-Bug #2069061 to investigate using `selectin` as the more
appropriate long term solution.
Change-Id: I437ab71f01483b b8adea96757a6d7 dd6253df182
Closes-bug: #2068761
Related-Bug: #2069061