Merge "BUG-362: add some diagnostic information Changed Remote RPC Server Implementation to log an error when host IP address is not found and stop itself. Earlier it would raise exception that the configuration subsystem could not handle."

Merge "BUG 408"

Merge "Fix incorrect validation in SwitchConfig"

Fix incorrect validation in SwitchConfig

Change-Id: Ie9267b97b8742f9bd5ebbd85a4f0dc05bf1cebd9
Signed-off-by: Alessandro Boch <aboch@cisco.com>

Merge "Lower logger level in OF1.0 Discovery Service"

BUG 408

- added null safe comparison
- added unit test
- added SLF4j
- added log message in method StrIpToIntIp in case of exception

Change-Id: Id15dc4f80d419df5e73ab79fea4e8ab25ec82968
Signed-off-by: Michal Rehak <mirehak@cisco.com>

operation, store, path and data properties added to Data change event model

Change-Id: I0b1c39e23506988528614b33c4a49cf8e00802ea
Signed-off-by: Martin Bobak <mbobak@cisco.com>

Merge "Fixed publishDataChangeEvent in 2phase commit"

Merge "Increase timeout for waiting for broker service in sal-binding-it."

Merge "Remove unnecessary warn log from config subsystem"

Merge "Fix for bug #236 and bug #240 Have made changes in opendaylight-table-types.yang"

Merge "Improve netconf negotiator messages"

Merge "Improve logging in NetconfClient, logging-bridge. Increase connection timeout in sal-netconf-connector."

Merge "Maven archetype for config subsystem aware bundles"

Merge "Fix of bug in Notificator class"

Merge "Fix Plugin execution not covered in Eclipse for config subsystem"

Lower logger level in OF1.0 Discovery Service

- Discovery Service has a function to attempt to snoop
  non controller generated or malformed LLDP frames.
  The whole function block is in a try catch block,
  as exceptions are well expected.
  In case an exception is hit though, the function logs
  it as a warn.
- Change the log level to debug, do not print a stack trace,
  print the source MAC and the port on which it was received.

Change-Id: I82b459b16356b0a57934b3c26c993d20a2b53dc2
Signed-off-by: Alessandro Boch <aboch@cisco.com>

Improve netconf negotiator messages

Change-Id: I8b36f2fa6173055e5fc34783873a55cdf240589d
Signed-off-by: Robert Varga <rovarga@cisco.com>

Merge "Requestor FRM to update the sw views on remote execution"

Merge "Minor change to resource name regex"

Merge "Fix for BUG 402: Subnet creation with NeutronAPIService sometimes fails with a 409 error"

Minor change to resource name regex

Change-Id: Ia087f56b7fd51406e1e81a4b1e7aaf3996f0250d
Signed-off-by: Alessandro Boch <aboch@cisco.com>

Merge "The Switch Event thread is the thread that processes events coming from all switches. Right now its very fragile as an unhandled exception can cause the thread to die causing the controller to be unusable. Making the thread more tolerant of exceptions and making the shutdown of the thread more deterministic."

Fix Plugin execution not covered in Eclipse for config subsystem

yang-test, yang-test-plugin

Change-Id: I9be015e1221ad9ff143370bb68837e0c55558a3e
Signed-off-by: Maros Marsalek <mmarsale@cisco.com>

Fix of bug in Notificator class

Deleting of listner from listenersByStreamName map was corrected
try, finally block was added for locking, unlocking of access to map

Change-Id: I60ecb9c247af18436f348d4f4827c7f75fdfaeaa
Signed-off-by: Jozef Gloncak <jgloncak@cisco.com>

The Switch Event thread is the thread that processes events coming from all switches.
Right now its very fragile as an unhandled exception can cause the thread to die causing
the controller to be unusable. Making the thread more tolerant of exceptions and making
the shutdown of the thread more deterministic.

Change-Id: I31c31f012229de8e124c0c13fa651513daaf6776
Signed-off-by: Asad Ahmed <asaahmed@cisco.com>

Merge changes I8290457c,If84199a1

* changes:
  Stats Mgr to skip equality check on flow stats
  Apply configs' name convention to authorization roles and resource groups

Fix for BUG 402: Subnet creation with NeutronAPIService sometimes fails
with a 409 error

Correct typo and bring hostRoutes out as own init block

