Add SFC shell's command to show Service Function Paths

Add SFC shell's command to show Service Function Paths, as described in
[1].

[1] https://git.opendaylight.org/gerrit/#/c/61059/

Change-Id: If11acd10e0ec38de14fee038958ac4a68015db96
Signed-off-by: David Suarez <[email protected]>

Add SFC shell's command to show Service Function Chains

Add SFC shell's command to show Service Function Chains, as described in
[1],

[1] https://git.opendaylight.org/gerrit/#/c/61059/

Change-Id: Iceadac2c1a39b4c93e950503d5e23924e07e3f39
Signed-off-by: David Suarez <[email protected]>

Add missing relative path

Change-Id: I358b05a938f8b4ef812f1174e33b58d434cbba93
Signed-off-by: Stephen Kitt <[email protected]>

Add SFC shell's command to show SFFs

Add SFC shell's command to show SFFs, as described in [1],

[1] https://git.opendaylight.org/gerrit/#/c/61059/

Change-Id: I3a8197d70fef6073a6713f7fbbe30ff01ae43a0f
Signed-off-by: David Suarez <[email protected]>

Bump versions by x.(y+1).z for next dev cycle

Change-Id: Ib7c6fc7f5861f48c09c6603c147ad55f322d343d
Signed-off-by: Thanh Ha <[email protected]>

Bump odlparent 2.0.2 to 2.0.4

Change-Id: I9c74470018d63d8b693068c7ce7373cabb4cf3ed
Signed-off-by: Stephen Kitt <[email protected]>

Add support for the new SFC shell

Add support for the new SFC shell. This is the first commit of a series
adding all the new CLI commands to the Karaf console to show the
provisioned SFC's model entities as described in [1].

[1] https://git.opendaylight.org/gerrit/#/c/61059/

Change-Id: I04498cbbe23aa0ab3ffb22aba87432217b0fe34b
Signed-off-by: David Suarez <[email protected]>

Replace size()==0 by isEmpty()

Replace size()==0 by isEmpty(). It improves readability and avoid "=0".

Change-Id: Id7c7d52a4c7ad41238b32922fcfca3db3c493fee
Signed-off-by: David Suarez <[email protected]>

Add private constructor to this utility class

Add private constructor to this utility class.

Change-Id: Ifd9a3eff6b23712b7ef2ca3f0d6cc82ffa3b1266
Signed-off-by: David Suarez <[email protected]>

Join and simplify if(s) statements

Join and simplify if(s) statements

Change-Id: I22a2532f5abf68043906f0cfdea77e6e80db51f6
Signed-off-by: David Suarez <[email protected]>

Add the spec document for the new SFC shell

Add the specification documents for the new feature SFC Command Line
Interface (CLI). This shell will include several commands to show the
provisioned SFC's data model.

Change-Id: I09af7269a327ee6cfe35e824d91e8c0c5a8894f7
Signed-off-by: David Suarez <[email protected]>

Fix some logs

Fix some logs to comply with the recommended usage for slf4j.

Change-Id: I6e45bde2cc239ec652f1efb04f5b6523e8bb93eb
Signed-off-by: David Suarez <[email protected]>

Add a readAll method to get all the Service Chains

Add a readAll method to get all the Service Chains.

Change-Id: I6f95d23b75c90c848a94e50e13cc18d38c1bb31c
Signed-off-by: David Suarez <[email protected]>

Add a readAll method to get all the Service Function Types.

Add a readAll method to get all the Service Function Types.

Change-Id: I0028800cc3a87b5eec2102604420da0941124404
Signed-off-by: David Suarez <[email protected]>

Cleanup

Minor changes to improve code quality and readability.

Change-Id: I4fada8d1e21fef132484b36d6fcf857686330cca
Signed-off-by: David Suarez <[email protected]>

Improve the generation of docs within the project

Add configuration files and static HTML elements that allow to generate
documents with the same look&feel that the ones to be published.

To generate SFC documents just go into the doc directory and type "tox",
the tool will generate an HTML site into the _build directory.

