Merge "DHCP for TOR VMs"

Merge "BUG 6076: protect from NPE when port contains ipV6"

Merge "Modified the aclservice-api.yang."

Merge "Commit includes: 1> NeutronVPN support for floating IPs associaton to fixed IPs"

Modified the aclservice-api.yang.

1)Removed the security group and security rule uuid. The key from ietf
will be used instead.
2)Changed the direction to be an attribute of access-list-entries.

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

DHCP for TOR VMs

- Handling deletion of L2GwConnection
- Corrected the event handling during tunnel state up and down events
- Handling exception during UcastLocalMac events properly.

Signed-off-by: Kency Kurian <kency.kurian@ericsson.com>
Change-Id: I6859f3d7feb55794d124c3e1aad5ed1227849b84

Commit includes:
1> NeutronVPN support for floating IPs associaton to fixed IPs

    a. Extra dependency in fibmanager-shell bundle on fibmanager-impl also
       removed.
    b. Changes to FloatingIpInfo yang model to take router input as string
       rather than uint32.
    c. Resolved conflicts for the above.

2> Karaf CLI for DHCPService

3> Check for vlan/gre network provider in neutronvpn

4> SNAT : Fixed the neutron code to update the IPv4address in External
   Router model

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

BUG 6076: protect from NPE when port contains ipV6

Change-Id: I9353672f0d543160ca7a7a253c875fd261f2704f
Signed-off-by: Shlomi <shlomi.alfasi@contextream.com>

.gitignore .factorypath created by m2e-apt

https://github.com/vorburger/opendaylight-eclipse-setup now bundles
https://github.com/jbosstools/m2e-apt, which creates .factorypath files
all over the place.

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

Merge "BUG-6070  dpid changes as ports added to bridge"

BUG-6070  dpid changes as ports added to bridge

Fixed by adding (default) config to auto generate a
mac address and assign it to br-int using
other_config:hwaddr

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

Merge "Use <> Java 7 operator"

Merge "Enhance host config to add multiple configs per host"

Merge "Use <> Java 7 operator"

Merge "Simplify boolean expressions"

Merge "Add revision for acl.yang now that it is included in mdsal"

Merge "Bug 6056 - Wrong logging in NetvirtSfcStandaloneOF13Provider"

Add revision for acl.yang now that it is included in mdsal

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

Merge "Initial draft for the netvirt AclService yang."

Merge "Simplify boolean expressions"

Merge "Mechanical clean-up: semicolons, default access"

Merge "BUG:6009 Handling the FlowRemove Event message from the Event Listener"

Use <> Java 7 operator

Change-Id: Ic524731144e92652b8809bf722db743d22006f50
Signed-off-by: Stephen Kitt <skitt@redhat.com>

Use <> Java 7 operator

Change-Id: I01ae9fd3d761d86325e425d0a76c631675205446
Signed-off-by: Stephen Kitt <skitt@redhat.com>

Simplify boolean expressions

Change-Id: Ie0fa8ea6286883e5304eddb63ba3fcd58805f00e
Signed-off-by: Stephen Kitt <skitt@redhat.com>

Simplify boolean expressions

Change-Id: I239a8cdfca687bdb9ebec85fa4b67adc97517a7a
Signed-off-by: Stephen Kitt <skitt@redhat.com>

Mechanical clean-up: semicolons, default access

* DHCPOptions shouldn't "extend Object", that's the default;
* interface methods don't need "public", that's the default;
* enum constructors don't need "private", that's the default;
* remove duplicate semicolons.

Change-Id: I60c05c1454b3d7bfac22e90c58d7b35bbd323a8a
Signed-off-by: Stephen Kitt <skitt@redhat.com>

Mechanical clean-up: semicolons, default access

* interface methods don't need "public", that's the default;
* enum constructors don't need "private", that's the default;
* remove duplicate semicolons.

Change-Id: I1740214e4d2e47a79e7f7681f94a26803ebe15d2
Signed-off-by: Stephen Kitt <skitt@redhat.com>

BUG:6009 Handling the FlowRemove Event message from the Event Listener

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

Merge "Bug 6014 - Named Thread pool Executors for better debugging"

Bug 6056 - Wrong logging in NetvirtSfcStandaloneOF13Provider

Change-Id: I0dba2fab675c00fac1e736118dd3e74b96c9ab9c
Signed-off-by: Dileep Ranganathan <dileep.ranganathan@intel.com>

