Sfc-ui patch set based on feedback (I, II, III)

+ Added webconsole as boot feature for sfc-karaf

+ SFF set as default view on UI access
+ SN topology showing data from SFF as default
+ Table view for service-function-type
+ RESTconf URL kept in storage of web browser persistently
+ System info tab: karaf features + karaf logs
+ Top menu responsive - in future should be even more improved
+ Fixed UI layout for low-res screens

Change-Id: If10de42271e47ef888c417afb433dcd602065a60
Signed-off-by: Andrej Kincel <[email protected]>

imported nsh_encode and removed build_packet from service_classifier
added parameters into build_packet()

Change-Id: Ib26795461d7ef6ee7b8829d1c00170dc5fa2e51c
Signed-off-by: Jim Guichard <[email protected]>

initial python service classifier basic implementation

Change-Id: I835d43604a1e16a5ed190222df591d5a30a6393a
Signed-off-by: Jim Guichard <[email protected]>

Add dependency on odlparent

Refers to odlparent as the upstream parent, thus bringing in the
third-party definitions such that they match the larger project-wide
settings.

Signed-off-by: Robert Varga <[email protected]>

Introduced Rendered Service Path

One of the problems with the current code is that the constructed
service path is written in the config data store by merging SFF/SF
information with config provided by the user. This is very bad and I
tried to avoid changing this but it needs to be done if we are serious
about SFC.

The problem is that after SFC provider modifies the SFP with data from
SFF, SF, etc and commit its, it triggers the listener again, which
recreates the path again, which triggers the listeners again, in an
infinite loop. Today I break the loop by using a trick but it makes
the code fragile, specially in the face of modifications, as opposed
to SFP creations.

So, thinking about making the code rock solid, this is what I will be
working on (let me know feedback):

- I will introduce a new yang model called “Rendered Service Path”.
The IETF architecture document actually has this construct. This will
be an    operational yang model. Meaning it can not be changed by the
user.

- When user creates a SFP, provider will include SFF/SF
information and save it in a RSP.  This will not trigger any listener
and information will    be protected from RESTconf.

- Modificationsto SFP by the  user will carry to RSP but not
vice-versa (of course). This makes the code much more reliable and
predictable.   It also makes it easier for the Python agent since
 there is no confusion about what is actually something it needs
 to act on (RSP) versus something
that might not be complete (SFP).

- Changed SF and SFF operational containers. Changed from leaf-list
  to list because it is error prone and tricky to update leaf-lists
  in ODL. It requires reading the list, copying, updating and
  writing it back. In the case of list a single indexe operations
  to add/delete/update is possible.

- Changed SFcProvider*API to use new DataStore APIs. This reduced a lot
  of repetitive code

- Removed dead code

Change-Id: I42cfda4c2a83c0e651523852c3c0dc064dbe7127
Signed-off-by: Reinaldo Penno <[email protected]>

Introduced strong synchronization

- Strong synchronization across listeners with Locks and synchronizaed
- Strong synchronization in checked future callback with semaphores
- Fixed several logs
- Added more javadocs
- Removed busy wait form SfcDataStoreAPIs

Change-Id: I4d58ea8c13be61b1fc6aac96e29400939a661b5c
Signed-off-by: Reinaldo Penno <[email protected]>

SB REST plugin & Karaf features
(port fixed after code review)

Change-Id: I44b0a086235557feda4a4d50b7560678624d0306
Signed-off-by: Konstantin Blagov <[email protected]>

Critical Bug fixes and Enhacements

- Added javadoc to many methods
- Introduced wrappers around datastore API. These wrappers take care of retries, logging,
  check whether transaction is in progress, etc
- Added more tests to Python dcloud regression. Now it checks for operational state
  correctness as well
- As much as possible changed most methods to use new datastore wrappers.
- Introduced rest:uti to SF Yang. This will allow python agent to manage SFs
- Removed dead code
- Fixed several conditions where operational state was not cleared after delete
- Fixed several concurrency issues
- Fixed SFcDataStore callback to take into account transactions in progress
- Fixed logging in multiple places
- Added more documentation to Python SFC agent and SFF thread

Change-Id: If90a0c45ed928b59a5f4634c18ed2819c6723c59
Signed-off-by: Reinaldo Penno <[email protected]>

Bugs and Enhancements to SB REST and SFF/SF Data Plane