Change-Id: I7f6f83cb6a63aa81a057b40c377ac49132a9fcb9
Signed-off-by: Ryan Moats <rmoats@us.ibm.com>

Requestor FRM to update the sw views on remote execution

- When FRM relay the flow add/remove/modify action to an executor
  FRM on another cluster node, it should be in charge of updating
  the sw view caches. As this happens inside the caller application's transaction.
  Currently the executor updates the cache, update which is not
  reflected in the requestor until the caller application commits the transaction.

Change-Id: I36f3efa2770e71c24e0d059d6bf08eee2c1684f5
Signed-off-by: Alessandro Boch <aboch@cisco.com>

Improve logging in NetconfClient, logging-bridge. Increase connection timeout in sal-netconf-connector.

Stop polluting log file with unnecessary 'Exception' messages by logging-bridge, print stack trace with exceptions using slf4j.
Increase connection timeout in sal-netconf-client from 5s to 20s.
Include label in NetconfClient exceptions.

Change-Id: If6819029fb79ea6ebead6b6745adf2c439d107da
Signed-off-by: Tomas Olvecky <tolvecky@cisco.com>

Fix unintentional breakage

Yangtools commit 6e7a42bd344f5928343ad67328bc8a8a734bea2e introduced a
new method which broke the controller-side proxy. Fix it up by simply
forwarning it through the delegate.

Change-Id: I4582acb3a6987d9d87d422a86283984a38cd31ba
Signed-off-by: Robert Varga <rovarga@cisco.com>

Merge "Added resource /streams/stream/<streamName>"

Merge "Added support for /modules, /modules/module, /operations resources"

Merge "Handle uncaught exceptions from Clustering Services in HostTracker"

Handle uncaught exceptions from Clustering Services in HostTracker

There are couple of timer tasks in host tracker which use caches
provided by clustening services. If a container is deleted, then
the cache allocated to that container is also removed. Although
TimerTasks do get cancelled when container is deleted, but if it
happens during the execution of timer thread, then the task will
run to completion (even though it was cancelled) and may try to
access a cache which has already been deallocated.

The accessing of this deallocated cache will result in clustering
service exception which needs to be handled by timer tasks

Also adding toString methods to IPHostId and IPMacHostId for more
readable output when diplaying hosttracker data structures

Change-Id: Id1c481360a4ff71b6eb0afab3c030ef011ca7dcd
Signed-off-by: Maurice Qureshi <maquresh@cisco.com>

Added resource /streams/stream/<streamName>

- added netty web socket server - port 8181

Subscriber can be registered for notifications in this flow:
1. Subscriber invokes RPC create-data-change-event-subscription with input element regarding to module sal-remote revision "2014-01-14"
2. Subscriber gets sream name as a response from server
3. Subscriber calls GET operation on /streams/stream/<streamName>
4. Subscriber gets response with status code 200 OK and header field Location contains URI of web socket server
5. Subscriber creates web socket http connection to obtained URI in step 4.
6. Subscriber can be unregistered from receiving notifications by sending CloseWebSocketFrame with reasonText containing stream name

Change-Id: I92925f5a50b2ceadf2cc8f0eaee2c99dd6a63d8d
Signed-off-by: Martin Sunal <msunal@cisco.com>

Added support for /modules, /modules/module, /operations resources

support for:
/restconf/modules
/restconf/modules/pathToMountPoint/yang-ext:mount
/restconf/modules/module/moduleName/revision
/restconf/modules/module/pathToMountPoint/yang-ext:mount/moduleName/revision
/restconf/operations
/restconf/operations/pathToMountPoint/yang-ext:mount

revision must be in format "yyyy-MM-dd"

test for URIs:
- /modules
- /modules/module/<module_name>/<module_revision>
- /operations
- /modules/......./yang-ext:mount
- /modules/module/........../yang-ext:mount/<module_name>/<module_revision>
- /operations/........../yang-ext:mount

Change-Id: I7169a25024598dd3b49d7960d1ef90bc40c5d1d5
Signed-off-by: Martin Sunal <msunal@cisco.com>
Signed-off-by: Jozef Gloncak <jgloncak@cisco.com>

Stats Mgr to skip equality check on flow stats

- Statistics Manager runs an equality check on the list of flow statistics
  received from the protocol plugin with its cached ones to decide whether
  to update or not, in order to avoid a cluster cache update.
  In presence of flows on the switch, the flow updated statistics are always
  different becasue of the milli and nano secs duration fields they contain