Merge "Initial changes for ACL Service"

Enhance host config to add multiple configs per host

Current implementation of host config allows one configuration
per host. This patch adds support for multiple configurations
per host. Also this patch removes the fixed host type values.
Neutron patch: https://git.opendaylight.org/gerrit/#/c/36767/.
Openstack doc: https://review.openstack.org/#/c/309630/

Change-Id: I2cfe421611d66741970963dc80184a0e470d8697
Signed-off-by: Ritu Sood <ritu.sood@intel.com>

Bug 5989: Thrown nullpointerexception while updating the port.

This patch is added to catch the NullPointerExceptions which could occur while updating the port.

Change-Id: Ia72959830b8b42f0d9c17f84cbddd30b96318d52
Signed-off-by: Abbas Pareedkunju <abbas.pareedkunju@tcs.com>

Merge "BUG-2714 OVSDB needs to be more proactive in reporting errors with underlying OVS instances."

Merge "L2 gateway functionality related fixes."

Bug 6014 - Named Thread pool Executors for better debugging

Enhances debugging and logging information.
Distinguishes thread names in executor pools.

Change-Id: I366a0ade4005004c8aec963863a3cbd8d2913878
Signed-off-by: Dileep Ranganathan <dileep.ranganathan@intel.com>

Initial draft for the netvirt AclService yang.

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

Initial changes for ACL Service

Change-Id: Ice8fbcfe3afc3f08197e6ae1b2b8bafe52fd037b
Signed-off-by: Shashidhar R <shashidhar.raja@ericsson.com>

BUG-5988: throws NullPointerException while creating a network without br-int interface.
  * Handled an exception properly wherever throws NullPointerException
    while creating a network without br-int interface.

Change-Id: Ie73cd46d5c857b90a6ff975883ac2764a60dd0d2
Signed-off-by: vinoth <m.vinoth@hcl.com>

Merge "BUG-5894 NullPointerException while deleting the interface from router.   * While deleting the interface from router,     checking the floatingIp's port uuid is null else     delete the respective floating Ip."

Merge "Natservice module - bug fixes for: 1) Non primary group entry not updated during primary switch over 2) NAPT reverse Traffic is not working & fib is not getting updated for New NAPT switch fix 3) Delete Router doesnt remove ext fixed IP fib entry"

BUG-5894 NullPointerException while deleting the interface from router.
  * While deleting the interface from router,
    checking the floatingIp's port uuid is null else
    delete the respective floating Ip.

Change-Id: I36ed6c25716cc1993de1cb110edf3a0ba47816e7
Signed-off-by: arthi.b <arthi.b@hcl.com>

Merge "Use DockerOvs + test connect to 2 OVSs"

Merge "Netvirt project : Neutron VPN enhancements to handle external networks"

Merge "BUG-5614: Ovsdb should not flood the packets to compute nodes unless tenant network exists in the compute node   * Before adding tunnel rules,     checking the network present or not in src and dst node.     If network present in both nodes adding the Vxlan port     in flood entry in src and dst.Else do not add vxlan ports."

L2 gateway functionality related fixes.

+ Earlier during l2gw connection delete, remote ucast macs to be deleted
were the combination from ELAN MAC table plus other Elan L2Gateway devices
local macs. This logic is changed. Now remote ucast macs for that particular elan is
read/filtered from node config DS.

+ If batch deletion operation fails, macs are deleted sequentially.

+ During deletion of logical switch, updated to delete entry from
LogicalSwitchDeletedTasks.

+ Renamed LogicalSwitchDeletedJob to DeleteLogicalSwitchJob which
reflects its functionality.

+ Corrected job key in DeleteL2GwDeviceMacsFromElanJob.

Change-Id: Ice6892c7a0bd8d703ba5527fa34afe613180230d
Signed-off-by: Somashekar Byrappa <somashekar.byrappa@ericsson.com>

Natservice module - bug fixes for:
1) Non primary group entry not updated during primary switch over
2) NAPT reverse Traffic is not working & fib is not getting updated for New NAPT switch fix
3) Delete Router doesnt remove ext fixed IP fib entry

Change-Id: I80e5493d2aaf44adb5b3d1bec5ef0874f7f521c5
Signed-off-by: Sumanth MS <sumanth.ms@ericsson.com>

BUG-2714 OVSDB needs to be more proactive in reporting errors with underlying OVS instances.

 * Logging an error message if br-int interface is not available while creating a network.
 * More discussion details with community can be found in below link,
   https://lists.opendaylight.org/pipermail/ovsdb-dev/2016-May/003206.html

