Skip to content

feat(neutron): add support for Neutron FWaaS - #3712

Open
Yaguang Tang (yaguangtang) wants to merge 6 commits into
mainfrom
feat/add-neutron-fwaas-support
Open

feat(neutron): add support for Neutron FWaaS#3712
Yaguang Tang (yaguangtang) wants to merge 6 commits into
mainfrom
feat/add-neutron-fwaas-support

Conversation

@yaguangtang

@yaguangtang Yaguang Tang (yaguangtang) commented Mar 3, 2026

Copy link
Copy Markdown
Member
  • Enabled Neutron FWaaS by default for OVN deployments instead of introducing a separate role-level feature flag.
  • Wired FWaaS through the Neutron OVN defaults:
    • adds firewall_v2 to the default OVN service plugins
    • configures the OVN FWaaS driver
    • mounts neutron_fwaas.conf for neutron-server, neutron-rpc-server, and db-sync
    • runs the neutron-fwaas DB migrations
  • Enabled the Horizon FWaaS dashboard by default so the UI matches the backend behavior.
  • Added tests for the Neutron role/chart wiring and Horizon panel defaults.
  • Added deployment docs and a release note for the new default behavior.

Related

@yaguangtang
Yaguang Tang (yaguangtang) force-pushed the feat/add-neutron-fwaas-support branch from 88ea0a8 to a4fd575 Compare March 3, 2026 10:43
@yaguangtang
Yaguang Tang (yaguangtang) force-pushed the feat/add-neutron-fwaas-support branch 2 times, most recently from 1ae7dfb to 2d42600 Compare June 12, 2026 07:14
@yaguangtang
Yaguang Tang (yaguangtang) force-pushed the feat/add-neutron-fwaas-support branch from f9ff291 to c1059cf Compare June 19, 2026 13:21
@yaguangtang
Yaguang Tang (yaguangtang) force-pushed the feat/add-neutron-fwaas-support branch from c1059cf to 95e390f Compare July 7, 2026 03:22

@mnaser Mohammed Naser (mnaser) left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

small comment but cuirious why there is a bunch of other changes, we should fix that

Comment thread roles/neutron/defaults/main.yml Outdated
Comment on lines +45 to +47

# Enable Neutron Firewall as a Service (FWaaS)
neutron_fwaas_enabled: false

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is there a reason not to just make it enable out of the box without a way to disable it?

@yaguangtang

Copy link
Copy Markdown
Member Author

The CI fails due to vexxhost/docker-neutron#1962 needed.

@yaguangtang

Copy link
Copy Markdown
Member Author

Mohammed Naser (@mnaser) The CI job fails due to neutron plugins aren't updated for a long time, vexxhost/docker-neutron#1327

Extend the default OVN Neutron wiring to include FWaaS so
operators do not need a separate feature flag to turn it
on.

This change adds the firewall_v2 service plugin to the
OVN defaults, configures the OVN FWaaS driver, mounts
neutron_fwaas.conf in the Neutron server, RPC server,
and db-sync jobs, runs the neutron-fwaas database
migrations, and adds role and Helm coverage for the
new wiring.

Signed-off-by: Yaguang Tang <yaguang.tang@vexxhost.com>
Keep the Horizon defaults aligned with the OVN Neutron
defaults by always loading the FWaaS dashboard panel
when Atmosphere deploys Horizon.

This keeps the FWaaS UI available out of the box
alongside the existing VPNaaS panel and adds a test
that covers the default panel list.

Signed-off-by: Yaguang Tang <yaguang.tang@vexxhost.com>
Describe the new default FWaaS behavior for OVN
deployments and capture it in the release notes so
operators know what Atmosphere now enables
automatically.

The deployment guide now explains the built-in FWaaS
wiring, and the release note summarizes the service
plugin, driver, migration, and dashboard changes.

Signed-off-by: Yaguang Tang <yaguang.tang@vexxhost.com>
Signed-off-by: Yaguang Tang <yaguang.tang@vexxhost.com>
Signed-off-by: Yaguang Tang <yaguang.tang@vexxhost.com>
@yaguangtang
Yaguang Tang (yaguangtang) force-pushed the feat/add-neutron-fwaas-support branch from 18556cb to 3c314b4 Compare August 6, 2026 13:56
Signed-off-by: Yaguang Tang <yaguang.tang@vexxhost.com>
@yaguangtang

Copy link
Copy Markdown
Member Author

Depends on vexxhost/docker-tempest#198 to pass tempest neutron plugin test cases.

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.

2 participants