- Expanded and cleared considerably the Python refernce implementation.
  - There are now mseparate modules for NSH encode, decode
  - Module for SFF client
  - Self contained module for SFF
  - Module of SFF with REST
  - An agent based module that creates SFF on the fly, meaning no
    pre-configuration needed. This will probably become the future
    of data plane side ODL SFC.
  - In next commits will enahnce agent to create SFs on the fly as well.
  - SFF now uses a dictionary indexed by sfp-id and sfp-index to
    lookup next SF/SFF. This is an considerable improvement over the
    preivous method.

 - Fixed an issue on the SB REST where SFP were not communicated down
 - Fixed an issue where SFPs were deleted more than once. Not really
   a biggie given DELETE is idempotent but still worth it.
 - Service locator Yang now can be configured with a local function

Change-Id: Ic89f175781f29eea7b2aeef7e9e748cc624be6bb
Signed-off-by: Reinaldo Penno <[email protected]>

Merge "Sfc-UI classifier functionality"

Merge "Sfc-provider classifier functionality"

SFF REST server that configures SFF data plane implementation

Fixed a few other bugs on reference SFF implementation

Change-Id: Ie4342470dee8b2981a85fa5d4ff89a19a21fde96
Signed-off-by: Reinaldo Penno <[email protected]>

Bugs in SFF refernece implementation

- When decoding NSH v3 packets, there were a couple of
  off-by-one erros

Change-Id: I110f8e2732034c6f85899efdaf03d8710928d93d
Signed-off-by: Reinaldo Penno <[email protected]>

Merge "Changed messages based on list feedback"

Changed messages based on list feedback

Changed and clarified some messages from error to warning or info.
  The conditions were not errors.

Changed REST abstract class exception handling to differentiate
 between REST Server not responding and invocation exception.

Change-Id: I7349f1d0c4fa3ceabfe1d7c1c006d12b269e40ab
Signed-off-by: Reinaldo Penno <[email protected]>

Merge "Several bug fixes and Python SFF/SF reference implementation"

Several bug fixes and Python SFF/SF reference implementation

Introduced a CheckedFuture callback implementation
Started using CheckedFuture for some transactions
Fixed a bug with SB REST where the URI was not extracted from model correctly
Fixed handling of exceptions in SB REST code
Added LOGs to several transaction
Added a new directory and Python reference implementations of SFF/SF
Fixed minor bugs with SFC python regression

Change-Id: I56811650900eb44818ae144368969bb01a8d5274
Signed-off-by: Reinaldo Penno <[email protected]>

Sfc-UI classifier functionality

+ UI modified according to changes proposed in commit:
  Sfc-provider classifier functionality

Signed-off-by: Andrej Kincel <[email protected]>

Sfc-provider classifier functionality

+ classifier yang model change
+ classifier entry listener
+ basic API for ACL

How it works:
Whenever Classifier is attached to SFP it creates mapping between SFP <-> ACL.
Then, Classifier entry listenter calls method from ACL API which will update ACL
entries to point to the SFP.

If it comes to change of ACL in Classifier, old ACL entries are unset,
thus are not pointing to the SFP anymore. The new ACL is updated
appropriately.

Change-Id: Iba179fedee05756dbd3101242a481866ddfcba24
Signed-off-by: Andrej Kincel <[email protected]>

SFC-UI - several fixes and improvements

+ Updated SFF table view
+ Fixes in SFF create/edit dialog
+ All select boxes can now be reset to no value selected (initial state)
+ Classifier create/edit dialog ehancements

Change-Id: Ib2cfaa097644bd1ccc63591f6c9cca5cb3b1b099
Signed-off-by: Andrej Kincel <[email protected]>
Signed-off-by: Martin Lauko <[email protected]>

SFC-UI - adapted to new YANG models

+ Support for classifier CRUD operations
+ Classifier is now attachable to SFC
+ SFF adapted to new YANG models
+ SFF now supports OVS augmentation
+ various bug fixes and improvements

Change-Id: I51f5625bf845be681a2f1407d976f182bedc05da
Signed-off-by: Andrej Kincel <[email protected]>
Signed-off-by: Martin Lauko <[email protected]>

SFC-UI - SFP support for context metadata

+ Context metadata can be attached to existing SFP
+ Bugfix for ACL create/edit dialog
+ Improvements in SFC create dialog

