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]>

Change package name to comply with Java checkstyle

Change the name of package transactional_writer to writer to comply with
Java checkstyle.

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

Docker environment + SFC e2e traffic tests

-Add some updates to 'dovs' utility (that simulates neutron networks)
-Add tools to test end to end traffic for Service Function Chaining
with NSH in Logical-SFF configurations
-Add a Simple Service Function HTTP Header Enrichment (with its own tests)
-Add brief documentation (README.asciidoc files)
-Add Test cases see below

Example demo for testing the Logical SFF end-to-end SFC traffic
-Start the Controller (e.g. that run on 172.28.128.3)
-Start the 'dovs' environment:
cd sfc-test/sfc-docker/
vagrant destroy -f && vagrant up && vagrant ssh
> cd /vagrant/sf_hhe/logical_sff

A. SFs in a DIFFERENT subnets
-1 Test traffic in two Service Function Chains
> sudo ./test_case_01_traffic_different_subnet.sh -o 172.28.128.3
-2 Test traffic in one Service Function Chain
 + Test traffic after moving one VM to other compute node
> sudo ./test_case_02_move_vm_different_subnet.sh -o 172.28.128.3

B. SFs in the SAME subnet
-3 Test traffic in two Service Function Chains
> sudo ./test_case_03_traffic_same_subnet.sh -o 172.28.128.3
-4 Test traffic in one Service Function Chain
 + Test traffic after moving one VM to other compute node
> sudo ./test_case_04_move_vm_same_subnet.sh -o 172.28.128.3

- Clean the network topology in 'dovs'
> sudo dovs clean --odl 172.28.128.3

Executing tests from A and B requires restart ODL Controller and dovs

-There are many logs that will be stored in ./logs
(e.g. To see TCP packages in client of first Chain for test case 01
 > tcpdump -qns 0 -A -r ./logs/client1_tcpdump.pcap tcp )

- Check SFC configuration
> ./GetSfcConfig.py -v -ip 172.28.128.3

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

Fix issues related to checkstyle compliance

Fix issues related to checkstyle compliance in module
sfc-ios-xe-renderer:
- Copyright sections
- Line lengths
- Order imports
- Declaration of constants: name, modifiers
- Formatting
- Remove redundant modifiers: public in interface methods, final, etc.
- Rename variables
- Move declaration of variables closer to their use

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

Clean up and fix RSP and service classifier remove issues

When a RSP is removed, the corresponding vpp configuration
should be removed and cleaned up, vpp classifier configuration
also should be removed correctly after a service classifier
is removed.

This patch makes sure the related VxlanGpe ports, nsh entries,
nsh maps, vpp classify tables and vpp classify sessions are
removed and cleaned up after RSP or service classifier is
removed.

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

Remove overall checkstyle activation in SFC

The overall checkstyle activation is not defined locally
in the specific projects as SFC anymore, but in the odlparent's POM.

This patch remove the local files stored in the
sfc-parent directory referenced by some modules like
the sfc-network-topology-utils.

The actual enforcement will be done on a per-module
basis, as in the following:

https://git.opendaylight.org/gerrit/#/c/51040/2/sfc-netconf/pom.xml

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

Fix issues related to checkstyle compliance

Fix issues related to checkstyle compliance in module sfc-provider:
- Copyright sections
- Line lengths
- Order imports
- Declaration of constants: name, modifiers
- Formatting
- Remove redundant modifiers: public in interface methods, final, etc.
- Rename variables
- Move declaration of variables closer to their use

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

Fix issues related to checkstyle compliance

Fix issues related to checkstyle compliance in module sfc-provider:
- Copyright sections
- Line lengths
- Order imports
- Declaration of constants: name, modifiers
- Formatting
- Remove redundant modifiers: public in interface methods, final, etc.
- Rename variables
- Move declaration of variables closer to their use

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

Merge "Fix issues related to checkstyle compliance"

Fix issues related to checkstyle compliance

