Bulk-add copyright headers to .xtend files

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

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

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

Merge "Instance identifier support"

Improve logging to indicate when configuration has been pushed

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

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

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"

Merge "fix typos and some Java improvements in ClusterManager"

Fixed inappropriate uses of log level INFO

This is done per the recommendation here:

https://wiki.opendaylight.org/view/Draft_Syslog_Level_Settings

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

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.

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

Fixed bug when global RPCs we're not forwarded

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

Added DEBUG log statements for transaction translation.

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

Merge "Optimization - calculate the subnet prefix only once."

Merge "Bump yangtools artifacts to reflect their uptodate versions"

Merge "Fix ClassCastException in logging bridge stop"

fix typos and some Java improvements in ClusterManager

Fix typos in error messages, use StringBuilder instead
of StringBuffer, append strings more efficiently,
and move a local variable to be a constant.

Change-Id: I23a4436449909cf02a6fce47e0f1f9a0ce80f0e4
Signed-off-by: Alissa Bonas <abonas@redhat.com>

Optimization - calculate the subnet prefix only once.

Before this change, subnet prefix was calculated on every
'isSubnetOf' call. Since the prefix is not changing unless
the subnet address itself changed, it's sufficient to do it only once
and keep it as part of the Subnet object.

Marked it as transient so it will not be serialized - it can be
calculated from the rest of the info of Subnet anytime after
deserialization.

Change-Id: Ie16bf80713820b566864bea0ff46298c93cc2d47
Signed-off-by: Alissa Bonas <abonas@redhat.com>

Merge "Avoiding hash collisions of a match with its reverse"

Minor bugfix in Topology Manager

- This fixes an incorrect assumption that subsequent edge-ADDED updates
  will always have equal edge properties

Change-Id: Ib124a4c260d7fce6bb8464f267c2e88cf0b35384
Signed-off-by: Yevgeny Khodorkovsky <ykhodork@cisco.com>

Fix ClassCastException in logging bridge stop

  - Additional fix to handle NPE during logging bridge start

Change-Id: Id6d4b727dd08db18a91b509cad99450fd41a5171
Signed-off-by: Prasanth Pallamreddy <ppallamr@cisco.com>

Make configuration push timeout configurable

This patch adds the option to specify a the pusher timeout, such that it
can be made either shorter or longer than the default 2 minutes.

Another improvement is the use of System.nanoTime(), making the deadline
work in face of calender time changing.

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

unitests: Remove extending TestCase from unitest classes.

Extending TestCase which is a Junit3 format, and using Junit4
annotations at the same time in same class causes confusion,
and can also lead to runtime problems.
An example for a problem - putting @Ignore on a test case which starts
with word "test" and is located in a class that extends TestCase will
actually NOT ignore the test because it will be interpreted as Junit3
and not as Junit4.
Removing extending TestCase will make the unitests to be interpreted
as Junit4 style and leave no room for confusion.

Change-Id: I2365f0921d64667a5489c42eeb53adadc45efdaa
Signed-off-by: Alissa Bonas <abonas@redhat.com>

Merge "Move NetconfUtil to netconf-util."

Merge "Default DataBroker does not read subtrees from Mountpoint"

Bump yangtools artifacts to reflect their uptodate versions

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