vpp renderer: fix netconf node capabilities check issue

netconf project commit ebce6c1ba9a22e420c87b45e5b20501dc7047730
broke vpp renderer, it changed netconf node topology model
, vpp renderer is using it to check if a netconf node is vpp
node or not.

This fix is to adapt to new netconf node topology model, it has
been verified in sfc104 vpp demo.

Change-Id: I191fdb298d560b207ab2cc20fc9e13f2f9730ebe
Signed-off-by: Yi Yang <yi.y.yang@intel.com>

Minor cleanup of sfc-provider unit tests

- removed all warnings
- removed old references to the removed OpendaylightSfc class
- removed duplicate entry in sfc-model/pom.xml
- patchSet 2: addressed code review comments

Change-Id: I149acf7c7e84ee98ffe12f2604bc15a78a87846f
Signed-off-by: Brady Johnson <brady.allen.johnson@ericsson.com>

Fix a bug in the refactored ScfListener

The SfcScfOfDataListener was recently refactored, replacing the
onDataChanged notification method by an onDataTreeChanged method.

In that change, the lines that initializes/closes the listeners got
lost, making the SCF feature deaf/blind to classifiers being added /
deleted.

Change-Id: I77104b30c87939156cd4458fef4a6f9859bbbabe
Signed-off-by: Miguel Duarte <miguel.duarte.de.mora.barroso@ericsson.com>

Remove flake8 warnings from sfc-test

Change-Id: Ie2a1bcd11ee6095e62158ffaf9cb24246f588967
Signed-off-by: George Paraskevopoulos <geopar@intracom-telecom.com>

BUG 7196 : Doc style changes to README file

At present the commands in this file are combined with
text lines. In this patch I have modified the style to
quick understanding.

Signed-off-by: venkatamahesh kotha <venkatamaheshkotha@gmail.com>
Change-Id: I60d163b94b649f6bafc40abaf00c7a8e75b5ba84

Removing last parts of the old renamed SFCOFL2 feature

- The feature was still in the karaf features as deprecated.
  It has now been removed.
- There was a stale reference to sfcofl2 in the pom.xml

Change-Id: I88b465977fd4cc65872e30763113c09114dae987
Signed-off-by: Brady Johnson <brady.allen.johnson@ericsson.com>

Replace onDataChanged method in SfcScfOfDataListener

Change-Id: I3bf931099d86287cf0deeba68c1406b8dc658fc7
Signed-off-by: Ursicio Martin <ursicio.javier.martin@ericsson.com>

No prefix required for type parameter of SF

sfc-ui,is adding prefix "service-function-type:" for SF type.
i.e if Service Function type is selected as 'firewall',SF type gets set
as "type":"service-function-type:firewall".

The prefix is not required to be added, it suffices to set type as
firewall/dpi/ids etc. The prefix is causing SF creation failure in
sfc_agent.

Bug:7179
Change-Id: I26381def21b987ff9b1ef46a0493791222e76ddd
Signed-off-by: swatideshpande <swati.deshpande@serro.com>

New configuration examples for SFC - Logical SFF

Change-Id: I07699cdd531576de3f030d22bc6cdc52bca54a26
Signed-off-by: eceujmm <ursicio.javier.martin@ericsson.com>

Add XML encoding to pom.xml and formatting.

Change-Id: I5eef334cb47ae509dba0bdb0d946fa283adc7476
Signed-off-by: David Suarez <david.suarez.fuentes@ericsson.com>

Integrate classifier w/ genius - commit #1

For the logical SFF feature, the existent SFC classifier has to be
integrated w/ genius, for 2 reasons:
  - currently, it writes to table 0, which is owned by genius. It should
    write to a different table
  - logical sff requires the classifier to steer packets based on the
    dpn IDs of the OVS switches hosted in the compute nodes where the
    classifier / first SFF are located

This commit prepares the ground for that functionality, which will be
added in the next commit.

The genius service binding is exposed as an API, thus
enabling the classifier to leverage it.

The openflow writer, is also exposed as an API, thus enabling the
classifier to also leverage its functionallity (install multiple flows
within the same transaction object).
  - Trasactional support for the classifier was not used, but for
    logical SFF scenarios *is* required, because we cannot bind an
    interface to genius and not install its respective flows.

Additionally, the TableIndexMapper is also modified, registering the
classifier table.

Finally, the one of the geniusRPCs is also modified, allowing for an
offset to be added - this way, the actions gotten from genius can be
appended to another list of actions.

Change-Id: If849f5f3f66ef295c48d76e0bc52d9b57d8891c4
Signed-off-by: Miguel Duarte <miguel.duarte.de.mora.barroso@ericsson.com>

Replace deprecated DataChangeListener in SF Scheduler Type

Change-Id: I793dda563e6e4788d8f5bbb6d1b36dc4cf4828f7
Signed-off-by: eceujmm <ursicio.javier.martin@ericsson.com>

