Merge "Adding Enum for Output -Port Values"

Merge "Revert "Reverting until appropriate Gerrit for Openflowplugin can be provided.""

Adding Enum for Output -Port Values

Signed-off-by: Gaurav Bhagwani <gaurav.bhagwani@ericsson.com>
Change-Id: Ic77422b4ce0b1f047df25ce820001707e90813d8
Signed-off-by: Ed Warnicke <eaw@cisco.com>

MD-SAL Statistics Manager - Adding support for individual flow stats
and aggregate flow statistics.

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

Merge "Added support for union yang types to netconf."

Merge "model of switch features in md-sal"

Merge "Fixed union serialization and deserialization"

Fixed union serialization and deserialization

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

model of switch features in md-sal

flow-node-inventory.yang:
added container switch-features into flow-node which contains list of
different capabilities

Change-Id: Icee61676c3e7e2dc2391aac8c4995f51d05d84a3
Signed-off-by: Juraj Sebin <jsebin@cisco.com>

Revert "Reverting until appropriate Gerrit for Openflowplugin can be provided."

This reverts commit e192dfabeb6003044e9c99da9694e4a6cd1361a1.

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

Merge "Make sure the target-ide are excluded by checkstyle"

Merge "This sanity test is sporatically failing in Jenkins for no good reason.  Reverting to restabalize "

Merge "Reverting until appropriate Gerrit for Openflowplugin can be provided."

This sanity test is sporatically failing in Jenkins for no good reason.  Reverting to restabalize 

Revert "Bug 116 - Revisit SanityTest"

This reverts commit effdf86b58e39060b75e6090596b123ec6e8ef69.

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

Reverting until appropriate Gerrit for Openflowplugin can be provided.

Revert "VLAN bug fixes"

This reverts commit 18e3ef9f1d021b9f5fef70e8a5310b399c4c6459.

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

Merge "VLAN bug fixes"

Added support for union yang types to netconf.

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

Make sure the target-ide are excluded by checkstyle

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

Merge "Notifying listeners when a subnet gets added or removed"

VLAN bug fixes

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

Notifying listeners when a subnet gets added
or removed

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

Merge "Openflowyang - added StripVlan, nwTos changes"

Openflowyang - added StripVlan, nwTos changes

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

Merge "Send SystemName TLV in LLDP message."

Merge "Add Set<String> getAvailableModuleFactoryQNames() to config-api."

Merge "Added xml storage adapter bundle fragments for config persister."

Merge "Added more granular error reporting during commit validation."

Merge "removed parent dependency on pax-exam-container-native"

Add Set<String> getAvailableModuleFactoryQNames() to config-api.

Allow querying Config Registry and each transaction for set of qnames belonging to ModuleFactory classes.
This is helpful for netconf - allows checking for race conditions between yang store and config-manager.
Modify config code generator to add @ModuleQName annotation to each abstract factory class.

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

Added xml storage adapter bundle fragments for config persister.

File and Directory based xml adapters are now available.
Only File xml adapter is used so far to persist current configuration.

Directory based persister is still using plain text to keep backwards compatibility.

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

Send SystemName TLV in LLDP message.

Some applications may need System Name TLV, therefore include
SystemName TLV in the LLDP message sent by Controller

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

Added more granular error reporting during commit validation.

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

Merge "Improve logging of persister and netconf client."

Merge "Remove jolokia dependency from config-subsystem."

removed parent dependency on pax-exam-container-native

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

Improve logging of persister and netconf client.

Add a debug message measuring time spent waiting for netconf server to reply in NetconfClient.
Log with error level failure to send/recieve netconf message in config-persister-impl.
Add TODOs to config-manager's dependency resolver for detecting cycles.

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

Merge "Bug 116 - Revisit SanityTest"

Remove jolokia dependency from config-subsystem.

Remove jolokia client from config-util and tests that were starting jolokia.

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

Merge "Get rid of legacy ref_ prefixes in configuration"

Merge "Added support for persistance of service references to netconf."

Merge "Add service reference binding to config registry."

Get rid of legacy ref_ prefixes in configuration

Change-Id: I0c3a6f05bcfcf71ebf857ce7e2355af008f68793
Signed-off-by: Robert Varga <robert.varga@pantheon.sk>

Added support for persistance of service references to netconf.

Created reference names under services/ subtree are now persisted and available for further edit-config operations.

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

Add service reference binding to config registry.

Add a database for persisting service references between commits. In netconf modules
are wired together via link in /services/service xml subtree. Each service has an
xpath pointer back to a /modules/module that implements the service. Consumer of a
service then maintains a link to the service via its namespace and reference name.
This commit fixes problem of reference names not surviving commits.

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

Merge "Fixed deserialization of IdentityRefs in Restconf URI."

