Merge "Fixed YangStore exception thrown at startup"

Merge "TableFeatures Update,correcting InstanceIdentifier"

MD-SAL Statistics Manager- types and capabilities in group-features model is bitmap, so changed to bitmap from enum

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

Merge "Fix NPE in FlowProgrammerService in openflow plugin"

Merge "Adding another constructor to ContainerFlowConfig to take dlVlan. Being an immutable object, there is no other way to set the Vlan (other than via NB Serialization)"

Adding another constructor to ContainerFlowConfig to take dlVlan.
Being an immutable object, there is no other way to set the Vlan (other than via NB Serialization)

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

Merge "1. Fixed not sending of events to the node that created the routing table entry 2. Deprecated listener pattern and introduced the WB pattern based on code review 3. Included some documentation and logging package 4. Modified/Added unit test case"

TableFeatures Update,correcting InstanceIdentifier

Signed-off-by: Gaurav Bhagwani <gaurav.bhagwani@ericsson.com>
Change-Id: I3722482a52a8b26cd48474838d21bf3ecc22a580

Merge "MD-SAL Statistics Manager -Added group-id to group-statistics API"

1. Fixed not sending of events to the node that created the routing table entry
2. Deprecated listener pattern and introduced the WB pattern based on code review
3. Included some documentation and logging package
4. Modified/Added unit test case

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

MD-SAL Statistics Manager -Added group-id to group-statistics API

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

Merge "Fixed NPE in MD-FRM."

Fixed YangStore exception thrown at startup

Fixed a bug in yang store service that caused cache to be updated at illegal state

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

Fixed NPE in MD-FRM.

There was an uninitialized ConcurrentMap.  Initialized it.

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

Merge "Added flags for flow removed event"

Merge "Ganymed ChannelManager hack for subsystem command, functionality for subsystem added - tabulators replaced"

Response for PUT and POST operations

- added response as is written in http://tools.ietf.org/html/draft-bierman-netconf-restconf-02#page-37
- tests

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

Fix NPE in FlowProgrammerService in openflow plugin

- In default container have installed flows matching on input port.
  When creating the first container, default container flows are removed.
  On switch notification about flow removal, a NPE is hit in FlowProgrammerService
  because of the assumption the newly created container already has ports assigned.

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

Merge "tests for empty leaf, namespace changing, multi simple node"

Merge "Added config SI for io.netty.util.Timer and config module wrapper for HashedWheelTimer, moved netty's config SIs into netty-config-api. Added and fixed dependencies to distribution pom.xml."

Merge "Dispose HashedWheelTimer instances properly in netconf client and server"

Dispose HashedWheelTimer instances properly in netconf client and server

Prevents resource leak exception from netty

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

tests for empty leaf, namespace changing, multi simple node

following tests were added
- N elements in leaf list are red as N simple node instances
- Json [null] value is translated to simple node with null value
- for composite node for which were updated namespaces of its childs
  can't be namespaces changed again

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

Added flags for flow removed event

Signed-off-by: Deepthi V V <deepthi.v.v@ericsson.com>
Change-Id: I4765c5620f6a49fb55a993b80e1132fe73e15326

Do not re-register config instances to OSGi service registry if old instance is reused.

Keep old OSGi service registration when reusing old instance. Also move @Deprecated flag in ModuleInternalTransactionalInfo where it belongs.

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

Added config SI for io.netty.util.Timer and config module wrapper for HashedWheelTimer, moved netty's config SIs into netty-config-api.
Added and fixed dependencies to distribution pom.xml.

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

Add threadpool bundles to distribution.

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

Merge "Add blacklist capability to yang-store-impl."

Bind raw TCP netconf server to all network interfaces.

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

Add blacklist capability to yang-store-impl.

Add blacklist to overcome yuma problems when parsing buggy yang files in md-sal.

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

Ganymed ChannelManager hack for subsystem command, functionality for subsystem added
- tabulators replaced

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

