BUG 6266: Correct augmentation for service-function-forwarders-state

The augmentation overwrites the generated class:
sff-logical-sff-augmentation.

Change-Id: Id40961d82f41245e5377828660dd555927121847
Signed-off-by: danipeon <[email protected]>

Improved debugability of SFC flows

- Based on OPNFV SFC testing, I wanted to be able to easily
  find the code where the complicated transport egress flows
  are created, so I made individual cookie values for each
  of the flows.
- I also removed a pass-through method that wasnt needed to
  further simplify the code.
- Added Cookies for NSH_ETH, VLAN, and MPLS transport egress

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

update sfc103 demo default installation from scratch build to ODL integration

Change-Id: Iea5c3efa9781e99d5e207bac7cbdbf4f53b7c939
Signed-off-by: Guo Ruijing <[email protected]>

Dynamic insert & remove SF

1. refactor sfc103 demo
2. add dynmic insert & remove sf demo

Change-Id: I9b281c1afb989d8f5dc8fc96993027faa0bfd8ce
Signed-off-by: Guo Ruijing <[email protected]>

Merge "Remove wildcard imports"

Remove wildcard imports

Remove wildcard imports to comply with Open Daylight's coding
guidelines.

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

ios-xe renderer - encapsulation removed from SF

Change-Id: Ie82c7495311885da5dda66416271f22d3191215c
Signed-off-by: Vladimir Lavor <[email protected]>

Merge "Only the OVS Bridge name is needed on the SFF."

Make ui-module a provided dependency

Since ui-bundle already includes all of ui-module,
we should make it a provided dependency so it does
not get packaged into the distribution.

Based on a recomendation by Robert Varga and
Alexis Talhouet, and this patch:
https://git.opendaylight.org/gerrit/#/c/41720

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

Only the OVS Bridge name is needed on the SFF.

Previously the OVS Node was specified, but only the bridge name is needed.

Change-Id: Id5ecc2cb91091c306add8a98efcb59c9fca3e4e7
Signed-off-by: Martin Sunal <[email protected]>
(cherry picked from commit 3a40b7d71a64f5911ad9356273a3627eb6627bad)

Update of the vxlan-tool

We add features:

- Act as a firewall blocking specific TCP ports
To do so, use the flag -b when starting the tool.
For example, to block TCP port 22:

python vxlan_tool.py -i eth0 -d forward -b 22

- Decapsulate the NSH
Using the flag -f, it is possible to decapsulate packets
which come with NSH. For example:

python vxlan_tool.py -i eth3 -d forward -f

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

Fix build due to openflowplugin and ovsdb model upgrade

This is needed after https://git.opendaylight.org/gerrit/40798 and
https://git.opendaylight.org/gerrit/40820 are merged.

Change-Id: I13b5625ccd860804d84e6cfc845d643edd774b01
Signed-off-by: Lorand Jakab <[email protected]>

Merge "Enable honeycomb backed SFFs."

BUG 5028 - Test GBP and SFC against Li OFP

migration to openflowplugin Li

Change-Id: I42e3fce86b6a6eb34189ec55f6ea184c14720c95
Signed-off-by: Matej Perina <[email protected]>

Enable honeycomb backed SFFs.

Change-Id: I91382663afbd81b379771eca570bbba2087300f5
Signed-off-by: Ed Warnicke <[email protected]>

Yang definitions for the Logical SFF

  This yang model adds yang definitions for the logical SFF.
  The feature aims for a simpler configuration of SFC, which
  does not need to be reconfigured when working on scenarios where
  the SF are provided by virtual machines which can be executed /
  moved on different compute nodes

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

Addition of the sfc encapsulation type

Addition of the sfc encapsulation type
in the service function path and rendered
service path

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

Sort java import

run the following command to sort

FILES=`find . -type f | grep "/src/.*java$"`

for FILE in $FILES
do
    echo "replace $FILE"

    sed -n -e '1,/^import .*;$/p' $FILE | grep -v '^import .*;$' > $FILE.part1
    sed -n -e '/^import .*;$/p' $FILE > $FILE.part2
    sort -u $FILE.part2 > $FILE.part2.s
    sed -n  '/^import .*;$/,/^END/p' $FILE | grep -v '^import .*;$' > $FILE.part3
    sed -i -e '2,/^$/N;/^\n$/d' $FILE.part3
    cat $FILE.part1 $FILE.part2.s $FILE.part3 > $FILE
    rm -rf $FILE.part1 $FILE.part2 $FILE.part2.s $FILE.part3