SfcGeniusRpcClient singleton removal

SfcGeniusRpcClient implementation as a singleton was temporary: done
that way in order not to interfere with upcoming blueprint impl,
but needed to be fixed (e.g. it was making component test harder).
Fixed

Change-Id: Ief23609b13396cae35c3fe2c06027ce5b284d4b8
Signed-off-by: Diego Granados <diego.jesus.granados.lopez@ericsson.com>

Change clustered listener by the single one.

Change clustered listener by the single one since there is no
need for all the cluster members to get notified.

Minor changes in Copyrights.

Change-Id: I1d9067bd15a46f0e9fac917d3af1207df848b48d
Signed-off-by: David Suarez <david.suarez.fuentes@ericsson.com>

Bug 7181 - "Bug 6768: The OpendaylightSfc class should be refactored..." has a bug in sfc-provider.xml

Change-Id: Ib0174acf337e755fd12db0e6823c48dc50af9e17
Signed-off-by: g tomasini <guillermo.pablo.tomasini.redondas@ericsson.com>

Reorder the organization to avoid YANG editor complaints.

Minor change to avoid YANG editor complaints.

Change-Id: Ia4fca6cfa050a9030e300b32721fd265d0123167
Signed-off-by: David Suarez <david.suarez.fuentes@ericsson.com>

Bug 6768: The OpendaylightSfc class should be refactored...

Change-Id: I0e079aecee87768e40787b2a86209b7b36d30e9e
Signed-off-by: g tomasini <guillermo.pablo.tomasini.redondas@ericsson.com>

Enable activation of feature odl-sfc-genius

Add feature 'odl-sfc-genius' that functionally enables SFC integration
with Genius. This allows configuring a logical SFF and SFs attached to
this logical SFF via logical ports/interfaces (i.e. neutron ports).

Change-Id: Ieaec3f68d9bbb74324f97d5406b97176f35a1f20
Signed-off-by: Jaime Caamaño Ruiz <jaime.caamano.ruiz@ericsson.com>

Merge "Add a load/path-aware Service Function Scheduler API."

Add a load/path-aware Service Function Scheduler API.

Most of the scheduler code are based on the two existing Service Function Schedulers
(SfcServiceFunctionLoadBalanceSchedulerAPI.java, SfcServiceFunctionLoadBalanceSchedulerAPI.java)

Change-Id: I4dc98dd7da07308a50c23bf76343ab031d42e23a
Signed-off-by: Dongeun Suh <dongeunsuh@gmail.com>

Add sfc104 demo

Current sfc103 demo is totoally docker container based, so it can't run with OVS DPDK and VPP.

This sfc104 demo has many improvements compared to sfc103 demo, including:

  * Only install a VM once, then import this new VM for all Classifiers, SFs and SFFs.
  * VMs can be used for demos repeatedly, you mustn't repeat VMs installation.
  * ODL SFC is started by you instead of this demo, so you can use this demo to test your ODL SFC development branch.
  * it includes three demos for OVS, OVS DPDK and VPP, respectively. These demos leverage the same vagrant VMs, so you are unncessary to create new vagrant VMs.
  * You can copy sfc104 to any other place to run, it doesn't depend on any other sfc code.

Change-Id: I95f5d153ddde09e49bc934905931fe789408b4e3
Signed-off-by: Yi Yang <yi.y.yang@intel.com>

Update RSP on VM migration for sfc-genius

A VM migration is detected when the interface state corresponding to
the neutron port is created. This signals the end of a process that
started when that interface state was previously removed.

Once the interface state is created, all the RSPs relying on that
interface are re-rendered. This is achieved by first deleting the RSP
and then creating it again.

Change-Id: Ia1564976540a0eae2fd3ccfe3f9c5d59b63e1424
Signed-off-by: Jaime Caamaño Ruiz <jaime.caamano.ruiz@ericsson.com>

Unit Test for ServiceFunctionClassifierListener

Change-Id: Ib3ddede453888d216d53a43cc0a9031b1a7e5a32
Signed-off-by: eceujmm <ursicio.javier.martin@ericsson.com>

Bug 7089: Logical SFF - Operational datastore

LSFF operational datastore implementation, including:
- RSP state: now RSPs which use a logical SFF are
  updated after rendering with information about the
  data plane node used in each hop
- SFF state: Logical SFFs now keep a list of dpn-ids
  for all the switches participating in the LSFF,
  including the list of RSPs using each switch. Yang
  model was updated (the existing one already contained
  a section aiming to hold this info, but was incorrect)
- SFF state: dictionary info for logical SFF has been
  removed from the model (will not be implemented; was
  redundant with the dpnid info included in the RSPs)

