Merge "Prevent ConfigPusher from killing its thread"

Merge changes I8c23739a,Ia0e70828

* changes:
  Bulk-add copyright headers to java files
  Bulk-add copyright headers to .xtend files

Prevent ConfigPusher from killing its thread

The original code promoted pretty much every error to an unhandled
RuntimeException. Use the exceptions already in place to handle erros
occuring. Also handle IO error as a retriable operation.

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

Merge "Fix RPC forwarding related bugs in Binding Independent Connector"

Merge "Adding a couple of ovsdb config variables to the config.ini"

Adding a couple of ovsdb config variables to the config.ini

Change-Id: I5c524121130845bd8d42f545430f5f06293cfc25
Signed-off-by: Madhu Venugopal <mavenugo@gmail.com>

Migrate bundles' configuration mgmt to ConfigurationService

- Change current logic where read/save config is completely managed by each bundle,
  with moving it to configuration manager. This decoupling will allow configuration
  manager to decide how to persist the bundles' configurations to disk, withouth
  necessarely depend on java serialization as today controlelr does.
  Also now file names can be static and each bundle does not need to track the container
  context in the file name (which was lame). This allow controller to erase container
  startup files on container removal, which was not possible today.
- Global and in container configuration services to expose
  two APIs for the other bundles to retrieve and save their config
- Per container startup files organized in directories (directory name = container name)
- Container startup directory removed on container removal, and created on container creation if not present

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

Fix for dry run

Removed unneeded variables from commons/opendaylight/pom.xml
Expanded dependencyManagement in commons/opendaylight/pom.xml
Added explicte dependency on sanitytest to distribution pom.xml
Fixed dependencies in two integration tests
Removed module archetypes from pom.xml
Moved third-party/ganymed build from netconf to root pom.xml

Change-Id: Id4d1586bd86f375fa2bd4b64faf28682d402e4e9
Signed-off-by: Ed Warnicke <eaw@cisco.com>

Merge "One more fix to pass release-prepareonly"

Merge "Gerrit contains following fixes: 1) Fix for bug 284 - Added custom ip matching method to compare ip addresses with mask 2) Fix for bug 286 3) Fix for bug 379"

Merge "Changed multimaps to synchronized multimaps"

Bulk-add copyright headers to java files

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

Bulk-add copyright headers to .xtend files

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

Merge "Self assign controller MAC when cannot retrieve one from host"

Merge "Add a the logger config for debugging schema loading"

Merge "Improve logging to indicate when configuration has been pushed"

Self assign controller MAC when cannot retrieve one from host

- When Switch Manager is not able to retrieve the list of the host
  network interfaces, self assign a custom MAC address.
  This is a quick fix for Hydrogen release. User can still
  overwrite the custom MAC with the one of his choice via
  switchmanager REST API

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

Changed multimaps to synchronized multimaps

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

Merge "Instance identifier support"

Improve logging to indicate when configuration has been pushed

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

Gerrit contains following fixes:
1) Fix for bug 284 - Added custom ip matching method to compare ip addresses with mask
2) Fix for bug 286
3) Fix for bug 379

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

Add a the logger config for debugging schema loading

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

Merge "Improve client logging"

Instance identifier support

- codec for instance identifier (deserializer, serializer) was added
- instance identifier from XML and JSON files to simple node
- instance identifier from simple node to XML and JSON
- corrected augments with the same names

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

Fix RPC forwarding related bugs in Binding Independent Connector

  - Fix for set of small bugs which prevented invocation of RPC via Restconf

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

Merge "Minor fixes in md-sal and config."

One more fix to pass release-prepareonly

Add sanitytest as a dependency of the distribution else release would
fail

Change-Id: I69fa09c7f00b873586def2ba1452def3db3cb10f
Signed-off-by: Giovanni Meo <gmeo@cisco.com>

Merge "Turned down some logging from INFO to DEBUG in StatsAdapter"

Added listener registration when not reusing global instance

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

