Generate ADDED events for Node and NodeConnector

Components like switch manager need to see an ADDED event come in for Node and NodeConnector before
they accept changes for those elements. This commit first checks whether a Node/NodeConnector are
already known to the DataBrokerService. If they are not known then it notifies the listeners of an ADDED
event otherwise it notifies it of a CHANGED event

Also implemented node removal. NOTE: NodeConnector removed events do not seem to be coming through.

Change-Id: Idcbe7fdce37c16158614f62b56767bed624bc803
Signed-off-by: Moiz Raja <moraja@cisco.com>

Merge "DataPacketAdapter fix"

Merge "Lazily initialize jaxb context in NB application"

DataPacketAdapter fix

added incomingNodeConnector for RawPacket from incoming PacketReceived so DataPacketService dont discard it

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

Lazily initialize jaxb context in NB application

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

Fix method isSame in generated Abstract Modules.

When dealing with list of dependencies, == cannot be used. Instead,
both lists must be equal, and each dependency must be == to the old one.

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

Merge "Fixed Bug 250 - bits serialization between Binding Broker and DOM broker"

Fixed Bug 250 - bits serialization between Binding Broker and DOM broker

  - fixed serialization bug for YANG bits type when crossing brokers
  - Added regression test for bug, based on provided bug report
  - Decreased log level to debug and trace for some code generation
    functionality.

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

MD-SAL Statistics Manager - Changed Group/Meter Augmentataion location in operational data tree
Also added check, to only entertain multipart responses associated to the multipart request sent
by statistics-manager.

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

Merge "Persist service references as separate MBeans."

Merge "Switched codecs infrastructure to use yang-data-impl codecs"

Merge "Fixed RESTConf support for identity-ref build-in datatype"

Merge "Verification of keys in list"

Merge "Fix list of dependencies mapping for readable/writable jmx wrappers"

Switched codecs infrastructure to use yang-data-impl codecs

- Codecs are used for tranlation of JSON/XML values to SimpleNode values and vice versa
- Leafref is encoded as String

Change-Id: Ibe5b4a23d5dad6373a2dcf7d8723a7dff135cf2a
Signed-off-by: Martin Sunal <msunal@cisco.com>
Signed-off-by: Jozef Gloncak <jgloncak@cisco.com>
Signed-off-by: Tony Tkacik <ttkacik@cisco.com>

Fixed RESTConf support for identity-ref build-in datatype

- Created implementation of codec for identityref
- Identityref is supported in XML and JSON
- Tests

Change-Id: I813f1f52e990317b6adf30f1d0d8a164c60a4901
Signed-off-by: Martin Sunal <msunal@cisco.com>
Signed-off-by: Tony Tkacik <ttkacik@cisco.com>

Verification of keys in list

- verification of keys in list against list yang structure

Change-Id: Ieb46eea67a7d3173958184ced10cec1b853bb7fe
Signed-off-by: Martin Sunal <msunal@cisco.com>
Signed-off-by: Tony Tkacik <ttkacik@cisco.com>

Merge "Netconf SSH user authentication using UserManager"

Netconf SSH user authentication using UserManager

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

Merge changes I1474351f,I2ddc5ffa

* changes:
  FRM to properly handle timed flows in case of switch down/up
  Adding password salting

Merge "Use equalsIgnoreCase when comparing subnet name to default subnet. Using equals or == is not accurate enough. Also, move declaration of variable to be where it's used, to save its allocation before an 'if' validation that's not related to it."

Use equalsIgnoreCase when comparing subnet name to default subnet.
Using equals or == is not accurate enough.
Also, move declaration of variable to be where it's used,
to save its allocation before an 'if' validation that's not related to
it.

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

Added Missing DELETE to allowed CORS methods.

Change-Id: I4415666588fec3bb7f7c11b4850817ad006a8353
Signed-off-by: Thomas Bachman <tbachman@yahoo.com>

Merge "Changed zeromq-routingtable module to remoterpc-routingtable based on codereview comments"

Fix date initialization in NetconfOperationServiceImplTest.

Use SimpleDateFormat instead of Date(0) as the later fixes the date to UTC rendering local date to
1969-12-31 for timezones > 0

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

Persist service references as separate MBeans.

Create dummy MBean - ServiceReferenceMXBean - that contains link to implementation of the service as an attribute.
Add support for resolving of SR to dependency resolver. Each time user saves SR, create MBean and
add the interface QName together with user defined reference name to ObjectName of the newly created SR MBean.

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

Fix list of dependencies mapping for readable/writable jmx wrappers

Attributes for list of dependencies can be accessed from (testing) code

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

On openflow plugin restart, NPE in tx poller

- In TopologyServiceShim tx poller thread task does not
  check is IOFStatisticsManager provider is available.
  Protocol plugin restart is a way to consistently
  expose the issue.

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

FRM to properly handle timed flows in case of switch down/up

- When node reconnects to controller, plugin will send remove all flow message.
  FRM will receive flow removed message and should toggle the installInHw flag
  only for the timed static flows, not all of them. At the end a timed flow with
  original configured timeout (idle or hard) is no longer valid later on.
- Fix bug in flow ui

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

Changed zeromq-routingtable module to remoterpc-routingtable based on codereview comments

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

Merge "Added tests for reading: - operational data - configuration data"

MD-SAL Statistics Manager: Added support for queue statistics collection

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

Merge "Improve iteration over subnets map by using entrySet instead of keyset. It is more efficient to iterate  on the entrySet because it avoids the Map.get(key) lookup."

Merge "Simplify hasNodeConnector method in Subnet class. Reduce 'if' conditions for improved readability."

Adding password salting

- When A UserConfig object is created, generate its 64 bit long salt
  and use it to salt the clear text password before hashing.

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

Merge "Add consistency check of visible yang modules to netconf."

Merge "Fixed netconf monitoring."

Simplify method isMutualExclusive in Subnet.
Remove redundant 'if' statements.

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

Merge "Fixed bug in cross-broker RPC routing"

Simplify hasNodeConnector method in Subnet class.
Reduce 'if' conditions for improved readability.

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

Added tests for reading:
- operational data
- configuration data

Change-Id: I8d771ac061d0d7cc5c4187133e2bf0a3ddb87252
Signed-off-by: Ladislav Borak <lborak@cisco.com>

Improve iteration over subnets map by using entrySet instead of keyset.
It is more efficient to iterate  on the entrySet because
it avoids the Map.get(key) lookup.

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

Refactor Subnet.isSubnetOf - reduce number of 'if' statements.
Added unitests.

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

MD-SAL Statistics Manger - Added support for fetching port statistics

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

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 bug in cross-broker RPC routing

  - Fixed cross-broker (Binding to DOM) and (DOM to Binding)
    rpc routing by adding explicit routing and invocation strategies

  - Lowered reporting level for fingAugmentableArgument to DEBUG
    so it does not polute logs

  - Extracted and converted RpcProviderRegistry implementation
    from BindingAwareBrokerImpl (Xtend) to RpcProviderRegistryImpl
    (java).

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

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

Add consistency check of visible yang modules to netconf.

Netconf connector to config subsystem performs this check before a netconf session is established.
This check consists of comparing yang modules visible to config subsystem to modules visibile to yangstore.
If an inconsistency is detected, session negotiation is cancelled and the session is dropped.

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

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

Fixed netconf monitoring.

-added extension of ietf-netconf-monitoring
-fixed reporting of host address
-fixed reporting of transport
-added reporting of session identifier (client,persister)
-fixed send netconf message with additional header

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

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>