0742aba...
by
OpenStack Release Bot <email address hidden>
Update TOX_CONSTRAINTS_FILE for stable/2024.2
Update the URL to the upper-constraints file to point to the redirect
rule on releases.openstack.org so that anyone working on this branch
will switch to the correct upper-constraints list automatically when
the requirements repository branches.
Until the requirements repository has as stable/2024.2 branch, tests will
continue to use the upper-constraints list on master.
'pre-commit' tool [1] provides an extensible way to configure hooks
which can be run automatically before each commit. That way we can make
sure that code send to review is passing base static analysis checks
like e.g. bandit, flake8 or hacking.
For now this is optional to use and not integrated in e.g. tox.ini file
but we can integrate it there too as next step.
Fix _delete_ports_flowrules_by_id so that it expects only one port_id
Private method _delete_ports_flowrules_by_id was done in the way that it
was expecting list of ports given as "ports_id" variable but the only
place where this method was called is passing ID of the one port only.
As it was not list but string, iteraton was done over string elements
and that was causing plenty of messages send to the neutron server for
every char of the each deleted port if. This was overloading
neutron-ovs-agent.
This patch changes that private method to be
_delete_port_flowrules_by_id and to expect only one port id to be passed
to it as argument as there is no any place in the code where old method
would be called with list of the ports.