BUG 3548 - LLDP speaker doesn't start send LLDP packet on port up.

Listener on changes on changes on State subtree was added.

If instance of FlowCapableNodeConnector with port down is received
in NodeConnectorInventoryEventTranslator then it is stored to local map.
When event port up event is received then stored instance is looked up
in map, state part is replaced (with port up information) and it is sent
to LLDP.

Change-Id: Ic989a88c06b2e266c63375bd19e3ed2c2b4d4c2d
Signed-off-by: Jozef Gloncak <jgloncak@cisco.com>
(cherry picked from commit 5865c58f43808e730fdb9dc310741663c304112d)

BUG-3741 - groups and meters failing

 - fixed group deletion
 - fixed meter deletion
 - optimized instance identifiers for flowCapableNode

Change-Id: I4645b2bd2e9b2aded1c55015dc2ad8f5b1428517
Signed-off-by: Martin Bobak <mbobak@cisco.com>
(cherry picked from commit 01b5fb16c02331da1d9dd6b2ca40509f4c90785f)

BUG-3157: cbench tuneup - packetIn dynamic throttling proposal.

Limit for total amount of outstanding notification can be parametrized.
By default there is 120k. Every device gets proportional part of that
limit and corresponding limiter is notified when number of devices
changes and new proportions affect limits.

 - added parameter to configSubsystem
 - water marks adaptation proposal
 - sanitize limit computation

Change-Id: I257f1b91b8f5d512f512c1694be6be3a670128c6
Signed-off-by: Martin Bobak <mbobak@cisco.com>
(cherry picked from commit 024a822cccc79cd15ca7f9d5d67f54215c82a9d5)

- Creating separate bulk-o-matic features for the He and Li plugins
- fixed dependencies

Change-Id: I2de2addc4d31c7b03b5b9206367c9fe168dd4439
Signed-off-by: Jan Medved <jmedved@cisco.com>
Signed-off-by: Michal Rehak <mirehak@cisco.com>
(cherry picked from commit cca6476b40b353886ac1a501dcd3659a6aaa462b)

Merge "bulk support application - initial proposal"

bulk support application - initial proposal

 - added models to config subsystem config file
 - created simple implementation
 - added to ofp-li features
 - workarounded restconf bugs (flow-ref related)
 - added parents creation for dataStore way (all or first only)
 - clean up rpc intput models
 - fixed version

Change-Id: I1716b32738c8e430d674ff5f0476d55ab10b43fb
Signed-off-by: Martin Bobak <mbobak@cisco.com>
(cherry picked from commit 45c121d507dbd45995fa3efa56a9cef5154dee62)

Merge "Bug 3817: Fix action convertor for OF 1.3."

Bug 3817: Fix action convertor for OF 1.3.

The action convertor for flow entries of OpenFlow 1.3 wrongly used
instance builders for UDP source port in the conversion for the UDP
destination port. In the result, the OpenFlow plugin failed to install a
flow entry which has an action modifying UDP destination port.

Also, the action convertor wrongly used Icmpv4Type.class to create an
action modifying ICMPv4 code filed. And, the size of mask for ICMPv4
code and type fields was wrongly set into 0x0f, though the size of these
fields were 8 bits.

This patch fixes these issues so that the OF plugin can correctly
install flow entries which modify UDP destination port and ICMPv4
fields.

Change-Id: I8e972233bbea5741bb8bca39cbd281b6c00b763b
Signed-off-by: Hideyuki Tai <Hideyuki.Tai@necam.com>
(cherry picked from commit 05d964944576807810ef9e8622f56eb2cff4c4ce)

Bug-3794 - OFHandshake thread leak leads to OOM.
Fixed the thread pool cleanup

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

Bug 3792 - Change default flow timeout values to 0

Current OF plugin sets the following default values if idle and
hard timeouts are not set in flow body:

Idle_timeout=300
hard_timeout=600
as per openflow spec, if user don't set hard-timeout and idle-timeout
value explicitly, switch will set it to 0 and consider the flow permanent
flow. So existing behavior is clearly violating the specs.

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

