controller.git
9 years agoMerge "Adjust regenerated files to match what latest and greatest yang-tools will...
Ed Warnicke [Mon, 19 May 2014 12:29:24 +0000 (12:29 +0000)]
Merge "Adjust regenerated files to match what latest and greatest yang-tools will produce. This will avoid the annoying experience of having the following files shown as modified in git status."

9 years agoMerge "Store yang store snapshot cache using soft reference."
Ed Warnicke [Mon, 19 May 2014 12:28:20 +0000 (12:28 +0000)]
Merge "Store yang store snapshot cache using soft reference."

9 years agoMerge "Breaking up 6360: Adding base features"
Ed Warnicke [Mon, 19 May 2014 12:03:42 +0000 (12:03 +0000)]
Merge "Breaking up 6360: Adding base features"

9 years agoBug 992 - Fix broken netconf xml serialization. 93/6993/2
Tomas Olvecky [Wed, 14 May 2014 15:34:11 +0000 (17:34 +0200)]
Bug 992 - Fix broken netconf xml serialization.

Empty commit and restart triggered xml deserialization errors
in netconf. The problem is in type and name elements - they
should not be prefixed with 'prefix:'.

This fix triggers other bugs in md-sal modules, tracked
as dependencies of this bug. To test proper (de)serialization
with this fix, 01-netconf-connector.xml and 02-clustering.xml
must be delted.

Change-Id: I54d6ab23ec27ccbed924bf6abff95736f49dd5a3
Signed-off-by: Tomas Olvecky <tolvecky@cisco.com>
9 years agoMerge "Trigger a GC once initial configuration has been pushed"
Tony Tkacik [Mon, 19 May 2014 09:50:34 +0000 (09:50 +0000)]
Merge "Trigger a GC once initial configuration has been pushed"

9 years agoMerge "BUG-1027: rework lookup for clarity"
Tony Tkacik [Mon, 19 May 2014 09:28:43 +0000 (09:28 +0000)]
Merge "BUG-1027: rework lookup for clarity"

9 years agoMerge "BUG-1026: workaround race condition"
Tony Tkacik [Mon, 19 May 2014 09:25:07 +0000 (09:25 +0000)]
Merge "BUG-1026: workaround race condition"

9 years agoMerge "BUG-509: reorganize data tree abstractions."
Tony Tkacik [Mon, 19 May 2014 09:16:13 +0000 (09:16 +0000)]
Merge "BUG-509: reorganize data tree abstractions."

9 years agoBUG-1018 Implement BackwardsCompatible BI broker data notifications 14/7214/1
Maros Marsalek [Mon, 19 May 2014 09:10:14 +0000 (11:10 +0200)]
BUG-1018 Implement BackwardsCompatible BI broker data notifications

Finish support for notifying BI data change listeners in backwards compatible broker.

Change-Id: I221a1e6937c8f9c493444afd0756e6c23fd4b18f
Signed-off-by: Maros Marsalek <mmarsale@cisco.com>
9 years agoMerge "Bug 1029: Remove dead code: sal-binding-spi"
Tony Tkacik [Mon, 19 May 2014 09:08:48 +0000 (09:08 +0000)]
Merge "Bug 1029: Remove dead code: sal-binding-spi"

9 years agoMerge "Bug 1029: Remove dead code: sal-rest-connector-it"
Tony Tkacik [Mon, 19 May 2014 09:08:14 +0000 (09:08 +0000)]
Merge "Bug 1029: Remove dead code: sal-rest-connector-it"

9 years agoMerge "Bug 1029: Remove dead code: sal-dom-demo"
Tony Tkacik [Mon, 19 May 2014 09:07:59 +0000 (09:07 +0000)]
Merge "Bug 1029: Remove dead code: sal-dom-demo"

9 years agoMerge "Bug 1029: Remove dead code: sal-data-api"
Tony Tkacik [Mon, 19 May 2014 09:07:38 +0000 (09:07 +0000)]
Merge "Bug 1029: Remove dead code: sal-data-api"

9 years agoMerge "BUG-868: do not use InstanceIdentifier.getPath()"
Tony Tkacik [Mon, 19 May 2014 08:42:21 +0000 (08:42 +0000)]
Merge "BUG-868: do not use InstanceIdentifier.getPath()"

9 years agoMerge "Bug 1029: Remove dead code: clustered-data-store"
Tony Tkacik [Mon, 19 May 2014 08:12:16 +0000 (08:12 +0000)]
Merge "Bug 1029: Remove dead code: clustered-data-store"

9 years agoBUG-868: do not use InstanceIdentifier.getPath() 13/7213/1
Robert Varga [Mon, 19 May 2014 06:45:47 +0000 (08:45 +0200)]
BUG-868: do not use InstanceIdentifier.getPath()

This simplifies the NodeMapping class significantly by using the
utility methods shipped in InstanceIdentifier itself.

Change-Id: I1bb79bd4a16da9d78c600d21f67e3ca1926a0445
Signed-off-by: Robert Varga <rovarga@cisco.com>
9 years agoReplaced a couple of package imports with class imports 11/7211/1
Brent Salisbury [Mon, 19 May 2014 01:47:30 +0000 (21:47 -0400)]
Replaced a couple of package imports with class imports
in the L2switch sample bundle.

