Merge "Bug 3540: Implementation of FlowIds based on match data"

Merge "Bug 3607: Removing OpFlex from Sonar coverage."

Merge "BUG3619 - css mismatch with dlux"

BUG3619 - css mismatch with dlux

Change-Id: I15fe5a8bf9076a529ba11fe669b51dcc7bb8e007
Signed-off-by: Daniel Malachovsky <dmalacho@cisco.com>

Bug 3607: Removing OpFlex from Sonar coverage.

Change-Id: I2671696383ef89be351b35caa9d148d1071a0be8
Signed-off-by: Thomas Bachman <tbachman@yahoo.com>

Bug 3606: Fix features so UI pulls in backend.

The UI feature should pull in the backend feature. This
patch changes the UI feature so that it pulls in the
backend feature.

Change-Id: I281349277c09f708ed0db138b7059d2d36a8073e
Signed-off-by: Thomas Bachman <tbachman@yahoo.com>

Bug 3540: Implementation of FlowIds based on match data

Change-Id: Id5766bfbe3c065d5d4816402be73d79cee8f3871
Signed-off-by: Konstantin Blagov <kblagov@cisco.com>

BUG3582 - GBP UI - Subject detail in the Delivered policy
BUG3583 - GBP UI - Different element design in jointjs
BUG3584 - GBP UI -Positioning of elements
BUG3587 - GBP UI - missing links in expressed policy

Change-Id: Ic7606e6161416df15189f661639e8e22c11922b7
Signed-off-by: Daniel Malachovsky <dmalacho@cisco.com>

Merge "groupbased policy ui-rework"

Bug 3269 - Neutron External Gateway not functioning with floatingIP

Adding processing of NAT and External tables in OfOverlay

Change-Id: Ief7bf77fc2e805515488efb533ba1901dff63b59
Signed-off-by: Keith Burns (alagalah) <alagalah@gmail.com>

Bug 3243 increased test coverage #3

EndpointManager

Change-Id: Ibd034c26e9f5deaefa0f6cf36f5220021c0f916c
Signed-off-by: Kinsey Nietzsche <knietzsc@cisco.com>

groupbased policy ui-rework

reworked ui - pushed provisioning part to background
created governance part
-expressed policy
-delivered policy
-renderer state
-renderer configuration
-jointjs load fixed
- fixed design issues

created placeholders for other parts

Change-Id: I7a61934268e37899064be4c882206a2cbd1b7708
Signed-off-by: Juraj Sebin <jsebin@cisco.com>
Signed-off-by: Daniel Malachovsky <dmalacho@cisco.com>

Bug 3242: Tenant CONF -> OPER validation in PolicyEnforcer pattern not returning correct result

Change-Id: I05413d84ad1a7c6fbe7d0d07b39fe448b7f67186
Signed-off-by: Andrej Kincel <akincel@cisco.com>
Signed-off-by: Peter Žeby <peter.zeby@pantheon.sk>

Bug3318 removed groupbasedpolicy-openstackendpoint-config

Change-Id: I28ade2169917ff6cf62766f5607369df8d442761
Signed-off-by: Kinsey Nietzsche <knietzsc@cisco.com>

Incrementing versions by 0.1.0 for post-lithium master branch

Change-Id: Ie3302d76efa113ff30be151e4ed00e9062365f64
Signed-off-by: Thomas Bachman <tbachman@yahoo.com>

Bug3269 - Support for L3PrefixEndpoint and external endpoints in OfOverlay

Change-Id: I1cc40a963eca9b2a3f2ac2f334f7443c9a90e3cc
Signed-off-by: Keith Burns (alagalah) <alagalah@gmail.com>

PortSecurity changes for looping off EP instead of EPG.

Also adds support for external port.

Change-Id: Iafc3a961ef3ba9f60434ef8b2b65ce45b085fd1d
Signed-off-by: Keith Burns (alagalah) <alagalah@gmail.com>

Fixed issues with Inventory augmentations being over-written (ExternalInterfaces/Tunnel)

Externals
Neutron-ovsdb: InventoryHelper is writing these to CONF
SwitchManager is reading these from OfOverlayNodeConfig which comes from
CONF but it handles these inside a loop that is processing things from
FCNC.OPER.