Bug 2902 - NPE when apply_action doesn't contain action list

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

Bug 3041 - Meter and Meter Config messages not sent when controller starts after the switch

Openflow plugin explictly sends group/meter features statistics request
after successful switch handshake to the controller. Statistics-manager
receives multipart reply and ignores it, because it only process response
for the request sent by itself. So it does not make sense to send these
request from plugin.
Statistics manager also sends meter/group feature multipart request when
it's get notified that new node is added to the inventory. In case where
it gets the response immdiately from switch, sometime it creates race
condition where statistics manager don't find the transaction-id of the
request in the cache for which it received the response. This race condition
occures  because statistics manager store the key in cache once request send
operation finishes successfully in asyn manner. Processing the group/meter
response for the request send by plugin causing the reordering of the event
so that statistics processing happens before it adds transaction-id to the
cache. This race condition make statistics manager believe that the request
was not send by it and so it does not update the meter statistics capability
of the node and hence it stops sending the meter stats.

Patch 2 : Fixed unit test - plugin don't send meter/group feature request
now, so two less events.
Patch 3 : Fixed integration test for stable/lithium and master

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

Merge "Bug 3660 - Missing flow for match ARP."

Bug 3660 - Missing flow for match ARP.

Probably typo. Incorrect variable was evaluated.

Change-Id: I02ece504dda3603e8d5dbc60e6dda06d5f60ebbd
Signed-off-by: Jozef Gloncak <jgloncak@cisco.com>
(cherry picked from commit 33637cfb62d32ce59894c18de5e86e73c9047e70)

BUG 3603 - Flow without priority isn't processed.

If flow priority isn't specified then default value 0x8000 is used.
Operation no more fail with NPE.

Change-Id: I9de3457781478e83401a4b18000ded58af501654
Signed-off-by: Jozef Gloncak <jgloncak@cisco.com>
(cherry picked from commit 81f9238a2012999c4fe94c87486f4d7c9bf0f54a)

Bug 3616 - NPE if flow ref isn't provided.

Check of nullity was added.
If flow ref:
- was provided in input data then it is used.
- wasn't then new flow isn't stored in flow registry.
  Once statistics will come then allien ID will be generated
  for this flow.

Change-Id: Id1ee07ca38825077acfda3af82badbac92db79a5
Signed-off-by: Jozef Gloncak <jgloncak@cisco.com>
(cherry picked from commit 2679df2fb38df97b44c4ca6f60526a35dc0d018f)

Bug 3635 - fix npe when device disconnects during handshake

Change-Id: Ia92660d630ddbcefdfd37e887ab896c2550e88cc
Signed-off-by: Martin Bobak <mbobak@cisco.com>
(cherry picked from commit 3c0d578f2001e1114e6da8df8dfaf437d14f2a7a)

Bug 3569 - Missing TunIPv4DstConverter.

TunIPv4DstConverter was added to nicira. The class was created
according to TunIPv4SrcConverter.

Some helping methods were moved to MatchUtil.

Change-Id: Ic2e5fab847037258e56922620d2702e21294c2b7
Signed-off-by: Jozef Gloncak <jgloncak@cisco.com>
(cherry picked from commit ca3fb67fc8485dd72df46fcf63796bd433a8354e)

Bug 2987 - Set-Vlan action on OF1.3 Sends 2 remove flows (similar to addFlow)

Change-Id: I408e5d7931e9fab0c02b0b6eb84f7e8d8ed6bf9d
Signed-off-by: Claudio D. Gasparini <cgaspari@cisco.com>
Signed-off-by: Martin Bobak <mbobak@cisco.com>
(cherry picked from commit 8d4984a0e75093877652ae31b1e5dfb4d7a49fb9)

Bug 3620 - don't handle connection state in DeviceTransactionChainManagerProvider

Change-Id: Ief98f9c3d3e69b28a76a432714aaf56deedba385
Signed-off-by: Martin Bobak <mbobak@cisco.com>
(cherry picked from commit aa516b90b6f750cc30ab716efe8425f3906cf8e8)

