Workaround for karaf bug KARAF-3069

https://issues.apache.org/jira/browse/KARAF-3069

More recent maven-bundle-plugins are putting

Require-Capability: osgi.ee; filter="(&(osgi.ee=JavaSE)(version=1.6))"
into MANIFEST.MF.  This causes them to not
be loadable in karaf when using the equinox framework (though
felix works fine).

As noted in the bug, this is the workaround.

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

Merge "Bug 1262 - Document Transaction Chaining of MD-SAL"

 BUG 1606 - A read is attempted on a WRITE_ONLY transaction

Change-Id: I896d976ff387ecdc9ecfe6e4cb2ead75b15c15f5
Signed-off-by: Basheeruddin Ahmed <syedbahm@cisco.com>

Merge "changed the label for seed node config"

Merge "Configuration service uses an inconsistent hashcode in the clustered data structure"

Configuration service uses an inconsistent hashcode in the clustered data structure

- Using Enum in the Key field of a clustered data structure can lead
to propagation issues, because the clustering services implementation
relay on a consistent hashcode to be calculated. So replaced the Java
Enum used in configuration.implementation bundle with String.

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

Bug 1262 - Document Transaction Chaining of MD-SAL

Improves and clarifies the transaction chain concept and related APIs.

Change-Id: Ib509cdef33f8b51f8bd142b93657eace489e03a1
Signed-off-by: Tony Tkacik <ttkacik@cisco.com>
Signed-off-by: Robert Varga <rovarga@cisco.com>

Merge "Incorrect name of elements in yang case nodes"

Merge "Resolving Karaf Northbound API issues with a few related fixes"

Merge "Removed getClass comparison if FlowComparator."

Merge "Remove unused getter"

Merge "BUG-868: remove unused things"

Merge "BUG-868: remove deprecated call"

Resolving Karaf Northbound API issues with a few related fixes