Tunnels:
Neutron-ovsdb: InventoryHelper is writing these to OPER (which you never
see in POSTMAN… weird but squirrel).
SwitchManager handles these in two places: Looping on FCNC.OPER looking
for “*vxlan*", and processing OfOverlayNodeConfig from CONF.

Change-Id: I12a5738e3832228032368f0b5b45dcb050023a16
Signed-off-by: Keith Burns (alagalah) <alagalah@gmail.com>

Possible fix for OfOverlay overwrite (#2).

This is an attempt to fix an overwrite of data
in the Inventory Node's OfOverlay augmentation.
It changes the writes of the entire augmentation
to just a write of the tunnel elements.

Change-Id: I5a22dd31381558300544dfb3fcc154f18d71e337
Signed-off-by: Thomas Bachman <tbachman@yahoo.com>

Yang models for internal system health and fault aggregration.

- Instrumentation, leveraging data store instead of log to generate sane log messages.

Change-Id: I5e7eafd8a7f527c77fd5f47b3bc41b65b8cd0ed5
Signed-off-by: Keith Burns (alagalah) <alagalah@gmail.com>

Removal of OpenStackEndpoint YANG

Change-Id: I9be44f2bf640fa59026de2d7935bf61a40078ea6
Signed-off-by: Keith Burns (alagalah) <alagalah@gmail.com>

Merge "Bug 2814 - logical OR to AND in classifier"

Remove OpenStackEndpointGBP

- not needed. Openstack GBP API integration uses standard API and was back-ported to OpenStack Juno.

Change-Id: I1007410cdc6eabba5afdc362458aeddc4bb6bee4
Signed-off-by: Keith Burns (alagalah) <alagalah@gmail.com>

Bug 2814 - logical OR to AND in classifier

Fix of the logical OR behaviour within a rule and update of error messages accross all the
classifiers.

Change-Id: Id1e5157915772e13fcc5ff431321c40d3863ffdc
Signed-off-by: Tomas Cechvala <tcechval@cisco.com>

Added new FlowTables (NATMapper, ExternalMapper)
- IpPrefix support in DestMapper
- Implemented tableOffset as adding new tables that need skipping for NAT

Updated neutron-ovsdb to make NeutronGbpMapperListener

Added L3Endpoint for External Gateway Map

Change-Id: I1cdf13750ca3d28ab6bda6718e8983e95af44331
Signed-off-by: Keith Burns (alagalah) <alagalah@gmail.com>
Signed-off-by: Martin Sunal <msunal@cisco.com>

Merge "GBP UI fixes"

Changing of IidFactory

- solves compilation error

Signed-off-by: Martin Sunal <msunal@cisco.com>

Merge "Bug 3388 Added DNS and ICMP rules for DHCP EPG"

Merge "Bug 3269 Implementation of floating ip in neutron-mapper"

Merge "Bug 3359 - Duplication of methods for IID factory"

Bug 3388 Added DNS and ICMP rules for DHCP EPG

Change-Id: I7531ce8e391003ce415f6adb0ab2e52baece000e
Signed-off-by: Martin Sunal <msunal@cisco.com>

Bug 3243 increased test coverage #2

 - increased test coverage in groupbasedpolicy base
 - FIX PolicyScope.removeFromScope(

Change-Id: I5551736c8cdd81a9ebd270e2b4421454a0af0af9
Signed-off-by: Kinsey Nietzsche <knietzsc@cisco.com>

Merge "Bug 3051: Fix Unit Tests for IpPrefix."

Merge "Bug 3173 - group flows rendering fix."

Merge "Bug 3281 duplicate flow programming"

Merge "Removal of OpenContrail Renderer and config."

Bug 3359 - Duplication of methods for IID factory

Change-Id: Icbf2babc99aec5fda99f85fc392e62256777d372
Signed-off-by: Kinsey Nietzsche <knietzsc@cisco.com>

Merge "Adding EIC."

Bug 3269 Implementation of floating ip in neutron-mapper

- creates an endpoint representing floating ip port
- creates floating-ip-association-mappings
- creates endpoints-by-floating-ip-ports and floating-ip-ports-by-endpoints mappings
- contains workaround becaue of https://bugs.opendaylight.org/show_bug.cgi?id=3368

Signed-off-by: Martin Sunal <msunal@cisco.com>

Fix sonar test coverage reporting

 - added sonar-jacoco-listeners dependency - picks up jacoco report and
   transforms it into sonar report
 - modified jacoco-maven-plugin - so that sonar is able to pick up jacoco report

Change-Id: Iebd90d8afe96f9255ec11d427de970791316ff42
Signed-off-by: Michal Polkorab <michal.polkorab@pantheon.sk>

GBP UI fixes

Fixed labels in topology
Selecting topology when selecting items from right sidebar
Reorder buttons under Policy - moved L2L3 to bottom
added back and home buttons in #ff7300 color
network containment field is now selectbox
added required fields into endpoints
tenant added to registering EP
fixed access model wizard
fixed access model wizard design

Change-Id: Ie07073902959bacbb918530a7d1539bdc58b94d9
Signed-off-by: Juraj Sebin <jsebin@cisco.com>
Signed-off-by: Daniel Malachovsky <dmalacho@cisco.com>

Adding EIC.

This includes Endpoint Constraints in policy
resolution.

Change-Id: I29ed8014f598683f37174289ce66190a319c2775
Signed-off-by: Martin Sunal <msunal@cisco.com>
Signed-off-by: Thomas Bachman <tbachman@yahoo.com>

Bug 3243 increased test coverage #1

- increased test coverage in groupbasedpolicy base
- FIX EgKey.compareTo()

Change-Id: I189169fe1be6e60ed781c016d707f04429a1f950
Signed-off-by: Kinsey Nietzsche <knietzsc@cisco.com>

Bug 3269 Floating IP association in a model

Change-Id: Ieaab358da8f2b37712ec31d5064954d60aa6d30d
Signed-off-by: Martin Sunal <msunal@cisco.com>

Add Endpoints for external gateway and subnet.

- Endpoint for Router Gateway Interface
- L3PrefixEndpoints for subnet routes
- Subnet map for external gateways
- Check for port updates without tenant.

Change-Id: I8bc8825abb1f3c2972b79c73a434ee0047951948
Also-by: Martin Sunal <msunal@cisco.com>
Signed-off-by: Keith Burns (alagalah) <alagalah@gmail.com>
Signed-off-by: Martin Sunal <msunal@cisco.com>

Add underlay external port for inventory to OVSDB listener.

- leverages DevStack writing "other-config" for provider network and physical port.
- this it to determine egress from OfOverlay (external)

Change-Id: If346089a16d1cfb7d08e98499ed0555f02bd5b4f
Also-By: Thomas Bachman <tbachman@yahoo.com>
Signed-off-by: Keith Burns (alagalah) <alagalah@gmail.com>
Signed-off-by: Thomas Bachman <tbachman@yahoo.com>

Centralising DataStore and IidFactory.

- Needed this for L3PrefixEp.
- Can move other sub-project general helpers here
- Potentially can be structured as
-- General (all GBP)
-- Feature specific packages

Change-Id: I627b7ec6dcf82eda8ecb9b251e76f4ea77582ac3
Signed-off-by: Keith Burns (alagalah) <alagalah@gmail.com>

Example of adding parameters with config subsystem.

Change-Id: Idae0fcb0960345444a55bbd2b5013439493f53e9
Signed-off-by: Ed Warnicke <hagbard@gmail.com>

Bug 3281 duplicate flow programming

implemented Guava's Equivalence for Flow

Change-Id: I35dab16646d4da638bb8ebfe93cedea6659c1874
Signed-off-by: Konstantin Blagov <kblagov@cisco.com>

Bug 3173 - group flows rendering fix.

Fixed issue with missing group flows in POC. No need to reset connection between
ovs switches and controller anymore.

Signed-off-by: Tomas Cechvala <tcechval@cisco.com>

Removal of OpenContrail Renderer and config.

Multiple emails to Juniper have gone unanswered regarding:
- intentions to participate in Lithium
- documentation
- ongoing support.

We cannot have code we cannot support or test in the project for release.

Change-Id: Ie1c32e5c010c1f57cc26cc71c9f1f53e5752885b
Signed-off-by: Keith Burns (alagalah) <alagalah@gmail.com>

Merge "Adding neutron-ovsdb."

Bug 3051: Fix Unit Tests for IpPrefix.

This fixes the GBP unit tests that would
fail when the fix for Bug 3051 is mergedd
in yangtools.

Change-Id: I46eb019d55ea9e439ee7e14085f4a4b29538e6bc
Signed-off-by: Thomas Bachman <tbachman@yahoo.com>

Merge "Use StringBuilder instead of StringBuffer"

Adding neutron-ovsdb.

This adds a new listener for OVSDB southbound changes,
and uses that state to update state in the endpoint
registry, as well as create tunnel ports, if absent.

Change-Id: I0f5b09c684e10e77ab7845399f903f55e60ceba6
Signed-off-by: Thomas Bachman <tbachman@yahoo.com>
Signed-off-by: Keith Burns (alagalah) <alagalah@gmail.com>

Change Tunnel type in ChainAction

Change-Id: I2a61d4c7614f7fe1813a5f3ea94db09396de2000
Signed-off-by: Keith Burns (alagalah) <alagalah@gmail.com>

Use StringBuilder instead of StringBuffer

StringBuffer is overly synchronized, use a StringBuilder instead.

Change-Id: Ifb18d3ebb2ae464f5e634aaee8c7d969a98256de
Signed-off-by: Robert Varga <rovarga@cisco.com>

GBP UI initial commit

added GBP UI dlux module
added groupbasedpolicy-ui feature and bundle

Change-Id: I0ee3cb740d6a2164122a66eb6f4e04f61071e563
Signed-off-by: Juraj Sebin <jsebin@cisco.com>

Implement SFC integration

- Query based on Chainname
- Validate Chain before moving from CONF to OPER
- Added ChainAction
- Added things in SwitchManager/EndpointManager to modify READY state so we don't write flows to switches
  with no endpoints

Change-Id: I84d2549a444f10ddabed146f857af38d0e6369c0
Signed-off-by: Keith Burns (alagalah) <alagalah@gmail.com>

Getters and setters for boolean

Change-Id: Id5833518ca860f49de0243bc58c866cc1f6fcee5
Signed-off-by: Keith Burns (alagalah) <alagalah@gmail.com>

Bug fix for new mapping table.

The new mapping table needs to have the parent
nodes created.  This adds the true flag to cause
that to happen in the puts to the data store.

Change-Id: I2fe7477bc9c93d74a36c81213a1b3fac44636f6e
Signed-off-by: Thomas Bachman <tbachman@yahoo.com>

Fix of NPEs and attempt to fix Neutron mapper.

- Added utilities to only put Switch in READY when Endpoints attached.

- Also removes switch from ready when last EP is removed.
  This means we don't write and handle all OF switches connected to
  controller unless an EP is present. It means we play better with others.

Change-Id: I3b3e66259e2e43a201d298dd8af523cc5a3cf426
Signed-off-by: Keith Burns (alagalah) <alagalah@gmail.com>

Merge "Implementation of INeutronFloatingIPAware introduced"

Merge "Fixed missing tunnel type"

Implementation of INeutronFloatingIPAware introduced

Tenant NPE temp fix - REMOVE BEFORE PUBLISH
identification of router-gateway

Change-Id: Ibc84fcee36f7604fc0794f456bf153f21e52c56e
Signed-off-by: Martin Sunal <msunal@cisco.com>

Added neutron port to endpoint mapping and vice versa

Signed-off-by: Martin Sunal <msunal@cisco.com>

Fixed missing tunnel type

Change-Id: I23d5b1cad48f132f632da4e5b038953f0b9e0d32
Signed-off-by: Martin Sunal <msunal@cisco.com>

Merge "Moved assignment of TunID to SourceMapper"

Fix javadoc JDK8 compatibility

JDK8 is more picky about javadoc format, so fix them up to pass
compilation with JDK8.

Change-Id: Ib79d332135f4a29f98a1461a17ab86118d1a2678
Signed-off-by: Robert Varga <rovarga@cisco.com>

Merge "Removed calling of NeutronRouterAware API from NeutronPortAware"

Merge changes I8490a721,I71941875

* changes:
  Bug 3166 ui-backend introduced
  Moved util methods from PolicyResolver

Removed calling of NeutronRouterAware API from NeutronPortAware

Devstack added support of sending notification about CUD of router interface.
So hack in NeutronPortAware is not needed anymore.

Signed-off-by: Martin Sunal <msunal@cisco.com>

Bug 3166 ui-backend introduced

This is a backend based on RESTCONF.
Purpose of the backend is to provide data which has to be calculated by difficult processing
i.e. it reuses policy resolution process from groupbasedpolicy module.
UI can call RPCs on this backend.

Backend provides these RPCs:
 - get-endpoints-from-endpoint-group: Returns endpoints which are
        in the given endpoint group.
 - get-subjects-between-endpoint-groups: Returns subjects between
        endpoint group pairs where endpoint groups are from the given tenant.

Change-Id: I8490a7217c5725ccdfc6dda3467d3ba91c76eeb9
Signed-off-by: Martin Sunal <msunal@cisco.com>

Merge "Revert "Revert "Added feature dependency on odl-ovsdb-southbound-impl"""

Revert "Revert "Added feature dependency on odl-ovsdb-southbound-impl""

This reverts commit 8fd49cd72a4319e0cb4ceacbf6f77d2e820cfaa0.

Change-Id: Ica226425eeae14d952be136fb8eb6e2151bdc90b
Signed-off-by: Keith Burns <alagalah@gmail.com>

Moved util methods from PolicyResolver

Most of methods in PolicyResolver are util mehtods
so these methods are moved to new *ResolverUtils classes.

Signed-off-by: Martin Sunal <msunal@cisco.com>

Moved assignment of TunID to SourceMapper

- TunID determined by SourceEP ordinals
- Removes need for matching on Source generated reg values in DestinationMapper
- Incredibly useful for SFC when EPs are both on same node.

Change-Id: I6598f7b10edbe4624694e884f760cfb894ca31ad
Signed-off-by: Keith Burns (alagalah) <alagalah@gmail.com>

Merge "Fixed main issues of SwitchManager"

Fixed main issues of SwitchManager

SwitchManager is feeded from listeners
    FlowCapableNodeListener - FlowCapableNode from OPER DS on BASE scope of OF augmentation
    OfOverlayNodeListener - OfOverlayNodeConfig from CONF DS on BASE scope of OF overlay augmentation
    FlowCapableNodeConnectorListener - FlowCapableNodeConnector from OPER DS on BASE scope of OF augmentation

SwitchState is composed based on listeners' data:
    FlowCapableNode, OfOverlayNodeConfig, and FlowCapableNodeConnector-s
SwitchManager keeps track of SwitcheState-s in Map.

SwitchState can have 3 types of status:
  DISCONNECTED - SwitchState does not contain FlowCapableNode.
  PREPARING - SwitchState contains FlowCapableNode but information about tunnel(tunnelPort, tunnelIp) is missing
  READY - SwitchState contains FlowCapableNode, tunnelPort and tunnelIp

Even status of SwitchState is DISCONNECTED the SwitchManager still keeps track
until SwitchState contains configuration. This is because FlowCapableNode can disappear from OPER DS but
additional OfOverlayNodeConfig data are still in CONF DS.
Without this tracking we would loose OfOverlayNodeConfig data when SwitchState had been removed.

Change-Id: I7781fdfb0e306568587a1c69c2ab9f90e205180d
Signed-off-by: Martin Sunal <msunal@cisco.com>

Removed "reverse" from policy resolution process

Reversed EPG roles(cons/prov) based on ID complicates debugging.

Signed-off-by: Martin Sunal <msunal@cisco.com>

Revert "Added feature dependency on odl-ovsdb-southbound-impl"

This reverts commit aa03a46446c4f267fc8f88f109cae13b864b30bb.

Change-Id: I38200796d8f01ea18528f95e4b12f7f839eb983d
Signed-off-by: Keith Burns <alagalah@gmail.com>

Merge "Fixed EIC grouping"

Merge "Revert "Create the tunnel port in OVSDB, and connect.""

Revert "Create the tunnel port in OVSDB, and connect."

This reverts commit 967912af4cdcf89df665579708af27c51c808d9b.

Change-Id: I98ede9979ca8b0ce6eab26439ea9c649d975a714
Signed-off-by: Thomas Bachman <tbachman@yahoo.com>

Fixed EIC grouping

Signed-off-by: Martin Sunal <msunal@cisco.com>

Merge "Removing legacy distribution."

Create the tunnel port in OVSDB, and connect.

This patch creates a tunnel port for ofoverlay, and
optionally connects it to the controller (set by member
in the class).

Change-Id: Iac2f099cbe3521c8b9d53bf55752264543e4e22d
Signed-off-by: Ed Warnicke <eaw@cisco.com>

Added feature dependency on odl-ovsdb-southbound-impl

Change-Id: I955a6695d98432855fc008145ddc4bc822fd4d26
Signed-off-by: Ed Warnicke <eaw@cisco.com>

Removing legacy distribution.

This removes the (legacy) distribution build from
the repo, as it is no longer used and/or needed.

Change-Id: Id1481e6c90b8eebbaef5022c4dac26be8614aa17
Signed-off-by: Thomas Bachman <tbachman@yahoo.com>

Set odlparent, clean-up of pom.xml-s

Change-Id: I76ba8a551755ca30347fba59750ec0d7d02a1dad
Signed-off-by: Martin Sunal <msunal@cisco.com>

Backed out SFC Manager.

SFC Manager is removed -- will be replaced by
a subsequent patch.

Change-Id: Ifaba8e77eac8da7428ec959cf02b69fa4d5f9471
Signed-off-by: Thomas Bachman <tbachman@yahoo.com>

Merge "Move JSON-RPC lib to OpFlex."

Move JSON-RPC lib to OpFlex.

The JSON-RPC library is only used by the OpFlex
renderer. This patch moves the library under that
renderer for now (can be moved to a more general
place if it's needed by more than one renderer).

Change-Id: Icbefa208b63b0a36967f0834d7e7fbcf14bd4ac2
Signed-off-by: Thomas Bachman <tbachman@yahoo.com>

Fix order of base feature loading.

The order of the features in the features.xml file is
important -- this patch fixes the order so that the
ofoverlay feature comes up correctly.

Change-Id: Idf7c5e9ce7c026a67f7dd3c913f35abcea979119
Signed-off-by: Thomas Bachman <tbachman@yahoo.com>

Fixes for FlowTable in OFOverlay

- Added tunnel ordinal, removed VNID->sEPG
- Fixed FDID tunnels
- Remapped DestMapper to set above VNID
- Fixed bug in SourceMapper (object identity vs equality test)
- Assured that only BD-BD communication allowed for L2 Flows
- Set foundation for fixing conditions and implementing distributed
policy enforcement.
- NeutronMapper: set DHCP EPG to IGP Deny

Change-Id: Ic66e56b25700cea6a68815b86175e7909482dab9
Signed-off-by: Keith Burns (alagalah) <alagalah@gmail.com>

Merge "Routing fix in preparation for routing table support."

Merge "Parameter of classifiers changed in POC"

Routing fix in preparation for routing table support.

Need to refactor to ensure that local routing occurs with source subnet gateway,
as well as final destination MAC.

Change-Id: I1dc25728b2750eb273bf31a08401759b931fa952
Signed-off-by: Keith Burns (alagalah) <alagalah@gmail.com>

Additional Enhancements to the Endpoint RPC Registration
1- Allowed RPC registration to have its own dedicated thread pool since
   its shared among multiple renderers.
2- Made renderers only return their augmentations to RPC registration
   without being involved in building endpoints (which only handled by
   the RPC registry itself).

Change-Id: I5075404f2137ee5cb6df5bd5479dd8138ee4bb5c
Signed-off-by: Khaldoon Al-zoubi <khalAlzoubi@gmail.com>

Fixing concurrent access errors.

Change-Id: I3e7ba5760d396a646ab37e1e832f6ef2435779ba
Signed-off-by: Keith Burns (alagalah) <alagalah@gmail.com>