Claudio D. Gasparini [Sat, 28 Nov 2015 21:33:05 +0000 (22:33 +0100)]
BUG-2208: PCC-MOCK Stateful Sync Opt
Update pcc-mock to support:
+State Synchronization Avoidance.
+Incremental Synchronization
+PCE Triggered Initial Synchronization
+PCE Triggered Lsp Resynchronization
+PCE Triggered Full DB Resynchronization
Input
--state-sync-avoidance 10 5 5
--incremental-sync-procedure 10 5 5
--triggered-initial-sync
--triggered-re-sync
+DBVersion number after PCC reconnection
+Seconds to wait until disconnect
+Seconds to wait until PCC reconnects
Change-Id: I61f673177a086d969b6f097733351413b71ea8e2
Signed-off-by: Claudio D. Gasparini <cgaspari@cisco.com>
Claudio D. Gasparini [Fri, 8 Jan 2016 12:02:46 +0000 (13:02 +0100)]
BUG-2208: BUG-2208: Fix minors triggered resync bugs
Fix isTriggeredInitialSyncEnable
Change isSynchronized by method getSynchronized
Change-Id: Icef303f48fb0b192e4d62494e6462aca34e3eea8
Signed-off-by: Claudio D. Gasparini <cgaspari@cisco.com>
Milos Fabian [Mon, 11 Jan 2016 14:07:21 +0000 (15:07 +0100)]
Bug 4836 - tunnel's paths are not removed from ODL after tunnel status "Down" is reported
In a case when PCC reports tunnel has been shutdowned, but not removed
(LSP-ID=0, remove=false), ODL PCE should remove all previously reported
paths related to the tunnel as there are no paths signaled anymore.
Also, in this case PCC will not send report/s with explicit removal
of those paths.
Change-Id: I7b2c34431b094b27ac1fc55c2a75312e00868532
Signed-off-by: Milos Fabian <milfabia@cisco.com>
Milos Fabian [Mon, 4 Jan 2016 08:36:46 +0000 (09:36 +0100)]
Bug 4864 - BMP: No RIB table is created for non-MP peer
Initiate IPv4 Unicast RIB tables by default for all peers.
Fixes case when peer is not supporting
multiprotocol capability.
Change-Id: I8b133cfe621e5ab5c46cc41f604b68854bb4d607
Signed-off-by: Milos Fabian <milfabia@cisco.com>
Thanh Ha [Fri, 15 Jan 2016 04:44:22 +0000 (23:44 -0500)]
Bump versions by 0.1.0 for next dev cycle
Change-Id: Ie87462b78c86280ac0cd3cad758909225d4f57c0
Signed-off-by: Thanh Ha <thanh.ha@linuxfoundation.org>
Claudio D. Gasparini [Tue, 12 Jan 2016 12:25:53 +0000 (13:25 +0100)]
BUG-4714 : No routes from loc-rib are advertised
When new peer connects, already existent routes on EffectiveRibIn
Table are not advertised and stored on corresponding AdjRibOut
Peer table. Therefore AdjRiboutListener doesnt advertise
these routes.
Fix by apply routes, when supported tables are created.
Change-Id: Id263723ee685bbc61468e1ea532f0f57f1c6c6c7
Signed-off-by: Claudio D. Gasparini <cgaspari@cisco.com>
Claudio D. Gasparini [Sun, 10 Jan 2016 12:38:32 +0000 (13:38 +0100)]
BUG-4939: Remove BGP Peer structure from DS once session is close
When session is close remove whole Peer Structure from DS,
and create it again on Session up.
Change-Id: I7796983a98d8378b7fa80c61a9bed55bf46c836f
Signed-off-by: Claudio D. Gasparini <cgaspari@cisco.com>
Claudio D. Gasparini [Tue, 12 Jan 2016 11:45:51 +0000 (12:45 +0100)]
BUG-4962:LocRibWriter shouldn't announce routes to Application Peer
Fix by skip the step when Peer is Internal Role
Change-Id: I8c7932e9cc092d583c3db7b2ee03d38323499b73
Signed-off-by: Claudio D. Gasparini <cgaspari@cisco.com>
Iveta Halanova [Tue, 12 Jan 2016 12:28:17 +0000 (13:28 +0100)]
BUG-4834: resolve values for new best path state
Change-Id: I44c606aa4998a3e3a898d55e87c2f1978cf596dd
Signed-off-by: Iveta Halanova <ihalanov@cisco.com>
Claudio D. Gasparini [Sun, 10 Jan 2016 12:58:12 +0000 (13:58 +0100)]
BUG-4867: AdjRibOutListener is one per Table
AdjRibOutListener is one per Table, therefore we need to store all
of them for later close.
Change-Id: Ied60bd9046dc21f03cb6d45418f0348109c66ae5
Signed-off-by: Claudio D. Gasparini <cgaspari@cisco.com>
Claudio D. Gasparini [Thu, 7 Jan 2016 14:17:17 +0000 (15:17 +0100)]
BUG-2208: Fix wrong order on method calls
Fix wrong order on method calls which caused empty
stateful proposal
Change-Id: I3935e348a707657fb41eaa58f9cc545abd1aeab4
Signed-off-by: Claudio D. Gasparini <cgaspari@cisco.com>
Claudio D. Gasparini [Fri, 8 Jan 2016 12:51:57 +0000 (13:51 +0100)]
BUG-2208: Add missing return on PCCDispatcherImpl
Change-Id: Ie9ff5d9ce8ab54ed9682ddc9da48eba7d7534cc8
Signed-off-by: Claudio D. Gasparini <cgaspari@cisco.com>
Claudio D. Gasparini [Tue, 5 Jan 2016 15:59:27 +0000 (16:59 +0100)]
BUG-2208: Cancel PCEPSessionNegotiator Timers on session down
On session goes down PCEPSessionNegotiator Timers need to
be canceled
Change-Id: I82091efb6e8ffcda992f08bc3fdcbf1cf63a1ab5
Signed-off-by: Claudio D. Gasparini <cgaspari@cisco.com>
Claudio D. Gasparini [Fri, 8 Jan 2016 09:41:10 +0000 (10:41 +0100)]
BUG-2208: leaf triggered-initial-sync should be by default true
leaf triggered-initial-sync should be by default true
Change-Id: If0ccdd658805b8ae35999ceb270641c7d20a604d
Signed-off-by: Claudio D. Gasparini <cgaspari@cisco.com>
Milos Fabian [Mon, 4 Jan 2016 08:32:10 +0000 (09:32 +0100)]
Bug 4863 - BMP: PeerUp Notification serializer bugs
-fix NPE for optional Information TLV
-fix Sent/Received Open Message encoding order
Change-Id: I9de38b02b0b105b4828aaa140fa5f5861fa5d56e
Signed-off-by: Milos Fabian <milfabia@cisco.com>
Iveta Halanova [Fri, 18 Dec 2015 10:00:16 +0000 (11:00 +0100)]
BUG-4802: LS-SR enabling multiple SIDs
specifically: AdjSIDs, Peer-Set-SIDs, BindingSIDs
Change-Id: Ic75bcd6cec20ce70d8351a1f7891e3630369ae76
Signed-off-by: Iveta Halanova <ihalanov@cisco.com>
Claudio D. Gasparini [Wed, 30 Dec 2015 15:32:16 +0000 (16:32 +0100)]
BUG-4867: AdjRibOutListener still listening changes after peer Session closed
Once Peer session is closed. Listener needs to be removed.
Change-Id: Ie3575bdd42e9b0924e48f110652d0c595b42804c
Signed-off-by: Claudio D. Gasparini <cgaspari@cisco.com>
Claudio D. Gasparini [Mon, 4 Jan 2016 10:48:06 +0000 (11:48 +0100)]
Remove deprecated Qname.cachedReference method
Remove deprecated Qname.cachedReference method
Change-Id: I401b75a2a583b9fd048dba6ac7698dd84c47f915
Signed-off-by: Claudio D. Gasparini <cgaspari@cisco.com>
Claudio D. Gasparini [Wed, 30 Dec 2015 22:55:51 +0000 (23:55 +0100)]
Add a private constructor to hide the implicit public one
Add a private constructor to hide the implicit public one on
OpenConfigComparatorFactory.
Change-Id: Id24e1b3bb8c59e0eaed71093acd4443e75ac50db
Signed-off-by: Claudio D. Gasparini <cgaspari@cisco.com>
Claudio D. Gasparini [Wed, 30 Dec 2015 22:48:55 +0000 (23:48 +0100)]
Rename Label package name to match the regular expression
Change-Id: I31996acbcca5b3e247c977a3615f2b4050670488
Signed-off-by: Claudio D. Gasparini <cgaspari@cisco.com>
Robert Varga [Sat, 2 Jan 2016 11:43:03 +0000 (12:43 +0100)]
Use StandardCharsets.UTF_8
Do not look up the charset, but use a global shared instance.
Change-Id: I43ae1e93c0a13675585e69cf32bbf46d80c1ebfa
Signed-off-by: Robert Varga <robert.varga@pantheon.sk>
Milos Fabian [Thu, 17 Dec 2015 15:07:53 +0000 (16:07 +0100)]
BUG-4729: PCEP RP object priority can be set 0
Fix RP object priority parser - if priority is 0,
priority is not present.
Change-Id: I86a9ffb652763524440e9f1599b468bbf27704db
Signed-off-by: Milos Fabian <milfabia@cisco.com>
Milos Fabian [Fri, 18 Dec 2015 11:45:47 +0000 (12:45 +0100)]
Bug 4801 - FromExternalImportPolicyTest is failing at JDK7
Enable the test.
The failure was caused by:
https://git.opendaylight.org/gerrit/#/c/31365/
The change was reverted:
https://git.opendaylight.org/gerrit/#/c/31511/
Change-Id: I70b130b15ec57fc696a3af02b9dfc7f8f9e93ae7
Signed-off-by: Milos Fabian <milfabia@cisco.com>
Iveta Halanova [Mon, 23 Nov 2015 12:43:14 +0000 (13:43 +0100)]
Unit tests for openconfig-impl/moduleconfig
Change-Id: I179021c1214f127151defcc77cd2a6eaec6ab251
Signed-off-by: Iveta Halanova <ihalanov@cisco.com>
Milos Fabian [Wed, 16 Dec 2015 12:32:22 +0000 (13:32 +0100)]
Bug 4781 - BGP at work is not ready for application of 43-bgp-openconfig.xml
Change starting order - first BGP OpenConfig app is stared, then RIBImpl
is started.
The change avoids problem with reconfiguring RIBImpl instance at startup.
Change-Id: I8777bb0175b03bd4450076b6e916fc1d015d1394
Signed-off-by: Milos Fabian <milfabia@cisco.com>
Claudio D. Gasparini [Wed, 16 Dec 2015 08:55:16 +0000 (09:55 +0100)]
BUG-4780: New peers failed to connect after one gets connected
After one peer gets connected, new peer failed to connect.
Cause of the issue was that we were reusing DefaultPromise
from previous connection.
Fix by creating new promise for each connection.
Remove needless EventExecutor variables, use
GlobalEventExecutor.INSTANCE instead.
Change-Id: I2938b9f16087d5b004fdbd43bcf082e26bb8a071
Signed-off-by: Claudio D. Gasparini <cgaspari@cisco.com>
Iveta Halanova [Tue, 15 Dec 2015 13:51:57 +0000 (14:51 +0100)]
BUG-4740: No reconnect after negotiation failure
Change-Id: I31a152bcfc52f98a9e95a8a75b26d42d2da8a014
Signed-off-by: Iveta Halanova <ihalanov@cisco.com>
Milos Fabian [Thu, 17 Dec 2015 01:28:43 +0000 (02:28 +0100)]
BUG-4779: Openconfig mandatory descendant missing
Make sure mandatory leaf is allways present.
Map RIB module configuration on OpenConfig Bgp top level container.
Change-Id: I5f28a5fa2924ec2ddc721c69f0b2a67fdf374f63
Signed-off-by: Milos Fabian <milfabia@cisco.com>
Milos Fabian [Wed, 16 Dec 2015 17:18:17 +0000 (18:18 +0100)]
BUG-4792: Fix failing single feature tests
Use "odl-netty" feature from odlparent.
Ignore JUnit test FromExternalImportPolicyTest.testEffectiveAttributes
Will be fixed in additional patch.
Change-Id: I7baa260325cbf4fe6d317d71315e091fcf1d7a6b
Signed-off-by: Milos Fabian <milfabia@cisco.com>
Milos Fabian [Tue, 1 Dec 2015 09:08:23 +0000 (10:08 +0100)]
BUG-4696: BGP OpenConfig bugfixing
-introduce comparators, used when comparing current configuration state
with proposed state
-pass whole configuration (not only identifier) when removing
config module instance
-use MountPointListener instead of DataTreeChangeListener
-refactor implementations
Change-Id: I388dcd6ccb094a82d922740b86f721b5baf23e18
Signed-off-by: Milos Fabian <milfabia@cisco.com>
Milos Fabian [Fri, 11 Dec 2015 11:32:12 +0000 (12:32 +0100)]
BUG-4730: Update BGP-LS EPE
Update implementation to conform draft-ietf-idr-bgpls-segment-routing-epe-01
which is reflecting changes in latest linkstate segment routing extension draft.
-TLVs redefined with common structure
-changed TLVs codepoints
API (Yang data model) change:
renamed container "peer-sid" -> "peer-node-sid"
added container "peer-adj-sid"
Change-Id: I391862970fd751f54426e7222eb99519a68b4d99
Signed-off-by: Milos Fabian <milfabia@cisco.com>
Milos Fabian [Wed, 9 Dec 2015 00:30:56 +0000 (01:30 +0100)]
BUG-4689: Advertising performance fix
The TCP channel become unwritable too fast and too often, hence
use high & low write watermark option for channel.
Change-Id: Icd52f164153dde94cd7c61a0c38f805a5c0593cf
Signed-off-by: Milos Fabian <milfabia@cisco.com>
Iveta Halanova [Fri, 4 Dec 2015 09:31:42 +0000 (10:31 +0100)]
Sonar warning fix: IPv4RIBSupport
attribute name refactor
Change-Id: I89e58a8a1b4120b44085983ccf724f94b0370c27
Signed-off-by: Iveta Halanova <ihalanov@cisco.com>
Iveta Halanova [Wed, 2 Dec 2015 12:55:22 +0000 (13:55 +0100)]
Test coverage: rsvp-spi/subobjects
Change-Id: I29400140d6e5cb5846473896a20151e3a57fbdc5
Signed-off-by: Iveta Halanova <ihalanov@cisco.com>
Iveta Halanova [Wed, 2 Dec 2015 19:24:04 +0000 (20:24 +0100)]
BUG-4705: LS-SR MplsLabel as 20 rightmost bits
Change-Id: Ia8c47d4f9b16785b911025994c5b56cd6e7b78d9
Signed-off-by: Iveta Halanova <ihalanov@cisco.com>
Claudio D. Gasparini [Wed, 2 Dec 2015 15:43:49 +0000 (16:43 +0100)]
Refactor classes names
Refactor classes name to have same stetic in capitals
all, like BGP and PCEP.
Change-Id: I58fb5a3524e2628a4b7555621e34b2d1f3c3c362
Signed-off-by: Claudio D. Gasparini <cgaspari@cisco.com>
Iveta Halanova [Mon, 23 Nov 2015 07:02:56 +0000 (08:02 +0100)]
Unit Test: BGPOpenConfig
Change-Id: I501cc7c2ecde00e00d68535aa4b05ae1f3bbb52b
Signed-off-by: Iveta Halanova <ihalanov@cisco.com>
Iveta Halanova [Mon, 30 Nov 2015 14:28:53 +0000 (15:28 +0100)]
Test coverage: rsvp-spi
Change-Id: I4cdcecd0610c667e3d1f4bbe3e4871b77c869c7f
Signed-off-by: Iveta Halanova <ihalanov@cisco.com>
Thanh Ha [Tue, 1 Sep 2015 23:09:15 +0000 (19:09 -0400)]
Use odlparent jacoco configuration
Change-Id: I2948f02064ad38fa018b7ae05068a65d3a961ae7
Signed-off-by: Thanh Ha <thanh.ha@linuxfoundation.org>
Filip.Gregor [Fri, 27 Nov 2015 09:18:17 +0000 (10:18 +0100)]
Bug 4646: Fixed up duplicate use of reference statements.
minor fix of invalid yang files
Change-Id: Ic8dbcee91e9be2c78813291dd95239d3ca5895b6
Signed-off-by: Filip.Gregor <fgregor@cisco.com>
Thanh Ha [Sun, 29 Nov 2015 21:45:55 +0000 (16:45 -0500)]
Use odlparent-lite as artifacts parent
We created odlparent-lite to provide a minimal pom which allows projects
to publish to the correct server. Without this ${project}-artifacts may
fail to deploy to Nexus.
Change-Id: Icff4d53b0af400ecae86c6099ebba00ec0509c72
Signed-off-by: Thanh Ha <thanh.ha@linuxfoundation.org>
Robert Varga [Fri, 27 Nov 2015 11:27:57 +0000 (12:27 +0100)]
Eradicate use of commons-codec
Guava provides hex-codec, use that instead of commons-codec. Also turn
some methods into statics while we're in the area.
Change-Id: I9e950277995f78bfe6323b11b41ef840e427726a
Signed-off-by: Robert Varga <rovarga@cisco.com>
Thanh Ha [Fri, 27 Nov 2015 06:39:34 +0000 (01:39 -0500)]
Remove antlr4-runtime-osgi-nohead
This bundle was removed from yangtools so no longer exists. See:
https://git.opendaylight.org/gerrit/28272/
Change-Id: I7094edc12c3f12ad6ef8aee0e4c2cfaf98af7263
Signed-off-by: Thanh Ha <thanh.ha@linuxfoundation.org>
Milos Fabian [Wed, 25 Nov 2015 14:06:47 +0000 (15:06 +0100)]
BUG-4672: BMP client reconnection not working
-get rid of protocol-framework's reconnect strategy
-introduce simple initial reconnection as suggested in
https://tools.ietf.org/html/draft-ietf-grow-bmp-15#section-3.2
-BMP session should go to "up" state once the connection is established (channel is active),
introduced "IDLE" state of a BMP session
-added commented sample of active client (monired router) configuration
Introduces "IDLE" state of a BMP session,
session goes up once the channel is active.
Change-Id: Ibd78fe708162a710fdcc20c614cc1d9c1be6eb95
Signed-off-by: Milos Fabian <milfabia@cisco.com>
Milos Fabian [Thu, 26 Nov 2015 11:10:56 +0000 (12:10 +0100)]
BUG-3919: Support for non-MP advertisement
Add a support for old-school BGP-4 advertising,
if peer do not advertise IPv4-Unicast MP capability.
Revert changes in IPv4RIBSupport class.
Change-Id: I0f3a4c767f4f7048e4867ec3aab6c2b8dac6c008
Signed-off-by: Milos Fabian <milfabia@cisco.com>
Claudio D. Gasparini [Thu, 26 Nov 2015 15:23:31 +0000 (16:23 +0100)]
BUG-4569: Data-change-counter reconfiguration failure
Data-change-counter reconfiguration fails because of
race condition. Last task which removes model from
DS is not executed before new data-change-counter is
created and insert his own model on DS.
Fix by wait until last task is executed when close module.
Change-Id: Ic9349bf99d6a96fd44babdb874158138d7eaf6e8
Signed-off-by: Claudio D. Gasparini <cgaspari@cisco.com>
Milos Fabian [Thu, 26 Nov 2015 09:36:48 +0000 (10:36 +0100)]
BUG-4679: NPE in IPv4 prefix withdrawn Update msg serialization
When building Update message for withdrawing routes, check if
Multi-protocol is supported for this type of route.
If not, pack withdrawing routes as per RFC4271.
Change-Id: Ia9a36add74dd817bf0a460cfeaa5281f9b2fb50a
Signed-off-by: Milos Fabian <milfabia@cisco.com>
Claudio D. Gasparini [Wed, 25 Nov 2015 12:01:03 +0000 (13:01 +0100)]
Refactor Tunnel-Programming
Refactor Tunnel-programming and create new
TunnelInstrucctionExecutor classes.
Change-Id: I0be6fd4f5a050a9b81f8f7bed01b3eabb848fd25
Signed-off-by: Claudio D. Gasparini <cgaspari@cisco.com>
Iveta Halanova [Wed, 25 Nov 2015 10:50:55 +0000 (11:50 +0100)]
Remove redundant modifier
Change-Id: Ibfc7b03b4e73fd4d329af0b690967ce38772a2c9
Signed-off-by: Iveta Halanova <ihalanov@cisco.com>
Iveta Halanova [Wed, 25 Nov 2015 10:44:25 +0000 (11:44 +0100)]
BGPRibImplProvider sonar fix
Change-Id: I9c3101ba41b5cadadbbc1d1fc24b1b19c67296cb
Signed-off-by: Iveta Halanova <ihalanov@cisco.com>
Iveta Halanova [Wed, 25 Nov 2015 10:37:47 +0000 (11:37 +0100)]
OpenConfigUtil sonar fix
Change-Id: I16c4abaca3cd57e492695ae896fb54271b17db93
Signed-off-by: Iveta Halanova <ihalanov@cisco.com>
Iveta Halanova [Wed, 25 Nov 2015 10:28:37 +0000 (11:28 +0100)]
Rev130715Util minor fixes
misspelled Uitl -> Util
added private constructor
Change-Id: I949cceb518cdaa4a66b9b725d5974f09610b15f6
Signed-off-by: Iveta Halanova <ihalanov@cisco.com>
Iveta Halanova [Thu, 19 Nov 2015 14:27:31 +0000 (15:27 +0100)]
SrRangeTlv: inner area parsing/serializing
Inner area should be handled just in case of OSPF protocol.
Change-Id: I8468dfe0aeff6ed6c4ee7945f489b22bf8874075
Signed-off-by: Iveta Halanova <ihalanov@cisco.com>
Milos Fabian [Wed, 18 Nov 2015 14:00:33 +0000 (15:00 +0100)]
BUG-4617: BGP Flow-Spec Redirect to IP Action
-adds support for BGP FS Redirect to IP action
-implementation is based on draft-ietf-idr-flowspec-redirect-ip-02
-registered under Ex. comm type/subtype as per
https://tools.ietf.org/html/draft-ietf-idr-flowspec-redirect-ip-00#section-7
Change-Id: If889e1b65d944c71b85e0d27c57af4be7e08af6c
Signed-off-by: Milos Fabian <milfabia@cisco.com>
Claudio D. Gasparini [Wed, 18 Nov 2015 15:38:50 +0000 (16:38 +0100)]
Move releaseConnection from ReusableBGPPeer to BGPSessionListener
Move releaseConnection from ReusableBGPPeer to BGPSessionListener
Change-Id: Ief296eccc81cc00661462cc9ec861ff4acc379de
Signed-off-by: Claudio D. Gasparini <cgaspari@cisco.com>
Claudio D. Gasparini [Fri, 20 Nov 2015 13:25:52 +0000 (14:25 +0100)]
Refactor LinkstateNlriParser
Refactor LinkstateNlriParser to fix Sonar complains
Change-Id: If4f523bc7fe338f370b809df68c952ceef8e403f
Signed-off-by: Claudio D. Gasparini <cgaspari@cisco.com>
Claudio D. Gasparini [Fri, 20 Nov 2015 08:09:28 +0000 (09:09 +0100)]
Implement changedRoutes Test
Implement changedRoutes Test
Change-Id: I2026338225959325efefd9501248a0fa01cc6335
Signed-off-by: Claudio D. Gasparini <cgaspari@cisco.com>
Claudio D. Gasparini [Mon, 16 Nov 2015 13:55:05 +0000 (14:55 +0100)]
Remove unnecesary dependency
Remove unnecesary yang-ext dependency
Change-Id: I9a0434a756326ebe5d670d346a01b01648cb5c60
Signed-off-by: Claudio D. Gasparini <cgaspari@cisco.com>
Claudio D. Gasparini [Mon, 16 Nov 2015 13:40:06 +0000 (14:40 +0100)]
Remove workaround for bug 1276
Since bug 1276 is fixed, workaround is not required
anymore.
Change-Id: I5939f3ac94ab3b3f865c3c2920e4f491f39f9827
Signed-off-by: Claudio D. Gasparini <cgaspari@cisco.com>
Claudio D. Gasparini [Fri, 20 Nov 2015 13:49:02 +0000 (14:49 +0100)]
Refactor Stateful07TopologySessionListener
Refactor Stateful07TopologySessionListener to fix Sonar
complains.
Change-Id: Ib9548697baa6e405cadc1013857388df1379b99b
Signed-off-by: Claudio D. Gasparini <cgaspari@cisco.com>
Claudio D. Gasparini [Wed, 18 Nov 2015 15:18:28 +0000 (16:18 +0100)]
Create AbstractEROPathKeySubobjectParser
Create AbstractEROPathKeySubobjectParser for PCEP
Change-Id: Ifeb84955f2002d78871705c0e7091ac3df31e894
Signed-off-by: Claudio D. Gasparini <cgaspari@cisco.com>
Milos Fabian [Wed, 18 Nov 2015 10:32:58 +0000 (11:32 +0100)]
BUG-4616: Update Flowspec Redirect Ext. Communities
-implementation of handlers for BGP flowspec
Ipv4 and AS4 Redirect extended communities
ref.: http://tools.ietf.org/html/rfc7674
Change-Id: Ia4da00db3b62633206e643654e64a3faf5b76aa2
Signed-off-by: Milos Fabian <milfabia@cisco.com>
Milos Fabian [Wed, 18 Nov 2015 10:43:36 +0000 (11:43 +0100)]
Fix extended community parser key generator
Change-Id: Ie311b0f1a03b1475313f4c657ffa6d9147965ca9
Signed-off-by: Milos Fabian <milfabia@cisco.com>
Claudio D. Gasparini [Fri, 20 Nov 2015 14:06:37 +0000 (15:06 +0100)]
Refactor EffectiveRibInWriter
Refactor EffectiveRibInWriter to fix Sonar Complains
Change-Id: Ie5b5892b26f045cf683ca5af56307cad3de683ef
Signed-off-by: Claudio D. Gasparini <cgaspari@cisco.com>
Iveta Halanova [Thu, 19 Nov 2015 13:39:36 +0000 (14:39 +0100)]
BUG-4618: FS Redirect ipv6 EC handler
Change-Id: I460fb793c2b0cbf09e954214c8142e9bbfcb8666
Signed-off-by: Iveta Halanova <ihalanov@cisco.com>
Claudio D. Gasparini [Mon, 16 Nov 2015 14:33:04 +0000 (15:33 +0100)]
create AbstractNumericByteOperandParser
create AbstractNumericByteOperandParser and remove duplicate code.
Change-Id: Ibb58d848dd7f92780e824a867b680cb550b38107
Signed-off-by: Claudio D. Gasparini <cgaspari@cisco.com>
Iveta Halanova [Thu, 19 Nov 2015 08:03:22 +0000 (09:03 +0100)]
Test coverage: openconfig-impl
Change-Id: Ie1de0875a1f4fbe1aab3c602a5a75b78b3654d08
Signed-off-by: Iveta Halanova <ihalanov@cisco.com>
Claudio D. Gasparini [Fri, 13 Nov 2015 16:48:06 +0000 (17:48 +0100)]
BUG-4633: Use ietf-inet-type rev.
20130715 on bgp-openconfig
Add mssing dependencies.
Add missing dependencies required by openconfig
Reconfigure openconfig to use ietf-inet-type
revision
20130715.
Change-Id: I4c5f6960c1b1e5f364f0a4f9d9cf347677fa72cb
Signed-off-by: Claudio D. Gasparini <cgaspari@cisco.com>
Claudio D. Gasparini [Mon, 16 Nov 2015 14:07:21 +0000 (15:07 +0100)]
Add path-binding-tlv missing reference model
Add path-binding-tlv missing reference model
Change-Id: I14df2a1c1e21266d1b820729add1660e8024de49
Signed-off-by: Claudio D. Gasparini <cgaspari@cisco.com>
Claudio D. Gasparini [Mon, 26 Oct 2015 12:06:00 +0000 (13:06 +0100)]
BmpMonitorImplTest fails
Test BmpMonitorImplTest sometimes fails
because port still not free from previous test.
As a solution we set the Socket as reusable.
Change-Id: Idb4ecc50f43e2be57d5b11a7c72c09fd8e2de51d
Signed-off-by: Claudio D. Gasparini <cgaspari@cisco.com>
Claudio D. Gasparini [Mon, 16 Nov 2015 11:30:36 +0000 (12:30 +0100)]
BUG-4634: Prefix in withdrawn list is not ignored when announced in same msgUpd.
An UPDATE message SHOULD NOT include the same address prefix in the
WITHDRAWN ROUTES and Network Layer Reachability Information fields.
However, a BGP speaker MUST be able to process UPDATE messages in
this form. A BGP speaker SHOULD treat an UPDATE message of this form
as though the WITHDRAWN ROUTES do not contain the address prefix.
Fix by ingore common NLRI on withdrawn router
Change-Id: Ic9ebd3ce13f042f0dfcd2b28815442f8484add16
Signed-off-by: Claudio D. Gasparini <cgaspari@cisco.com>
Claudio D. Gasparini [Wed, 11 Nov 2015 12:23:46 +0000 (13:23 +0100)]
BUG-4622: Sending Empty Optional Attributes
Some of the parsers of Optional Attributes
only take in account the possibility of null
attribute and not of an empty one.
Fix by check it and dont parse them in case of
empty or null.
Change-Id: I34d801a001ed167e18646e228b9b9dfc03f7c882
Signed-off-by: Claudio D. Gasparini <cgaspari@cisco.com>
Robert Varga [Fri, 13 Nov 2015 09:41:16 +0000 (10:41 +0100)]
Remove unused dependency on yang-ext
Since the augmentation order is predictable now, do not depend on unused
yang-ext bundle.
Change-Id: I7e511a2f43f3f4e430ac6746a46ecefb13f10174
Signed-off-by: Robert Varga <rovarga@cisco.com>
Iveta Halanova [Wed, 4 Nov 2015 13:46:50 +0000 (14:46 +0100)]
Sonar warnings fix
Complexity issue - extracted switch into private method.
Renaming attributes and variables.
Removed unused attributes.
And other minor refactoring.
Change-Id: I335cb6ccce85b55e30c9537fe39f3dfb7754cc39
Signed-off-by: Iveta Halanova <ihalanov@cisco.com>
Robert Varga [Wed, 11 Nov 2015 10:13:08 +0000 (11:13 +0100)]
BUG-4598: upgrade openconfig models
bgp.yang conflicts with another model in ODL. As it turns out,
this has already been fixed in OpenConfig 1.1.0 release. Update the
models to resolve the conflict.
Change-Id: Iede4e57116b81f5f8d835b60958ae3d87b59d897
Signed-off-by: Claudio D. Gasparini <cgaspari@cisco.com>
Signed-off-by: Robert Varga <rovarga@cisco.com>
Signed-off-by: Milos Fabian <milfabia@cisco.com>
Iveta Halanova [Thu, 12 Nov 2015 12:15:54 +0000 (13:15 +0100)]
BUG-4593: BGP peer registry lookup false negative result
Added method into Ipv6Utill for creating Ipv6Address with uncompressed value.
Used when registring and deleting peer address.
Change-Id: Ifa96a3ec2f9f7cc3ee60329e888590647af40f64
Signed-off-by: Iveta Halanova <ihalanov@cisco.com>
(cherry picked from commit
0e5baad314c8374cc1d0dd4bcce9b2845d32da24)
Iveta Halanova [Wed, 11 Nov 2015 10:44:53 +0000 (11:44 +0100)]
BUG-4618: FS Redirect-IPv6 Extended Community
updated FS model for redirect-ipv6 ext.com and added to augmentations.
ref.: https://tools.ietf.org/html/draft-ietf-idr-flow-spec-v6-06#section-4
Change-Id: I65acbc4d47f9f0fe2375a8bcb920e1af83f08010
Signed-off-by: Iveta Halanova <ihalanov@cisco.com>
Claudio D. Gasparini [Wed, 11 Nov 2015 09:46:12 +0000 (10:46 +0100)]
BUG-3771: Use sync on FutureChannels intead of await
"Await" waits until future has finished and throw execution
if thread was interrupted."Sync"" instead waits for this
future until it is done, and re throws the cause of
the failure if this future failed.
When closing the module, first we ensure that channel
has been closed and if it didn't fail we proceed
with next closures.
Change-Id: I1240d6fdb6986b8c30251206d8027db7b1a987fa
Signed-off-by: Claudio D. Gasparini <cgaspari@cisco.com>
Milos Fabian [Wed, 11 Nov 2015 02:03:52 +0000 (03:03 +0100)]
BUG-4617: BGP-FS redirect to next-hop IP action
-added yang data model for "redirect to IP extended communities"
ref.: https://tools.ietf.org/html/draft-ietf-idr-flowspec-redirect-ip-02#section-3
Change-Id: I80e74b884e87e2fdbcc0a9feb0ed5f873f8abbb7
Signed-off-by: Milos Fabian <milfabia@cisco.com>
Milos Fabian [Wed, 11 Nov 2015 01:51:19 +0000 (02:51 +0100)]
BUG-4616: Update BGP-FS Redirect extended Communities
-added yang data model for redirect AS-4Byte
-added yang data model for redirect IPv4
ref.: https://tools.ietf.org/html/rfc7674
Change-Id: I4456f8bd7789f82418e852be74c010b1ee7038ee
Signed-off-by: Milos Fabian <milfabia@cisco.com>
Milos Fabian [Thu, 5 Nov 2015 09:16:58 +0000 (10:16 +0100)]
BUG-4568: Introduce BGP Extended Community registry
-new Extended Community registry
-parsers registration is based on type/subtype
-serializers registration is based on "extened community case"
-added new "transitive" field for extened communities
-other "transitive" fileds are deprecated in yang data model
-deprecated type/sub-type value from yang data model:
-values are not filled in during a parsing
-values are ignored during a serialization
-make the extended commnity attribute parser non-extensible:
-BGP path attribute serializers are registred in
a specific order, an extending of the extended community
handler breaks the contract
-added missing IPv4 Transitive/non-transitive Route Target/Route Origin
extened communities
-updated Flowspec extended communities
Change-Id: I2818b20c069da362f60a716862421bb185bc8571
Signed-off-by: Milos Fabian <milfabia@cisco.com>
Claudio D. Gasparini [Mon, 9 Nov 2015 11:07:28 +0000 (12:07 +0100)]
BUG-4594: ISIS PseudoNode NPE
NPE caused beacuse of wrong container.
Fix by: add missing is-is-router Container
which contains ISO System ID. Affecting
isis-node-case and isis-pseudonode-case.
Change-Id: I256c36ea980326bd39db4e8189ef092e20491c54
Signed-off-by: Claudio D. Gasparini <cgaspari@cisco.com>
Claudio D. Gasparini [Thu, 5 Nov 2015 15:20:10 +0000 (16:20 +0100)]
Remove gracefullyShutdown of EventLoopGroops
When instance is closed, boosgroud and workergroop
shouldnt be shutdown since they are required in case
of new instance created. If new instance is created
it will cause
RejectedExecutionException: event executor terminated
Change-Id: I58f9328140bc209aeafc7bfad14f883887447a59
Signed-off-by: Claudio D. Gasparini <cgaspari@cisco.com>
Claudio D. Gasparini [Thu, 5 Nov 2015 15:03:17 +0000 (16:03 +0100)]
Keep consistency under Optional parssword
Change-Id: Ib0f5901237ff370f7f9a6f143f1d9e8ec59405e9
Signed-off-by: Claudio D. Gasparini <cgaspari@cisco.com>
Vratko Polak [Fri, 6 Nov 2015 15:05:11 +0000 (16:05 +0100)]
Add openconfig-config to BGP feature dependencies
SingleFeatureTests in distribution jobs are failing.
Change-Id: Id6b6a55eb87519c27f63f6149f8dc505033c102f
Signed-off-by: Vratko Polak <vrpolak@cisco.com>
Milos Fabian [Wed, 23 Sep 2015 10:09:21 +0000 (12:09 +0200)]
BUG-4197: BGP OpenConfig Implementation
-BGP OpenConfig for ODL BGP RIB and peer/app-peer configuration (config-subsystem) and vice-versa
*RIBImpl <-> Global
*BGPPeer <-> Neighbor
*BGPApplicationPeer <-> Neighbor (belonging to special neighbors grupup "application-peers")
-support BGP configuration module to BGP OpenConfig mapping:
*once configuration of a particular BGP config module is created, the attributes
are mapped to OpenConfig API, configuration is stored and written do configuration DS
*once configuration of a particular BGP Configuration module is closed, a stored OpenConfig DTO
is removed and deleted from configuration DS
-support BGP OpenConfig API to BGP configuration module mapping
-application is listening to the changes in BGP OpenConfig DS subtree:
*when a new data are created or modified, the new BGP OpenConfig DTO is mapped to
BGP config module, the DTO is stored and config module is written to
controller-config's Modules subtree.
*when data is removed, the DTO is removed from inner store and BGP config module is removed
from controller-config's Modules subtree.
Motivation: Simplify configuration of the main ODL BGP components (mainly in programatic way) in a most used use-cases.
Limitations: BGP OpenConfig Yang data model is composed for configuration of one BGP speaker, ODL may have multiple instances of BGP Speaker
The change brings a dependecy on netconf's sal-nectonf-connector ot BGP project.
Change-Id: If245bd54d844bd9965637c83f382e82c6a5b9a53
Signed-off-by: Milos Fabian <milfabia@cisco.com>
Claudio D. Gasparini [Tue, 3 Nov 2015 06:36:45 +0000 (07:36 +0100)]
Remove duplicate class
Remove duplicate and unnecesaries classes
Change-Id: I5835cb41ef3dfd881478b5ed93930c9cc87cf066
Signed-off-by: Claudio D. Gasparini <cgaspari@cisco.com>
Iveta Halanova [Mon, 2 Nov 2015 11:26:19 +0000 (12:26 +0100)]
BUG-4552: Enhance LS-SR - discern protocols
Updated yang model and decoding methods.
Corrected and extended unit tests.
Change-Id: I4cb8e7ba18ebce2962d4a5155e910f1292d57174
Signed-off-by: Iveta Halanova <ihalanov@cisco.com>
Iveta Halanova [Thu, 29 Oct 2015 10:59:13 +0000 (11:59 +0100)]
Utilizing MplsLabel
Applied in BGP-LU and BGP-LS SR
Change-Id: I0736a96a71a2aca06bf9645df4fdf9b3afa967b9
Signed-off-by: Iveta Halanova <ihalanov@cisco.com>
Iveta Halanova [Mon, 2 Nov 2015 14:48:48 +0000 (15:48 +0100)]
Sonar minor issues fix: modifiers
Change-Id: Idc6504dc7e95f64452a9c3e11f60690bbec36168
Signed-off-by: Iveta Halanova <ihalanov@cisco.com>
Claudio D. Gasparini [Fri, 30 Oct 2015 14:01:01 +0000 (15:01 +0100)]
BGPCEP MD5 Password Fix
Password were checked in case if null
but not in case of empty.
Fix by creating an Optional from password where in
case of null or empty will be absent.
Change-Id: I67033deeccd5f634a73d941d0c3da23e8dfb644d
Signed-off-by: Claudio D. Gasparini <cgaspari@cisco.com>
Iveta Halanova [Fri, 30 Oct 2015 14:47:14 +0000 (15:47 +0100)]
Sonar issues fix: LS, FS
added final modifiers, solved complexity by extracting switches
to private methods, removed not used attributes.
Change-Id: Iab11adafac0b2519a5d07b26d638e8265ac6d0ff
Signed-off-by: Iveta Halanova <ihalanov@cisco.com>
Milos Fabian [Fri, 30 Oct 2015 10:43:07 +0000 (11:43 +0100)]
BUG-4557: Fix BMP initial config files
-reconnecting strategy belongs to dispacther configuration
-add expected protocol framework capability in bmp's "features.xml"
Change-Id: Icde460db308d6882e742456df3829d9fb7cac683
Signed-off-by: Milos Fabian <milfabia@cisco.com>
Milos Fabian [Fri, 30 Oct 2015 08:43:40 +0000 (09:43 +0100)]
Bug-4534: Fix yang model
remove an unexpected comma causing "line 124:13 extraneous input ',' expecting {'}', SEP, IDENTIFIER}"
Change-Id: Ia3dc19bdf95e3450b89bdcc00f33ad6898a55b67
Signed-off-by: Milos Fabian <milfabia@cisco.com>
Claudio D. Gasparini [Wed, 28 Oct 2015 14:57:49 +0000 (15:57 +0100)]
BUG-4545: PCEP SR- ERO/RRO codepoint configurable
Since we dont have IANA stablished code point for
SR- ERO/RRO subobjects, and there are new one
proposed by draft (ref) we made the code point
configurable, setting by default previouly used.
ref:https://tools.ietf.org/html/draft-ietf-pce-segment-routing-06
Change-Id: I2055f1cd6644ad1506cb6d2ceb05bcaf9f794dd4
Signed-off-by: Claudio D. Gasparini <cgaspari@cisco.com>
Iveta Halanova [Thu, 29 Oct 2015 15:04:16 +0000 (16:04 +0100)]
BUG-4447: BGP-LS SR extention
applied in LS YANG model
updated attributes parsers of types: link, node, prefix
unit tests correction
Change-Id: Ie6624360ce0947ac4f5523386c1700bb41f11565
Signed-off-by: Iveta Halanova <ihalanov@cisco.com>
Milos Fabian [Wed, 28 Oct 2015 10:42:18 +0000 (11:42 +0100)]
BUG-4197: use yang-ext's augment-identifier
-make sure that generated java APIs for augmentation have
always the same name.
Change-Id: I859460ca6265a5e52148af22d45d638b4d54030f
Signed-off-by: Milos Fabian <milfabia@cisco.com>
Jeff Liu [Mon, 3 Aug 2015 23:54:06 +0000 (16:54 -0700)]
make bmp session with reconnect strategy
Change-Id: Ia4a0bc455b0d920c13eb9d4e1804ff7d98e79b7e
Signed-off-by: Jeff Liu <bigdataops@gmail.com>
Milos Fabian [Sat, 3 Oct 2015 00:18:28 +0000 (02:18 +0200)]
BUG-4197: BGP OpenConfig SPI
Defines the APIs to track a BGP configuration modules status changes and
BGP OpenConfig data writers consumed as a service injected by config subsystem into
BGP configuration module.
The goal is to react on changes in the BGP configuration module instances - the configuration
modules are mapped to the OpenConfig BGP data model instances and kept in a datastore.
Change-Id: Ic38d5049d4d7e2f51f05a0e44e28626fc48e7ab7
Signed-off-by: Milos Fabian <milfabia@cisco.com>
Iveta Halanova [Tue, 29 Sep 2015 12:40:23 +0000 (14:40 +0200)]
BUG-4173: flowspec fixes
Fixed extraction of nodes in FSNlriParser.
Fixed model for extended communities and augmentation of bgp-rib route.
Corrected and simplified RIB support destination processing.
Unit test correction.
Change-Id: I3ddd1424703eb1e7f4a3506eeb4c02f641753452
Signed-off-by: Iveta Halanova <ihalanov@cisco.com>