Add Instruction deserializers

Inject instruction deserializers into DeserializerRegistry.
These deserializers will deserialize these instructions:

 - GoToTable
 - WriteMetadata
 - WriteActions
 - ApplyActions
 - ClearActions
 - Meter

See also: bug 7140

Change-Id: Ie38961458157cac46a4bdeba18c0e6c9020d3671
Signed-off-by: Tomas Slusny <tomas.slusny@pantheon.tech>

Add InstructionDeserializerInjector

Add injector where all instruction deserializers will be
registered and call it from general DeserializerInjector.
Create AbstractInstructionDeserializer and
AbstractActionInstructionDeserializer that will define
structure for all instruction deserializers.

See also: bug 7140

Change-Id: Icf76321ed0e51c004d003be585f40cfe4710b1b3
Signed-off-by: Tomas Slusny <tomas.slusny@pantheon.tech>

Add Action deserializers

Inject action deserializers into DeserializerRegistry.
These deserializers will deserialize these actions:

 - CopyTtlIn
 - CopyTtlOut
 - DecMplsTtl
 - DecNwTtl
 - Group
 - Output
 - PopMpls
 - PopPbb
 - PopVlan
 - PushMpls
 - PushPbb
 - PushVlan
 - SetField
 - SetMplsTtl
 - SetNwTtl
 - SetQueue

See also: bug 7140

Change-Id: I0f36092ca99de0ca51719c3c2099ebe1d904a72f
Signed-off-by: Tomas Slusny <tomas.slusny@pantheon.tech>

Add ActionDeserializerInjector

Add injector where all action deserializers will be registered and call it
from general DeserializerInjector.
Add AbstractActionDeserializer that will define structure for all action
deserializers.
Add ActionUtil#readAction that will try to deserialize action or use
extensions.
Add MessageCodeExperimenterKey for registering action deserializers with
experimenters.

See also: bug 7140

Change-Id: I1a1eaec5c9aa128f2c2d5e2a31b37086f2a56734
Signed-off-by: Tomas Slusny <tomas.slusny@pantheon.tech>

Add Match entry deserializers

Inject match entry deserializers into MatchDeserializer.
These deserializers will deserialize these match entries:

 - ArpOp
 - ArpSha
 - ArpSpa
 - ArpTha
 - ArpTpa
 - InPort
 - InPhyPort
 - EthSrc
 - EthDst
 - EthType
 - Metadata
 - VlanPcp
 - VlanVid
 - IpDscp
 - IpEcn
 - IpProto
 - TcpSrc
 - TcpDst
 - UdpSrc
 - UdpDst
 - SctpSrc
 - SctpDst
 - Icmpv4Code
 - Icmpv4Type
 - Icmpv6Code
 - Icmpv6Type
 - Ipv4Source
 - Ipv4Destination
 - Ipv6Source
 - Ipv6Destination
 - Ipv6ExtHeader
 - Ipv6Flabel
 - Ipv6NdSll
 - Ipv6NdTll
 - Ipv6NdTarget
 - MplsLabel
 - MplsBos
 - MplsTc
 - PbbIsid
 - TunnelId
 - TcpFlags

Change-Id: Ic584d4610f2b2230542e449d3fae3f4affcc2756
Signed-off-by: Tomas Slusny <tomas.slusny@pantheon.tech>

Add MatchDeserializerInjector

Add injector where all match and match entry deserializers will be
registered and call it from general DeserializerInjector.
Add MatchEntryDeserializer and MatchDeserializer to be able to actually
create match entry deserializers and deserialize match body.
Add new method to MatchExtensionHelper that can inject extension
augmentations one by one.

See also: bug 7140

Change-Id: Id15419539792cb4a21a5862be2dd423be34ab117
Signed-off-by: Tomas Slusny <tomas.slusny@pantheon.tech>

Add MessageDeserializerInjector

Add injector where all message deserializers will be registered and call
it from general DeserializerInjector
Add unit tests for MessageDeserializerInjector

See also: bug 7140