Change-Id: I8290457c1d0a43294e2449cb16c14d6c1e36481e
Signed-off-by: Alessandro Boch <aboch@cisco.com>

Maven archetype for config subsystem aware bundles

Change-Id: I0b109770f03917b923de3d4fd78d91f43eded4e2
Signed-off-by: Maros Marsalek <mmarsale@cisco.com>

Apply configs' name convention to authorization roles and resource groups

- Also include Switch config

Change-Id: If84199a1db6e1fc35c02c4b92493f99aeb8d8af8
Signed-off-by: Alessandro Boch <aboch@cisco.com>

Merge "Fix end of TLV in LLDP packet"

Merge "MD-SAL transaction chaining API"

Merge "Removed unused code from md-sal."

Merge "Restrict MD-SAL data operations to a single thread"

Merge "Clarify the semantics of put{Configuration,Operational}Data()"

Clarify the semantics of put{Configuration,Operational}Data()

This just adds documentation to how the put operations interact with
pre-existing data.

Change-Id: I01979df6a22f1a6d16c04c2cb0b97d7472d62040
Signed-off-by: Robert Varga <rovarga@cisco.com>

Make sure transaction applies whole removal list

Change-Id: I53d1cf10bce6e444f80b13d9ce57e5f1ef2eb5c9
Signed-off-by: Tony Tkacik <ttkacik@cisco.com>

Fixed bug when new childs were ommited during data store merge.

Change-Id: I0680d1354614d7597b0079445951d4a6f33d8d5c
Signed-off-by: Tony Tkacik <ttkacik@cisco.com>

Merge "Yang Changes to include list<actions> to PacketOut - Added list<actions> and buffer_id to PacketOut Message to handle group processing and Packet Modification through different actions"

MD-SAL transaction chaining API

This API complements the normal MD-SAL transaction API. Applications can
use this API to maintain inter-transaction ordering and data
consistency.

Change-Id: Ia603f7dacf081fb1528d149f4635c2f22101a7a0
Signed-off-by: Robert Varga <rovarga@cisco.com>

Restrict MD-SAL data operations to a single thread

MD-SAL does not give us transaction ordering guarantees, which mean that
data may hit the data store than what the application intended. Change
the model to single-threaded until proper transaction ordering APIs are
defined.

Change-Id: I6c6375480082b94c614ab2602a2857e0f4779ee8
Signed-off-by: Robert Varga <rovarga@cisco.com>

Merge changes I805ec065,Idc9995b1

* changes:
  Change unprotected netconf address from 0.0.0.0 to 127.0.0.1 .
  Refactor config-persister: clean up exception handling and netconf client.

Merge "Fix typo and add more descriptive error"

Merge changes Iae4ee2f2,I148c22c4,I1dab84ae

* changes:
  Additional test for instance-identifier
  BUG 388: corrected media type in /operations resource
  BUG 312: Removed legacy RESTCONF draft 01 syntax

Merge "updated test coverage for AutodetectDirectoryPersister test coverage raised on config-persister-file-xml-adapter"

Merge "minor test fix"

Merge "Add configurable connection timeout to netconf client."

Merge "Added description to meter types yang model."

Fix typo and add more descriptive error

When comparing keys, also provide a hint to the node which violated the
key constraint.

Change-Id: I02a9ba3c887bcbc3af34fe770f662e627c3a4414
Signed-off-by: Robert Varga <rovarga@cisco.com>

Bump yangtools to 0.6.2-SNAPSHOT

This will pick up upcoming yangtools release.

Change-Id: I3c6f8faf8888eaa3a71a9f0e16cc242b6f958832
Signed-off-by: Robert Varga <rovarga@cisco.com>

Yang Changes to include list<actions> to PacketOut
- Added list<actions> and buffer_id to PacketOut Message to handle group processing and Packet Modification through different actions

Signed-off-by: Gaurav Bhagwani <gaurav.bhagwani@ericsson.com>
Change-Id: I5875f9792f6a036bf2b4060ba377f1565e66d2fd

Additional test for instance-identifier

- test for instance identifier
- correction of ignored test (adding revisions to wrong tests)
- instance identifier with leaflist predicate:
-- serialize codec
-- XML output
-- JSON output