1. Upgrading Apache Tomcat Patch version from 7.0.32 to 7.0.53 (https://git.opendaylight.org/gerrit/10110)
2. No impact to the legacy distribution.
3. Fixing the Jackson - Jersey binding by fixing the Classloader confusion where multiple bundles exports javax.ws.rs
4. Adding a temporary hack for the Http authentication by adding a karaf-tomcat-security bundle
   The existing security bundle is rift with dependencies into a lot of deeply chained AD-SAL NSF and needs a lot of pruning.
   Hence in order to make progress in Karaf testing and development, Have bypassed it temporarily and we will fix it soon.

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

Small fix on dependencies for karaf features.

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

Remove unused getter

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

BUG-868: remove unused things

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

BUG-868: remove deprecated call

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

Changed gossiper test to use akka system's LocalRefProvider

This improves unit test's performance and reliability

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

Added more tests for Bucket store

Changed bucket store tests to use akka's LocalRefProvider from ClusterRefProvider
to improve performance. Also added more unit tests.

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

changed the label for seed node config

Change-Id: Iafbd7d781f0fb41bff8e0194954809febecf909c
Signed-off-by: Basheeruddin Ahmed <syedbahm@cisco.com>

Merge "Karaf odl-nsf-managers feature fails due to missing topologymanager shell. Fixing that."

Removed getClass comparison if FlowComparator.

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

Merge "Bug 1586: Do not use JaxRS 2.0 unnecessarily"

Merge "Converted Inventory Manager to use Transaction Chaining"

Merge "Fixed possible NPE in flow  reconciliation"

Merge "Bug 1585 - switch features removed after node updated"

Merge "Revert "Revert "BUG-1425: Integrated new Binding to Normalized Node codec for write path"""

Merge "SupervisorStrategy caught throwable - logging"

Merge "Correct the self usage in ThreePhaseCommitCohort Seems this correction missed from another commit"

Merge "BUG-1568 Getting rid of StreamGobller from netconf ssh client."

Merge "BUG-1541 Make EXI and clientConnectTimeout configurable in testtool."

Merge "Fix race condition in SSH NETCONF server"

Incorrect name of elements in yang case nodes

Name of child node among all cases of choice has to be unique.

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

Bug 1586: Do not use JaxRS 2.0 unnecessarily

As it turned out our container uses jax-rs 1.0 so we removed
unnecessarily call to 2.0 constructor.

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

Merge "Patch to eliminate sleeps in RemoteRpc tests"

BUG-1568 Getting rid of StreamGobller from netconf ssh client.

We have another thread in SshClientAdapter that works as StreamGobller
http://www.ganymed.ethz.ch/ssh2/FAQ.html#blocking

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

Merge "NETCONF scalability: share instances"

Merge "BUG-1493: split off recursion tracking and rework it"

Merge "BUG-1493: Add test suite documentation"

Karaf odl-nsf-managers feature fails due to missing topologymanager shell. Fixing that.

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

Patch to eliminate sleeps in RemoteRpc tests

Instead of using sleeps to wait for message processing to complete
the tests now installs a probe on the subject(actor). Subject(actor) sends
message to the probe when it processes message. The test monitors the
probe for specific messages and knows when the message has been
processed by the subject(actor).

Patch#2: Fixed probe creation and updated assertions

Change-Id: I10652b59e4138adbe6c6821d4411b06e1b3f1598
Signed-off-by: Moiz Raja <moraja@cisco.com>
Signed-off-by: Abhishek Kumar <abhishk2@cisco.com>

Small fixes to feature files.

As it turns out... the default artifact type
for config files is jar from commons/opendaylight/pom.xml

This is not corrected there because jar *is* the thing that
is used by the legacy distributions...

But if we don't correct it here, if you try to test a feature
against an empty .m2... the test will fail.

(Yes, this is how I spend my evenings, emtpying out my .m2
directory :( )

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

Merge "Increasing the time-out to allow for message processing."

SupervisorStrategy caught throwable - logging

logging stacktrace at warning level
patch2: increased timeout for one of the test case as build was failing

Change-Id: Ia09dcfacd90f11ad725c7601e0a71ac3983f0dbf
Signed-off-by: Basheeruddin Ahmed <syedbahm@cisco.com>

Correct the self usage in ThreePhaseCommitCohort
Seems this correction missed from another commit

Change-Id: Iacd8055b0e59238cf3377f250de0f0e17f4e3e98
Signed-off-by: Basheeruddin Ahmed <syedbahm@cisco.com>

BUG-1493: split off recursion tracking and rework it

This patch reworks tracking of recursion state into a separate class.
This allows us to unify the four call sites and provide a simple
heuristic. A state object is allocated each time we enter a new child
and its buidlers are collected when we emerge from it -- resulting in
more accurate event reporting, eliminating false positives.

Further optimization allows us to stop recursing as soon as we know no
listeners are affected by the outcome at a particular level.

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

BUG-1493: Add test suite documentation

This patch adds some sorely needed documentation.

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

Merge "BUG 932 - Swagger HTTP POST contains incorrect object"

NETCONF scalability: share instances

Scalability testing shows that the netconf session code:
- retains XML document reference
- does not reuse QName instances

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

BUG-1541 Make EXI and clientConnectTimeout configurable in testtool.

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

Fix race condition in SSH NETCONF server

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

Merge "Fix exception when netconf device provides YIN schemas"

BUG 932 - Swagger HTTP POST contains incorrect object

POST link now contains specific JSON structure which ends with POST suffix
and contains all containers and lists which are direct subchildren of
current node (URI link). It is list of possible objects which can be used
in POST request. Concretelly are these objects specified below this *POST
JSON object and are also all specified in Parameters section of GUI POST
URI link with ** prefix. Two stars (**) mean that only one of parameters
with ** prefix should be specified.

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

Increasing the time-out to allow for message processing.

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

Added a missing dependency to codahale's metrics
package in remoterpc bundle

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

Merge "Bug 1534: Changed blocking calls to async in dist data store"

Merge "Custom mailbox that is bounded and instrumented."

Merge "Moving common code out of remote rpc to clustering common bundle"

Merge "Turn on feature testing for those features that pass"

Merge "Moved the resources to a separate plugin so they can be used by others."

Merge "Revert "Added the Buffer-Id to packet-in.""

Converted Inventory Manager to use Transaction Chaining

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

Fixed possible NPE in flow  reconciliation

Code assumed non-null lists when processing augmentation
but Binding Specification allows null to be present.

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

Revert "Added the Buffer-Id to packet-in."

This reverts commit 41a6be9bc180cd81e24e5872ca0682e15b37a7f6.

This revert was because it *appears* this patch is breaking integration (not sure how yet).

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

Fix exception when netconf device provides YIN schemas

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

Merge "Added the Buffer-Id to packet-in."

Bug 1585 - switch features removed after node updated

- use tx.merge rather than tx.put when node or nodeConnector is updated

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

Custom mailbox that is bounded and instrumented.

This is for use with Akka actors. It enables monitoring
of actor's mailbox size via JMX.

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

Turn on feature testing for those features that pass

(includes a fix for features/config)

From this point onward, those features should
break the build if they are broken.

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

Moved the resources to a separate plugin so they can be
used by others.

The idea here would be to allow the projects to build
their own local runnable instances, as well as integration,
while keeping things in sync.

A couple of notes:
1)  Please be patient, this is a pull apart of work to
get features working in general, so there may be
some extra things in commons/opendaylight/pom.xml
for that are for that follow on patch.  Mea culpa.

2)  I created an opendaylight-karaf-empty distribution
that *just* pulls in the resources to be used
for feature testing (see next patch)

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

Propogating cohort failure back to calling actor

Created new ThreePhaseCommitCohortFailureTest negative test cases