Change-Id: I28e2795148597a1c15cb52dec46728a4324a4d0a
Signed-off-by: Tomas Slusny <tomas.slusny@pantheon.tech>

Add PortMessageService

Add PortMessageService that will convert Port to PortMessage and send
this meesage to serialization.
Update SalPortServiceImpl to use PortMessageService if supported and
enabled.

See also: bug 7139

Change-Id: I73a452770a745c5c7a7debe95cbcf7e04afebb89
Signed-off-by: Tomas Slusny <tomas.slusny@pantheon.tech>

Add GroupMessageService

Add GroupMessageService that will convert Group to GroupMessage and
send this meesage to serialization.
Update SalGroupServiceImpl to use GroupMessageService if supported
and enabled.

See also: bug 7139

Change-Id: I07c2f8949f4b89d0cb921d82dfaed3159ca7da8e
Signed-off-by: Tomas Slusny <tomas.slusny@pantheon.tech>

Add MeterMessageService

Add MeterMessageService that will convert Meter to MeterMessage
and send this meesage to serialization.
Update SalMeterServiceImpl to use MeterMessageService if supported and enabled.

See also: bug 7139

Change-Id: I158ddc58deb62394c50bc2e6d195abaa3f283532
Signed-off-by: Tomas Slusny <tomas.slusny@pantheon.tech>

Add FlowMessageService

See also: bug 7139

Change-Id: Idb4f5b40ce8f033deee209eec1596df2b8ff7d10
Signed-off-by: miroslav.macko <miroslav.macko@pantheon.tech>

Add AbstractMessageService

Add AbstractMessageService that will define structure for
all other message services.

Modify AbstractService#handleServiceCall not to be final.

See also: bug 7139

Change-Id: I0bcfe8c0a5ba8ad572427240b29d7522ce0c5cf5
Signed-off-by: miroslav.macko <miroslav.macko@pantheon.tech>

Add configuration parameter for single layer

 - Add use-single-layer-serialization to openflow-provider-config.yang
 - Pass this new configuration parameter to DeviceContext

See also: bug 7139

Change-Id: I00557cd1c0c637fcf83c42e47808a5bf96a57a1b
Signed-off-by: Tomas Slusny <tomas.slusny@pantheon.tech>

Remove redundand case in nicira-action.yang

    - created probably as unattended copy/paste
    - equal cases are forbidden by yang spec and next version
      of yang parser will check this rule

See: BUG-7424

Change-Id: I83c551dc5dd613163c3f8a47a443081d71f3c328
Signed-off-by: Michal Rehak <mirehak@cisco.com>

Fix failing test.

Yangtools fixed their issue and merged it.

Change-Id: I47c3397145a53c2d2db72093b153a40da5cee571
Signed-off-by: Jozef Bacigal <jozef.bacigal@pantheon.tech>

Merge "Bug 7764 - Do no throw warning on explicit task cancellation"

Bug 7764 - Do no throw warning on explicit task cancellation

Change-Id: I9477823ffd1b4e9b28484ae8cde166a2d03f5a9f
Signed-off-by: Anil Vishnoi <vishnoianil@gmail.com>
(cherry picked from commit d813c7f656b4dd6f460b66a2fa303a448f6a4e07)

NAT Action added in Nicira Extension

Nat action is added as a port of conntrack action, it is an optional
action field.

Change-Id: Iec97d9d859e0e2060abc3a31ceb929c1b1e43b4a
Signed-off-by: Aswin Suryanarayanan <asuryana@redhat.com>

Bug 7769 - Master branch is broken because of the test failures.

Change-Id: I0f3d787011d388947b10f4fe94ed20e4a270293c
Signed-off-by: Anil Vishnoi <vishnoianil@gmail.com>

Merge "Fix comparison between port numbers in match"

Fix PacketInV10TranslatorTest

Use object instead hardcoded string

Change-Id: I6408a927e0c6c186e78884a9851f1dbea56456aa
Signed-off-by: miroslav.macko <miroslav.macko@pantheon.tech>

