x/tap-as-a-service-tempest-plugin
Code Issues Proposed changes

Fix taas-tempest-plugin gate

Browse Source 
Make pep8 job work on focal:
* adopt constraints files
* ignore W504 Line break occurred after a binary operator

Make tempest tests work by changing the default m1.medium flavor to d1.

Change-Id: Iea267fe07f3acfd344cb9843c0a79b924c2b30cc
This commit is contained in:
elajkat 2020-09-17 11:28:23 +02:00
parent 7d763bebb8
commit 993d76b82e
5 changed files with 10 additions and 9 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
neutron_taas_tempest_plugin/config.py
View File

@ -42,6 +42,6 @@ TaaSPluginGroup = [
default='ubuntu', default='ubuntu',
help='Name of ssh user to use with advanced image in tests.'), help='Name of ssh user to use with advanced image in tests.'),
cfg.StrOpt('advanced_image_flavor_ref', cfg.StrOpt('advanced_image_flavor_ref',
default='m1.medium', default='d1',
help='Valid flavor to use with advanced image in tests.'), help='Valid flavor to use with advanced image in tests.'),
] ]

4
neutron_taas_tempest_plugin/tests/scenario/manager.py
View File

@ -466,8 +466,8 @@ class NetworkScenarioTest(ScenarioTest):
port_map = [(p["id"], fxip["ip_address"]) port_map = [(p["id"], fxip["ip_address"])
for p in ports for p in ports
for fxip in p["fixed_ips"] for fxip in p["fixed_ips"]
if netutils.is_valid_ipv4(fxip["ip_address"]) if netutils.is_valid_ipv4(fxip["ip_address"]) and
and p['status'] in p_status] p['status'] in p_status]
inactive = [p for p in ports if p['status'] != 'ACTIVE'] inactive = [p for p in ports if p['status'] != 'ACTIVE']
if inactive: if inactive:
LOG.warning("Instance has ports that are not ACTIVE: %s", inactive) LOG.warning("Instance has ports that are not ACTIVE: %s", inactive)

2
requirements.txt
View File

@ -2,4 +2,4 @@
# of appearance. Changing the order has an impact on the overall integration # of appearance. Changing the order has an impact on the overall integration
# process, which may cause wedges in the gate later. # process, which may cause wedges in the gate later.
pbr!=2.1.0,>=2.0.0 # Apache-2.0 pbr>=5.5.0 # Apache-2.0

8
test-requirements.txt
View File

@ -2,12 +2,12 @@
# of appearance. Changing the order has an impact on the overall integration # of appearance. Changing the order has an impact on the overall integration
# process, which may cause wedges in the gate later. # process, which may cause wedges in the gate later.
hacking!=0.13.0,<0.14,>=0.12.0 # Apache-2.0 hacking>=3.0.0,<3.1 # Apache-2.0
coverage!=4.4,>=4.0 # Apache-2.0 coverage>=5.2.1 # Apache-2.0
python-subunit>=1.0.0 # Apache-2.0/BSD python-subunit>=1.4.0 # Apache-2.0/BSD
sphinx!=1.6.6,!=1.6.7,>=1.6.2 # BSD sphinx!=1.6.6,!=1.6.7,>=1.6.2 # BSD
oslosphinx>=4.7.0 # Apache-2.0 oslosphinx>=4.7.0 # Apache-2.0
oslotest>=3.2.0 # Apache-2.0 oslotest>=4.4.1 # Apache-2.0
os-testr>=1.0.0 # Apache-2.0 os-testr>=1.0.0 # Apache-2.0
reno>=2.5.0 # Apache-2.0 reno>=2.5.0 # Apache-2.0

3
tox.ini
View File

@ -26,8 +26,9 @@ commands = {posargs}
[flake8] [flake8]
# E123, E125 skipped as they are invalid PEP-8. # E123, E125 skipped as they are invalid PEP-8.
# W504 Line break occurred after a binary operator
show-source = True show-source = True
ignore = E123,E125 ignore = E123,E125,W504
builtins = _ builtins = _
exclude=.venv,.git,.tox,dist,doc,*openstack/common*,*lib/python*,*egg,build exclude=.venv,.git,.tox,dist,doc,*openstack/common*,*lib/python*,*egg,build