Change-Id: Ibb7155b8f0395b3908f5d4e70cee6993894b9b77
Signed-off-by: Andrej Kincel <[email protected]>

SFC-UI added metadata functionality

+ new tab NSH metadata
+ possibility to define context/variable metadata
+ attachment to SFP/ACL is prepared but disabled at the moment
  (because of missing YANG models)

Change-Id: Ief2b7dae731b92d928ee04211a61e14c48e9b316
Signed-off-by: Andrej Kincel <[email protected]>

Fixes to SouthBound REST

    - When a SFF is created/modified, we send to the REST URI founds in the
      SFF config ("rest-uri") instead of a generic URI
    - When a SFP is created/modified, we send only to the SFFs found in the
      service-hop list. For this we also use the rest-uri found in the SFF
      config

Change-Id: I8bd4905c6ededb9fb5003adaa00bca338673c8a1
Signed-off-by: Reinaldo Penno <[email protected]>

Fixes and Enhancements to REST and Generics API

Signed-off-by: Reinaldo Penno <[email protected]>

More fixes and Robustness changes

Signed-off-by: Reinaldo Penno <[email protected]>

Several Fixes in the REST code and Listeners

Signed-off-by: Reinaldo Penno <[email protected]>

Fixed bootstrap files

+ added order to service-function-chains
+ updated boostrap files for sfc-distribution
 (service-function-forwarders were loaded properly in karaf but not in
 sfc-distribution)

Change-Id: Ia5aa80c95b0bfdf1cfbfeb42d3bb3b07b3e76f2b
Signed-off-by: Andrej Kincel <[email protected]>
Signed-off-by: Reinaldo Penno <[email protected]>

Modified yang models, setup scripts and provider

Signed-off-by: Reinaldo Penno <[email protected]>

Modified SFF OVS Model and Python scripts

- Changed SFF and SFF OVS models after studying dcloud setup
  Now, each sff data plane locator is augment to have bridge name
  and other OVS data. Same for sff-sf data plane locators
- Modified SFF bootstrap config to account for change
- Created Python script to setup all dcloud config in ODL in a single sweep.
- Modified gitignore to ignore pyc files

Signed-off-by: Reinaldo Penno <[email protected]>

New Classifier Model

- Classifier model
- Metadata model reference in SFP
- Removed old model directory

Signed-off-by: Reinaldo Penno <[email protected]>

Typdefs for Metadata Model

Signed-off-by: Reinaldo Penno <[email protected]>

Metadata Model

- Any number of Context headers and TLV headers can be
  build independently
- Headers can be attached to SFPs

Signed-off-by: Reinaldo Penno <[email protected]>

Merge "Enabling Acl entries for ping"

Merge "SFC-UI - added classifier functionality"

SFC-UI - added classifier functionality

+ added ACL (classifier) tab
+ added support for all types of service locators
+ allowed addition of plain SFF to SFP
+ validation is now showing incorrectly filled inputs
+ ipv4/ipv6 validation now supports ip/mask CIDR notation
+ fixed modal dialogs css (is not stretching the window anymore)
+ added new tests
+ various bug fixes and improvements

Signed-off-by: Andrej Kincel <[email protected]>
Signed-off-by: Martin Lauko <[email protected]>

big 2133: odl-mdsal-apidocs need to be part of SFC Karaf distribution

Change-Id: I850b25843dd04b288ef3f9e8ebc749dcdd969f6b
Signed-off-by: Paul Quinn <[email protected]>

Overall hardening of the code

- Overall hardening of the code
- Expansion of regression testing

Change-Id: Ib62aae644d85e47ee8c14fc8566556db413082aa
Signed-off-by: Reinaldo Penno <[email protected]>
(cherry picked from commit 0ed2f0614b0076e3fbe22b2beedae52df0b9c28e)

Incrementing versions by 0.1.0 for post-helium master branch

Change-Id: I1ca0a618b1a412f847fdfa91a2304c5be8730d3d
Signed-off-by: Reinaldo Penno <[email protected]>

Enabling Acl entries for ping

Change-Id: If96e6264e8a97f7e2e4b0ea5b6ed264d046e38b1
Signed-off-by: Shuva Jyoti Kar <[email protected]>

Enabling subnet-mask as input

