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

Merge "Rename package sfc_lisp to sfclisp to comply with checkstyle."

Rename packages to comply with checkstyle

Rename packages to comply with checkstyle:
- org.opendaylight.sfc.sfc_netconf.provider to
  org.opendaylight.sfc.netconf.provider

- org.opendaylight.sfc.sfc_netconf.provider.api to
  org.opendaylight.sfc.netconf.provider.api.test

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

Rename package sfc_lisp to sfclisp to comply with checkstyle.

Change package name to comply with checkstyle

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

Enforce checkstyle in sfc-model module

Checkstyle enforcement for sfc-model module.

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

BUG-7555. Crash on RSP rendering when using a Logical SFF

The bug was triggered because of the code uncorrectly comparing two
different DpnIdType objects using the == comparator. Even though
both objects refer to the same dpnid, the objects are not the same,
thus triggering the error path (asking genius for a tunnel interface
between two different compute nodes but passing the same dpnid for
both tunnel ends). Incorrect component tests corrected; an additional
test for the condition will also be included in CSIT

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

Merge "Model changes for VXLAN transport/encap"

Merge "Clean the usage of boolean expressions"

Clean the usage of boolean expressions

Clean the usage of boolean expressions and other minor improvements for
readability.

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

Model changes for VXLAN transport/encap

- Added VXLAN transport
- Added single chain SF parameter

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

Make code more explicit and less redundant

  - Make better use of java types
  - Remove duplicated code
  - Remove compiler warnings

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

Merge "GUI - Add default redirect on app start"

GUI - Add default redirect on app start

Due Dlux split we removed Topology app from odl-dlux-core feature.
There were also default redirect to that app, so it was landing page, when
DLUX was started.
This patch adds default route to GUI in this project.

More info:
https://wiki.opendaylight.org/view/Weather#Rename_some_odl-dlux-.2A_features_to_odl-dluxapps-.2A

Change-Id: I025ffe3403392458207af0fea52b57af75bb22b0
Signed-off-by: Daniel Malachovsky <[email protected]>

Replace Arrays.asList by Collections.singletonList.

It is a minor performance improvement.

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

Move sfc-scf-openflow into sfc-classifiers

We want to add VPP classifier for sfc, so do this change to adapt to multiple sfc classifiers.

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

sfc104 demo: fix two issues led by proxy environment variable and recent sfc YANG model changes

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

Merge "Remove explicit type boxing."

Remove explicit type boxing.

Autoboxing is applied improving readability.

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

Add private constructor.

Add private constructor to utility class to avoid instantiation.

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

Refactor the Classifier Processor classes.

By making the flow generator classes return FlowDetails instead of
FlowBuilders, both the OpenflowClassifierProcessor and the
ClassifierRspsUpdateListener classes are simpler, since they are no
longer responsible of building the FlowDetails object.

The dpdk flow generation was also taken to the classes
implementing the ClassifierInterface (BareClassifier &
LogicallyAttachedClassifier), thus making the processAce simpler.

This enables the processAce and processAcl methods on those classes to
become simpler, thus reducing their cyclomatic complexity - overall
indicator of complexity.

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

Remove technical debt from the project

Break package cycles between scfofrenderer,
scfofrenderer.logicalclassifier, and scfofrenderer.rspupdatelistener
packages.

This was found by analyzing the sonarcube results, and means that these
3 modules are tightly coupled. By restructuring the packages, the
circular dependencies are fixed.

Modified the OpenflowClassifierProcessor, attempting to make it simpler.
In a latter commit, its methods will be split, thus simplifying it quite
a lot.

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

Modified JSON files in resources to adapt them to deprecated fields in Service Functions
Added one configuration example using bidirectional field in SF type

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

Merge "Removing deprecated SF fields"

Removing deprecated SF fields

- The SF fields nsh-aware and request-reclassification were
  deprecated in Beryllium. The fields were added to the
  SF-type, so use those instead.

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

Classifier reacts to first SF location updates

Whenever the first SF in an RSP migrates to another compute node, the
classifier rules are updated accordingly - whether sending the packet
through a different tunnel connected to the new compute node, or, in
scenarios when the SF moved to the compute node where the classifier is
installed, by jumping directly to the SFC transport ingress table.

The helper methods on the OpenflowClassifierProcessor were moved into a
new class - ClassifierHandler.

The ClassifierRspUpdateDataGetter was finished, providing 2 methods:
  - filter all ACLs whose ACEs impact the supplied RSP
  - filter all ScfClassifier objects that implement those ACLs

