Skip to content

Remove FWaaS v1 support#2320

Closed
EmilienM wants to merge 1 commit intogophercloud:masterfrom
EmilienM:removal_fwaasv1
Closed

Remove FWaaS v1 support#2320
EmilienM wants to merge 1 commit intogophercloud:masterfrom
EmilienM:removal_fwaasv1

Conversation

@EmilienM
Copy link
Copy Markdown
Contributor

FWaaS v1 was deprecated in the Newton cycle and removed entirely in the Stein cycle.

FWaaS v1 was deprecated in the Newton cycle and removed entirely in the Stein cycle.
@coveralls
Copy link
Copy Markdown

Coverage Status

Coverage increased (+0.01%) to 79.931% when pulling d628c67 on EmilienM:removal_fwaasv1 into 1e844ba on gophercloud:master.

@theopenlab-ci
Copy link
Copy Markdown

theopenlab-ci bot commented Jan 13, 2022

Build failed.

mdelord pushed a commit to ovh/gophercloud that referenced this pull request Oct 7, 2022
This runs a Github Action to:
* Deploy Devstack with Neutron
* Run acceptance/openstack/networking
* Unskip a lot of tests
* Do not test FWaaS v1, too old and will be removed from Gophercloud via gophercloud#2320
* Skip FWAAS v2 testing after Ussuri (service was removed); and deploy it only on
  Ussuri and Train, so it can still be tested.
* Fix a test for FWAAS, where ICMP rule can't have a port in source and
  source/dest have to be comma separated ranges.
* Skip TestAgentsRUD because it can't work with OVN (default in devstack
  now). This test will have to be reworked in the future.
* Fix TestDNSPortCRUDL test (some typos)
* Skip TestLayer3RouterAgents because it can't work with OVN (default
  in devstack now). This test will have to be rewored in the future.
* Skip TestDNSFloatingIPCRDL which doesn't work with ML2/OVN
* Skip LBAAS (v1 and v2) from Neutron API, it was removed and replaced by Octavia.
  We'll need to deprecate and remove it from Gophercloud as well.
* In TestMTUNetworkCRUDL, reduce the MTU to 1440. With OVN + Geneve, we
  can't go above 1442, so let's set 1440 for the test to be safe.
* Skip VlanTransparent, Trunk and QoS testing if the extension is not there.
* Post logs if we encounter a failure
@EmilienM EmilienM closed this by deleting the head repository Sep 5, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants