bgpcep.git
8 years agoFix PingPong transaction race in EffectiveRibWriter. 03/24603/3
Dana Kutenicsova [Wed, 29 Jul 2015 12:35:10 +0000 (14:35 +0200)]
Fix PingPong transaction race in EffectiveRibWriter.

To avoid race conditions between transactions, create one only when
we need one.

Change-Id: I1052adb326db68aeaac700a0f0169bba3f0b8999
Signed-off-by: Dana Kutenicsova <dkutenic@cisco.com>
8 years agoBUG-3888 : fix comparing ASnumbers 96/24396/6
Dana Kutenicsova [Wed, 22 Jul 2015 12:56:46 +0000 (14:56 +0200)]
BUG-3888 : fix comparing ASnumbers

In order to allow BGP connection, expected remote AS (configured in ODL)
needs to match AS number received in peer Open message. Added expected
remote AS to BGP Session Preferences to allow for this comparison.

Moved validate method from client validator to registry that is used
anytime a peer connects.

This commit is intended to be cherry-picked. Refactoring commit
will be followed up in master branch.

Change-Id: Ie8bfd8b51ee7654b284fe0b0d4ce1db5e365d55b
Signed-off-by: Dana Kutenicsova <dkutenic@cisco.com>
8 years agoRemoved pcep-stateful02 implementation 22/22722/6
Dana Kutenicsova [Tue, 16 Jun 2015 18:57:17 +0000 (20:57 +0200)]
Removed pcep-stateful02 implementation

EOL for pcep-stateful02 is scheduled for Beryllium

Change-Id: I55b58c350273c760366aabb92bae03010d622f42
Signed-off-by: Dana Kutenicsova <dkutenic@cisco.com>
8 years agoFixed some java8 non-compliant javadocs. 89/24689/2
Dana Kutenicsova [Fri, 31 Jul 2015 07:58:34 +0000 (09:58 +0200)]
Fixed some java8 non-compliant javadocs.

Change-Id: I4c067a1466f6373d1e3d6f1582dfb13af5775a40
Signed-off-by: Dana Kutenicsova <dkutenic@cisco.com>
8 years agoFixed SupportedTables map. 23/24623/1
Dana Kutenicsova [Thu, 30 Jul 2015 10:39:44 +0000 (12:39 +0200)]
Fixed SupportedTables map.

We were putting a different NodeIdentifierWithPredicates
to supported tables map, than we were getting.

Change-Id: I4b5264448a833620af9e019c4f49561e6b34d1f1
Signed-off-by: Dana Kutenicsova <dkutenic@cisco.com>
8 years agoStart using Codecs and CodecsRegistry instead of RIBSupportContext. 96/22996/6
Dana Kutenicsova [Fri, 19 Jun 2015 12:40:15 +0000 (14:40 +0200)]
Start using Codecs and CodecsRegistry instead of RIBSupportContext.

Change-Id: I27b46f2c8c02b92e2f93cdd2cba5b52f43891991
Signed-off-by: Dana Kutenicsova <dkutenic@cisco.com>
8 years agoCreated new classes for serializing/deserializing only. 95/22995/5
Dana Kutenicsova [Fri, 19 Jun 2015 11:03:42 +0000 (13:03 +0200)]
Created new classes for serializing/deserializing only.

Codecs and RibSupportContext should be separate
concepts.

Change-Id: I953c36526775105bbee12d3530984a5ee1d9925c
Signed-off-by: Dana Kutenicsova <dkutenic@cisco.com>
8 years agoTCPMD5 support for bmp 41/24441/7
Jeff Liu [Wed, 22 Jul 2015 03:44:57 +0000 (20:44 -0700)]
TCPMD5 support for bmp

Change-Id: I8e35c1213699c8096a47b12783ed864d396a8520
Signed-off-by: Jeff Liu <bigdataops@gmail.com>
8 years agobmp-impl unit test for MonitorImplModule and Factory 13/24513/2
Iveta Halanova [Mon, 27 Jul 2015 08:19:11 +0000 (10:19 +0200)]
bmp-impl unit test for MonitorImplModule and Factory

Change-Id: Iaa216c5bdb146a95cef3e638f465c36747b588cc
Signed-off-by: Iveta Halanova <iveta.halanova@pantheon.sk>
8 years agobmp-impl unit tests 11/24511/2
Iveta Halanova [Fri, 24 Jul 2015 12:55:20 +0000 (14:55 +0200)]
bmp-impl unit tests

Change-Id: I08163ce8406063fba0edd2e742de51534e01b19e
Signed-off-by: Iveta Halanova <iveta.halanova@pantheon.sk>
8 years agoBug-3998: Allow to create multiple instances of instruction scheduler 31/24331/2
Milos Fabian [Sat, 18 Jul 2015 08:49:11 +0000 (10:49 +0200)]
Bug-3998: Allow to create multiple instances of instruction scheduler

Replace container with list of instruction queues. Queue is identified by it's name.
This change allows to create multiple instances and prevent conflicts i.e. in replicated cluster
setup.

Change-Id: I0ed279cf78c83a4d3bc80694d17333ad22376532
Signed-off-by: Milos Fabian <milfabia@cisco.com>
8 years agobmp-spi unit tests 89/24389/2
Iveta Halanova [Wed, 22 Jul 2015 11:55:51 +0000 (13:55 +0200)]
bmp-spi unit tests

Change-Id: I8f4a2b4c4c814c3b117bc5b141a5383c6df08cc5
Signed-off-by: Iveta Halanova <iveta.halanova@pantheon.sk>
8 years agoModule names/files bug fix 81/24381/4
Iveta Halanova [Wed, 22 Jul 2015 09:11:42 +0000 (11:11 +0200)]
Module names/files bug fix

Change-Id: I886779d05325a9f3b08495a784a40e0209a37184
Signed-off-by: Iveta Halanova <iveta.halanova@pantheon.sk>
8 years agoSonar warning fix: too long method 77/24377/2
Iveta Halanova [Wed, 22 Jul 2015 06:57:21 +0000 (08:57 +0200)]
Sonar warning fix: too long method

