sfc.git
7 years agoBump to odlparent 3.1.0 and yangtools 2.0.3 87/69887/3
Stephen Kitt [Thu, 22 Mar 2018 17:24:30 +0000 (18:24 +0100)]
Bump to odlparent 3.1.0 and yangtools 2.0.3

Change-Id: Iab5a1c87fae57ec45524f8be10ceef3aee716ab5
Signed-off-by: Stephen Kitt <[email protected]>
7 years agoRELENG-63: Fix duplicate bundles in features 83/70283/5
Jaime Caamaño Ruiz [Tue, 3 Apr 2018 12:11:50 +0000 (14:11 +0200)]
RELENG-63: Fix duplicate bundles in features

Change-Id: I921a0b78e2a6da5ff9f23a9d2a5357ecbf86b5a0
JIRA: RELENG-63
Signed-off-by: Jaime Caamaño Ruiz <[email protected]>
7 years agoUse ietf-ip.yang from mdsal instead of our own 84/70784/2
Lorand Jakab [Wed, 11 Apr 2018 14:34:04 +0000 (17:34 +0300)]
Use ietf-ip.yang from mdsal instead of our own

Change-Id: I692e6301fc0e3303e13e5dd1893443bd7cab0b05
Signed-off-by: Lorand Jakab <[email protected]>
7 years agoAdd Karaf build profile 97/70597/2
Thanh Ha [Sat, 7 Apr 2018 04:44:12 +0000 (00:44 -0400)]
Add Karaf build profile

Project local Karaf distros are handy for devs to test their work
however is unneeded by autorelease builds and should not be released
as part of the Simultanious Release. Add a profile that is active by
default so that default behaviour is unchanged however allows the
autorelease project to disable building this module.

Issue: RELENG-86
Change-Id: Id968ab6ce79d7067ad71c78634cb8a29b4b2edb7
Signed-off-by: Thanh Ha <[email protected]>
7 years agoImprove vpp utils logging on transaction failure 50/68950/2
Jaime Caamaño Ruiz [Thu, 1 Mar 2018 14:36:00 +0000 (15:36 +0100)]
Improve vpp utils logging on transaction failure

Change-Id: Ib07ffe823d9133811d5117b967ebba5bfa1d666d
Signed-off-by: Jaime Caamaño Ruiz <[email protected]>
7 years agoFix formatting of license statements and remove some redundancies 30/70430/1
David Suarez [Fri, 6 Apr 2018 13:47:55 +0000 (15:47 +0200)]
Fix formatting of license statements and remove some redundancies

Change-Id: Iaeaacb2ad82108b2266a22188be4c862928debc3
Signed-off-by: David Suarez <[email protected]>
7 years agoRemove sfc-ui module (DLUX) 08/70308/2
David Suarez [Wed, 4 Apr 2018 17:32:12 +0000 (19:32 +0200)]
Remove sfc-ui module (DLUX)

SFC's User Interface (module sfc-ui) is based on DLUX project which is
not properly maintained, so it's been decided to removed this dependency
from all Opendaylight projects.

Change-Id: I797ff02078f7ea4afa4b72d5f8a67d5c2fcd5524
Signed-off-by: David Suarez <[email protected]>
7 years agoRemove sal-binding-config dependency 13/70213/2
Tom Pantelis [Fri, 30 Mar 2018 17:12:08 +0000 (13:12 -0400)]
Remove sal-binding-config dependency

It's CSS-related and not needed and will be going away.

Change-Id: Iad88f3e8bc58dd51846ca7f80b527b080a7d154f
Signed-off-by: Tom Pantelis <[email protected]>
7 years agoMigrate listener's deprecated methods to use the new ones 71/69571/3
David Suarez [Fri, 16 Mar 2018 20:07:51 +0000 (21:07 +0100)]
Migrate listener's deprecated methods to use the new ones

Change-Id: I1b0fa95c8e2f33b86c5a394efbb6ededed8eb64c
Signed-off-by: David Suarez <[email protected]>
7 years agoMigrate listener's deprecated methods to use the new ones 72/69572/3
David Suarez [Fri, 16 Mar 2018 20:34:06 +0000 (21:34 +0100)]
Migrate listener's deprecated methods to use the new ones

Change-Id: Ida842033c0f4221a0bcc0ea63db40c2c1d68e7c1
Signed-off-by: David Suarez <[email protected]>
7 years agoUse odl-jackson-2.8 26/70126/2
Stephen Kitt [Tue, 27 Mar 2018 00:24:13 +0000 (17:24 -0700)]
Use odl-jackson-2.8

We should use the odl-jackson-2.8 feature to fulfill our dependencies
on Jackson.

Change-Id: If859f41c3e8f94819376267db65bc578dfe51e82
Signed-off-by: Stephen Kitt <[email protected]>
7 years agoBump java-uuid-generator to 3.1.5 26/70026/3
Robert Varga [Fri, 23 Mar 2018 14:39:37 +0000 (15:39 +0100)]
Bump java-uuid-generator to 3.1.5

The new version removes an explicit dependency on log4j, hence
removing it from packaging.

Change-Id: I3ba1ebc656e0d1c4e78092a8831c1d804cd48bfd
Signed-off-by: Robert Varga <[email protected]>
7 years agoMigrate listener's deprecated methods to use the new ones 73/69573/3
David Suarez [Fri, 16 Mar 2018 20:52:54 +0000 (21:52 +0100)]
Migrate listener's deprecated methods to use the new ones

Change-Id: Ib9e91aba0300231f6bf1af6b89d0622759b944d1
Signed-off-by: David Suarez <[email protected]>
7 years agoSFC-215: Fix sfc-103 demo issues 13/68813/4
Jaime Caamaño Ruiz [Tue, 27 Feb 2018 11:38:58 +0000 (12:38 +0100)]
SFC-215: Fix sfc-103 demo issues