Change-Id: If3e3d9c874f7a64064dbec1d2cded819457f5feb
Signed-off-by: David Suarez <[email protected]>

Support more chain egress methods for logical sff

Some scenarios, like origin and destination of the encapsulated packet
being on the same node, require more flexibility at chain egress.
Support was added to provide the classifier the possibility to handle
the de-encapsulation of the nsh packet and forwarding the original
packet to destination with the help of extra metadata included in the
nsh context.

As before if C1 is 0, packet will be de-encapsulated by SFC and sent
back to dispatcher under the assumption that a different service will
handle the encapsulated packet.

Otherwise, C1 indicates an IP address were the traffic must be sent by
SFC on chain egress. In such cases, introduced in this commit, if C2 is
0, the packet traffic will be sent conserving its original nsh
encapsulation. If C2 is not 0, the packet will be de-encapsulated and
sent through vxlan with C2 as tunnel id, as before.

Also introduced in this commit is the support for C1 to be a local IP
address. The packet will be send either to the dispatcher table if C2 is
0 assuming a chain egress classifier will handle it immediately after,
or to the internal tunnel table if C2 is not 0.

Change-Id: I7ca4f65f3b0142d03399ae096858b010427f8f01
Signed-off-by: Jaime Caamaño Ruiz <[email protected]>

Convert sfc-vnfm-tacker to blueprint

Change-Id: I396d7a702207e92c622c322f7f45c22802d177ee
Signed-off-by: Tom Pantelis <[email protected]>
Signed-off-by: Jaime Caamaño Ruiz <[email protected]>

Improve the handling of InterruptedException

Improve the handling of InterruptedException.

InterruptedExceptions should either be rethrown or the thread should be
re-interrupted by calling Thread.interrupt().

Change-Id: I26a283f9a92d3df66a3e0041a1aab229c7c619e8
Signed-off-by: David Suarez <[email protected]>

Remove sfc-rsp-manager

This sub-project was added by https://git.opendaylight.org/gerrit/#/c/38222/
over a year ago but is not used and basically only contains a stubbed
CSS Module. Maybe the intent was to imlplement this Module at some point.
If that's the case then we would want to use blueprint and not CSS. Either
way I just removed the entire sub-project.

Change-Id: Ic0e5b2ff6c05d777d4bd6ca17e525a0913e975ca
Signed-off-by: Tom Pantelis <[email protected]>

Convert sfc-test-consumer to blueprint

Change-Id: I01767167fc4982803031751273c1349202301e0d
Signed-off-by: Tom Pantelis <[email protected]>

Merge "Cleanup minor issues"

Fix the listener registration

The listener was not properly registered.

Minor changes.

Change-Id: I17931ec1483d517c9c1e15ab63ead4c90a933bf3
Signed-off-by: David Suarez <[email protected]>

Cleanup minor issues

Cleanup minor issues.

Change-Id: I63d1761c867735a66de7c8291182921e59ffd788
Signed-off-by: David Suarez <[email protected]>

Add execution permission to test scripts

Change-Id: I39333034d95c59a0a63df46df03f2b72e01cc059
Signed-off-by: Jaime Caamaño Ruiz <[email protected]>

Add support for SFC in only one node in dovs

In order to be able to test the topology where client, server and SF are
in one node, the support is added in dovs.

Change-Id: I2098607ca3ac2e7b98f885282a6d3532135ef65c
Signed-off-by: Manuel Buil <[email protected]>
Signed-off-by: Jaime Caamaño Ruiz <[email protected]>

Migrate the rest of the listeners to use blueprint annotations

Migrate the rest of the sfc-provider module's listeners to use blueprint
annotations.

Blueprint annotations is the recommended configuration mechanism for
Dependency Injection [1].

Some minor cleanup.

Change-Id: Ib1d7bf0c3dfcb482042934f144263188d43efc04
Signed-off-by: David Suarez <[email protected]>

Migrate ServiceFunctionChain listener to use blueprint annotations

