aclservice AclInterfaceStateListener update() TODO replace with comment

Change-Id: Ic3bd87e29e35b9d424c5fcfefed43fc88cc0bd1d
Signed-off-by: Michael Vorburger <vorburger@redhat.com>
Signed-off-by: Aswin Suryanarayanan <asuryana@redhat.com>
(cherry picked from commit 3754686eba6045589371ad429eaa1dec80e7b2e6)

Merge "Bug 6688 - Patch port is not correctly associated to ELAN" into stable/boron

Merge "Bug 6677: Create ext-routers when a router is created with ext-gw" into stable/boron

Merge "Bug 6687: Fix NPE when updating ExternalNetwork" into stable/boron

Merge "Code for myMAC in the L3VPN pipeline" into stable/boron

Merge "Bug 6628 - Handling missing router entities DMAC table flows" into stable/boron

Merge "Bug 6609: when 2 vm belonging to the same NETWORK/SUBNET get created in different COMOUTE NODE - ping between those 2 did not work" into stable/boron

Merge "Support multiple routers per external GW" into stable/boron

Merge "Fix bad design of AclClusterUtil to make it pluggable for e2e tests" into stable/boron

Merge "Bug 6666: Making sure no 0 datapathID is used when adding interfaces to the model, and when the node updated with the datapathID, create the relevant interfaces" into stable/boron

Bumping versions by 0.0.1 for next dev cycle

Change-Id: Ie3881039c368b1f5f60d1a1a9003f291f2e3ef86
Signed-off-by: Anil Belur <abelur@linuxfoundation.org>

Bug 6677: Create ext-routers when a router is created with ext-gw

When a router is created with an External Network, no ext-routers entry
is created. This flow is used by horizon & several tempest tests.

Change-Id: I33d520a24db1c6f455c98213e31e081c8c8509e9
Signed-off-by: Koby Aizer <koby.aizer@hpe.com>

Bug 6687: Fix NPE when updating ExternalNetwork

Happens when there are routers which are not associated with
a DPN yet.

Change-Id: I80f25439484d2e8c9f813826bec880a406421ee2
Signed-off-by: Koby Aizer <koby.aizer@hpe.com>

Bug 6688 - Patch port is not correctly associated to ELAN

Change-Id: Iaa96881879ea1fc32dfa3001abd45bbb0595ce91
Signed-off-by: Tali <tali.ben-meir@hpe.com>

Code for myMAC in the L3VPN pipeline

Code to program the L3_GW_MAC_TABLE.

Change-Id: I02252a9904c24c8ff37338b4b52cef49e73e9382
Signed-off-by: eceghkl <manu.b@ericsson.com>

Bug 6666: Making sure no 0 datapathID is used when adding interfaces to the model, and when the node updated with the datapathID, create the relevant interfaces

Change-Id: I213afac94544dc4e806cb684fc12126e358747b3
Signed-off-by: Tomer <tomer.pearl@hpe.com>
(cherry picked from commit 63056c5d9b18fc030b26c5c810172ea6302ae297)

Bug 6628 - Handling missing router entities DMAC table flows

This is part of the L2 L3 pipeline reversal.

Change-Id: If91c3cf3a06ddfb5dbf1317fc8dea0a72616e25f
Signed-off-by: yair <yair.zinger@hpe.com>

Support multiple routers per external GW

Was finally able to see the basic tempest scenario test
passing with this patch! (with a small tempest workaround
to disable ping to router interface, which is currently not
supported in netvirt)

It includes the following changes:
 * Create a vpninstance (with a new vpnId) for the neutron external network
 * Associate the external interface with that vpninstance
 * Partially revert changes done in [1] & [2], to make sure external
   floating IP & NAT rules are associated with the new vpnId
 * Fix ExternalNetworkGroupInstaller to look for the gateway IP in the
   under the new vpnId. Also do some refactoring to allow better debug messages
   for ARP resolving.
   Also, drop traffic sent to external gw traffic if its mac is not resolved (Bug 6653)

This code leverages current implementation of NAT tables to translate external vpnId
to internal vpnIds for specific floating / PNAT IPs.

