Remove LLDPDiscoveryUtils.macToString()

This method is inefficient and not used anywhere. Remove it to keep
our code clean.

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

Migrate topology-lldp-discovery to use Uint types

There are a few callers of legacy methods here, migrate them.

Change-Id: Ie8f552be5fb8c6a0d8bcef04dc7db7327a9edae7
JIRA: OPNFLWPLUG-1099
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Migrate table-miss-enforcer to use Uint types

There are a few callers of legacy methods here, migrate them.

Eclipse is also flagging Map.get(0) as unlikely -- which is accurate
and means the test is not verify things completely.

JIRA: OPNFLWPLUG-1099
Change-Id: I47ddec6b39a6e0f789d4da1c88bf9e00832d7150
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Cleanup ReconciliationServiceImpl

The first hint here is the raw ConcurrentHashMap, which turns out
to be immediately overwritten in constructor. After this is fixed
it becomes obvious the field is actually final, not volatile.

Further investigation shows we can improve use of RpcErrorBuilder,
making the code a lot simpler.

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

Migrate notification-supplier to use Uint types

There are a few callers of legacy methods here, migrate them.

JIRA: OPNFLWPLUG-1099
Change-Id: Ibee194f83afa4f54fc76c5c4cbefea957d749017
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Fix various warnings

We can make methods static and do not have to specify generic
arguments. Fix these Eclipse-reported notices.

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

Use Assert.assertThrows()

ExpectedException.none() has been deprecated, migrate to use
the appropriate replacement, which is assertThrows().

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

Migrate Mockito methods

Mockito.verifyZeroInterations() has been deprecated as was
ArgumentMatchers.isNull(Class). Migrate users to their replacements.

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

Migrate fr-sync to use Uint types

There are a few instances where we do not use Uint types but rely
on them being converted. Fix those up.

JIRA: OPNFLWPLUG-1099
Change-Id: I6def1f9da310021ace4670be313d932663affb57
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Fix use of raw types in fr-sync

Use star-notation with constructor to fix a warning, also use
wildcard for registration.

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

Migrate forwardingrules-manager to uint types

FRM has a few warnings around uint types, fix them up.

JIRA: OPNFLWPLUG-1099
Change-Id: I5ba7bfd67bd51e9bfc19b9bf59d232c74f2fcea5
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Migrate bulk-o-matic to use Uint types

All callers of these method are using a primitive short, hence
change the method signature to take that and explicitly convert
it to Uint8. This fixes one deprecation warning and further eclipse
notices around boxing.

JIRA: OPNFLWPLUG-1099
Change-Id: Iaffc9984322f460129975436d88e8c048ca63fa2
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Clarify DsCallback.tableId nullness

All callers are using non-null tableId in primitive form, do not
force its boxing.

JIRA: OPNFLWPLUG-1099
Change-Id: I3c8dbf1689ede25568002e5394e89f1126005535
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Migrate arbitratorreconciliation to uint types

There is just a single violation, fix it up.

JIRA: OPNFLWPLUG-1099
Change-Id: I8bca4d9faed14dd39e69f36915b680155c185504
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Mark static methods

A few methods do not impact object state, make them static to
clarify that (and remove related Eclipse warnings).

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

Fix addAugmentation() callers

There are two invocations that got through the last patch, fix them
up.

JIRA: OPNFLWPLUG-1098
Change-Id: Ia1a586e1664d9084851d90e49a09ce62e662ba41
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Clean up version setters

On-wire version is an unsigned byte, use an Uint8 constant to
represent them, speeding things up and eliminating deprecation
warnings.

JIRA: OPNFLWPLUG-1099
Change-Id: Iea384add91ba13dbcb20ad86b60cdb0e23dabeb3
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Merge "Migrate uint/ByteBuf interactions"

Merge "Do not pull javax.inject into features"

Merge "FRM code cleanup."

Clear Flow registry on all flow remove

The RPC to clean all flows for an OF switch does not clear the flow descriptors for the deleted flows,
so when RPC Add flow comes later, it finds old descriptor and generates alien ID. Solution is to delete all flow descriptors
when RPC to clean all flows is received.

JIRA : OPNFLWPLUG-1101

Change-Id: I7bc6c6c54ef80c9a8be2a054cb0259dbf268b15d
Signed-off-by: Harshini M <hm@luminanetworks.com>

FRM code cleanup.

Change-Id: I35697d2544464d6cdc4ee88d13978eaabcf33353
Signed-off-by: Shweta Chaturvedi <shweta.chaturvedi@ericsson.com>

Bump versions by x.(y+1).z for next dev cycle