Change-Id: Ic1f9cbbe77c4b5f51724c8efec52bfa9860ab6b4
Signed-off-by: vinoth <m.vinoth@hcl.com>

Netvirt project : Neutron VPN enhancements to handle external networks

Neutron VPN enhancements to handle ExternalNetworks inclusion into VPN.  Also handles association of externalnetworks as gateways to neutron routers.

Fixes to ExternalNetwork enhancement to neutronvpn.

Fix to ensure association/disassociation of external networks to VPN.

Fix to handle internal router interfaces for SNAT.

Change-Id: If8bcd6e4832080c38a87a14e22f98ba97a8613fb
Signed-off-by: Kiran N Upadhyaya <kiran.n.upadhyaya@ericsson.com>

Use DockerOvs + test connect to 2 OVSs

Do not merge before  https://git.opendaylight.org/gerrit/39461

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

L2Gateway: Fixes related to L2 gateway functionality.

+ Removed the check isL2GwDeviceConnected during l2 gw connection delete.
- This change ensures cleaning of data from cache and config DS
  which avoids having stale data when l2gw device is down.
- The stale data was blocking deletion of configs from
  l2gw device during subsequent l2gw connection (having
  same network id) deletes.

+ Few L2GatewayDevice class variables are updated by
  using Set instead of List to avoid duplicates. Also
  toString() method is overridden to print minimal info.

+ doesLocalUcastMacExistsInCache check is added during
  handling of the newly discovered L2gw device mac. This
  check avoids unnecessary configurations found during
  scalability testing.

Change-Id: Iab6be8d67645518e0e78f9ce883b305867c81958
Signed-off-by: Somashekar Byrappa <somashekar.byrappa@ericsson.com>

Merge "Enable NetVirt Maven site"

Enable NetVirt Maven site

This patch enables the Maven site, fixes up all the URLs and corrects
blocking Javadoc errors.

Change-Id: I534e904a7bd31394bcf833c1403d77ec689ea2ef
Signed-off-by: Stephen Kitt <skitt@redhat.com>

L2Gateway: Handling Hwvtep node tunnel ip update. Plus two more fixes.

+ Hwvtep node tunnel ip update is handled only in case of first time
configuration of tunnel IP (i.e., None to valid IP address).
Subsequent updates i.e., changing from one tunnel IP to another is not
handled.

+ Replaced HwvtepNodeListener with HwvtepPhysicalSwitchListener which will
handle Hwvtep node add/remove/update scenarios.

+ Changed logical switch listener to cluster-wide DCN listener to cache
ELAN L2 Gw details on all the cluster nodes when logical switch is
created.

+ Changed remote mcast mac listener to cluster-wide DCN listener.

Change-Id: Icb5efe578102f22650fc9b4ce4dcc8b7022398fe
Signed-off-by: Somashekar Byrappa <somashekar.byrappa@ericsson.com>

Merge "Remove unsued nic dependencies"

Merge "Remove fcapsmanager and application"

Remove unsued nic dependencies

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

Bug 5939 - Communications through external gateway not working

The problem happens because the GatewayMacResolver is not resolving the
MAC address for the external gateway.  GatewayMacResolver is failing
because externalSubnet == null in triggerGatewayMacResolver.  This is
happening due to changes from [0] (a fix for BUG-5291).

In [0], a change was made to only set GatewayIP if port.getDeviceOwner()
== "network:router_interface"; however, in our case,
port.getDeviceOwner() == "network:router_gateway", so the Gateway IP
isn't set.

In this fix, I also set GatewayIP if port.getDeviceOwner() ==
"network:router_gateway".

[0] https://git.opendaylight.org/gerrit/#/c/37136/

Change-Id: I49b089900e4f85581beb8634580439dca2413ce5
Signed-off-by: Andre Fredette <afredette@redhat.com>

Remove fcapsmanager and application

Fcaps manager and application have been moved to Genius
https://git.opendaylight.org/gerrit/#/c/39107/

Change-Id: Ic71cacb4fccb526e8ad1e70d5538289a6361ca84
Signed-off-by: Vishal Thapar <vishal.thapar@ericsson.com>

use the right hop for more than one sf

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

Merge "Added support for enable/disable security on a port dynamically."

