Provide callback feature when modal is visible

Change-Id: I50cc978f1a32bca94d53db84d2c281ddf9099d52
Signed-off-by: Sapan Shah <sapshah@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>

Merge "Log transactions being allocated"

Move NetconfUtil to netconf-util.

Implement simple acceptance netconf client test that can connect to a remote machine. Clean up
helper functions from netconf-impl and persister-impl to netconf-util.

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

Default DataBroker does not read subtrees from Mountpoint

Changed default behaviour of DataBroker, which involved
also read from remote Netconf nodes to return only
data from local store. To access Netconf device
as a datastore one must get MountInstance for it -
MD-SAL instance exclusive for that device.

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

Log transactions being allocated

We do get logs when the transaction processing starts, unfortunately we
do not get the context in which the transaction was created. This fixes
the deficiency by logging when the transaction is first allocated. Also
adds explicit logs about transaction state transitions.

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

md-sal: commons-codec artifact version sync

md-sal is using both 1.7 and 1.8 versions of commons-codec.  The rest of
the controller is using 1.7, so bring md-sal in line with that.

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

Avoiding hash collisions of a match with its reverse

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

Changed port number for test server in ClientRequestHandlerTest
When tests are run in parallel, the port number in ServerRequestHandlerTests and ClientRequestHandlerTests
could have conflicted. Changed the port number to avoid conflict.

Change-Id: I63fae99458fc9109fdbfc5f063ca4289b10aa30c
Signed-off-by: Abhishek Kumar <abhishk2@cisco.com>

Merge "More defensive RPC handling in DOM Broker"

Merge "sal-remoterpc-connector: sync ch.qos.logback:logback-classic version"

More defensive RPC handling in DOM Broker

  - RPCs registered to DOM broker now must comform to the schema
    - at the time of registration of RPC implementation,
      the YANG schema must be present for that RPC.
      This does not affect RPC registrations for Binding Aware Broker.

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

Merge "Commented out Sys.outs in runtime generated code."

Merge "Update to new version of the artifacts proposed by jenkins-controller-version-changes-11"

Merge "Fix bug in hello message: module with more revisions."

Merge "fix of Bug 314"

Merge "Fix: removed tabs from yang files."

Merge "Tests for loading data with augmented schema nodes with equal names"

Merge "Clean up unnecessary imports in 01-md-sal.xml ."

Merge "The config now is loaded only from the coordinator. The previous way of loading the config only if empty had some side-effects."

Merge "Migrate all artifact versions to dependencyManagement"

sal-remoterpc-connector: sync ch.qos.logback:logback-classic version

The following commit:

  8079442 Enhancements to remote rpc server. Using zmq router-dealer bridge

introduced a new version dependency on ch.qos.logback:logback-classic.
Let's make this dependency consisten with the rest of the project by
moving from 1.0.12 back to 1.0.9.

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

Merge "InputStream for RSA file should not be static"

Merge "Changed codec for Identityref in JSON transformation"

Merge "OSGI version change to 5.0.0 in ganymed subproject"

Merge "Enhancements to remote rpc server. Using zmq router-dealer bridge to make the server async. Enhancements to remote rpc client. Using zmq router-dealer bridge to make the client async. Added configuration for remote rpc in Configuration subsystem On Server startup reading of routingtable to populate remoterpcs was giving exception - fixed the same Client was not registered with Server and addRPCImplementation was failing - fixed the same ServerImpl was not getting registered as listener to RoutingTable hence announcement was not received on remote - fixed the same Patch 6: Some unit tests were intemittently hanging. ZMQ Test server was not gracefully shutting down."

The config now is loaded only from the coordinator. The previous way of loading
the config only if empty had some side-effects.

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

Merge "Bug fix for bug 328 https://bugs.opendaylight.org/show_bug.cgi?id=328#add_comment"

Bug fix for bug 328
https://bugs.opendaylight.org/show_bug.cgi?id=328#add_comment

Casts node_id to long before instantiatiing an Node

Change-Id: I846bfe11cef49adeea65d9d2cce6a0d7444b93f5
Signed-off-by: Dave Tucker <dave.j.tucker@hp.com>

Migrate all artifact versions to dependencyManagement

- Migrate all the artifacts to use dependencyManagement to centralize
the version changes in controller and make sure the dry-run for
release could actually rewrite those.

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

Commented out Sys.outs in runtime generated code.

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

Update to new version of the artifacts proposed by jenkins-controller-version-changes-11

Change-Id: If1cdd56839fc0e15cb2c1fa946897b58d7bcfa37
Signed-off-by: jenkins-controller <jenkins-controller@opendaylight.org>

fix of Bug 314

- corrected POST operation

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

Fix bug in hello message: module with more revisions.

When yang store discovers new yang files, it should keep all modules
provided by the parser. The old behavior was that only only one
module instance was chosen. This change results in hello message
containing all revisions of all modules.

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

Tests for loading data with augmented schema nodes with equal names

Tests loading and normalization of JSON and XML data to composite/simple node structure when
schema node structure contains several nodes with equal name which were
added through augmentation.

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

Clean up unnecessary imports in 01-md-sal.xml .

Also remove unused import of ietf-inet-types in odl-sal-netconf-connector-cfg.yang .

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

Fixing a NullPointerException in Neutron Interface (Bug-323)