Change-Id: I4fdc2f39e465fa0266e4d202fbf5f7b69a2de4fa
Signed-off-by: Iveta Halanova <iveta.halanova@pantheon.sk>
8 years agoSonar issue fix: generic exception 75/24375/1
Iveta Halanova [Wed, 22 Jul 2015 06:09:07 +0000 (08:09 +0200)]
Sonar issue fix: generic exception

Change-Id: I4a10483dce22fe1bc8817f849ef0a39223fd0307
Signed-off-by: Iveta Halanova <iveta.halanova@pantheon.sk>
8 years agoBmp TlvUtilTest fix 36/24336/3
Iveta Halanova [Tue, 21 Jul 2015 08:51:25 +0000 (10:51 +0200)]
Bmp TlvUtilTest fix

Change-Id: Ibd3f960b8e61df6f272d00a01f0df39d074796d0
Signed-off-by: Iveta Halanova <iveta.halanova@pantheon.sk>
8 years agoBUG-3823 : fixed Sonar issues for PCEP/programming 83/24283/3
Dana Kutenicsova [Mon, 20 Jul 2015 11:42:58 +0000 (13:42 +0200)]
BUG-3823 : fixed Sonar issues for PCEP/programming

Mostly reduced method complexity.

Change-Id: Id1eed6d27598389ec126e4e590c067a95a61125c
Signed-off-by: Dana Kutenicsova <dkutenic@cisco.com>
8 years agoBUG-3823 : fixed sonar issues in BGP 82/24282/3
Dana Kutenicsova [Mon, 20 Jul 2015 11:42:21 +0000 (13:42 +0200)]
BUG-3823 : fixed sonar issues in BGP

Mostly reduced method complexity.

Change-Id: Ib3bb6a6cc95b854afb61534dc35a45652fc2d2c8
Signed-off-by: Dana Kutenicsova <dkutenic@cisco.com>
8 years agoBUG-3823 : fixed sonar issues in linkstate 81/24281/2
Dana Kutenicsova [Mon, 20 Jul 2015 11:41:31 +0000 (13:41 +0200)]
BUG-3823 : fixed sonar issues in linkstate

Mostly reduced method complexity.

Change-Id: I215f6b1c65ba8f3d29bbcfe4fef99f0df59990cc
Signed-off-by: Dana Kutenicsova <dkutenic@cisco.com>
8 years agoBUG-139: PCEP TLVs extensions implementation 01/24201/13
Iveta Halanova [Thu, 16 Jul 2015 13:23:09 +0000 (15:23 +0200)]
BUG-139: PCEP TLVs extensions implementation

- yang model augmented by LSP-DB-VERSION and SPEAKER-ENTITY-ID
- parsers/serializers implementation
- integration into Lsp and Open objects
- expansion of unit tests

Change-Id: I07d2da05f27a5fdb1fe87ebdb58793ab59a08012
Signed-off-by: Iveta Halanova <iveta.halanova@pantheon.sk>
8 years agoSonar issue fixed. 68/24268/1
Dana Kutenicsova [Sun, 19 Jul 2015 20:13:20 +0000 (22:13 +0200)]
Sonar issue fixed.

serializeNlri() was too complex, split it into
various small methods.

Change-Id: Ibc6623f379beaa077bee193b5d0721504d976dfe
Signed-off-by: Dana Kutenicsova <dkutenic@cisco.com>
8 years agoSonar issue fix: BestPathSelector 38/24238/1
Iveta Halanova [Fri, 17 Jul 2015 12:35:35 +0000 (14:35 +0200)]
Sonar issue fix: BestPathSelector

Replaced if-then-else statement by a single return statement.

Change-Id: Ia5e5e162a3337e01cc26253e586de6eefd989680
Signed-off-by: Iveta Halanova <iveta.halanova@pantheon.sk>
8 years agoBUG-139: Implement PCEP state synchronization avoidance 86/24086/18
Iveta Halanova [Tue, 14 Jul 2015 08:17:10 +0000 (10:17 +0200)]
BUG-139: Implement PCEP state synchronization avoidance

Advertising Support of Synchronization Optimizations
- defined yang model for STATEFUL-PCE-CAPABILITY TLV
- implemented SyncOptimizationsCapabilityTlvParser
- parser/serializer Activator
- Config-subsystem integration
- tests: update and extension

Change-Id: Ief4a928a1cd3e6c18e35f13fa1e49053351eebd7
Signed-off-by: Iveta Halanova <iveta.halanova@pantheon.sk>
8 years agoBUG-3840 : introduced Supported tables 91/24091/6
Dana Kutenicsova [Wed, 8 Jul 2015 07:19:34 +0000 (09:19 +0200)]
BUG-3840 : introduced Supported tables

When advertizing to other peers, we need to keep track
of afi/safi that a client supports. Not having them
causes exceptions in the logs, as the particular
AdjRibsOut was not created.

Change-Id: I3bea92666c200c8dceb83e3c4f144b9afe1f0305
Signed-off-by: Dana Kutenicsova <dkutenic@cisco.com>
8 years agoPCEP LSP obj parser: missing PathBindingTlv serialization 25/24225/1
Iveta Halanova [Fri, 17 Jul 2015 06:31:03 +0000 (08:31 +0200)]
PCEP LSP obj parser: missing PathBindingTlv serialization

Change-Id: Ife794f9d485cdffc2dc8c1efb9d5e77ccc2c170e
Signed-off-by: Iveta Halanova <iveta.halanova@pantheon.sk>
8 years agoBug: Fix Cycles issues 82/24182/1
Claudio D. Gasparini [Thu, 16 Jul 2015 08:02:00 +0000 (10:02 +0200)]
Bug: Fix Cycles issues

Change-Id: I18e58d635dbc7cbd33d1b57afd7269278e25567c
Signed-off-by: Claudio D. Gasparini <cgaspari@cisco.com>
8 years agoBUG: Fix Sonar Issues 50/24150/2
Claudio D. Gasparini [Wed, 15 Jul 2015 09:17:41 +0000 (11:17 +0200)]
BUG: Fix Sonar Issues