Merge "BUG-5813: Vxlan ports should not be removed in table 110 flow entry unless last VM instance removed from the openstack node.   * Before deleting the Vxlan port in flow entry it should     check whether the deleted vm instance is last or not.     If it is the last vm instance Vxlan port should be delete from     source node in flow entry else vxlan port shouldn't be delete."

updates for sfc models

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

Added support for enable/disable security on a port dynamically.

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

Change BGP thrift port to 7744

BGP Thrift port is 6644 which conlicts with NetIDE.
Refer: https://wiki.opendaylight.org/view/Ports

Change-Id: I8931fa78f53afea643d6856b005c740670036ef0
Signed-off-by: Vishal Thapar <vishal.thapar@ericsson.com>

BUG-5813: Vxlan ports should not be removed in table 110 flow entry unless last VM instance removed from the openstack node.
  * Before deleting the Vxlan port in flow entry it should
    check whether the deleted vm instance is last or not.
    If it is the last vm instance Vxlan port should be delete from
    source node in flow entry else vxlan port shouldn't be delete.

Change-Id: Iadc9003ed81b1e8e475d629f80ccd961242ef859
Signed-off-by: hari.pr <hari.pr@hcl.com>

Merge "Fix Neutron VPN to handle Routers."

Merge "Added support in neutron and it utils for SGs"

Fix Neutron VPN to handle Routers.

This fix enables Neutron VPN module inside netvirt to
handle routers and router interfaces, thereby enabling
internal VPN to become functional.

Change-Id: I527e182c5ad38b0cabf457dee465763dcd30dd0c
Signed-off-by: Vivekanandan Narasimhan <n.vivekanandan@ericsson.com>

Added support in neutron and it utils for SGs

Methods in NeutronUtils for creating SecurityGroups
Added param in NeutronNetItUtils.createPort for security groups

Initially commited in draft mode

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

Use Genius for all infra modules in vpnservice

Change-Id: I30e64d9a0b0e80f463ef364da08ae7d9750b5d72
Signed-off-by: Vishal Thapar <vishal.thapar@ericsson.com>

Namespace changes for vpnservice modules

Change-Id: I4830703f94a218c6b4ccd97076ca852472bb16d5
Signed-off-by: Vishal Thapar <vishal.thapar@ericsson.com>

Fix CSIT failures

These changes bring in patch
https://git.opendaylight.org/gerrit/#/c/38361/ from vpnservice to address CSIT failures

Change-Id: I66064b35046d8078c82a0ba6cf68d20a763757f2
Signed-off-by: Vishal Thapar <vishal.thapar@ericsson.com>

BUG-5614: Ovsdb should not flood the packets to compute nodes unless tenant network exists in the compute node
  * Before adding tunnel rules,
    checking the network present or not in src and dst node.
    If network present in both nodes adding the Vxlan port
    in flood entry in src and dst.Else do not add vxlan ports.

  * During last instance removal form the node,
    checking the network present or not in src node.
    If src node doesn.t have the network remove the tunnel entries in src node
    and also remove the Vxlan port of src node from dst node.

Change-Id: Ic8e6010e50c04882bb5afe86c2dcd4695911c366
Signed-off-by: hari.pr <hari.pr@hcl.com>

Fix NPE in FibManagerProvider

Change-Id: I216a5ca030472fce08e49ec49833d0a23988de5d
Signed-off-by: Vishal Thapar <vishal.thapar@ericsson.com>

Initial merge of vpnservice code

Based on vpnservice hash: commit f267eb8b08e303ccc40131361fe82f2ad5ea1628

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

Imported vpnservice as a subtree

Change-Id: I31c80009a31b0b97c1d10246e844be69e81125f4

Merge "Integration of fcaps applications"

Merge "IDManager fixes for restart scenario"

Integration of fcaps applications

Change-Id: If2fffa37ebed025b203b280ebf7b88a606c4823d
Signed-off-by: Viji J <viji.j@ericsson.com>

Merge "Integration of Fcaps framework"

Updated L2Gw changes in "neutronvpn", "elanmanager" and "dhcpservice" modules

- Used entity ownership service to handle l2gw requests only in one cluster node.
- Registered for entity ownership for elan entity to process all the l2gw requests.
- Taken out the datastore jobcoordinator jobs into separate classes.
- Handled external tunnel state events.

Change-Id: I39a2a24c8927e48fc7d0299de008cfff9ca6d2b4
Signed-off-by: Shashidhar R <shashidhar.raja@ericsson.com>

Merge "Remove ovsdb related in resources"