Merge "Replace mockito-all by mockito-core (see Bug 7662)"

Bug 7499 - ensure statistics scheduler does not die and keep trying while the controller keeps the ownership of the device

Change-Id: Ia5923f200b6c7a8888d4e3c96fdf8358603d4ac9
Signed-off-by: Jon Castro <castro.jon@gmail.com>

Merge "BUG-7608: use blueprint action-provider/action-service"

Bug 7453 - FlowRemoved doesn't have Removed Reason Information

On flow being removed from switch on idle timeout, onFlowRemoved() api was
invoked and while conversion reason was not being converted and added, so
the application using API onFlowRemoved() was not getting the reason for
flow being removed from the switch.
This change will provide the reason invoking onFlowRemoved() API

Change-Id: Id8b957b5b2cc581215fb84b8ca84d1d9a04e5619
Signed-off-by: eeiillu <suja.t@ericsson.com>

BUG-7608: use blueprint action-provider/action-service

The contract for RPC services containing Routed RPCs (RFC7950
actions) with respect to routing is different from normal RPCs.

In order to properly express dependencies, it needs a concept
of a promise to register instantiations (action-provider)
and the dependency on that promise (action-service). Use these
concepts instead of realying on rpc-service which happens to
work due to sal-remoterpc-connector (incorrecty) providing
the equivalent of action-provider for all Routed RPCs defined
in the local SchemaContext.

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

Replace FindBugs :jsr305 by full :annotation (Bug 7663)

Change-Id: I05b3a0fd993c246617808d33d743d04976e0286c
Signed-off-by: Michael Vorburger <vorburger@redhat.com>

Add isComplete callback to commitEntry

Implement commitEntry with isComplete callback
from https://git.opendaylight.org/gerrit/#/c/51064/ to
OpenflowPlugin OutboundQueueProviderImpl.

See also: bug 7140

Change-Id: I7c4224e029f175a54b46246e1d409356f2447b16
Signed-off-by: Tomas Slusny <tomas.slusny@pantheon.tech>

Fix comparison between port numbers in match

Properly extract port number from NodeConnectorId when comparing
port numbers, as NodeConnectorId is basically string, and sometimes
when going back from device, it do not contains datapath id
or prefix (for example in new serialization/deserialization path).

See also: bug 7139

Change-Id: I89999b71dba2e834ea1f37f42b5cddf1082bdd12
Signed-off-by: Tomas Slusny <tomas.slusny@pantheon.tech>

Replace mockito-all by mockito-core (see Bug 7662)

Change-Id: Iceeb2f1082a6ed3be04e99e17b2e6c3c3c4ebb31
Signed-off-by: Michael Vorburger <vorburger@redhat.com>

Fix Direct statistics RPC - actions part

Handle actions for direct statistics.

See also: bug 7232

Change-Id: Ib29f9a760bf123bac1745ea2fe8bc7e6bd3de73c
Signed-off-by: miroslav.macko <miroslav.macko@pantheon.tech>

RPC opendaylight-direct-statistics:get-flow-statistics not taking nicira
extension match

This is just a draft review, still testing in progress

Added new MatchPath RPCFLOWSSTATISTICS_FLOWANDSTATISTICSMAPLIST_MATCH.

This will differentiate between the augment model between notification and
rpc.

Change-Id: If80ff86522ba406577b15c4a3d8a30ee364001db
Signed-off-by: D Arunprakash <d.arunprakash@ericsson.com>

Merge "Bug7485 Make statistics poller parameters configurable."

BUG-5222: do not pull in odl-mdsal-xsql

xsql is deprecated, buggy and never was meant for production
use. Do not pull it in.

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

Bug7485 Make statistics poller parameters configurable.

Currently the the time intervals used for polling the statistics
are harcoded. (basicTimerDelay, maximumTimerDelay).
This patch helps makes parameters more configurable by defining
current default values in yang. Using BluePrint extensions user
can now configure the parameters without changing the source code.