Change-Id: Ibd1f4a2e077f7dfc096854f08b62537fe2584165
Signed-off-by: Shuva Jyoti Kar <[email protected]>

Acl-flow fixed
Signed-off-by: Shuva Jyoti Kar <[email protected]>
Change-Id: I10a5db61e73502b27644448310bb2fd571763e72
Signed-off-by: Shuva Jyoti Kar <[email protected]>

Bug 2146: SFC broken due to new auth requirement (correct fix)

HTTPBasicAuthFilter added to jersey REST client
It attaches Authorization: Basic YWRtaW46YWRtaW4= to HTTP header
It means that username and password is hardcoded to "admin", "admin"

Change-Id: Ib164a8736c5793a670b55347b703f249bdd63e66
Signed-off-by: Andrej Kincel <[email protected]>

Support for multiple data-plane locators

Change-Id: I0f04145d238298a9ec4bddf9102b851e361c7787
Signed-off-by: Shuva Jyoti Kar <[email protected]>

Disabling egress transport table

Change-Id: Id08686d0083c52f21b8c4abd19fa918ca656e4bf
Signed-off-by: Shuva Jyoti Kar <[email protected]>

EthType matches not required
Signed-off-by: Shuva Jyoti Kar <[email protected]>

Getting rid of vlan_vid/different priorities flow
Signed-off-by: Shuva Jyoti Kar <[email protected]>
Change-Id: I7c61fdc3c527b4ad5f2e7f37527d2a7c562b9163
Signed-off-by: Shuva Jyoti Kar <[email protected]>

Resolved NoClassDefFoundError at runtime
Signed-off-by: Shuva Jyoti Kar <[email protected]>
Change-Id: I28ecf0139191f945d68a4be1185cc6167149d1bb
Signed-off-by: Shuva Jyoti Kar <[email protected]>

Multiple Mac matches to address multiple chains

Change-Id: Iae7312446ff1ab9b646c7ebd62bbd4cf03e03f91
Signed-off-by: Shuva Jyoti Kar <[email protected]>

Unit tests support for sfc-ui

karma-unit.tpl.js - config for karma test runner

package.json - config for npm package manager (contains packages required
to execute tests)

README.md - added step by step guide for karma installation and test execution

Change-Id: I19807a8a8404f7dfa1ffbef191ff0ec7970376a6
Signed-off-by: Andrej Kincel <[email protected]>

Merge "Fix to write mac/ip to avoid frm bugs Signed-off-by: Shuva Jyoti Kar <[email protected]>"

Merge "Unit tests for provider API"

Fix to write mac/ip to avoid frm bugs
Signed-off-by: Shuva Jyoti Kar <[email protected]>
Change-Id: I3f1971ad23c4ae2b58b52199407fcfa3933241f5
Signed-off-by: Shuva Jyoti Kar <[email protected]>

Bug 1948: Separate out restconf features

In order to avoid a maven project cycle in solving
Bug 1948, we need to separate restconf features.
Note, this is a first step, suffixing everything
with -new.  Subsquently, after everywhere using
odl-restconf has been fixed to use this new repo,
we will deprecate the ones in the mdsal features.xml
and rename these to not have the -new.

This patch just adds the dependency to features/pom.xml

Change-Id: I6de8b6bcaeec75eef41df14dbac58bbedec3a779
Signed-off-by: Ed Warnicke <[email protected]>

Unit tests for provider API

Change-Id: Ifef803488b660069ebb516c408bf88fbf0921c02
Signed-off-by: Konstantin Blagov <[email protected]>

Merge "serve index.html on sfc/ root directory request Activator.java code cleanup"

serve index.html on sfc/ root directory request
Activator.java code cleanup

Change-Id: Iad292289c8f5d8faaa6830b0c2e155b31489d722
Signed-off-by: malauko <[email protected]>

Bug fix pack for SFC-UI

Added missing angular-xeditable library (was not included because 'dist' is in .gitignore)
Fixed REST PUT requests
Fixed SFC/SFP table reload
Fixed SFC symmetric property
Fixed automatic addition of required SFF into SN

Signed-off-by: Andrej Kincel <[email protected]>, Martin Lauko <[email protected]>

Karaf final changes
Signed-off-by: Shuva Jyoti Kar <[email protected]>
Change-Id: Ia053649d19913d172dbc6993622ca4882a8cb27c
Signed-off-by: Shuva Jyoti Kar <[email protected]>