Operational datastore updates are performed transactionally
and asynchronously (in a separate thread) in order not
to delay the release of the RSP rendering lock. A separate
DS transaction than the one used for flows is used (the
one for flows is not exposed to the renderer; exposing it
breaks encapsulation principle in SfcFlowWriterImpl ->
considered not worth)

Change-Id: Iedc28fc37bd772218cb22d686931f9e6a22ae656
Signed-off-by: Diego Granados <diego.jesus.granados.lopez@ericsson.com>

 Bug 7087: ServiceFunctionClassifierListener - Invalid Update Operation

Change-Id: I710d9b88547a31d72b1a3231aa398440b4df2f6a
Signed-off-by: eceujmm <ursicio.javier.martin@ericsson.com>

Porting SFF/SF update changes from stable/boron

- Originally fixed in stable/boron for BUG 7039
- couldnt do a straight cherry-pick, so the changes
  had to be manually ported.

Change-Id: Ie9c51ed14564bb3077591f913c94b641c0c98683
Signed-off-by: Brady Johnson <brady.allen.johnson@ericsson.com>

Enable NSH-based service function chaining on OVS DPDK

For OVS DPDK use case, dpdk0 is a physical Ethernet
interface, br-int needs to be configured IP address
to handle management traffic, you can imagine dpdk0
as ethX. In addtion, VxLAN-gpe + NSH implementation
for DPDK in Intel OVS NSH patches also needs some
complementary flows to receive and send VxLAN-gpe +
NSH packets. So we have to add some flows similar to
the below for br-int in case of OVS DPDK:

table=0, in_port=1 actions=NORMAL
table=0, in_port=LOCAL actions=output:1

Here port 1 is dpdk0.

This patch can detect OVS DPDK and dpdk0 therefore add
necessary flows into SFC table 0, it will do
nothing for non-DPDK ovs.

In order to make sure br-int is configured correctly
and can work with next SFF, you must add ARP entry and
route entry for br-int, here is an example for reference:

$ sudo ovs-appctl tnl/arp/set br-int 192.168.50.75 08:00:27:bf:ae:0d
$ sudo ovs-appctl ovs/route/add 192.168.50.75/24 br-int

Change-Id: Ieea5599690ea7046f85d2b59926625042b2dbfe3
Signed-off-by: Yi Yang <yi.y.yang@intel.com>

Bug 6278: Switch to use odlparent's karaf-parent

Change-Id: Ic379884ea79e0c648dae997ab234c4ac0b0e477f
Signed-off-by: Ryan Goulding <ryandgoulding@gmail.com>

Minor changes.

Changes in copyright section and a method name.

Change-Id: I5f088c99a516daf474f8f2f2f76cfb7094b2a3c8
Signed-off-by: David Suarez <david.suarez.fuentes@ericsson.com>

Replace deprecated DataChangeListener in SF Classifier.

Change deprecated DataChangeListener by the new DataTreeChangeListener
in Service Function Classifier.

Change-Id: Ic10ec32c9c508396ad0dbef194da45a914428f40
Signed-off-by: David Suarez <david.suarez.fuentes@ericsson.com>
Signed-off-by: David Suarez <david.suarez.fuentes@ericsson.com>

Fix unchecked operation warnings on sfc-genius tests

Change-Id: I95b3b71e7ebbf0bbc76bbac057aa176922e87271
Signed-off-by: Jaime Caamaño Ruiz <jaime.caamano.ruiz@ericsson.com>

SFC openflow renderer: remove RuntimeException as a communication mechanism

Conflicts:
sfc-renderers/sfc-openflow-renderer/src/main/java/org/opendaylight/sfc/ofrenderer/processors/SfcRspProcessorLogicalSff.java

Change-Id: Ib79a96e5341326c5a0beb94946e5c7d0eca0fa65
Signed-off-by: eceujmm <ursicio.javier.martin@ericsson.com>

Corrections in mac handling for logical sff

Included changes:
- nextHop: inner dst mac address was being changed instead of
     the one in the eth encapsulation header (fixed). Also, added
     the src mac address, so the packet can be correctly forwarded
     (by switching src and dst mac) after SF processing
- changed src mac address at chain egress so the packet won't
     be dropped because of bad src mac at subsequent pipeline
     stages