Change-Id: I2dbcf59e974cea99565ab254c528d4f7ea8dcec0
Signed-off-by: Brent Salisbury <brent.salisbury@gmail.com>
9 years agoBug 1029: Remove dead code: sal-binding-spi 08/7208/1
Ed Warnicke [Sun, 18 May 2014 23:01:43 +0000 (18:01 -0500)]
Bug 1029: Remove dead code: sal-binding-spi

opendaylight/md-sal/sal-binding-spi is dead code that is not being
built and hasn't for some time.  Its complicating keeping
versions up to date and verifying that they are up to date.
If it needs to be revived in the future it can be retrieved
from git.

Change-Id: I2698f4bfa687e24e65989c1257c5f80d84ebe262
Signed-off-by: Ed Warnicke <eaw@cisco.com>
9 years agoBug 1029: Remove dead code: sal-schema-repository-api 99/7199/1
Ed Warnicke [Sun, 18 May 2014 20:41:32 +0000 (15:41 -0500)]
Bug 1029: Remove dead code: sal-schema-repository-api

opendaylight/md-sal/sal-schema-repository-api is dead code that is not being
built and hasn't for some time.  Its complicating keeping
versions up to date and verifying that they are up to date.

If it needs to be revived in the future it can be retrieved
from git.

Change-Id: Iad92d3a9486b6b20e898bf96ad78e3b941a30a3e
Signed-off-by: Ed Warnicke <eaw@cisco.com>
9 years agoBug 1029: Remove dead code: sal-dom-it 98/7198/1
Ed Warnicke [Sun, 18 May 2014 20:39:26 +0000 (15:39 -0500)]
Bug 1029: Remove dead code: sal-dom-it

opendaylight/md-sal/sal-dom-it/ is dead code that is not being
built and hasn't for some time.  Its complicating keeping
versions up to date and verifying that they are up to date.

If it needs to be revived in the future it can be retrieved
from git.

Change-Id: Id7f32c333c0407130203991fb37ce128abea3e33
Signed-off-by: Ed Warnicke <eaw@cisco.com>
9 years agoBug 1029: Remove dead code: sal-rest-connector-it 93/7193/1
Ed Warnicke [Sun, 18 May 2014 18:00:33 +0000 (13:00 -0500)]
Bug 1029: Remove dead code: sal-rest-connector-it

opendaylight/md-sal/test/sal-rest-connector-it is dead code that is not being
built and hasn't for some time.  Its complicating keeping
versions up to date and verifying that they are up to date.

If it needs to be revived in the future it can be retrieved
from git.

Change-Id: I2ab837c661552022792ff7f02a1c2b68f7928501
Signed-off-by: Ed Warnicke <eaw@cisco.com>
9 years agoBug 1029: Remove dead code: sal-dom-demo 92/7192/1
Ed Warnicke [Sun, 18 May 2014 17:59:02 +0000 (12:59 -0500)]
Bug 1029: Remove dead code: sal-dom-demo

opendaylight/md-sal/sal-dom-demo is dead code that is not being
built and hasn't for some time.  Its complicating keeping
versions up to date and verifying that they are up to date.

If it needs to be revived in the future it can be retrieved
from git.

Change-Id: I77357764b8a779c444ed173e9e8a29a227ba3bf7
Signed-off-by: Ed Warnicke <eaw@cisco.com>
9 years agoBug 1029: Remove dead code: sal-data-api 91/7191/1
Ed Warnicke [Sun, 18 May 2014 17:57:04 +0000 (12:57 -0500)]
Bug 1029: Remove dead code: sal-data-api

opendaylight/md-sal/sal-data-api is dead code that is not being
built and hasn't for some time.  Its complicating keeping
versions up to date and verifying that they are up to date.

If it needs to be revived in the future it can be retrieved
from git.

Change-Id: I28d9a8f63ff0398766d0d8c29f18a27d75642d2e
Signed-off-by: Ed Warnicke <eaw@cisco.com>
9 years agoBug 1029: Remove dead code: clustered-data-store 90/7190/1
Ed Warnicke [Sun, 18 May 2014 17:50:40 +0000 (12:50 -0500)]
Bug 1029: Remove dead code: clustered-data-store

opendaylight/md-sal/clustered-data-store/ is dead code that is not being
built and hasn't for some time.  Its complicating keeping
versions up to date and verifying that they are up to date.

If in the future it needs to be revived it can be retrieved from
git.

Change-Id: Ie8365d5cb48e806a9d9a9fc0175478ef21277cbd
Signed-off-by: Ed Warnicke <eaw@cisco.com>
9 years agoBUG-509: reorganize data tree abstractions. 07/7107/2
Robert Varga [Thu, 15 May 2014 13:14:03 +0000 (15:14 +0200)]
BUG-509: reorganize data tree abstractions.

This patch has no functional changes, it only rearranges code such that
the flow of data and object lifecycle becomes more obvious. This will
make metadata lifecycle easier to manage.

Change-Id: I10fd912433b72e25729d163c27699a5a5c47e305
Signed-off-by: Robert Varga <rovarga@cisco.com>
9 years agoBUG-1027: rework lookup for clarity 53/7153/1
Robert Varga [Sun, 18 May 2014 10:10:21 +0000 (12:10 +0200)]
BUG-1027: rework lookup for clarity

This patch gets rid of the nasty 'catch Exception', making the lookup
fail as soon as a DeserializationException is thrown. We account for
most of the checked exceptions, so this should not be a problem.