Ideally, this condition should never occur. But there seems to be a consistency issue in the ML2 driver
which could potentially result in this condition. While we are addressing the root-cause in ODL ML2
driver, we should also patch our side of the interface as well to avoid failing for such conditions when
it is harmless for a few usecases.

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

InputStream for RSA file should not be static

- input stream is used in AuthProvider constructor
  and closed

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

Enhancements to remote rpc server. Using zmq router-dealer bridge to make the server async.
Enhancements to remote rpc client. Using zmq router-dealer bridge to make the client async.
Added configuration for remote rpc in Configuration subsystem
On Server startup reading of routingtable to populate remoterpcs was giving exception - fixed the same
Client was not registered with Server and addRPCImplementation was failing - fixed the same
ServerImpl was not getting registered as listener to RoutingTable hence announcement was not received on remote - fixed the same
Patch 6: Some unit tests were intemittently hanging. ZMQ Test server was not gracefully shutting down.

Change-Id: I6054443b39394f82258522205ccd4be470f597f0
Signed-off-by: Basheeruddin Ahmed <syedbahm@cisco.com>
Signed-off-by: Abhishek Kumar <abhishk2@cisco.com>

Merge "Fix for Bugs 324"

Merge "run.sh: fix -status command exit codes"

Merge "Updated port-types yang file to adhere to pyang validation. OF Plugin review: https://git.opendaylight.org/gerrit/#/c/4434/ Signed-off-by: Deepthi V V <deepthi.v.v@ericsson.com> Change-Id: I4e1d0dd355864d7ed3f6f9ce4251d20db40a5a05"

Fix: removed tabs from yang files.

Signed-off-by: Milos Fabian <milfabia@cisco.com>
Change-Id: I1f6fcba1c5dfc3eddb53adc89950ed20795dddfb

Merge "pyang fixes for table/meter types"

pyang fixes for table/meter types

Signed-off-by: Prasanna Huddar <prasanna.huddar@ericsson.com>
Change-Id: I0a60d90b0859257e1c466f1a353aa57840f2072d

Changed codec for Identityref in JSON transformation

- codec for Identityref which is used in JSON transformation can use mount point

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

Updated port-types yang file to adhere to pyang validation.
OF Plugin review: https://git.opendaylight.org/gerrit/#/c/4434/
Signed-off-by: Deepthi V V <deepthi.v.v@ericsson.com>
Change-Id: I4e1d0dd355864d7ed3f6f9ce4251d20db40a5a05

OSGI version change to 5.0.0 in ganymed subproject

Change-Id: I11a43745e91ae4d9d39862120b457eb557574447
Signed-off-by: Robert Gallas <rgallas@cisco.com>

Fix for  bug 288
https://bugs.opendaylight.org/show_bug.cgi?id=288

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

Fix for Bugs 324

Small fix to handle an NPE in TopologyMapping

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

Use version 0.6.0-SNAPSHOT of yangtools everywhere in controller.

As of yangtools commit f5e9689b95d945c22c7a38870681cab899326de9, all projects
in yangtools repo have version 0.6.0-SNAPSHOT. This commit bumps all
yangtools dependencies to this version.

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

pyang fixes for base model type. changes do not affect plugin build

Signed-off-by: Prasanna Huddar <prasanna.huddar@ericsson.com>
Change-Id: I1828a86aa1b88467435c136fbb4882ecb409fbfc

pyang fixes for inventory and topology models

Signed-off-by: Prasanna Huddar <prasanna.huddar@ericsson.com>
Change-Id: I0ef0e3a7a0ca96743d5c1d304b3a9f70714593c5

pyang fixes for service models

Signed-off-by: Prasanna Huddar <prasanna.huddar@ericsson.com>
Change-Id: I425794e060f352ccc3788c0123b1f50c832b89dc

pyang fixes for statistics module

Signed-off-by: Prasanna Huddar <prasanna.huddar@ericsson.com>
Change-Id: I7dd4c3c5bb5478abe02bf744a99163f9d76680f3

pyang error fixes for management models.
Error fixed:
   name should be same as module name.
   all warning removed.

Signed-off-by: Prasanna Huddar <prasanna.huddar@ericsson.com>
Change-Id: Ic2f510fb0a5618faaedfa396ff9e13be4f206c59

Do not wait for non-existing modules

The two capabilities removed are exposed from bgpcep concepts, so md-sal
configuration should not depend on them.

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

run.sh: fix -status command exit codes

Currently -status command exits with exit code 0 to represent both
'running' and 'not running' state when pidfile is found and -1 when
pidfile is not found (likely not running).  Overloading exit code 0 makes
wrapper scripts incapable of differentiating 'running' from 'not running.'
Make 0 == running, 1 == not running, and -1 likely not running.

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

Merge "Move protocol framework from BGPCEP project"

Merge "Fix thread safety issue with StartExi operation"

Move protocol framework from BGPCEP project

This moves the current parts of BGPCEP protocol framework into the
controller project, eliminating the circular dependency forced by
netconf being resident in this project.

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

Remove dependecies on bgpcep concepts/util

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

Merge "Do not instantiate constant lists on each message"

Merge "Switch to using yangtools version of mockito-configuration"

Merge "Fixed NetconfDevice to store capability info."

Merge "increased test coverage"

Merge "Switch testing configuration for md-sal tests to xml."

Merge "Add deprecated warning for plaintext configuration files in config persister"

Merge "Update ietf yang dependencies from rev130712 to rev131021."

Merge "Rename initial config files from *.xml.conf to *.xml"

Merge "removing the old FRM code"