BUG-3157: performance tuning - Use a simple lookup table for prefix conversion

This is a clear CPU/memory trade-off, where we have a pre-allocated
bytearray for valid prefixes. We then lookup the appropriate offset
into this array when performing a mask operation. If the user requests
the actual array, we return a Arrays.copyOf(), cutting just the
appropriate bytes out. Memory cost is around 2K.

Change-Id: Idd679084bd3be4a31331a38ecf978b07d738f495
Signed-off-by: Robert Varga <rovarga@cisco.com>
(cherry picked from commit c59861d4957cb69badda3737b146a0c98f9fec71)

BUG-3363: code optimization and cleanup - Fix a few warnings

Unused imports, methods that can be static, deprecated imports, missing
type arguments.

Change-Id: I04f7cee19ab1832e58e5b7c91e3f4bcbe55d2a1f
Signed-off-by: Robert Varga <rovarga@cisco.com>
Signed-off-by: Michal Rehak <mirehak@cisco.com>
(cherry picked from commit aedfc8ddeac646cd11b364e4faf1fca803175ef0)

Bug 3594 - error when updating flow via restconf

If flow is updated via rpc update-flow where flow-ref
instance-identifier isn't specified then nothing is
deleted from flow hash to id map stored in operational DS.

Change-Id: Ia9151b9d8a85a920ef70d7328ef2dda769be297c
Signed-off-by: Jozef Gloncak <jgloncak@cisco.com>
(cherry picked from commit 0ca58c24157ab10d3a140ceffe5e3cbd4a7991de)

BUG-3363: code optimization and cleanup - Use Strings.isNullOrEmpty()

A convenience method making for an easier read.

Change-Id: I7d5868999e4644fdcc2e67657e44264dc950f4f8
Signed-off-by: Robert Varga <rovarga@cisco.com>
(cherry picked from commit a589f3912de430ad4889fa776325bebabf4ffa34)

BUG-3363: code optimization and cleanup - Do not instantiate explicit arrays

Since Iterators.forArray() is variadic, we can simplify the call a bit.

Change-Id: Ib66035f0d18d76c2c6c8f8af0ad7f5915038b72a
Signed-off-by: Robert Varga <rovarga@cisco.com>
(cherry picked from commit 804e6d8da21c4f7b5fbb91ae08e90ef51f723294)

BUG-3363: code optimization and cleanup - Use Arrays.copyOf(byte[], int)

This is a simpler version of Arrays.copyOfRange(), with equivalent
effect.

Change-Id: Iabf03022e2ace5d083fbe485b8b0e110658c457a
Signed-off-by: Robert Varga <rovarga@cisco.com>
(cherry picked from commit 72153d9785596d79ad1261bf1b866939e0767e84)

BUG-3363: code optimization and cleanup - Do not open-code array copy operations

System.arrayCopy() is the proper tool for the job, use it. Also add
final modifiers to arguments and organize imports.

Change-Id: I123c30c3e85b92a7b5d9a9f237cda019b7c362e0
Signed-off-by: Robert Varga <rovarga@cisco.com>
(cherry picked from commit 5f2ce7a9a8bb4758dc94ff543b251d6a8f4b3e47)

BUG-3157: performance tuning - Use a TrieMap as the ConcurrentMap implementation

TrieMap is space-optimized and has a high fanout, which means it will
reclaim space when not needed and has shallower structure than a
ConcurrentHashMap. It also re-hashes automatically and concurrently.

Change-Id: I1e40dbb5588c1e42475e12d73f7cadf46b909594
Signed-off-by: Robert Varga <rovarga@cisco.com>
(cherry picked from commit 5faeb7ea76404e4da3620e63432b6038800d4de7)

BUG-3363: code optimization and cleanup - Make flowRegistry concurrent

Traces indicate that updating the flow registry can be a source of
contention. As it turns out all operations we perform on it can be
expressed in terms of ConccurentMap, so let's take advantage of that.