Signed-off-by: Thanh Ha <zxiiro@gmail.com>
Change-Id: I11d5024bc231e92a43fda9ce84b76a9e49768759

Do not pull javax.inject into features

This dependency is not needed at runtime and can interfere with Jersey,
do not pull it in.

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

Use released version of infrautils

infrautils-1.8.0 has been released, use released version instead
of snapshots.

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

Migrate uint/ByteBuf interactions

Migrate use of Uint helpers provided by yangtools, which offer
better performance and ergonomics. This is the first step, migrating
production code, so that it is validated using existing tests.

JIRA: OPNFLWPLUG-1099
Change-Id: I5f396bff3b7d2eaff25c4b3bf6d5eab28add4b44
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Merge "Bump odlparent/yangtools/mdsal/controller"

Fix use of deprecated uint compat methods

Inline conversion to Uint8 instead of a simple short, thus fixing
a few warnings.

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

Cleanup MatchUtil.createEmptyV10Match()

Use Uint constants for zero values instead of legacy widened types
and cache full wildcards object.

JIRA: OPNFLWPLUG-1099
Change-Id: I2b3ce1d135e6b3304ae311206eb2076589c45f67
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Clean up use of deprecated methods

This is mostly simplification of addAugmentation() calls, but also
contains a few list type migrations.

JIRA: OPNFLWPLUG-1098
Change-Id: Ib32b2b34b5642d9a9ccd06f89e7b757fbf115f03
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Merge "Add SwitchCertificate attributes in TLS failure notification"

Merge "Add INFO.yaml for openflowplugin"

Merge "Added JSON and XML payloads tabs with RFC 8040 URL"

Bump odlparent/yangtools/mdsal/controller

Adopt upstream versions:
- odlparent-7.0.5
- yangtools-5.0.5
- mdsal-6.0.4
- controller-2.0.3

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

Added JSON and XML payloads tabs with RFC 8040 URL

Signed-off-by: Dhiraj Sharma <dhiraj.8.sharma@gmail.com>
Change-Id: I4efd1b236dae9a93c02cad1c9e3a99600d45d3b4

Add SwitchCertificate attributes in TLS failure notification

JIRA : OPNFLWPLUG-1094

Change-Id: I00aa25b3dfc33d10333a43b8bc67a2515f8fc874
Signed-off-by: Harshini M <hm@luminanetworks.com>

Merge "Fix ExperimenterActionSerializerKey initializer use"

Merge "Make methods static"

OPNFLWPLUG-1093 : Exceptions Multiple entries with same key seen in openflowplugin

description: Post aluminium mri changes node statistics rpc response was not successful,
             as flowstatisticsmaplist list contain duplicate flow-Id keys,removed flowstatisticskey from Flowstatistics
Signed-off-by: dheenadayalan b <dhinua@gmail.com>
Change-Id: I014e18630d1531df2f13e4d0c83defa183cadf0d
Signed-off-by: dheenadayalan b <dhinua@gmail.com>

Fix ExperimenterActionSerializerKey initializer use

Fix raw type use and convert tests to use Uint32 constants,
fixing migration warnings.

JIRA: OPNFLWPLUG-1099
Change-Id: I4c77735129403db278df2888e208ee2f11f20fab
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Make methods static

Eclipse points out these methods can be made static, clean up those
warnings. This very slightly improves performance as well as makes
it clear these methods are not touching object state.

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

Migrate IetfYangUtil.bytesFor() users

There are more types handled by IetfYangUtil, hence its naming is
evolving. Convert use of bytesFor() to to use macAddresBytes().

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

Remove use of Byte(byte) constructor

This constructor is deprecated for removal, do not use it.

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

Add INFO.yaml for openflowplugin

Signed-off-by: Anil Belur <abelur@linuxfoundation.org>
Change-Id: I44797e0800afb83644247167083c2da6d23b329f

Openflow Event logging using log4j - addition/modification of few logs

Signed-off-by: dheenadayalan b <dhinua@gmail.com>
Change-Id: I9a4c0a74611eb2efcb359cac203959bdc4b8429e
Signed-off-by: dheenadayalan b <dhinua@gmail.com>

Correct ampersand escapes

Ampersands need to be escaped in XML. Since we are using XML highlight,
the examples failed to lex, leading to a rtd job failure.

Change-Id: I233e018b706c65354fd7a68f78f66505e2f34e30
Signed-off-by: Dhiraj Sharma <dhiraj.8.sharma@gmail.com>
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Fix MRI feature references

OFP features are referencing a number of MRI features without using
ranged imports. Fix this up, improving our ability to upgrade
seamlessly.

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