Merge "Ganymed ChannelManager hack for subsystem command"

Merge "AbstractConfigTest - exposed BundleContext and ServiceRegistration mock."

Merge "Fixed quotation mark in action-types.yang"

Merge changes Ibc19654a,I17a345ea,Id0377e20

* changes:
  test of namespace suplement (from xml)
  test of namespace supplement
  Xml -> Composite node test

Merge "Json to composite node test"

Fix for Bug 144, Bug 147, 148 - improved codec

  - Added defensive locking mechanism to class generation code
  - Provider's and Consumer's crossing boundatories between
    brokers are blocked util their YANG schema is ready and processed.
  - Added set of APIs to make generated codec contracts and interactions
    more explicit.
  - Extracted augmentation serialization / deserialization to sepparate
    generated codec (one codec per augmentation type).
  - Fixed NPE

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

test of namespace suplement (from xml)

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

test of namespace supplement

test suplement of namespace when data are red from json file. to obtain
correct namespace the corresponding yang file is loaded and to supplement
it to composite node the method createConfigurationData of RestconfImpl
class is used

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

Xml -> Composite node test

Test for empty xml elements (for leaf, leaf list), for container(list)
which contains as subnodes leaf, leaflist, container, list.

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

Json to composite node test

contains tests for:
- elements leaf, leaf list, list, container under list or container
- json top level elements check
- check of read empty simple data like leaf, leaf list

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

Merge "bug 152"

Ganymed ChannelManager hack for subsystem command

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

Merge "fix of Bug 149"

Merge "added own XML to Composite node translation"

Merge "fix of Bug 145"

ID's are made in sync with openflow specification. Flow/Group/Table/Port/Queue/Meter all Id's where string. The Spec points to uint32. So all ID have been moved to uint32 type.

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

fix of Bug 145

- https://bugs.opendaylight.org/show_bug.cgi?id=145
- Broker service is tracked by ServiceTracker

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

AbstractConfigTest - exposed BundleContext and ServiceRegistration mock.

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

Fixed quotation mark in action-types.yang

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

bug 152

add a null check when setting delgates, as when implementations are unregistered, registrations are closed, the delegate is set to null. this was causing a null pointer exception.

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

Exposed components of Binding Aware MD-SAL via config subsystem.

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

Merge "moved forwardingrulesmnaager to mdsal directory"

moved forwardingrulesmnaager to mdsal directory

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

Merge "changed the pom.xml to include proper integration test"

fix of Bug 149

- added missing providers

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

added own XML to Composite node translation

- JsonReader is used for translation of XML to CompositeNode.
It creates CompositeNode and its structure with CompositeNodeWrapper and SimpleNodeWrapper
implementations. It allows to add namespaces after translation.

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

Bugfix generated method in Abstract ModuleFactory template.

Fix generated ModuleFactory#isModuleImplementingServiceInterface: Use isAssignableFrom instead of equality check when
testing if possibly super class provided as a parameter can be used instead of possibly more concrete class of implemented
service interface.

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

changed the pom.xml to include proper integration test

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

Merge "Small fix to make Eclipse happy"

Merge "MD-SAL Statistics Manager - Temporary disabling (config=true) state for stats model"

Small fix to make Eclipse happy

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

Merge "Topology Manager changes for cross checking Production switches"

MD-SAL Statistics Manager - Temporary disabling (config=true) state for stats model

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

Merge "Added flow event listener"

MD-SAL Statistics Manager - Implement request and notifications and minor modifications to the models

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

Added flow event listener

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

Error notification Message modelling

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

Topology Manager changes for cross checking Production switches

Incorporating review comments from Jason

Change-Id: I42b649d099346ac7187037ae2c5f7efd27260293
Signed-off-by: Maurice Qureshi <maquresh@cisco.com>

Merge "Added parsing of namespace in Json"