Change-Id: Ie037f085b39da5a0e6743798695c372ebd604a9e
Signed-off-by: Claudio D. Gasparini <cgaspari@cisco.com>
8 years agoRib-impl unit test for export policies 82/24082/4
Iveta Halanova [Mon, 13 Jul 2015 12:09:00 +0000 (14:09 +0200)]
Rib-impl unit test for export policies

Change-Id: I094f9fab6774b824d8f1cbea9a8d851036f1853a
Signed-off-by: Iveta Halanova <iveta.halanova@pantheon.sk>
8 years agoBug-3780: Fixed 41-bgp-example init config file 90/24090/1
Milos Fabian [Tue, 14 Jul 2015 10:17:16 +0000 (12:17 +0200)]
Bug-3780: Fixed 41-bgp-example init config file

bgp-peer was refering to incorrect bgp-rib service type

Change-Id: I5b5dfc94bc073677e9fe4c7fe495cff573b1a2b0
Signed-off-by: Milos Fabian <milfabia@cisco.com>
8 years agoBUG-2873 : Remove dependency protocol-framework on PCEP 31/23431/11
Claudio D. Gasparini [Fri, 26 Jun 2015 07:10:08 +0000 (09:10 +0200)]
BUG-2873 : Remove dependency protocol-framework on PCEP

Change-Id: I612108eeba62385b455fa7d8f4e633f41486a7aa
Signed-off-by: Claudio D. Gasparini <cgaspari@cisco.com>
8 years agoBUG-2823: Simplify AS_PATH encoding 75/23775/22
Iveta Halanova [Mon, 13 Jul 2015 05:43:24 +0000 (07:43 +0200)]
BUG-2823: Simplify AS_PATH encoding

Expaned yang model by leaf-lists as-set and as-sequence as new type and marked
old type as obsolete.
Changed bgp-parser functionality to support just new type.
Changed bgp-rib functionality to support just new type.
Updated and expanded unit tests.

Change-Id: I1a832167afa9c19f4de358704d122d969b8b6fa8
Signed-off-by: Iveta Halanova <iveta.halanova@pantheon.sk>
8 years agoBUG-2873 : Remove dependency protocol-framework on BGP 76/23276/14
Claudio D. Gasparini [Thu, 25 Jun 2015 12:24:43 +0000 (14:24 +0200)]
BUG-2873 : Remove dependency protocol-framework on BGP

Change-Id: Id04890456a517ca416f18be3de5bb3e777a5bfe9
Signed-off-by: Claudio D. Gasparini <cgaspari@cisco.com>
8 years agoBestPathSelector bug fix 47/24047/4
Iveta Halanova [Mon, 13 Jul 2015 07:23:22 +0000 (09:23 +0200)]
BestPathSelector bug fix

expanded unit test BestPathSelectorTest and corrected BestPathSelector code.

Change-Id: I0591e7c7dbe455c852faed677b80694ef0dd9e63
Signed-off-by: Iveta Halanova <iveta.halanova@pantheon.sk>
8 years agoBug: Fix missing break 49/24049/1
Claudio D. Gasparini [Mon, 13 Jul 2015 07:09:39 +0000 (09:09 +0200)]
Bug: Fix missing break

Change-Id: If67090fbbff9baa160d58df5d9ccbb4884670853
Signed-off-by: Claudio D. Gasparini <cgaspari@cisco.com>
8 years agoSonar Issues fix 18/23818/4
Iveta Halanova [Tue, 7 Jul 2015 12:16:36 +0000 (14:16 +0200)]
Sonar Issues fix

Change-Id: I1af7e2a6926b6d8717e862382f5540beab2df3a0
Signed-off-by: Iveta Halanova <iveta.halanova@pantheon.sk>
8 years agoTest coverage: yang.bmp.spi 28/23728/5
Iveta Halanova [Fri, 3 Jul 2015 09:06:50 +0000 (11:06 +0200)]
Test coverage: yang.bmp.spi

Change-Id: Iaea73e11132e278e6be094f8bdab6dcf5ba56d75
Signed-off-by: Iveta Halanova <iveta.halanova@pantheon.sk>
8 years agoAdd <relativePath/> for autorelease scripts 48/23748/3
Thanh Ha [Sat, 4 Jul 2015 03:04:18 +0000 (23:04 -0400)]
Add <relativePath/> for autorelease scripts

Autorelease requires <relativePath/> for parents that live outside of a
project so that it's scripts can properly find and replace this value
for use in the autorelease build.

Change-Id: I00f6a37c71d8a261316648eb2f00f3b3e7382ebe
Signed-off-by: Thanh Ha <thanh.ha@linuxfoundation.org>
8 years agoIllegalArgument Exception changed to Preconditions checks 70/23770/8
Iveta Halanova [Mon, 6 Jul 2015 07:31:10 +0000 (09:31 +0200)]
IllegalArgument Exception changed to Preconditions checks

Change-Id: I30f607cbc6cbe1e6abd20b0bb375a4073e28cd17
Signed-off-by: Iveta Halanova <iveta.halanova@pantheon.sk>
8 years agoBug-3075: BMP application integration test 00/22200/6
Milos Fabian [Tue, 9 Jun 2015 09:44:46 +0000 (11:44 +0200)]
Bug-3075: BMP application integration test

-doploys client to BMP app. and send messages, check datastore

Change-Id: I6fadd5450bd2ff81a84b9b5cd38260187905a984
Signed-off-by: Milos Fabian <milfabia@cisco.com>
8 years agoBug-3075: augment BMP pre-policy RIB and post-policy RIB 99/22199/8
Milos Fabian [Tue, 9 Jun 2015 09:31:01 +0000 (11:31 +0200)]
Bug-3075: augment BMP pre-policy RIB and post-policy RIB

-augment BMP RIBs with BGP routes extensions (multiprotocol, linkstate, flowspec)

