The migration tool does not register the oslo.log options so fails to
set up its logger. Ensure that the options are loaded before
setup_logging call to fix it.
Closes-Bug: #2067557
Change-Id: I902f49f4efa0da80021c2eda4b7fcb63cc1310b0
devstack-gate has been retired, remove it from the
'required-projects' section of job definitions.
Change-Id: Ic8131b825066bcc5297152924d7c43c245a06045
The current method of creating a default security group may be under
an upper level context. If there is an exception of creating a
duplicate primary key for the default security group, it will cause
the context to need to be rolled back, but not rolled back, which
will affect subsequent database operations.
Through the submission of patch8, a default security group was
created every time. Upon reviewing the [1] log "Default FWG
was concurrently created", it was determined that duplicate primary keys
were actually triggered and the issue(#2061883) was fixed.
Closes-Bug: #2061883
[1] https://storage.bhs.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_94a/916968/8/check/neutron-tempest-plugin-fwaas/94a09c8/controller/logs/screen-q-svc.txt
Change-Id: Ib2e412ed5e5d4c4fcb4f0abe71d4750fb78bce27
The str(url) function in SQLAlchemy hides the password.
For a URL string that is to be re-used, use
render_as_string(hide_password=False).
Co-Authored-By: Mike Bayer <mike_mp@zzzcomputing.com>
Change-Id: I00e8bab328b3969c503d682fac6d3f3f811d9c68
'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.
[1] https://pre-commit.com/
Change-Id: I781f46a4b3a3df9dd4a3345e569cede0bb5dd955
Add file to the reno documentation build to show release notes for
stable/2024.1.
Use pbr instruction to increment the minor version number
automatically so that master versions are higher than the versions on
stable/2024.1.
Sem-Ver: feature
Change-Id: If74eaf7f7d10994ec91e2ec676bb6eb4b2f484a6
hacking 3.0.x is too old.
Try to synchronize pylint ignore and extension list with
other Networking projects.
With new pip the order of packages is not relevant, so the
related comment from requirements.txts is removed, see pip
documentation:
https://pip.pypa.io/en/stable/cli/pip_install/#installation-order
Change-Id: I99a2d30149088d3d71d56351d180e665c38686ef
tox now always recreates an env although the env is shared using envdir
options.
~~~
$ tox -e genpolicy
genpolicy: recreate env because env type changed from
{'name': 'genconfig', 'type': 'VirtualEnvRunner'} to
{'name': 'genpolicy', 'type': 'VirtualEnvRunner'}
~~~
According to the maintainer of tox, this functionality is not intended
to be supported.
https://github.com/tox-dev/tox/issues/425#issuecomment-1011944293
Change-Id: I069f590d873ea7a0e689936664c07843c2706fd5
These translation sections are not needed anymore, Babel can generate
translation files without them.
Change-Id: I7eb99b96ea1e9a1a96e28c53760d9d747670d6e4
This removed usage of LBaaS constants from unit tests. LBaaS was
retired some time ago and these constants will be removed.
Change-Id: I2951c866cdfbd88bcba9fb8d299f592cd6a44dff
This option is not needed since [1], Also set
tempest_plugins var to setup neutron-tempest-plugin
in tempest venv.
[1] https://review.opendev.org/c/openstack/devstack/+/558930
Change-Id: I595f81ac10443b6484b8b0bc0a12b5b58e75044f
Add file to the reno documentation build to show release notes for
stable/2023.2.
Use pbr instruction to increment the minor version number
automatically so that master versions are higher than the versions on
stable/2023.2.
Sem-Ver: feature
Change-Id: I5f7c8b7b019852558289e94dd18d4867b033b3cb
The mock third party library was needed for mock support in py2
runtimes. Since we now only support py38 and later, we can use the
standard lib unittest.mock module instead.
Change-Id: I009b6e65424ba3b662949baa1226fbf0ff14af80
Since [1] (release 1.11.0), the Alembic operations "now enforce
keyword-only arguments as passed as keyword and not positionally"
(from the commit message).
This change is compatible with the previous versions (as confirmed
in the CI).
[1]df75e85489
Closes-Bug: #2019948
Change-Id: Iedf9a47a80a2775f73e7873ce3e55c4152d0f564
If the status of the firewall group is consistent with the status
to be updated, there is no need to notify the plugin to update the
status.
The processing of the agent requires a certain amount of time. If
firewall group is updated during this period, updating the firewall
group status at this time may cause the firewall group to be in an
incorrect state.
Partial-Bug: #2021457
Change-Id: I316827259367d78d7dbb57888ad41408d44c43f6
Currently, we determine that the firewall group is in use based on
its ACTIVE status. But the firewall group may have just updated
the port and is currently PENDING_UPDATE status, deletion should
not be allowed at this time.
This patch changes the judgment method for deleting firewall
groups, no longer based on their status. But like other neutron
resources, based on whether or not they are associated.
Closes-Bug: #2018967
Depends-On: https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/883826
Change-Id: Ib7ab0daf9f6de45125ffc9408f865fc0964ff339
These exception definitions are already in neutral-lib.
And it has already been directly referenced in neutron lib
in the code.
Change-Id: I9f0d5efc3d86bdab97a188ea880780b19e2a7e43
Now, this patch will support standard_attrs for
firewall_group/firewall_rule/firewall_policy.
Closes-Bug: #1986906
Change-Id: Ib7b06d604a0950a104215bcf4386e14b77d20d12
Neutron recently merged a commit enabling secure rbacs by default
[1], breaking several unit tests of networking-fwaas.
This patch changes the necessary test calls to be admin and
requires neutron >= 23.0.0.0b2.
[1]670cc383e0
Related-Bug: #2019097
Change-Id: I60ad379f9fc94919581f461fd6a731cfe4baba30
Add file to the reno documentation build to show release notes for
stable/2023.1.
Use pbr instruction to increment the minor version number
automatically so that master versions are higher than the versions on
stable/2023.1.
Sem-Ver: feature
Change-Id: I63a4a2131a6d569a120346dc20e8a413cdae93f4
Since we use usedevelop=True, as a result neutron-fwaas
installation happens at 'develop-inst' phase and it installs
requirements from requirements.txt without considering
tox constraints, This can lead to issue where broken
requirements are installed in docs job and as a result
it will fail.
In order to get constraints be considered for this case, add
requirements.txt to deps for docs env(same as [1]).
[1 https://review.opendev.org/q/Ic8f6472fe708d12f8fc129729465367083e1095a
Change-Id: Iea9556f6f025ad2f3dfa38fd401aa64414bd561f