done

Change-Id: I3d94e479f60549b9b09c380eaa43dadc8b9ed256
Signed-off-by: Guo Ruijing <[email protected]>

Upgrade sfclisp to latest ietf-inet-types

Change-Id: I73587a3d76c2ed32216c00aeb6f10de516e9ffa4
Signed-off-by: Lorand Jakab <[email protected]>

Break sfc-openflow-renderer's cyclic package dependencies

This change is an strict refactoring and it does not modify
functionality.

In order to break the cyclic dependecies the following moves have been
performed:

- Classes SfcOfProviderUtils, SfcOfBaseProviderUtils, and
SfcSynchronizer have been moved to new package ofrenderer/utils.

- Listeners have been moved to new package ofrenderer/listeners.

- Processors have been moved to new package ofrendere/processors.

- Tests have been updated accordingly.

Now the dependecy graph (roughly) is:

- ofrenderer package uses almost everything but only the yang module
uses it.

- listeners package uses openflow, processors, sfg, and utils.

- processors package uses openflow, sfg, and utils.

- openflow package uses only sfg.

- utils and sfg packages use nothing.

Change-Id: Ia59e12a45694978fc75f5a78b38065fa23c68353
Signed-off-by: Jose-Santos Pulido Garcia <[email protected]>

Updating/Fixing Yang models

- Fixing models so that they pass --ietf
- Updating inet-types and common-yang accordingly
- Fixing all associated code accordingly
- Adding proper change to AbstractDataStoreManager.java for
  ServiceFunctionTypes
- Took care of Ruijing comments
- Took care of Martin comment
- Merged all code
- Add IETF ACL Draft 7

Change-Id: I180494555042034b687044e55bbda816bc6865ec
Signed-off-by: Reinaldo Penno <[email protected]>
Signed-off-by: Guo Ruijing <[email protected]>

Fix unit tests in sfc-provider - SfcServicePathId

Change-Id: Ibd5cd91a2d0e03282ce7a3c9ba2171b2f425ac11
Signed-off-by: Vladimir Lavor <[email protected]>

Service functions are read from datastore. not from netconf device

Change-Id: I85e9a326b3860676ae63365a2a09de0039e23eb7
Signed-off-by: Vladimir Lavor <[email protected]>

Retry transaction when accessing temporary locked netconf device

Change-Id: I56b02e769974c6f32c2c67a150390bc2ed5f2029
Signed-off-by: Vladimir Lavor <[email protected]>

Create reversed rsp name in separate method

Change-Id: I1570d10a0761d030a800dda719208a985d6e4aff
Signed-off-by: Vladimir Lavor <[email protected]>

sfc ios-xe renderer is now using the same ned.yang file as gbp

Change-Id: Icda2e90aa729665e307ee59e1bdbac251f6e64d3
Signed-off-by: Vladimir Lavor <[email protected]>

Fix sfc-provider unit test which could occasionaly fail

Change-Id: Ic2b248527fd46760c99e932b8cae284a19b147d0
Signed-off-by: Vladimir Lavor <[email protected]>

Merge "Fix for bug 6072: race condition in test SfcIpv4PacketInHandlerTest.pktInPurgeBuffering"

Added rsp processing state notifications to ios-xe-renderer

Change-Id: I7e97b6381b2063370447af6d39af9544b70d9095
Signed-off-by: Vladimir Lavor <[email protected]>

Netconf capability check in ios-xe-renderer for sfc

Change-Id: I6a57f49eac36cd557bd996e2a789d8e8a1f39f14
Signed-off-by: Vladimir Lavor <[email protected]>

Fix for bug 6072: race condition in test SfcIpv4PacketInHandlerTest.pktInPurgeBuffering

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

Write openflow flows to datastore in a single transaction per RSP

- Much faster (3x faster on RSP creation; 4x faster on deletion)
- Transactionality will guarantee flow coherence even in the
  event of errors during writeup / deletion