Migrate ServiceFunctionChain listener to use blueprint annotations.

Blueprint annotations is the recommended configuration mechanism for
Dependency Injection [1].

Some minor cleanup.

[1] https://wiki.opendaylight.org/view/BestPractices/DI_Guidelines

Change-Id: I80f69e4fdf7c57e4448299434423d440d7515ba4
Signed-off-by: David Suarez <[email protected]>

Bump odlparent from 2.0.0 to 2.0.2

Bump odlparent from 2.0.0 to 2.0.2.

Change-Id: I16f10b7603c2492386fff7e6c4b64f3f987900ab
Signed-off-by: David Suarez <[email protected]>

Migrate ServiceFunctionForwarder listener to use blueprint annotations

Migrate ServiceFunctionForwarder to use blueprint annotations. Blueprint
annotations reduce follows the recommended guidelines for DI [1].

Some minor cleanup.

Change-Id: I8a2889a6fc1871be9d4c4019539e030f10d840a4
Signed-off-by: David Suarez <[email protected]>

Migrate ServiceFunction listener to use blueprint annotations

Migrate ServiceFunction listener to use blueprint annotations. Blueprint
annotations reduce follows the recommended guidelines for DI [1].

Some minor cleanup.

[1] https://wiki.opendaylight.org/view/BestPractices/DI_Guidelines

Change-Id: I296bdb7c6cacc07b8c65209e4cfb881c369d296e
Signed-off-by: David Suarez <[email protected]>

Add a marker file for the project to generate a proper Javadoc

Add the deploy-site.xml marker file that allows the SFC merge job to
generate a proper Javadoc. This Javadoc will be published on [1].

[1] https://nexus.opendaylight.org/content/sites/site/

Change-Id: I4a66cfaa9aaf439bbfa6f7b3c4cc8836b28868e3
Signed-off-by: David Suarez <[email protected]>

Add javax.inject dependency sfc-parent POM

Adding java.inject to sfc-parent to allow modules to include annotations
(@Singleton, @Inject, etc.).

Minor cleanup: moving modules at the beginning of file to better
identify what the aggregator is building.

Change-Id: Ic8ae00c3396b0f9dba8c72849e158eedc4c571b9
Signed-off-by: David Suarez <[email protected]>

Add blueprint maven plugin to sfc-parent POM

Following the best practices in DI described in [1], this
patch add support to SFC for generating the blueprint xml file directly
from annotations (@Singleton, @Inject, etc.).

The patch adds also support for other OSGi annotations as
@osgiservice by including pax-cdi-api.

[1]
https://wiki.opendaylight.org/view/BestPractices/DI_Guidelines#NEW:_.40OsgiService_.26_.40OsgiServiceProvider_.28pax-cdi-api.29

Change-Id: I3d06169cac7db125f70827c6bd1689b74b102590
Signed-off-by: David Suarez <[email protected]>

Remove direct usage of OpenFlowJava models

Remove direct usage of OpenFlowJava models in SfcOpenflowUtils and
replace them with usage of OpenFlowPlugin models.

Resolves: bug 8807
See also: bug 8747

Change-Id: I6c13740e9d2ed60fa0db0a29353b0b199bdb9c8a
Signed-off-by: Tomas Slusny <[email protected]>

Merge "Adding blueprint template"

Add infrautils version to parent pom

Add infrautils project version to the sfc parent pom so
that it can be referenced by other other modules.

Change-Id: Ia1cea8ea352b724061b71faaadc78276be8284f6
Signed-off-by: David Suarez <[email protected]>

Adding blueprint template

- All new features must first have a blueprint.
- This blueprint templates is based on OpenStack blueprints
  and was copied from the Netvirt project.

Change-Id: Id019f22065bdde327a6a4c391f1b8971a9190530
Signed-off-by: Brady Johnson <[email protected]>
Signed-off-by: David Suarez <[email protected]>

Add files to be ignored

Change-Id: Ib4d83e049ba4713524853055cbdc4fd2e9456b4d
Signed-off-by: David Suarez <[email protected]>

