Merge "Add postman collection with restconf requests to get datapath id's and create transport zone when using the VPN Service NetVirt."

Merge "IT fixes due to blueprint migration"

Merge "Fixed problem with wrong class in blueprint configuration for dhcp commands"

IT fixes due to blueprint migration

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

Add postman collection with restconf requests to get
datapath id's and create transport zone when using the
VPN Service NetVirt.

I got the initial postman collection with the
"Create TZA" request from Vic Pickard.

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

Changes in netvirt-vpnservice to accomodate genius service-binding model change

Description : Genius datamodel for service-binding is getting changed
to accomodate generic service bindings at ingress/egress. There will be compilation failures
in netvirt-vpnservice once this change is merged, raisinfg a review for the changes needed in netvirt-vpnservice
to solve the compilation errors

Link to the genius review : https://git.opendaylight.org/gerrit/#/c/40767/

Change-Id: Ia6fd1a1c2a00f3900687530020ceb8d46f4d11c2
Signed-off-by: Faseela K <faseela.k@ericsson.com>

Fixed problem with wrong class in blueprint configuration for dhcp commands

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

Remove southbound's config system file reference..

As it was removed here: https://git.opendaylight.org/gerrit/#/c/40639/

Change-Id: Ic47b22a5691edac23f944fec39eca22862acf9ea
Signed-off-by: Alexis de Talhouët <adetalhouet@inocybe.com>

Merge "Clean up pom files"

Merge "Bug:6066 Added log message for tunnelports creation."

Merge "BUG-6017 java.lang.NullPointerException at org.opendaylight.ovsdb.openstack.netvirt.impl.BridgeConfigurationManagerImpl.          createBridges(BridgeConfigurationManagerImpl.java:407)"

Merge "BUG-6021 java.lang.NullPointerException at org.opendaylight.ovsdb.openstack.netvirt.          impl.NeutronL3Adapter.storeNetworkInCleanupCache(NeutronL3Adapter.java:1564)."

Bug:6066 Added log message for tunnelports creation.

Change-Id: I7a613231ed5892d24586d1c29a875a526c564f46
Signed-off-by: ananthip <ananthip@hcl.com>

Merge "Changed the description and id in aclservice-api.yang"

BUG-6017 java.lang.NullPointerException at org.opendaylight.ovsdb.openstack.netvirt.impl.BridgeConfigurationManagerImpl.
         createBridges(BridgeConfigurationManagerImpl.java:407)

    * There is no external bridge interface while reading from southbound bridge node.
    * Here highlighting the exception with existing conditional code.

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

BUG-6021 java.lang.NullPointerException at org.opendaylight.ovsdb.openstack.netvirt.
         impl.NeutronL3Adapter.storeNetworkInCleanupCache(NeutronL3Adapter.java:1564).

   * Added check for nuetron network is not null while processing a neutron network UUID.

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

Merge "DHCP with Metadata"

DHCP with Metadata

Add configuration to allow using the neutron dhcp-agent
or the vpnserice dhcp implementation.
vpnservice includes a minimal dhcp service implementation
and does not use the neutron dhcp agent.
The older netvirt relies on the neutron dhcp-agent with metdata.

Change-Id: Idfee0c6a76834a339541527af23b3adbea073e7f
Signed-off-by: Oded Shvartz <oshvartz@redhat.com>

Changed the description and id in aclservice-api.yang

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

acl-service: enable checkstyle on error

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

prep netvirt vpnservice for enabling checkstyle on error

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

prep netvirt for enabling checkstyle on error

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

translator: remove dependency of neutron.spi

netvirt still had a dependency on netutron.spi. which seems accidentally
remained when netvirt migrated from I*Aware AD-SAL to MD-SAL.
This patch removes its dependency and finish its migration with the only
dependency is neutron yang model.
In order to avoid circular dependency, NeutronModelsDataStoreHelper was
moved from neutron.util to net-virt.

Change-Id: I09628c39a2df85bc18d930639965d5a466134658
Signed-off-by: Isaku Yamahata <isaku.yamahata@intel.com>

neutron: unbreak of neutron northbound yang model revise

This patch provides the corresponding change needed to fix
compilation issues after the Yang model revise
Neutron changes:
https://git.opendaylight.org/gerrit/#/q/project:neutron+branch:master+topic:yang-model-revise

Change-Id: I15850089233d72038a70e1edcb4db1e69445fe57
Signed-off-by: Ritu Sood <ritu.sood@intel.com>
Signed-off-by: Isaku Yamahata <isaku.yamahata@intel.com>

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>

Clean up pom files

Change-Id: Ifc83ffad2deb9da0e5a39a01816e2f6659499fbb
Signed-off-by: Alexis de Talhouët <adetalhouet@inocybe.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>