Simplify StatisticsManagerActivator/StatisticsProvider contract

Get rid of the need for the StatisticsManagerActivator to keep around
the Provider session -- it is only ever referenced from
StatisticsProvider.start(). Same goes for the various services which
are set into it.

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

Centralize NodeStatisticsAger creation

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

Optimize statistics cleanup

Perform a single transaction for cleaning up the flow. Also use
System.nanoTime() for keeping the expiry time and keep it constant.

Furthermore, use HashMap instead of ConcurrentHashMap to gain better
memory peformance. This requires synchronizing between the statistics
update and statistics cleanup -- which actually is okay, as it closes
the race between an update happening precisely when a flow entry would
be retired. With the lockless design that entry could be lost, simply
because we'd get undefined ordering between cleanup and update
transaction commit.

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

Do not allocate/commit transactions in tight loops

A transaction is additive and there's little harm in commiting empty
ones. Move allocation/commit outside of tight loops, such that we get
proper bulking.

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

Fix yangtools API incompatibility

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

Merge "Do not use deprecated StringBufferInputStream"

Merge "Fix a warning"

Merge "Remove deprecated API elements from protocol framework"

Merge "Fix typo in match types yang model"

Merge "Changed model versions to dependencies"

Do not use deprecated StringBufferInputStream

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

Fix a warning

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

Rewrite POSSIBLE_ENDS and POSSIBLE_STARTS

These two are really simple byte arrays, initialized them as such
withouth going through strings/UTF8.

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

Merge "Some methods should be final"

Merge "Expose ChannelFuture from sendMessage()"

Merge "Getter methods should be final"

Merge "Fix a warning and make field final"

Merge "Remove use of deprecated Exceptions"

Merge "minor pom cleanup to eliminate duplicate group ID specification when the group ID is the same as the parent pom, just removes some warnings from eclipse, should not be any functional difference"

Some methods should be final

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

Merge "Fix warnings reported in toaster"

Merge "Fix a few eclipse-reported warnings"

Expose ChannelFuture from sendMessage()

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

Getter methods should be final

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

minor pom cleanup to eliminate duplicate group ID specification when the group ID is the same as the parent pom, just removes some warnings from eclipse, should not be any functional difference

Change-Id: I396126408fff54d7120d3ae94b2a5c447d14f8d2
Signed-off-by: David K. Bainbridge <dbainbri@ciena.com>

Merge "Get rid of NetconfMessageFactory"

Merge "Get rid of DeserializerException"

Merge "Get rid of using ProtocolHandlerFactory, part 2"

Merge "Get rid of using ProtocolHandlerFactory, part 1"

Merge "Cleanup dependencyManagement section"

Merge "Remove repositories/pluginRepositories/distributionManagement"

Merge "Remove duplicate properties"

Merge "SwitchMgrNB: URL does not allow nodeconnectorId with forward slash"

Fix for BUG 410

Ensure lower case alpha hex representation is included in MAC
regex match in all cases.

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

SwitchMgrNB: URL does not allow nodeconnectorId with forward slash

Change-Id: I489d5ae559b8c1e35a3c7b9749f7476618b2765b
Signed-off-by: Pramila Singh <pramisin@cisco.com>

Merge "Fixed bugs in getUnsigned methods of NetUtils class."

Cleanup dependencyManagement section

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

Remove repositories/pluginRepositories/distributionManagement

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

Remove deprecated API elements from protocol framework

The use of these concepts has been deprecated in version 0.4.0, which
was released. The only in-tree user was netconf, which was refactored to
not to rely on these concepts, so bump the version to 0.5.0 and purge
the  code.

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

Fix a warning and make field final

Drive-by fix
- additionalHeader should be final,
- the class is not serializable, no uuid is needed

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

Remove use of deprecated Exceptions

DocumentedException and DeserializerException are deprecated and were
only used by deprecated interfaces which we no longer use. Stop using
them.

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

Get rid of NetconfMessageFactory

This removes the use of deprecated ProtocolMessageFactory and
ProtocolHandlerFactory. Just instantiate appropriate Netty handlers
instead.

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

Get rid of DeserializerException

We can throw anything, so use NetconfDeserializerException instead.

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

Get rid of using ProtocolHandlerFactory, part 2

Now that we do not rely on passing it around, we can safely keep
NetconfHandlerFactory wired directly to the
ProtocolMessageEncoder/Decoder.

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

Get rid of using ProtocolHandlerFactory, part 1

This gets rid of completely dead code in NetconfSshClientDispatcher. It
already talks directly to the initializer.

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

Remove duplicate properties

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

Merge "Cleanup imports/whitespace in MD-SAL"

Merge "Adding FlowRef,GroupRef and MeterRef for the ErrorNotification Message"

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."

Cleanup imports/whitespace in MD-SAL

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

Merge "BUG 408"

Fixed bugs in getUnsigned methods of NetUtils class.

 * Fixed bugs that getUnsignedByte(byte) and getUnsignedShort(short)
   handled 0 incorrectly.
 * Added tests for getUnsignedByte(byte) and getUnsignedShort(short).

Change-Id: Ie67fb5c07bed31d16d4a52068dda7735fb7cd5e2
Signed-off-by: Hideyuki Tai <h-tai@cd.jp.nec.com>

Fix typo in match types yang model

Change-Id: Ieb1d4e0808042d8278d67e400db02f8141d7425d
Signed-off-by: Lorand Jakab <lojakab@cisco.com>

Changed model versions to dependencies

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

Merge "Fix incorrect validation in SwitchConfig"

Fix incorrect validation in SwitchConfig

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

Fix warnings reported in toaster

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

Fix a few eclipse-reported warnings

- unused suppresswarnings
- missing serialVersionUuid
- missing generic arguments

Change-Id: Ie741e88bfe161a9e4613b4076892cbcabdd0375c
Signed-off-by: Robert Varga <rovarga@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>