Change-Id: I3d0440933ebc30ba32a06690ebe326e18addf613
Signed-off-by: Robert Varga <rovarga@cisco.com>
9 years agoBUG-1026: workaround race condition 49/7149/1
Robert Varga [Sun, 18 May 2014 08:19:27 +0000 (10:19 +0200)]
BUG-1026: workaround race condition

This patch provides a workaround: move the netconf-connector config past
any other configuration that may be present, such that it initializes
last. This will ensure that any models referenced have been instantiated
by the time the mount instance is announced.

Change-Id: Iba3587806932c307474e708209a39cba1d63ee5d
Signed-off-by: Robert Varga <rovarga@cisco.com>
9 years agoChange in sal.connectionservice.notifyNodeDisconnectFromMaster 39/7139/1
Alessandro Boch [Sat, 17 May 2014 20:40:14 +0000 (13:40 -0700)]
Change in sal.connectionservice.notifyNodeDisconnectFromMaster

- On node disconnect from Master, notify the respective protocol plugin, not all of them

Change-Id: I2bfb30d2f649a1bc3f07be60150a814bcd1b8a45
Signed-off-by: Alessandro Boch <aboch@cisco.com>
9 years agoMerge "Bug 967: Do not translate leaf Instance Identifiers."
Ed Warnicke [Fri, 16 May 2014 15:15:24 +0000 (15:15 +0000)]
Merge "Bug 967: Do not translate leaf Instance Identifiers."

9 years agoBug 967: Do not translate leaf Instance Identifiers. 11/7111/1
Tony Tkacik [Fri, 16 May 2014 11:02:33 +0000 (13:02 +0200)]
Bug 967: Do not translate leaf Instance Identifiers.

Improved extraction of Instance Identifiers from
DOM Data Change Event.

Logs in console are poluted with DeserializationException,
which fails to deserialize DOM Instance Identifier
to Binding Instance Identifier.

Added explicit check against schema context to
see if Instance Identifier is representable
in Binding Format, if not we will not report
is as part of subtree Data Change event.

Change-Id: Iaee2390fd089e0a2d66ec658aab4e7dab2a93a46
Signed-off-by: Tony Tkacik <ttkacik@cisco.com>
9 years agoBug 1022 - Add ability to lookup dependent Module's attribute. 10/7110/1
Tomas Olvecky [Fri, 16 May 2014 11:24:49 +0000 (13:24 +0200)]
Bug 1022 - Add ability to lookup dependent Module's attribute.

Add 2 methods to DependencyResolver to allow modules look up values
of dependent bean attributes.

Change-Id: I2ebee82cc1260a84a16f5e69b32c9234bf576e87
Signed-off-by: Tomas Olvecky <tolvecky@cisco.com>
9 years agoBug 624 - Separate netty and exi specific classes from netconf-util. 31/6931/2
Tomas Olvecky [Tue, 13 May 2014 13:26:28 +0000 (15:26 +0200)]
Bug 624 - Separate netty and exi specific classes from netconf-util.

As preparation step for splitting netconf into service and protocol,
create netconf-netty-util bundle with classes dependent on protocol
handlers - netty, exi, ssh.

Change-Id: I22ba17d0fa4f86c5f64cb3258a55f2c54da85dee
Signed-off-by: Tomas Olvecky <tolvecky@cisco.com>
9 years agoMerge "Add lispflowmapping specific configuration options"
Giovanni Meo [Fri, 16 May 2014 08:24:09 +0000 (08:24 +0000)]
Merge "Add lispflowmapping specific configuration options"

9 years agoMerge "Node Name in non-default container should default to Node Name in Default...
Andrew Kim [Thu, 15 May 2014 22:00:43 +0000 (22:00 +0000)]
Merge "Node Name in non-default container should default to Node Name in Default Slice"

9 years agoNode Name in non-default container should default to Node Name in Default Slice 04/7004/2
Pramila Singh [Wed, 14 May 2014 22:35:13 +0000 (15:35 -0700)]
Node Name in non-default container should default to Node Name in Default Slice

Change-Id: I81670c824bfcb4d7f400ef224abf329b6bc20abd
Signed-off-by: Pramila Singh <pramisin@cisco.com>
9 years agoBug 953 - Enable mounted RPC calls via RestConf 15/6815/15
Devin Avery [Thu, 8 May 2014 13:00:29 +0000 (09:00 -0400)]
Bug 953 - Enable mounted RPC calls via RestConf

Modify RestconfImpl to allow RPC calls via mount points.
===
MountPointImpl - wired pass through call for invokeRpc that was missing.
RestconfImpl - abstract the execution of RPC out to an "RpcExecutor" class.
Remaining classes - added tests to cover new code, including the toaster.yang file (modified from original) to test various RPC calls.

- Patch Set 8 - fixed unit tests which were failing (one failure due to missing future enhancement, another due to missing test rpc in yang file).
- Patch Set 10 - fixed run failure by addressing package import.
- Patch Set 13 - Cleaned up exception throws / used Preconditions checking.
- Patch Set 14 - Rebased
- Patch Set 15 - Added back change that was accidently removed during merge / rebase.