Bug 116 - Revisit SanityTest

  - Sanity test is now an integration test
  - Introduce maven-controller-plugin to handle startup & shutdown
    of controller process and run away processes
  - Fix run.bat issue with -start option

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

Merge "Bug 211 - Fixed codec generation when transitive dependencies (parents) are not available"

Fixed deserialization of IdentityRefs in Restconf URI.

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

Bug 211 - Fixed codec generation when transitive dependencies (parents) are not available

  - Classloader for each encountered generated class is added to javassist ClassPool
    if not already there, to make sure we have full visibility into supertype hierarchy
    of class

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

Merge "Controller ignores switch, if no ports are present"

Merge "Fixing a recent breakage in Flow programmer introduced by https://git.opendaylight.org/gerrit/#/c/3493/"

Fixing a recent breakage in Flow programmer introduced by https://git.opendaylight.org/gerrit/#/c/3493/

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

User manager to hash with SHA-384

- When strong password check is enabled, hash the
  users password with SHA-384 instead of SHA-1.
  (Password salting will be considered in future)
- Other minor changes: organize parameters, remove
  printStackTrace()

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

Deleting the Default subnet gateway returns success

- Need to let caller know this operation is not allowed

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

Adding of specific flow validation in of plugin

- An of 1.0 switch silently accepts a flowmod which
  match does not have proper hierarchy of fields set.
  It installs the flow discarding the incorrect fields.
  This would cause inconsistency between flows in hw
  and flows in sw. The application needs to be notified
  that the flow would not be accepted as is by the openflow
  switch.

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

Fix to allow RESTCONF PUTing of Flows

- Due to a bug in the JAXB spec, which is propogated to
JAXRS, which forbids '/', even escaped, in path segments
in URLs... we can't use the nesting by refs under /flows.

- Therefore I've moved flow config to
nodes -> node -> table -> flow

- Making that change in the existing FlowConsumerImpl proved extremely
hard.  Due to the press of time I've introduced a very simple set of
FlowProvider, FlowCommitHandler, FlowTransaction, FlowTransactionValidator,
to cleanly separate the logic.  We can migrate over the validation rules as
need be.

PatchSet 11: Abstract the Transaction.  Add Groups.
PatchSet 12: Fixes for Meters

PatchSet 13: Yet another rebase

PatchSet 14: Squashing https://git.opendaylight.org/gerrit/#/c/3304/8

Must proceed: http://git.opendaylight.org/gerrit/3596/1
in openflowplugin

Change-Id: I1f1cd04a04f7a30630062725e63112cda4b049f1
Signed-off-by: Ed Warnicke <eaw@cisco.com>
Signed-off-by: Tony Tkacik <ttkacik@cisco.com>

Controller ignores switch, if no ports are present

 - At switch connection, InventoryServiceShim mistakenly skips to
   inform the global listeners if no ports are configured on the switch.
   This is not visible with software switches because they always
   advertise they have the OFLocal port active.
   Being Connection Service a listener of global inventory service,
   it will not get to know about the switch presence, and in subsequent
   queries from InventoryServiceShim it will reply the switch is not
   locally connected to this controller, causing Inventory Shim to
   not inform its upper layer listeners about the switch. Therefore
   no other controller service will get to know the switch is conencted.

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

Merge "Changed maximumEntries to correct int rather than long"

Merge "Prevent password hash from being shown"

Change portId to accept a generic data-type

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

Merge "Added conflict handling between configuration and state choice nodes. -unique node naming required"

Merge "Corrected response status codes from restconf"

Merge "Implementation for enabling remote rpc calls between 2 instances of md-sal"

Merge "Improved support for default yang statements in configuration subsystem"

Merge "Fix for bug 211, where direct write and read of augmentation was not processed correctly"

Merge "Fix errors reported by pyang"

Merge "Update service references to new format"

Fix for bug 211, where direct write and read of augmentation was not processed correctly

  - Fixed TypeNotAvailable issue with end-to-end Openflow

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

Merge "Validation Issue with DlSrc,DlDst MacAddress"

Added conflict handling between configuration and state choice nodes.
-unique node naming required

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

Corrected response status codes from restconf