Change-Id: Id06cb1e569b0d3c653c142bf5f5af5a24fd6e3ea
Signed-off-by: Sai Marapareddy <sai.marapareddy@gmail.com>

Merge "BUG 7502: Switch throws an error message for the action type"

Merge "Remove use of triemap"

Remove use of triemap

Triemap is not used anywhere, do not declare it as
a dependency.

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

Bug 7071: adding support for fin-timeout

Add support for serialization/deseralization and encoding/decoding of fin-timeout action.
This action currently is created using the learn action, so we nead only deseralization/decoding way (when flow statistics is sent to odl),
but this patch suppors also the other way of insalling flows from odl, in case it will be needed in the future.

Change-Id: I061100fab61e8d3398fddde98e76493a5afef468
Signed-off-by: Slava Radune <slava.radune@hpe.com>

BUG 7502: Switch throws an error message for the action type

       (move:NXM_NX_REG1[0..19]->OXM_OF_MPLS_LABEL[] )
       error message: OFPAT_BAD_MATCH due to wrong OXMClassCode.
       code-review contains the fix to address correct OXMclasscode for the matchtype.

Change-Id: I113e76111b4ca84e2a7c1f0f443af7330f33328f
Signed-off-by: epgoraj <p.govinda.rajulu@ericsson.com>

Merge "Bug 7481 - Flows with nicira actions get corrupted after cluster failure"

Merge "Add GroupMessageSerializer"

Merge "Add PortMessage serializer"

Merge "Add MeterMessage serializer"

Merge "Add FlowMessage serializer"

Merge "Add Instruction serializers"

Merge "Add InstructionSerializerInjector"

Merge "Add Action serializers"

Merge "Add ActionSerializerInjector"

Merge "Add Match entry serializers"

Merge "Add MatchSerializerInjector"

Merge "Add MessageSerializerInjector"

Add GroupMessageSerializer

Inject GroupMessageSerializer into OFJ. This serializer will
serialize GroupMod message to raw bytes.

See also: bug 7138

Change-Id: Ib2763904506d199d1a817449f4c08f1ec267f0cf
Signed-off-by: Tomas Slusny <tomas.slusny@pantheon.tech>

Add PortMessage serializer

Inject PortMessageSerializer into OFJ. This serializer will
serialize PortMod message to raw bytes.

See also: bug 7138

Change-Id: Iea3c90111cfc83b297e54559c4846a7949cc15df
Signed-off-by: Tomas Slusny <tomas.slusny@pantheon.tech>

Add MeterMessage serializer

Inject MeterMessageSerializer into OFJ. This serializer will
serialize MeterMod message to raw bytes.
Added unit tests for ExperimentedIdMeterBandTypeSerializerKey

See also: bug 7138

Change-Id: I6b46f6f9e7e960055692c0a33932be373a783b47
Signed-off-by: Tomas Slusny <tomas.slusny@pantheon.tech>

Bug 7481 - Flows with nicira actions get corrupted after cluster failure

Change-Id: I7fe988b54af3823e0fad672f1f3c5d7750c9095a
Signed-off-by: Tali <tali.ben-meir@hpe.com>

Bug 7415 Reducing the severity of the log message

Whenever there is a change in configuration of l2switch,
bundle refresh happens. When bundles refresh, openflowplugin
pushes the same flow. Changing the severity of of the exception
to warning since it is expected to have a flow with same id.

Change-Id: I69874d14b83053edc97b6d299c8fdb05d7ccdd8c
Signed-off-by: Sai MarapaReddy <sai.marapareddy@gmail.com>
Author: Sai MarapaReddy <sai.marapareddy@gmail.com>

Merge "Create YANG builders for all relevant model types"

Merge "Add Serialization and Deserialization injectors"

Add FlowMessage serializer

Inject FlowMessageSerializer into OFJ.
This serializer will serialize FlowMod message to raw bytes.

See also: bug 7138
Change-Id: I83038bdaab160a757dda78c4356af5a1e976d402
Signed-off-by: Tomas Slusny <tomas.slusny@pantheon.tech>