adding sfcofl2 karaf features

Change-Id: Id8e9e61bb990a5a253e9b13f97b0eebc78768317
Signed-off-by: ChrisPrice <[email protected]>

Bug 1867: fix to features files

Change-Id: I9acac68f8d9edee4875b252e46e6c0a0855d8b04
Signed-off-by: Ed Warnicke <[email protected]>

Merge "added missing bundle dependency"

added missing bundle dependency

Change-Id: I137487a5209a355a5be580742d54d737e96b6278
Signed-off-by: David Goldberg <[email protected]>

Merge "adding SFC features adding sfclisp"

adding SFC features
adding sfclisp

Change-Id: I3e4ac5a0fae056ca019a5475152b29fc0e6f890d
Signed-off-by: [email protected] <[email protected]>

Merge "SFC Python Regression Test for Karaf"

Added Karaf feature for SFC-UI

New odl-sfc-ui Karaf feature is added to odl-sfc-all.
SFC-UI is available on http://localhost:8181/sfc/index.html

This solution is only temporary because it's not using Dlux Karaf.
Instead it's only using dlux-web mvn bundle as dependency, then it inserts SFC-UI
project files into it creating merged SFC-UI bundle.

Signed-off-by: Andrej Kincel <[email protected]>, Martin Lauko <[email protected]>

Fix for missing features dependencies

fixing duplicate entries

Change-Id: Ie2229a4d273fec67dcb453a7c67a34b96a06ec1e
Signed-off-by: Ed Warnicke <[email protected]>

SFC Python Regression Test for Karaf

Change-Id: I5c59f4e298763df7eaea1f6a7383bd352296566d
Signed-off-by: Reinaldo Penno <[email protected]>

fixing bootfeatures: sfc-all and restconf load at startup

Change-Id: I055128ac8598c06d1064c6ad2a89e06b81707741
Signed-off-by: paulq <[email protected]>

Fix for SFC bootstrap config files loading

Change-Id: If54e22d65957d0162e032bf7ecee55902de860a7
Signed-off-by: Andrej Kincel <[email protected]>, Martin Lauko <[email protected]>

inital sfc feature for karaf integration

Change-Id: I9fd54bdb56f9df245b9a6ed8dd0433d86592429a
Signed-off-by: [email protected] <[email protected]>
Signed-off-by: Ed Warnicke <[email protected]>

added lisp updater bundle for service functions and service function forwarders with lisp address

Change-Id: Id7bc013a8c87eeb0a3d75149eb37c4a240b70da6
Signed-off-by: David Goldberg <[email protected]>

SFC-OF-L2 implementation
Signed-off-by: Shuva Jyoti Kar <[email protected]>
Change-Id: I30bf3eece86290f54ffad6112a2245b92880904b
Signed-off-by: Shuva Jyoti Kar <[email protected]>

Revert "added lisp updater for service functions and service function forwarders with lisp address"

This reverts commit 2fd4810db9dc9a22e472a40da37bf0cc91cee760.

Revert "added lisp updater for service functions and service function forwarders with lisp address"

This reverts commit 2fd4810db9dc9a22e472a40da37bf0cc91cee760.

Gal, David,
I am reverting this patch.

As noted in the comments to patch:
https://git.opendaylight.org/gerrit/#/c/10669/
This patch breaks the existing sfc-provider.

You were specifically asked not to do so in
the review for 10699 to which you provided no response.

The existing agreed architecture is extremely flexible and
allows for multiple independent providers, as you can see
in the openflow provider here:
https://git.opendaylight.org/gerrit/#/c/10631/
and here the provider for the ovsdb folks here:
https://git.opendaylight.org/gerrit/gitweb?p=ovsdb.git;a=tree;f=ovs-sfc;h=8fa462c53b4c92b50a292ec3bc2a5b3e0ba22282;hb=HEAD
An additional provider from you guys that does something LISP
specific is very welcome, but breaking existing code the day
before code freeze is not.

Let me know if you need a hand getting your provider in and
a karaf feature for it.

Change-Id: Ie84ed0d4540af9e12ac20d55291871a133b7a793
Signed-off-by: Ed Warnicke <[email protected]>

added lisp updater for service functions and service function forwarders with lisp address