patch 2: updated based on Tom's code review comments
patch 3: reformatted Shard class
patch 4: minor comments taken care of.
patch 5: review comments taken care of
patch 6/7: getSender/getSelf issue fixed
Change-Id: I403d8e7cbf44739b16a21345c645b7b66f9f8ac6
Signed-off-by: Basheeruddin Ahmed <syedbahm@cisco.com>

Moving common code out of remote rpc to clustering common bundle

Change-Id: Iad469f809ac55b2d873ff4d48e912ad66184c68d
Signed-off-by: Harman Singh <harmasin@cisco.com>

Merge "Bug 1430: Obtain config params from config system"

Merge "Modifications to error handling in restconf"

Merge "BUG-1541 Netconf device simulating testtool"

Merge "BUG-1541 Extract AuthProvider interface"

Merge "BUG-1535 Ignore exception on toaster.close"

Merge "BUG-1558 Fix BindingAwareActivator ignoring removedService call."

Merge "Fix precondition formatting"

BUG-1541 Netconf device simulating testtool

Produces executable jar file.

The jar can simulate arbitrary number of netconf devices that listen on configured ports
Schemas are loaded from a provided folder
Can also generate ODL initial config files for started simulated devices to support testing with ODL distribution

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

BUG-1541 Extract AuthProvider interface

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

BUG-1535 Ignore exception on toaster.close

It tries to delete data from store, but if the broker gets closed it cannot remove data since the broker throws exception

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

BUG-1558 Fix BindingAwareActivator ignoring removedService call.

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

Bug 1534: Changed blocking calls to async in dist data store

Changed the read and commit methods to use async akka actor calls instead of
submitting tasks to a ListeningExecutorService that do blocking akka
calls. This obviates the need for the ListeningExecutorService.

Change-Id: I7a3f369917431067ad1817a3ab53a358bc21f123
Signed-off-by: tpantelis <tpanteli@brocade.com>

Merge "bug 537 - Node Reconciliation"

Revert "Revert "BUG-1425: Integrated new Binding to Normalized Node codec for write path""

This reverts commit be2b35105f8e10ada7d73f4523a5a931c57323b3,
reinstating the new codecs, with hopefully all the users able to cope.

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

Merge "Bug 1503: RestconfError "data-missing" is error code 404."

Merge "BUG-1554: fill out ID/KEY"

bug 537 - Node Reconciliation

https://wiki.opendaylight.org/view/OpenDaylight_OpenFlow_Plugin:Backlog:Node_Status_Reconciliation

* FRM Reconciliation Notify Listener (DataChangeScope.BASE for FlowCapableNode in DataStore/OPERATIONAL)
* FlowCookieProducer - MapHolder for an unique relation flow_cookie to flow_id via FRM
* FlowStatTracker changes
        * search flow_id by flow_cookie for exist flow
* add flow_id - flow_cookie for new Flow
* FRM migration for new DataBroker API
* merge with new DataStore/OPERATIONAL hierarchy (Flow vs. FlowStatistics)
* fixed flow delete - generated cookie is bypassed with mask=0

In additional commit has fixed Bug 282 - Pre-configuration of flows causes exception
+ remove the transaction validators - not needed for now

Change-Id: Iadc2b111780524b9a8fd86cfc248e5fa194e6638
Signed-off-by: Vaclav Demcak <vdemcak@cisco.com>
Signed-off-by: Michal Rehak <mirehak@cisco.com>

BUG-1554: fill out ID/KEY

When a list is being inserted it is required to fill out both its key
and its mandatory leaves.

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

Merge "Adding more unit tests for remote rpc connector and Integrating routing table"

Merge "Hotfix bug 1556 - FRM: NPE thrown in notification"

Bug 1503: RestconfError "data-missing" is error code 404.

The base specification listed the error code to be 409, which is in
direct violation on HTTP. The restconf specification will be fixed in
the upcoming draft, meanwhile we are going for the new, sane behavior
regardless.

Change-Id: I7a6cf67a794f8348d746ba94b7de9580d0472a61
Signed-off-by: Vratko Polak <vrpolak@cisco.com>
Signed-off-by: Robert Varga <rovarga@cisco.com>

Hotfix bug 1556 - FRM: NPE thrown in notification

* FlowCapableTopologyExpoerter - flow:1 is not exist because it is not created correctly

Change-Id: Ifa5b984329796e5df7aa0f71b1932b73d0ff99a0
Signed-off-by: Vaclav Demcak <vdemcak@cisco.com>

Merge "Revert "BUG-1425: Integrated new Binding to Normalized Node codec for write path""

Revert "BUG-1425: Integrated new Binding to Normalized Node codec for write path"

This reverts commit eb2bd48a9148a4fbeae292ffd03e30083c2ff024.

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

Merge "write /nodes/node/table/0 with ensure parents when node first appears"

Merge "Bug-1520 Config subsystem:config-impl tests"

write /nodes/node/table/0 with ensure parents when node first appears

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