Add Instruction serializers

Inject instruction serializers into InstructionSerializerInjector.
These serializers will serialize these intrusctions to raw bytes:

  - ApplyActions
  - ClearActions
  - GoToTable
  - Meter
  - WriteActions
  - WriteMetadata

Change-Id: I72a254b897234b3e7f29c0ea2d1496f0bc517e43
Signed-off-by: Tomas Slusny <tomas.slusny@pantheon.tech>

Add InstructionSerializerInjector

Add injector where all instruction serializers will be registered and call
it from general SerializerInjector
Add AbstractInstructionSerializer and
AbstractActionInstructionSerializer
to unify all instruction serializers
Add unit tests

Change-Id: Id2c2462c950b127636f040b452226b2216e097ec
Signed-off-by: Tomas Slusny <tomas.slusny@pantheon.tech>

Add Action serializers

Inject action serializers into OFJ. These serializers will serialize
these actions to raw bytes:

  - CopyTtlIn
  - CopyTtlOut
  - DecMplsTtl
  - DecNwTtl
  - GroupAction
  - OutputAction
  - PopMplsAction
  - PopPbbAction
  - PopVlanAction
  - PushMplsAction
  - PushPbbAction
  - PushVlanAction
  - SetMplsTtlAction
  - SetNwTtlAction
  - SetQueueAction
  - SetFieldAction
  - DropAction
  - SetVlanIdAction
  - SetVlanPcpAction
  - StripVlanAction
  - SetDlSrcAction
  - SetDlDstAction
  - SetNwSrcAction
  - SetNwDstAction
  - SetTpSrcAction
  - SetTpDstAction
  - SetNwTosAction

See also: bug 7138

Change-Id: I5b42b0b2d2afd3468768e28bd11bfdcfcbbbc142
Signed-off-by: Tomas Slusny <tomas.slusny@pantheon.tech>

Add ActionSerializerInjector

Add injector where all action serializers will be registered and
call it from general SerializerInjector.
Add AbstractActionSerializer and AbstractSetFieldActionSerializer to be able
to actually create action serializers.
Add unit tests for ActionSerializerInjector
Add ActionUtil#writeAction that tries to determine what type of
action we are serializing, and serializes it.

See also: bug 7138

Change-Id: I25549b453a0a36d38f9b42de7b8087a41d438f7b
Signed-off-by: Tomas Slusny <tomas.slusny@pantheon.tech>

Add Match entry serializers

Inject match entry serializers into MatchSerializer.
These serializers will serialize these entries to raw bytes:

  - EthernetDestination
  - EthernetSource
  - EthernetType
  - InPhyPort
  - InPort
  - Metadata
  - VlanPcp
  - VlanVid
  - IpDscp
  - IpEcn
  - IpProto
  - TcpSource
  - TcpDestination
  - UdpSource
  - UdpDestination
  - SctpSource
  - SctpDestination
  - Icmpv4Type
  - Icmpv4Code
  - Icmpv6Type
  - Icmpv6Code
  - Ipv4ArbitraryBitMaskSource
  - Ipv4ArbitraryBitMaskDestination
  - Ipv4Source
  - Ipv4Destination
  - TunnelIpv4Source
  - TunnelIpv4Destination
  - ArpOp
  - ArpSourceTransportAddress
  - ArpTargetTransportAddress
  - ArpSourceHardwareAddress
  - ArpTargetHardwareAddress
  - Ipv6ArbitraryBitMaskSource
  - Ipv6ArbitraryBitMaskDestination
  - Ipv6Source
  - Ipv6Destination
  - Ipv6Label
  - Ipv6NdTarget
  - Ipv6NdSll
  - Ipv6NdTll
  - Ipv6ExtHeader
  - MplsLabel
  - MplsBos
  - MplsTc
  - Pbb
  - TunnelId
  - TcpFlags

Change-Id: I886e990aebc4ce02e5c7b571d06078aaf8033e07
Signed-off-by: Tomas Slusny <tomas.slusny@pantheon.tech>