Change-Id: I84a02000c6672ad73d03b8939e274f291939beb2
Signed-off-by: Milos Fabian <milfabia@cisco.com>
8 years agoFixed unit tests after yangtools change. 67/23767/3
Dana Kutenicsova [Mon, 6 Jul 2015 08:34:46 +0000 (10:34 +0200)]
Fixed unit tests after yangtools change.

Commit in yangtools master: https://git.opendaylight.org/gerrit/#/c/23673/
Commit in yangtools lithium: https://git.opendaylight.org/gerrit/#/c/23045/

Change-Id: Ifab39c3188e80e793ecf384d496d853809e940fb
Signed-off-by: Dana Kutenicsova <dkutenic@cisco.com>
8 years agoBug-3075: BMP feature 98/22198/7
Milos Fabian [Tue, 9 Jun 2015 09:23:51 +0000 (11:23 +0200)]
Bug-3075: BMP feature

-integrate BMP with karaf
-defines "odl-bgpcep-bmp-all" feature
-starts all required bundles/features and initial config files

Change-Id: I145fbc7ca437ca9ff0cda322dabb5f347be19257
Signed-off-by: Milos Fabian <milfabia@cisco.com>
8 years agoBug-3075: BMP initial configuration 97/22197/6
Milos Fabian [Tue, 9 Jun 2015 08:50:19 +0000 (10:50 +0200)]
Bug-3075: BMP initial configuration

-initial configuration xml files
-instatiate BMP extension provider, dispatcher and application

Change-Id: I633c45d61dfa463c275c90bd1a68e8755f494c6d
Signed-off-by: Milos Fabian <milfabia@cisco.com>
8 years agoTest coverage: bmp.spi.registry 80/23680/6
Iveta Halanova [Thu, 2 Jul 2015 09:36:21 +0000 (11:36 +0200)]
Test coverage: bmp.spi.registry

Change-Id: I2165c8e7861aa4667845c8ea8bbba14e2fa99469
Signed-off-by: Iveta Halanova <iveta.halanova@pantheon.sk>
8 years agoBug-3075: BMP application configuration 96/22196/7
Milos Fabian [Tue, 9 Jun 2015 08:32:08 +0000 (10:32 +0200)]
Bug-3075: BMP application configuration

-integrate BMP application with config subystem
-creates instance of BMP application
-configurable binding port and IP address of server instance

Change-Id: Id95fffc0612a746e3ea37f4de7af43d4df25265d
Signed-off-by: Milos Fabian <milfabia@cisco.com>
8 years agoBug 2208 - pcc-mock: enhancemnets 53/22853/4
Ladislav Borak [Wed, 10 Jun 2015 14:56:04 +0000 (16:56 +0200)]
Bug 2208 - pcc-mock: enhancemnets