Change-Id: Iae145699cc5111a589eba421bc1082f88f39ee18
Signed-off-by: Robert Varga <rovarga@cisco.com>
(cherry picked from commit f355ec96d1d7c99038650839c133db0aff8b043c)

BUG-3363: code optimization and cleanup - Fix unsynchronized access to marks

We must not iterate over marks while not holding the lock, as we risk a
concurrent modification. Mkae sure we synchronize properly.

Also minimize the synchronized sections around flowRegistry. These will
be removed in the next patchset.

Change-Id: I5575fa972aea6d8055d578da2d3418c25286668d
Signed-off-by: Robert Varga <rovarga@cisco.com>
(cherry picked from commit 06068449350ddee54597a38db39c678f93fe2ab3)

BUG-3363: code optimization and cleanup - Do not leak mutable collection

Flow descriptors should not be modified outside this class. Make sure
that is the case by wrapping it in an Collections.unmodifiableMap().

Change-Id: I188287d8d10911f6faabd5f31e028714ae005bce
Signed-off-by: Robert Varga <rovarga@cisco.com>
(cherry picked from commit 9930a5d6b7cfeed87e4e3308882b5375bea10f59)

BUG-3363: code optimization and cleanup - Fix eclipse warnings

Unused imports, missing serialVersionUID, unneeded cast. Formatting
changes courtesy autosave cleanup actions.

Change-Id: If85ef69d426b129170d5b55d0bd6893a901d50cc
Signed-off-by: Robert Varga <rovarga@cisco.com>
(cherry picked from commit d25cd407fd8f5d038349db4bb2407d023afe3150)

Bug 3599 - Li - transaction chain manager should not propagate connection context in notification

Change-Id: Ia03c152cec6c7ea3afbb29e5778b53cc1d80fd83
Signed-off-by: Martin Bobak <mbobak@cisco.com>
(cherry picked from commit 571b41953805ad4c14abbbd031fc86f4dfccf747)

Bug 3597 - added session statistics for switch connect/disconnect events

    Change-Id: I75f388721a678ba00e10b352bd87d1b3ca9833c3
Signed-off-by: Martin Bobak <mbobak@cisco.com>
Change-Id: Ia702621740bbc7190813ff286d6b76158bd59f64
Signed-off-by: Martin Bobak <mbobak@cisco.com>
(cherry picked from commit 1ab03d3178b3d81f46ef6c94ba8e742327e89921)

BUG-3592: topology not starting (Li)

 - added config subsystem files to Li features

Change-Id: Ibcdfa83ab47777a6b3d739b024a1453d17b48791
Signed-off-by: Michal Rehak <mirehak@cisco.com>
(cherry picked from commit bea0a6e84adcc91ae681b2a07c69a168d72764e5)

BUG-3580: statistics teardown improvement

 - statistics context cancels next scheduled statistics poll

Change-Id: I3a4c1fb63d32f8ecc660da38455284bb142a62c6
Signed-off-by: Martin Bobak <mbobak@cisco.com>
(cherry picked from commit c8b1875c9f8dcb4f5a4db29b33285990abb1149b)

BUG-3579: device disconnection cleanup

 - connection + handshake revisited
 - device, rpc, statistics contexts/managers revisited
 - cleaned up logging
 - adapted tests
 - added propagating to connectionCtx close for timeout

Change-Id: I990012ecb34fd3a229c1988cbd1c2b71af6ec98e
Signed-off-by: Michal Rehak <mirehak@cisco.com>
Signed-off-by: Martin Bobak <mbobak@cisco.com>
(cherry picked from commit d54deb3112a9ee9cf71571cffd9fb40806d5bed2)

Bug:3048-Handle duplicate nodeRemoved and nodeConnectorUpdated notifications to prevent tx failures

NodeRemoved notifications are received twice by the InventoryManager, in quick successions
If the duplicate node removal operation is clubbed with other node-removal operations in transaction chain, it fails the whole tx chain even
though we retry one time.

This patch caches the deleted nodes in-order to ignore the duplicate notifications. The cache entries are expired after 10s, to reduce the memory consumption.