Change-Id: Ib214b4c82ffc487ee2f7b912e65d5c189b82154a
Signed-off-by: Devin Avery <devin.avery@brocade.com>
9 years agoMerge "Bug 509: Improve conflict detection of leaf merges"
Tony Tkacik [Thu, 15 May 2014 15:50:38 +0000 (15:50 +0000)]
Merge "Bug 509: Improve conflict detection of leaf merges"

9 years agoMerge changes Ibe0145be,I0764bfa9
Tony Tkacik [Thu, 15 May 2014 15:50:08 +0000 (15:50 +0000)]
Merge changes Ibe0145be,I0764bfa9

* changes:
  BUG-509: remove DataAndMetadataSnapshot.Builder.getDataTree()
  BUG-509: remove StoreMetadataNode.children()

9 years agoMerge "BUG-509: remove StoreUtils.initialChangeEvent()"
Tony Tkacik [Thu, 15 May 2014 15:49:44 +0000 (15:49 +0000)]
Merge "BUG-509: remove StoreUtils.initialChangeEvent()"

9 years agoMerge "BUG-509: StoreUtils should operate on NormalizedNode"
Tony Tkacik [Thu, 15 May 2014 15:49:11 +0000 (15:49 +0000)]
Merge "BUG-509: StoreUtils should operate on NormalizedNode"

9 years agoBug 509: Improve conflict detection of leaf merges 22/7022/1
Robert Varga [Thu, 15 May 2014 14:55:02 +0000 (16:55 +0200)]
Bug 509: Improve conflict detection of leaf merges

Original conflict detection for merge action on leaves
only looked at node version and subtree version, which
caused the transaction to fail even if the leaf was
unmodified and contained the same value.

Conflict detection now checks if the leaves are not same,
and only in that case it checks for concurrent writes.

Change-Id: I5e43233e86adf96e3dad2a93cdcf512b3d17977a
Signed-off-by: Tony Tkacik <ttkacik@cisco.com>
Signed-off-by: Robert Varga <rovarga@cisco.com>
9 years agoBUG-509: remove DataAndMetadataSnapshot.Builder.getDataTree() 17/7017/1
Robert Varga [Thu, 15 May 2014 12:37:19 +0000 (14:37 +0200)]
BUG-509: remove DataAndMetadataSnapshot.Builder.getDataTree()

This getter and its associated field is completely unused. Remove it to
improve clarity.

Change-Id: Ibe0145bee610c7a2b147852f43d0492ca5061adc
Signed-off-by: Robert Varga <rovarga@cisco.com>
9 years agoMerge "Bug 1002 - REST POST transaction is not commited"
Tony Tkacik [Thu, 15 May 2014 12:02:38 +0000 (12:02 +0000)]
Merge "Bug 1002 - REST POST transaction is not commited"

9 years agoMerge "Cosmetics: fix typo facilitate"
Ed Warnicke [Thu, 15 May 2014 11:12:31 +0000 (11:12 +0000)]
Merge "Cosmetics: fix typo facilitate"

9 years agoBug 1002 - REST POST transaction is not commited 90/6990/4
Vaclav Demcak [Wed, 14 May 2014 12:39:10 +0000 (14:39 +0200)]
Bug 1002 - REST POST transaction is not commited

fix REST POST create workflow (create data only), fix according to specification
  http://tools.ietf.org/html/draft-bierman-netconf-restconf-03#section-2.4

I'm missing implementation for ietf-restconf:errors, so I can't do more for now
  http://tools.ietf.org/html/draft-bierman-netconf-restconf-03#page-48

* fix a check for an existing node in DataStore
* add warn log and a correct ResponseException
* fix tests for BrokerFacade

Change-Id: I68c95e831f42cae4df0608a40c585251a631693f
Signed-off-by: Vaclav Demcak <vdemcak@cisco.com>
9 years agoBUG-509: remove StoreMetadataNode.children() 15/7015/1
Robert Varga [Thu, 15 May 2014 09:10:36 +0000 (11:10 +0200)]
BUG-509: remove StoreMetadataNode.children()

This patch removes the now-unneeded access, which means the children
iteration order is no longer part of StoreMetadataNode, allowing us to
use more memory-efficient HashMap instead of LinkedHashMap. Let's do
precisely that.

Change-Id: I0764bfa9861f9df4d3014151db96edf1024e3f9b
Signed-off-by: Robert Varga <rovarga@cisco.com>
9 years agoBUG-509: remove StoreUtils.initialChangeEvent() 14/7014/1
Robert Varga [Thu, 15 May 2014 08:56:48 +0000 (10:56 +0200)]
BUG-509: remove StoreUtils.initialChangeEvent()

This method is unused. Remove it, such that StoreUtils are now
completely independent of org.opendaylight.controller.md.sal, making
them a candidate for move to yang-data-*.

Change-Id: Idf945aee6fd4a80f8ca58fc50bcc997880a60ec0
Signed-off-by: Robert Varga <rovarga@cisco.com>
9 years agoBUG-509: StoreUtils should operate on NormalizedNode 13/7013/1
Robert Varga [Thu, 15 May 2014 08:47:00 +0000 (10:47 +0200)]
BUG-509: StoreUtils should operate on NormalizedNode

This prunes the use of MetadataNode from StoreUtils. It turns out it is
completely unnecessary. This removes the last user of
Metadata.getChildren(), which will be removed in a subsequent patch.