Added classifier creation in dovs

To create the classification config we had to use the RestAPI directly.
Using this patch we can use the dovs CLI

Change-Id: I784109c192b22a5590fa3a440fc19a722a023d1c
Signed-off-by: Manuel Buil <[email protected]>

Store SFF Openflow table offsets

- Simple DataObject used internally to specify SFF
  OpenFlow table offsets. To be used primarily to
  pass information from the SFC OpenFlow renderer
  to the sfc-statistics module.

Change-Id: I5c13d081f01e97bc00a3678f33c128f57e347e28
Signed-off-by: Brady Johnson <[email protected]>

Bump odlparent to 2.0.0

- Needed for Karaf 4 migration.

Change-Id: I8e717a7b129433724c5888ff93180d04b7161b27
Signed-off-by: Brady Johnson <[email protected]>

Merge "Improving SFC OpenFlow Flow Names and IDs"

Improving SFC OpenFlow Flow Names and IDs

- Previously the Flow id was an incrementing counter,
  which makes it difficult to easily retrieve flows
  from the data store by the flow ID.

Change-Id: Ie2a87e2748d05efd29b516aa6e2c126e483fdcd7
Signed-off-by: Brady Johnson <[email protected]>

Bug 8543: Pop_nsh should be last

pop_nsh action is no the last one and thus the NSH fields are not copied
correctly

Change-Id: I8f0d8429709cf287989eee803fcc334113967a6d
Signed-off-by: Manuel Buil <[email protected]>

Bug 5995: remove org.json

Nothing appears to use org.json any more in SFC, remove it.

Change-Id: If7d9e909a5ad7e93c1a1f894bde6df7754cb6dc8
Signed-off-by: Stephen Kitt <[email protected]>

Migrating SFC Nitrogen to Karaf4

- Removing Karaf3 features configuration

Change-Id: I5cfda9d9898b3d0acd909df3cb63b45aadda66c5
Signed-off-by: Brady Johnson <[email protected]>
Signed-off-by: Stephen Kitt <[email protected]>

Migrate to odlparent 1.9.0

Change-Id: I473b4304f9ace966bdf64eb17b4d1b4b35d4b9a1
Signed-off-by: Thanh Ha <[email protected]>

Migrate to odlparent 1.8.0-Carbon

Per request of odlparent project we are downgrading all Nitrogen
projects to use the released odlparent 1.8.0-Carbon to allow for the
odlparent project to start performing semver style releases.

Change-Id: Idaadab64a3c227ba6df9acb50607a77b4fd9e5b2
Jira: RELENG-159
RT: 41406
Signed-off-by: Thanh Ha <[email protected]>

Fix minor issues

Fix some minor issues like:
- Useless assignments
- Local variables not used
- Missing logging for some exceptions
- Readability issues

Change-Id: Ibf294ef4d8a69a93869a9505291cf9daf17535a9
Signed-off-by: David Suarez <[email protected]>

Remove redundant modifier

Remove redundant modifier.

Change-Id: I2630144ec06123b223b891f4f21412b6af783999
Signed-off-by: David Suarez <[email protected]>

Remove <prerequisites><maven> from pom.xml to avoid WARNING

[WARNING] The project ... uses prerequisites which is only intended for
maven-plugin projects but not for non maven-plugin projects. For such
purposes you should use the maven-enforcer-plugin. See
https://maven.apache.org/enforcer/enforcer-rules/requireMavenVersion.html

This is already done in odlparent.

Change-Id: I5837047b6a940c91155e7f8f6871fcbbd335a75b
Signed-off-by: David Suarez <[email protected]>

BUG 8367 (2): Avoid certain unicode chars from random test input

A new subtle, random problem was detected in autorelease. Full
unicode space use in the random test input generation has been
removed (really hard to do well; makes junit fail all the time)
Now, random standard characters / numbers are used instead.

Change-Id: If7b1298f368c6fdffc701173b616c1f83b9def99
Signed-off-by: Diego Granados <[email protected]>