- Use Ubuntu 16 as Ubuntu 15 is no longer available
- Fix ODL distribution URL path
- Use latest ODL snapshot available instead of a fixed version
- Removed unused and un-existent requirement 'messages'

Change-Id: I6e9588dff79d82be3a077acf21478b43238e17a4
JIRA: SFC-215
Signed-off-by: Jaime Caamaño Ruiz <[email protected]>
7 years agoMigrate listener's deprecated methods to use the new ones 20/69520/3
David Suarez [Thu, 15 Mar 2018 08:03:03 +0000 (09:03 +0100)]
Migrate listener's deprecated methods to use the new ones

Change-Id: I124fec07636d2ce86e018e0aada73225d4bfdeeb
Signed-off-by: David Suarez <[email protected]>
7 years agoSFC-217: Fix sfc-py pep8 issues and packaging 74/68874/2
Jaime Caamaño Ruiz [Wed, 28 Feb 2018 12:48:58 +0000 (13:48 +0100)]
SFC-217: Fix sfc-py pep8 issues and packaging

sfc-python-verify has been failing for some time. After fixing it [1],
some issues have arised in sfc-py:

- Fix pep8 issues.
- Refactor jenkins_build_package.sh so that it does not build python3
  itself as it is already available in the enviroment on which
  sfc-python-verify runs. Anyway, it has also being removed form the
  job for the time being as per [1].

[1] https://git.opendaylight.org/gerrit/#/c/68873

Change-Id: I2f0ae12bb6fef56bce50137515a23e024d08b4ee
Signed-off-by: Jaime Caamaño Ruiz <[email protected]>
7 years agoRemove deprecated abstract listeners 84/69384/2
David Suarez [Mon, 12 Mar 2018 15:37:33 +0000 (16:37 +0100)]
Remove deprecated abstract listeners

These abstract listeners were superseded by the Genius' listeners. They
were deprecated in Nitrogen and now we delete them.

Change-Id: Iac27ae94535b03a77b816678a17f03fe3b50ed59
Signed-off-by: David Suarez <[email protected]>
7 years agoClean up fragile Unit Test 63/69463/2
ebrjohn [Tue, 13 Mar 2018 20:55:25 +0000 (15:55 -0500)]
Clean up fragile Unit Test

Change-Id: If23f0d1acda11a57d52686608dce43785071dea1
Signed-off-by: Brady Johnson <[email protected]>
7 years agoFix some unit tests for listeners 33/69333/3
David Suarez [Sat, 10 Mar 2018 10:41:25 +0000 (11:41 +0100)]
Fix some unit tests for listeners

Change-Id: Idbee6a8ae6047af5fb377b4f1f13535e95c8cccb
Signed-off-by: David Suarez <[email protected]>
7 years agoRefactor to change packages' names 38/69238/3
David Suarez [Thu, 8 Mar 2018 09:11:20 +0000 (10:11 +0100)]
Refactor to change packages' names

org.opendaylight.sfc.ofrenderer package is replaced by a more canonical
name: org.opendaylight.sfc.renderers.openflow

Change-Id: I170fefdca841d7deeb86d42d587a4478c4073467
Signed-off-by: David Suarez <[email protected]>
7 years agoRemove unused reference to AsyncDataChangeEvent 38/69038/2
Tom Pantelis [Sat, 3 Mar 2018 17:26:02 +0000 (12:26 -0500)]
Remove unused reference to AsyncDataChangeEvent

AsyncDataChangeEvent is deprecated and scheduled for
removal.

Change-Id: Ie140ad16235479e3f79401a91988731742454cc6
Signed-off-by: Tom Pantelis <[email protected]>
7 years agoSFC-213 SFC statistics dont always work 42/68842/7
ebrjohn [Tue, 27 Feb 2018 15:18:41 +0000 (09:18 -0600)]
SFC-213 SFC statistics dont always work

- The OpenFlow NodeId is not available on
  the SFF, so the statistics cant work.
- There's a race condition between when SFC
  creates the OVS bridge for an SFF and when
  we try to get the bridge DPID.

Change-Id: I05b2d0da61f8147ffea0a995b3efef38cbf05b31
Signed-off-by: Brady Johnson <[email protected]>
7 years agoRefactor to change packages' names 88/69088/2
David Suarez [Mon, 5 Mar 2018 21:31:13 +0000 (22:31 +0100)]
Refactor to change packages' names

sfc-ios-xe-renderer package is replaced by more canonical name:
org.opendaylight.sfc.renderers.iosxe

Change-Id: I86e9f2e355e86823300c4de7003d4133f1dfe95e
Signed-off-by: David Suarez <[email protected]>
7 years agoRefactor sfc-vpp-renderer 55/68955/3
David Suarez [Thu, 1 Mar 2018 15:14:32 +0000 (16:14 +0100)]
Refactor sfc-vpp-renderer

- Migrate to use Blueprint
- Use Blueprint annotations
- Migrate listeners to use Genius' ones

Change-Id: Iaec54c9d59bb078e1e455fd10aff96155af9c042
Signed-off-by: David Suarez <[email protected]>
7 years agoMinor change in .gitignore 54/68954/2
David Suarez [Thu, 1 Mar 2018 15:08:59 +0000 (16:08 +0100)]
Minor change in .gitignore

Change-Id: Icb8095295dbb30eba28a624c608aaeb1b4a4fdda
Signed-off-by: David Suarez <[email protected]>
7 years agoFix sfc-104 demo issues 49/68949/2
Jaime Caamaño Ruiz [Thu, 1 Mar 2018 14:26:57 +0000 (15:26 +0100)]
Fix sfc-104 demo issues

Change-Id: Idfd3cc4b3fdca31188c4a4c2132ee06ed3ebe3a1
JIRA: SFC-218
Signed-off-by: Jaime Caamaño Ruiz <[email protected]>
7 years agoSFC-218: Fix sfc-scf-vpp wiring 48/68948/2
Jaime Caamaño Ruiz [Thu, 1 Mar 2018 14:25:01 +0000 (15:25 +0100)]
SFC-218: Fix sfc-scf-vpp wiring