Add MatchSerializerInjector

Add injector where all match and match entry serializers will be
registered and call it from general SerializerInjector.
Add AbstractMatchEntrySerializer and MatchSerializer to be able to
actually create match entry serializers and serialize match body.
Add AbstractExperimenterMatchEntrySerializer to serialize match
entries with experimenter id.
Add match extension serialization to MatchSerializer, using match
extension converters. This needs to be changed later, to also remove
use of match extension converters.

See also: bug 7138

Change-Id: Ifa3f0d52b384b173cf639f7d00fd5a4a3e0d30f4
Signed-off-by: Tomas Slusny <tomas.slusny@pantheon.tech>

Revert the functional behavior change introduced through

https://git.opendaylight.org/gerrit/#/c/47138/3

Signed-off-by: Anil Vishnoi <vishnoianil@gmail.com>
Change-Id: I0b8fc5b1e0763bb2b91302647037e1153fd41dc1
(cherry picked from commit 43b49485d25ebdf0dca58d46c14100a4aa7c6324)

Add MessageSerializerInjector

Add injector where all message serializers will be registered
and call it from general SerializerInjector
Add AbstractMessageSerializer to unify all message serializers
Add unit tests for MessageSerializerInjector

See also: bug 7138

Change-Id: Ie5f0652e31316ca33b6b6fb10d84bf9972a1fed5
Signed-off-by: Tomas Slusny <tomas.slusny@pantheon.tech>

Create YANG builders for all relevant model types

Add new *message builders to flow, group, meter and port
YANG models to allow us to serialize and deserialize them
properly.
Add dependency on openflowjava for model-flow-base
Update set-tp-src and set-tp-dst yang with ip-protocol

See also: bug 7137

Change-Id: Ie552e5b06778ec0d0d803e34b9628fb1c627d1ab
Signed-off-by: Tomas Slusny <tomas.slusny@pantheon.tech>

Add Serialization and Deserialization injectors

Add SerializerInjector and DeserializerInjector that will
allow us to register new serializers and deserializers in
OpenflowPlugin easily.
Add abstract classes for unit tests to speed up unit test
development.

See also: bug 7136

Change-Id: I3135dd3d0ca49585f274f5c63f4ab6b35b525b6d
Signed-off-by: Tomas Slusny <tomas.slusny@pantheon.tech>

Bug 7335 - Flow update rejected by switch generates
faulty flow entry in operational DS

When flow update request comes to plugin, it cleans up
the flow hashmap for the updated flow, but it does not
clean up the flow with the assumption that in the next
statistics cycle, this stale flow in operational data
store will be overwritten by the new modified flow,
but as reported in the issue, if installation of modified
flow fails, it will leave the stale flow in the data store.

Change-Id: I47905f980aabc83134c6eccfbf0655d48e71484b
Signed-off-by: Anil Vishnoi <vishnoianil@gmail.com>

Merge "BUG-6890:Flow-Removed Notification configuration"

Merge "Add new API's for LLDP"

Bug 7209 - Null Pointer Exception in LearnCodecUtil when add learn flow for ipv6

The exception caused by using ByteBuf::readShort
(valid range: -32,768 32,767) for ipv6 etherType '0x86dd'
The fix is to use ByteBuf::readUnsignedShort

Change-Id: I238535565a66692b5d4dbe39b8cb7809867f3cde
Signed-off-by: Vinh Nguyen <vinh.nguyen@hcl.com>
(cherry picked from commit e9deca32b7e455ba805adc35aa49d635b2a2537e)

Add new API's for LLDP

  *Right now LLDP time period is hard coded
  *This patch enables to set the time period dynamically,
   which increases the performance of the CPU
  *Increase in CPU performance is observed clearly when we do scaling

Change-Id: I959e151c1e0f11799617bc3ae679cfb7539e764a
Signed-off-by: senthil <senthil.b@tataelxsi.co.in>