The ClassifierRspsUpdateListener was added logic on how to identify the
movement of the first SF in any RSP, and also triggers the classifier
flows update through the ClassifierRspUpdateProcessor.

The ClassifierRspUpdateProcessor follows the same principle of the
OpenflowClassifierProcessor - build all the flows to be installed. A new
class was created to separate the concerns (classifier updates have the
RSP as its single argument, whereas classifier add/delete have the ACL
and the ScfClassifier objects as arguments).

Everything else are tests (+- 600 lines)

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

Merge "Deprecating the SFC and RSP symmetric flags"

Handle RSP updates on the classifier APIs

The APIs leveraged by the classifier were changed enabling it to update
its flows when the first SF in an RSP migrates to another compute node.

New classes (to be implemented in the next commit):
  - ClassifierRspUpdateDataGetter
This class queries MD-SAL for all ACLs whose ACEs target the supplied
RSPs, and also queries MD-SAL for all Classifiers enforcing the filtered
ACLs.

  - ClassifierRspsUpdateListener
Identifies relevant RSP updates (RSPs whose 1st SF neutron port changed
data-plane IDs), and processes that RSP update - deleting the old
impacted flows, and creating new ones.

API changes:
  - OpenflowClassifierProcessor
The OpenflowClassifierProcessor now *has* a BareClassifier, and a
LogicallyAttachedClassifier object, injected through the constructor.

Which classifier(Bare vs LogicallyAttached) is used for each processed
request depends on the type of SFF to which it is connected to.

  - SfcNshHeader
The SfcNshHeader class now encapsulates an RSP object, which can be set
through a different constructor. Using this allows us to save DS reads
on RSP updates, since we are notified with the new object.

  - SfcScfOfProcessor
The SfcScfOfProcessor class got simpler, and now does not have to
encapsulate and inject the transaction object into the OpenflowWriter
and OpenflowClassifierProcessor.

  - SfcScfOfRenderer
The SfcScfOfRenderer class now also features an update handler for RSPs.
Also, it injects the required dependencies into the
OpenflowClassifierProcessor and OpenflowWriter objects, while making
them share the same transaction objects.

  - SfcOfFlowWriterInterface
The SfcOfFlowWriterInterface and its subclass now encapsulate an
EventBus, and expose a transactional API. Clients that want to share the
same transaction object as the OpenflowWriter register themselves, and
are notified whenever the OpenflowWriter commits a transaction.

Also, the FlowDetails class manipulated by the SfcOfFlowWriterInterface
now encapsulates the used RSP path ID, making it easier / more efficient
for the SfcOfFlowWriterInterface clients to delete a set of related
flows.

Log messages are tuned throughout the classifier code, along with the
used log levels.

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

Deprecating the SFC and RSP symmetric flags

- Chain symmetry is now a function of the SF-type bidirectionality flag
- The SFP symmetry flag, if present, will override the SF-type
  bidirectionality flag
- Minor cleanup of the SfcProviderRpcTest test

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

Merge "Add blueprint for sfc-genius module"

Add unit test for logical SFF RSP Processor with SFs in different compute nodes

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

Add blueprint for sfc-genius module

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

BUG 7350: NullPointerException when creating RSP without any topology node connected

The bug is triggered when looking for a dpdk port during RSP
rendering. An unchecked-for-null iteration is performed in
the topology nodes list. Now the list of nodes is checked
for null before iterating

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

BUG 7204 : Typo in method names of sfc-vnfm-tacker

There is a typo in method names of sfc-vnfm-tacker
In this patch it is changed accordingly

Change-Id: I5f2b66979ce9944cb753d68a124b91024cc8c30c
Signed-off-by: venkatamahesh kotha <[email protected]>

BUG 4221 When deleting a RSP remove symmetric path if necessary

Solution and Unit Test for:
Bug 4221 - When deleting a RSP need to remove symmetric path if it exists

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

Updates to SFC Proof of Transit feature.

1. Add RPC trigger to enable SFC Proof of Transit feature.
2. Add changes to handle RPC trigger to enable the feature.
3. Augmentation updates to RSP.
4. Add unit tests to verify the changes.
5. Design guide and documentation: https://github.com/ciscoDevNet/iOAM

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

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 <[email protected]>

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 <[email protected]>

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 <[email protected]>

Merge "Fix clearly badly broken Blueprint wiring"