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

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"

Add missing license headers

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

Adding UT for SFCOFL2 package:

    - Checking if flows written in datastore are properly built

Change-Id: Ia475c2f5471226551913af1fc3a5a770ac28155b
Signed-off-by: Ricardo Noriega <[email protected]>

BUG-4697: move features to top level

RFC6020 is explicit about where a feature can be defined
(https://tools.ietf.org/html/rfc6020#section-7.18.1). Fix the model so
it does not block yangtools enforcement patch.

Change-Id: I555cefeb39ed9c50904b16765fc9353cc9dcaef1
Signed-off-by: Robert Varga <[email protected]>

Add SFC Standalone classifier demo

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

Merge "update according to yang model change"

update according to yang model change

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

Merge "SfcNetworkTopologyUtils cleanup"

SfcNetworkTopologyUtils cleanup

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

The changes to this code were done to support the recent SFC demo done by
Sam Hague and Tim Rozet.  Keith Burns also used a version of it in a
GBP-based demo.

This code was modified to
* run with Python 2.7
* swap the IP SA and DA when forwarding.
* Work for either UDP port 4790 or 6633
* Don't forward NSH frames unless service index is > 1
* Added verbose option to turn off prints when in forward mode

Options:
* -i: Interface to receive, and optionally send packets on
* -d: Mode: 'dump' or 'forward'.  Default is dump.
* -v: Verbose: 'on' or 'off'.  Default is 'on', and 'off' only works
      in 'forward' mode.

Example Use:
sudo python vxlan_tool.py -i 'eth1' -d 'forward' -v 'off'

Change-Id: I14ddefcd2bdde3c681f69878ad817f464cb520d4
Signed-off-by: Andre Fredette <[email protected]>

Addition of SF DPL to SFF DPL mapping.

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

SFC Standalone Classifier

1. Add unit test for SfcMatch
2. Add unit test for SfcScfOfScfDataListener
3. Add unit test for SfcScfOfScfProcessor
4. fix bug in SfcServicePathIdTest

Todo List:
1. add more unit tests
2. move SfcOvsPortUtils to sfc-ovs
3. move SfcNshHeader to sfc-provider

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

Update and re-enable sfclisp

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

Merge "Remove sfclisp from build"

Remove sfclisp from build

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

SFC data model to config OpenFlow table offsets

- For now, the Sfc OpenFlow Renderer data model only
  has attributes for OpenFlow tables, but in the future
  more attributes can be added.
- Introduced SfcSynchronizer class to be able to synchronize
  FlowTable offset changes with Rsp modifications.
- Introduced SfcL2OfRendererDataListener, that listens for
  changes to Sfc OpenFlow Renderer data model changes.
- Not UnitTested yet, waiting for patch 29986 to be merged,
  after which SfcL2FlowProgrammerOFimpl UT will be refactored.
- TableEgress feature not implemented yet. I want to refactor
  SfcL2FlowProgrammerOFimpl first, which will make implementing
  the feature much easier.
- Patch Set 2 - rebase

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

Change to classifier.py fixing issue with dict keys.

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

Allow socket reuse in IPTABLES classifier

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

Merge "SFC UI - broken forms fix"

Merge "sfc_ovs.api cleanup"

sfc-provider.api cleanup

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

sfc_ovs.api cleanup

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

SFC UI - broken forms fix

- ACL table now shows data
- add/edit ACL data is working
- fixed template directive to not rewrite existing data

Change-Id: Icc3d40a9962053f09e40d7834b5f3b1e994d68e1
Signed-off-by: Daniel Malachovsky <[email protected]>
Signed-off-by: Stanislav Jamrich <[email protected]>
Signed-off-by: Daniel Malachovsky <[email protected]>

Move network topology APIs from sfc-provider to sfc-util

This will utterly remove dependency on l2switch from
sfc-provider, network topology APIs are put into a bundle
sfc-network-topology-utils under sfc-util.

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

Merge "Change sfc-util for accommodating multiple bundles"

Merge "SfcUICosmeticChanges2"

Change sfc-util for accommodating multiple bundles

Current sfc-util only has one bundle, it is actually
openflow utils, network topology utils will be moved
into this in order that sfc-provider won't depend on
l2switch. So this commit renames current sfc-util
bundle to sfc-openflow-utils and chnage sfc-util to
accommodate other utils.

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

Merge "Remove executable bit on pom files"

Implementation of Service Function Type Definitions

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

SfcUICosmeticChanges2

- added ngTable search placeholder

Change-Id: I4671b496344a07a70770ec9c73046be6c847e3c5
Signed-off-by: Stanislav Jamrich <[email protected]>

Merge "SFC UI cosmetic fixes"

Merge "sfc_ovs.util cleanup"

SFF SF-dict sff-sf-dpl is a name, not a DPL

- Instead of redefining the DPL in this dictionary,
  refer to the SF DPL name.
- Allows for much better referential integrity
- Patch Set 6 removed duplicate SF type
- Patch Set 7 fixes the python test failures
- Patch Set 8 adds sf retrieval from ODL to the sfc_agent
- Patch Set 9 and 10 rebasing
- Patch Set 12 and 13 rebasing again
- Patch Set 16 fixed problem with python list vs dict

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

SFC UI cosmetic fixes

- cosmetic fixes in SFC UI

Change-Id: I4049941f1f656fc3fb14c60107f0cce946547437
Signed-off-by: Stanislav Jamrich <[email protected]>

Remove executable bit on pom files

pom.xml files are metadata and do not require executable permissions.

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

SFC provider unit test clean up

    - Replacing C-style arrays to Java lists

Change-Id: Icbd3459a66d358d61027367d5abd854ed8c4ff81
Signed-off-by: Ricardo Noriega <[email protected]>

Merge "SFCOFL2 Unit Test improvements"

sfc_ovs.util cleanup

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

Yang model for Abstract Type Definition of Service Functions

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

SFCOFL2 Unit Test improvements

- Now using mockito to track calls to the OpenFlow renderer

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

Fixed tests using default value

- fixed tests based on change
https://git.opendaylight.org/gerrit/#/c/28251/

Change-Id: Ifbdc51ed7dd68ac5b2c8e4b56260643880e59134
Signed-off-by: Martin Sunal <[email protected]>

Add code style check

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

Add NSH C3 C4 move action

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

Merge "Yin files creation fix"

Merge "Fix for ACL YANG changes"

Fix for ACL YANG changes

- URI sent to iptables classifier changed, yet Task sending old one.

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

Adding sequential path generation algorithm.

- Random generation breaks sfcofl2 vlan demo
- Also added impl code for the service path id RPCs
- Sequential generation Unit tests included

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

Feature clean up

- centralised and homogenised
- removed openflowplugin-all and netconf-all !!!!
- no more spurious log messages during startup
- removed un-needed depedencies
- removed redundant feature dependencies inside feature defn.

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

Pom file clean up.

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

Yin files creation fix

- added search path to convert_yang_to_xml.sh script
- added yang files which are included into sfc models in sfc-model folder

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

Revert "Bug 4544 - SFC-VNFM Interface Implementation"

This reverts commit b31d270a05ea744ff8d539745e04738d836cf119.

Needs further discussion.

Change-Id: I22da812c27c83eb2084a403de5b665d1a0d5eb4e
Signed-off-by: Johnson Li <[email protected]>

Bug 4544 - SFC-VNFM Interface Implementation

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

Bug 4507

 - Fixed script generating YIN files from Yang modules in the project.
 - added empty yang2xml folder
 - fixed many of the JS errors
 - deleted unnecessary files

 - more info @ https://bugs.opendaylight.org/show_bug.cgi?id=4507

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

Bug 4528 - SFC SBRest Broken

- Commit https://git.opendaylight.org/gerrit/#/c/28539/
  introduced a new Yang model sfc-common.yang with typedefs
  for names used throughout other models.

- Although the typedefs resolve to strings, the typedefs
  themselves are instantiated as objects. Therefore in order
  to get the actual value of, say, the SF name from a SF
  object, first you get a handle to the object associated
  with the typedef and then perform a getValue().

- Introduced try/catch around jersey client methods
  in order to catch future issues

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

Fix for YANG revisions not being explicit in import.

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

Merge "Add exception handle for SfcProviderRenderedPathAPI & SfcNshHeader"

Add exception handle for SfcProviderRenderedPathAPI & SfcNshHeader

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

Fixing sfc-scf-openflow naming/class convention

- Module, namespace and file name need to be consistent in
  sfc-scf-openflow-provider-impl.yang

- Fixes on .yang required corresponding changes in package
  name, class name, etc

- Fixed all possible Yang issues in sfc-provider-impl.yang.
  I do not think this is the cause of any issues but fixed
  everything according to pyang

- Fixed initial config files

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

fix NullPointerException in SfcLispUtil

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

Bug 4481 - Standalone SFC classifier support

  1. add more ACL match for SFC classifier
  2. clean up some code
  3. add some exception handle

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

Merge "Append NSH-aware SF to vxlan_dump.py and rename as vxlan_tool.py"

Append NSH-aware SF to vxlan_dump.py and rename as vxlan_tool.py

Usage: vxlan_tool.py [-h] [-i INTERFACE] [-d dump | forward]

This is a VxLAN/VxLAN-gpe + NSH dump and forward tool, you can use it to dump
and forward VxLAN/VxLAN-gpe + NSH packets, it can also act as an NSH-aware SF
for SFC test when you use --forward option, in that case, it will
automatically decrease nsi by one.

optional arguments:
  -h, --help            show this help message and exit
  -i INTERFACE, --interface INTERFACE
                        Specify the interface to listen
  -d {dump,forward}, --do {dump,forward}
                        dump/foward VxLAN/VxLAN-gpe + NSH packet

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

Fix charset encode issue in sfc-py/MANIFEST.in

Before fix:
$ iconv -t ascii sfc-py/MANIFEST.in
iconv: illegal input sequence at position 0
$

After fix:
$ iconv -t ascii sfc-py/MANIFEST.in
graft nfq
graft test
include start_agent.sh tox.ini README.rst

prune unused
global-exclude *.pyc
recursive-exclude * __pycache__
$

https://pypi.python.org/pypi/sfc has same issue because of this
bad encode, this results in "pip3 install sfc" failure.

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

Merge "Introducing typedef and leafrefs for strings."

Merge "Add a SFC NSH test tool vxlan_dump.py"

Add a SFC NSH test tool vxlan_dump.py

This is a pure python script, it can dump Ethernet
header, IP header, UDP header, NSH base header and
NSH context header, this is very helpful for SFC NSH
test.

It includes much code from sfc-py, the intention
is to make sure it is self-contained and hasn't too many
dependencies and packeges installation, in addition,
it does need some changes, the original APIs can't meet
its requirements.

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

Fix errors in compute_internet_checksum in corner case

When data length is odd, compute_internet_checksum will
hit a corner case, this results in two errors:
  1. pad = bytearray('\x00')
  2. out-of-boundary access to 'data'

This commit fixes them and it is verified in my python-based
SF.

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

Introducing typedef and leafrefs for strings.

This is to prepare for new data model, introduce
type safety and code-readability.

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

Bug 4500: Clean up Classifier/RSP dependency

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

Merge "SFC - fixed broken SFC UI by DLUX patch - fixed bugs about new version of angular >1.4 - changed SFC compatibility to DLUX - SFC is depend just only on DLUX core - changed SFC UI translations to partial (like DLUX core) - removed yang2xml folder with Yin files - changed service-function-metadata module to   service-function-path-metadata in sfc.services.js"

Bug 4452 - Multiple Context Header Allocation Support

This commit adds a SFC yang model as part of the
design of allows multiple SFC clients besides GBP.

https://lists.opendaylight.org/pipermail/sfc-dev/2015-September/001886.html

- The SFF metadata fatures design follows the recommended approach
  for capability and model evolution:
- https://tools.ietf.org/html/draft-ietf-netmod-rfc6087bis-04#page-31
- The common model (SFF) does not need to be touched
- All new augmentation and features remain on separate files
- Separate files can be reved and changed as needed

- RSP model was patched so client can request metadata allocation.
- Added new provisional code to test new RSP RPC
  parameters

- This code is safe in that it does not require any GBP changes yet

- Rebased to new classifier

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

Merge "Bug 4481 - Standalone SFC classifier support"

Bug 4481 - Standalone SFC classifier support

1. Standalone SFC classifier framework
2. Already test in uni-direction SFC

TODO:

1. plan to add exception handle
2. plan to add bi-directory SFC
3. plan to add VLAN
4. plan to add classifier stats

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

Change protected to public in SfcProviderServiceFunctionMetadataAPI

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

Merge "Bug 4458 - implement SfcOpenflowUtils as SFC util module"

Bug 4458 - implement SfcOpenflowUtils as SFC util module

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

Add sfclisp postman collection for testing

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

Merge "Removed methods containing reflection"

Removed methods containing reflection

- removed all methods with name *Executor
- removed SfcProviderAbstractAPI which is not needed anymore

Change-Id: Ib7f0e2ad3ad5c43f4d06b3c93363eee0b2070ea2
Signed-off-by: Martin Sunal <[email protected]>

Merge "Bug 4474 - Add metadata in rendered path data store"

Bug 4474 - Add metadata in rendered path data store

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

Bug 4445 : Clean up pom.xml in SFC project

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

SFC
- fixed broken SFC UI by DLUX patch
- fixed bugs about new version of angular >1.4
- changed SFC compatibility to DLUX - SFC is depend just only on DLUX core
- changed SFC UI translations to partial (like DLUX core)
- removed yang2xml folder with Yin files
- changed service-function-metadata module to
  service-function-path-metadata in sfc.services.js

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

Merge "Bug 4453 - service function metadata provider support"

Merge "Bug 4443 - Add default classifier table for SFC"

Merge "Bug 4445 : Clean up pom.xml in SFC project"

Bug 4453 - service function metadata provider support

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

Merge "Bug 4444 : Add system interface for SFC data plane locator"

Bug 4443 - Add default classifier table for SFC

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

Bug 4444 : Add system interface for SFC data plane locator

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

Bug 4445 : Clean up pom.xml in SFC project

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

Fix bug 4450 random failure in SfcProviderSfstEntryDataListenerTest

Change-Id: I5660aaa0aa4d4e1fced3d286e10cef2089dd9048
Signed-off-by: Hongli Chen <[email protected]>