Integrate controller-2.0.2

This bumps versions to the following:
- odlparent-7.0.3
- yangtools-5.0.3
- mdsal-6.0.2
- controller-2.0.2

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

Merge "All openflow nodes are not getting listed in the output of getallnodes  in controller nodes"

Integrate controller-2.0.1

This bumps versions to the following:
- odlparent-7.0.2
- yangtools-5.0.2
- mdsal-6.0.1
- controller-2.0.1

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

All openflow nodes are not getting listed in the output of getallnodes  in controller nodes

Signed-off-by: dheenadayalan b <dhinua@gmail.com>
Change-Id: If64f441b211489828867ad9f2091260992ca7a5e
Signed-off-by: dheenadayalan b <dhinua@gmail.com>

OPNFLWPLUG-1090: ConcurrentModificationException: null when connecting cbench switches

Issue:

CSIT job using cbench was frequently failing because some switches are
getting their initial connection rejected.

Observation and Fix:

On analyzing the logs it was found that the failure had been caused by
the sessionstats collection during initial handshake. "ConcurrentModification"
exception was thrown as multiple switch sessions were trying to modify
the SESSION_EVENTS map which reulted in the connection getting closed.

This has been fixed by making the SESSION_EVENTS threadsafe now.

Change-Id: I2d0ec7c00d7c893d6271942f5fdbe9815069d4d0
Signed-off-by: Gobinath <gobinath@ericsson.com>
(cherry picked from commit bd28cb92e0f6f3bca79a04a5e5a57c617b6cb552)

Set version to Aluminium

Signed-off-by: Thanh Ha <zxiiro@gmail.com>
Change-Id: I178492ba8affcb8eb74ee1e1e8ac5758a64b3727

Update MRI projects for Aluminium

This bumps versions to the following:
- odlparent-7.0.1
- yangtools-5.0.1
- mdsal-6.0.0
- controller-2.0.0

Change-Id: I4f23c953622f39186f82ac1da487f00f3a9707ab
Signed-off-by: Somashekhar Javalagi <somashekhar.manohara.javalagi@ericsson.com>

Merge "OPNFLWPLUG-1080: Nodes not populated in operational/opendaylight-inventory:nodes after Controller Isolation OPNFLWPLUG-1082: When switch changes owner, all cluster members delete switch inventory operational"

Fix checkstyle violations

Upgraded checkstyle is finding these, fix them up.

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

Remove invalid annotations

@Nullable on primitive 'int' or 'byte' is invalid, remove it.

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

OPNFLWPLUG-1080: Nodes not populated in operational/opendaylight-inventory:nodes after Controller Isolation
OPNFLWPLUG-1082: When switch changes owner, all cluster members delete switch inventory operational

Signed-off-by: VenkataSatya Jonnadula <rsankar@luminanetworks.com>
Change-Id: I6ceb3e71dcf86bd83fc9a3f03621502ac5144e95

Merge "OPNFLWPLUG-1087: ODL controller to provide view of openflow node reconciliation status"

MDSAL API migration changes

Desc: List of MdSAL API migration changes.

Signed-off-by: Somashekhar Javalagi <somashekhar.manohara.javalagi@ericsson.com>
Change-Id: I857c25b6bfcfe80160f735bac91746d113c844f3
Signed-off-by: dheenadayalan b <dhinua@gmail.com>

OPNFLWPLUG-1087: ODL controller to provide view of openflow node reconciliation status

Signed-off-by: dheenadayalan b <dhinua@gmail.com>
Change-Id: Ibf3e02fe4352a2bd559b838f8788ca18bc2ab83d
Signed-off-by: dheenadayalan b <dhinua@gmail.com>

Merge "Reduce use of deprecated constructors"

Use Magnesium controller

As part of controller transition to being MRI, downgrade the version
used to Magnesium release. This will be bumped to Aluminium proper
during the MRI window.

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

Reduce use of deprecated constructors

Integer-based constructors are deprecated, this adds a dent to
remove their usage.

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

OPNFLWPLUG-1083: Stats frozen after applying 2 sec delay in OF channel

Signed-off-by: VenkataSatya Jonnadula <rsankar@luminanetworks.com>
Change-Id: Ie7fa92f2307ac5751f0c14133d582262f1c46f89

OPNFLWPLUG-1085: Switch-idle events weren't consumed by inbound handler during read time-out,
                 hence moved idle handler above inbound handler from end of channel pipeline

desc: Reverting the changes provided for OPNFLWPLUG-1039, by moving idle handler above the inbound handler
within channel pipeline, as the switchidle event message are fired after channel messages are consumed.