- Initial work by Brady Johnson <[email protected]>

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

Fix for Bug 5879 - Wrong run-time check for variable value

Change-Id: I1ea76572dc40fd634070f4bfa2b7718ca4db1be8
Signed-off-by: Suvitha Balu <[email protected]>

Rename sfcofl2_demo to sfc-openflow-renderer_demo

This patch does not modify code at all.  Only paths and a few comments
have been modified.

Change-Id: I8c6e6d35934e6669d8353d6dc8fc24e2d5038a9c
Signed-off-by: Jose-Santos Pulido Garcia <[email protected]>

Merge "Load karaf features during demo"

updated gitignore for local log files

Change-Id: I9791dbe8c18f49c8371664d1042fc6f6cdc5c5b5
Signed-off-by: Keith Burns (alagalah) <[email protected]>

Load karaf features during demo

Change-Id: I4d2d90059b025dfcfc2ea6081cc2bca0f8b3d4ea
Signed-off-by: Guo Ruijing <[email protected]>

Introduce IOS-XE (Netconf) renderer for SFC

Signed-off-by: Vladimir Lavor <[email protected]>
Change-Id: I3ee145fa0a4e5e9d4da7035f7d47d1add5ee011c

Fix SfcOfRendererDataListener's table offset check

SfcOfRendererDataListener.processConfig(...) used the wrong magic
number.

Bug: 5415
Change-Id: I1cb1e9964eb3528a413a679e93e9318feddbf8f9
Signed-off-by: Jose-Santos Pulido Garcia <[email protected]>

Rename sfcofl2 to sfc-openflow-renderer

Directory sfcofl2 is renamed to sfc-renderers/sfc-openflow-renderer.
Everything in sfcofl2 is renamed accordingly.

Feature odl-sfcofl2 is marked as deprecated. It depends on the new
odl-sfc-openflow-renderer feature.

Change-Id: I1fd74a04629dd31df1348b794479413d11d5f80f
Signed-off-by: Jose-Santos Pulido Garcia <[email protected]>

Fix wrong run-time check on variable in SfcScfOfProcessor.java

Bug: 5871
 - Patch-set 2: added solution to bug 5879

Change-Id: I4733d2874d4a097ff155f4e0dd835c644b8f32c5
Signed-off-by: ecejjar <[email protected]>

RSP manager yang model for SFC

Change-Id: Idc12ac53b48e235293256bc522949a2ff25d1e29
Signed-off-by: Vladimir Lavor <[email protected]>

Add sfc-renderers pom module

The sfc-renderers module will group the RSP renderers (including the
to-be-renamed sfcofl2).

Change-Id: I04776e6fa5f1a4680e1c266c8fa321d6087ffa9d
Signed-off-by: Jose-Santos Pulido Garcia <[email protected]>

Remove maskLength field references in LISP

The maskLength field of the MappingRecord object in lispflowmapping was
deprecated in Beryllium and removed in Boron. The maskLength is now part
of the Eid object, so having a separate (unused) maskLength field is
both redundant and confusing.

Change-Id: Iea6be2b39305e75ed2f324f6f066058fd5c740a2
Signed-off-by: Lorand Jakab <[email protected]>

Derive karaf distribution pom from karaf-parent

This picks up the recent changes to the karaf distro from a common
place.

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

Support building with org.json 20090211