Change-Id: I778e77b826576b8fa53e040a825d615dab7d253f
Signed-off-by: Robert Varga <rovarga@cisco.com>
9 years agoCommit generated test files after copyright was removed. 12/7012/1
Tomas Olvecky [Thu, 15 May 2014 07:54:26 +0000 (09:54 +0200)]
Commit generated test files after copyright was removed.

This prevents from changing files during build.

Change-Id: I1319d5f01d0c532cae2a680f59f5d76f3128da8a
Signed-off-by: Tomas Olvecky <tolvecky@cisco.com>
9 years agoTrigger a GC once initial configuration has been pushed 03/7003/1
Robert Varga [Wed, 14 May 2014 22:04:17 +0000 (00:04 +0200)]
Trigger a GC once initial configuration has been pushed

This introduces a single run of GC once we complete the initial
configuration.

The reasoning behind that is that we generate a fair amount of metadata
which we are likely to retain unmodified for extended periods of time.
We take care to store it in both memory and CPU efficient format, which
unfortunately requires us to do some copying around to compact it,
leaving a sizeable chunk of garbage on the heap.

Change-Id: I81af7d317e3b6f82a431e78b5fb439d8cb5f1997
Signed-off-by: Robert Varga <rovarga@cisco.com>
9 years agoBug 967: Fixed incorrect delete was addressing augmentation of augmentation 82/6782/2
Tony Tkacik [Wed, 7 May 2014 11:10:40 +0000 (13:10 +0200)]
Bug 967: Fixed incorrect delete was addressing  augmentation of augmentation

DataNormalizationOperation was constructed incorrectly, if augmentation
was already augmented, which leaded to incorrect construction of instance
identifier for delete.

Patchset 2: Fixed normalization in cases of mixin inside mixin
e.g. augmentation containing choice with cases.

Change-Id: I580e5dfe41ce504775ac454b66368037f4d707bb
Signed-off-by: Tony Tkacik <ttkacik@cisco.com>
9 years agoMerge "BUG 1001 Failing test - RestGetOperationTest.getStreamsTest"
Tony Tkacik [Wed, 14 May 2014 10:16:45 +0000 (10:16 +0000)]
Merge "BUG 1001 Failing test - RestGetOperationTest.getStreamsTest"

9 years agoMerge "Add Pagination to Neutron Networks Northbound API"
Giovanni Meo [Wed, 14 May 2014 09:45:10 +0000 (09:45 +0000)]
Merge "Add Pagination to Neutron Networks Northbound API"

9 years agoMerge "BUG-372 Add reconnecting to netconf-connector for unexpected session drop"
Tony Tkacik [Wed, 14 May 2014 09:02:00 +0000 (09:02 +0000)]
Merge "BUG-372 Add reconnecting to netconf-connector for unexpected session drop"

9 years agoBUG 1001 Failing test - RestGetOperationTest.getStreamsTest 84/6984/1
Jozef Gloncak [Wed, 14 May 2014 08:23:24 +0000 (10:23 +0200)]
BUG 1001 Failing test - RestGetOperationTest.getStreamsTest

responseBody variable now contained some data and therefore it wasn't xml
tag <streams/> but <streams></streams>

Change-Id: I73004f848f0dfc5f2929f8f8ea2dbd6afc25d35a
Signed-off-by: Jozef Gloncak <jgloncak@cisco.com>
9 years agoAdjust regenerated files to match what latest and greatest yang-tools will produce. 78/6978/1
Flavio Fernandes [Wed, 14 May 2014 03:26:40 +0000 (23:26 -0400)]
Adjust regenerated files to match what latest and greatest yang-tools will produce.
This will avoid the annoying experience of having the following files shown as modified
in git status.

Change-Id: I581de6082164ef1b7368e0487c4244869bb83e93
Signed-off-by: Flavio Fernandes <ffernand@redhat.com>
9 years agoMerge "REST API to fetch Node Property"
Andrew Kim [Tue, 13 May 2014 18:30:11 +0000 (18:30 +0000)]
Merge "REST API to fetch Node Property"

9 years agoMerge "Bug 617: Remove extend files from sal-rest-connector"
Tony Tkacik [Tue, 13 May 2014 15:02:41 +0000 (15:02 +0000)]
Merge "Bug 617: Remove extend files from sal-rest-connector"

9 years agoBUG-372 Add reconnecting to netconf-connector for unexpected session drop 34/6934/1
Maros Marsalek [Tue, 13 May 2014 14:10:59 +0000 (16:10 +0200)]
BUG-372 Add reconnecting to netconf-connector for unexpected session drop

Netconf connector will reattempt connection if the session to remote device is dropped.
Increase sleep factor in netconf connector from 1.0 to 1.5.
This prevents connection attempt every second in case remote device is not comming back for longer time.
Connection attempts span increases gradually.

Change-Id: Icc4b993942e9cc6897c0da858eb867afa4c8aa87
Signed-off-by: Maros Marsalek <mmarsale@cisco.com>
9 years agoMerge "Bug 809: Enhancements to the toaster example"
Tony Tkacik [Tue, 13 May 2014 13:52:24 +0000 (13:52 +0000)]
Merge "Bug 809: Enhancements to the toaster example"

9 years agoBUG-981: Fixed incorrect condition in ResolveDataChangeEventsTask 29/6929/1
Robert Varga [Tue, 13 May 2014 11:52:10 +0000 (13:52 +0200)]
BUG-981: Fixed incorrect condition in ResolveDataChangeEventsTask