Change-Id: Iae4ee2f2ff3c1b3445958f1d69673df9f56b0e2a
Signed-off-by: Jozef Gloncak <jgloncak@cisco.com>

BUG 388: corrected media type in /operations resource

There were added these media types for /operations resource:
application/yang.operation+json
application/yang.operation+xml
Corrected handling of requests which have empty body and Content-Type has some value or is missing

Change-Id: I148c22c4c6c28d0d13f2b717fe629b42d5c73f70
Signed-off-by: Martin Sunal <msunal@cisco.com>

Change unprotected netconf address from 0.0.0.0 to 127.0.0.1 .

Plaintext netconf server was bound to any local interface. This is a security risk
because there is no authentication. The ssh server should be used as public endpoint instead.

Change-Id: I805ec065548e017dd2244d37e3275d379761e490
Signed-off-by: Tomas Olvecky <tolvecky@cisco.com>

updated test coverage for AutodetectDirectoryPersister
test coverage raised on config-persister-file-xml-adapter

Change-Id: I027fd57baeb233271c88499c29e4708a735bb10d
Signed-off-by: Martin Bobak <mbobak@cisco.com>

Refactor config-persister: clean up exception handling and netconf client.

Split ConfigPersisterNotificationHandler into registrator and listener, remove unnecessary
reference to NetconfClient.
Refactor ConfigPusher: each method now returns computational results instead of void. Avoid
unnecessary NetconfClient reference leak - close it after every use. Rename generic 'timeout'
to 'maxWaitForCapabilitiesMillis'. Avoid throwing and catching SAXException and IOException
as there is no recovery from them; keep only ConflictingVersionException.
ConfigPersisterActivator now actually uses the configurable timeout, the property is renamed
from 'pushTimeout' to 'maxWaitForCapabilitiesMillis'.

Change-Id: Idc9995b11dd2f19a90598c88eb7337d39d63fc27
Signed-off-by: Tomas Olvecky <tolvecky@cisco.com>

minor test fix

Change-Id: I3dce2b98d5d7452a9363150b91827eb3dcc03e60
Signed-off-by: Martin Bobak <mbobak@cisco.com>

Fix Helper sorting issue

Fix sorting in tables, for undefined values

Change-Id: Id9c09aa9b9de93f07c4063e21e6dac578d13913f
Signed-off-by: Sapan Shah <sapshah@cisco.com>

Removed unused code from md-sal.

Change-Id: Ia64a118ec7c75d0abf5800f7eee98ae708b8e50a
Signed-off-by: Martin Vitez <mvitez@cisco.com>

Display Vlan Priority in Flow Stats in TS GUI

also display Priority values, greater than 32767

Change-Id: Idc067ffe3b9e859c1f2ce2eeec7d5e569f318db6
Signed-off-by: Sapan Shah <sapshah@cisco.com>

Fix end of TLV in LLDP packet

Change-Id: Id4bc88bcf792758ff73f9d143481aca2a6ef3c95
Signed-off-by: Diti Bhatia <dibhatia@cisco.com>

Merge "Handling the case where vlan priority is set to 0."

Merge "Double the SSL input packet buffer size"

Handling the case where vlan priority is set to 0.

Change-Id: I0d9735343633c2efc1f6a3a5c768aad3f7c88c90
Signed-off-by: Asad Ahmed <asaahmed@cisco.com>

Add configurable connection timeout to netconf client.

Make connection timeout configurable. Previous behavior was having hardcoded connection timeout set to 1 minute.
Make NetconfConnectorModule more configurable, allow (and this is new default) to retry on connection failures forever.

Change-Id: Ia9280cff1b751a4e7318e1e0331175c960823dad
Signed-off-by: Tomas Olvecky <tolvecky@cisco.com>

BUG 312: Removed legacy RESTCONF draft 01 syntax

Change-Id: I1dab84ae70ef39964ead1b14efd58ae2e3eee087
Signed-off-by: Martin Sunal <msunal@cisco.com>

Added description to meter types yang model.

Signed-off-by: Deepthi V V <deepthi.v.v@ericsson.com>
Change-Id: Iedb115fa2178178d79292705f787502df56ab844

Merge "Add javadoc generation for yang modules in netconf subsystem"

Merge "Removed javadoc declaration from md-sal to enable parent javadocs settings."