Merge "Config-manager: add synchronized to BlankTransactionServiceTracker."

Merge "Config-netconf: Remove unnecessary capability advertistments from hello message"

Merge "Modify the java-client to use the build-helper-maven-plugin"

MD-SAL Statistics Manager - Added sending request and caching response

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

Added parsing of namespace in Json

- if Json element contains namespace ( namespace:elementName ) then it is added to Node
- added test for status 404 in XML part

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

Modify the java-client to use the build-helper-maven-plugin

- Current northbound.java-client uses custom install-file and
deploy-file to actually deploy multiple artifacts. Now that has the
issue of telling to which repo the artifact should go, while it's way
simpler to let maven figure out via the distributionManagement. To
rescue in this situation the build-helper-maven-plugin can attach
additional files to an artifact distinguing them via the classifier
selector. This change is to use this feature

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

Config-manager: add synchronized to BlankTransactionServiceTracker.

Add synchronization mechanism to prevent ConflictingVersionException when loading bundles in parallel.

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

Config-netconf: Remove unnecessary capability advertistments from hello message

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

Modified flow to add setField. MeterID and group ID are needed for plugin

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

Merge "Test composite node with empty conts and lists to Json"

Get for flow/meter/group and flow update

Signed-off-by: Gaurav Bhagwani <gaurav.bhagwani@ericsson.com>
Change-Id: Id8a545c37d75e6edc9826d809d6846f8bff4a43a

Test composite node with empty conts and lists to Json

Method for building composite node with empty elements (containers, lists)
was added.

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

Merge "Fix for Bug 144"

Fix for Bug 144

  - Fixed bug in generated code which tried to access non-existing field
  - Added common root for tests dealing with models and their storage.
  - Added regresion unit test to test situation from original bug report.

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

Merge "Flow removed switch event. Group and Meter update RPC"

Merge "Fix two neutron service defects"

Flow removed switch event. Group and Meter update RPC

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

Merge "Prevent NullPointerException in config-util"

Merge "Added Json to CompositeNode translation"

Added Json to CompositeNode translation

- added JsonReader.java to provide translation from Json (InputStream) to CompositeNode
- Json has to be in one of these two formats:
  {"foo":{....}} for Container foo
  {"foo":[{....}]} for List foo
- added preconditions to translation from Node to Json:
  Data of Container and List has to be represented as CompositeNode
  Data of LeafList and Leaf has to be represented as SimpleNode
- request for unexisting data has response - not found 404

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

Prevent NullPointerException in config-util

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

Added flow and group NSF.
This review will be checked-in after 2617 review is done.
Added Flow Add/delete RPC methods.

Signed-off-by: Tony Tkacik <ttkacik@cisco.com>
Signed-off-by: Gaurav Bhagwani <gaurav.bhagwani@ericsson.com>
Change-Id: I01f65c956adb5b9d28e08f3fec2985023ee7f2e4

Merge "MD-SAL Statistics Provider"

Fix two neutron service defects

Ensure that singleton neutron network objects are properly
initialized on create.  Add case insensitive check to port
MAC address on create.

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

Merge "Replay the installedSwView data structure to populate node flows and group flows."

Merge "Added meter, action, group, flow models, mask and transactions support."

Merge "Fixed performance issues with implementation of BA-to-BI mapping"

Added meter, action, group, flow models, mask and transactions support.

   - Updated SAL-Compatibility
   - Updated Group Consumer
   - Updated Meter Consumer

Change-Id: I4854eb822723c3fc8fccdf6b05241d7c03fdb60d
Signed-off-by: Prasanna Huddar <prasanna.huddar@ericsson.com>
Signed-off-by: Ed Warnicke <eaw@cisco.com>
Signed-off-by: Tony Tkacik <ttkacik@cisco.com>

Replay the installedSwView data structure to populate
node flows and group flows.

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

MD-SAL Statistics Provider

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