- Table 0 (classifier) table is no longer written by Logical
     transport processor in any circumstance (logical SFF is
     always used in cooperation with Genius -> Genius owns
     table 0 always (it is used for service binding)
- Some test improvements

Change-Id: I1f344604801f0855cc35ea27885de91427766f24
Signed-off-by: Diego Granados <diego.jesus.granados.lopez@ericsson.com>

Change deprecated DataChangeListener.

Change deprecated DataChangeListener by the new DataTreeChangeListener.

Change-Id: I81f84548f97930da80ebb72ef07d9c6599220189
Signed-off-by: David Suarez <david.suarez.fuentes@ericsson.com>

fix for netconf model change

 - fixes changes made in https://git.opendaylight.org/gerrit/#/c/43972/,
   which was breaking SFC build

Change-Id: Ib29dfed9f6f80493b4ef0d8ff5ab5677e45666ca
Signed-off-by: Diego Granados <diego.jesus.granados.lopez@ericsson.com>

Bind SFC service to interface in sfc-genius

Performs bind of SFC service for the logical interfaces handled through
Genius. Includes adding the SFC terminating service action to the
interface node, which handles SFC traffic incoming through tunnel, if
it is the first interface bound to SFC service on the node.

Some comments about the solution implemented through this patch:

- For Boron it is expected to have a independent vxlan-gpe tunnel mesh
that uses a VNID of 0 for all SFC traffic.
- Using a rather uninspired flow priority and cookie for the rule that is
added on the genius dispatcher table through the binding. Feedback
welcome.

Change-Id: Iee00923a1e18e61902dc2c2218f7a4e84717da27
Signed-off-by: Jaime Caamaño Ruiz <jaime.caamano.ruiz@ericsson.com>

Bump SFC versions and remove incorrent parent versions

sfc-ui/module/pom.xml included a hardcoded version reference
from the dlux project. Now it is dependent on sfc-ui and
sfc-ui depends on sfc parent pom.

Change-Id: I5da41297b8ef7836b03856e32ebdeb5d223412db
Signed-off-by: Miguel Angel Munoz Gonzalez <miguel.angel.munoz.gonzalez@ericsson.com>

Change deprecated DataChangeListener by the new DataTreeChangeListener.
It adds two abstract listeners to simplify the process of creating new
listeners.

Change-Id: Ia00fded0029fa34e9b9bf85c2d779094d8ebff6f
Signed-off-by: David Suarez <david.suarez.fuentes@ericsson.com>

Hardcode sfc-ui version to avoid compilation warning

sfc-ui module version is set to fixed valud 0.5.0-SNAPSHOT
temporarily to fix the blocker issue caused by
https://git.opendaylight.org/gerrit/#/c/46730/

Further discussions are being held to remove the version tag
from this module and inherit from parent project, maybe
requiring bumping the versions.

Change-Id: I62d19f6a3c48af2d3be48accf1f27867d57bf874
Signed-off-by: Miguel Angel Munoz Gonzalez <miguel.angel.munoz.gonzalez@ericsson.com>

Fix NPE when clearing SFF initialization flows

NPE whenever initialization flows did not exist (for instance, no RSP
ever created). Fixed by checking their existence before filtering
the ones that belong to orphan SFFs.

Change-Id: Id5601c630395551d031658de8e74f847904c6124
Signed-off-by: Miguel Duarte <miguel.duarte.de.mora.barroso@ericsson.com>

Remove technical debt from the project.

Added a couple of tests to SfcOfRspTransactionalProcessorTest, thus
checking that attempting to delete a nonexistent RSP does not crash the
process.

Hid the constructors of some classes that only contain static methods.

Refactored the OpenFlowProgrammer, thus removing plenty of duplicated
code.

Logged the exceptions on the SfcOfFlowWriterImpl, thus removing sonar
warnings.

Change-Id: I8771897ec5026f255c8def7e7627275eb5ec90c8
Signed-off-by: Miguel Duarte <miguel.duarte.de.mora.barroso@ericsson.com>

Remove Maven warnings for using variable in version

This commit fixes the warning for using a variable in the
version tag of the artifact.
The artifact should get the version from the parent pom.
Note that variables in version tag should not be allowed.

Change-Id: I2cafc4609e27c74ea0874b4a0c3bbf4fc1707cdd
Signed-off-by: Miguel Angel Munoz Gonzalez <miguel.angel.munoz.gonzalez@ericsson.com>

Logical SFF component test

Features a component test of the RspProcessor used when the SFC uses a
logical SFF. The test features and end to end scenario of 2 SFs,
connected through a logical SFF entity. The genius RPC, and the genius
data utils are mocked.

The test provided 'simulates' a single compute node being used, since
the mocked genius RPC is returning the same dataplane ID, independently
of the logical interface being queried.

Notes:
- Added a couple of methods to the SfcGeniusDataUtils class, thus
entirely replacing the API of SfcLogicalInterfaceOfUtils.
- Removed SfcLogicalInterfaceOfUtils, since SfcGeniusDataUtils provides
the same functionality, but using better abstractions, and has been
extensively unit tested.
- Replaced the calls to this API throughout the code, namely, in the
processor classes.
- Changes add to be made to the RspBuilder class, enabling logical SFFs to
be used in unit/component tests.
- An empty constructor had to be added to the OpenFlowProgrammer class, so
that it could be mocked / spied in unit tests.

Change-Id: I32de81363ad69d6a8ca0cb32aaa061887ebbf76f
Signed-off-by: Miguel Duarte <miguel.duarte.de.mora.barroso@ericsson.com>

Add Genius-based app coexistence (table index mapping)

Added logic for performing translation between a set of external SFC
table indexes and the table indexes which are reserved by Genius for those tables.
For doing so, the client module (sfc renderers using Genius) first builds a
SfcTableIndexMapper (using the convenience SfcTableIndexMapperBuilder
class) by providing the table indexes it uses for each SFC table (this
mechanism exists in order not to have a circular dependency between
sfc-genius and the sfc renderer). Later, at flow writing time, the flow
programmer will use the mapping class in order to retrieve the correct
table indexes for each SFC table

In short, this logic allows to perform Genius-based application coexistence
at table level, while keeping the previous application coexistence when
genius is not used

Change-Id: I536ce79fde7d1960db9d3192a11ba9088e367806
Signed-off-by: Diego Granados <diego.jesus.granados.lopez@ericsson.com>

Update ODL integration maven repo

Change-Id: Ib640308b72bc5d90d9a24a59c1d3dd3fc060cc85
Signed-off-by: Guo Ruijing <ruijing.guo@intel.com>

Move sfc-provider listeners to their own package following the typical
structure
in other bundles.

Change-Id: I0f30502ff96d9bb94a6b0eac809e3f8b2f1ce084
Signed-off-by: David Suarez <david.suarez.fuentes@ericsson.com>

Add BP wiring for sfc-pot

Change-Id: I3cac8051a891fd2a7fe229c8eaeaaebc992f1573
Signed-off-by: Rashmi Pujar <rpujar@inocybe.com>

Add BP wiring for sfc-sb-rest

Change-Id: I5d55db2e18842e7f915b383a2c9976fca76e980e
Signed-off-by: Rashmi Pujar <rpujar@inocybe.com>

Add BP wiring for sfc-lisp

Change-Id: I822a4d22113ea97cc6766af6f69816d2cb24fde8
Signed-off-by: Rashmi Pujar <rpujar@inocybe.com>

Add BP wiring for sfc-vpp-renderer

Change-Id: I5cd39e31c4cadabe9eaf5b253b31ee97296df2a1
Signed-off-by: Rashmi Pujar <rpujar@inocybe.com>

Add BP wiring for sfc-scf-openflow

Change-Id: I48692a63f6ffe3d10c4442b9d1cc11c7adc58a4b
Signed-off-by: Rashmi Pujar <rpujar@inocybe.com>

Add blueprint wiring for sfc-ovs bundle

Change-Id: Ib5feb7fdfacc7528ff43bfdab2b45eb16cdce3cb
Signed-off-by: Rashmi Pujar <rpujar@inocybe.com>

Utilities for Genius related datastore operations

  - Utils for genius data retrieval, namely the MAC address of the service
    function, connected through a neutron port, and the name of the
    logical interface to which a service function is connected to. This
    data is required for the Logical SFF feature.
  - Unit Tests of the supplied classes

Change-Id: I7acd13504b489330854a6a79da07c5f0433c06da
Signed-off-by: Miguel Duarte <miguel.duarte.de.mora.barroso@ericsson.com>

Minor formatting and checkstyle changes to improve readability.

Change-Id: I4a9832c419d00c7ad50b4000b15534e0759b3b71
Signed-off-by: David Suarez <david.suarez.fuentes@ericsson.com>

New transport processor for Logical SFF

Created a new RSP processor class for chains which use Logical SFFs.
On such chains, SFF configuration is trivial (it only defines SFF name),
while SFs attached to the logical SFF define use Logical Interfaces as DPL.
That logical interfaces are used to gather all required information for
path rendering via Genius project

Transport protocols are the same than in the NshEth processor (that is,
Eth+NSH between the SFF and SF, and VXGPE+NSH between the SFFs)

Change-Id: Ic8175c02006f7b6c08120941b7e535d551c3f56a
Signed-off-by: Diego Granados <diego.jesus.granados.lopez@ericsson.com>

Unbind SFC service from interface on sfc-genius.

Performs unbind of SFC service for the logical interfaces handled
through Genius. Includes removing the SFC terminating service action
from the interface node, which handles SFC traffic incoming through
tunnel, if there are no more interfaces bound to SFC service on the
node.

A handler is setup for this task, SfcGeniusServiceHandler, which
coordinates the task through more sub-handlers specialized in read and
write operations through asynchronous APIs. JDK8 new CompletableFuture
is leveraged to asynchronously chain read & write asynchronous
operations.

For the time being, the module stores in-memory the data plane node
location of each interface (dpnInterfaces map), which is an information
needed to handle the terminating service actions. On a later patch, this
will be stored in the SFC operational data store.

The implementation is asynchronous for the following reasons:
- Javadoc of APIs recommend using the asynchronous interfaces over
  the synchronous ones.
- With the introduction of JDK8, more code facilities are available to
  produce asynchronous readable code.
- Being sfc-genius an independent small module, it's deemed a good
  chance to get a taste for it.

Change-Id: I145b4eb0aa6a4f17ff0c3bd8344a0486d8d7be96
Signed-off-by: Jaime Caamaño Ruiz <jaime.caamano.ruiz@ericsson.com>

Fix a possible null pointer on RSP deletion

  - remove wildcard imports from SfcOfRspTransactionalProcessorTest
  - fix a null pointer exception that accured whenever an RSP was
    deleted and for some reason it did not get properly added to the
    datastore

Change-Id: I50888a9fbacf0fcf53f4aae4af2344758efb0bc8
Signed-off-by: Miguel Duarte <miguel.duarte.de.mora.barroso@ericsson.com>

Merge "Handle data change events on sfc-genius listeners"

Merge "Bug 6657: Added init-method into ios-xe-renderer blueprint xml"

Handle data change events on sfc-genius listeners

sfc-genius needs to be aware of the following data change events:

- IETF Interface State

  When service function migrate from one location to another,
  the state of the associated logical interface is removed
  and then added again. Attribute LowerLayerIf reflects it's
  new location. On such a case, sfc-genius needs to trigger
  re-renderization of the affected RSPs.

- SFC Service Function State

  When a service function becomes involved with one or more RSP,
  sfc-genius needs to undertake the appropiate actions towards
  Genius to bind the SFC service to the associated logical interfaces.

Initially, data store change events are handled on a statically shared
thread provided by AsyncDataTreeChangeListenerBase. Being this is a
Genius utility class shared also with genius and netvirt, jobs are
taken off to a different single thread executor where there are
enqueued and processed synchronously in order.

Another single thread executor will be used as continuation through the
callbacks of the different asynchronous APIs (Data store reads,
transactions, Rpcs) that are to be used later on.

Change-Id: Ib50c9e63a58b06eafb29fb539d0b4e0d760f7aa5
Signed-off-by: Jaime Caamaño Ruiz <jaime.caamano.ruiz@ericsson.com>

Bug 6657: Added init-method into ios-xe-renderer blueprint xml

Change-Id: I1c1c92592d3654c598740aad46723afcd95b711d
Signed-off-by: Vladimir Lavor <vlavor@cisco.com>

Avoid run_demo.sh retries calculation error

Change-Id: Ia36b03e633ae9375531d7c349d63d1aa45a228b0
Signed-off-by: Sergio Arroutbi Braojos <sergio.arroutbi.braojos@ericsson.com>

Bug 5969 - Flows left behind in the OVSs

  - Modified the initialization flows cleaning behavior, this time
    accounting individually for each SFF: whenever an SFF does not
    feature in any RSP, its flows are deleted
  - Added unit testing of this particular behavior
  - Tiny changes in the InstanceIdentificar instantiation in
    SfcOfFlowProgrammerImpl.java

Change-Id: I2382a65adb5c6d19a477ee2643941d00cfdf5bee
Signed-off-by: Miguel Duarte <miguel.duarte.de.mora.barroso@ericsson.com>

Increase time after docker build

Change-Id: I88cd7e5c9f207166a603dee89a4c1c2aebe55d99
Signed-off-by: Guo Ruijing <ruijing.guo@intel.com>

Removed non-existing required capability from sfc-ovs initial configuration

Required yang model does not exist, not possible to install
sfc-openflow-renderer feature. File sfc-provider-impl.yang
was removed here https://git.opendaylight.org/gerrit/#/c/42661/18

Change-Id: I82a90aafba4615b01f568f67910d92d8d69327b4
Signed-off-by: Vladimir Lavor <vlavor@cisco.com>

Bug 6649 - Fix sfc-encapsulation placement

Added sfc-encapsulation at root level of service-function-path
in the yang model. It was previously wrongly placed on the
service function path hops.

Added logic to handle a default value: 'nsh' for vxlan-gpe
transport type, and 'transport' in every other case.

Included updated sample jsons.

Change-Id: Idfc205cbf8e636feed89ba7d2aa07f7078848123
Signed-off-by: Jaime Caamaño Ruiz <jaime.caamano.ruiz@ericsson.com>

Update sfc103 to docker based version

- Additionally the demo can be executed several times without vagrant vm destruction
- Compliant with OVS 2.5.90 NSH patches written by Yi Yang

Change-Id: I13b5561c263b5fc76511adff42010bf51696f572
Signed-off-by: Guo Ruijing <ruijing.guo@intel.com>
Signed-off-by: Ricardo Diaz Campos <ricardo.diaz.campos@ericsson.com>

Reorder imports and disable flake8 quality assurance checkings for them in sfc_agent

- The sfc_agent is not working because of the order of some imports. This change requires
  disable fleak8 quality assurance checkings. The complete module should be refactored
  to avoid the checkings cancellation (noqa flag).

Change-Id: I23b78cf85462453a805d28324691f271806f572f
Signed-off-by: Ricardo Diaz Campos <ricardo.diaz.campos@ericsson.com>

Support VxLAN-gpe in sfc103 demo setup

Make sure it can work with https://github.com/yyang13/ovs_nsh_patches

Change-Id: Iea4f5222620474d127d942751d777f46e34eb57c
Signed-off-by: Yi Yang <yi.y.yang@intel.com>

Enable sfc-py to support VxLAN-gpe + Eth + NSH use case

Change-Id: I0970b987f60216240e8e932f2df06f0c941d4257
Signed-off-by: Yi Yang <yi.y.yang@intel.com>

Bug 6640 - SFC VPP Renderer can't work with Honeycomb and VPP

 - Updated v3po.yang and vpp-nsh.yang
 - Added BridgeDomain "SFCVPP"
 - Finished integration test with Honeycomb and VPP
 - Fixed several integration issues

Change-Id: Ie695f4398446e8dee43b1dc406385e3c1e1a47de
Signed-off-by: Yi Yang <yi.y.yang@intel.com>

Bug 6648 Fix: Not possible to add additional Service Function's data plane locators

This commit fixes exception given (500 http error code) returned
when a second data plane locator is added to an existing service function

This bug is filed as 6648: https://bugs.opendaylight.org/show_bug.cgi?id=6648

Change-Id: I20463bc3a05e509528417de738f17a8df3e49b07
Signed-off-by: Miguel Angel Munoz Gonzalez <miguel.angel.munoz.gonzalez@ericsson.com>

Remove unused code & fix pep8 issues

Change-Id: I4fb3807ba6edb59d42bc81cc49e818ff8610fdbe
Signed-off-by: Guo Ruijing <ruijing.guo@intel.com>

Add blueprint wiring for sfc-openflow-renderer

Change-Id: Ibde7b70e52aaa26413e662a6c77d2a5079c112f9
Signed-off-by: esergbr <sergio.arroutbi.braojos@ericsson.com>

Add blueprint wiring for sfc-ios-xe-renderer

Change-Id: I5e907324257cecfe4b7cc76bc62e88e3420919f4
Signed-off-by: Rashmi Pujar <rpujar@inocybe.com>

Add blueprint wiring for sfc-netconf

Change-Id: I88826203e126553fa8a65febca84c08605257cb1
Signed-off-by: Rashmi Pujar <rpujar@inocybe.com>

Add blueprint wiring for sfc-provider

Change-Id: Ib4992cb0dbae907d2e7b86b7df3f4f50edea81c9
Signed-off-by: Rashmi Pujar <rpujar@inocybe.com>

BUG 6613 - avoid randomly generated test input from randomly breaking test

Avoid randomly generated test input from randomly breaking test
by generating invalid XML chars at Surefire test report

Change-Id: Iac01d2b97a877ecb40b574395fcbc0e6b39f26e7
Signed-off-by: Diego Granados <diego.jesus.granados.lopez@ericsson.com>

V3po.yang model updated

Change according to https://git.opendaylight.org/gerrit/#/c/44300/1

Signed-off-by: Vladimir Lavor <vlavor@cisco.com>
Change-Id: I4b0d0231089bcb75a647553ee283a5ea1f50796b

BUG 6355 - reduce the size of the karaf distro

- The sfc-vnfm-tacker module doesnt need the jersey containers
  at runtime, so the scope was changed to test
- https://bugs.opendaylight.org/show_bug.cgi?id=6355

Change-Id: I093ab139363fec570bc5d2339903dcb624dd5c94
Signed-off-by: Brady Johnson <brady.allen.johnson@ericsson.com>

Fixed UT in sfc openflow renderer

Change-Id: Ic6a570d84eb757c4b270ad7ca7eb5a0ae19f0c61
Signed-off-by: Vladimir Lavor <vlavor@cisco.com>

Merge "BUG 6398 - fix failing unit tests"

BUG 6403 - outport cant be the same as inport

- the inport cant be the same as the outport. if they are,
  outport must be the string INPORT, else the pkt will be
  dropped according to openflow spec.
- https://bugs.opendaylight.org/show_bug.cgi?id=6403

Change-Id: If16523ddd5eae0f94806f0bbdc7e1af1f4869267
Signed-off-by: Brady Johnson <brady.allen.johnson@ericsson.com>

BUG 6398 - fix failing unit tests

- they were commented out so as not to hold things up
- now they've been uncommented and fixed
- https://bugs.opendaylight.org/show_bug.cgi?id=6398

Change-Id: I8e49089ae7bed5c62e83602cec5578646c86e379
Signed-off-by: Brady Johnson <brady.allen.johnson@ericsson.com>

Support new actions in new ovs nsh patches

- Change sfcofl2 to support new actions
- Change sfc classifier to support push_nsh action
- Verified by sfc103 demo
- Rebasing
- Changing topic back to topic/37937, since it was changed in previous patch set
- Added output to vxgpe port to transport egress to SF flows
- Change output:vxlangpe to IN_PORT (for gbpsfcenv demo)

Change-Id: I44e583df91e5034a6eed6e49b1e90fcd81f7c455
Signed-off-by: Yi Yang <yi.y.yang@intel.com>

Add classifiers topology graph for sfc-demo/sfc103h

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

Bump versions by 0.1.0 for next dev cycle

Change-Id: I8ad7da6fd17c68cc38f5ee28c5d2d6256f66e4d4
Signed-off-by: Thanh Ha <thanh.ha@linuxfoundation.org>

Bug fix in vxlan_tool.py

TCP blocking feature was not working when ETH+NSH

Change-Id: I8afb361e494cd8d3de245c971786b46fab5b89a1
Signed-off-by: Manuel Buil <manuel.buil@ericsson.com>

vxlan_tool.py: support VxLAN-gpe + ETH + NSH

Change-Id: I490ccf9190198280e14916e16eb9edacd8bb422a
Signed-off-by: Yi Yang <yi.y.yang@intel.com>

Demo 103 Vagrant + Docker

A version of sfc103 demo using just one Vagrant machine and six Docker
containers:

- The Vagrant machine hosts the ODL controller
- The Docker containers deployed also on the Vagrant machine
  (with  docker-compose) act as service nodes

In theory, this version of the demo requires less machine resources
(memory and cpu) than the original one.

Change-Id: I46870c35af3afbd760a40a21c166eb7cc96c8c96
Signed-off-by: ericdia <ricardo.diaz.campos@ericsson.com>

Rename sfc-genius provider package to impl

It's a more appropiate name and avoids the export of any bundle

Change-Id: Ie039679f4fe849dadb2a91002159b4b69363b67c
Signed-off-by: Jaime Caamaño Ruiz <jaime.caamano.ruiz@ericsson.com>

Add configuration examples for SFC - Logical SFF

Change-Id: I7377fd16f2ed6e6defecbe1e2ac97a5517b54f41
Signed-off-by: eceujmm <ursicio.javier.martin@ericsson.com>

Merge "Fix build failure caused by following OpenFlow patch https://git.opendaylight.org/gerrit/#/c/42630/"

Fix build failure caused by following OpenFlow patch
https://git.opendaylight.org/gerrit/#/c/42630/

Change-Id: I0e4877a04040714765a23f733d4a725764a24948
Signed-off-by: Anil Vishnoi <vishnoianil@gmail.com>

Add VPP Renderer

VPP is a high performance data plane, https://wiki.fd.io/view/VPP/What_is_VPP%3F
has detailed introduction. It can replace ovs and act as a virtual switch.

Another project honeycomb (https://wiki.fd.io/view/Honeycomb) provides REST and
NETCONF APIs for VPP in order that ODL controller can control VPP very easily.

VPP Renderer will render SFC RSP to corresponding objects in VPP, this can enable
VPP to act as a SFF, VPP Renderer also can handle the use case VPP acts as SF NSH
proxy.

Change-Id: Ia5c0e196797674e878fae5642daaaab4dc1f6d25
Signed-off-by: Yi Yang <yi.y.yang@intel.com>

Merge "Introduce SFC Proof of Transit via in-band OAM"

Add new module sfc-genius

This commit adds the skeleton of new module sfc-genius.
Sfc will start using odl genius project as a generic
network service provider and this module will contain
several specific items related to this handling.

Among other things, it will contain:
* Listeners for SFC configuration events that require
  any Genius related action.
* Listeners for Genius operational events that require
  any SFC related action.
* Genius related utilities

Change-Id: Ia26269d7765f17a5836c3054eef9e6912e9ebc0c
Signed-off-by: Jaime Caamaño Ruiz <jaime.caamano.ruiz@ericsson.com>

Do not pull JUnitParams into runtime

This is used only for testing, hence the dependency
should be scope to test.

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

BUG 6268: service-function-dictionary grouping is redundant as it is

The grouping service-function-dictionary is already defined in the SFF
configuration with the same name. It was not desired to repeat the name
nor reuse the same fields, so additionally to change the name,
the following fields should be removed:

     list sff-interfaces
     leaf failmode

Additionally in this grouping, the grouping dpn-locator is used and
it should be renamed to dpn-grouping since the dpn-id defined inside
is not going to be defined as other locators.

Change-Id: Ib2042251ad6850da007979ad9dc861b4465b4a7d
Signed-off-by: danipeon <daniel.peon.quiros@ericsson.com>

Merge "Add configuration examples for SFC"