BUG 8367. Avoid certain unicode chars from random test input

Avoid randomly-generated test input from containing certain
unicode characters (in the private-unassigned ranges) which
cause trouble with some regexps internally used by junit
Parameterised test runner

Change-Id: Iaa0fa034f40c7084f5485d70685e985e0b082c9b
Signed-off-by: Diego Granados <[email protected]>

BUG 8316: non-NSH packets should be redispatched

SFC is the first serviced registered for the SFs interfaces. If packets do not
have NSH, they should be sent back to the dispatcher table so that they can be
processed by other services. However, right now they are dropped in table 83.

Change-Id: I15bb49bfaaf0f1956c3238243cbeed210342d6cc
Signed-off-by: Manuel Buil <[email protected]>

Complete the README of dovs

I added the SFC instructions which were not included

Change-Id: I942e3b64c49df440a4a8711267885531ad50087f
Signed-off-by: Manuel Buil <[email protected]>

BUG 8249 Correctly handle Logical SFF Chain egress

- If NSH C1 is present, then write C1/C2 to Ipv4TunDst/Vnid
- Minor fix for UT
- When NSH C1 is present, also egress to Vxgpe flow-based tunnel

Change-Id: Ia149c30850c7e55bb7bcd678f73d217ff7231a76
Signed-off-by: Brady Johnson <[email protected]>

Fix checkstyle problems not detected by the current version

This change is required for overall move to new Checkstyle version, see
https://git.opendaylight.org/gerrit/#/q/topic:bumpCheckstyle

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

Bump versions by x.(y+1).z for next dev cycle

Change-Id: I3205e3d12d13531d3bd68751d0c49d09892188c6
Signed-off-by: Anil Belur <[email protected]>

Fix wait for parent interface and device-id

- Do not wait for parent interface if not using ODL.
- Add device-id to neutron ports of guest namespaces, use port uuid as
  value for now.

Change-Id: Ic8fd0b00b6d927e1054cf83e96a85366e9fb1c8d
Signed-off-by: Jaime Caamaño Ruiz <[email protected]>

Bug 4628: Name sfc-vnfm-tacker configfile

Also, add dependency to feature poms.

Change-Id: Id05a5f8014ef8692925837cabec3ccd286502093
Signed-off-by: Vratko Polak <[email protected]>

Add execution permissions to setup_sh_hhe.sh

Fixes the 'Permission denied' error when bringing up the sfc-docker
vagrant.

Change-Id: Ie0ad555b7c47fb19bf21923a2b59fa19f2262763
Signed-off-by: Jaime Caamaño Ruiz <[email protected]>

Migrate features to Karaf4

 - Move old features folder to features/features-sfc
 - Create new features4-sfc folder for Karaf4 pom.xml
 - Create module-specific folders with their respective pom.xml files

Bug: 7629 - Karaf 4 migration: provide Karaf 4 sfc features

Change-Id: I7b5126780b1ae463bd416eeecbd0def18fa2a44e
Signed-off-by: Juan Vidal <[email protected]>

add Mininet wrapper to create SFC topologies and add a MAC Chaining demo

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

add mac chaining implementation as a new SFC OpenFlow Renderer

- Fix minor review issues and add unit tests for mac chaining processor
Change-Id: I309f1c641160ea16f945bea84aeabead6d2be8b7
Signed-off-by: Rafael <[email protected]>

Align dovs to latest carbon updates

- check parent interface after ovs tap add
- use a segmentation id other than 0

Change-Id: I9a36dbf835c5725daba47d67d01c4c7287161427
Signed-off-by: Jaime Caamaño Ruiz <[email protected]>

Declare private constructors for utility classes

Minor change: Declare private constructors for utility classes and make
those classes final.

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

Fix minor issues related to types boxing and unboxing

Fix minor issues related to types boxing and unboxing.

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

Remove duplicate declaration of itm-api in sfc-genius

Change-Id: I6f3a848c3b79fcc001d6c99df45e8fb665bcc5c0
Signed-off-by: Juan Vidal <[email protected]>