If we revert to org.json 20090211 (see
https://git.opendaylight.org/gerrit/36537 for details), we need to
handle JSONException which is a checked exception in that version.
This change is forwards-compatible.

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

Use https for Nexus in SFC project

Change URLs for Nexus release and snapshot in pom.xml

Change-Id: Idcf254328d59a3d72f848d667eb0521e48654896
Signed-off-by: Miguel Angel Munoz Gonzalez <[email protected]>

Do not deploy the karaf artifact

The karaf artifact is only used for testing. It does not need to be
released to the world.

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

More feature properties clean-up

These slipped through the cracks...

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

Merge "Clean up dependency properties"

Merge "BUG 5170 - statistics should contain timestamp for data historization"

Clean up dependency properties

Migrate to features-parent for features-sfc and use that to drop a
number of dependency properties (pulling in the dependency management
from odlparent).

Remove obsolete properties.

Note that this upgrades PowerMock to 1.6.4 (odlparent now provides
dependency management for Mockito and PowerMock since they need to be
upgaded in sync).

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

Bug 4401:move access-lists-state from ietf-acl.yang

Change-Id: I20e670ee755d82585f72f681a23acf7d391c09d8
Signed-off-by: Ruijing Guo <[email protected]>
Signed-off-by: Robert Varga <[email protected]>

Checkstyle Issues in VNFM

- Compilation actually fails due to these
- Two checkstyle errors

Change-Id: I066c44f6fe343f8c233e95911d22c66272c4c0ca
Signed-off-by: Reinaldo Penno <[email protected]>

BUG 5170 - statistics should contain timestamp for data historization

Change-Id: I8ca833ef7053cf625dcc166f06560a99bd9e8f84
Signed-off-by: Matej Perina <[email protected]>

Merge "UT Adding PacketInHandler UT and misc cleanup"

BUG 5078 : Fixing AppCoexistence table offsets

- https://bugs.opendaylight.org/show_bug.cgi?id=5078
- the offsets werent compensating for the following:
  -- TABLE_INDEX_CLASSIFIER=0 - which is not used for AppCoexistence
  -- TABLE_INDEX_TRANSPORT_INGRESS=1 - which is table 0 for AppCoexistence
  Example: tableBase=20, TABLE_INDEX_PATH_MAPPER=2, should return 20
- Also fixed table egress, so it can be set to 0
- Added SfcL2OfRendererDataListenerTest

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

UT Adding PacketInHandler UT and misc cleanup

- added SfcIpv4PacketInHandlerTest.java
- removed SfcL2FlowWriterTest.java and use mockito instead

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

Bug 5072: Fail to create ACL

Change-Id: Ib002e91ae3e074f6bbfbeda816078498914203a6
Signed-off-by: Ruijing Guo <[email protected]>

Merge "Bug 4977 authentication with tacker fails"

Bump yangtools to 1.0.0-SNAPSHOT

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

Merge "Append ETH+NSH and VxLAN-gpe+NSH send as well as ETH+NSH forward features to vxlan_tool.py"

Bug 4977 authentication with tacker fails

Change-Id: Ic1cda403933d3be444e97fdb6211d4d21e5ff85e
Signed-off-by: Michal Cmarada <[email protected]>

Dont create match any flow on Table 0 for AppCoexist

- https://bugs.opendaylight.org/show_bug.cgi?id=4981

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

Bump versions by 0.1.0 for next dev cycle

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

Merge "implementation of VNFM SPI for Tacker use case"

Merge "Use merge when adding sff bridge"

Use merge when adding sff bridge

The current code uses a put transaction to add the sff bridge to the datastore. This causes the existing bridge to be erased and then recreated. This causes any existing flows already programed to be removed.

Patch set 2: Add fix for unit-test

Change-Id: I1bf223a31fa29c31c8171877fb8b6149885eb93f
Signed-off-by: Sam Hague <[email protected]>

implementation of VNFM SPI for Tacker use case

Change-Id: I59de026fd8ec60f77ddb41e6bda480fb22e07d12
Signed-off-by: Michal Cmarada <[email protected]>

sfc-vnfm-spi

Change-Id: I6d95e49e2d77c5a47cc434dfabb47667f7acaad9
Signed-off-by: Keith Burns (alagalah) <[email protected]>
Signed-off-by: Michal Cmarada <[email protected]>

Augmenting SF DPL with OVS port name

- Bugzilla: https://bugs.opendaylight.org/show_bug.cgi?id=4893
- Patch Set 2 : rebase, address code review comments,
                and add resubmit for table egress
- Patch Set 3 : modify unit tests for App Coexistence resubmit table

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

Append ETH+NSH and VxLAN-gpe+NSH send as well as ETH+NSH forward features to vxlan_tool.py

- Support ETH+NSH packet forward
- Support ETH+NSH and VxLAN-gpe+NSH packet send
- Add --no-swap-ip option to disable source and destination IP swap for some use cases
- Wrap all the prints to functions
- Clean up duplicate code block

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

Merge "Update sfclisp to use ServicePath EIDs"

SFC App Coexistence

    - This patch fixes a previously reverted patch:
      https://git.opendaylight.org/gerrit/#/c/31509/
    - The base table functionality was already implemented, which
      offsets the tables.
    - The TransportIngress table will always be table 0, even if
      the base table has been set.
    - All Goto Table Instructions must also use the table offset.
    - The LastHop TransportEgress table should now goto an Egress
      table instead of outputing to a switch port for Chain Egress.
    - Added relevant Unit Tests
    - For now just match on NSH NSP for vxlan nsh transport Ingress.
      In a later patch we will match on ingress switch port.
    - Patch-set 4 : remove IP from vxlan nsh transport Ingress flow.

Outstanding issues with this patch:
https://bugs.opendaylight.org/show_bug.cgi?id=4929

Change-Id: Ic312b91dedd41b90a677ce680aab5b56a1f9f6f7
Signed-off-by: Brady Johnson <[email protected]>
Signed-off-by: Keith Burns (alagalah) <[email protected]>

Update sfclisp to use ServicePath EIDs

Update sfclisp to insert ServicePath to ELP mappings, instead of
per ACE SourceDests, for each RSP.

Change-Id: Ic8378e050a83d50b8a506e574c2fd34b75d6b892
Signed-off-by: Florin Coras <[email protected]>

Bug 4131 - SFF has no initialization flows

SFFs are still considered as active even after initialization
flows get removed. The flows are cleared when a last RSP is
deleted. Fixed by removing cleared SFFs from list of
initialized forwarders.

Change-Id: I8d5646760ac9e373dee46599541adedbf39dc786
Signed-off-by: Tomas Cechvala <[email protected]>

Merge "Revert "SFC App Coexistence""

Revert "SFC App Coexistence"

This reverts commit 1a1b94e9e2b5c611307c0eb91815a95b1b864a2c.

Change-Id: If4de4c43aa3888936ed04c48fb2e673003b24a46
Signed-off-by: Keith Burns (alagalah) <[email protected]>

Merge "Remove Classifier State from sfc-ui"

Merge "Revert "SFC App Coexistence""

Revert "SFC App Coexistence"

This reverts commit 1a1b94e9e2b5c611307c0eb91815a95b1b864a2c.

Change-Id: Icbad1e79f83427573d26923748bec79ff1263924
Signed-off-by: Keith Burns (alagalah) <[email protected]>

Merge "SFC Demo Rest Regression Fixes"

Merge "SFC App Coexistence"

SFC Demo Rest Regression Fixes

- The idea of this commit is to allow a multi-SFf, multi-SFF demo without
  the need for VMs. All in software under a single interface. This allows
  for much quicker demos and experimentation.
- In order to have multi-SFFs the Agent now constructs data paths on a per SFF basis. Each
  thread looks up the net hops on its own data path.
- sfc_agent.py became too big. Moved all ODL apis into a separate file. Now sfc_agent
  only has the REST server and related functions
- BUG: End of chain packet had wrong offset.
- BUG: More fixes related to Yang model changes
- BUG: NSH packet type should come from NSH header and not global arg
- BUG: Fixed find_sf_locator function. Too many scope problems.
- BUG: Fixed all JSON messages related to demo regression
- BUG: Fixed initial agent info message
- Still need to work on SB Rest side. It will be another commit
- Slightly improved Agent start script by allowing passing ODL IP:port
- Implemented requirements.txt python file

Multi-SFF, multi-SF passes

INFO:sfc.common.services:SFF Server service received packet from ('192.168.1.7', 6008):
INFO:sfc.common.services:dpi service received packet from ('192.168.1.7', 30001):
INFO:sfc.common.services:dpi service received packet from ('192.168.1.7', 30001):
INFO:sfc.common.services:SFF Server service received packet from ('192.168.1.7', 40001):
INFO:sfc.common.services:SFF Server service received packet from ('192.168.1.7', 30001):
INFO:sfc.common.services:firewall service received packet from ('192.168.1.7', 30002):
INFO:sfc.common.services:firewall service received packet from ('192.168.1.7', 30002):
INFO:sfc.common.services:SFF Server service received packet from ('192.168.1.7', 40002):
INFO:sfc.common.services:SFF Server service received packet from ('192.168.1.7', 30002):
INFO:sfc.common.services:napt44 service received packet from ('192.168.1.7', 30003):
INFO:sfc.common.services:napt44 service received packet from ('192.168.1.7', 30003):
INFO:sfc.common.services:SFF Server service received packet from ('192.168.1.7', 40003):
ERROR:sfc.common.services:Could not determine next service hop. SP: 934, SI: 252
INFO:sfc.common.services:End of Chain. Sending packet to 200.200.200.1 20000

Change-Id: I274bed3b67f70a1a394b5063edd4bed56dd95ee1
Signed-off-by: Reinaldo Penno <[email protected]>

Move SfcOvsPortUtils to SfcOvsUtil

Change-Id: I1595405edf26d433d22efa03d08643508e3b02b8
Signed-off-by: Ruijing Guo <[email protected]>

SFC App Coexistence

- The base table functionality was already implemented, which
  offsets the tables.
- The TransportIngress table will always be table 0, even if
  the base table has been set.
- All Goto Table Instructions must also use the table offset.
- The LastHop TransportEgress table should now goto an Egress
  table instead of outputing to a switch port for Chain Egress.
- Added relevant Unit Tests
- Patch Set 3 - rebase
- Patch Set 4 - minor fixes
- Patch Set 6 - Added configurable udpVxlanPort to VxlanTransportIngress flows

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

Bug 4843 - SFC Basic Rest Regression broken after Yang models change

- This first patch fixes all JSON messages for the basic rest regression
- More fixes to come
- Fixed typo

Change-Id: I122a343bb9b9daff81fe157fbcf0aa2132b965c5
Signed-off-by: Reinaldo Penno <[email protected]>

Merge "SFF Form"

Merge "Refactoring RspProcessor"

Clean up SfcOpenflowUtilsTest and re-enable all tests

nextHopGroupIdArray isn't used, remove it.
(int) random() is always 0 (because of rounding), use the default
Random() constructor.
Drop redundant casts.
static methods are final, drop the redundant final modifier.
All the parameter creation methods are used but untrackable, suppress
the unused warnings.
Remove commented code.
Fix shouldThrowExceptionForCreateSetAction() and re-enable it.

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

Refactoring RspProcessor

- Moved convoluted logic from configureIngress() and
  configureEgress() to individual methods for each table.
- Transport specific (mpls, nsh, vlan) logic has been
  moved to a newly created class hierarchy of SfcRspProcessors.
- RspProcessorTest updated to reflect changes. Previously
  there were several duplicate calls to the same Flow
  Programmer methods being made, this no longer occurs.

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

Remove Classifier State from sfc-ui

Classifier state was removed in https://git.opendaylight.org/gerrit/#/c/28584/
Fixed classifier state service removal

Change-Id: Ib6114ce74ef2c7fd14bc74f68170759f10ef415a
Signed-off-by: Ruijing Guo <[email protected]>
Signed-off-by: Stanislav Jamrich <[email protected]>

Cleaning up SfcL2FlowProgrammerOFimpl

- instead of creating a thread class for each type
  of table modification, just create the flow and
  have one thread class for writing to, and another
  for removing from the data-store
- some simple API and logic cleanups
- more cleanups to SfcL2FlowProgrammerOFimpl coming
  in the RspProcessor refactor:
  https://git.opendaylight.org/gerrit/#/c/30859/
- PatchSet 3: Adding javadoc
- PatchSet 4: Rebase
- PatchSet 5: Rebase
- PatchSet 6: Minor changes from code review comments

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

Merge "sfc.sfc_lisp.provider.api cleanup"

Merge "Adding buffering for TCP Proxy PktIn packets"

Merge "Add missing license headers"

Merge "Add SFC Standalone classifier demo"

sfc.sfc_lisp.provider.api cleanup

Change-Id: Ic7ec747ea10ef75e6133ddb3db195239021350e0
Signed-off-by: Matej Perina <[email protected]>

Adding buffering for TCP Proxy PktIn packets

- Avoids updating data store for every packet in a flow
- The buffer key is the src/dst IP
- buffer max time is 60 seconds

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

Merge "BUG-4697: move features to top level"