Signed-off-by: David Goldberg <[email protected]>

Fix for missing exports from sfc

sfc-model adds ietf-acl.yang and packet-fields.yang but does not export them so other bundles fail to resolve when loading since they can't find them.

org.osgi.framework.BundleException: The bundle "org.opendaylight.ovsdb.ovssfc_0.0.1.SNAPSHOT [137]" could not be resolved. Reason: Missing Constraint: Import-Package: org.opendaylight.yang.gen.v1.urn.ietf.params.xml.ns.yang.ietf.acl.rev140520; version="[0.0.0,1.0.0)"

Change-Id: Id1db0f3c6cd666c6e5f463e94f81a99304a4532c
Signed-off-by: Sam Hague <[email protected]>

Fix for SFC-UI initial commit

Change-Id: Ia68677f302494dd3b0a81e913c64f251b810e990
Signed-off-by: Andrej Kincel <[email protected]>

Initial commit for SFC-UI

It's just collection of all sfc-ui source files enclosed in maven bundle.
Needs to be added to oncoming SFC Karaf as feature.
Then this feature should be pluggable into DLUX project.

Change-Id: I8f7704bbeda6b6e1287c3c0eeb86983298b19a80
Signed-off-by: Andrej Kincel <[email protected]>

Bug Fixes and Enhancements

- Bug due to recent callable change: When using more than one executor concurrently
   we need multiple threads in the pool or system deadlocks
- Bug: external-ids in Openstack/OVS Yang needs to be a list
- Enhancement: Changed SFP Yang so that it is a collection of hops that are a combination of SF & SFF
- Bug: Changed SF yang so that it can contain a list of locators instead of single container
- Fixed initial config files accordingly
- Bug: SFP Data Listener needs to listen to scope BASE otherwise we create an inifite loop

Change-Id: I2243e82b530baf549685cbcb7722fcb7f4517e24
Signed-off-by: Reinaldo Penno <[email protected]>

Moved provider API to subpackage .api; solved minor scope issues

Change-Id: I88ea03f4ee075e0467cb79a57447ee9e0e6c6869
Signed-off-by: Konstantin Blagov <[email protected]>

API classes changed to Callable
Added put/read/delete methods
Applied new DataBroker transactions API
Added dependencies for unit testing
Outdated classes to instantiate a SFC moved to sub-package

Change-Id: I69758c6212e1c8be81cdefbac4e7ebda41ae786c
Signed-off-by: Konstantin Blagov <[email protected]>

Add openflow bundles to sfc to fix openflow problems

Change-Id: I79b2e130be04f1e83bdc2e8481370cc5b8f904ee
Signed-off-by: Sam Hague <[email protected]>

Fixed most reported issues and general improvements

ACL model now references SFP name
Modified Data Plane Locator group and used within SF, SFF and SF dictionary
Added trasnport type to Data Plane locator group
Added more necessary information to SFF OVS model
SFP model has service index for each SF and consequently we populate
  SI for each SF
Modified ACL model by inserting a list under top container
Modified original configuration files accordingly
Added newer ietf-yang-types dependency to POM
Modified Log configuration from DEBUG to ERROR
Modified SF model to indicate whether SF is nsh-aware

Change-Id: Ifcf83e0841bea7624a5d9e52e866f26abb6c21d1
Signed-off-by: Reinaldo Penno <[email protected]>

Added apply logback.xml configuration to Logback service on sfc-provider initialization
Added '.hg' to .gitignore

configuration: logger for sfc-provider to sfc.log file and send messages to syslog server

Change-Id: Ibdcfc59bf89f53e486aca0645cd03205a67b78aa
Signed-off-by: malauko <[email protected]>

Bug Fixes and More Fine Tuning of Yang Models

- Further decoupling of SFF and SF
- Fixed bugs as part of moving to new DataBroker
- Fixed bootstrap files to account for modified Yang models
- Fixed RPCs accordingly

Change-Id: Icf7ae1c4d645696044cc5564c9e16eb45970e0c9
Signed-off-by: Reinaldo Penno <[email protected]>

Decoupling of SFF and SF. More bug fixes

- Decoupling SFF and SF in a upcoming commit. Today when you create
  a SF, we create a SFF if it does not exist. But any coupling
  between the models is really bad because it makes us have to do
  garbage collection and synching of data. So, the user will have
  to explicitly create a SFF and in its configuration also specify
  the list of SFs attached to it.
