Bugzilla will be upgraded to version 5.0. The upgrade date is tentatively scheduled for 2 December 2018, pending final testing and feedback.
Bug 1385341 - [RFE] [Neutron] OVS firewall driver - full support with kernel based OVS
[RFE] [Neutron] OVS firewall driver - full support with kernel based OVS
Status: CLOSED ERRATA
Product: Red Hat OpenStack
Classification: Red Hat
Component: openstack-neutron (Show other bugs)
10.0 (Newton)
Unspecified Unspecified
high Severity medium
: Upstream M3
: 12.0 (Pike)
Assigned To: Jakub Libosvar
Toni Freger
: FutureFeature, Triaged
Depends On:
Blocks: 1442136 1501603
  Show dependency treegraph
 
Reported: 2016-10-16 07:17 EDT by Nir Yechiel
Modified: 2018-02-05 14:02 EST (History)
9 users (show)

See Also:
Fixed In Version: openstack-neutron-11.0.0-0.20170807223712.el7ost
Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of:
Environment:
Last Closed: 2017-12-13 15:49:23 EST
Type: Bug
Regression: ---
Mount Type: ---
Documentation: ---
CRM:
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---


Attachments (Terms of Use)


External Trackers
Tracker ID Priority Status Summary Last Updated
Launchpad 1626010 None None None 2017-07-03 08:18 EDT
Launchpad 1697593 None None None 2017-07-03 08:18 EDT
OpenStack gerrit 385085 None None None 2017-07-05 16:05 EDT
OpenStack gerrit 472692 None None None 2017-07-05 16:05 EDT
OpenStack gerrit 473899 None None None 2017-07-05 16:05 EDT
Red Hat Product Errata RHEA-2017:3462 normal SHIPPED_LIVE Red Hat OpenStack Platform 12.0 Enhancement Advisory 2018-02-15 20:43:25 EST

  None (edit)
Description Nir Yechiel 2016-10-16 07:17:33 EDT
Description of problem:

With RHOSP 10 we introduced a new OVS firewall driver as a technology preview (see BZ 1310654). The new firewall driver is capable of filtering packets based on specified security groups using Open vSwitch only, thanks to integration with conntrack. 

The purpose of this RFE bug is to track the remaining gaps in order to graduate this feature into full support. Apart from full test coverage we also want to ensure that there is a robust upgrade/migration path between the current (iptables based) firewall driver and the OVS firewall driver so that customers would be able to smoothly (and automatically) convert their existing configuration and security-groups policies.
Comment 1 Nir Yechiel 2016-10-16 07:21:40 EDT
Note: this RFEs talks specifically about OVS with its native kernel datapath. OVS-DPDK is out of scope and will be scoped/tested separately.

Link to previous, RHOSP 10 BZ (tech preview offering): https://bugzilla.redhat.com/show_bug.cgi?id=1310654

Link to related OSP director BZ: https://bugzilla.redhat.com/show_bug.cgi?id=1367678
Comment 9 Nir Yechiel 2017-06-22 09:29:40 EDT
For RHOSP 12, this will be fully supported for greenfield deployment only. Migration path will be provided in a future release.
Comment 11 Ihar Hrachyshka 2017-07-05 16:04:29 EDT
My understanding is that the only missing bits to claim completion here are a bunch of neutron patches already posted for review and in good traction, and QE coverage.

For the latter, quoting Frank: "For the firewall driver, OVS-DPDK is expected to be tested, and a minor upgrade test has to be developed: RHOSP12 to RHOSP12.z1 has to work. However, RHOSP11 -> RHOSP12 has not to be tested as security groups are not supported with OVS-DPDK in RHOSP11."

dev:
- https://review.openstack.org/#/c/385085/ (has +w)
- https://review.openstack.org/#/c/473899/ (has +w)
- https://review.openstack.org/#/c/472692/ (has -1, needs Kuba's attention)

qe:
- minor upgrade with the new driver
- ovs-dpdk with the new driver
Comment 12 Ihar Hrachyshka 2017-07-05 16:06:20 EDT
Moving to POST since all patches seem to be on review.
Comment 13 Ihar Hrachyshka 2017-07-05 16:07:17 EDT
For the record, I created a separate RFE to follow up on migration path from iptables to openvswitch firewall driver: https://bugzilla.redhat.com/show_bug.cgi?id=1468035
Comment 14 Ihar Hrachyshka 2017-07-05 16:10:31 EDT
Clarification for comment#11: it's not clear if DPDK is in scope for the RFE, since Nir suggested before that we should focus on kernel path only. It's to be decided between PMs.
Comment 15 Ihar Hrachyshka 2017-07-05 16:25:12 EDT
Moving back to ON_DEV since it seems like POST is supposed to mean 'merged upstream' while ON_DEV is 'patches posted for review'.
Comment 17 Assaf Muller 2017-07-17 18:57:11 EDT
Quoting Hugh and Perry on the rhos-program ML:

Hugh: "I'll ask Perry to weigh in, but this looks like a reasonable request
to me, +1 pending his ACK."

Perry: "I reviewed the bz and looked at the gerrit reviews as well and also
spoke to Assaf a bit, and based on the above explanation I feel that
ACKing this is the right call."

Meanwhile, 2 out of the 3 patches needed to resolve outstanding bugs have been merged, and we expect the third to merge very soon. At that point we can flip the bug to MODIFIED and then ON_QA. It also happens that the QE test plan was already designed and executed so we're pretty far along in this RFE's cycle.
Comment 18 Assaf Muller 2017-07-20 12:01:28 EDT
The remaining work item here is https://review.openstack.org/#/c/385085/, other patches were recently merged. The linked patch presently has one +2, should merge soon.
Comment 19 Assaf Muller 2017-07-21 08:20:06 EDT
Last patch merged.
Comment 21 Toni Freger 2017-11-08 07:30:40 EST
Tested on latest osp12 with openstack-neutron-11.0.1-3.el7ost.noarch.rpm     
Setup; 3 controllers and 2 computes

According to CI reports all neutron/security group tests passed successfully.
Comment 25 errata-xmlrpc 2017-12-13 15:49:23 EST
Since the problem described in this bug report should be
resolved in a recent advisory, it has been closed with a
resolution of ERRATA.

For information on the advisory, and where to find the updated
files, follow the link below.

If the solution does not work for you, open a new bug report.

https://access.redhat.com/errata/RHEA-2017:3462

Note You need to log in before you can comment on or make changes to this bug.