Resolve the service reference before intantiating proxy for it

The original code assumed that if it has a ServiceReference, that
service does not disappear. Now it resolves it before deciding whether
to use a proxy.

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

Turned down some logging from INFO to DEBUG in StatsAdapter

Change-Id: Ibc5b6b766d99f2044baa147b0985c293778d5041
Signed-off-by: Ed Warnicke <eaw@cisco.com>

Fixed some exceptions in the Adapters

Change-Id: Icf4c0802a41bc68026117cff38d080aeea87aef1
Signed-off-by: Ed Warnicke <eaw@cisco.com>

Dummy commit

Change-Id: I662e2d15d7fcc26157c963634a3d748974ad006c
Signed-off-by: Ed Warnicke <eaw@cisco.com>

Merge "Reduce verbosity/criticality of inconsistent yangstore messages"

Merge "Add support for identity-ref config attributes to config/netconf subsystem"

Merge "Fix for bug - 369 Minor fix in building NodeConnector for AD-SAL Output action"

Reduce verbosity/criticality of inconsistent yangstore messages

Directs yangtools warnings into the logfile and updates yangstore
messages such that we know when the store reaches consistent state after
being inconsistent.

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

Add support for identity-ref config attributes to config/netconf subsystem

Change-Id: I79d5147f1d6bb1acd6b5a374bb76f48b6a3aaf55
Signed-off-by: Maros Marsalek <mmarsale@cisco.com>
Signed-off-by: Tomas Olvecky <tolvecky@cisco.com>
Signed-off-by: Robert Varga <rovarga@cisco.com>

Update test to be more careful and add logs

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

Fix for bug - 369
Minor fix in building NodeConnector for AD-SAL Output action

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

Merge "Turned down some logs in StatisticsProvider from INFO to DEBUG"

Merge "Fix JMX plugin mismatch with yangtools"

Fix JMX plugin mismatch with yangtools

https://git.opendaylight.org/gerrit/#/c/4775/ introduced an incompatible
StaticLoggerBinder to the classpath, so make the same change here.

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

Turned down some logs in StatisticsProvider from INFO to DEBUG

Change-Id: I34729faa14d81e902add8acd59e2bfc895ddd87e
Signed-off-by: Ed Warnicke <eaw@cisco.com>

Remove clustered MD-SAL/datastore from distribution

It is not being built, so attempting to distribute it would result in
release failure in the controller project.

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

Bug 319: Fixed two-phase commit verification of data.

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

Extract dom-serializer-api|impl from binding-broker-impl

Change-Id: I1b2844bfac5062fd15c1f011c3104b3dddb33b51
Signed-off-by: Tony Tkacik <ttkacik@cisco.com>
Signed-off-by: Maros Marsalek <mmarsale@cisco.com>

Improve client logging

This logs the cause of the failure to instantiate client and excalates
it to info if it's not done in 30 attempts.

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

Minor fix in flow IP Matching

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

Merge "Gerrit contains following minor enhancements: 1) Send statistics requests whenever new flow capable node connects to the controller 2) Clean up transaction-id cache for expired Ids 3) Remove Tx id when last part of multipart response received."

Merge "fix for frm to not act on duplicate mode change notifications"

Merge "Relaxing a strict check in updateNode in SwitchManager to handle a condition in which the adaptor fails to call the addNode."

Small fix to finally fix dryrun.

Turns out that the lack of directory hiearchy matching
our pom hierarchy requires some redeclaration of
properties.

Change-Id: I5b608ba9599ac5d9d54d1898f4f45f68001ada5e
Signed-off-by: Ed Warnicke <eaw@cisco.com>

fix for frm to not act on duplicate mode change notifications

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

Relaxing a strict check in updateNode in SwitchManager to handle a condition in which the adaptor fails to call the addNode.

The existing ADSAL code is already robust enough to handle the scenario of NodeProps not containing the a node Key.
Also, there are cases in which an application might just want to see an update event for processing
(especially when the addNode events are missing).