- Changed sfc_provider_config.json to pretty format.
- Changed SFC chain model to allow symmetric paths
- Fixed a few Yang modules with lines larger than 70 chars.
- Fixed a few Yang descriptions according to new arch draft-01
- Fixed LOG statement in SnDataListener()
- Fixed service-function-forwarders.json to use new model syntax.
- Fixed OVS yang namespace issue

Change-Id: I56227cd8ef1d8a78386c75f3c3b6fcd97f34c1cc
Signed-off-by: Reinaldo Penno <[email protected]>

Fixes several bugs on bootsrap code, Yang models and related

- Fix sfc-initial.xml to reference new Async Data Broker
- Fix SfcProviderConfig(). Static variables are initialized in order they are declared.
- Fix SfcProviderConfig(). Test if file exists
- Fix SFcProviderConfig(). Moved reading config file to a separate method so we can handle
    failure gracefully. If reading is inside constructor we can not test for failure.
- getPutBootstrapData() was trying to commit initial config too early - datastore not ready.
  Moved to a ScheduledExecutorService with delay of 15 seconds.
- Commented out recent changes to data-plane-locator (Mac address). During testing
  I found that the system becomes unusable due to a Yangtools bug.
  https://bugs.opendaylight.org/show_bug.cgi?id=1467
- Fixed all Yang modules to be IETF compliant

Change-Id: Ie04890166382027a0564be2a8e2eb2fef3693032
Signed-off-by: Reinaldo Penno <[email protected]>

Yang Model changes for simple use case.

Change-Id: I1492d86696a08bb99df5d63461515e56d6183794
Signed-off-by: ebrjohn <[email protected]>

JSON configuration file for provider

Change-Id: I7138fff9ee552737476a764d39106e6496b110a1
Signed-off-by: Konstantin Blagov <[email protected]>

Migrating SFC to new Data Broker

- All necessary changes to migrate to new data broker
- Changed SFF model to better accomodate OVS architecture

Change-Id: I313232085bf22284860d4d064707854658699645
Signed-off-by: Reinaldo Penno <[email protected]>

removed unnecessary imports

Change-Id: Iad22697de10d264dcbfab6cc5eb6248727841a3c
Signed-off-by: Konstantin Blagov <[email protected]>

added copyright headers

Change-Id: I08f61fb837f1bcd3cd9892564d4d4e55df568a60
Signed-off-by: Konstantin Blagov <[email protected]>

Bootstrapping data from JSON configuration files

Change-Id: I1731cd536d8889c10b60e620c516e542136ec9e7
Signed-off-by: Konstantin Blagov <[email protected]>

Merge "Bug fixes"

Bug fixes

- When a SF is deleted, we need to clean up the operational state.

- When a new service path is added at the top level it replaces existing
  ones that need to be removed from the service function state list

- When deleting a SF from SFF we were actually deleting the entire SFF.
  Not detected until tests with partial overlapping PUTs

Change-Id: Ie960761ed0d05b444748fe3b7187a9f4f7e3a48f
Signed-off-by: Reinaldo Penno <[email protected]>

added mandatory SFName parameter setting

Change-Id: Ifff85bab29fe5f467e0609466d62829201e902a8
Signed-off-by: Konstantin Blagov <[email protected]>

Several Enhancements and bug fixes

- Completely decoupled SF and SFC.
- Completely decoupled SFP and SFC
- SFC is just a blueprint to build SFPs
- SFC can be deleted and SFPs stay
- If SF is deleted/updated, SFPs/SFF
  are deleted/updated accordingly
- Enhanced REST API
- All APIs are using reflection and factories.
  Meaning API implementation can be changed without changing signature
- Hardening of the code
- Modified Yang models by introducing operational containers.
- Fixed concurrency issue.

Change-Id: I8478b4ce3b0d2ab3092b5458fcd09116b6b1664f
Signed-off-by: Reinaldo Penno <[email protected]>

Merge "Bug fixes"

Bug fixes

- Fixed bug in path-id generation. Due to concurrency variable is now atomic
- Change LOG.info to LOG.debug

Change-Id: Iae29cf18a55bf5acf6c6c52b0076ee7ce92972f0
Signed-off-by: Reinaldo Penno <[email protected]>