Signed-off-by: dheenadayalan b <dhinua@gmail.com>
Change-Id: I1c1a2306eb298ba72aad2480a8f34fa6affa9a28
Signed-off-by: dheenadayalan b <dhinua@gmail.com>

Merge "OPNFLWPLUG-1086: Reconciliation framework failure when starting cbench tool for the first time"

OPNFLWPLUG-1086: Reconciliation framework failure when starting cbench tool for the first time

Issue:

CSIT job using cbench had consistent failure with the error message
starting starting with a "closed connection ... exiting" message.
The karaf logs indicated the switches getting disconnected because of
"reconciliation framework" failure.

Observation and Fix:

On analyzing the logs it was found that the "flownodereconciliation"
service of the reconciliation framework was throwing an uncaught
exception (ConcurrentModificationException) which had resulted in
the reconciliation framework failure.

This has been fixed and logs have been enhanced to debug similar issues.

Change-Id: I3cbcea2da32c342c83846f2041d82e921d7acf08
Signed-off-by: Gobinath <gobinath@ericsson.com>
(cherry picked from commit 1fdc1a25f9703550c678b4cd83188de26ad9b19d)

Rework MatchEntrySerializer

MatchEntrySerializer has a filter, which is being applied before
main serialization, and that is the only way it is being used --
we check if the match is present and if it is we run serialize().

This leads to quite a bit of code duplication as well as inefficient
execution, MatchSerializer performs two invocations and each of them
performs at least one traversal of the match.

Change matchTypeCheck() to serializeIfPresent(), which means
MatchSerializer does not invoke two method. This also allows us to
optimize the serialization process in AbstractMatchEntrySerializer
and its subclasses by traversing the match only once. This rework
makes it obvious that MatchEntrySerializer should not be an
OFSerializer, at all, as the inherited methods just confuse the
workflow.

Resulting code structure removes quite a bit of duplicated code
from the individual EntrySerializer classes and lends itself to
sharing functionality through common subclasses -- reducing the
number of possible method implementations at various places, thus
allowing better devirtualization and inlining.

The second part deals with the logic of serializing the header,
as all specific serializers have a constant header, which we can
easily precompute and emit with a single ByteBuf.writeInt() call.
Since external serializers have had the option to dynamically emit
a different header, this capability is retained via HeaderWriter
helper which is attached to each serializer. Since the number of
serializers is limited, this additional object poses little memory
overhead and is partially offset by the reduction of bytecode coming
from the elimination of oxmField/oxmClass/valueLength getters.

Because all our serializers have a constant length, we only provide
one concrete implementation of HeaderWriter, which is
ConstantHeaderWriter. Unless an additional implementation is provided
externally, this means that all header serialization calls are
readily devirtualized via CHA, thus the cost of this indirection
is zero.

JIRA: OPNFLWPLUG-1068
Change-Id: I49b7a3c5770e939d082db8479f6d70a689d4c944
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

OPNFLWPLUG-1083: Stats frozen after applying 2 sec delay in OF channel

Signed-off-by: VenkataSatya Jonnadula <rsankar@luminanetworks.com>
Change-Id: I3a34ed327237ec2f765291cc45305683f561ae88

Bump versions by x.y.(z+1)

Signed-off-by: jenkins-releng <jenkins-releng@opendaylight.org>
Change-Id: Idcbd5dc14cae5925e588837bafb1abf1839b88ed

OPNFLWPLUG-1074: table stats not available after a switch flap

Problem: csit test cases failing due to empty table stats

Root Casue: First time when table stats are collected, they
are written into operational inventory. But consequently at
the time of writing flow stats to datastore, table stats
transaction was still not complete. So when fow stats are
written with empty table stats which are read from
operational inventory

Fix: Collecting flow stats first followed by table stats,
which prevents overwrite of table stats.

Signed-off-by: Somashekhar Javalagi <somashekhar.manohara.javalagi@ericsson.com>
Change-Id: I39bf7ffe9b12a4d3a03efbec95f75dd1a0ef70f6

Merge "OPNFLWPLUG-1084 Device operational is not getting created if device reconciliation is not enabled"

Revert "All-dpns are not getting listed in the"

This reverts commit 566c4ac8fcd1feaa501955791790a247f3e69b07.

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

Revert "Fix for DPNs not connecting to controller"

This reverts commit 6b6281680872611d795261dbf232c0131235bd51.

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

Revert "CSC  to provide a view of DPN re-sync"

This reverts commit 674bb50091db4628a50c08606f8f8b9f3c8a5e55.

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

CSC  to provide a view of DPN re-sync
status from CLI

