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"

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

SFF Form

- implemented yang changes for SFF
- fixed some cosmetic issues

Change-Id: I1fb8b100381b68438acaee62eefbcbab18986f79
Signed-off-by: Stanislav Jamrich <[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]>