Fix issues related to checkstyle compliance in module
sfc-provider:
- Copyright sections
- Line lengths
- Order imports
- Declaration of constants: name, modifiers
- Formatting
- Remove redundant modifiers: public in interface methods, final, etc.
- Rename variables
- Move declaration of variables closer to their use

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

Add VPP classifier

From vpp, honeycomb, hc2vpp and nsh_sfc 17.01 on,
vpp can act as sfc classifier as ovs-dpdk did before,
this patch adds a new classifier sfc-scf-vpp, it can
translate IETF ACLs into vpp classify tables and
sessions. With this commit, sfc can achieve NSH-based
service function chaining only based on vpp instead ovs,
this makes people have another choice to do service
function chaining.

By the way, NSH implementation has been part of vpp
releases, so we needn't any extra patch for NSH.

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

Fix issues related to checkstyle compliance

Fix issues related to checkstyle compliance in module
sfc-openflow-renderer:
- Copyright sections
- Line lengths
- Order imports
- Declaration of constants: name, modifiers
- Formatting
- Remove redundant modifiers: public in interface methods, final, etc.
- Rename variables
- Move declaration of variables closer to their use

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

Merge "Fix issues related to checkstyle compliance"

Merge "Fix issues related to checkstyle compliance"

Merge "Delete SFF VXGPE port upon SFF deletion"

Fix issues related to checkstyle compliance

Fix issues related to checkstyle compliance in module
sfc-openflow-renderer:
- Copyright sections
- Line lengths
- Declaration of constants: name, modifiers
- Formatting
- Group overloaded methods
- Remove redundant modifiers: public in interface methods, final, etc.
- Move variable declarations to be closer to their usage

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

Fix issues related to checkstyle compliance

Fix issues related to checkstyle compliance in module
sfc-openflow-renderer:
- Copyright sections
- Line lengths
- Declaration of constants: name, modifiers
- Formatting
- Group overloaded methods
- Remove redundant modifiers: public in interface methods, final, etc.
- Move variable declarations to be closer to their usage

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

Change package name to comply with checkstyle

Change package name to comply with checkstyle rules and other minor
changes to follow its rules.

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

Delete SFF VXGPE port upon SFF deletion

- Added sfc_ovs/provider/listener unit tests

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

docker environment to test sfc

This directory provides:

- An utility, 'dovs', to setup neutron networks where compute nodes
  are containerized openvswitch instances and VMs are isolated network
  namespaces connected to such instances.
- A vagrant driven virtual environment prepared to run dovs

The plan is to further add utilities to test SFC on top of this
environment, and will also be used for CSIT SFC.

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

Merge "Bug 3712: SFP validation of SF types at creation time"

Minor changes in sfc-genius module to comply with checkstyle

Minor changes in sfc-genius module to comply with checkstyle.

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

Bug 3712: SFP validation of SF types at creation time

This commit makes use of DOMDataTreeCommitCohorts (callbacks
that can be registered in order to participate in commit
validation) in order to make sure that newly-added datastore
SFPs are consistent with previously defined SFCs and SFs.
Specifically, it is checked that the types for the SFs the
SFP references are consistent with the SF types that the
SFC specifies.

Since the datastore commit transaction is kept open during
validation, this validation must be as fast as possible.
Controller guidelines about not opening new datastore
reads in order to validate a DS write have been followed
to the maximum extent; for this reason, a datastore cache
has been deployed to cover both SFs and SFCs. On systems
with less than 500 SFs & SFCs deployed, caches will provide
total coverage (i.e. validation will never need to access
datastore)

Testing: manually tested (complex interactions between
controller-mdsal-scf are too hard to simulate). Proper CSIT
tests will be delivered to the sfc suites in the integration
project.

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

Update vpp YANG models to 17.01 release

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

Rename sfc-ios-xe-renderers packages to comply with checkstyle

Rename sfc-ios-xe-renderers packages to comply with checkstyle.

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

Bug 7554: Update GUI after deprecating nsh-aware in SF and other changes in model

After the following commits, GUI should be updated:
1) https://git.opendaylight.org/gerrit/#/c/49687/
   Removing deprecated SF fields (nsh-aware)