Signed-off-by: Kamal Rameshan <kramesha@cisco.com>
Change-Id: Ifce9a8240e4a6bac00815be1f494f0077fd957d1
(cherry picked from commit fce9a8240e4a6bac00815be1f494f0077fd957d1)

Bug 3564 - Li - ofp when transaction chain manager exists in WORKING state, drop new connection

Change-Id: I75810f705774a75264558166bd7570eb9bae41e6
Signed-off-by: Martin Bobak <mbobak@cisco.com>
(cherry picked from commit d2cc6d8fc130e6086cc75f98b210297fd4f85be3)

Bug-3104: Bypassing MultimessageCollector

 - added service for on-the-fly processing (flow stats only)

Change-Id: I6308bc1b38009deb6187ac2ee57b94a9b3d29145
Signed-off-by: Martin Bobak <mbobak@cisco.com>
(cherry picked from commit cfda2cbdee0e34a67202bf7c740b841ef0a53a2b)

Bug 3557 - Li -ofp when device reconnects, it has to wait for previous transaction chain to complete

TransactionChain should be checked for its lifecycle status when device reconnects

Change-Id: I272da2eae4d81802c0dc3e70308989daa6dd8162
Signed-off-by: Martin Bobak <mbobak@cisco.com>
(cherry picked from commit 8fd4238633295a741f65e2e82a883dfb401bb5b3)