[1] https://git.opendaylight.org/gerrit/#/c/43303/
[2] https://git.opendaylight.org/gerrit/#/c/43102/

Change-Id: I8d3019389d7fe6fc923a509082e7a13d61a91444
Signed-off-by: Koby Aizer <koby.aizer@hpe.com>

Fix bad design of AclClusterUtil to make it pluggable for e2e tests

Having AclClusterUtil as a "Bean" with a @PostContruct start() method
but then a static method (isEntityOwner()) which relies on that start()
is bad design - it's either static or it's not, not half-half.  It's
better not be static, and have an interface; this way the (upcoming)
end-to-end AclServiceTest can easily plug in an "always true"
implementation.

Change-Id: Ia2c07ba1cfc5b7447fcd10ad88e1008fca3df35f
Signed-off-by: Michael Vorburger <vorburger@redhat.com>
(cherry picked from commit cc82e251a77ad76659036a9bb57516539dd17cef)

Bug 6609: when 2 vm belonging to the same NETWORK/SUBNET get created in different COMOUTE NODE - ping between those 2 did not work

Fix for avoiding race condition while programming remote BC
group in participating DPNs. With earlier code, there was a chance of this
group getting overwritten by another thread

Change-Id: I09ae213c69ae971011661e713aaa92788eec1ea7
Signed-off-by: Gobinath <gobinath@ericsson.com>

Merge "Bug 6623: Fix NPE in AclServiceUtils" into stable/boron

Bug 6623: Fix NPE in AclServiceUtils

This patch fixes couple of exceptions when an IPv6 address
is present in allowed_address_pairs.

Change-Id: If0660b299500b7c00609d97484ea42961bb73dd9
Signed-off-by: Sridhar Gaddam <sgaddam@redhat.com>
Signed-off-by: Aswin Suryanarayanan <asuryana@redhat.com>

Fixes Bug 6624

Null check is added for dpid and aclUuidList.

Change-Id: Ib11bcd599474d474ee8157d44249d0e82484363b
Signed-off-by: Aswin Suryanarayanan <asuryana@redhat.com>

Merge "Fix Checkstyle violations related to exception handling in elanmanager." into stable/boron

Fix Checkstyle violations related to exception handling in elanmanager.

read
https://wiki.opendaylight.org/view/BestPractices/Coding_Guidelines#IllegalCatch

This change also includes a few additional non-Exception handling
related minor Checkstyle fixes, and can now enable enforcement of
Checkstyle for elanmanager.  (This is not directly related to the
exception handling fixes, but those were simply the last Checkstyle
violations left, and in order to ensure future non-regression, we can
now, finally, enable enforcement.)

Change-Id: If233f3d856b2f5402ef551f8ceceb242376b671c
Signed-off-by: Michael Vorburger <vorburger@redhat.com>
Signed-off-by: Sam Hague <shague@redhat.com>

Fix NPE and other exceptions

Change-Id: I691c475c6d032e543d741872afee7bbe8ec0c0ea
Signed-off-by: Sam Hague <shague@redhat.com>

Add config for neutronvpn

Change-Id: I63945c786808a93a8c825ee4862eb8f76cc7593c
Signed-off-by: Sam Hague <shague@redhat.com>

Merge "Use right docker instance" into stable/boron

Merge "Bug 6614: Fix NPE in AclInterfaceCacheUtil" into stable/boron

Merge "Fix FloatingIp add GARP handling" into stable/boron

Use right docker instance

Change-Id: Id30c7034674fdf08557a8fccd63a37b7b4fa1fed
Signed-off-by: Sam Hague <shague@redhat.com>

Bug 6608: Solve TZ name mismatch

Change-Id: I1d567bec97773a44ce16f1127b4ee43bdb0cf1c5
Signed-off-by: Slava <slava.radune@hpe.com>

Bug 6614: Fix NPE in AclInterfaceCacheUtil

Change-Id: I9337c8fb2beffb6e5b07d186974c75a3ec25740f
Signed-off-by: Sridhar Gaddam <sgaddam@redhat.com>

Fix FloatingIp add GARP handling

Consolidate VpnFloatingIPHanlder
Send GARP only for VM OVS
Reuse SendArpRequestInput
Change-Id: I0ff992d3175470163a77f4f216c5febeeb6b53a3
Signed-off-by: Ravit Peretz <ravit.peretz@hpe.com>

