Stephen Kitt [Fri, 7 Jul 2017 15:50:31 +0000 (17:50 +0200)]
Upgrade to odlparent 2.0.2
Change-Id: I9660748964bf09f3db84481c62e7cce7843ef7bd
Signed-off-by: Stephen Kitt <skitt@redhat.com>
Tomas Cere [Mon, 10 Jul 2017 14:12:07 +0000 (16:12 +0200)]
Simplify RC4 removal
Change-Id: Ib26c8c7a7bad5e156f33c444593db0c161af0b59
Signed-off-by: Tomas Cere <tcere@cisco.com>
Tomas Cere [Mon, 10 Jul 2017 14:02:19 +0000 (16:02 +0200)]
Use RSA for ssh server
The bouncy-castle bump increased the default size of DSA to 2048 which
some clients can choke on. We shouldn't use DSA anyhow so switch it up
to 4096 RSA everywhere.
Change-Id: I936eb240a534367fff550d25dcedc3de069c6654
Signed-off-by: Tomas Cere <tcere@cisco.com>
Tomas Cere [Fri, 7 Jul 2017 10:08:02 +0000 (10:08 +0000)]
Merge "Bug 8725: Remove netconf device mounting via the config system"
Tomas Cere [Fri, 7 Jul 2017 08:53:00 +0000 (08:53 +0000)]
Merge "Skip SFT in features-netconf-connector"
Tom Pantelis [Thu, 29 Jun 2017 11:48:52 +0000 (07:48 -0400)]
Bug 8725: Remove netconf device mounting via the config system
Change-Id: Ida7a236f0bd75978e07a64b38497ccf8aa75fea8
Signed-off-by: Tom Pantelis <tompantelis@gmail.com>
Yrineu Rodrigues [Fri, 30 Jun 2017 19:13:48 +0000 (16:13 -0300)]
BUG-8752: apidoc explorer show/hide and list operations no longer works
This patch fixes the APIdoc explorer after change from
org.json to jackson
Change-Id: I9168c5aa08b820df38ac9a2413abba08a567cccf
Signed-off-by: Yrineu Rodrigues <yrineu.rodrigues@serro.com>
Tom Pantelis [Fri, 30 Jun 2017 13:51:09 +0000 (09:51 -0400)]
Skip SFT in features-netconf-connector
odl-netconf-clustered-topology and odl-netconf-topology are mutually
exclusive so running SFT may fail intermittently due to timing. This
will be addressed in a future release of odlparent to disable SFT
for feature-repo-parent by default and we can revert this later
although it doesn't hurt to disable locally.
Change-Id: Id91e36ea857179beba0e4a862ca9ab1821735025
Signed-off-by: Tom Pantelis <tompantelis@gmail.com>
Tomas Cere [Thu, 29 Jun 2017 13:33:06 +0000 (13:33 +0000)]
Merge "Use final field as lock in AsyncSshHandlerWriter"
Tomas Cere [Thu, 29 Jun 2017 12:55:42 +0000 (12:55 +0000)]
Merge "Bug 8071 - Do not fail on modules with invalid revision during resolution of ietf-netconf-yang-library/modules-state node"
Andrej Mak [Mon, 17 Oct 2016 11:12:02 +0000 (13:12 +0200)]
Use final field as lock in AsyncSshHandlerWriter
Change-Id: I355e626b90e5cd27dfb5da96ef76c13a36f77dde
Signed-off-by: Andrej Mak <andrej.mak@pantheon.tech>
miroslav.kovac [Fri, 31 Mar 2017 09:36:51 +0000 (11:36 +0200)]
Bug 8124 - get-schema rpc fails to send error message
Change-Id: If2d87beec2dd8c1b3579d10ddcb008d35effa289
Signed-off-by: miroslav.kovac <miroslav.kovac@pantheon.tech>
Tomas Cere [Wed, 28 Jun 2017 11:40:52 +0000 (11:40 +0000)]
Merge "Remove the need for synthetic methods"
Tomas Cere [Wed, 28 Jun 2017 08:58:38 +0000 (08:58 +0000)]
Merge "Make netconf utilize encrypted passwords only"
Jakub Morvay [Tue, 27 Jun 2017 18:34:20 +0000 (18:34 +0000)]
Merge "Convert yanglib project to blueprint"
Colin Dixon [Tue, 27 Jun 2017 14:09:28 +0000 (10:09 -0400)]
Make netconf utilize encrypted passwords only
Change-Id: I5c178c4e3865e374f39e326ceb9c4e24be269590
Signed-off-by: Ryan Goulding <ryandgoulding@gmail.com>
Signed-off-by: Atul Gosain <agosain@brocade.com>
Signed-off-by: Colin Dixon <colin@colindixon.com>
Tomas Cere [Tue, 27 Jun 2017 13:40:00 +0000 (13:40 +0000)]
Merge "Use lambdas to remove references to Function"
Robert Varga [Thu, 25 May 2017 10:41:45 +0000 (12:41 +0200)]
Use lambdas to remove references to Function
Change-Id: I8e45aca6ce8c3af4845502784ade1d859b375f09
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
Jakub Morvay [Tue, 27 Jun 2017 11:37:31 +0000 (11:37 +0000)]
Merge "Add blueprint wiring for restconf connector"
Tomas Cere [Tue, 27 Jun 2017 10:28:26 +0000 (10:28 +0000)]
Merge "Don't loose cause when reporting parsing errors."
Gwenael Lambrouin [Thu, 22 Jun 2017 12:27:42 +0000 (14:27 +0200)]
BUG-8697: select correct transformer for schemaless netconf mounts
Getting or editing the configuration of a schemaless-mounted netconf device
from the code of an ODL application results in a NullPointerException.
This patch fixes the NetconfBaseOps constructor so that the RPC structure
transformer dedicated to schemaless mount points is actually selected.
Change-Id: Ib0b2d1c9910613f4354bdcb4f1c56e9be36a95b2
Signed-off-by: Gwenael Lambrouin <gwenael.lambrouin@b-com.com>
(cherry picked from commit
32621d57701de889651097ce5c1bfc3bb5f43ce9)
Tom Pantelis [Mon, 26 Jun 2017 14:39:03 +0000 (10:39 -0400)]
Convert yanglib project to blueprint
Added blueprint XML to instatiate the YangLibProvider. The yanglib
yang module was converted to a container that is stored in the DS and
provided via the clustered-app-config element. The CSS YangLibModule
was removed.
Change-Id: Ie45fb8261665fea38a4153c01484986d8a0272c4
Signed-off-by: Tom Pantelis <tompantelis@gmail.com>
Tomas Cere [Mon, 15 May 2017 10:38:20 +0000 (12:38 +0200)]
Don't loose cause when reporting parsing errors.
Change-Id: I5e5a07c00f29784a6289aa1ed24187ed6f42e833
Signed-off-by: Tomas Cere <tcere@cisco.com>
Tom Pantelis [Thu, 1 Jun 2017 19:07:43 +0000 (15:07 -0400)]
Add blueprint wiring for restconf connector
Added the blueprint XML wiring and deprecated the CSS config yang
and modules. The CSS Module classes are essentially no-ops but
remain for backwards compatibility until CSS is completely
removed.
Change-Id: I6f5e7f369958ead9ce00a7fc6c8afe551738f195
Signed-off-by: matthieu <mcauffiez@inocybe.com>
Signed-off-by: Tom Pantelis <tompantelis@gmail.com>
Stephen Kitt [Thu, 22 Jun 2017 16:08:24 +0000 (18:08 +0200)]
Build features-... instead of features4-...
The directories were renamed, but not the artifacts.
Change-Id: I4abe486b54c6ff2220ee83ee3a6c274914932e43
Signed-off-by: Stephen Kitt <skitt@redhat.com>
Jakub Morvay [Wed, 21 Jun 2017 11:41:46 +0000 (13:41 +0200)]
Bump odlparent dependency to 2.0.0
This updates odlparent dependency to 2.0.0 version. Furthermore, karaf 3
features are eliminated.
Change-Id: I8e18e032ea9cfb36f3913dcbfa7c3d1d6cd9ee59
Signed-off-by: Jakub Morvay <jmorvay@cisco.com>
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
Jakub Morvay [Wed, 21 Jun 2017 14:41:58 +0000 (14:41 +0000)]
Merge changes I0d93e3c4,I34aca892,I74162d9a,Icd48e366
* changes:
Remove netconf-auth service-type config yang
Remove netconf-mapping-api service-type config yang
Remove netconf-notifications-api service-type config yang
Remove netconf-api service-type config yang
Jakub Morvay [Wed, 21 Jun 2017 07:52:43 +0000 (07:52 +0000)]
Merge "Switch from config-parent in pom files"
Tomas Cere [Tue, 20 Jun 2017 14:40:07 +0000 (14:40 +0000)]
Merge "Remove import-package declaration"
Tomas Cere [Tue, 20 Jun 2017 14:40:00 +0000 (14:40 +0000)]
Merge "Get rid of stale prerequisite for maven"
Tomas Cere [Tue, 20 Jun 2017 13:39:59 +0000 (13:39 +0000)]
Merge "Migrate XML output to yang-data-codec-xml"
Tomas Cere [Tue, 20 Jun 2017 13:39:20 +0000 (13:39 +0000)]
Merge "Run NormalizedNode parsing inline if possible"
Ryan Goulding [Tue, 20 Jun 2017 00:22:03 +0000 (20:22 -0400)]
Get rid of stale prerequisite for maven
It is commonly accepted that ODL requires maven 3.3.9+ now. This
change removes a stale prerequisite for maven so that it is more
consistent with project guidelines.
Change-Id: Idd3bbe1c7ecbbc9b790fe50b7b29d8a263d98bd5
Signed-off-by: Ryan Goulding <ryandgoulding@gmail.com>
Tomas Cere [Tue, 20 Jun 2017 12:26:09 +0000 (12:26 +0000)]
Merge "Bug 5995: remove org.json"
Tomas Cere [Tue, 20 Jun 2017 12:25:37 +0000 (12:25 +0000)]
Merge "Bug 8622: convert sal-rest-docgen to Jackson"
Robert Varga [Tue, 13 Jun 2017 16:21:08 +0000 (18:21 +0200)]
Remove import-package declaration
Maven exclusions should not be necessary.
Change-Id: I32c843efcd965624797a61ceff1b1ed4ce80d2e6
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
Robert Varga [Tue, 13 Jun 2017 16:40:52 +0000 (18:40 +0200)]
Remove the need for synthetic methods
Using private methods from anonymous classes forces javac to create
synthetics. Lower isolation to package-private to call them directly.
Change-Id: I52a5d641f275d5a2e9924e60910641ccdce42281
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
Tomas Cere [Tue, 20 Jun 2017 09:44:01 +0000 (09:44 +0000)]
Merge "Use SimpleSchemaContext instead of EffectiveSchemaContext"
Tom Pantelis [Sat, 17 Jun 2017 12:46:07 +0000 (08:46 -0400)]
Remove netconf-auth service-type config yang
The service-type identities aren't used so remove them.
Change-Id: I0d93e3c46e93db31d4515c55296c937f4f7a4d19
Signed-off-by: Tom Pantelis <tompantelis@gmail.com>
Tom Pantelis [Sat, 17 Jun 2017 12:43:43 +0000 (08:43 -0400)]
Remove netconf-mapping-api service-type config yang
The service-type identities aren't used so remove them.
Change-Id: I34aca892bdd88f6ffa50c63a933a6cd6266cd75c
Signed-off-by: Tom Pantelis <tompantelis@gmail.com>
Tom Pantelis [Sat, 17 Jun 2017 12:41:18 +0000 (08:41 -0400)]
Remove netconf-notifications-api service-type config yang
The service-type identities aren't used so remove them.
Change-Id: I74162d9a055c9d2757a3781b19de5d9dc4054f85
Signed-off-by: Tom Pantelis <tompantelis@gmail.com>
Tom Pantelis [Sat, 17 Jun 2017 12:35:42 +0000 (08:35 -0400)]
Remove netconf-api service-type config yang
The service-type identities aren't used so remove them.
Change-Id: Icd48e3666773ca65a3f08a65b2377c0eaf830992
Signed-off-by: Tom Pantelis <tompantelis@gmail.com>
Tom Pantelis [Sat, 17 Jun 2017 02:41:40 +0000 (22:41 -0400)]
Switch from config-parent in pom files
Most netconf bundles don't or no longer use the CSS so they
don't need to use config-parent.
Change-Id: I59757e8e81709f4904615b0e3829bc5ef569000b
Signed-off-by: Tom Pantelis <tompantelis@gmail.com>
Tom Pantelis [Wed, 14 Jun 2017 18:42:41 +0000 (14:42 -0400)]
Remove UriInfo from JSONRestconfService API methods
https://git.opendaylight.org/gerrit/#/c/48369/ added a UriInfo parameter
to several JSONRestconfService API methods in Carbon. However this was an
undocumented API change which will break downstream users. Further,
there is no available implementation of UriInfo so each user would have
to provide an implementation which would pretty much render the API
unusable. We do not need to expose UriInfo in this API so I've remove the
parameters.
We'll need to cherry-pick to stable/carbon for SR1 to minimize breakage.
Change-Id: I74285413c2a3a92510398a57ea82567b646c6d27
Signed-off-by: Tom Pantelis <tompantelis@gmail.com>
Stephen Kitt [Wed, 14 Jun 2017 12:14:24 +0000 (14:14 +0200)]
Bug 5995: remove org.json
This switches sal-rest-connector to Jackson and Gson and cleans up the
features.
Change-Id: I587ab4638374fccb344e9a769ee2793e16863fef
Signed-off-by: Stephen Kitt <skitt@redhat.com>
Stephen Kitt [Tue, 13 Jun 2017 16:25:59 +0000 (18:25 +0200)]
Bug 8622: convert sal-rest-docgen to Jackson
This removes the requirement for org.json in sal-rest-docgen, which
causes bundle wiring issues in Karaf 4 (sal-rest-docgen ends up
exporting org.json, which Jackson’s compatibility layer consumes,
creating a dependency loop which causes refreshes anytime anything
touches the sal-rest-docgen).
Change-Id: I877f3e567fdeba3258a19a40f1f9e31bb19df24d
Signed-off-by: Stephen Kitt <skitt@redhat.com>
Robert Varga [Mon, 12 Jun 2017 16:56:37 +0000 (18:56 +0200)]
Use SimpleSchemaContext instead of EffectiveSchemaContext
EffectiveSchemaContext is an implementation detail of the parser,
SimpleSchemeContext is a beta-level API from yang-model-util.
Change-Id: I524f9e91562b5fa8d6d942f40bf957ec97e40e88
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
Thanh Ha [Tue, 13 Jun 2017 18:41:35 +0000 (14:41 -0400)]
Migrate to odlparent 1.9.0
Change-Id: Ib0dfd4ded0c67e04ae7e4f4d4cc537cf2c632486
Signed-off-by: Thanh Ha <thanh.ha@linuxfoundation.org>
Robert Varga [Sat, 20 May 2017 09:57:22 +0000 (11:57 +0200)]
Migrate XML output to yang-data-codec-xml
The codecs provided yang-data-impl are superseded by those in
codec-xml. This patch provides the first part of migration.
Change-Id: If777defcc6218faf5597289956b800d68dcb5e5a
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
Robert Varga [Sun, 21 May 2017 16:31:48 +0000 (18:31 +0200)]
Run NormalizedNode parsing inline if possible
While JSON does not guarantee object ordering, we do not have to
fall back to building internal string if we have the target node
set.
If the target has been set, we call directly to parser with our
current parser, which will be moved forward -- which is more
efficient.
Change-Id: I311cc3525944fd2327183c200b3ed9dbee9e915a
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
Robert Varga [Mon, 12 Jun 2017 13:12:52 +0000 (15:12 +0200)]
Adjust to EffectiveSchemaContext movement
This code depends directly on parser internals. Adjust the import,
but really this should be fixed by subclassing AbstractSchemaContext
from yang-model-util.
Change-Id: I57c84ec7e721bf0bd1e6a6eef1d8ccd0562f8b74
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
David Suarez [Sat, 3 Jun 2017 16:06:08 +0000 (18:06 +0200)]
Fix checkstyle problems not detected by the current version
This change is required for overall move to new Checkstyle version, see
https://git.opendaylight.org/gerrit/#/q/topic:bumpCheckstyle
Change-Id: I4fac148d693e5de5c96485243a5469a22f1a8237
Signed-off-by: David Suarez <david.suarez.fuentes@ericsson.com>
Marek Gradzki [Mon, 5 Jun 2017 14:28:44 +0000 (16:28 +0200)]
BUG-8085: create missing parent augmentation node
Augmentation nodes do not exist in serialized form
(e.g. in edit-config message), but are required by DataTree-based
DOMDataBroker implementations, so should be created if data from
augment is present.
This patch creates missing augment nodes by issuing merge on augmentation
before put operation (based on current behaviour for ListSchemaNodes).
Change-Id: If657ae96e914fc46617099042a833c7d4d5883b7
Signed-off-by: Marek Gradzki <mgradzki@cisco.com>
Thanh Ha [Tue, 6 Jun 2017 02:18:22 +0000 (22:18 -0400)]
Migrate to odlparent 1.8.0-Carbon
Per request of odlparent project we are downgrading all Nitrogen
projects to use the released odlparent 1.8.0-Carbon to allow for the
odlparent project to start performing semver style releases.
Jira: RELENG-159
RT: 41406
Change-Id: Ib81afe4e0e7eb5587044bade9d50d00bc98906e1
Signed-off-by: Thanh Ha <thanh.ha@linuxfoundation.org>
Tomas Cere [Mon, 5 Jun 2017 13:39:45 +0000 (13:39 +0000)]
Merge "Un-deprecate JSONRestconfService(Impl)"
Tomas Cere [Wed, 31 May 2017 12:53:57 +0000 (14:53 +0200)]
BUG 8566 direct writes to ordered list fail
Incorrect handling of direct writes into a mapEntry which tries
to merge an empty parent list instead of an exists check but doesnt
correctly handle the possibility of an OrderedMapNode as a parent.
Change-Id: Ia2c75cc50e6ff4c9c7c9be36846c08cc85cff491
Signed-off-by: Tomas Cere <tcere@cisco.com>
Tom Pantelis [Fri, 2 Jun 2017 03:41:01 +0000 (23:41 -0400)]
Un-deprecate JSONRestconfService(Impl)
Patch https://git.opendaylight.org/gerrit/#/c/56391/ deprecated
JSONRestconfService and JSONRestconfServiceImpl however no reason
was given in the patch message (in fact the patch was about fixing
checkstyle violations) nor were any comments added to the classes
as to why they're deprecated.
There really shouldn't be any reason to deprecate these classes
as they are useful and are used in applications. Therefore I remove
the @Deprecated annotation.
Change-Id: I335c8e8665cc2b489e4d3b690d8cac7dbd157b4f
Signed-off-by: Tom Pantelis <tompantelis@gmail.com>
Tomas Cere [Mon, 29 May 2017 10:44:20 +0000 (10:44 +0000)]
Merge "Test RestconfWrapperProviders"
Tomas Cere [Mon, 29 May 2017 10:43:11 +0000 (10:43 +0000)]
Merge changes Ieb46b050,I6bff6467
* changes:
Bug 8533: Not possible to invoke RPC on mount points with new Restconf
Bug 7933: NPE when posting using XML
Jakub Toth [Thu, 13 Apr 2017 09:50:44 +0000 (11:50 +0200)]
Test RestconfWrapperProviders
Change-Id: Id95c607bba19c56c6924938ba5ef107f8937a76b
Signed-off-by: Jakub Toth <jatoth@cisco.com>
Signed-off-by: Ivan Hrasko <ivan.hrasko@pantheon.tech>
Ivan Hrasko [Tue, 23 May 2017 07:27:43 +0000 (09:27 +0200)]
Bug 8533: Not possible to invoke RPC on mount points with new Restconf
- put identifer of RPC into InstanceIdentiferContext
instead of mount point identifer which leads to fail
in unit test
- when working with mount point use mount point's
schema context to find RPC module, previously used
controller's schema context leads to NPE
- repaired imports in RestconfApplication to use
new implementation
Change-Id: Ieb46b050e881c89ea58bb9f521be69554733d575
Signed-off-by: Ivan Hrasko <ivan.hrasko@pantheon.tech>
Tomas Cere [Tue, 23 May 2017 10:43:13 +0000 (10:43 +0000)]
Merge "Bug 7933: NPE when posting using XML"
matus.kubica [Thu, 18 May 2017 14:32:26 +0000 (16:32 +0200)]
Bug 7933: NPE when posting using XML
PUT operation:
- fix issue when putting malformed XML body did not
fail with appropriate status.
Change-Id: I6bff6467f4b99bed9adf1ac39def54822899761e
Signed-off-by: matus.kubica <matus.kubica@pantheon.tech>
Signed-off-by: Ivan Hrasko <ivan.hrasko@pantheon.tech>
Robert Varga [Sun, 21 May 2017 15:28:36 +0000 (17:28 +0200)]
Introduce abstract body readers
{Json,Xml}{NormalizedNode,ToPatch}BodyReader classes share some
common code. Factor it out into abstract classes to reduce code
duplication.
Change-Id: I7358151f617f5327fc514ac2066b2aee883b2fba
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
Robert Varga [Sun, 21 May 2017 14:13:33 +0000 (16:13 +0200)]
Use URI.create()
We wrap 'new URI(String)' in a try-catch block, which is not really
necessary -- use URI.create(String) to get pretty much the same
behavior (except for the exception thrown).
Change-Id: Id9baa17dddb564b2be6d2258a1d73947a67c15f6
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
Robert Varga [Sun, 21 May 2017 14:12:48 +0000 (16:12 +0200)]
Pre-instantiate operations
There is no point in creating the operations over and over again.
Instantiate them immediately and keep them in an ImmutableSet.
Change-Id: I952bcf026b9d0de99252530e8fc44863408adb3e
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
Ivan Hrasko [Fri, 19 May 2017 12:39:16 +0000 (14:39 +0200)]
Bug 7933: NPE when posting using XML
- NPE was thrown because leaf value was not parsed,
thus data value was null
- fixed by adding condition to parse leaf nodes
- log warning when unknown schema node type was
not parsed
- fixed in old and new restconf
Change-Id: I13c8e419e35999980548e09f6963b0d407a45b31
Signed-off-by: Ivan Hrasko <ivan.hrasko@pantheon.tech>
Robert Varga [Sat, 20 May 2017 12:05:46 +0000 (14:05 +0200)]
Fix PatchEditOperation
Enums support methods, so instead of a switch dispatch use normal
property. Also simplify users so they do not pass strings around.
Change-Id: I3818f3d6a4020cee2f2e43fb8ec110e0bc872ba6
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
Robert Varga [Sat, 20 May 2017 11:46:12 +0000 (13:46 +0200)]
Optimize JsonToPatchBodyReader
Use a StringBuilder instead of a StringBuffer, use switch statements
and du not use string concat when using StringBuilder.append().
Change-Id: Icaf68a7a92dae6aece4e9382ba4a03bac429d4ec
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
Robert Varga [Sat, 20 May 2017 10:19:08 +0000 (12:19 +0200)]
Fix various warnings
The code is full of eclipse warnings around static methods, raw types
and similar. Fix that up.
Change-Id: I8aa84b8c0fbde5c32ab855dfc215205501ce2777
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
Ivan Hrasko [Tue, 16 May 2017 12:14:27 +0000 (14:14 +0200)]
Bug 8153: enforce check-style rules for netconf
- enforce checkstyle in all modules + cleanup
Change-Id: I64b003c8f8f554445423e54642ef46bb95bc90b1
Signed-off-by: Ivan Hrasko <ivan.hrasko@pantheon.tech>
Ivan Hrasko [Mon, 15 May 2017 13:50:39 +0000 (15:50 +0200)]
Bug 8153: enforce check-style rules for netconf
- call-home
Change-Id: I1b5e2db2d6b0a4d3591e56d773cfcd4ffc13ae67
Signed-off-by: Ivan Hrasko <ivan.hrasko@pantheon.tech>
Ivan Hrasko [Mon, 15 May 2017 08:52:02 +0000 (10:52 +0200)]
Bug 8153: enforce check-style rules for netconf
- netconf-testool
Change-Id: Ic7e772768596d7e9e66e74896029f72caadeff9e
Signed-off-by: Ivan Hrasko <ivan.hrasko@pantheon.tech>
Ivan Hrasko [Wed, 17 May 2017 10:54:57 +0000 (12:54 +0200)]
Bug 8351: enforce check-style rules for restconf
- enforce checkstyle in all modules + cleanup
Change-Id: I9d8dc0d8461a11582029debcba8512ef6f0ed96a
Signed-off-by: Ivan Hrasko <ivan.hrasko@pantheon.tech>
Ivan Hrasko [Mon, 15 May 2017 14:27:34 +0000 (16:27 +0200)]
Bug 8351: enforce check-style rules for restconf
- sal-rest-docgen
Change-Id: I848691f539b189dffcd9ff10fadf5d3c708a9792
Signed-off-by: Ivan Hrasko <ivan.hrasko@pantheon.tech>
matus.kubica [Tue, 2 May 2017 14:17:30 +0000 (16:17 +0200)]
Bug 8351: Enforce check-style rules for restconf - sal-rest-connector
Organize Imports for Checkstyle compliance.
Checkstyle compliance: line length.
Checkstyle compliance: various types of small changes.
Checkstyle compliant Exception handling.
Checkstyle final clean up & enforcement.
Add the fail on violation flag into the pom.xml .
Change-Id: I81b704c5ed79dead57e5e37c925b4aaa8579444a
Signed-off-by: matus.kubica <matus.kubica@pantheon.tech>
Signed-off-by: Ivan Hrasko <ivan.hrasko@pantheon.tech>
Ivan Hrasko [Tue, 16 May 2017 08:55:04 +0000 (10:55 +0200)]
Bug 8455: Yang Patch response is not having the error details
- add error-info to error message
Change-Id: I0d637871969ff91850749228e92bda2055c71eef
Signed-off-by: Ivan Hrasko <ivan.hrasko@pantheon.tech>
Ivan Hrasko [Mon, 15 May 2017 07:27:53 +0000 (09:27 +0200)]
Bug 8153: enforce check-style rules for netconf
- netconf-cli module
Change-Id: Ifb0be378ff9f3ddc697bb34d139c5fc76b824907
Signed-off-by: Ivan Hrasko <ivan.hrasko@pantheon.tech>
matus.kubica [Wed, 19 Apr 2017 09:29:42 +0000 (11:29 +0200)]
Bug 8153: Enforce check-style rules for netconf - mdsal-netconf-connector
Organize Imports for Checkstyle compliance.
Checkstyle compliance: line length.
Checkstyle compliance: various types of small changes.
Checkstyle compliant Exception handling.
Checkstyle final clean up & enforcement.
Add the fail on violation flag into the pom.xml .
Change-Id: Ib50c1754c97932714d4eb19b7e34a818af8a1a77
Signed-off-by: matus.kubica <matus.kubica@pantheon.tech>
Signed-off-by: Ivan Hrasko <ivan.hrasko@pantheon.tech>
miroslav.kovac [Thu, 6 Apr 2017 08:47:59 +0000 (10:47 +0200)]
Bug 8071 - Do not fail on modules with invalid revision during
resolution of ietf-netconf-yang-library/modules-state node
After parsing xml with invalid revision string it failed instead
of returning only modules with correct revision.
It reads an xml, creates a new xml string using only modules with
correct revision and logs the ones that has wrong revision. New xml
string will be parsed to normalized node.
Change-Id: Ie51163864f3547560840cb7d52ea8762437a6b1e
Signed-off-by: miroslav.kovac <miroslav.kovac@pantheon.tech>
Tomas Cere [Thu, 11 May 2017 10:22:17 +0000 (10:22 +0000)]
Merge "Bug 8351: Enforce check-style rules for restconf - sal-rest-docgen"
Tomas Cere [Thu, 11 May 2017 10:20:50 +0000 (10:20 +0000)]
Merge "Bug 8153: Enforce check-style rules for netconf - yanglib"
miroslav.kovac [Mon, 3 Apr 2017 12:39:19 +0000 (14:39 +0200)]
Bug 8074 - actor name [test-device] is not unique!
When creating a new actor with same name as an old one, it might fail
becuase old actor might not be completely gone at this point.
This creates a slaveActor which will never be closed, but instead
every time that mountpoint is closed or master is changed, it would
update NetconfNodeActor data and keep the same actor.
Only time that actor is closed is when node was deleted. Or slave
became a master
The name of the actor is removed - letting akka to handle that -
log the akka created name.
Change-Id: I6d10db7b8902048632e336fb5a9701a797b363c6
Signed-off-by: miroslav.kovac <miroslav.kovac@pantheon.tech>
Jakub Morvay [Wed, 10 May 2017 13:18:29 +0000 (13:18 +0000)]
Merge "Resolve exception from registerClusterSingletonService"
Andrej Mak [Tue, 9 May 2017 11:17:48 +0000 (13:17 +0200)]
Bug 8405: Add close check to NetconfDevice
Since schema resolution runs in its own thread, it is possible,
that handleSalInitializationSuccess is called when
NetconfDeviceCommunicator was closed meanwhile. Add check to
prevent this.
Change-Id: If93d32b26f0b98c4c0d47fdd65fdb5104db20bc5
Signed-off-by: Andrej Mak <andrej.mak@pantheon.tech>
miroslav.kovac [Wed, 10 May 2017 09:05:14 +0000 (11:05 +0200)]
Resolve exception from registerClusterSingletonService
registerClusterSingletonService return RuntimeException in
case of problems with registration and client should implement
strategy to resolve this issue
Change-Id: If6350969f1d1c2917c78850f4c66629b52137b0b
Signed-off-by: miroslav.kovac <miroslav.kovac@pantheon.tech>
matus.kubica [Tue, 2 May 2017 07:54:31 +0000 (09:54 +0200)]
Bug 8153: Enforce check-style rules for netconf - yanglib
Organize Imports for Checkstyle compliance.
Checkstyle compliance: line length.
Checkstyle compliance: various types of small changes.
Checkstyle compliant Exception handling.
Checkstyle final clean up & enforcement.
Add the fail on violation flag into the pom.xml .
Change-Id: I841ed5ba3ba57f91e18a82015c7582195cec5562
Signed-off-by: matus.kubica <matus.kubica@pantheon.tech>
Jakub Morvay [Sat, 6 May 2017 11:55:03 +0000 (11:55 +0000)]
Merge "Bug 7891: Add ssh feature to netconf-netty-util"
Jakub Morvay [Sat, 6 May 2017 11:53:10 +0000 (11:53 +0000)]
Merge "Bug 8364: Set bundle context when module reloads"
Tomas Cere [Fri, 5 May 2017 13:52:24 +0000 (13:52 +0000)]
Merge "Bug 8361 - NetconfConnector cannot be created due to NullPointerException"
matus.kubica [Tue, 25 Apr 2017 12:38:20 +0000 (14:38 +0200)]
Bug 8153: Enforce check-style rules for netconf - netconf-topology-singleton
Organize Imports for Checkstyle compliance.
Checkstyle compliance: line length.
Checkstyle compliance: various types of small changes.
Checkstyle compliant Exception handling.
Checkstyle final clean up & enforcement.
Add the fail on violation flag into the pom.xml .
Change-Id: I690d1d6c21f042874977f1030f216b08c4c21c45
Signed-off-by: matus.kubica <matus.kubica@pantheon.tech>
Andrej Mak [Fri, 5 May 2017 09:05:46 +0000 (11:05 +0200)]
Bug 8364: Set bundle context when module reloads
Fix for bug Bug 8214 handles waiting for AAA, when Restconf
is installed for the first time. Override handleChangedClass
to do the same thing for reinstall case. Also close old
module to release bound ports and started threads.
Change-Id: I00f5f330fca39bb63f66517dddd8d1371ae353e8
Signed-off-by: Andrej Mak <andrej.mak@pantheon.tech>
Martin Mihálek [Wed, 3 May 2017 11:24:02 +0000 (13:24 +0200)]
Bug 8361 - NetconfConnector cannot be created due to NullPointerException
- Adds Nullpointer checks in RemoteDeviceConnecterImpl
- If Yang module capabilities are missing its override flag is set to false
- If Non module capabilities are missing its override flag is set to false
Change-Id: I25d172e94a1013d79c0c7db37c930c89ad2ed94f
Signed-off-by: Martin Mihálek <mamihale@cisco.com>
Andrej Mak [Wed, 3 May 2017 07:06:48 +0000 (09:06 +0200)]
Bug 7891: Add ssh feature to netconf-netty-util
Odl-netconf-netty-util needs sshd-core bunde to work.
It is provided by karaf. However, if odl-netconf-netty-util
is placed to featuresBoot, ssh provided by karaf isn't
started, so ResolutionException is thrown. To prevent this
add ssh feature to odl-netconf-netty-util.
Change-Id: Iaf5ecddaa726ea892191b5cf47095c1df3b096f6
Signed-off-by: Andrej Mak <andrej.mak@pantheon.tech>
Tomas Cere [Wed, 3 May 2017 14:01:52 +0000 (14:01 +0000)]
Merge "Bug 8153: Enforce check-style rules for netconf - sal-netconf-connector"
Tomas Cere [Wed, 3 May 2017 13:34:51 +0000 (13:34 +0000)]
Merge "Bug 8153: Enforce check-style rules for netconf - netconf-topology"
matus.kubica [Fri, 21 Apr 2017 09:49:55 +0000 (11:49 +0200)]
Bug 8153: Enforce check-style rules for netconf - netconf-monitoring
Organize Imports for Checkstyle compliance.
Checkstyle compliance: line length.
Checkstyle compliance: various types of small changes.
Checkstyle compliant Exception handling.
Checkstyle final clean up & enforcement.
Add the fail on violation flag into the pom.xml .
Change-Id: If5e2448d5ae5a312bde0295ed52df9f8314afb6d
Signed-off-by: matus.kubica <matus.kubica@pantheon.tech>
Tomas Cere [Wed, 3 May 2017 11:25:07 +0000 (11:25 +0000)]
Merge "Bug 8153: Enforce check-style on messagebus-netconf"
matus.kubica [Wed, 3 May 2017 10:56:12 +0000 (12:56 +0200)]
Bug 8351: Enforce check-style rules for restconf - sal-rest-docgen
Organize Imports for Checkstyle compliance.
Checkstyle compliance: line length.
Checkstyle compliance: various types of small changes.
Checkstyle compliant Exception handling.
Checkstyle final clean up & enforcement.
Add the fail on violation flag into the pom.xml .
Change-Id: If4a4a3b17f8a0b93433fb41813f2560b334fd45c
Signed-off-by: matus.kubica <matus.kubica@pantheon.tech>
Jakub Morvay [Wed, 3 May 2017 08:55:55 +0000 (08:55 +0000)]
Merge "Do not use InetAddress.getByName()"