Robert Varga [Mon, 5 May 2014 13:22:45 +0000 (15:22 +0200)]
BUG-731: do not assign to parameters
It is convenient to coerce the argument value, but this approach is a
bit more clean and avoids the sonar warning.
Change-Id: I8648e09c5c02bb082321c3a9a858142eaabdc9ea
Signed-off-by: Robert Varga <rovarga@cisco.com>
Robert Varga [Mon, 5 May 2014 13:19:35 +0000 (15:19 +0200)]
BUG-731: do not throw RuntimeException
Let's turn this into an IllegalArgumentException, with a proper message.
Change-Id: I5bb149de83001ea6817c6f58fb1d5d9be41907e1
Signed-off-by: Robert Varga <rovarga@cisco.com>
Robert Varga [Mon, 5 May 2014 13:16:09 +0000 (15:16 +0200)]
BUG-731: rework catch clausule
This eliminates warning about rethrowing exceptions.
Change-Id: I46a3531a7753c049d78919c0134d714a509b29a6
Signed-off-by: Robert Varga <rovarga@cisco.com>
Robert Varga [Mon, 5 May 2014 12:29:00 +0000 (14:29 +0200)]
BUG-731: move to new APIs
This fixes "Overriding method merely calls super".
Change-Id: I02794a2c2a118cafa847138ad22199a18df4a83c
Signed-off-by: Robert Varga <rovarga@cisco.com>
Robert Varga [Mon, 5 May 2014 12:18:48 +0000 (14:18 +0200)]
BUG-731: fix keyword ordering
Fixes abstract/protected keyword order.
Change-Id: I1d486b1113e0a5dc7f46a6220203e25c806ebd19
Signed-off-by: Robert Varga <rovarga@cisco.com>
Robert Varga [Mon, 5 May 2014 09:47:53 +0000 (11:47 +0200)]
Remove BGPCEP distribution
This distribution is now defunct, so let's remove it.
Change-Id: Ifdfe2d5f19f6f1988ac287f0772a55d794cc1b69
Signed-off-by: Robert Varga <rovarga@cisco.com>
Robert Varga [Mon, 5 May 2014 09:40:56 +0000 (11:40 +0200)]
Add abstract method documentation
This just clarifies the API contract for identifierForKey.
Change-Id: Ia855cc69e36271e317ffc529d0963dc0a314e722
Signed-off-by: Robert Varga <rovarga@cisco.com>
Tony Tkacik [Mon, 5 May 2014 09:04:25 +0000 (11:04 +0200)]
Enabled replace action in PCEP Topology Provider
With legacy compatibility APIs default action of datastore
is merge, enabled replace operation, in order to
assume correct behaviour.
Change-Id: I9302a535e1ea4b4e001e45dae28015dcf45b3362
Signed-off-by: Tony Tkacik <ttkacik@cisco.com>
Robert Varga [Sun, 4 May 2014 19:48:26 +0000 (21:48 +0200)]
BUG-731: JMX attributes are static now
The JMX attribute fields are now public static so let's not use object
reference when accessing them.
Change-Id: I1613188a11d528f06ef30183f1eade1c4d60ab79
Signed-off-by: Robert Varga <rovarga@cisco.com>
Robert Varga [Sun, 4 May 2014 19:34:55 +0000 (21:34 +0200)]
BUG-731: remove implied modifier
All methods in an interface are always public, no need to re-state that.
Change-Id: Ibb1c87ab2f2c0a81636607fbfa95dd109ccdc0d6
Signed-off-by: Robert Varga <rovarga@cisco.com>
Robert Varga [Sun, 4 May 2014 19:33:19 +0000 (21:33 +0200)]
BUG-731: remove unused imports
These imports are only used in javadoc -- so let's not do that and
specify the name in full in the javadoc.
Change-Id: I90ebe8da4ff84ca7620d1713630899492a3b09fa
Signed-off-by: Robert Varga <rovarga@cisco.com>
Robert Varga [Sun, 4 May 2014 19:29:47 +0000 (21:29 +0200)]
BUG-731: utility classes should be final
There's no reason to subclass these -- they cannot be instantiated,
anyway.
Change-Id: I6e07c8eba054fb9d6ad5a6c5ee1e12a6c83acd7d
Signed-off-by: Robert Varga <rovarga@cisco.com>
Robert Varga [Sun, 4 May 2014 19:27:09 +0000 (21:27 +0200)]
BUG-731: Raw RuntimeExceptions should not be thrown
They really tell us very little -- so convert them to
IllegalStateExceptions where appropriate.
Change-Id: I885570a0a0ae7972b2227dcd08bf83dcb9f86c5a
Signed-off-by: Robert Varga <rovarga@cisco.com>
Robert Varga [Sun, 4 May 2014 19:20:53 +0000 (21:20 +0200)]
BUG-731: do not use temporary variables
These are single-use and can be easily simplified, so get rid of them.
Change-Id: I50c074fa184f59cfe39df34af05c60b20f6b035a
Signed-off-by: Robert Varga <rovarga@cisco.com>
Robert Varga [Sun, 4 May 2014 19:16:28 +0000 (21:16 +0200)]
BUG-731: do not declare Exception to be thrown
These 7 methods either throw only a subclass of RuntimeException,
or IOException. Declare them accordingly.
Change-Id: I253e659dde725df004f63f156448591ef5e85bbb
Signed-off-by: Robert Varga <rovarga@cisco.com>
Dana Kutenicsova [Sat, 3 May 2014 21:09:10 +0000 (23:09 +0200)]
BUG-730 : added unit tests for stateful02
Change-Id: I0673520a53e5c2c46e0c76bf3a4da8f5a8ef6219
Signed-off-by: Dana Kutenicsova <dkutenic@cisco.com>
Dana Kutenicsova [Sun, 20 Apr 2014 09:15:17 +0000 (11:15 +0200)]
Fixed typo.
Change-Id: Ic346974d03a358a377f36acfbc14c2723222e4e9
Signed-off-by: Dana Kutenicsova <dkutenic@cisco.com>
Robert Varga [Fri, 2 May 2014 05:13:30 +0000 (07:13 +0200)]
BUG-632: supress unneeded round-trips to JNI
This fixes Java logic to not update keys which match.
Change-Id: I7ff22f74cb82066367e4bb77c319ab11eb97eb99
Signed-off-by: Robert Varga <rovarga@cisco.com>
Robert Varga [Thu, 1 May 2014 20:16:47 +0000 (22:16 +0200)]
BUG-632: Fix NIO package thinko
It turns out we cannot just proxy around the Selector and
SelectorProvider -- we have to have our own. This is okay, as we can
nicely drop this in.
Change-Id: Idcda5b263bc584fbd9157f0d0fc57d4af82d08d4
Signed-off-by: Robert Varga <rovarga@cisco.com>
Robert Varga [Thu, 1 May 2014 19:03:36 +0000 (21:03 +0200)]
BUG-632: fix netty binding
It turns out netty is not really friendly to us subclassing its channel
implementations. Rework the implementation such that it subclasses the
netty generic substrate.
Change-Id: Ia25f23a5791fd43d617130f5b11d7cbde713e75f
Signed-off-by: Robert Varga <rovarga@cisco.com>
Robert Varga [Thu, 1 May 2014 14:30:35 +0000 (16:30 +0200)]
BUG-632: add initial configuration bits for TCP-MD5
This patch adds the controller-config bit for tcp-md5 and modifies both
BGP and PCEP configurations to take advantage of it. Since the runtime
support is not universally available, these options are left
deactivated.
Change-Id: I61d455966aada9e0cc68dadab724994ba8dcb794
Signed-off-by: Robert Varga <rovarga@cisco.com>
Robert Varga [Thu, 1 May 2014 14:28:12 +0000 (16:28 +0200)]
BUG-632: fix configuration interactions
This commit fixes various problems in configuration module code, so it
actually works. Key problems addressed:
- netty integration was not returning proper instance
- BGP/PCEP modules need to look at actual implementation mbean, not the
service reference
Change-Id: Iaaf03a5fd94f4a2953f671015bee8f635a4c69d5
Signed-off-by: Robert Varga <rovarga@cisco.com>
Robert Varga [Thu, 1 May 2014 15:31:10 +0000 (17:31 +0200)]
BUG-636: fix PCEP MD5 keys not being propagated
This patch fixes the slight oversight of not passing MD5 keys not being
passed down to the dispatcher.
Change-Id: I54246a7a427c7de4d0e9cc9a4e574653bfe9a956
Signed-off-by: Robert Varga <rovarga@cisco.com>
Robert Varga [Thu, 1 May 2014 16:05:09 +0000 (18:05 +0200)]
BUG-370: update Netty version to 4.0.17
Change-Id: I633489a6d90f5718cdc1bf6e325f0e3ef024d34b
Signed-off-by: Robert Varga <rovarga@cisco.com>
Robert Varga [Wed, 30 Apr 2014 05:53:27 +0000 (07:53 +0200)]
BUG-731: fix unused imports and pom.xml override
There are two unused mockito imports and an config artifact version
override -- fix all three of them.
Change-Id: Id6e5b9b2ed1b5e0427fb10f56b065540f728fe0d
Signed-off-by: Robert Varga <rovarga@cisco.com>
Robert Varga [Tue, 29 Apr 2014 12:43:02 +0000 (14:43 +0200)]
BUG-635: implement MD5 auth option for BGP peers
This adds the wiring for the BGP dispatcher to make use of client/server
channels. It also adds validation support.
Change-Id: Ieb2dc4c60dfcd92d8e7a3f732198ad07a734853e
Signed-off-by: Robert Varga <rovarga@cisco.com>
Robert Varga [Sat, 26 Apr 2014 12:27:54 +0000 (14:27 +0200)]
BUG-636: Add configuration option to PCEP
This patch adds the ability to configure a RFC2385 key on a PCEP server
session. The configuration is accepted only if the runtime supports it,
e.g. PCEPTopologyProviderModule checks for presence before ever
attempting to use it.
Change-Id: I349cdef08655ec56ba2ed413f67ab187b0a9fc0c
Signed-off-by: Robert Varga <rovarga@cisco.com>
Robert Varga [Sun, 27 Apr 2014 10:27:54 +0000 (12:27 +0200)]
BUG-634: add tcpmd5 configuration binding
Change-Id: I64d400fb7083a5e9de8a88d067d4d16358fabfff
Signed-off-by: Robert Varga <rovarga@cisco.com>
Dana Kutenicsova [Tue, 29 Apr 2014 08:57:55 +0000 (08:57 +0000)]
Merge "Remove bgp-update-api-config"
Robert Varga [Tue, 29 Apr 2014 05:19:31 +0000 (07:19 +0200)]
Remove bgp-update-api-config
This bundle has been evacuated before Hydrogen release, but retained for
compatibility purposes. It can be safely removed now.
Change-Id: Ic870a7fd255ed42f0f4aba1ef7a0b9327f9e4504
Signed-off-by: Robert Varga <rovarga@cisco.com>
Milos Fabian [Mon, 28 Apr 2014 13:38:38 +0000 (15:38 +0200)]
Bug 187 - PCEP's message parser should ignore unknow objects with flag P=0
-Fixed: ObjectRegistry#parseObject returns null, so such objects are ignored in message parser.
Change-Id: I14251257e55b0495e592ff213c0aa48cffc15640
Signed-off-by: Milos Fabian <milfabia@cisco.com>
Robert Varga [Sun, 27 Apr 2014 09:54:36 +0000 (11:54 +0200)]
BUG-634: improve netty interface
This introduces dedicated ChannelFactory implementations, which allow
for non-default KeyAccessFactory use in the MD5-enabled sockets.
Change-Id: I7ae2e7400076123f6627fdd4b300130da1799539
Signed-off-by: Robert Varga <rovarga@cisco.com>
Tomas Olvecky [Wed, 9 Apr 2014 11:48:47 +0000 (13:48 +0200)]
Resolve Bug:681 - Fix bgpcep after config-manager tests have been changed.
This must be merged after https://git.opendaylight.org/gerrit/#/c/5964/
is merged.
Change-Id: I2b56c5a5bd4f138b58c96a70fcb8296ba2eff261
Signed-off-by: Tomas Olvecky <tolvecky@cisco.com>
Robert Varga [Sat, 26 Apr 2014 17:21:17 +0000 (19:21 +0200)]
BUG-592: use new InstanceIdentifier APIs
This patch gets rid of deprecation warnings stemming from changes to
InstanceIdentifier APIs. These have been optimized to elide as many
list copies as possible and expose utility functions which are very
useful for building temporary instance identifier objects. We attempt to
use them as much as possible.
Change-Id: I7cd8e116eaf2080974ba6d943279f6586298ca50
Signed-off-by: Robert Varga <rovarga@cisco.com>
Robert Varga [Thu, 24 Apr 2014 19:43:35 +0000 (21:43 +0200)]
BUG-634: add connecticity test suite
This patch adds two tests, which test both the positive (matching key)
and negative (mismatched key) scenarios.
Change-Id: Ibee31baa126a54d9cff4d905febad46a812bea21
Signed-off-by: Robert Varga <rovarga@cisco.com>
Robert Varga [Sun, 27 Apr 2014 08:57:16 +0000 (10:57 +0200)]
Easier felix switch
Adjust the comment block such that it is easier to switch to felix for
debugging purposes.
Change-Id: I0207cd1be5158172af5148a5960551aed580aed3
Signed-off-by: Robert Varga <rovarga@cisco.com>
Robert Varga [Fri, 25 Apr 2014 08:53:00 +0000 (10:53 +0200)]
Make sure tests are run in a deterministic order
This simple patch makes sure our order of tests is deterministic and
repeatable. Unit tests should generally be nice and cleanup after
themselves, but that is not always the case and having the ability to
re-create reported conditions is more important than speed.
Change-Id: I51c756a21916e533dd0271db4f6f9e82963f06aa
Signed-off-by: Robert Varga <rovarga@cisco.com>
Robert Varga [Tue, 22 Apr 2014 08:56:47 +0000 (10:56 +0200)]
BUG-634: Rework KeyAccess to work on maps
This fixes an invalid assumptions about how the TCPMD5 API works. It
turns out each FD has a detabase of various keys on a per-address basis,
so we really need to promote the single key into a map of keys.
Also fixes a bundle misconfiguration which prevented it from being
useful in OSGi environment.
Change-Id: Idcbb26383b4e787d7eb8d5d2ce183f5c5a782b2e
Signed-off-by: Robert Varga <rovarga@cisco.com>
Robert Varga [Mon, 21 Apr 2014 05:12:10 +0000 (07:12 +0200)]
BUG-634: add tcpmd5-nio tests
This adds basic tests to the NIO package. These validate correct
operation of MD5ChannelOptions and MD5SocketChannel.
Change-Id: Ic8667416cc575730a814b83ae39a88335b9e611d
Signed-off-by: Robert Varga <rovarga@cisco.com>
Milos Fabian [Fri, 25 Apr 2014 10:42:25 +0000 (12:42 +0200)]
Randomized port number for PCEPTopologyProviderModule instance in PCEPTopologyProviderModuleTest.
Change-Id: Id362d0d9d6b775ddab3ce376dca78f81b97647fe
Signed-off-by: Milos Fabian <milfabia@cisco.com>
Milos Fabian [Tue, 15 Apr 2014 13:06:16 +0000 (15:06 +0200)]
Bug 474 - Added missing config junit tests.
-added config tests in BGP's modules:
-linkstate-config
-parser-spi-config
-topology-provider-config
-rib-spi-config
-created AbstractRibImplModuleTest:
-abstract test class contains mocked md-sal stuff and methods for creating related config module instances
-test classes, where RIBImplModule is injected, or tested directly, should extends this abstract class
-added config tests in PCEP's modules:
-ietf-stateful07
-impl-config (parser modules)
-spi-config
-topology-provider-config
-tunnel-provider-config
-changed dead-timer-value and keep-alive-timer-value type from uint16 into uint8 in pcep-session-proposal-factory config modules
-added config tests in Programming's modules:
-impl-config
-added AbstractInstructionSchedulerTest abstract test class:
-contains mocked md-sal stuff and methods for creating related config module instances
-tests classed, where InstructionSchedulerImplModule is injected, or directly tested, should extends this abstract class
-fixed config tests dependency management
-other minor fixes
Change-Id: I7a439eafb80fd52c4363058d498a3de62129542b
Signed-off-by: Milos Fabian <milfabia@cisco.com>
Robert Varga [Wed, 23 Apr 2014 12:32:26 +0000 (12:32 +0000)]
Merge "Bug 498 - Fixed PCEP and BGP testtools"
Robert Varga [Mon, 21 Apr 2014 05:11:03 +0000 (07:11 +0200)]
BUG-634: add tcpmd5-api tests
Introduce basic tests for API utility classes.
Change-Id: Ic87581edb2d0a9ed767176c23df681b5f14dcaa2
Signed-off-by: Robert Varga <rovarga@cisco.com>
Robert Varga [Mon, 21 Apr 2014 05:03:36 +0000 (07:03 +0200)]
BUG-634: rework packaging
First we split out tcpmd5-api component, which contains the basic
interfaces and classes.
Second we rename tcpmd5-core to tcpmd5-nio to better reflect the fact
this is the binding to java.nio package. We also introduce
DefaultKeyAccessFactoryFactory, which provides a facade to selecting the
best-functioning KeyAccessFactory.
Third we introduce NativeSupportUnavailableException to indicate when
the native support library fails to load/initialize.
Finally we fix invalid javaChannel() reference in tcpmd5-netty, which
should have been called on the superclass.
Change-Id: I1243189ab728abecc473b3aa5a38e9e167b80aad
Signed-off-by: Robert Varga <rovarga@cisco.com>
Robert Varga [Sat, 19 Apr 2014 07:41:51 +0000 (09:41 +0200)]
BUG-634: Rework getKey() implementation
As it turns out, getsockopt(TCP_MD5SIG) is not implemented, which means
we cannot read back the key associated with the channel. This patch
makes sure we cache in the KeyAccess layer and removes the unused native
method.
Change-Id: I941c60afa7380ffddd5745f0e71286e6fcabcdcc
Signed-off-by: Robert Varga <rovarga@cisco.com>
Dana Kutenicsova [Wed, 23 Apr 2014 09:04:06 +0000 (09:04 +0000)]
Merge "BUG-634: Fix .so not being included in the .jar"
Dana Kutenicsova [Wed, 23 Apr 2014 09:01:46 +0000 (09:01 +0000)]
Merge "BUG-634: Fix JNI bugs"
Milos Fabian [Wed, 23 Apr 2014 07:45:36 +0000 (09:45 +0200)]
Bug 498 - Fixed PCEP and BGP testtools
-added missing registration of extension parsers/serializers
-refactored ServiceLoaderBGPExtensionProviderContext - BGPExtensionProviderContext hold as a singleton
Change-Id: Idc5991a017c9bbc7550a1d4170f88f2086fd6416
Signed-off-by: Milos Fabian <milfabia@cisco.com>
Robert Varga [Thu, 17 Apr 2014 13:24:55 +0000 (15:24 +0200)]
BUG-634: Fix .so not being included in the .jar
This patch ensures the native dynamic library is included in the
tcpmd5-jni jar file. Also changes packaging to bundle, implements proper
library loading and adds a unit test to validate basic functions.
Change-Id: I966ef49723de25ae05f75f719a2e0169d161f773
Signed-off-by: Robert Varga <rovarga@cisco.com>
Robert Varga [Fri, 18 Apr 2014 16:55:14 +0000 (18:55 +0200)]
BUG-634: Fix JNI bugs
This patch fixes issues discovered while implementing testcases:
- Fix handlers not being retained
- Fix failure to lookup LoggerFactory.getLogger()
- Remove getsockopt(TCP_MD5SIG), as it is not implemented in Linux
- Fix key removal mechanics
- Make sure struct tcp_md5sig contains the address
Change-Id: Id7185a33d00e794c9c2bfa1a7c4a9bf685a2ed59
Signed-off-by: Robert Varga <rovarga@cisco.com>
Robert Varga [Tue, 22 Apr 2014 08:57:23 +0000 (10:57 +0200)]
BUG-509 Specify ordering in lists
It turns out we need explicit markers for lists where the order of
things is important. This adds such annotations to PCEP object lists.
Change-Id: Ifb695910c93bf65b154ecf86acee5ce99279eb2e
Signed-off-by: Robert Varga <rovarga@cisco.com>
Robert Varga [Sat, 19 Apr 2014 19:23:41 +0000 (21:23 +0200)]
Fix warnings about overridden versions
We specify versions in dependencyManagement, no need to repeat them
here.
Change-Id: Id72c8b1c47e3b665ad32e5dc9da9491addfef161
Signed-off-by: Robert Varga <rovarga@cisco.com>
Robert Varga [Thu, 17 Apr 2014 14:41:01 +0000 (16:41 +0200)]
BUG-472: Fix IT breakage from downstream
Change-Id: If6db3f9443289175889574f4ea8a122ddc390629
Signed-off-by: Robert Varga <rovarga@cisco.com>
Dana Kutenicsova [Wed, 16 Apr 2014 17:21:03 +0000 (17:21 +0000)]
Merge "BUG-509: Fix topology not being augmented"
Robert Varga [Tue, 15 Apr 2014 07:08:02 +0000 (09:08 +0200)]
BUG-731: use IllegalArgumentException
This gets rid of non-descriptive RuntimeException in favor of
IllegalArgumentException.
Change-Id: Idd4f3452eb0a2b2ea7235fdfe5545490209e9993
Signed-off-by: Robert Varga <rovarga@cisco.com>
Robert Varga [Tue, 15 Apr 2014 07:02:50 +0000 (09:02 +0200)]
BUG-731: do not declare Exception
These exceptions are specified by the super method. We have no use for
them, so follow sonar and remove them.
Change-Id: I67ad3d5764e92eba337da5ca0ab58d4bb5dbd215
Signed-off-by: Robert Varga <rovarga@cisco.com>
Robert Varga [Tue, 15 Apr 2014 07:04:30 +0000 (09:04 +0200)]
BUG-731: remove unused field
This filed is no longer used. Remove it and get rid of the warning.
Change-Id: Iced3c01061ca28ec5c889a905db8e5840f47793c
Signed-off-by: Robert Varga <rovarga@cisco.com>
Robert Varga [Mon, 14 Apr 2014 10:57:21 +0000 (12:57 +0200)]
BUG-509: Fix topology not being augmented
The new datastore is picky about augmentations. More precisely it is
thorough in checking that the data destined to a particular adddress is
supposed to be there according to schema.
This is most noticable when a Binding-Aware application uses groupings,
as tthe Binding Specification does not discern between the individual
grouping instantiations -- allowing applications to easily attach
augmentations to a grouping instance and then use it in a context where
that augmentation has not been made.
This patch fixes one such instance, as reported by the following:
Caused by: java.lang.IllegalArgumentException: Supplied QName (urn:opendaylight:params:xml:ns:yang:pcep:crabbe:initiated:00?revision=2014-01-13)initiation is not valid according to schema container stateful
at com.google.common.base.Preconditions.checkArgument(Preconditions.java:119) ~[na:na]
at org.opendaylight.controller.md.sal.common.impl.util.compat.DataNormalizationOperation.fromSchemaAndPathArgument(DataNormalizationOperation.java:497) ~[na:na]
at org.opendaylight.controller.md.sal.common.impl.util.compat.DataNormalizationOperation$DataContainerNormalizationOperation.getChild(DataNormalizationOperation.java:226) ~[na:na]
at org.opendaylight.controller.md.sal.common.impl.util.compat.DataNormalizationOperation$CompositeNodeNormalizationOpertation.normalize(DataNormalizationOperation.java:165) ~[na:na]
at org.opendaylight.controller.md.sal.common.impl.util.compat.DataNormalizationOperation$CompositeNodeNormalizationOpertation.normalize(DataNormalizationOperation.java:144) ~[na:na]
at org.opendaylight.controller.md.sal.common.impl.util.compat.DataNormalizationOperation$CompositeNodeNormalizationOpertation.normalize(DataNormalizationOperation.java:185) ~[na:na]
at org.opendaylight.controller.md.sal.common.impl.util.compat.DataNormalizationOperation$CompositeNodeNormalizationOpertation.normalize(DataNormalizationOperation.java:144) ~[na:na]
at org.opendaylight.controller.md.sal.common.impl.util.compat.DataNormalizationOperation$CompositeNodeNormalizationOpertation.normalize(DataNormalizationOperation.java:185) ~[na:na]
at org.opendaylight.controller.md.sal.common.impl.util.compat.DataNormalizationOperation$CompositeNodeNormalizationOpertation.normalize(DataNormalizationOperation.java:144) ~[na:na]
at org.opendaylight.controller.md.sal.common.impl.util.compat.DataNormalizationOperation$CompositeNodeNormalizationOpertation.normalize(DataNormalizationOperation.java:185) ~[na:na]
at org.opendaylight.controller.md.sal.common.impl.util.compat.DataNormalizationOperation$CompositeNodeNormalizationOpertation.normalize(DataNormalizationOperation.java:144) ~[na:na]
at org.opendaylight.controller.md.sal.common.impl.util.compat.DataNormalizationOperation$CompositeNodeNormalizationOpertation.normalize(DataNormalizationOperation.java:182) ~[na:na]
at org.opendaylight.controller.md.sal.common.impl.util.compat.DataNormalizationOperation$CompositeNodeNormalizationOpertation.normalize(DataNormalizationOperation.java:144) ~[na:na]
at org.opendaylight.controller.md.sal.common.impl.util.compat.DataNormalizer.toNormalized(DataNormalizer.java:88) ~[na:na]
at org.opendaylight.controller.md.sal.common.impl.util.compat.DataNormalizer.toNormalized(DataNormalizer.java:63) ~[na:na]
at org.opendaylight.controller.md.sal.binding.impl.BindingToNormalizedNodeCodec.toNormalizedNode(BindingToNormalizedNodeCodec.java:57) ~[na:na]
at org.opendaylight.controller.md.sal.binding.impl.BindingToNormalizedNodeCodec.toNormalizedNode(BindingToNormalizedNodeCodec.java:51) ~[na:na]
at org.opendaylight.controller.md.sal.binding.impl.AbstractForwardedTransaction.doPutWithEnsureParents(AbstractForwardedTransaction.java:107) ~[na:na]
at org.opendaylight.controller.md.sal.binding.impl.ForwardedBackwardsCompatibleDataBroker$ForwardedBackwardsCompatibleTransacion.putOperationalData(ForwardedBackwardsCompatibleDataBroker.java:218) ~[na:na]
at org.opendaylight.controller.md.sal.binding.impl.ForwardedBackwardsCompatibleDataBroker$ForwardedBackwardsCompatibleTransacion.putOperationalData(ForwardedBackwardsCompatibleDataBroker.java:195) ~[na:na]
at org.opendaylight.bgpcep.pcep.topology.provider.AbstractTopologySessionListener.onSessionUp(AbstractTopologySessionListener.java:153) ~[na:na]
at org.opendaylight.bgpcep.pcep.topology.provider.AbstractTopologySessionListener.onSessionUp(AbstractTopologySessionListener.java:58) ~[na:na]
at org.opendaylight.protocol.pcep.impl.PCEPSessionImpl.sessionUp(PCEPSessionImpl.java:391) ~[na:na]
at org.opendaylight.protocol.framework.AbstractProtocolSession.handlerAdded(AbstractProtocolSession.java:51) ~[na:na]
at io.netty.channel.DefaultChannelPipeline.callHandlerAdded0(DefaultChannelPipeline.java:484) [bundlefile:4.0.17.Final]
... 24 common frames omitted
Change-Id: Ie9d160a3a66c39ca3b4a86c50127c9b9080f2e6f
Signed-off-by: Robert Varga <rovarga@cisco.com>
Robert Varga [Mon, 14 Apr 2014 09:21:04 +0000 (09:21 +0000)]
Merge "Fixed write of uptodate to explicitly set routes."
Dana Kutenicsova [Fri, 11 Apr 2014 08:46:13 +0000 (10:46 +0200)]
Fixed dissapearance of links because of metric value not present.
Change-Id: Ibad22fee30a57f343ba1e3d52867fc0806bdc9f9
Signed-off-by: Dana Kutenicsova <dkutenic@cisco.com>
Dana Kutenicsova [Wed, 9 Apr 2014 16:02:52 +0000 (18:02 +0200)]
RSVP Error code in stateful02 updated actally reflect stateful02 draft.
Change-Id: I349910316608bbb95bccbe0fc43ddb5d88ca9042
Signed-off-by: Dana Kutenicsova <dkutenic@cisco.com>
Dana Kutenicsova [Tue, 8 Apr 2014 11:12:28 +0000 (13:12 +0200)]
BUG-670 : stateful07 now displaying LSP related errors in RESTCONF
Change-Id: Ia036d0fa962f919b47b772c8b04d45cc87afceef
Signed-off-by: Dana Kutenicsova <dkutenic@cisco.com>
Tony Tkacik [Wed, 26 Mar 2014 18:21:44 +0000 (19:21 +0100)]
Fixed write of uptodate to explicitly set routes.
Generated copy constructors does not do full
copy of provided class, only of common super type
fields, so it lead to ommision of routes.
Added workaround which explicitly copies routes
from original Tables TO to newly constructed.
Change-Id: I875284016cffecd84ede841c54feef776e981921
Signed-off-by: Tony Tkacik <ttkacik@cisco.com>
Dana Kutenicsova [Sat, 5 Apr 2014 20:27:27 +0000 (22:27 +0200)]
Added unit tests to MesaageRegistry.
Change-Id: I8f7b9072ef9305cf8cfb30c420f3d8e95876a8c3
Signed-off-by: Dana Kutenicsova <dkutenic@cisco.com>
Dana Kutenicsova [Sat, 5 Apr 2014 18:54:10 +0000 (20:54 +0200)]
Added unit tests for bgp-parser-api.
Change-Id: Ifdd15e880a695d705f5689d0d3e7b48bce78cc25
Signed-off-by: Dana Kutenicsova <dkutenic@cisco.com>
Dana Kutenicsova [Fri, 4 Apr 2014 13:36:54 +0000 (15:36 +0200)]
Added tests for LinkstateAdjRIBsIn.
Change-Id: I488fddbf57730af587c29937fe146f2850329ea0
Signed-off-by: Dana Kutenicsova <dkutenic@cisco.com>
Milos Fabian [Fri, 4 Apr 2014 12:16:13 +0000 (14:16 +0200)]
Remove final keyword from Stateful02LspObjectParser class definition, to be the class extensible.
Change-Id: Ie4be02f2ca02ea02e6333a6e8f4df55de2a351b2
Signed-off-by: Milos Fabian <milfabia@cisco.com>
Dana Kutenicsova [Fri, 28 Mar 2014 17:34:31 +0000 (18:34 +0100)]
BUG-64 : refactor BGP parser to use ByteBuf
Change-Id: Ibbd892d29fe709facaf06cf03273ee0ceea174a9
Signed-off-by: Dana Kutenicsova <dkutenic@cisco.com>
Milos Fabian [Mon, 31 Mar 2014 10:39:00 +0000 (12:39 +0200)]
Bug 467 - Reconnect strategy configuration deleted from bgp/rib-impl-config
-fixed RIBImplModuleTest
-fixed bgp initial configuration
-fixed integration tests
Change-Id: I2909d8ca47911de525d4201c8d858f8225cf0192
Signed-off-by: Milos Fabian <milfabia@cisco.com>
Dana Kutenicsova [Tue, 1 Apr 2014 09:37:57 +0000 (09:37 +0000)]
Merge "Increase the service lookpu timeout"
Robert Varga [Thu, 27 Mar 2014 20:10:22 +0000 (21:10 +0100)]
Increase the service lookpu timeout
This should fix the intermittent errors we have been seeing.
Change-Id: Ia8244cefa2602cb616f4ffabb4f218b76cec697b
Signed-off-by: Robert Varga <rovarga@cisco.com>
Robert Varga [Thu, 27 Mar 2014 01:23:01 +0000 (02:23 +0100)]
Prepare for InstanceIdentifier change
InstanceIdentifier.getPathArguments() will have its return changed.
Prepare by using getPath() instead. Also fixes a warning about using raw
Future.
Change-Id: Idf7f2c4acc6cd4b77c2666ceede87f3cd465f9a3
Signed-off-by: Robert Varga <rovarga@cisco.com>
Dana Kutenicsova [Mon, 31 Mar 2014 13:51:25 +0000 (15:51 +0200)]
BUG-604 : Holdtimer value now calculated as the lower from local/remote session prefs.
Change-Id: I08bb324fec9816cd9110c24601a322c982810303
Signed-off-by: Dana Kutenicsova <dkutenic@cisco.com>
Dana Kutenicsova [Mon, 31 Mar 2014 11:26:42 +0000 (13:26 +0200)]
BUG-594 : added check for AS numbers beyond 2B range.
Change-Id: I30f870191dfccc5996b5fcb0b78400b4690a46be
Signed-off-by: Dana Kutenicsova <dkutenic@cisco.com>
Robert Varga [Wed, 26 Mar 2014 00:10:15 +0000 (01:10 +0100)]
Make sure we set the key in the linkstate route entry
This fixes a missing key when storing the link state route. In order to
make it efficient, we need to pass back the InstanceIdentifier which
will be holding that entry. This also has the nice property of
increasing efficiency for IPv4/IPv6 routes by reusing the route key.
Change-Id: I777594feb283a7e5874da5cf8eaf6a97af49f1ab
Signed-off-by: Robert Varga <rovarga@cisco.com>
Robert Varga [Sat, 29 Mar 2014 15:30:36 +0000 (16:30 +0100)]
netconf-client is no longer needed by persister
Change-Id: I7cb0390231eb41a93103146465b4746ec1ecb11e
Signed-off-by: Robert Varga <rovarga@cisco.com>
Robert Varga [Tue, 25 Mar 2014 15:42:29 +0000 (16:42 +0100)]
Reduce logback dependency scope to test
There's no need to have it compile.
Change-Id: I7ab37e2f6b2f72b96210561df0dd8acabd05d11f
Signed-off-by: Robert Varga <rovarga@cisco.com>
Dana Kutenicsova [Fri, 21 Mar 2014 13:48:29 +0000 (14:48 +0100)]
BUG-64 : initial rewrite, MessageRegistry.
Change-Id: I6aaeb69545f05204c7de3399ce95fa061960e0fa
Signed-off-by: Dana Kutenicsova <dkutenic@cisco.com>
Robert Varga [Wed, 19 Mar 2014 15:33:38 +0000 (16:33 +0100)]
Do not build JNI unconditionally
This patch introduces the "jni" profile, which triggers build of JNI
artifacts. This fixes the build on non-Linux machines, where the JNI
stuff will be pulled from nexus.
Change-Id: I2ca973b4b3067ac05a7297339c21c23581fd3dea
Signed-off-by: Robert Varga <rovarga@cisco.com>
Robert Varga [Fri, 21 Mar 2014 13:17:34 +0000 (14:17 +0100)]
Remove another distributionManagement
Change-Id: Icef92503e7edcafd7a6911529908b26db7707e71
Signed-off-by: Robert Varga <rovarga@cisco.com>
Dana Kutenicsova [Fri, 21 Mar 2014 08:34:57 +0000 (08:34 +0000)]
Merge "Initial code drop of Netty.io integration"
Dana Kutenicsova [Thu, 20 Mar 2014 10:48:51 +0000 (11:48 +0100)]
BUG-64 : initial rewrite, LabelRegistry.
Change-Id: Ie40b980cd3745a6674aa43b7403a36b9074754e7
Signed-off-by: Dana Kutenicsova <dkutenic@cisco.com>
Dana Kutenicsova [Mon, 17 Mar 2014 12:16:37 +0000 (13:16 +0100)]
Fixed delegate predefined value.
Change-Id: I61954d1022e136a7e59840c9358465c7c51a0858
Signed-off-by: Dana Kutenicsova <dkutenic@cisco.com>
Dana Kutenicsova [Fri, 14 Mar 2014 11:41:37 +0000 (12:41 +0100)]
BUG-64 : initial rewrite, ObjectRegistry.
Change-Id: I92764b8f0c44cab1b5b5c83f60f3153be873afda
Signed-off-by: Dana Kutenicsova <dkutenic@cisco.com>
Dana Kutenicsova [Wed, 19 Mar 2014 13:14:17 +0000 (13:14 +0000)]
Merge "Remove unused distributionManagement sections"
Robert Varga [Wed, 19 Mar 2014 12:52:30 +0000 (13:52 +0100)]
Remove unused distributionManagement sections
Change-Id: I5fbd37a261c79ae56616c5439239d4d4c28c3705
Signed-off-by: Robert Varga <rovarga@cisco.com>
Milos Fabian [Wed, 19 Mar 2014 12:23:45 +0000 (13:23 +0100)]
Move AbstractVendorSpecificTlvParser from pcep-spi to pcep-impl.
Change-Id: Ie3c622d9effc7454af387ad1607467d56a97b526
Signed-off-by: Milos Fabian <milfabia@cisco.com>
Robert Varga [Sun, 24 Nov 2013 19:25:55 +0000 (20:25 +0100)]
Initial code drop of Netty.io integration
This is binding glue allows for integration of the base TCP-MD5 code
with Netty.io library.
Change-Id: Id574736e4f3f8e21f75e79591190120b4495943b
Signed-off-by: Robert Varga <robert.varga@pantheon.sk>
Tomas Olvecky [Wed, 19 Mar 2014 08:43:36 +0000 (09:43 +0100)]
Fix bgpcep distribution after yangstore was removed.
Also fix RIBImplModuleTest which should work with yang parser directly.
Change-Id: Ib10eef8d3bc6e06abbd6ae0b42f7a612bff3bd8b
Signed-off-by: Tomas Olvecky <tolvecky@cisco.com>
Dana Kutenicsova [Wed, 19 Mar 2014 08:22:39 +0000 (08:22 +0000)]
Merge "Initial support for RFC2385"
Tomas Olvecky [Thu, 6 Mar 2014 10:02:23 +0000 (11:02 +0100)]
Fix logging.bridge file location in config.ini .
Change-Id: Ic2713ce4543df977dafd89caa5323eec574ce611
Signed-off-by: Tomas Olvecky <tolvecky@cisco.com>
Dana Kutenicsova [Fri, 14 Mar 2014 10:37:24 +0000 (11:37 +0100)]
BUG-64 : initial rewrite, EROSubobjectRegistry.
Change-Id: I3d8e8f6d23ae112c5f4cc045030fb71f0a4d5316
Signed-off-by: Dana Kutenicsova <dkutenic@cisco.com>
Milos Fabian [Fri, 14 Mar 2014 14:40:35 +0000 (15:40 +0100)]
Fix TimedReconnectStrategyFactory - optional parameters must be objects not simple types.
Change-Id: I8078f4824587f6d1bfd7d2046cc8c403289272b1
Signed-off-by: Milos Fabian <milfabia@cisco.com>
Dana Kutenicsova [Thu, 13 Mar 2014 14:14:38 +0000 (15:14 +0100)]
BUG-64 : initial rewrite, RROSubobjectRegistry.
Change-Id: I02b515720c62228ce382e39a11252b8fde0dd33b
Signed-off-by: Dana Kutenicsova <dkutenic@cisco.com>
Dana Kutenicsova [Thu, 13 Mar 2014 12:21:17 +0000 (13:21 +0100)]
BUG-64 : reformat TlvRegistry, to skip using getType method.
Change-Id: If3dd450f148e255357eb11d230b94cb7ab058da8
Signed-off-by: Dana Kutenicsova <dkutenic@cisco.com>
Dana Kutenicsova [Tue, 11 Mar 2014 12:55:50 +0000 (13:55 +0100)]
BUG-64 : initial rewrite, XROSubobjectRegistry.
Change-Id: I845831d51830433dd88a881b63e215baeef0a7c8
Signed-off-by: Dana Kutenicsova <dkutenic@cisco.com>
Milos Fabian [Thu, 13 Mar 2014 12:01:04 +0000 (13:01 +0100)]
Session reconnect strategy updated to factory.
Change-Id: I57ba847641609198492cfb6c361564f003d6af27
Signed-off-by: Dana Kutenicsova <dkutenic@cisco.com>
Signed-off-by: Milos Fabian <milfabia@cisco.com>
Milos Fabian [Fri, 14 Mar 2014 09:10:00 +0000 (10:10 +0100)]
Fixed integration-tests: add missing bundle.
Change-Id: I7dfe93eafaa913dfef247ce4b01c529cdd26d35c
Signed-off-by: Milos Fabian <milfabia@cisco.com>
Robert Varga [Tue, 11 Mar 2014 14:26:52 +0000 (14:26 +0000)]
Merge "BUG-444: Refactored BGP-RIB configuration modules tests."
Milos Fabian [Tue, 11 Mar 2014 13:57:14 +0000 (14:57 +0100)]
BUG-444: Refactored BGP-RIB configuration modules tests.
-Added tests for BGPTableTypeImplModule
-Fixed validations in RIBImplModule
Change-Id: Iaaba1f6ceb52858d6b54c0b52774dab3b39596ad
Signed-off-by: Milos Fabian <milfabia@cisco.com>