Merge "BUG 6598 listen to a change on BridgeRefEntry" into stable/boron

Merge "Fix NPE when updating vpnInterface" into stable/boron

Merge "Fix DMAC flows for Vlan E/W provider" into stable/boron

Fix NPE when updating vpnInterface

Change-Id: Ib7c3a8ab804947d018fc21c2fedcbf8894346c03
Signed-off-by: Koby Aizer <koby.aizer@hpe.com>

BUG 6598 listen to a change on BridgeRefEntry

in cases a bridge is created without dpnId the creation of TZ fails. we need to re-try when the entry is updated.

Change-Id: Ifc808b8895911df69703adf4122828a7056a0d6e
Signed-off-by: Shlomi <shlomi.alfasi@hpe.com>

BUG: 6582 copying DHCP service config file to etc/..config

cherry-picked from Icaf6e9e9f18f70b74d98dcfbbbc6ba2b5f0531ad

Change-Id: Iec72489b4c8c38703524608af7dc116680b9fd6d
Signed-off-by: epgoraj <p.govinda.rajulu@ericsson.com>
Signed-off-by: Sam Hague <shague@redhat.com>

Merge "Revert "BUG: 6582 copying DHCP service config file to etc/..config"" into stable/boron

Revert "BUG: 6582 copying DHCP service config file to etc/..config"

This reverts commit 3fe4cd801dbd0305d82502be19fe7f76e86f499b.

Change-Id: I7410bdf98cd76e4ca4d1f14dd137167d195f32e3
Author: Sam Hague <shague@redhat.com>
Signed-off-by: Sam Hague <shague@redhat.com>

fix security groups IT

Change-Id: I6cb9b226ed15241cb206e412352e9af02ef5a766
Signed-off-by: Sam Hague <shague@redhat.com>

Merge "BUG: 6582 copying DHCP service config file to etc/..config" into stable/boron

Merge "Bug 6592: Adding output to group of external network when using NAPT" into stable/boron

Merge "Bug 6509 - External network group tables removal is not working - indirect type" into stable/boron

Bug 6592: Adding output to group of external network when using NAPT

https://bugs.opendaylight.org/show_bug.cgi?id=6592

Change-Id: I40b40f5f65c177b3a77a3206c95a4720ed0e66de
Signed-off-by: Tomer <tomer.pearl@hpe.com>

Fix DMAC flows for Vlan E/W provider

This fix addresses two bugs related only in that
they both prevent vlan provider networks from
working:

1) Wrong (remote) dmac flows generated for Vlan provider networks
https://bugs.opendaylight.org/show_bug.cgi?id=6583

2) Mac learning in vlan provider networks clobbers static flows with wrong flows
https://bugs.opendaylight.org/show_bug.cgi?id=6584

The fix for the (1) is to test if the net is
of type vlan and then retrieve the correct action.

The fix for (2) is to simply disable configuration
of remote dmac flows when the interface from whence
entered the unknown smac is a vlan or flat network
provider interface.

Change-Id: Id3431e94d96dac5230679fcb70fe0a79efbd96a4
Signed-off-by: Josh <jhershbe@redhat.com>

Reduce IPv6 Periodic Unsolicited RA timer interval

In some situations it was noticed that flow programming is taking
some time. Due to this, the Router Soliciation requests from the
VMs are not punted to the controller (VMs are able to acquire IPv6
address only upon arrival of unsolicited Router Advts from the ODL
controller). Currently the periodic unsolicited RA interval is set
to 5 mins. This patch reduces the interval to 60 seconds to mitgate
such situations.

In a future patch, we would enhance the periodic RA interval to an
updated algorithm which will provide a better user experience.

Change-Id: I2eb4061b487cfa709ef02b7d093481799294adb3
Signed-off-by: Sridhar Gaddam <sgaddam@redhat.com>

Merge "BUG: 6569 Exceptions thrown while removing mac-entries" into stable/boron

Merge "aclservice Java @Inject etc. instead of XML for "inner" wiring" into stable/boron

Bug 6509 - External network group tables removal is not working - indirect type