*new functionality
- refuse operations with not delegated LSP
- refuse removing of non PCE-initiated LSPs
- delegate PCC's LSPs to the first PCE at start-up
- support "return delegate" request from PCE for PCC's LSPs, re-delegate, after the "redelegation timeout" expires, to the next PCE in cyclic order
- support "delegate" and "return delegation" for PCE-initiated LSPs, delete LSP if "state-timeout" expires
- if PCE connects, report all LSPs (PCC's and PCE-initiated) with actual state

*new input arguments
--redelegation-timeout <seconds>, default 0 - start timer when delegation for PCC's LSP is returned or when PCE fails;
hold delegation for PCE until expiration; timout is cancelled when LSP is delegated to PCE

--state-timeout <seconds>, default -1 (infinite) - start timer when delegation is returned or when PCE fails;
after timeout expiration, LSP is removed; timeout is cancelled when LSP is delegated to PCE

*new PCEP error added
- LSP is not PCE-initiated Error-type=19, Error-value=9

Change-Id: I7bb4cdf03dd04f9caddc13cf7a70c913aa7e6e2b
Signed-off-by: Ladislav Borak <lborak@cisco.com>
Signed-off-by: Milos Fabian <milfabia@cisco.com>
8 years agoBug-3919: Advertisement is always done using MP_REACH 84/23684/9
Claudio D. Gasparini [Thu, 2 Jul 2015 14:14:52 +0000 (16:14 +0200)]
Bug-3919: Advertisement is always done using MP_REACH

Fix Check whenever Tablekey is empty do not advertise using MP_REACH

Change-Id: Ia77e8fbdf841c61b82e85965001ef0f1a183e369
Signed-off-by: Claudio D. Gasparini <cgaspari@cisco.com>
8 years agoAdds new utility function 25/23725/1
Milos Fabian [Fri, 3 Jul 2015 09:21:50 +0000 (11:21 +0200)]
Adds new utility function

converts IpAddress and PortNumber to InetSocketAddress

Change-Id: I43f5e6af0316aeb0dfe2ab901d3c396118da707b
Signed-off-by: Milos Fabian <milfabia@cisco.com>
8 years agoBug-3075: BMP application 95/22195/7
Milos Fabian [Tue, 9 Jun 2015 08:00:47 +0000 (10:00 +0200)]
Bug-3075: BMP application

-implementation of application layer
-handle messages->transform->write do data-store

*BmpMonitoringStation:
-top level of application
-instantiates lower level of application
-creates instance in data-store
*RouterSessionManager:
-manages sessions with remote devices (routers)
*BmpRouter:
-represents application layer of session with particular device
-message handling/delegation of per-peer messages
-manages peers as reported by router
*BmpRouterPeer
-represents peer reported by remote device
-per-peer messages handling
-delegates
*BmpRibInWriter
-handling of Update messages and their transformation to Rib
-uses BGP RIB extensions service

Change-Id: I5f00197f7f96d6feb3f21ddd5726c464c3aad1f3
Signed-off-by: Milos Fabian <milfabia@cisco.com>
8 years agoAdds utility function that converts InetAddress to IpAddress 97/23697/1
Milos Fabian [Thu, 2 Jul 2015 16:05:17 +0000 (18:05 +0200)]
Adds utility function that converts InetAddress to IpAddress

Change-Id: Id19c064888755a17fb99de070de3182538a1349e
Signed-off-by: Milos Fabian <milfabia@cisco.com>
8 years agoSonar Warning: removed unused parameter 69/23669/2
Iveta Halanova [Thu, 2 Jul 2015 08:44:08 +0000 (10:44 +0200)]
Sonar Warning: removed unused parameter

Change-Id: Iab6fa30f3c0e53b239a8a088cb15e1a21e742f05
Signed-off-by: Iveta Halanova <iveta.halanova@pantheon.sk>
8 years agoTest coverage: bmp.spi.parser 68/23668/2
Iveta Halanova [Thu, 2 Jul 2015 06:04:35 +0000 (08:04 +0200)]
Test coverage: bmp.spi.parser

Tests expansion

Change-Id: I5a0c470fb7f40bd098f72e182c0537a657f52bea
Signed-off-by: Iveta Halanova <iveta.halanova@pantheon.sk>
8 years agoSonar Warning Fix: complexity issues 34/23634/8
Iveta Halanova [Wed, 1 Jul 2015 08:35:15 +0000 (10:35 +0200)]
Sonar Warning Fix: complexity issues

NodeChangedListener:
- extracted for cycle into private updateTransaction method
Stateful07TopologySessionListener
- extracted building the Path variable
FSNlriParser
- switch extraction into private method setFlowspecType

Change-Id: Iec4d2d179527560150046531cc02fd198ea3d323
Signed-off-by: Iveta Halanova <iveta.halanova@pantheon.sk>
8 years agoSonar Warning Fix: reused parameter 37/23637/2
Iveta Halanova [Wed, 1 Jul 2015 11:11:36 +0000 (13:11 +0200)]
Sonar Warning Fix: reused parameter

moved obj Object into updated method
created new Object instead of reusing parameter

Change-Id: I0694a8e45f9c9070dd894c3382e8e39ec13beaf1
Signed-off-by: Iveta Halanova <iveta.halanova@pantheon.sk>
8 years agoSonar Warning Fix: anonymous class reduction 29/23629/2
Iveta Halanova [Wed, 1 Jul 2015 07:31:54 +0000 (09:31 +0200)]
Sonar Warning Fix: anonymous class reduction

Change-Id: I93661cd78f12231e6989c7800606dd46fc75c3e7
Signed-off-by: Iveta Halanova <iveta.halanova@pantheon.sk>
8 years agoSonar Warning Fix: BmpMessageConstants 89/23589/2
Iveta Halanova [Tue, 30 Jun 2015 12:00:14 +0000 (14:00 +0200)]
Sonar Warning Fix: BmpMessageConstants

set class as final

Change-Id: Iedd9912c7a0c8b5ada130c9270b6ffb621e3e856
Signed-off-by: Iveta Halanova <iveta.halanova@pantheon.sk>
8 years agoSonar Warning Fix: LocRibWriter 25/23625/2
Iveta Halanova [Wed, 1 Jul 2015 06:28:08 +0000 (08:28 +0200)]
Sonar Warning Fix: LocRibWriter

decrease of update method complexity
exctracted two for cycles into two private methods

Change-Id: I10003e4a87dccf251b1f06a8b7a964b64b51bc8b
Signed-off-by: Iveta Halanova <iveta.halanova@pantheon.sk>
8 years agoSonar Warning Fix: RIBImpl - log exception 28/23628/1
Iveta Halanova [Wed, 1 Jul 2015 06:51:25 +0000 (08:51 +0200)]
Sonar Warning Fix: RIBImpl - log exception

Change-Id: I4bcb9ab05c62366e679d7d6d95d9d6d88de9d47f
Signed-off-by: Iveta Halanova <iveta.halanova@pantheon.sk>
8 years agoUnit tests for AttributeOperations. 38/23238/13
Dana Kutenicsova [Tue, 30 Jun 2015 13:50:46 +0000 (15:50 +0200)]
Unit tests for AttributeOperations.

Change-Id: I4a02fb11e25497d9ff3538c3103481bfa5b9f049
Signed-off-by: Dana Kutenicsova <dkutenic@cisco.com>
8 years agoBUG-3892 : specify LeafSet as ordered 95/23595/1
Dana Kutenicsova [Tue, 30 Jun 2015 13:45:47 +0000 (15:45 +0200)]
BUG-3892 : specify LeafSet as ordered

When creating own ClusterId list it must be
ordered too.

Change-Id: I261f3757cd9ffafa6016d84f6238ce34aeedc69a
Signed-off-by: Dana Kutenicsova <dkutenic@cisco.com>
8 years agoBug-2848: BGP-LS EPE extension: Protocol Id 78/23578/1
Iveta Halanova [Tue, 30 Jun 2015 09:21:49 +0000 (11:21 +0200)]
Bug-2848: BGP-LS EPE extension: Protocol Id

Added new protocol id value (bgp-epe)

Change-Id: I0d092464d9ea5253ce1f37c6db6355fec8fd6389
Signed-off-by: Iveta Halanova <iveta.halanova@pantheon.sk>
8 years agoBug-2848: BGP-LS EPE extension: Peer Sid 96/23396/6
Iveta Halanova [Fri, 26 Jun 2015 09:31:41 +0000 (11:31 +0200)]
Bug-2848: BGP-LS EPE extension: Peer Sid

Parsering/serializing for Peer-SID, Peer-Set-SID
Added parsering/serializing of type and length of (Lan)AdjacencySegmentIdentifier

Change-Id: I56b4c2c131c205fa727f4261fc9cd06d4b39dd63
Signed-off-by: Iveta Halanova <iveta.halanova@pantheon.sk>
8 years agoBug-2848: BGP-LS EPE extension: EPE Descriptors 08/23408/6
Iveta Halanova [Fri, 26 Jun 2015 09:44:51 +0000 (11:44 +0200)]
Bug-2848: BGP-LS EPE extension: EPE Descriptors

Parsering and serialization for Local and Remote node descriptors expanted by:
- BGP router id
- member As number

Change-Id: I1c25f76aa1a7947f7ff459bf4f9019116e055f49
Signed-off-by: Iveta Halanova <iveta.halanova@pantheon.sk>
8 years agoBUG-3879 : Fixed NPE in OspfPseudonodes case. 30/23530/2
Dana Kutenicsova [Thu, 25 Jun 2015 13:36:34 +0000 (15:36 +0200)]
BUG-3879 : Fixed NPE in OspfPseudonodes case.

OspfPseudonodes weren't properly extracted from BI to BA.

Change-Id: I30a6b163806af69c9c8efa77a5d9f208228d54fa
Signed-off-by: Dana Kutenicsova <dkutenic@cisco.com>
(cherry picked from commit 044b654e168135e1a272b7a5f4fcf0f44ed469aa)

8 years agoBUG-3892 : added ordered-by user statement to clusterIds 20/23520/2
Dana Kutenicsova [Mon, 29 Jun 2015 15:51:06 +0000 (17:51 +0200)]
BUG-3892 : added ordered-by user statement to clusterIds

Without this statement, LeafSet does not guarantee the order
of clusterIds. We need to maintain the order as our clusterId
needs to be first when advertising routes.

Change-Id: Ic5d7ca2fb514a31cffc958a8a69ee3f57e3d2636
Signed-off-by: Dana Kutenicsova <dkutenic@cisco.com>
8 years agoBUG-3839 : fixed various minor bugs related to AS Path BPS 56/23256/3
Dana Kutenicsova [Tue, 23 Jun 2015 13:38:02 +0000 (15:38 +0200)]
BUG-3839 : fixed various minor bugs related to AS Path BPS

- fixed classcast from Long to String
- fixed wrong NodeIdentifier
- fixed wrong algorithm in BPS

Change-Id: I7d87534d4e027d62a13fca7da13af81cb5f89caa
Signed-off-by: Dana Kutenicsova <dkutenic@cisco.com>
8 years agoBUG-3860 : fixed config through RESTCONF 65/23265/2
Dana Kutenicsova [Thu, 25 Jun 2015 11:07:13 +0000 (13:07 +0200)]
BUG-3860 : fixed config through RESTCONF

Not closing loc-rib caused many loc-ribs to be created
through config-pusher causing NodeModificationExceptions.

Change-Id: I9b168615875f02a40b120b13cc92e9d5bdcd42be
Signed-off-by: Dana Kutenicsova <dkutenic@cisco.com>
8 years agoFix Sonar warnings: bgp-bmp-impl 97/23497/2
Iveta Halanova [Mon, 29 Jun 2015 06:26:41 +0000 (08:26 +0200)]
Fix Sonar warnings: bgp-bmp-impl

Change-Id: I50a3d1c10204cd421246a0d5f838bcfd83e3ff9d
Signed-off-by: Iveta Halanova <iveta.halanova@pantheon.sk>
8 years agobgp-bmp-spi sonar warnings 17/23417/5
Iveta Halanova [Fri, 26 Jun 2015 13:03:39 +0000 (15:03 +0200)]
bgp-bmp-spi sonar warnings

Change-Id: I167e53a743aaae073107addc86214d60e4f15302
Signed-off-by: Iveta Halanova <iveta.halanova@pantheon.sk>
8 years agoBug-3075: BMP dependency management 94/22194/7
Milos Fabian [Thu, 18 Jun 2015 09:11:28 +0000 (11:11 +0200)]
Bug-3075: BMP dependency management

-use project's dependency management
-clean dependecies in BMP poms

Change-Id: I4bbeae6d164d882ad33336b34dfc6a8712eeb9cd
Signed-off-by: Milos Fabian <milfabia@cisco.com>
8 years agobgp-bmp-spi cycle issue fix 30/23430/2
Iveta Halanova [Fri, 26 Jun 2015 14:22:38 +0000 (16:22 +0200)]
bgp-bmp-spi cycle issue fix

Moved BmpTlvRegistry to spi.parser package

Change-Id: I55a39113617bf844503936ad93e7c7a438ff7840
Signed-off-by: Iveta Halanova <iveta.halanova@pantheon.sk>
8 years agoBug-2848: BGP-LS EPE extension yang model correction 10/23410/4
Iveta Halanova [Fri, 26 Jun 2015 12:07:15 +0000 (14:07 +0200)]
Bug-2848: BGP-LS EPE extension yang model correction

Added epe descriptors to local descriptors
Removed from link descriptors

Change-Id: I434d5db602b599b50f8517ecd4b1b05c9f027048
Signed-off-by: Iveta Halanova <iveta.halanova@pantheon.sk>
8 years agonagasena lib: group id change 14/23414/1
Iveta Halanova [Fri, 26 Jun 2015 12:11:58 +0000 (14:11 +0200)]
nagasena lib: group id change

Change-Id: I1a93d8ebf081649b9918790f47a868400313c17d
Signed-off-by: Iveta Halanova <iveta.halanova@pantheon.sk>
8 years agoBug-3714: Generalize RIBSupport 58/22858/5
Milos Fabian [Thu, 18 Jun 2015 07:38:57 +0000 (09:38 +0200)]
Bug-3714: Generalize RIBSupport

Changed instance idetifier for writing/deleting routes operations in RIBSupport.
Now identifier has to point to table's routes choice data node.
The change allows to properly use write/delete routes DOM operations out of "rib-api" module namespace.

Change-Id: I4b877683fdb5232179da05eab78120187a2fcf4f
Signed-off-by: Milos Fabian <milfabia@cisco.com>
8 years agoBug-3075: BMP server dispatcher configuration 93/22193/5
Milos Fabian [Tue, 9 Jun 2015 07:24:55 +0000 (09:24 +0200)]
Bug-3075: BMP server dispatcher configuration

-integrates BMP dispatcher with config subsystem

Change-Id: If2b953361ae53ca8d3093bc19a0de2e892cb42cc
Signed-off-by: Milos Fabian <milfabia@cisco.com>
8 years agoBug-3075: BMP server dispatcher 92/22192/5
Milos Fabian [Tue, 9 Jun 2015 07:19:14 +0000 (09:19 +0200)]
Bug-3075: BMP server dispatcher

-based on netty protocol framework
-dispatcher is able to create BMP servers
-BMP server is binded to particular IP address and port
-listening for incomimg connections
-initializes channel pipeline - message and session handler

Change-Id: Ibe15ef3fbde8369cdac320db82ba18f3f1e46d5f
Signed-off-by: Milos Fabian <milfabia@cisco.com>
8 years agoBug-3075: BMP session implementation 91/22191/6
Milos Fabian [Tue, 9 Jun 2015 07:12:41 +0000 (09:12 +0200)]
Bug-3075: BMP session implementation

-basic channel reading/message handling
-delegating messages to application layer
-error handling, end of input detection
-simple session factory

Change-Id: I827c44413a4cc7cfa6066ceb2c63b8d9c85c49bc
Signed-off-by: Milos Fabian <milfabia@cisco.com>
8 years agoUse {{VERSION}} where possible 82/23282/1
Robert Varga [Thu, 25 Jun 2015 17:11:51 +0000 (19:11 +0200)]
Use {{VERSION}} where possible

Using properties is not something what we want to do, so switch where
supported.

Change-Id: I298167354857b832aebf48346238d22ac1b6c929
Signed-off-by: Robert Varga <rovarga@cisco.com>
8 years agoBug-2848: BGP-LS EPE extension yang model 01/23101/3
Milos Fabian [Mon, 22 Jun 2015 14:10:44 +0000 (16:10 +0200)]
Bug-2848: BGP-LS EPE extension yang model

Two new Node Descriptors Sub-TLVs:
-BGP-ROUTER-ID
-Confederation Member
A new Protocol-ID for Link-Type NLRI.
Two new BGP-LS Attribute Sub-TLVs:
-Peer-SID
-Peer-Set-SID
-ref.: https://tools.ietf.org/html/draft-ietf-idr-bgpls-segment-routing-epe-00

Change-Id: I78a32e4961ad73181b61750f681a496a91c8f9c5
Signed-off-by: Milos Fabian <milfabia@cisco.com>
8 years agoGet rid of BestPathState FIXMEs. 41/23241/3
Dana Kutenicsova [Wed, 17 Jun 2015 18:29:00 +0000 (20:29 +0200)]
Get rid of BestPathState FIXMEs.

Created method to parse AsPath segments
from BI to BA.

Change-Id: Ibc49da3ddfb0ba5e1bb0a8da562c00e069df3923
Signed-off-by: Dana Kutenicsova <dkutenic@cisco.com>
(cherry picked from commit 8640c8d02873b9ce26b8ac9150804cd7a873edb2)

8 years agoBUG-3075: BMP Session and dispatcher API 90/22190/6
Milos Fabian [Tue, 9 Jun 2015 07:02:08 +0000 (09:02 +0200)]
BUG-3075: BMP Session and dispatcher API

-BMP sesssion provides basic message handling as a part of pipeline
and decouples connection layer and application layer - delegated to
session listener
-BMP dispatcher is capable of creating BMP servers

Change-Id: I8cc5c6e71c8f6a592efe06f58152dd96640579af
Signed-off-by: Milos Fabian <milfabia@cisco.com>
8 years agoBug-3075: BMP Messages parser/serializer junit tests 33/20833/13
Milos Fabian [Mon, 25 May 2015 09:01:38 +0000 (11:01 +0200)]
Bug-3075: BMP Messages parser/serializer junit tests

Change-Id: Id2c67171f4d05f935fd9bfe3efe7dfe0c95a83cd
Signed-off-by: Ladislav Borak <lborak@cisco.com>
Signed-off-by: Milos Fabian <milfabia@cisco.com>
8 years agoBug-3075: netty encoder/decored for BMP messages 06/21106/5
Milos Fabian [Tue, 26 May 2015 09:41:23 +0000 (11:41 +0200)]
Bug-3075: netty encoder/decored for BMP messages

-integrate message registry with netty pipeline handlers

Change-Id: I8207b57ba7d859ff0a877c8eb6d5e4d7ddbc93ef
Signed-off-by: Milos Fabian <milfabia@cisco.com>
8 years agoBug-3075: BMP activator 05/21105/6
Milos Fabian [Tue, 26 May 2015 09:40:06 +0000 (11:40 +0200)]
Bug-3075: BMP activator

-registers all message and TLV parsers/serializers
-integrate BMP activator with config subystem
-create instance of activator, consume BGP extensions

Change-Id: I745fac383ff6c7e9d275ce94ed426b2751aacec6
Signed-off-by: Milos Fabian <milfabia@cisco.com>
8 years agoBug-3075: BMP TLV parsers/serializers implementation 04/21104/4
Milos Fabian [Tue, 26 May 2015 09:29:58 +0000 (11:29 +0200)]
Bug-3075: BMP TLV parsers/serializers implementation

Change-Id: Id3e9759e86f2054482d08e86fd5e8632a5db59ee
Signed-off-by: Milos Fabian <milfabia@cisco.com>
8 years agoBug-3075: BMP message parsers/serializers implementation 03/21103/5
Milos Fabian [Tue, 26 May 2015 09:27:10 +0000 (11:27 +0200)]
Bug-3075: BMP message parsers/serializers implementation

Change-Id: Ie0b8ee9c0086ffcebff9d5d767d87c198673f9fd
Signed-off-by: Milos Fabian <milfabia@cisco.com>
8 years agoFix SrNodeAttributesParser + test 59/22859/12
Iveta Halanova [Mon, 22 Jun 2015 13:06:52 +0000 (15:06 +0200)]
Fix SrNodeAttributesParser + test

expanded tests by parsering serialized outputs
fixed seralization, parsering

Change-Id: I8382ab97d1b19791ba5395a69d1a9873be6cf8e5
Signed-off-by: Iveta Halanova <iveta.halanova@pantheon.sk>
8 years agoBug-3075: BMP extension provider configuration 92/21092/4
Milos Fabian [Tue, 26 May 2015 05:51:19 +0000 (07:51 +0200)]
Bug-3075: BMP extension provider configuration

-configuration service for BMP extension provider and extension consumer

Change-Id: I53829a09a973b4da8ebda62d897e9958ce4fb317
Signed-off-by: Milos Fabian <milfabia@cisco.com>
8 years agoBug-3075: BMP parsers/serializers and registry abstraction 91/21091/4
Milos Fabian [Tue, 26 May 2015 07:48:59 +0000 (09:48 +0200)]
Bug-3075: BMP parsers/serializers and registry abstraction

-implemented common functionality for BMP parsers/serializers
-implemented simple registr, extension provider and consumer

Change-Id: Ie3b042d8b20b2f25d15c40bea529a45a4f9db1e4
Signed-off-by: Milos Fabian <milfabia@cisco.com>
8 years agoBug-3075: BMP parser/serializer and registry APIs 89/21089/4
Milos Fabian [Tue, 26 May 2015 07:40:22 +0000 (09:40 +0200)]
Bug-3075: BMP parser/serializer and registry APIs

-Java API for BMP message and TLV parsers/serializers
-Java API for BMP registry and extension consumer/provider

Change-Id: I4ea6fe6418b964bcf8976d85724d120efb128a5f
Signed-off-by: Milos Fabian <milfabia@cisco.com>
8 years agoBug-3075: BMP application yang model 27/21127/5
Milos Fabian [Tue, 26 May 2015 12:33:59 +0000 (14:33 +0200)]
Bug-3075: BMP application yang model

-representing data model for BMP application

Change-Id: I81c78516850829746df08a5d68ce9a2992d8d022
Signed-off-by: Milos Fabian <milfabia@cisco.com>
8 years agoBug-2802: BGP Update message serialization bug 30/22830/2
Milos Fabian [Wed, 17 Jun 2015 15:55:07 +0000 (17:55 +0200)]
Bug-2802: BGP Update message serialization bug

The next-hop value from MP reach attribute is not stored in AdjRibIn,
thus next-hop is missing in output Update message.
Copy next-hop value from MP reach to next hop attribute to keep it in DS.

Change-Id: I866af30ecc2cd0baa0ca1ca2f763dc03176ac009
Signed-off-by: Milos Fabian <milfabia@cisco.com>
(cherry picked from commit 7505f0031b58a178994ed90e12d0a7bb111ae8e1)

8 years agoRemoved duplicate logging from message parsers. 79/22979/4
Dana Kutenicsova [Fri, 19 Jun 2015 09:47:20 +0000 (11:47 +0200)]
Removed duplicate logging from message parsers.

Message hex dumps are already printed out via Encoder/Decoder
classes, no need to print them in here as well.

Unified log messages and renamed some variables to make
code more readable.

Change-Id: I9bdcfed891ee1175d28ba1b993f1f22c9774da3e
Signed-off-by: Dana Kutenicsova <dkutenic@cisco.com>
8 years agoBGPCEP Tests 29/23029/2
Claudio D. Gasparini [Wed, 17 Jun 2015 11:11:38 +0000 (13:11 +0200)]
BGPCEP Tests

PCEPExtensionProviderContextTest
LinkstateRIBSupportTest
FlowspecModuleTest

Change-Id: Ieec83b4dec1b010355a1001cf0f5adda4094c524
Signed-off-by: Claudio D. Gasparini <cgaspari@cisco.com>
(cherry picked from commit bfb96610200e708c010048c4ffde17c98e97000e)

8 years agoAdded a bit more tests for correctness and code coverage. 91/22891/2
Dana Kutenicsova [Thu, 18 Jun 2015 14:23:40 +0000 (16:23 +0200)]
Added a bit more tests for correctness and code coverage.

Change-Id: I21f1497279d4b297b533b5dc0872b545e594b11a
Signed-off-by: Dana Kutenicsova <dkutenic@cisco.com>
8 years agoBUG-3362 : improved tests for application peer 89/22989/2
Dana Kutenicsova [Mon, 15 Jun 2015 21:53:38 +0000 (23:53 +0200)]
BUG-3362 : improved tests for application peer

Change-Id: I7fd771d534b1950ff08b47e7126009d8d3afe768
Signed-off-by: Dana Kutenicsova <dkutenic@cisco.com>
(cherry picked from commit 9fee7709aee0234aa264f8d38bd3127957872cc0)

8 years agoRibSupportUtils Unit Test 64/22864/4
Iveta Halanova [Thu, 18 Jun 2015 09:20:02 +0000 (11:20 +0200)]
RibSupportUtils Unit Test

Change-Id: I520fea6782847f3c29ffbfb520f133944613aa99
Signed-off-by: Iveta Halanova <iveta.halanova@pantheon.sk>
8 years agoUnit test: PCCEndPointIpv4ObjectParser 83/22883/4
Iveta Halanova [Thu, 18 Jun 2015 13:07:07 +0000 (15:07 +0200)]
Unit test: PCCEndPointIpv4ObjectParser

Test coverage from 5.6% to 100%

Change-Id: I33855fba9d89d30cfaef9cec325020833d4a41e7
Signed-off-by: Iveta Halanova <iveta.halanova@pantheon.sk>
8 years agoUnit test: MultiProtocolCapabilityHandler 89/22889/5
Iveta Halanova [Thu, 18 Jun 2015 14:13:10 +0000 (16:13 +0200)]
Unit test: MultiProtocolCapabilityHandler

from 19.5% to 100% test coverage

Change-Id: I3b7ff5e31ef2c2bd33561366902f3d87fc40e531
Signed-off-by: Iveta Halanova <iveta.halanova@pantheon.sk>
8 years agoUnit test for Ipv4NlriParser 54/22754/4
Iveta Halanova [Wed, 17 Jun 2015 05:26:38 +0000 (07:26 +0200)]
Unit test for Ipv4NlriParser

created Ipv4NlriParserTest
correction of:
- check of nlri instance type
- serialize nlri in <length, prefix> order

Change-Id: I00c06b352aabe4eb94a492c737377ec1efefec99
Signed-off-by: Iveta Halanova <iveta.halanova@pantheon.sk>