BUG:6997 supporting OXM_OF_MPLS_LABEL in nicira extensiona

        Description : currently under nicira-extension, MPLS support is not
        present in nicira Move action. Mainly in ECMP use-case we need to use
        mpls_label under nicira_move_action type. The mpls_label will be using as
        example ( move:NXM_NX_REG1[0..19]->OXM_OF_MPLS_LABEL[] )

Change-Id: Ia497b963f30bf9ffcd09430db8ed85661a64859f
Signed-off-by: epgoraj <p.govinda.rajulu@ericsson.com>
Signed-off-by: Periyasamy Palanisamy <periyasamy.palanisamy@ericsson.com>

Merge "Bug 6820 - Implement SalExperimenterMpMessageService"

lower log level when stats come before flow is written to deviceflowregistry

Change-Id: Ib24b9e97c86b2d1a24610204b61fcf9237871f0e
Signed-off-by: Jamo Luhrsen <jluhrsen@redhat.com>
(cherry picked from commit ac114ea16d60aff29106fdf687e2f35a33f7e0d6)

BUG-6890:Flow-Removed Notification configuration

Changing the flag to enable/disable flow removed
notification.

Currently the flow-removed notification is controlled by the flag
notification-flow-removed-off which is on by default(value set to false).
However this logic being double-negative causes some confusion to the
end-user. Thus modifying the flag to enable-flow-removed-notification
and setting it to true by default.

The associated logic involved at oyther places have been modified as
well as a part of this commit.

Change-Id: I1452de6e3e72030608c89168f19328d9c6aa8f1c
Signed-off-by: Shuva Kar <shuva.jyoti.kar@ericsson.com>

Merge "Bug 7058 - [Helium Plugin]Stats collection issue when controller disconnect the device"

Merge "BUG-6890: Statistics-polling configuration"

Bug 7058 - [Helium Plugin]Stats collection issue when controller disconnect the device

Change-Id: I50164bf9c3f09097d211b9f40a3f2325dee38d45
Signed-off-by: Anil Vishnoi <vishnoianil@gmail.com>

Bug 6820 - Implement SalExperimenterMpMessageService

Change-Id: Ibfbdea25957b8259a6b5070fb0598172d44108bc
Signed-off-by: yunyunhan <han.yunyun@zte.com.cn>

BUG-6890: Statistics-polling configuration

Changing the current flag to enable/disable statistics
polling.

Currently the statistics polling is controlled by the flag
is-statistics-polling-off which is off by default(value set to false).
However this logic being double-negative causes some confusion to the
end-user. Thus modifying the flag to is-statistics-polling-on and setting
it to true by default.

The associated logic involved at oyther places have been modified as
well as a part of this commit.

Change-Id: Ia9a027433611912c11ff0f3166aa6f68efdc6913
Signed-off-by: Shuva Kar <shuva.jyoti.kar@ericsson.com>

Bug 7011 - Race condition in statistics collection related transaction chain handling

Change-Id: Iabc18b2814bf2f49bb6159f0bb8f9f55eda2e07d
Signed-off-by: Anil Vishnoi <vishnoianil@gmail.com>
Signed-off-by: Shuva Kar <shuva.jyoti.kar@ericsson.com>

Merge "Implement SalExperimenterMpMessageService"

Optimize port number lookups

- Remove unnecessary OpenflowPortsUtil#init() and
  OpenflowPortsUtil#close() methods and add static constructors
  instead, so BiMaps will be initialized only when really needed,
  and we do not need to call OpenflowPortsUtil#init() method
  everywhere
- Cache inverse port BiMaps in OpenflowPortsUtil to greatly reduce
  lookup time in these BiMaps in some methods
- Try to use more short version instead of deprecated
  OpenflowVersion to reduce deprecation and to reduce
  memory use (less conversions from OpenflowVersion to short)
- Use proper MAX port constant when finding max port number
- Simplify some conditions in OpenflowPortsUtil
- Use proper OF_URI_PREFIX contant in InventoryDataServiceUtil

Resolves: bug 7046

Change-Id: I1ea033963fb48416d004398e2ee72909231aaea6
Signed-off-by: Tomas Slusny <tomas.slusny@pantheon.tech>