Change external network group tables type to All instead of Indirect

This is a workaround fix to the 6509
https://bugs.opendaylight.org/show_bug.cgi?id=6509
The real cause for the bug will be solve in Openflowplugin in the future
https://bugs.opendaylight.org/show_bug.cgi?id=6492

Change-Id: I8bb48a644e6d815f1fe52588aef6ddc668df4956
Signed-off-by: yair <yair.zinger@hpe.com>

BUG: 6582 copying DHCP service config file to etc/..config

Change-Id: Icaf6e9e9f18f70b74d98dcfbbbc6ba2b5f0531ad
Signed-off-by: epgoraj <p.govinda.rajulu@ericsson.com>

BUG: 6569 Exceptions thrown while removing mac-entries

Change-Id: I40b3ad8865994f9481dedff3c14074dc55c9db9e
Signed-off-by: epgoraj <p.govinda.rajulu@ericsson.com>

Fixes the SSH drop from DHCP namespace

1)Changed netvirt neutron DCL to match the neutron object
representation.

2)Changed the TCP_SYN rule per to make it per port. Added IPv6 SYN drop
rule.

Change-Id: I94cef12de99f87b156e54cc07d7ea135a1549522
Signed-off-by: Aswin Suryanarayanan <asuryana@redhat.com>

Merge "correct default prootcol handling in stateless ACL" into stable/boron

aclservice Java @Inject etc. instead of XML for "inner" wiring

Some of the painfully hand-written, and often wrong and
out-of-sync with code, ODL Blueprint XML can be automatically generated
by the blueprint-maven-plugin from annotations in Java code.

This has two main advantages: 1. less XML to hand write (wrong
sometimes), and maintain;  2. easy to get a DI framework (blueprint or
another one..) to inject classes - great e.g. for more upcoming end2end
style tests.

Change-Id: Ifb86b00de680bec8ca494519225a55a7a1e6fbb1
Signed-off-by: Michael Vorburger <vorburger@redhat.com>
Signed-off-by: Stephen Kitt <skitt@redhat.com>

Bug 6559: Create TZ for routed networks

Change-Id: Id3fda9454b5c73aaa6c5de696036309a8cc0f53b
Signed-off-by: Slava <slava.radune@hpe.com>

correct default prootcol handling in stateless ACL

Change-Id: I0f87bfc77bd8c2d44fb4f8c522486779138a9df2
Signed-off-by: Slava <slava.radune@hpe.com>

Merge "Fixed the use of index NwConstants" into stable/boron

Bug 6486: add security-group-mode options

Change-Id: I62d46b4fec6f2e406b319972a068671647330477
Signed-off-by: Sam Hague <shague@redhat.com>

Fixed the use of index NwConstants

There was a change to support L2 before L3
https://git.opendaylight.org/gerrit/#/c/43126/
Change the use of constnats to the new way of work

Change-Id: I493e80fac8c49d667cf7d9a498d0f7a82892b687
Signed-off-by: yair <yair.zinger@hpe.com>

Merge "Fix Checkstyle violations (formatting only, no real changes)" into stable/boron

Merge "Bug 6556 - Fix to send the first unsolicited RA message for a Router interface" into stable/boron

Fix Checkstyle violations (formatting only, no real changes)

All of changes made here are purely "formatting", and none change code
behaviour in any way.  This should thus be safe to merge ASAP.

Change-Id: I9214f254426aae90b20d31c73422363ca69b177e
Signed-off-by: Michael Vorburger <vorburger@redhat.com>

Bug 6555 - Translation layer skip the RSP creation API call

Change-Id: Id02c7efed8d78960c7b74c813980b42a88335143
Signed-off-by: Anil Vishnoi <vishnoianil@gmail.com>

Bug 6556 - Fix to send the first unsolicited RA message for a Router interface

On creation of the router interface, the 1st unsolicited RA is sent before
the association of the subnets with the interface. So the RA packet was not
transmitted as no valid gateway address was available for the interface.
The unsolicited RA is tranmitted after network/subnet association.

Change-Id: Ifcf8e0dbe395070ff2a1731b9cf2c151ed024e9d
Signed-off-by: kalaiselvik <Kalaiselvi_K@Dell.com>