2) https://git.opendaylight.org/gerrit/#/c/49546/
   Deprecating the SFC and RSP symmetric flags

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

SFC Proof of Transit SB changes.

1. Add changes to handle RPC trigger to generate PoT configuration.
2. Add timer wheel and configuration refresh related timer code.
3. Add core SFC PoT configuration generation APIs
4. Utilities for listening to node arrivals and sending via NETCONF
5. Add unit tests.
6. Design guide and documentation: https://github.com/CiscoDevNet/iOAM

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

Use operational for DPN term. service actions

Remove in-memory handling of DPNs for terminating service actions.
Replace with listener for changes on the existing DPN to RSP map
already available on operationa data store
(service-function-forwarder-logical.yang, dpnid-rsps grouping).

In detail:

- Main module interface has been moved and renamed, from
  impl/handlers/ISfcGeniusInterfaceServiceHandler to
  impl/SfcGeniusServiceManager. This is because the module
  is no longer centered on interface events as it now responds
  to DPN related events.
- SfcGeniusServiceManager has two new methods, bindToNode and
  unbindFromNode, to create and remove the SFC terminating service
  action on the nodes.
- SfcGeniusServiceManager no longer has an interfaceStateDown method
  which only purpose was to remove the SFC terminating service action
  on VM migration. Note that now this is delayed up until the VM
  migration completes: interfaceStateUp on a new Node triggers
  the re-rendering of the appropiate RSPs which updates the map
  of DPN to RSP, which in turn triggers the removal and creation
  of the SFC service terminating action on the appropiate nodes.
- A new listener, SfcGeniusSffDpnStateListener, that in response
  to changes on the DPN to RSP map, will trigger SFC service
  binding or unbinding towards the DPN.

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

Act on sonar reported issues / technical debt

Stop using the 'order' local variable in the SfcOfFlowProgrammerImpl
methods; since we're only appending the action to the action list, I
think it is more explicit / less error-prone to indicate the 'order' as
the current-list size (always append to the end of the list)

The configureNshNscTransportEgressFlow in the SfcOfFlowProgrammerImpl
class had a bug; the next protocol was not deing correctly defined as
NSH - as a result, that action list only featured a single action, which
is to set the output port. Now it has an action setting the next
protocol to NSH, and another setting the output protocol.

Tests are updated accordingly.

Used a StringBuilder instead of a StringBuffer in the SfcOfRspProcessor
because the StringBuffer is thread safe, which, in this scenario only
leads to throttled performance.

Removed the TUN_GPE_NP_NSH constant from yet another class.

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

Enforce checkstyle in sfc-sfclisp module

Checkstyle enforcement for sfc-sfclisp module:
- Remove commented blocks
- Formatting
- Lines length
- Re-factor switch
- Replace catching of generic exceptions
- Move overloaded methods
- Fix Javadocs
- Protected constructor for utility classes
- Order imports
- Change attributes names

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

Enforce checkstyle in sfc-genius module

Checkstyle enforcement for sfc-genius module
- Formatting
- Fix Javadocs
- Order imports
- Change attributes names
- Replace catching of generic exceptions
- Lines length
- Enforce rule in POM

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

LSFF Fix: add NSH as NP at SFF->SFF egress

Also test changes

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

Enforce checkstyle in sfc-netconf module

Checkstyle enforcement for sfc-netconf module
- Protected constructor for utility classes
- Formatting
- Fix Javadocs
- Order imports
- Change attributes names
- Replace catching of generic exceptions
- Remove commented blocks

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

Replace mockito-all by mockito-core (see Bug 7662)

Change-Id: I884d88f940955788bf7d5f0194183e198c9fad0e
Signed-off-by: Michael Vorburger <[email protected]>

Split sfc-vpp-utils bundle from sfc-vpp-renderer for sfc-scf-vpp

sfc-scf-vpp (SFC VPP classifier) needs to use common APIs from
sfc-vpp-renderer, so split them as a single bundle sfc-vpp-utils
for sharing between sfc-vpp-renderer and sfc-scf-vpp (the following
commit will implement it).

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