Merge "Implementing Host Config data update"

Merge "Added isPortSecurityEnabled check to enable/disable SG."

Integration of Fcaps framework

Change-Id: I68429a51ba6cd8a7d49a426b78fad866efdf126e
Signed-off-by: Viji J <viji.j@ericsson.com>

Merge "Natservice module bug fixes"

postman: use 1 for netvirt table offset

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

Natservice module bug fixes

Change-Id: I8535d504b50d6b1a1a7fa8e14d51212596017b4c
Signed-off-by: Sumanth MS <sumanth.ms@ericsson.com>

Vxlan/Gre co-existence,Alarms,tunnelstate,TR fixes

Change-Id: Ia253c413dc198dce371b6f87c2c67af2bdb324b5
Signed-off-by: HemaTG <hema.gopalkrishnan@ericsson.com>

Merge "DHCP Handling for TOR VM"

Fixes in interface manager and aliveness monitor

Removing HwVTEP tunnel entries from topology configuration and operational
datastores
cli command to display internal tunnels is fixed to show the interface
name along with the datapath node identifier
A check has been add to update interface operational state only of it
exists in the datastore
Tunnel to DC gateway should not have key "flow" set
Tap interface not getting deleted from interface operational datastore is
fixed
Interface operational status has been handled to get updated based on port
up/down events
Repopulation of monitor configuration after re-enabling tunnel monitor has
been fixed
Optimizations in aliveness monitor error handling code

Change-Id: I19f0ec5fb0133575e6e08a741f3f0f9c4b805ba7
Signed-off-by: Faseela K <faseela.k@ericsson.com>
Signed-off-by: Kiran N Upadhyaya <kiran.n.upadhyaya@ericsson.com>

DHCP Handling for TOR VM

- As the TOR floods the packets to the CSS mesh, some logic should be
  added such that only one CSS punts the DHCP packet to the controller and
  the other CSS drops the DHCP packet.
- The designated CSS is chosen for each TOR-ELAN pair and it is stored in
  the datastore.

Change-Id: I695f1a7600938c1e4f6411f9752de20bf44349cd
Signed-off-by: Kency Kurian <kency.kurian@ericsson.com>

Implementing Host Config data update

This patch implements the logic to fill host config data
based on the yang model defined in the Neutron Northbound
https://git.opendaylight.org/gerrit/#/c/36767/.

Updated patch after addressing review comments

Change-Id: I1ea6b315ab9de70dc1c1849ba905eee2eb51d94e
Signed-off-by: Ritu Sood <ritu.sood@intel.com>

BFD monitoring changes

Enabling/Disabling BFD monitor was fixed. Updates to the  monitor
parameters were not being considered while writing to the data store.

Change-Id: Iaa9a71b353a78d8ac626a81dbe43e1587b8bbdf8
Signed-off-by: Kiran N Upadhyaya <kiran.n.upadhyaya@ericsson.com>

Merge "BUG 5783: Ping test is FAILED from VM to Invisible IP which is hosted non-nexthop."

Tunnel monitoring changes

Code revision in Aliveness Monitor files related to fetching mac address
from data store.
Monitor profile creation for both the tunnel endpoints was fixed.
Changes in JUNIT test cases related to tunnel monitoring.

Change-Id: I0843ee855f854a3fd98d212516292d8b34a8409c
Signed-off-by: Kiran N Upadhyaya <kiran.n.upadhyaya@ericsson.com>

Merge "LLDP monitor interval update fixes"

BUG 5783: Ping test is FAILED from VM to Invisible IP which
is hosted non-nexthop.

a. Ensure that SubnetRoutePacketHandler can
work in deployments where DPNs (i.e., Computes)
are connected to different PLs of an ODL Cluster
(i.e., different ODL instances).  More specifically,
this leads to a deployment where the nexthop
reports to one PL but the invisible IP is in a DPN
that is reporting to a different PL in a clustered
environment.

b. Ensure that SubnetRoutePacketHandler can
work with apply_actions rule used in FIB Table
(i.e., Table 22).

c. Create Table 22 (SubnetRoute Table) rules
as default Table Miss Flows instead of dynamically
creating them on the fly in FibManager.

Change-Id: I86e0b3ff8761b054603bc9bb7853340078ad653d
Signed-off-by: Suraj Ranjan <suraj.ranjan@ericsson.com>
Co-Authored-By: Vivekanandan Narasimhan <n.vivekanandan@ericsson.com>