Merge "Add two node IT" into stable/boron

Add two node IT

Change-Id: Idd5aa1cf2be711420d96938769a293a1822b7407
Signed-off-by: Sam Hague <shague@redhat.com>
fix checkstyle for NetvirtIT

Change-Id: I51517f242d063ab2b68be692a96cf7c7b24bfec8
Signed-off-by: Sam Hague <shague@redhat.com>

Fix broken legacy NetvirtIT

Change-Id: I60cd9814190a162302b6bad15348010f63c9324b
Signed-off-by: Sam Hague <shague@redhat.com>
(cherry picked from commit 599b209edc6e04315cfb6275799a217390b1019c)

Merge "fix garp packet out creation" into stable/boron

Merge "Fix bugs related to wrong exception handling" into stable/boron

fix garp packet out creation

Change-Id: I4dd681f5f8a42070c49654dd6a27587046892ef0
Signed-off-by: Shlomi <shlomi.alfasi@hpe.com>

Fix bugs related to wrong exception handling

see discussion in thread "Re: [netvirt-dev] Exception Handling in
elanmanager-impl" on netvirt-dev@lists.opendaylight.org.  Checkstyle
raised these, and is right.

This resolves all Checkstyle related violations related to IllegalCatch.

Change-Id: Ie826b4688cd4b5dee8e0f33e98542319093cfa8c
Signed-off-by: Michael Vorburger <vorburger@redhat.com>

Merge "elanmanager CLI console commands impl should never catch Exception (II)" into stable/boron

Merge "Fixes Bug 6514" into stable/boron

Merge "Bug 6532: Fix ACL IPv6 VM to VM communication on same network" into stable/boron

elanmanager CLI console commands impl should never catch Exception (II)

Thank You to Checkstyle for pointing these out.  Please see
https://wiki.opendaylight.org/view/BestPractices/Coding_Guidelines#IllegalCatch
for background & details.

This finishes up & completes had started in a previous Gerrit.

Change-Id: Iddd35fa699010bc4934e55c8bf6b339997effd38
Signed-off-by: Michael Vorburger <vorburger@redhat.com>

Merge "Removing Blind imports across the module" into stable/boron

Fixes Bug 6514

The AclInterface cache was getting cleaned on a interface state change.
Now it will be cleaned up only on port delete.

Change-Id: Ia7fbb3fa77055271bed4ccad1d0155ce1615f309
Signed-off-by: Aswin Suryanarayanan <asuryana@redhat.com>

Merge "Fix NPE in FloatingIpGarpHandler.sendGarpOnInterface" into stable/boron

Bug 6532: Fix ACL IPv6 VM to VM communication on same network

This patch programs the necessary flows in the ACL (Stateful version)
to allow IPv6 VM to VM communication on the same network.

Change-Id: I1fa4bf5210c1ebfe73658800796f2c53ec88441f
Signed-off-by: Sridhar Gaddam <sgaddam@redhat.com>

Merge "Gratitious ARP handling added in ARP cache feature of L3VPN" into stable/boron

Merge "VpnPortIpToPort Listener for ELAN" into stable/boron

Fix NPE in FloatingIpGarpHandler.sendGarpOnInterface

Change-Id: I03ecf452559649391ffee5b3597ef9cf95479224
Signed-off-by: Ravit Peretz <ravit.peretz@hpe.com>

Move odl-netvirt-openstack-it to IT bundle

Change-Id: I6041e27ee2ef8ec213b2301981c9a15364175448
Signed-off-by: Sam Hague <shague@redhat.com>
Signed-off-by: Stephen Kitt <skitt@redhat.com>

Removing Blind imports across the module

This commit removes blind imports across the netvirt module.

Change-Id: I3b6d70cd6e83384eaf54f2a2f97495ae47df9e49
Signed-off-by: Suraj Ranjan <suraj.ranjan@ericsson.com>

Gratitious ARP handling added in ARP cache feature of L3VPN

As a part of the ARP cache feature of L3VPN, a timer runs once the ARP of an enterprise host is learned and is cleared as soon as this timer expires. When this host sends ARP packets(GARP for instance),the ARP cache timeout has to be reset again. Handling for it has been added.

