Remove sfc-provider-impl capability from sfc-test-consumer-initial.xml

sfc-provider-impl has been migrated to Blueprint,
so the capability will never appear.

Change-Id: Id14ea9529e8928815fd8f933298bc3f34acfae9f
Signed-off-by: Vratko Polak <vrpolak@cisco.com>

Remove sfc-bootstrap module

Since SFC is already using Blueprint as dependency injection framework,
the sfc-bootstrap module is not needed anymore.

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

BUG 7269: System Info tab,no data displayed

Added parameter to call SfcRestangularSvc.getCurrentInstance().oneUrl
When enter to System Info tab, debugging javascript, an error
is displayed in the console.

Change-Id: Ic86186e0fd89cfbee01cab8db7a3b8864ef1a6cb
Signed-off-by: Alberto Quintana Heras <alberto.quintanaheras@altran.com>

Merge "Fix clearly badly broken Blueprint wiring"

Fix clearly badly broken Blueprint wiring

Unable to find property descriptor dataProvider on class
org.opendaylight.sfc.sfc_netconf.provider.api.SfcNetconfServiceFunctionAPI
org.osgi.service.blueprint.container.ComponentDefinitionException: Unable
to find property descriptor dataProvider on class
org.opendaylight.sfc.sfc_netconf.provider.api.SfcNetconfServiceFunctionAPI

Indeed SfcNetconfServiceFunctionAPI's setDataProvider() method is static,
so you cannot do <property name="dataProvider" ref="dataBroker"/> for
<bean id="sfcnetconfservicefunctionapi"
  class="org.opendaylight.sfc.sfc_netconf.provider.api.SfcNetconfServiceFunctionAPI">

Thus, at the very least, you would need a fake non-static setter there,
(although that's very ugly, static-is-evil; but that's beyond this change).

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

Minor refactor of the VPP Renderer

- removing circular dependencies between listener and renderer packages
- changing listeners to use AbstractDataTreeChangeListener
- patchSet 4: changes due to code review comments

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

BUG 6347: Fine tune RSP removal on SFF update

Remove only RSPs affected by SFF dictionary change. If a dictionary
entry is changed or removed, an RSP with a hop on that SFF for the SF
corresponding to that dictionary entry will be removed. Otherwise, it
won't be removed for such a change.

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

Fix comparison of required and available capabilities

    - upon netconf node capabilities are compared. But
      because of model change in netconf available
      capabilities are no longer plain list of strings.
      Currently netconf provides a AvailableCapability
      object with additional leaves and string names
      must be extracted first and then compared.

Change-Id: Ia4e17a55cc8ab04714fb5b53a882e148593089ad
Signed-off-by: Michal Rehak <mirehak@cisco.com>

BUG 7243 : Modify the checkbox text color to be visible

At present the text is not visible as it sync with background
color. In this patch I have modified to be visible to users

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

Add empty checkstyle-suppressions.xml

Due to change 48715 [0] in odlparent, now checkstyle configuration
inherited from odlparent looks for a checkstyle-suppressions.xml file.
Rather than overriding the configuration, an empty file to add
exceptions in the future is added. This fixes a build failure.

 [0] https://git.opendaylight.org/gerrit/#/c/48715/

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

Merge "BUG 7242 : Correction in the command for starting sfc_agent"

Merge "Integrate classifier w/ genius - commit #3"

BUG 7242 : Correction in the command for starting sfc_agent

When we execute with the specified command it is throwing error
as we are not specifying .py to sfc_agent. If we execute with
.py extension there is no errors

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

Integrate classifier w/ genius - commit #3

This commit finishes the integration of the current SFC classifier w/
genius.

The remaining methods of the API providing in the previous commits are
implemented, specifically:

The genius service binding w/ the interfaces to classify is finished,
and is called from the OpenflowClassifierProcessor.

The LogicalClassifierDataGetter is finished, providing all necessary
data-retrieval methods required by the LogicallyAttachedClassifier.

The flow generation logic made in the LogicallyAttachedClassifier is
finished.

Unit testing of the remaining classes is added.

Quicker bail-out of the openflowWriter when no flows are to be writen or
deleted.

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

BUG 7171: ACL configuration via GUI Broken

The current yang model for ACLs is ietf-access-control-list(2016-02-18).
sfc-ui, is constructing ACL requests as per
ietf-acces-control-list(2015-03-17)causing breakage.

The acl key has changed from "acl-name" to "acl-type acl-name",
in Rev 2016-02-18.

Fix done in sfc-ui module for acl key change as well as schema change
for all ietf-access-list config requests.
Input box added to "Create Access List" screen for configuring ACL Type.
Fix added for service-classifier requests to adhere to yang model Rev
service-function-classifier(2014-07-01)

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

Merge "Integrate classifier w/ genius - commit #2"

Redundant if(x>0) brushing befor for( ;x<0; )

Change-Id: I2477c76b1558ca49aec4a0946efa37325728cd0c
Author: Guillermo Tomasini <guillermo.pablo.tomasini.redonas@ericsson.com>
Signed-off-by: guillermo tomasini <guillermo.pablo.tomasini.redondas@ericsson.com>

Integrate classifier w/ genius - commit #2

Major refactor of the SfcScfOfProcessor class, which had too many
responsabilities. These were split among the following classes:

SfcScfOfProcessor:
Processes adding classifier flows into a number of OVS switches.
Client of the OpenflowClassifierProcessor class, from which it gets
a list of all the flows to install among the OVSs, writing them in a
single transaction object.

OpenflowClassifierProcessor:
Encapsulates a transaction object, and processes the creation of
ACLs (along w/ respective ACEs) into the OVS switches.
Client of the LogicallyAttachedClassifier class.

ClassifierInterface:
Responsible for building the classifier flows required for the
logical SFF feature. WIP, and will feature in the next commit.
There are 2 types of ClassifierInterface:
  - LogicallyAttachedClassifier - empty implementation; will be uploaded
    in the next commit
  - BareClassifier - 'Legacy' scenario, moved into a wrapper class

LogicalClassifierDataGetter:
Utility class used to fetch data relevant to the OpenflowClassifier
Processor and the LogicallyAttached classifier.
Encapsulates the genius RPC, and fetches data from genius, SFC, and
IETF data models. WIP, and will feature in the next commit.

The SfcScfOfUtils are also refactored, building the required flows, so
that the SfcScfOfProcessor can install them afterwards, in a single
transaction object.

Blueprint injection of the RpcProvider (required for the logical SFF
feature).

Add unit testing of the legacy scenarios, and well as w/ DPDK flow
generation.

In this commit, while the way the classifier works is drastically
changed, no new functionallity is added. It finished paving the way
for integrating the new feature, making it easier to be understood,
and easier to unit test.

In the next - and last - commit, I will complete the APIs of the
forementioned classes, thus enabling the classifier to be attached
to a logical SFF.

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

sfc104 demo: check openflow renderer and sfc classifier initialization

- sleep longer time for vpp node connection
- check openflow renderer and sfc classifier initialization

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

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>