Implement SalExperimenterMpMessageService

Add missing implementation for experimenter multipart message
RPC service and register it as routed RPC implementation

Resolves: bug 6820

Change-Id: I5646f93dd6375b729e995a9e60e6d688f1508559
Signed-off-by: Tomas Slusny <tomas.slusny@pantheon.tech>

Improve cleanup after device disconnected event

- Ensure that all contexts are removed from managers at once
  at end of disconnect chain right before unregistration from
  cluster singleton service
- Prevent closing some services multiple times during device
  disconnect chain
- Prevent some NPEs in DeviceInitializationUtils what was caused
  by connecting and disconnecting device with no time in between
- Unify all openflow managers to implement OFPManager interface
- Ensure that when we will fail to set device to SLAVE we will wait
  until device is removed from operational DS and then continue

Resolves: bug 6672
See also: bug 6710, bug 5271

Change-Id: I3845c7d2a9147125b185d3abcc3fc8dd63ba5da8
Depends-On: I15f89cbd43310b0a8e4b7ac78595eeaaa82a65ee
Signed-off-by: Tomas Slusny <tomas.slusny@pantheon.sk>

Merge "BUG-6890:"

Fix direct statistics

- Initialize statistics gathering

Resolves: bug 7007

Change-Id: I0970b3c690f0771dbd1637405e3064a984b1ef4c
Signed-off-by: miroslav.macko <miroslav.macko@pantheon.tech>

Bug: 6930 Notiifcation-suppliers was broken

Review contains changes required for making notification-supplier as per
Blueprint Injection method
- Removed Config Subsystem parts - Yang, Config XML, Modules/ModuleFactory
- Added Blueprint XML for notification-supplier
- Removed Config Dependency from features Pom
- Removed references to Config XML

Change-Id: Ifbdcb21140903a668ad0b8dd6e2df666b7e3e37f
Signed-off-by: GauravBhagwani <gaurav.bhagwani@ericsson.com>

BUG-6890:

Fix to provide default values for configurable parameters by
handling the NumberFormatException in case the configurable
parameters are either null or absent

Change-Id: Idb37565167778a900732df2dfef428847b0efd3b
Signed-off-by: Shuva Kar <shuva.jyoti.kar@ericsson.com>

Fix flow matching function

- If flow is not present in DeviceFlowRegistry,
  don't write to operational DS.
- Update SalFlowService "update" method
- Update Unit tests

Resolves: bug 6917

Change-Id: If07e94be2d2e8564a2d39d88aec0ea5ea6ac7d19
Signed-off-by: miroslav.macko <miroslav.macko@pantheon.tech>

Merge "BUG-6890: Edit to cfg file reflecting that statistics collection is turned on by default"

Revert "Revert "Move the Openflow connections blueprint to OpenflowJava project""

This reverts commit e84b8d9e2c0e38a950db4295a61b24dd77538e01.

Change-Id: I1c79ddcb0041d4599e82fa5f6ed227d530dfab97
Signed-off-by: Michal Polkorab <michal.polkorab@pantheon.sk>

Merge "Remove RoleManager and RoleContext"

BUG-6890: Edit to cfg file reflecting that statistics
collection is turned on by default

Change-Id: I8a97cb11b1b98189a04daed50961b4a912113749
Signed-off-by: Shuva Kar <shuva.jyoti.kar@ericsson.com>

Remove RoleManager and RoleContext

- no more functionality for role manager (after Singleton changes)
- all remain functionality from role-mng and ctx were moved to device-mgn and ctx

Change-Id: I1cc1ef23c45da7b9a4288194caa7f88f7534e409
Signed-off-by: Jozef Bacigal <jozef.bacigal@pantheon.tech>

BUG-6890: Updated the cfg file with detailed description
of usage

Change-Id: Ibe772a29107296cb96cf4aa48c1d3cb2dee4f3b2
Signed-off-by: Shuva Kar <shuva.jyoti.kar@ericsson.com>