Change-Id: I6411448c85fa739ba513b9ceff3fef4c6536108d
Signed-off-by: Gobinath <gobinath@ericsson.com>

VpnPortIpToPort Listener for ELAN

* Listens to VpnPortIpToPortListener changes
* Add SMAC and DMAC flows
* Update ELAN operational DS

Change-Id: I70280b5b4e1560e9725015d54f2345401d66f432
Signed-off-by: eceghkl <manu.b@ericsson.com>

Bug 6515: Fix NPE for an IPv6 subnet without gateway

This patch addresses the exception in ipv6service when a IPv6
subnet is created without the gateway_ip.

Change-Id: Ibfa71e1775e15a51e717557ad080634d46d1220c
Signed-off-by: Sridhar Gaddam <sgaddam@redhat.com>

Merge "Fix boron rc1 build by removing netvirt-it-utils" into stable/boron

Fix boron rc1 build by removing netvirt-it-utils

Change-Id: I0661c19bbbfdc53ef59f1ab01f489c90e80d3c48
Signed-off-by: Sam Hague <shague@redhat.com>

BUG-6455 - due to blueprint migration, the order in which the respective
ConfigActivator.start() methods is called for Netvirt and
Netvirt-Providers is now random.

This could cause timing issues where dependencies are not yet loaded
when modules like NetVirt starts up.

We need to inject the dependency of OvsdbInventoryService for
NetvirtProviders so that NetvirtProviders has that service loaded before
it starts.

Change-Id: I431554dbd76c9cad71e6d49916a8eb9e4d456f71
Signed-off-by: Bertrand <bertrand.low@hcl.com>

Merge "Netvirt - Neutron QoS rate limit support" into stable/boron

Netvirt - Neutron QoS rate limit support

Adds Neutron QoS support to Neutronvpn.

Supports listening to neutron nb for
qos policy and bandwidth limit rule changes and configuring
corresponding ovsdb ports.

Supports:
- add, update, remove of qos policy assigned to Neutron port
- add, update and delete of bandwidth limit
  rules assigned to a Neutron port.
- add, update and delete of bandwidth limit rules
  assigend to a Neutron network.

todo:
- dscp marking rules (for port and network)

Testing procedure:
1.  Create neutron network, subnet and instance.
2.  Create neutron qos policy and rules via postman.
3.  using postman assign policy to postman, remove, add rules, etc.
4.  Result:  corresponding ovsdb port ingress policing attributes
    are configured appropriately.

v2 - rebase with master
v3 - rebase with master - blueprint changes
v4 - address review comments
     use different source for mdsalutils
     fix LOGGER class
v5 - rebase to master
v6 - rebase to master

Change-Id: Ibc9e0c8c2883eebf83d5f836a91ab8bc6c5e6469
Signed-off-by: Eric Multanen <eric.w.multanen@intel.com>

Bug 6473: Fix for CSIT SSH issue.

 * If security rules created with CIDR as 0.0.0.0/0 in TCP Custom rules
   it creates nw_dst=0.0.0.0 because of this behavior TCP packets are
   getting dropped.
 * Fix: If CIDR created with 0.0.0.0/0, nw_dst and nw_src will be
   removed from table 40 and table 90.

Change-Id: I82fdfa49d4da545619259572de8e0d2f571107b1
Signed-off-by: bharathi.m <bharathi.m@hcl.com>

Merge "Fixed the vpninstance creation" into stable/boron

Merge "Refactor aclservice to for all the acl types to inherit the same abstract class" into stable/boron

Fixed the vpninstance creation

The vpn instance was created in the NeutronVpnManager and there was a bug.
It could not create it without the external gateway info.
Should now create it after external gateway info creation

Change-Id: I203693ca69a48d06b69bb8acc9868a899371fcea
Signed-off-by: yair <yair.zinger@hpe.com>

Refactor aclservice to for all the acl types to inherit the same abstract class

Depends on:
https://git.opendaylight.org/gerrit/#/c/44453

Change-Id: I7bb81dc1249a4fcf2684bad879198be00aff0be4
Signed-off-by: Slava <slava.radune@hpe.com>