Change-Id: I73ad125c433f145bba93ef8b0c56f0cfb344a30d
JIRA: SFC-218
Signed-off-by: Jaime Caamaño Ruiz <[email protected]>
7 years agoRemove binary jar and download when running test 27/68827/2
ebrjohn [Tue, 27 Feb 2018 16:37:11 +0000 (10:37 -0600)]
Remove binary jar and download when running test

Change-Id: I15bbbb8a56d2e208b472d096ed85c9e166132258
Signed-off-by: Brady Johnson <[email protected]>
7 years agoRefactor to change packages' names 87/68387/2
David Suarez [Sun, 18 Feb 2018 17:34:52 +0000 (18:34 +0100)]
Refactor to change packages' names

sfc_vpp_renderer package is replaced by more canonical name:
org.opendaylight.sfc.renderers.vpp

Change-Id: I08a361eb9aab2f692a2a125b4447a1fc3dd5c7ad
Signed-off-by: David Suarez <[email protected]>
7 years agoSFC-214: Fix sb-rest wiring 12/68812/2
Jaime Caamaño Ruiz [Tue, 27 Feb 2018 10:50:39 +0000 (11:50 +0100)]
SFC-214: Fix sb-rest wiring

Missing build plugin and incorrect bean naming was preventing the
blueprint wiring to actually happen. Fixed the SFF listener which
was missing the wiring annotations. Got rid also of the unnecessary
provider module.

Change-Id: Id3b6ef73cd274a108b07a50fedbe94eab6123cf3
JIRA: SFC-214
Signed-off-by: Jaime Caamaño Ruiz <[email protected]>
7 years agoSFC-216: Fix exception message check for bad macs 18/68818/1
Jaime Caamaño Ruiz [Tue, 27 Feb 2018 13:15:05 +0000 (14:15 +0100)]
SFC-216: Fix exception message check for bad macs

The IllegalArgumentException message thrown when constructing a bad
MacAddress (or any IllegalArgumentException thrown for any yang type
value that fails a regex pattern check) changed with [1].

[1] https://git.opendaylight.org/gerrit/#/c/68461/

Change-Id: I5b6a73773cb6b4742d004b6bc6ac7ae6e38e4d66
JIRA: SFC-216
Signed-off-by: Jaime Caamaño Ruiz <[email protected]>
7 years agoBump versions by x.(y+1).z for next dev cycle 50/68750/1
Anil Belur [Mon, 26 Feb 2018 07:41:46 +0000 (17:41 +1000)]
Bump versions by x.(y+1).z for next dev cycle

Change-Id: Ie6a846caecb2cdec46cced1398dcf0c8ab11e42f
Signed-off-by: Anil Belur <[email protected]>
7 years agoSFC-210: Remove config RSP on delete RSP rpc (II) 70/68370/5
Jaime Caamaño Ruiz [Fri, 16 Feb 2018 14:40:45 +0000 (15:40 +0100)]
SFC-210: Remove config RSP on delete RSP rpc (II)

First patch did not really deal with the fact the previous behavior of
the RPC was to return once the operational RSP was already deleted.
This patch aligns the solution of the delete RPC to the one adopted
for the create RPC which respects previous behavior. And solves the
corresponding test case that fails in the basic SFC csit suite.

Change-Id: I7d21665c32f3358b083f43932da3285c11d7c1fe
JIRA: SFC-210
Signed-off-by: Jaime Caamaño Ruiz <[email protected]>
7 years agoAvoid exception on default scheduler handling 67/68267/4
Jaime Caamaño Ruiz [Tue, 13 Feb 2018 12:05:58 +0000 (13:05 +0100)]
Avoid exception on default scheduler handling

Default scheduler selection is handled through an exception that is
printed out in the logs and looks rather ugly. Replaced by a proper
cleaner log message.

Change-Id: I1fd8e6ffe91eaa45ef21a15942c58d12983a5f9d
Signed-off-by: Jaime Caamaño Ruiz <[email protected]>
7 years agoFixes for mac chaining demo 66/68266/5
Jaime Caamaño Ruiz [Wed, 14 Feb 2018 19:07:48 +0000 (20:07 +0100)]
Fixes for mac chaining demo

- Fix the traffic  example given in the instructions
- Fix the demo not working with a remote ODL
- Set python output to unbuffered for dummy SFs and GW
  so that output is actually written while using the demo
- Fix ovs node id value although commented out for now as it has side
  effects: when set, some values are overwritten on OVS topology node,
  which changes the node's datapath id.
- Provided a vagrant file with a mininet environment

Change-Id: Ic001ff9ecd7a91ee6fc56b90fff1fe1d56565edb
Signed-off-by: Jaime Caamaño Ruiz <[email protected]>
7 years agoFix openflow node mess in mac chaining classifier 65/68265/4
Jaime Caamaño Ruiz [Wed, 14 Feb 2018 10:43:11 +0000 (11:43 +0100)]
Fix openflow node mess in mac chaining classifier

Mac chaining SCF classifier was obtaining the openflow node
to write flows to through different ways on different places:
sometimes it would use the service node while others would
use the methods of SfcOvsUtils. Also it would sometimes try to
obtain the SFF from the openflow node when it already has it
readily available through a class property. So cleaned up this
which allowed also to clean up the ClassifierInterface a bit.

Change-Id: I8fe070e2a3a70a17b4520012de9358617b51d63f
Signed-off-by: Jaime Caamaño Ruiz <[email protected]>
7 years agoFix NPEs and improved debugging in SCF classifier 64/68264/3
Jaime Caamaño Ruiz [Tue, 13 Feb 2018 12:45:34 +0000 (13:45 +0100)]
Fix NPEs and improved debugging in SCF classifier

Fixed some methods that would return null if some mandatory input data
was also null. Users of these methods were not prepared for this even
thought there were unit tests asserting this behavior.