- POST /restconf/config/* - 204 no msg body
- POST /restconf/operations/* - 200 msg body or 204 no msg body or 500 operation failed
- PUT /restconf/config/* - 200 no msg body

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

Improved support for default yang statements in configuration subsystem

Generated code for configuration contains default values with correct constructor if specified for leaves
Fixed binary type (de)serialization

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

Implementation for enabling remote rpc calls between 2 instances of md-sal

 - This provides implementation for enabling remote rpc calls between 2 instances of md-sal.
   The current implementation enables remote execution of globally unique services in the
   cluster. For details, please refer to this wiki page
   (https://wiki.opendaylight.org/view/Zeromq_connector). This wiki page is a draft.
 - Added relativePath in pom so that parent pom can be found.
 - Removed dependency to sal-infinispan-routingtable
 - Exported "impl" as well from zeromq-routingtable. Fixed dependencies in RouterTest.
 - Removed oss.sonatype release repo from md-sal pom. ODL nexus repo mirrors it.
 - Updated server code to handle exception
 - Server code now uses WB pattern instead of listerner pattern.
 - Fixed pom so that parent can be resolved
 - Rebased due to changed in unmerged dependency
 - Added state machine to RpcSocket.
 - Added unit tests to RpcSocketTest and SocketManagerTest.
 - Added CompositeNode methods to ExampleConsumer & XML files for creation of CompositeNodes
 - Added CompositeNode testcases to RouterTest
 - Translated scala code to java
 - Added code to convert CompositeNode to xml and back to help
 - with serialization.
 - Added more unit and integration tests.

This is squash for:

https://git.opendaylight.org/gerrit/2882
https://git.opendaylight.org/gerrit/3022
https://git.opendaylight.org/gerrit/3028
https://git.opendaylight.org/gerrit/3159

Change-Id: I44739fd8ad61043c2e786875bb7787e3fa68e435
Signed-off-by: Abhishek Kumar <abhishk2@cisco.com>
Signed-off-by: Tony Tkacik <ttkacik@cisco.com>
Signed-off-by: Alex Fan <railor33@gmail.com>

Merge "Fix a typo in debug message"

Fix errors reported by pyang

Change-Id: Ia94339d7e9e759d02070a94507a340c80bc46100
Signed-off-by: Robert Varga <robert.varga@pantheon.sk>

Update service references to new format

Change-Id: I725ef68c983564db9e9bdcc8404696c0a3df71bc
Signed-off-by: Robert Varga <robert.varga@pantheon.sk>

Merge "Fixed sal-netconf-connector readConfiguration"

Fixed sal-netconf-connector readConfiguration

  - Promoted Netconf client to v1.1.
  - Added reconnect retry.
  - Fixed readConfiguration for mounted netconf device.
  - Fixed serialization issue in restconf related to mount points with unknown schema.

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

Merge "Fixed add/delete/modify RPC for group/flow/remove reach the test provider"

Fix a typo in debug message

Change-Id: Ia37a2b49ae0cfb1196997d82df04d5edb7fffba7
Signed-off-by: Robert Varga <robert.varga@pantheon.sk>

Merge "Added transactions statistics for BI Broker."

Merge "Updated xpath pointing to a module instance under services subtree."

Merge "Refactor yang-jmx-generator and -plugin to support list of dependencies."

Refactor persister: Add ability to publish multiple snapshots while loading initial configuration.

Persister adaptors now return list of snapshots to be pushed to netconf at server startup. Persister maintains an ordered list of adaptor
instances, each with its configuration. During server startup it iterates the list backwards until non empty list of snapshot is returned.
Each snapshot can depend on different capabilities, once all capabilities for given snapshot are announced by server, persister pushes this
snapshot.

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

Fixed add/delete/modify RPC for group/flow/remove reach the test provider

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

Validation Issue with DlSrc,DlDst MacAddress

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

Fix for adding a static flow with action as OUTPUT for generic type

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

Prevent password hash from being shown

The URL 'http://controller-ip:8080/admin/users' no longer displays the
user's password hashes

Change-Id: I0664bf21d1087a437ab9420c9e39b3129620aefa
Signed-off-by: Andrew Kim <andrekim@cisco.com>

Changed maximumEntries to correct int rather than long

-Added maximumEntries to needed parts in NodeTableStatistics

Change-Id: I07fa635562b8514aad17cf4a9c6f22a419dfb750
Signed-off-by: Katelyn Kasperowicz <kkaspero@cisco.com>

Merge "1. Corrected Ip Protocol match field. 2. Added mask for Ipv6 Extension header match field"

Merge "modified group"

Merge "Initial implementation of netconf monitoring module according to tools.ietf.org/html/rfc6022"

Added transactions statistics for BI Broker.

Added implementation of gathering transaction statistics when binding independent broker is used.

Signed-off-by: Lukas Sedlak <lsedlak@cisco.com>
Change-Id: Ia7fb9374575be62bb6bd6b2ace3112d28524bde5

Refactor yang-jmx-generator and -plugin to support list of dependencies.

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

Merge "Used log instead of exception"

1. Corrected Ip Protocol match field.
2. Added mask for Ipv6 Extension header match field

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

Merge "Fix for CLI- AddMDFlow having problem"

modified group

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

Used log instead of exception

- logging was added instead of throwing exception in problematic part

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

MD-SAL Statistics Manager - Fixed keys of group and meter models and their sub component

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