Change-Id: I6153f394803e43abf77e871f993d9ca7a1bedd45
Signed-off-by: Shweta Chaturvedi <shweta.chaturvedi@ericsson.com>

Fix for DPNs not connecting to controller

Change-Id: If708df53a0102bb85fe0040b260f821b4d35773c
Signed-off-by: Somashekhar Javalagi <somashekhar.manohara.javalagi@ericsson.com>

All-dpns are not getting listed in the
output of display all-dpns in cics

Signed-off-by: Shweta Chaturvedi <shweta.chaturvedi@ericsson.com>
Change-Id: Id4198d385fd636c5cedefbbf16a99b852ba36265

Optimize ReconcileUtil.safe*()

Node exposes nonnullTables() etc., which we can take advantage of
to simplify code.

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

Merge "Code refractor and logging improvements"

Merge "Code Refractor and logging improvements"

Merge "NETVIRT-1638 Modify yang modules such that it can be validated using pyang"

Code refractor and logging improvements

Signed-off-by: Shweta Chaturvedi <shweta.chaturvedi@ericsson.com>
Change-Id: I0b03ed4aae82f11c3d6dbe266120d00653b4ee1d

Code Refractor and logging improvements

Signed-off-by: Shweta Chaturvedi <shweta.chaturvedi@ericsson.com>
Change-Id: I85091cb808a35981c128d3f2194c15b17d92d85b

OPNFLWPLUG-1084 Device operational is not getting created if
device reconciliation is not enabled

Signed-off-by: Shweta Chaturvedi <shweta.chaturvedi@ericsson.com>
Change-Id: I7d72c004d71a9b7f47c2eedc8a399bfa8547dde9

NETVIRT-1638 Modify yang modules such that it can be validated using pyang

Fixing pyang warning and errors.

Signed-off-by: Nishchya Gupta <nishchyag@altencalsoftlabs.com>
Change-Id: Ic66aa4a4394b5b6132b4027e044da5b165e2912e

Moving threadPool to a common place.

It was creating one threadPool for each connection
and failed to close it, now using common thread pool
for all the incoming dpn connections on 6653.

Change-Id: I60372c32c9e32f65f83eb678d741a12202702d52
Signed-off-by: Shweta Chaturvedi <shweta.chaturvedi@ericsson.com>

Bump to odlparent-6.0.4/yangtools-4.0.6/mdsal-5.0.9

Pick up latest fixes and upgrades.

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

Bump odlparent/yangtools/mdsal to 6.0.3/4.0.3/5.0.6

Pick up the latest upgrades and fixes.

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

Merge "Failed to cancel service reconciliation, When controller become slave."

Error log correction

Change-Id: Ie00ff4c3008aaf3fba2af4245f09c021195829c3
Signed-off-by: Shweta Chaturvedi <shweta.chaturvedi@ericsson.com>

Avoid holding netty threads during resync callbacks

Change-Id: Ic6b78c053a9971b3be83ee1aa98f3ad822394c1b
Signed-off-by: Shweta Chaturvedi <shweta.chaturvedi@ericsson.com>

Merge "Grep not working for Karaf commands"

Read action throwing NPE

Issue Description:
When process alien action throwing NullPointerException:
java.lang.NullPointerException: null
        at org.opendaylight.openflowplugin.openflow.md.core.extension.ActionExtensionHelper.processAlienAction(ActionExtensionHelper.java:46)
        at org.opendaylight.openflowplugin.impl.protocol.deserialization.util.ActionUtil.readAction(ActionUtil.java:66)

Signed-off-by: Ning Zhang <zhangninglc@inspur.com>
Change-Id: I4008eef616bb08de9333442a4f21c3f21b12c65e

Merge "OPNFLWPLUG-1078: Notify device TLS authentication failure messages"

Merge "OPNFLWPLUG-1075: Making Device Oper transactions atomic"

OPNFLWPLUG-1075: Making Device Oper transactions atomic

Signed-off-by: VenkataSatya Jonnadula <rsankar@luminanetworks.com>
Change-Id: I7b2ed09633367e01229c7ea83063525c688821e8

OPNFLWPLUG-1078: Notify device TLS authentication failure messages

Signed-off-by: VenkataSatya Jonnadula <rsankar@luminanetworks.com>
Change-Id: I2d6acae15e94676b23764c62f4094123014e0093

Grep not working for Karaf commands

SupressWarnings is used to suppress checkstyle warnings thrown upon
using System.out

This covers the changes for "southbound-cli" in openflowplugin

Change-Id: I3491068d5cb14cfac8afe42f648445fcd7f0cdc1
Signed-off-by: gobinath <gobinath@ericsson.com>