As there is no good reason for that data to possibly be null, make them
fail with a controlled NPE and remove unit tests.

Change-Id: I16baa44d1ad6a844aef2a33cc7f3121be7f12c98
Signed-off-by: Jaime Caamaño Ruiz <[email protected]>
7 years agoSFC-212: Fix scf rendering with no RSP metadata 62/68262/4
Jaime Caamaño Ruiz [Tue, 13 Feb 2018 12:05:00 +0000 (13:05 +0100)]
SFC-212: Fix scf rendering with no RSP metadata

Fixes a problem where the rendering of the classifier flows would fail
if the RSP did no include the optional NSH metadata.

Change-Id: Ib8c4bef199b704db202a3c71faee1a93dea77068
Signed-off-by: Jaime Caamaño Ruiz <[email protected]>
7 years agoSFC-211: Fix service function path state not deleted 61/68261/3
Jaime Caamaño Ruiz [Wed, 14 Feb 2018 15:24:04 +0000 (16:24 +0100)]
SFC-211: Fix service function path state not deleted

Change-Id: Icd413d85002cda4e51f4a003bb71260f266dc984
JIRA: SFC-211
Signed-off-by: Jaime Caamaño Ruiz <[email protected]>
7 years agoSFC-210: Remove config RSP on delete RSP rpc 60/68260/2
Jaime Caamaño Ruiz [Wed, 14 Feb 2018 10:41:35 +0000 (11:41 +0100)]
SFC-210: Remove config RSP on delete RSP rpc

Change-Id: I6cf06398e6046dabcf3339ddeb1f68a910a8c882
Signed-off-by: Jaime Caamaño Ruiz <[email protected]>
7 years agoFix lisp NPEs on Logical SFF and more 10/68110/2
Jaime Caamaño Ruiz [Fri, 9 Feb 2018 14:53:25 +0000 (15:53 +0100)]
Fix lisp NPEs on Logical SFF and more

Change-Id: I95fabe89de7ca0b02aa64fe2191ca3c945e564ad
Signed-off-by: Jaime Caamaño Ruiz <[email protected]>
7 years agoUT for SfcStatistics 83/67883/7
ebrjohn [Fri, 2 Feb 2018 23:12:57 +0000 (17:12 -0600)]
UT for SfcStatistics

Change-Id: Ib84e5ecfff1be0cc51cb78b9b5189cd23894d9f9
Signed-off-by: Brady Johnson <[email protected]>
7 years agoSFC-208: Add hardening for create RSP RPC 58/67858/1
Jaime Caamaño Ruiz [Fri, 2 Feb 2018 10:52:14 +0000 (11:52 +0100)]
SFC-208: Add hardening for create RSP RPC

Change-Id: I05125e0e3486572b98c756a9fed6d525ba57c529
JIRA: SFC-208
Signed-off-by: Jaime Caamaño Ruiz <[email protected]>
7 years agoConfig data store listener for RSP creation 07/65007/15
ebrjohn [Wed, 1 Nov 2017 18:21:06 +0000 (13:21 -0500)]
Config data store listener for RSP creation

- Previous RSP creation via RPC is deprecated, but still supported.
- Now its possible to create RSPs with a write to the SFP config.
- SFP config creation will trigger an RSP config creation, which
  triggers an RSP oper creation.
- Deleting or modifying SFs/SFFs now deletes the SFP, not the RSP
- Still pending: Unit tests

Change-Id: I9cf947abdac6433d0e7d961c372f43f0f6d572c8
Signed-off-by: Brady Johnson <[email protected]>
Signed-off-by: Jaime Caamaño Ruiz <[email protected]>
7 years agoRSP statistics feature 50/66750/8
ebrjohn [Sat, 23 Dec 2017 00:30:09 +0000 (18:30 -0600)]
RSP statistics feature

- RSP statistics are retrieved on-demand by calling an RPC
- To be completed next: Unit Tests

Change-Id: If90657ad339a77bc50e6809ccbb1425c5eb31872
Signed-off-by: Brady Johnson <[email protected]>
7 years agoRefine affected RSP search on SFF update 32/67432/3
Jaime Caamaño Ruiz [Mon, 22 Jan 2018 16:44:15 +0000 (17:44 +0100)]
Refine affected RSP search on SFF update

Previously, any SFF locator update would cause the removal of any RSP
associated with that SFF. With this patch, in regards to a SFF locator
update, an associated RSP will be removed only if any of these
conditions are met:

* The SFF locator is used in a SF dictionary entry for an SF associated
with the RSP.

* The RSP is associated with more SFFs than the updated SFF so that the
updated locator could be used in a SFF-SFF hop.

This improvement is particularly meaningful for the logical SFF scenario
in combination with the changes introduced for the directional-dpl spec.
The logical SFF singleton is no longer an empty SFF and will be updated
frequently as paths are created or removed. It is fundamental that other
paths are not affected when they shouldn't.

The current data model does not allow for optimum processing and there
might be performance impacts on big SFFs, as the logical SFF most likely
will be. For a future release, there is the option to enhance the data
model to optimize this.

Change-Id: I16660e241779ff24bf644e4377e12965b12f695b
Signed-off-by: Jaime Caamaño Ruiz <[email protected]>
7 years agoUse directional dpls for openflow rendering 56/66956/11
Jaime Caamaño Ruiz [Mon, 8 Jan 2018 22:34:03 +0000 (23:34 +0100)]
Use directional dpls for openflow rendering

Change-Id: I36ceff0509d0d33d9e34e6cec4f30df100c3cd06
Signed-off-by: Jaime Caamaño Ruiz <[email protected]>
7 years agoRemove legacy sfc-test-consumer feature 27/67127/4
ebrjohn [Fri, 12 Jan 2018 17:58:04 +0000 (11:58 -0600)]
Remove legacy sfc-test-consumer feature

- It hasnt been used for a long time