sfc-vnfm-tacker checkstyle enforcement

Fix issues related to checkstyle compliance in module
sfc-vnm-tacker:
- Copyright sections
- Line lengths
- Declaration of constants: name, modifiers
- Formatting
- Rename variables
- Move declaration of variables closer to their use
- Exception catching
- printStackTrace()
- Remove unused variables
- Enforcement in pom.xml

Change-Id: Ia6e917833a4f060ae23d83f4aca0e9487b3a0b10
Signed-off-by: Faseela K <[email protected]>

add api changes needed by MAC Chaining implementation

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

Remove sfc-jsonconfig module

This module contains some configuration files that are not needed anymore.

Change-Id: Ided20e1e3c1341bf33bd41a9e53ef0e88ee9641c
Signed-off-by: Juan Vidal <[email protected]>

Fix issues related to checkstyle enforcement for module
sfc-rsp-manager

- Copyright sections
- Line lengths
- Enforcement in pom.xml

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

Remove sfc-network-topology-utils

This submodule does not contain anything, its contents was removed some time
ago. It can be safely removed from repository.

Change-Id: I7dca0dd1cee3c66aa808c6b7d95f0fac13ae7965
Signed-off-by: Juan Vidal <[email protected]>

Fix issues related to checkstyle enforcement for module
sfc-scf-openflow

- Copyright sections
- Line lengths
- Declaration of constants: name, modifiers
- Formatting
- Rename variables
- Move declaration of variables closer to their use
- Enforcement in pom.xml

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

Update sfc104 demo to support nsh proxy and vpp classifier

- Use ovs 2.6.1 + DPDK 16.07
- Enable vpp classifier for vpp demo
- Add an option "--nshproxy" for ./run_demo.sh
- Enable nsh proxy support for ovs, ovs_dpdk and vpp demos

You can run nsh proxy demo with option --nshproxy.

  $ ./run_demo.sh --nshproxy ovs | ovs_dpdk | vpp

Change-Id: I2d9982f6d9f77b0ec4f669940195e19bab48a5ab
Signed-off-by: Yi Yang <[email protected]>

Fix issues related to checkstyle enforcement for module
sfc-vnfm-spi

- Copyright sections
- Line lengths
- Declaration of constants: name, modifiers
- Formatting
- Rename variables
- Move declaration of variables closer to their use
- Enforcement in pom.xml

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

One Chain Service Functions

- If the SF flag one-chain-only is TRUE, then the Service
  Function can only be used in one Service Chain.
- This feature will be used mainly by VXLAN non-NSH,
  once implemented
- Once the VXLAN, non-NSH feature is implemented, if a
  chain is created whose SFs have one-chain-only set TRUE,
  then reclassification wont be neccessary at SF ingress.
- Fixes for code review comments.

Change-Id: Ia6b5584a9c1297ed5f678a3dd875fd57ab3c9044
Signed-off-by: Brady Johnson <[email protected]>

BUG-6859 - Binding generator v1 refactoring

Fix compilation failure after binding generator move.

Change-Id: Idd71975642e173e0779f281cb64e7aceb27efa73
Signed-off-by: Martin Ciglan <[email protected]>

Fix issues related to checkstyle compliance

Fix issues related to checkstyle compliance in module
sfc-sb-rest:
- Copyright sections
- Order imports
- Line lengths
- Formatting
- Rename variables
- Remove unused variables
- Declaration of constants: name, modifiers
- Exception catching
- Move declaration of variables closer to their use
- Checkstyle enforcement in pom.xml

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

Fix issues related to checkstyle enforcement for module
sfc-vpp-utils

- Copyright sections
- Line lengths
- Declaration of constants: name, modifiers
- Formatting
- Rename variables
- Move declaration of variables closer to their use
- Enforcement in pom.xml

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

Fix issues related to checkstyle compliance

Fix issues related to checkstyle compliance in module
sfc-pot:
- Copyright sections
- Line lengths
- Order imports
- Formatting
- Declaration of constants: name, modifiers
- Rename variables
- Move declaration of variables closer to their use
- Enforcement in pom.xml

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

