Merge "Remove redundant "public" from Java interface"

Merge "Added existsVPN API and refactored NeutronVPN"

Fix typo in factory-method name

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

Added existsVPN API and refactored NeutronVPN

1. existsVPN API added in VPNMgr, removed from NeutronVPN
2. Indentation, logging, import statement changes
3. Removed unused router-dpns yang model
4. Added notifications for router assc/dissc to/from VPN
5. Refactored a couple of validations

Change-Id: I139b5809b6519de5be3618b0accdb7987b551729
Signed-off-by: Abhinav Gupta <abhinav.gupta@ericsson.com>

Remove redundant "public" from Java interface

I'm about to enforce this as a Checkstyle rule in
https://git.opendaylight.org/gerrit/#/c/41302/

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

Merge "Instead of augment hwvtep, added a new entity which references the logical switch ETree YANG model cosmetic change"

Fix breakage due to aclservice yang models changes

This patch changes yang models https://git.opendaylight.org/gerrit/#/c/41234/2
and broke netvirt.

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

Merge "Initial changes for IPv6 Service"

Merge "clean aclservice files"

Merge "added Security Rule listener to handle coversion from security rule to acl model"

Instead of augment hwvtep, added a new entity which references the logical switch
ETree YANG model
cosmetic change

Change-Id: I33ad5e3a435b7634e0dc3032ae9dfcf2a09f6d09
Signed-off-by: Guy Sela <guy.sela@hpe.com>

clean aclservice files

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

Merge "Support for IT ping feature"

remove unused features to shorten build time

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

Support for IT ping feature

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

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

Merge "Multiple fixes in neutronvpn code"

Merge "Bug 5860 Fix port event wrong ordering"

Merge "ACL yang updated to include allowed-address-pairs with Interface augmentation"

Multiple fixes in neutronvpn code

Commit includes:

1> Fixing NPE in NeutronVpnUtils when no network-type is provided from
northbound

2> NVPN: neutron northbound support for gateway mac for external
networks

3> Use vif_type to determine portname prefix

4> NeutronVPN: Handling fixed-floating IP dissociation for floatingIP
delete

5> NeutronVPN additional -ve validations via ECM for network/router
assc/dissc

  a. Added validations for the following:

  a1) Associate network(s) to vpn:
    i) network already assc to another VPN
    ii) network's subnet added to some router as interface

  a2) Dissociate network(s) from vpn:
    i) network's subnet added to some router as interface

  a3) Associate router to vpn:
    i) router assc to another vpn
    ii) network corresponding to router's subnet interface already assc to VPN

  a4) Dissociate router from vpn:
    i) router assc to another vpn

  b. Updating VPNMaps once assc/dissc are done in assc/dissc network(s) from
     VPN

6> NeutronVPN: adding yang model and notifications for router-dpn
map

  a. Added a container to maintain mapping between neutron router and
  DPN(s) on which vpn-interfaces for router are present.

  b. Added notifications for router associated/dissociated to/from vpn

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

Merge "Neutron port listener updated to handle security group configurations."

ACL yang updated to include allowed-address-pairs with Interface augmentation

Neutron supports default anti-spoofing capabilities with
allowed-address-pair Neutron port attribute which also gets
enabled/disabled with port_security_enabled. As this is based on SG flag,
updating ACL yang to support same with Interface augmentation.

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

Bug 5860 Fix port event wrong ordering

If the southbound event arrives before the northbound event the
tenant network is not there yet and therefore the flows for
this port will not be created.

This patch add new logic to the PortHandler (northbound event)
to verify if the southbound event was already arrived and to
add the missing flows.

Change-Id: Id8ad03b6de8afff0b64ca3af3457281ddbaf0f74
Signed-off-by: oshvartz <oshvartz@redhat.com>

Merge "Added support for fixed Security group."

added Security Rule listener to handle coversion from security rule to acl model

Change-Id: I4d7c80e78c58b5739e085bc3ad6f4e1199f1ee49
Signed-off-by: gujele <imanilyadav@gmail.com>

Neutron port listener updated to handle security group configurations.

During Neutron port add/update, Security group details are updated to corresponding ACL Config DS.

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

Initial changes for IPv6 Service

Change-Id: Ib10708e7010360eacb00f89655f0dbb59359a6c5
Signed-off-by: Sridhar Gaddam <sgaddam@redhat.com>
Signed-off-by: kalaiselvik <Kalaiselvi_K@Dell.com>
Signed-off-by: Mohnish Anumala <Mohnish_Anumala@Dell.com>

Add blueprint wiring for neutron renderer

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

Added support for fixed Security group.

1) Added DHCP and Arp rules
2) Added default connection tracking rules.

The rules will be added to the ovs only after integrating with genius.

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

Add blueprint wiring for openstack/net-virt-providers

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

Add blueprint wiring for openstack/net-virt

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

Add blueprint wiring for openstack/net-virt-sfc

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

Remove getModuleName()/getInstanceName() in IT

see: https://git.opendaylight.org/gerrit/#/c/39891/

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

Fix breakage due to SFC yang models changes

This patch modify many yang models used by Netvirt-SFC
https://git.opendaylight.org/gerrit/#/c/40209/

- ietf-access-control-list.yang is now in MDSAL
review mismatch and pom dependency mismatch

- ietf-packet-fields.yang
has been move to MDSAL
is using a newer version of ietf-yang-type

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

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"