Change-Id: I1141abe111f9ea74e376aa2940ef575f292c8405
Signed-off-by: Brady Johnson <[email protected]>
7 years agoFix in the firewall mode of vxlan_tool 85/67585/1
Manuel Buil [Thu, 25 Jan 2018 17:40:57 +0000 (18:40 +0100)]
Fix in the firewall mode of vxlan_tool

The firewall mode of vxlan_tool was not compatible with the ETH + NSH
encapsulation. This patch fixes that through offset which is different in the
two cases we support (the other one is ETH + IP + UDP + VXLAN + ETH + NSH)

Change-Id: Ib8d84bd8d4c06e91b146a68d1a749ecb3eb6070f
Signed-off-by: Manuel Buil <[email protected]>
7 years agoEliminate unnecessary explicit yangtools dependencies 25/67325/1
Tom Pantelis [Fri, 19 Jan 2018 02:44:46 +0000 (21:44 -0500)]
Eliminate unnecessary explicit yangtools dependencies

Changed the sfc-parent pom to derive from bindng-parent so
yangtools version and dependencies comes from mdsal.

Change-Id: Idb649ef2367af82afd0fc3ed6ed4577aa7740f8b
Signed-off-by: Tom Pantelis <[email protected]>
7 years agoBump yangtools to 2.0.1 18/67318/2
Luis Gomez [Thu, 18 Jan 2018 23:50:44 +0000 (15:50 -0800)]
Bump yangtools to 2.0.1

Change-Id: Iaf9d1af326a2fc11f48f211040efead95e178e8d
Signed-off-by: Luis Gomez <[email protected]>
7 years agoEmergency path with odlparent/yangtools fixes 19/67219/2
ebrjohn [Tue, 16 Jan 2018 20:42:59 +0000 (14:42 -0600)]
Emergency path with odlparent/yangtools fixes

- Removed unecessary mandatory attributes. The mandatory
  attribute is now being enforced with the new yangtools.
- Fixed some UT that werent setting mandatory fields.
- Temporarily commented out broken IosXeRspProcessorTest

Change-Id: I22e96ce469fd12531b672faad61f158685e0bd7c
Signed-off-by: Brady Johnson <[email protected]>
7 years agoBump to yangtools-2.0.0 and odlparent-3.0.2 13/66513/5
Tom Pantelis [Fri, 15 Dec 2017 15:28:38 +0000 (10:28 -0500)]
Bump to yangtools-2.0.0 and odlparent-3.0.2

Change-Id: Ib2001a498f36ae3242a5f74b4fd729da91a0a359
Signed-off-by: Tom Pantelis <[email protected]>
Signed-off-by: Stephen Kitt <[email protected]>
7 years agoMove the SFC OpenFlow flow names 15/66215/4
ebrjohn [Mon, 4 Dec 2017 23:36:43 +0000 (17:36 -0600)]
Move the SFC OpenFlow flow names

- Moving the constants to a common util class so they can
  be easily accessed by other SFC entities, like the forth
  coming RSP statistics feature.
- The SFC OpenFlow renderer will use the statistics feature,
  but the statistics feature needs the SFC flow names, so
  this change will help avoid a cyclic feature dependency.

Change-Id: I69b35eb905aafca92f33149fbe24a4fad472040c
Signed-off-by: Brady Johnson <[email protected]>
7 years agoUse centOS vault package repository for sfc-docker 16/66816/2
Jaime Caamaño Ruiz [Fri, 29 Dec 2017 14:07:13 +0000 (15:07 +0100)]
Use centOS vault package repository for sfc-docker

Change-Id: Id07475d5cbc7978f97cd42abd45b65c20d7103d1
Signed-off-by: Jaime Caamaño Ruiz <[email protected]>
7 years agoMove local karaf to common directory 21/66321/3
Ryan Goulding [Fri, 8 Dec 2017 15:04:02 +0000 (10:04 -0500)]
Move local karaf to common directory

Most projects have elected to put the local distribution
in the "karaf" TLD.  For consistency sake, this moves
sfc-karaf to the "karaf" directory.

Change-Id: I7929c770459f77f9cf998c27298f4655c408b641
Signed-off-by: Ryan Goulding <[email protected]>
7 years agoBind to multiple logical interfaces of SF 75/66875/1
Jaime Caamaño Ruiz [Thu, 4 Jan 2018 14:03:08 +0000 (15:03 +0100)]
Bind to multiple logical interfaces of SF

Because of directional DPL, SF can have up to a pair of logical
interface locators. This patch adds support to bind/unbind to both
instead of just the one when the SF is included in a rendered path.

Even if the path is not symmetric and only one DPL is really used, there
is no problem binding to all the interfaces of the SF as long as its DPN
is initialized as a SFF, which happens when the SF intervenes in a RSP.

Current limitation is that all the SF interfaces must located on the
same DPN. To overcome this limitation, the logical SFF operational
information needs to be enhanced.

Change-Id: I911eeffa113e040eef9a1add20edcfb8534221ee
Signed-off-by: Jaime Caamaño Ruiz <[email protected]>
7 years agoSet reverse path flag on RSP 74/66874/1
Jaime Caamaño Ruiz [Thu, 4 Jan 2018 09:40:07 +0000 (10:40 +0100)]
Set reverse path flag on RSP

Change-Id: I1577f2c08c7fda55412239d0a73929d59a15e56b
Signed-off-by: Jaime Caamaño Ruiz <[email protected]>
7 years agoFix odlparent 3 Checkstyle issues 12/66512/2
Tom Pantelis [Fri, 15 Dec 2017 16:46:52 +0000 (11:46 -0500)]
Fix odlparent 3 Checkstyle issues

Change-Id: I525c8a677032f9b1c874a9170223b2c63777887e
Signed-off-by: Tom Pantelis <[email protected]>
7 years agoAPI changes for the rsp-config feature 67/66567/3
ebrjohn [Mon, 18 Dec 2017 16:27:28 +0000 (10:27 -0600)]
API changes for the rsp-config feature