Bug 2862 - flow ID is unknown value in operational store. (e.g. id: #UF*200-3)

Fix for old Helium code base.
This issue only occurs when user don't set cookie value while installing the flow
and that's valid flow.

Cookie is used in flow comparison for the applications using match extensions
in their flow body. As of now openflowplugin don't use match extensions
in flow comparison, that can create a scenario where more then one stored flow
can match to any stats flow, if stored flows differ only by match extension.
Once match extensions are part of flow comparison, we should remove cookie
from flow comparison.

Cookie is an optional field, so user might not set it, but if switch
get flow without cookie value , it will use 0 as a default cookie value
and return cookie=0 when openflowplugin fetch the flow stats from switch.
In this scenario flow comparison will fail. This fix make sure that
if user didn't set cookie value while flow installation, skip the cookie
comparison while augmenting flow stats.

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

BUG 2862 - alien flow ID when priority is missing.

Problem was:
- flow was sent to device
- attempt to write flow registry - fails (NPE) because missing priority
- flow was written to device
- statistics bring info about flow
- flow isn't found in flow registry therefore alien ID is generated for it.

Solution:
- sending of flow to device is executed only after storing flow to flow
registry. If it is unsuccessful then nothing is sent to device. Flow is thus
stored only in config DS.

Change-Id: I371d4030c41ac0866fb3615b8c2380aef5b283e2
Signed-off-by: Jozef Gloncak <jgloncak@cisco.com>
(cherry picked from commit 19622f6f0ea04a46bffda0d7900c2b2b7a1c88c4)

Bug 2862 - alien flows after flow update.

In Helium OF plugin is mapping of flow hashes to flow IDs stored
in operational DS. If flow data which are used for hash creating
(e. g. match) are changed then it isn't possible to find flow ID
in mapping stored in operational DS. Therefore deletion of hash
was added directly to update handling method - updateFlow.

Change-Id: If2f12d52d4bb8ca484ce31f1545f7493fe14364b
Signed-off-by: Jozef Gloncak <jgloncak@cisco.com>
(cherry picked from commit 70651e7540863872f6e8d94377ccdf9b53b0bf93)

BUG-2668: Handshake Exception

 - fixing bunch of exception by checking connection state before
   processing fist hello message from device

Change-Id: I8fcf9e9eeeb163e4fee30e9dabef810aabf1bf67
Signed-off-by: Michal Rehak <mirehak@cisco.com>
(cherry picked from commit 3b1e6fb5a6649abe7df47d2b3bb181229cb5cbc1)

Bug 3350 - Provide override for features detected during handshake

Some devices doesn't provide features considered by OF protocol as
mandatory. We introduce configurable variable that overrides features
check.

Change-Id: I8f7fe9c31f00b785bd19b5f0e79b8162fa7a2a4b
Signed-off-by: Martin Bobak <mbobak@cisco.com>
(cherry picked from commit 4085d17036f47c4b9bcc5113b079309b64fbc390)

Merge "BUG-3157: performance tuning - Do not user CopyOnWriteArrayList"

Bug 3341: Fix feature pom dependency issue

Change-Id: Ia9c3fecd2c5edcd080829bf5654c278c77c5bb0a
Signed-off-by: Ed Warnicke <hagbard@gmail.com>

Bug 3341: Partial fix, wires topology-manager

This patch is a partial fix to Bug 3341.
It wires topology-manager with the config subsystem,
and makes it require-capabilities the models it depends on.

Change-Id: I26baef419534f7d05d56350847d2248844f91a1d
Signed-off-by: Ed Warnicke <hagbard@gmail.com>

Bug 3341: Partial fix, wires topology-lldp-discovery

This patch is a partial fix to Bug 3341.
It wires topology-lldp-discovery with the config subsystem,
and makes it require-capabilities the models it depends on.

Change-Id: I9a70ea5bda5b5720e86d4591f29737eaa4d7ea7a
Signed-off-by: Ed Warnicke <hagbard@gmail.com>

Bug 3341: Partial fix, wires inventory-manager

This patch is a partial fix to Bug 3341.
It wires inventory-manager with the config subsystem,
and makes it require-capabilities the models it depends on.

Change-Id: Id0f2891b47e0290411a382eb2ef2bc82c0a53c9e
Signed-off-by: Ed Warnicke <hagbard@gmail.com>

BUG-3157: performance tuning - Do not user CopyOnWriteArrayList

Filling it is expensive, use a simple ArrayList.

Change-Id: I231ee15e177d4c689f1729dc7e55f19f4e286222
Signed-off-by: Robert Varga <rovarga@cisco.com>
(cherry picked from commit 01acc1e6693c8ad65fed069393fc05b5d44a655b)

BUG-3157: performance tuning - NodeConnectorRef cache

 - introducing port number to NodeConnectorRef cache in deviceContext

Change-Id: I805f8d54967331b1f42924c7d9f0e24a24f24749
Signed-off-by: Vaclav Demcak <vdemcak@cisco.com>
(cherry picked from commit 88aeb167f9cc81dcad79884dc3763cbda85296ae)

Bug-3431:NullPointerException from OpenflowPlugin.application.StatisticsManagerImpl

Change-Id: I5a4b45dd9d32a2968596f8c466d85c91132d6ee2
Signed-off-by: Kamal Rameshan <kramesha@cisco.com>
(cherry picked from commit 5a4b45dd9d32a2968596f8c466d85c91132d6ee2)

Bug 3437 - clean event time measurement average value of extreme values

Change-Id: Icc131dd1443ee2e76f2aeaee42bbe65a3c22786d
Signed-off-by: Martin Bobak <mbobak@cisco.com>
(cherry picked from commit b50f59687da67b75ec743e1dff164ef88ddc7b6d)

BUG-3157: perfomance tuning - cache translator lookups

Once a DeviceContext is instantiated, it has a fixed version, so we can
pre-cache the two most-used translators.

Change-Id: I9d324f14e3003301bf9107e2e6c12a189c0b6c80
Signed-off-by: Robert Varga <rovarga@cisco.com>
Signed-off-by: Martin Bobak <mbobak@cisco.com>
(cherry picked from commit f29048e1e03900f2bce34e0b6dd196488dd34d39)

Bug 3435 - add time counter for selected operations on device

Change-Id: Ie96860f8a253b57b82a8fb7e9577a8774b09a31f
Signed-off-by: Martin Bobak <mbobak@cisco.com>
(cherry picked from commit 667dd701f243e6344e864615559bbeb93d80bae8)

BUG-3157: Use InstanceIdentifier.firstIdentifierOf()

Instead of instantiating multiple instance identifiers, make sure to
express our intent in terms of the InstanceIdentifier API. In this
particular case we are looking for the identifier capped at the
specified key.

Change-Id: I0284c582959fb00527116335b0def93e71fa535e
Signed-off-by: Robert Varga <rovarga@cisco.com>
(cherry picked from commit f983b20e843c13e7d27f9c4261c719c1192066bf)

BUG-3157: Fix a generic warning in  DropTestRpcSender

Not adding generic arguments is just being lazy.

Change-Id: Ib43a5c68a60ac838c8b3ab458af4aaaa7fbbbaba
Signed-off-by: Robert Varga <rovarga@cisco.com>
(cherry picked from commit 11f256f8c68db67fc9cd894d1766553b2f3d6562)

Bug 3412 - Device should not be propagated if initial stats collection fails

Change-Id: Ib9bf397331f9011e6909caeefa8cf403747d005f
Signed-off-by: Martin Bobak <mbobak@cisco.com>
(cherry picked from commit 42f8e33bbd8649cd4f78b1873e69b77ae00b10c4)

Bug 3407 - Don't reconnect Node without a full clean of OPERATIONAL DataStore.

Note: A reconnection scenario could bring some problem with TransactionChain for a huge data sets.
So we have to stop all PostHandShake processes as soon as is possible and wait for a full Node
disconnecting what includes OPERATIONAL DataStore clean too.

* add checkExisteingNode method in TransactionChainManager
* add synchronized check for tx in submit method
* fix tests

Change-Id: Ib2b772f6800528860ce881204d53bbec35e09b97
Signed-off-by: Vaclav Demcak <vdemcak@cisco.com>
(cherry picked from commit 3668ca6d68b8762c6d69f41f4d1d95be8b17520e)

Bug 3393 - don't silence exceptions in tx chain manager

Log events where tx chain operations fail.
Also do not print stacktrace in info level.

Change-Id: I50af32d2fa802506fddb4a218193f4ce9fac0278
Signed-off-by: Martin Bobak <mbobak@cisco.com>
(cherry picked from commit 396b210498ced73badea844afaad81e601d75a30)

Bug 3390 - schedule statistics polling instead of polling right away

After successfull initial statistics gathering we don't
want to poll next statistics gathering right away. We schedule
polling job instead.

Change-Id: I1ed68227f33ca53fc58ce7e3d69aa345d65fdd57
Signed-off-by: Martin Bobak <mbobak@cisco.com>
(cherry picked from commit 71c7dc8480f1de90ec1015b3451bbdf12b66673a)

Bug 3389 - Scheduling time adaptation calculated per device

Calculated time should not be shorter than basic timer
delay which is currently 3000 ms.

Change-Id: I9e534e9268e462b3b572a2bc12a8f80d7de97146
Signed-off-by: Martin Bobak <mbobak@cisco.com>
(cherry picked from commit 5fb386b9d2ca54fd446c1435f2bceaeb6c1990f0)

BUG-3157: performance tuning - Skip int->Integer boxing

We really need the integer object, so don't keep it as a primitive type.
Also make sure we run equals on a constant reference. Finally do not
instantiate a zero integer.

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

Merge "Bug 3367 - StatManagerImpl scalable statPolling per stat type on device"

Merge "Bug 3367 - StatManagerImpl scalable statPolling per stat type on device"

Bug 3367 - StatManagerImpl scalable statPolling per stat type on device

Note: Running all stats for all device statistics in one time is
realy not good idea. So we'd like to bring a possibility for
scalable statistics aproach (one stat in one device in one time).

* remove gatherDynamicData(MultipartType multipartType) method

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

Bug 3367 - StatManagerImpl scalable statPolling per stat type on device

Note: Running all stats for all device statistics in one time is
realy not good idea. So we'd like to bring a possibility for
scalable statistics aproach (one stat in one device in one time).

* change getherDynamicData() like a chain for device statistics

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

Merge "Cleanup IpConversionUtil"

Merge "Fix warnings in TableFeaturesConvertor"

BUG 2723 - Topology spoofing via LLDP - LLDPSpeaker

 - added extra authorizer to LLDP payload
 - used version property
 - added logging
 - fixed it and features test

Add TLV CustomSec field to creation of LLDP packet.

REMARK:
controller patch https://git.opendaylight.org/gerrit/#/c/21174/ has to
be merged first

Change-Id: I234305e827817aef2dcec820869bddca91fc2b33
Signed-off-by: Jozef Gloncak <jgloncak@cisco.com>
Signed-off-by: Michal Rehak <mirehak@cisco.com>
(cherry picked from commit e20d25cfda275b06826538726595682d88f92182)
(cherry picked from commit 041211583c485316c432eb33801c60fe33a05daa)
(cherry picked from commit 00033e6051c2451f78fb9521dbb2775569180d6c)

BUG 2723 - Topology spoofing via LLDP - hash check in topology-discovery

Checking of CustomSec (TLV field in LLDP packet). Value of CustomSec from LLDP
packet has to be equal to hash value which is computed in
topology-lldp-discovery artifact. Hash value is obtained as MD5 value
calculated from concatenation of strings:
 - node connector ID
 - pseudo PID of running JAVA karaf

Method getValueForLLDPPacketIntegrityEnsuring() prepare array of bytes which
will be after hashing used to check integrity of LLDP packets. Ensuring that
LLDP packet wasn't modified. (extra authenticator; CVE-2015-1611 CVE-2015-1612)

Change-Id: Ic8f50c88e7d8e3722d8d83a01ffa94a96bde313f
Signed-off-by: Jozef Gloncak <jgloncak@cisco.com>
(cherry picked from commit 67eed66d24b20d03645140d40b44d16ce53e1210)

Version bump for master after cutting of stable/lithium branch

Change-Id: Id6e3567a6e2e19174881d8090ee60ae7130f99a9
Signed-off-by: Ed Warnicke <hagbard@gmail.com>

Merge "Speed up packetin throttling"

Merge "Bug 3328: Set icmpv4-match into OF10 match."

Speed up packetin throttling

Perform an atomic operation instead of taking a lock in the fast path.

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

Merge "Clean TransactionChainManager"

Merge "Change StatisticsContext"

Clean TransactionChainManager

Note: We don't need to check time or nr. of operations for
WriteTransaction. We would like to submit all collected stats
to DataStore ASAP.

* remove timer from TransactionChainManager
* remove calc nr. of operations
* add visible submitTransaction method

TODO: fix commeted-out tests

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

Cleanup IpConversionUtil

Follow JLS modifier ordering, do not use string concat to coerce
Strings, do not open-code unsigned byte-to-int conversion, remove
trailing whitespace.

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

Fix warnings in TableFeaturesConvertor

Remove unnecessary casts. Fix test to not issue warnings, too.

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

schedule next poll only after previous is done

Statistics will be gathered only after previous run was completed.

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

Merge "Statistics polling is scheduled per one device"

Merge "Un-obfuscate srcVlanMatch"

Merge "Use an ImmutableMaps for constant lookups"

Merge "Fix TableFeaturesConvertor table"

Merge "Use constants for Ordering objects"

Merge "Make OrderComparator.build() use generic arguments"

Merge "Add VLAN match constants"

Merge "Optimize FlowConvertor"

Merge "Make MatchReactor a proper constant"

Merge "Simplify CommonService interface"

Merge "HashUtil erradicated"

Statistics polling is scheduled per one device

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

HashUtil erradicated

Hashing flow values for its future identification is not done anymore. We
use repective object's haschode and equals methods.

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

Un-obfuscate srcVlanMatch

Eclipse complains about the possibility of a null-derefence. While it is
not possible, rewrite the code to make that obvious. Also move to
MoreObjects to remove deprecation warning.

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

Use an ImmutableMaps for constant lookups

The lookup maps are meant to be immutable, make sure that is the case.

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

Fix TableFeaturesConvertor table

This mapping problem has gone unnoticed until conversion to ImmuableMap.
Fix it here so it can be cherry-picked.

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

Use constants for Ordering objects

These objects are used heavily, make sure we share instances.

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