Change-Id: I6561f2f34220b2372b142307754f086a2abe42fd
Signed-off-by: Madhu Venugopal <mavenugo@gmail.com>

Gerrit contains following minor enhancements:
1) Send statistics requests whenever new flow capable node connects to the controller
2) Clean up transaction-id cache for expired Ids
3) Remove Tx id when last part of multipart response received.

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

Merge "Fix for NullPointerException"

Small fix for version skew to fix dryrun

Change-Id: Ie54347a79a11e06c3c33cdfae85a837ea5927365
Signed-off-by: Ed Warnicke <eaw@cisco.com>

Merge "Remove dependency on Exificient due to licensing incompatibility"

Merge "Pull zeromq initial configuration out of 01 config file"

Merge "Make MD-SAL use opendaylight parent"

Minor fixes in md-sal and config.

- extended Export-Package declaration in config/shutdown-api.
- added CodeGenerator in md-sal/sal-binding-config
- fixed version of yangtools in md-sal/sal-remote

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

Merge "Yangtools version police: remove references to non-current versions"

Merge "Added test for MouontPoints and URI"

Merge "Reduce informations from logging.bridge"

Remove dependency on Exificient due to licensing incompatibility

Exificient is licensed under GPLv2 with no intention to add the
classpath exception (witnessed in at
http://sourceforge.net/p/exificient/discussion/856595/thread/6017013e/).
Remove all dependencies on it, pending reimplementation using a more
open library (OpenEXI looks like a good candidate).

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

Make MD-SAL use opendaylight parent

Since MD-SAL code does not comply with coding style enforced by the
checkstyle plugin, disable its execution for now -- BUG-272 tracks the
need to re-enable it.

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

Yangtools version police: remove references to non-current versions

Unifies the version property to yangtools.version, force restconf to use
the versions specified in the md-sal parent and bump its reference to
restconf-api to 0.6.0-SNAPSHOT.

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

Pull zeromq initial configuration out of 01 config file

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

md-sal: sync commons-lang dependency

topology-lldp-discovery is currently specifying 2.6 commons-lang
dependency whereas the controller is dependent on 2.4.  Bring md-sal in
sync w/ the controller by dropping back to 2.4.

Change-Id: Ifb340ceef25f866cf606c4733ade80e78aea0e31
Signed-off-by: Chris Wright <chrisw@sous-sol.org>

Added test for MouontPoints and URI

Change-Id: Ibb18ac011f0523e18595e4fff0900a492d3c5e25
Signed-off-by: Ladislav Borak <lborak@cisco.com>

Merge "Remove powermock dependency from md-sal."

Merge "Provide single point for enforcing config naming convention"

Remove powermock dependency from md-sal.

Remove unused dependency.

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

Reduce informations from logging.bridge

Logging bridge maps 1-1 the log level from OSGi logging service to
slf4j. By design the OSGi framework set al INFO level all the modules
start and stop, which by pure chance, is also inline with our
guideline. Even though the compliance is satisfied there is a
usability issue because the console is flooded with many start stop
events given there are many bundles hence setting to WARN level the
log level for logging.bridge

Change-Id: I0dc484c49bd5966ad7dcdc01d55b9c28cfcf8217
Signed-off-by: Giovanni Meo <gmeo@cisco.com>

Provide single point for enforcing config naming convention

- Provide generic ConfigurationObject class which other will extend
- Class provides method for validate string based on default private regex
- Regex enforces 1 to 256 alphanumeric chars with addition of -+/*.()[]@
- Default regex can be overwritten with one set in config.ini
- Have configs which need name validation to extend new class.
  Change is bacward compatible, existing config file are correctly
  deserialized

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

Small fix to make dryrun happy.

Change-Id: I28a55d464c1e9ce4201ab3e44683982a1f525601
Signed-off-by: Ed Warnicke <eaw@cisco.com>

Merge "Binary masks in match types yang model changed to numbers. Plugin code review corresponding to this change: https://git.opendaylight.org/gerrit/#/c/4638/"

Binary masks in match types yang model changed to numbers.
Plugin code review corresponding to this change: https://git.opendaylight.org/gerrit/#/c/4638/

Patch3: Fixed masks to use the same type as values (MacAddress for MacAddress and
IPv6FlowLabel for IPv6FlowLabel) rather than arbitrary uint* values.

Signed-off-by: Deepthi V V <deepthi.v.v@ericsson.com>
Change-Id: Icbc84312c0a2f26738ab9d84f6af34783c4aad5d
Signed-off-by: Ed Warnicke <eaw@cisco.com>

Merge "Fix config transaction handling in netconf."

Merge "sal-remote yang"

Merge "Provide callback feature when modal is visible"

Provide callback feature when modal is visible

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

Merge "Add support to resolve docker container names for clustering support. Currently on works when all containers are on a single docker host"

Add support to resolve docker container names for clustering support. Currently on works when all containers are on a single docker host

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

Fix for NullPointerException

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

Fix config transaction handling in netconf.

When config manager fails a transaction because of optimistic lock failure, netconf
must clean up this transaction as well. Fix TransactionProvider to abort transaction
if commit fails. Only exception is ValidationException where user should
have an option to fix the configuration and commit again.

Turn off netconf client reuse in persister: this narrows the possibility of
hitting concurrent modification with other config transactions. Each retry
to push a snapshot is now done with a newly obtained client, so new transaction
will be started each time.

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

sal-remote yang

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

Include the incomming port in PacketOut message.
Without this flood messages do not omit the incomming port on the switch.

Change-Id: I932d00c1f17b95678cb73136aefa05fcc5718a95
Signed-off-by: Elek Jani <elekjani@gmail.com>

Fixed incorrect null check.

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

Merge "Change target to ${project.build.target} in a bunch of pom file. Add some xtend-gen source dir to the build path. The commit purpose is to eliminate errors in eclipse."

Merge "Make configuration push timeout configurable"

Merge "Gerrit contains following changes : 1) Fix for bug 284. Added functionality that listen to data store *remove* changes and cleanup the relevant statistics from operational data store. 2) Added functionality that listen to config data store *create* changes for Flow,Group, Meter & Queue and send statistics request to switch to get respective stats. 3) Added functionality to periodically remove stale stats from operational data store. Clean up thread invokes after every two cycle of stats collection. 4) Removed unnecessary local caching."

Merge "md-sal: commons-codec artifact version sync"

Gerrit contains following changes :
1) Fix for bug 284. Added functionality that listen to data store *remove* changes and
cleanup the relevant statistics from operational data store.
2) Added functionality that listen to config data store *create* changes for Flow,Group,
Meter & Queue and send statistics request to switch to get respective stats.
3) Added functionality to periodically remove stale stats from operational data store.
Clean up thread invokes after every two cycle of stats collection.
4) Removed unnecessary local caching.

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

Make AbstractSessionNegotiator react to exceptions

While the SessionNegotiator is present on the pipeline it should react
to exceptions happening on the channel by failing negotiation.

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

Introduced advanced strategy for parsing of YANG schemas

  - This strategy increases convergence time, because it does not
    parse files with unsatisfied dependencies.

  - New strategy for schema parsing in runtime uses two separate parser
     Minimalistic - is used when YANG source is discovered and scans only
     for module name, revision and dependency information
     Full - parses set of YANG sources and creates model representation.

     When bundle is added / removed - tracker gets yang sources,
     registers it to SchemaResolver, which immediatelly uses
     Minimalistic parser to obtain base information.

     Then context of all known sources and their dependencies is sorted
     and filtered only to use sources which has their dependencies resolved.

     Full parser is then invoked only for models, which have dependencies
     resolved.

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

Merge "Fixed bug when global RPCs we're not forwarded"

Merge "Added DEBUG log statements for transaction translation."

Merge "Fixed inappropriate uses of log level INFO"