- Modifying the "config: true/false" attribute on
  several of the RSP data model nodes.

Change-Id: If04109d12fe2226e3959a875237dfc41e7d6fd5b
Signed-off-by: Brady Johnson <[email protected]>
7 years agoSFC Statistics RPC operations definitions 26/66526/2
ebrjohn [Fri, 15 Dec 2017 21:25:43 +0000 (15:25 -0600)]
SFC Statistics RPC operations definitions

- API changes for the rsp_stats feature

Change-Id: Ie69642d46fc7ae2977f51cba987301d3997de64b
Signed-off-by: Brady Johnson <[email protected]>
7 years agoAdd yang model for directional data plane locators 69/66569/4
Jaime Caamaño Ruiz [Mon, 18 Dec 2017 16:55:39 +0000 (17:55 +0100)]
Add yang model for directional data plane locators

Plus updated spec to align it to the latest model proposal.

Change-Id: I7984d8e55decdeb91cd5611ad396b8a14f3492de
Signed-off-by: Jaime Caamaño Ruiz <[email protected]>
7 years agoSpec for the new RSP statistics feature 13/66213/7
ebrjohn [Mon, 4 Dec 2017 22:37:31 +0000 (16:37 -0600)]
Spec for the new RSP statistics feature

Change-Id: I88299b00386b8805f9ca69f3023b373661483338
Signed-off-by: Brady Johnson <[email protected]>
7 years agoSpec for directional data plane locators 56/66356/5
Jaime Caamaño Ruiz [Mon, 11 Dec 2017 19:32:01 +0000 (20:32 +0100)]
Spec for directional data plane locators

Change-Id: Ie7cfa1ddea8a4c3be318abb57489000a8770ec74
Signed-off-by: Jaime Caamaño Ruiz <[email protected]>
7 years agoRefactor sfc-scf-vpp module 91/65691/2
David Suarez [Sat, 18 Nov 2017 12:18:49 +0000 (13:18 +0100)]
Refactor sfc-scf-vpp module

- Proper use of Blueprint to wire all the components: convert some
  classes into @Singleton, remove the SfcScfVppRenderer, use annotations
  instead of XML, remove some useless parameters.
- Clean up
- Migrate SfcScfVppDataListener to use Genius listeners.

Change-Id: Ie7c608fd76b3cef44989d2b0c6c926f2d47c6f25
Signed-off-by: David Suarez <[email protected]>
7 years agoSFC-205: Fix wrong table when removing scf flows 56/65356/3
Jaime Caamaño Ruiz [Thu, 9 Nov 2017 09:46:10 +0000 (10:46 +0100)]
SFC-205: Fix wrong table when removing scf flows

Fixes SFC SCF classifier removing flows from incorrect table. Refactored
ClassifierInterface to provide the correct table numbers and for
MacChainingClassifier to also implement it.

Change-Id: I4383209ce24d0d3dce14f8f498c91abaf36b6c09
Signed-off-by: Jaime Caamaño Ruiz <[email protected]>
7 years agoSFC-206: Fix no sfc-scf-openflow listeners @runtime 48/65748/1
Jaime Caamaño Ruiz [Mon, 20 Nov 2017 17:38:49 +0000 (18:38 +0100)]
SFC-206: Fix no sfc-scf-openflow listeners @runtime

sfc-scf-openflow listeners were not being instantiated by blueprint
because their constructor parameters were not singletons that could be
injected. Introduced in b5fabe5.

Fixed by creating the listeners manually.

Bug: https://jira.opendaylight.org/browse/SFC-206
Change-Id: If3a01d691cbce8478537fd50cdead461d6740e9c
Signed-off-by: Jaime Caamaño Ruiz <[email protected]>
7 years agoMigrate sfc-ovs listeners to Genius' Listeners framework 40/65640/2
David Suarez [Thu, 16 Nov 2017 20:23:53 +0000 (21:23 +0100)]
Migrate sfc-ovs listeners to Genius' Listeners framework

Change-Id: I02019c49787c45cd45d3d27d089e5bc0a140416d
Signed-off-by: David Suarez <[email protected]>
7 years agoMigrate sfc-pot-netconf-renderer to use Genius' Listeners framework 65/65665/2
David Suarez [Fri, 17 Nov 2017 13:06:24 +0000 (14:06 +0100)]
Migrate sfc-pot-netconf-renderer to use Genius' Listeners framework

- Migrate the listeners to use the new framework available in Genius
- Replace Blueprint XML entries by the corresponding annotations
- Make some utilities and helpers proper singletons instantiated via BP
- Other minor changes and clean up

I'll rise a follow up patch to improve the design.

Change-Id: Ia343627fdb9cc4c93384a2499623faf040857316
Signed-off-by: David Suarez <[email protected]>
7 years agoAnnotate some parameters as @Nonnull 69/65669/3
David Suarez [Fri, 17 Nov 2017 13:50:23 +0000 (14:50 +0100)]
Annotate some parameters as @Nonnull

This allows to remove some checks.

Change-Id: I37125b35da5d37f9556d05223ad0cfe83070e0a1
Signed-off-by: David Suarez <[email protected]>
7 years agoAnnotate some parameters as @Nonnull 73/65673/1
David Suarez [Fri, 17 Nov 2017 14:42:04 +0000 (15:42 +0100)]
Annotate some parameters as @Nonnull

Change-Id: I1615fd920527f675243fe433289b7e66947eb8a8
Signed-off-by: David Suarez <[email protected]>
7 years agoFix broken formatting in rsp-config-datastore docs 58/65558/1
Ryan Goulding [Wed, 15 Nov 2017 17:06:10 +0000 (12:06 -0500)]
Fix broken formatting in rsp-config-datastore docs

"code-block" takes exactly one argument, the name of the language
contained by the code-block.  This fixes the formatting issue.