sfc-test-consumer - Checkstyle enforcement

Fix issues related to checkstyle compliance in module
sfc-test-consumer:
- Copyright sections
- Line lengths
- Declaration of constants: name, modifiers
- Formatting
- Rename variables
- Move declaration of variables closer to their use
- Exception catching
- Remove unused variables
- Enforcement in pom.xml

Change-Id: I6c82ded6654b044486934b46a19bc04ed7e8992e
Signed-off-by: Faseela K <[email protected]>

Merge "Fix issues related to checkstyle enforcement for module pot-netconf-renderer"

Fix issues related to checkstyle compliance

Fix issues related to checkstyle compliance in module
sfc-sb-rest:
- Copyright sections
- Order imports
- Line lengths
- Formatting
- Rename variables
- Remove unused variables
- New class AclExporter to avoid having 2 classes in the same file
- New class RspExporter to avoid having 2 classes in the same file
- New class SfExporter to avoid having 2 classes in the same file
- New class SfgExporter to avoid having 2 classes in the same file
- New class SfstateExporter to avoid having 2 classes in the same file
- New class SfstExporter to avoid having 2 classes in the same file
- Remove duplicate constants
- Move declaration of variables closer to their use
- Declaration of constants: name, modifiers

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

Fix issues related to checkstyle enforcement for module
pot-netconf-renderer

- Copyright sections
- Line lengths
- Declaration of constants: name, modifiers
- Formatting
- Rename variables
- Move declaration of variables closer to their use
- Exception catching
- Remove unused variables
- Enforcement in pom.xml

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

sfc-vpp-renderer - enforce checkstyle

Change-Id: I39c179bc22651087f3dd526adbbecece293d84c7
Signed-off-by: Faseela K <[email protected]>

Fix package names to comply with naming conventions

New arrangement of packages to comply with naming conventions.

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

Add missing <name> fields for pom.xml files

This is used by autorelease scripts to automatically parse which project
is failing a build and report to the mailing list automatically. We need
names in the format:

    ODL :: <groupId> :: <artifactId>

This patch formats in the same format as found in the startup archetypes
patch found here: https://git.opendaylight.org/gerrit/52522

Change-Id: Ib62021d93035eff6ada8584e12e68ce4dba3d651
Signed-off-by: Anil Belur <[email protected]>

Fix a minor checkstyle violation in sfc-provider module

Fix a minor checkstyle violation in sfc-provider module.

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

Get rid of double brace initializers

Double brace initialization is evil. In fact, it is considered a java
anti-pattern, since code becomes harder to read, and for each instance
instantiated through this, an anonymous class is created, whose
constructor code is the contents of the double brace.

Simply put, we're better off without it.

Change-Id: I162ca93b2afa7115a41ccc8b7e1f15b475ad0e6f
Signed-off-by: Miguel Duarte <[email protected]>

Fix issues related to checkstyle compliance

Fix issues related to checkstyle compliance in module
sfc-ovs:
- Copyright sections
- Line lengths
- Order imports
- Declaration of constants: name, modifiers
- Formatting
- Rename variables
- Move declaration of variables closer to their use
- Exception catching
- Default clause for switch
- Remove unused variables
- Enforcement in pom.xml

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

Add NSH proxy support in openflow renderer

You can set proxy data plane locator for one SF
to enable NSH proxy for this SF.

        "sf-data-plane-locator": [
            {
                "name": "firewall-1-dpl",
                "port": 4789,
                "ip": SF2_IP,
                "transport": "service-locator:vxlan",
                "service-function-forwarder": "SFF2",
                "service-function-proxy:proxy-data-plane-locator": {
                    "port": 4790,
                    "ip": SF2_PROXY_IP,
                    "transport": "service-locator:vxlan-gpe"
                }
            }
        ]