When we insert a new subtree, we need to make sure to call serialization
on the entire, such that any new elements which appear have their
classed attached to paths.

This reuses a previous code path, which pruned the walk tree as soon as
it encountered a node which did not have any listeners attached (thus
improving performance when we're gathering listeners).

In this specific case, though, this check is not correct: we need to
walk the path because it may be placing new data in the subtree.

At the same time add debug statements which made pinpointing this
problem possible.

Change-Id: I1687ba45817a84ea4a888496a90defc5896216c9
Signed-off-by: Tony Tkacik <ttkacik@cisco.com>
Signed-off-by: Robert Varga <rovarga@cisco.com>
9 years agoREST API to fetch Node Property 15/6915/1
Sapan Shah [Tue, 13 May 2014 00:06:28 +0000 (17:06 -0700)]
REST API to fetch Node Property

Change-Id: Iad693deefcc3a3f5420e8e19ddb81919c788738f
Signed-off-by: Sapan Shah <sapshah@cisco.com>
9 years agoMerge "Bug-574 Added a 4th RPC Invocation strategy for the case where the RPC has...
Tony Tkacik [Mon, 12 May 2014 18:48:31 +0000 (18:48 +0000)]
Merge "Bug-574 Added a 4th RPC Invocation strategy for the case where the RPC has an output, but no input"

9 years agoAdded .DS_STORE and .metadata to .gitignore 09/6909/1
Ed Warnicke [Mon, 12 May 2014 17:48:42 +0000 (12:48 -0500)]
Added .DS_STORE and .metadata to .gitignore

Change-Id: I95a89d26593064d8a471a8cc7bd6c48922109ece
Signed-off-by: Ed Warnicke <eaw@cisco.com>
9 years agoMerge "BUG-704 Remove pax from netconf identity-ref test."
Tony Tkacik [Mon, 12 May 2014 16:35:07 +0000 (16:35 +0000)]
Merge "BUG-704 Remove pax from netconf identity-ref test."

9 years agoBug-574 Added a 4th RPC Invocation strategy for the case where the RPC has an output... 90/6190/2
Mayank Agarwal [Mon, 14 Apr 2014 21:58:31 +0000 (14:58 -0700)]
Bug-574 Added a 4th RPC Invocation strategy for the case where the RPC has an output, but no input

Change-Id: I7d1a596f3bb1362044cc240a15bdb9bb7d094c55
Signed-off-by: Mayank Agarwal <mayagarw@cisco.com>
9 years agoMerge "Bug 508: Improved error reporting for failed canCommit phase."
Ed Warnicke [Mon, 12 May 2014 14:41:49 +0000 (14:41 +0000)]
Merge "Bug 508: Improved error reporting for failed canCommit phase."

9 years agoMerge "Bug 981: Fixed accidental shaddowing of data in Data Change Event."
Giovanni Meo [Mon, 12 May 2014 13:53:28 +0000 (13:53 +0000)]
Merge "Bug 981: Fixed accidental shaddowing of data in Data Change Event."

9 years agoStore yang store snapshot cache using soft reference. 93/6893/1
Tomas Olvecky [Mon, 12 May 2014 12:58:53 +0000 (14:58 +0200)]
Store yang store snapshot cache using soft reference.

Change-Id: I9b159db83ba204b4a636f2314fd4fc2e7b6f654c
Signed-off-by: Tomas Olvecky <tolvecky@cisco.com>
9 years agoBug 508: Improved error reporting for failed canCommit phase. 91/6891/3
Tony Tkacik [Mon, 12 May 2014 10:33:58 +0000 (12:33 +0200)]
Bug 508: Improved error reporting for failed canCommit phase.

Original implementation did not provide enough information
when transaction precondition failed.

Introduced new Exception DataPreconditionFailedException
which captures fail reason and path to subtree
for which precondition failed.

This helps debug accidental overwrites / conflicting
writes between transactions.

Change-Id: I0133fd3fdb8e6492bf457dab7b20efa67fd6e2af
Signed-off-by: Tony Tkacik <ttkacik@cisco.com>
9 years agoMerge "Bug 849: Fixed NPE in Translated Data Change Events."
Giovanni Meo [Mon, 12 May 2014 11:05:43 +0000 (11:05 +0000)]
Merge "Bug 849: Fixed NPE in Translated Data Change Events."

9 years agoBUG-704 Remove pax from netconf identity-ref test. 34/6834/2
Maros Marsalek [Fri, 9 May 2014 12:58:08 +0000 (14:58 +0200)]
BUG-704 Remove pax from netconf identity-ref test.

Make this test a regular unit test without pax.

Change-Id: I0f9767d860f25bb60494ded2011e3f398fbcf43f
Signed-off-by: Maros Marsalek <mmarsale@cisco.com>
9 years agoBug 981: Fixed accidental shaddowing of data in Data Change Event. 83/6883/4
Tony Tkacik [Sun, 11 May 2014 20:46:24 +0000 (22:46 +0200)]
Bug 981: Fixed accidental shaddowing of data in Data Change Event.

Not all Normalized DOM InstanceIdentifier has their counterpart
in Binding-Aware representation. Added explicit check to
detect mixin instance-identifiers which could lead to
incorrect deserialization into maps and shadowing parent data.

Change-Id: I66166f50c636ce0dfcd962efac7fc488da622336
Signed-off-by: Tony Tkacik <ttkacik@cisco.com>
9 years agoMerge "BUG-648: Improve performance by reusing old data nodes"
Tony Tkacik [Mon, 12 May 2014 07:05:37 +0000 (07:05 +0000)]
Merge "BUG-648: Improve performance by reusing old data nodes"

9 years agoMerge "BUG-980: stop emiting copyright"
Tony Tkacik [Mon, 12 May 2014 07:04:12 +0000 (07:04 +0000)]
Merge "BUG-980: stop emiting copyright"

9 years agoMerge "BUG-731: RoutedRegistration.close() should not throw"
Tony Tkacik [Mon, 12 May 2014 07:03:49 +0000 (07:03 +0000)]
Merge "BUG-731: RoutedRegistration.close() should not throw"

9 years agoMerge "Remove overriding versions"
Tony Tkacik [Mon, 12 May 2014 07:03:30 +0000 (07:03 +0000)]
Merge "Remove overriding versions"

9 years agoMerge "Unify maven-bundle-plugin version at 2.4.0"
Tony Tkacik [Mon, 12 May 2014 07:03:15 +0000 (07:03 +0000)]
Merge "Unify maven-bundle-plugin version at 2.4.0"

9 years agoMerge "BUG-972: correct Precondition"
Tony Tkacik [Mon, 12 May 2014 07:02:55 +0000 (07:02 +0000)]
Merge "BUG-972: correct Precondition"

9 years agoMerge "Add serialVersionUID to Serializable"
Tony Tkacik [Mon, 12 May 2014 07:02:33 +0000 (07:02 +0000)]
Merge "Add serialVersionUID to Serializable"

9 years agoMerge "BUG-868: stop using InstanceIdentifier.builder()"
Tony Tkacik [Mon, 12 May 2014 07:02:17 +0000 (07:02 +0000)]
Merge "BUG-868: stop using InstanceIdentifier.builder()"

9 years agoBUG-648: Improve performance by reusing old data nodes 86/6886/2
Robert Varga [Thu, 8 May 2014 16:29:30 +0000 (18:29 +0200)]
BUG-648: Improve performance by reusing old data nodes

This patchset gives yang-data-api the chance to optimize copy
operations we execute when creating a new generation.

Change-Id: Ib23a87e116a0a04f7fd4f2b709d7239e213ea744
Signed-off-by: Robert Varga <rovarga@cisco.com>
9 years agoAdd serialVersionUID to Serializable 70/6870/2
Robert Varga [Sun, 11 May 2014 07:16:29 +0000 (09:16 +0200)]
Add serialVersionUID to Serializable

Serializable classes have to have serialVersionUID. Add it where it's
missing.

Change-Id: I23689005878280b6ee9f97fb12ae0b15151f20d7
Signed-off-by: Robert Varga <rovarga@cisco.com>
9 years agoBUG-868: stop using InstanceIdentifier.builder() 69/6869/3
Robert Varga [Sun, 11 May 2014 06:41:18 +0000 (08:41 +0200)]
BUG-868: stop using InstanceIdentifier.builder()

The static helper method has been deprecated. Use either the non-static
one, or child()/augmentation() helpers.

Change-Id: Ic69057d2db669153dde60ba1797e8139623a755c
Signed-off-by: Robert Varga <rovarga@cisco.com>
9 years agoBUG-981: remove deprecated use of constructors 68/6868/2
Robert Varga [Sun, 11 May 2014 05:35:34 +0000 (07:35 +0200)]
BUG-981: remove deprecated use of constructors

This removes the use of all constructors, such that we initialize in one
go.

Change-Id: Iff461a878fb7061b0ea1c8efea6a34b833645784
Signed-off-by: Robert Varga <rovarga@cisco.com>
9 years agoMerge "BUG-893: do not store classPool"
Tony Tkacik [Sun, 11 May 2014 16:46:53 +0000 (16:46 +0000)]
Merge "BUG-893: do not store classPool"

9 years agoMerge "BUG-642: get rid of sneakyThrow"
Tony Tkacik [Sun, 11 May 2014 16:46:25 +0000 (16:46 +0000)]
Merge "BUG-642: get rid of sneakyThrow"

9 years agoCosmetics: fix typo facilitate 66/6866/1
Flavio Fernandes [Sun, 11 May 2014 03:27:02 +0000 (23:27 -0400)]
Cosmetics: fix typo facilitate

Change-Id: I351e285dd128f1d633f4940624096348b065f16a
Signed-off-by: Flavio Fernandes <ffernand@redhat.com>
9 years agoBug 716: Errors on controller shutdown 54/6554/5
tpantelis [Tue, 22 Apr 2014 16:05:23 +0000 (12:05 -0400)]
Bug 716: Errors on controller shutdown

    Fixed miscellaneous errors on shutdown:
        -  FlowProvider, GroupProvider and MeterProvider: all throw an
           UnsupportedOperationEx from their close methods with a TODO message.
           Implemented close to close the commitHandlerRegistration.

        - NetconfSSHServer: logs a socket closed exception due to closing the socket
          on shutdown. We can ignore this and not log the error (if 'up' is false).

        - RestconfProvider: NPE in stop method calling session.close.
          The 'session' member is never initialized or otherwise used so removed it.

        - RuntimeMappingModule and SchemaServiceImplSingletonModule:
          IllegalStateException from bundleContext.ungetService() call because the
          bundle has already been shutdown. Ignore ex as this can occur
          normally.

Need someone to commit this!

Change-Id: I31d9d6d66418dda7f5b73c2ad12bb251f3689643
Signed-off-by: tpantelis <tpanteli@brocade.com>
9 years agoBug 935 Fixed, added order key in instruction and action as flow yang model expects it. 39/6639/2
Amit Mandke [Thu, 1 May 2014 20:48:55 +0000 (13:48 -0700)]
Bug 935 Fixed, added order key in instruction and action as flow yang model expects it.

Change-Id: Ib7a82db71f4915496574ec2b8731cb0161351309
Signed-off-by: Amit Mandke <ammandke@cisco.com>
9 years agoBUG-893: do not store classPool 55/6855/1
Robert Varga [Sat, 10 May 2014 13:43:45 +0000 (15:43 +0200)]
BUG-893: do not store classPool

This prevents storing a reference to the classPool. It is not necessary
and it makes thinking about accesses much easier.

Change-Id: Iebbdac46c3ca445bbe36ba6efd19d3150d728f7f
Signed-off-by: Robert Varga <rovarga@cisco.com>
9 years agoBUG-642: get rid of sneakyThrow 53/6853/1
Robert Varga [Sat, 10 May 2014 11:25:21 +0000 (13:25 +0200)]
BUG-642: get rid of sneakyThrow

This patch rids us of unneeded usage of Exceptions.sneakyThrow. It
generally hides problems and is only useful for generated (or sloppy)
code.

Change-Id: Ibb73f4546b1785c4bcb3e6c033249f58efd1d2d6
Signed-off-by: Robert Varga <rovarga@cisco.com>
9 years agoBUG-980: stop emiting copyright 27/6827/3
Robert Varga [Fri, 9 May 2014 07:33:44 +0000 (09:33 +0200)]
BUG-980: stop emiting copyright

This is a stop-gap quickfix to stop doing the wrong thing.

Change-Id: I51a2ca012ea605632a737788005cf4143d9c1f44
Signed-off-by: Robert Varga <rovarga@cisco.com>
9 years agoBUG-891: do not drop events silently 36/6836/1
Robert Varga [Fri, 9 May 2014 15:22:19 +0000 (17:22 +0200)]
BUG-891: do not drop events silently

As it turns out, infrastructure silently drops things on the floor if
exceptions happen. When these exceptions occur, they hint at major
problems. Promote tham to warnings.

Change-Id: Ib4afdc7ff1a17e562211b3cfbb30735a592d0a2d
Signed-off-by: Robert Varga <rovarga@cisco.com>
9 years agoBUG-731: RoutedRegistration.close() should not throw 30/6830/1
Robert Varga [Fri, 9 May 2014 10:16:50 +0000 (12:16 +0200)]
BUG-731: RoutedRegistration.close() should not throw

Routed RPC implementations should be allowed to unregister themselves
without incurring a checked exceptions. This allows downstream users to
elide having to deal with raw Exception.

Change-Id: Ia356c6272da3d22a375c954e192f79786ab42395
Signed-off-by: Robert Varga <rovarga@cisco.com>
9 years agoMerge "BUG-969 Move fake read on nodes/node to separate thread."
Tony Tkacik [Fri, 9 May 2014 08:06:58 +0000 (08:06 +0000)]
Merge "BUG-969 Move fake read on nodes/node to separate thread."

9 years agoMerge "Making it easier for cygwin users to run OpenDaylight using run.sh. With this...
Giovanni Meo [Fri, 9 May 2014 07:45:34 +0000 (07:45 +0000)]
Merge "Making it easier for cygwin users to run OpenDaylight using run.sh. With this change, the users will only have to add the following line to their .bashrc to make it run: export CLASSPATH=".;lib/*;""

9 years agoBUG-969 Move fake read on nodes/node to separate thread. 26/6826/1
Maros Marsalek [Fri, 9 May 2014 07:27:55 +0000 (09:27 +0200)]
BUG-969 Move fake read on nodes/node to separate thread.

This will prevent blocking of config thread.

Change-Id: Ieb2e1c192ef91775153837956e6e6a428985c2d1
Signed-off-by: Maros Marsalek <mmarsale@cisco.com>
9 years agoBug 809: Enhancements to the toaster example 79/6379/7
tpantelis [Fri, 18 Apr 2014 19:10:47 +0000 (15:10 -0400)]
Bug 809: Enhancements to the toaster example

See https://bugs.opendaylight.org/show_bug.cgi?id=809 for details.

Need someone to commit this!

Change-Id: Icb9697de77d844d816ec61a0e7a6ac2100d185ae
Signed-off-by: tpantelis <tpanteli@brocade.com>
9 years agoBreaking up 6360: Adding base features 30/6730/8
Mathieu Lemay [Tue, 6 May 2014 23:50:35 +0000 (19:50 -0400)]
Breaking up 6360: Adding base features

https://git.opendaylight.org/gerrit/#/c/6360/29

Change-Id: I1df1bd970d184b3c363a94b68a608a8e2ed734ef
Signed-off-by: Ed Warnicke <eaw@cisco.com>
Signed-off-by: Mathieu Lemay <mlemay@inocybe.com>