Change-Id: Ib7f4fc6727bd2fb4378c5c87eb46e4fda6f4015d
Signed-off-by: Ryan Goulding <[email protected]>
7 years agoUpdates to create RSPs via config data store 06/65006/4
ebrjohn [Wed, 1 Nov 2017 18:21:06 +0000 (13:21 -0500)]
Updates to create RSPs via config data store

- deprecate RSP creation via RPC
- Update RSP data model changing "config false" fields.

Change-Id: Ifbc2e81b0fb56325f8a8c0e874d126e161a45ecb
Signed-off-by: Brady Johnson <[email protected]>
7 years agoRemove trailing spaces and tabs in resource files 19/63419/10
David Suarez [Fri, 10 Nov 2017 15:07:18 +0000 (16:07 +0100)]
Remove trailing spaces and tabs in resource files

Remove trailing spaces to pass the checkstyle rules included in [1].

[1] https://git.opendaylight.org/gerrit/#/c/48710/

Change-Id: I5b67bc919be751bfab04f74b43ffdba88fdde010
Signed-off-by: David Suarez <[email protected]>
7 years agoSpec for RSP creation in config data store 98/64998/4
ebrjohn [Wed, 1 Nov 2017 16:53:03 +0000 (11:53 -0500)]
Spec for RSP creation in config data store

Change-Id: Ib6e25bd9424ae65a8a1b8bbc102978138d2f3c63
Signed-off-by: Brady Johnson <[email protected]>
7 years agoFix some checkstyle issues for the upcoming odlparent 3.0 43/65343/2
David Suarez [Thu, 9 Nov 2017 00:45:09 +0000 (01:45 +0100)]
Fix some checkstyle issues for the upcoming odlparent 3.0

Change-Id: If2e6542e28f50d1c80bc696250cd62f28c3f79f9
Signed-off-by: David Suarez <[email protected]>
7 years agoRemove deploy-site.xml 05/64905/2
Thanh Ha [Mon, 30 Oct 2017 21:16:59 +0000 (17:16 -0400)]
Remove deploy-site.xml

With the javadoc jobs now available to generate javadoc it is time to
remove the maven-sites and related files.

Change-Id: Icc219da3bc0a3ac639692d041f651f85a086e5f5
Signed-off-by: Thanh Ha <[email protected]>
7 years agoRefactor listeners in sfc-sb-rest module to remove off-loading twice 03/65003/6
David Suarez [Wed, 1 Nov 2017 21:55:29 +0000 (22:55 +0100)]
Refactor listeners in sfc-sb-rest module to remove off-loading twice

Change-Id: Ibf10e63fbed0366981dfd2f9c01d46f1f8626674
Signed-off-by: David Suarez <[email protected]>
7 years agoRefactor listeners in sfc-ios-xe-renderer module 90/65090/2
David Suarez [Fri, 3 Nov 2017 12:09:30 +0000 (13:09 +0100)]
Refactor listeners in sfc-ios-xe-renderer module

All listeners in this module have been migrated to use the new listener
framework from Genius. Taking advantage of this migration process they
now use BP annotations to be wired. Renderers have been migrated to use
BP as well, so the SfcIosXeRenderer component (in charge of wiring) is
not needed anymore.

Change-Id: Ib3f059900e9018d1fe91a2e447c78e20d6503970
Signed-off-by: David Suarez <[email protected]>
7 years agoMigrate sfc-genius listeners 93/64393/4
Jaime Caamaño Ruiz [Fri, 13 Oct 2017 11:49:18 +0000 (13:49 +0200)]
Migrate sfc-genius listeners

Change-Id: I2b29f8e0925be3235f746a879e8e587dbaae7a39
Signed-off-by: Jaime Caamaño Ruiz <[email protected]>
7 years agoFix the broken build 41/65041/1
David Suarez [Thu, 2 Nov 2017 11:11:17 +0000 (12:11 +0100)]
Fix the broken build

Change-Id: I81461de5e14b17f0c29d0762d62159e667a48b04
Signed-off-by: David Suarez <[email protected]>
7 years agoRefactor listeners in sfc-sb-rest module 16/64916/1
David Suarez [Mon, 30 Oct 2017 23:59:44 +0000 (00:59 +0100)]
Refactor listeners in sfc-sb-rest module

- Use Genius Listeners Framework
- Use blueprint annotations so they can be directly handled by BP
- Some minor fixes
- Checkstyle

Change-Id: I3608e5c74a1b5a9af3d9e707d93c30b2bc4bd61b
Signed-off-by: David Suarez <[email protected]>
7 years agoRefactor listeners in sfc-scf-openflow module 77/64777/2
David Suarez [Thu, 26 Oct 2017 22:19:53 +0000 (00:19 +0200)]
Refactor listeners in sfc-scf-openflow module

- Use Genius Listeners Framework
- Use blueprint annotations so they can be directly handled by BP
  instead of the SfcScfProcessor singleton class
- Some minor fixes
- Checkstyle

Change-Id: I0c8029e4c8db2f65c76006e838108deb57d33141
Signed-off-by: David Suarez <[email protected]>
7 years agoDeprecate the abstract listeners included in sfc-provider 54/64754/2
David Suarez [Thu, 26 Oct 2017 06:07:25 +0000 (08:07 +0200)]
Deprecate the abstract listeners included in sfc-provider

The AbstractClusteredDataTreeChangeListener and
AbstractDataTreeChangeListener are replaced by the classes present in
Genius' package org.opendaylight.genius.datastoreutils.listeners.

Change-Id: I2fcb0187e2b3f43b7ea9d872c50de1543e89f384
Signed-off-by: David Suarez <[email protected]>
7 years agoMigrate the rest of the sfc-provider's listeners 41/64741/2
David Suarez [Wed, 25 Oct 2017 20:15:00 +0000 (22:15 +0200)]
Migrate the rest of the sfc-provider's listeners

Migrate the rest of the sfc-provider listener to use Genius' listeners