Then openflow renderer will send VxLAN-gpe + NSH packet
to NSH proxy instead of ths SF, vxlan_tool.py is
changed to support NSH proxy and NSH-unaware SF. NSH proxy
will map VxLAN-gpe + NSH to VxLAN and send VxLAN packet to
NSH-unaware SF. Every session's NSH header is saved in
NSH proxy in order that it can be sent back to SFF intactly.

You can run vxlan_tool.py as NSH proxy by the below command

$ sudo python ./vxlan_tool.py --do=nsh_proxy -i eth1 -v on

Change-Id: I0a34cf5b68709c960218dfdf723a03f532d61305
Signed-off-by: Yi Yang <[email protected]>

Fixes for the SFC103 demo

- the recent Eth+NSH patch introduced a bug that broke the demo
- changing the Vagrantfile to use Ubuntu trusty, since wiley is not available
- Patch Set 3 : removing unecessary LOG

Change-Id: Ibcd2ea2f219bdc57d4d23257b7c58988dcf1e97a
Signed-off-by: Brady Johnson <[email protected]>

Change from odl-genius to odl-genius-api dependences

Both SFC openflow renderer & SFC classifier dependence on
feature "odl-genius" (added during logical SFF development)
is breaking SFC for classic (i.e. not cloud-based) deployments.
This is because depending on the odl-genius features has
the consequence of instructing the feature to start the
Genius-based application coexistance (e.g. to take ownership of table 0),
which is wrong for non-cloud scenarios.
This feature dependence has been changed to odl-genius-api, which
allows code dependences to be resolved without starting
Genius-based application coexistence.

Change-Id: I1eb0921e3c6358f3a142b950e6c09cdf91928916
Signed-off-by: Diego Granados <[email protected]>

BUG 7182: remove M2E lifecycle mapping

This is handled by odlparent.

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

Change package name to comply with Java checkstyle

Change the name of package sfc_ovs to ovs to comply with
Java checkstyle.

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

Adding Eth+NSH Transport

- Implementation for Vxgpe+NSH from classifier-SFF and SFF-SFF, but
  Eth+NSH from SFF-SF.
- The SFFs should have 2 DPLs, one for Vxgpe+NSH and another for Eth+NSH.
- Added SfcRspProcessorNshEth
- Added Unit tests in SfcOfFlowProgrammerTest and SfcOfRspProcessorTest
- Fixes for Code Review comments
- Fixed checkstyle violations
- Fixes for Unit Tests

Change-Id: Iecd0cf7dd9af3395202e809ceda1f2f728228232
Signed-off-by: Brady Johnson <[email protected]>

Add NSH proxy support for VPP renderder

vpp 17.01 and nsh_sfc 17.01 can act as a NSH proxy,
this commit enables VPP renderer to support NSH-unaware
Service Function, in that case, users need to specify
proxy data plane locator for this SF as below:

     "sf-data-plane-locator": [
         {
             "name": "firewall-1-dpl",
             "port": 4789,
             "ip": SF2_DP_IP,
             "transport": "service-locator:vxlan",
             "service-function-forwarder": "SFF2",
             "service-function-proxy:proxy-data-plane-locator": {
                  "port": 4790,
                  "ip": SF2_PROXY_DP_IP,
                  "transport": "service-locator:vxlan-gpe"
             }
         }
     ]

SFF will send VxLAN-gpe+NSH packet to NSH proxy specified by
proxy data plane locator, NSH proxy will send VxLAN+Ethernet
to SF, NSH proxy maintains VxLAN-gpe+NSH to VxLAN+Ethernet
mapping table in order that SFF will get correct VxLAN-gpe+NSH
packets from NSH proxy.

Change-Id: If3ecc82afd6b38142b143e43f541a270f4caa470
Signed-off-by: Yi Yang <[email protected]>

Fix issues related to checkstyle compliance

Fix issues related to checkstyle compliance in module
sfc-openflow-utils:
- Copyright sections
- Line lengths
- Order imports
- Declaration of constants: name, modifiers
- Formatting
- Rename variables
- Move declaration of variables closer to their use
- Exception catching
- Enforcement in pom.xml

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