Merge "Minor bug fix in stale flow statistics data cleanup"

Merge "BUG 340: Resolved instance-identifier translation in JSON"

Merge "Added utility SynchronizedTransaction wrapper implementation."

Merge "Bug 387 : Fixed Binding Independent Connector to properly serialize output."

Merge "BUG 392: Resolved translation to JSON behind mount point"

Merge "Fix of issue 377 for Notifications resolving."

Merge "Added explicit locking of the data change listener / commit handler registration"

BUG 340: Resolved instance-identifier translation in JSON

Change-Id: Ia8853c00798674251b39941a675e9c359aa68a00
Signed-off-by: Martin Sunal <msunal@cisco.com>

BUG 392: Resolved translation to JSON behind mount point

Change-Id: Icee18d06679dcfa935b4c832b0b6673936d60a10
Signed-off-by: Martin Sunal <msunal@cisco.com>

Double the SSL input packet buffer size

Change-Id: I9aabf4e99c55eb788314f02755eb91df7f97a8da
Signed-off-by: Jason Ye <yisye@cisco.com>

Added utility SynchronizedTransaction wrapper implementation.

Change-Id: Ic35661a59fde08f0ec0b72121cd2e3dedf1de480
Signed-off-by: Tony Tkacik <ttkacik@cisco.com>

Minor bug fix in stale flow statistics data cleanup

Change-Id: I72ba43b33a2580bd07c3eef2d4c85a3a8c265a51
Signed-off-by: Anil Vishnoi <avishnoi@in.ibm.com>

Bug 387 : Fixed Binding Independent Connector to properly serialize output.

Change-Id: Ibabb08983ca31182e087edbdf6821bd1a0216516
Signed-off-by: Robert Gallas <rgallas@cisco.com>
Signed-off-by: Tony Tkacik <ttkacik@cisco.com>

Fix of issue 377 for Notifications resolving.

Fixed issue when Notification didn't pass from Netconf client to registered and configured ODL plugin application. After Fix, when notification is send to ODL,
notification is forwarded through md-sal directly into application that is registered for listening on mount point where notification arrive.

Change-Id: I4e0c644efb72ba0c26c75e9bd7452afbb0f4897c
Signed-off-by: Lukas Sedlak <lsedlak@cisco.com>
Signed-off-by: Ed Warnicke <eaw@cisco.com>
Signed-off-by: Tony Tkacik <ttkacik@cisco.com>

Added explicit locking of the data change listener / commit handler registration

  - This locking fixes ConcurrentModificationException in cases
    when transaction processing is computing affected commit handlers
    and data change listeners.

Change-Id: Icdd9a2ea36102f681a5d82390c3b9d942560b878
Signed-off-by: Tony Tkacik <ttkacik@cisco.com>

Removed javadoc declaration from md-sal to enable parent javadocs settings.

Change-Id: Ic0207a98d8755cb1ea12d570e37d55e76c94f4b6
Signed-off-by: Martin Vitez <mvitez@cisco.com>

Add javadoc generation for yang modules in netconf subsystem

Added org.opendaylight.yangtools.yang.unified.doc.generator.maven.DocumentationGeneratorImpl generator
to yang-maven-plugin configuration in order to generate informational html page about yang modules.

Affected yang modules: ietf-netconf-monitoring, ietf-netconf-monitoring-extension.

Change-Id: I83d3089256a21515cc4fb3320a81956af590e1ce
Signed-off-by: Maros Marsalek <mmarsale@cisco.com>

TS GUI does not show ToS field in flow stats

Change-Id: I4b0bfc1988ea4fcc3eae02a73c3aebae1622f382
Signed-off-by: Sapan Shah <sapshah@cisco.com>

BUG-362: add some diagnostic information
Changed Remote RPC Server Implementation to log an error when
host IP address is not found and stop itself. Earlier it would
raise exception that the configuration subsystem could not
handle.

Change-Id: I5ea843f8c1d86b5cd923081ff9ba82e94c29c4d2
Signed-off-by: Robert Varga <rovarga@cisco.com>
Signed-off-by: Abhishek Kumar <abhishk2@cisco.com>

Merge "Fix For Bug #335 Bug Description : Remove SW2 property "description" to a node - not working for both OF13 and OF10 mininet."

Merge "Fix for bug #343 Have made changes in HostTracker.java Submitting again"