Change-Id: Idafd8020f1f7620ae22e7d894ac0c0a38e7285fb
Signed-off-by: David Suarez <[email protected]>
7 years agoMigrate ServiceFunctionChainListener to Genius' Listeners 30/64730/3
David Suarez [Wed, 25 Oct 2017 18:36:40 +0000 (20:36 +0200)]
Migrate ServiceFunctionChainListener to Genius' Listeners

Change-Id: I26eaa162202c497fd286d1c028216c519997e69b
Signed-off-by: David Suarez <[email protected]>
7 years agoMigrate ServiceFunctionForwarderListener to Genius' Listeners 28/64728/1
David Suarez [Wed, 25 Oct 2017 17:36:25 +0000 (19:36 +0200)]
Migrate ServiceFunctionForwarderListener to Genius' Listeners

Change-Id: Ie09ee0efe4fde576b366a95a85ff41cd55f87488
Signed-off-by: David Suarez <[email protected]>
7 years agoMigrate ServiceFunctionListener listener to use the new Listeners FW 29/64629/4
David Suarez [Mon, 23 Oct 2017 23:33:02 +0000 (01:33 +0200)]
Migrate ServiceFunctionListener listener to use the new Listeners FW

Migrate the ServiceFunctionListener listener to use the new Listeners
framework available since [1].

[1] https://git.opendaylight.org/gerrit/#/c/59177/

Change-Id: Ie7306de4cf4af48d88a645dd872a3143337be5c7
Signed-off-by: David Suarez <[email protected]>
7 years agoMinor cleanup 28/64628/2
David Suarez [Mon, 23 Oct 2017 23:09:30 +0000 (01:09 +0200)]
Minor cleanup

More changes will follow.

Change-Id: Ib4a0a2adb59ea206fc7d27cf68201732653db508
Signed-off-by: David Suarez <[email protected]>
7 years agoMinor cleanup 08/64608/2
David Suarez [Mon, 23 Oct 2017 13:51:02 +0000 (15:51 +0200)]
Minor cleanup

Change-Id: I5023ca7b29220611c4b5545ce800ea958cee111f
Signed-off-by: David Suarez <[email protected]>
7 years agoRefactor sfc-pot to non-static 90/64590/3
David Suarez [Sun, 22 Oct 2017 19:01:05 +0000 (21:01 +0200)]
Refactor sfc-pot to non-static

The pom.xml files both in sfc-pot and sfc-provider need to be changed
so they generate autowire.xml files properly.

Change-Id: I4363ebecaab832844561d74689719deeb31c1e60
Signed-off-by: David Suarez <[email protected]>
7 years agoMove this service to use blueprint annotations 89/64589/2
David Suarez [Sun, 22 Oct 2017 08:29:45 +0000 (10:29 +0200)]
Move this service to use blueprint annotations

Move this service to use blueprint annotations instead of xml file.
Some simplifications.

Change-Id: Ic6937e5dab03c26097a4b6c3d1967187b64dc7b4
Signed-off-by: David Suarez <[email protected]>
7 years agoUnbind SFC service when removing SFs 92/64392/2
Jaime Caamaño Ruiz [Fri, 13 Oct 2017 11:49:18 +0000 (13:49 +0200)]
Unbind SFC service when removing SFs

This prevents leftover SFC bindings when the SFs are removed quickly
after the RSPs.

Change-Id: Idf81d862111b054ecd4681de295973f2b01c2e9e
Signed-off-by: Jaime Caamaño Ruiz <[email protected]>
7 years agoFix some NPEs related to logical SFF 63/64163/5
Jaime Caamaño Ruiz [Wed, 11 Oct 2017 13:41:09 +0000 (15:41 +0200)]
Fix some NPEs related to logical SFF

Change-Id: I3d92cf53c61e86a693a0d68f08e9c23f05499186
Signed-off-by: Jaime Caamaño Ruiz <[email protected]>
7 years agoFix checkstyle issues 56/64256/5
David Suarez [Fri, 13 Oct 2017 17:37:17 +0000 (19:37 +0200)]
Fix checkstyle issues

These utility classes should be declared as final.

Change-Id: Iedacd1ecfcecf0a42a7d4d21879cf7ae98609dbd
Signed-off-by: David Suarez <[email protected]>
7 years agoFix checkstyle issues 60/64260/2
David Suarez [Fri, 13 Oct 2017 17:52:42 +0000 (19:52 +0200)]
Fix checkstyle issues

These utility classes should be declared as final.

Change-Id: I6bcaeb367e0408289e5fca4e0640779fea4a0b98
Signed-off-by: David Suarez <[email protected]>
7 years agoFix checkstyle issues 62/64262/2
David Suarez [Fri, 13 Oct 2017 18:02:25 +0000 (20:02 +0200)]
Fix checkstyle issues

These utility classes should be declared as final.

Change-Id: I06b193536f6eccadebb819c67916b6f489933cfc
Signed-off-by: David Suarez <[email protected]>
7 years agoFix checkstyle issues 54/64254/3
David Suarez [Fri, 13 Oct 2017 17:21:11 +0000 (19:21 +0200)]
Fix checkstyle issues

These utility classes should be declared as final.

Change-Id: I7f94c771c0d9c5be05a0a8ccf13d1b8d4924cdfd
Signed-off-by: David Suarez <[email protected]>
7 years agoFix checkstyle issues 59/64259/3
David Suarez [Fri, 13 Oct 2017 17:49:49 +0000 (19:49 +0200)]
Fix checkstyle issues

These utility classes should be declared as final.

Change-Id: I9f391df1c5582da2b0c554c2b395e3c22d41140a
Signed-off-by: David Suarez <[email protected]>
7 years agoFix checkstyle issues 61/64261/2
David Suarez [Fri, 13 Oct 2017 17:54:42 +0000 (19:54 +0200)]
Fix checkstyle issues

These utility classes should be declared as final.

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