Merge "Fix deprecation warnings in learning-switch"

Merge "Do not parse string multiple times"

Merge "Remove unneeded boxing"

Merge "Mark static methods"

Merge "GetFlowNodeCache cli"

Merge "Inventory status check"

Merge "Update active committers info"

Inventory status check

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

GetFlowNodeCache cli

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

Increase flow and group dtcn listener changes

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

Update active committers info

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

Fix deprecation warnings in learning-switch

Convert use of both legacy lists as well as widened types,
eliminating deprecation warnings.

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

Do not parse string multiple times

This code block is a simple switch, really, if not even something
simpler.

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

Remove unneeded boxing

Use of Integer creates a box-and-unbox operation when coupled
with parseInt() and switch(), just use a plain int.

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

Mark static methods

A number of methods can be made static, make sure they are marked
as such.

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

Migrate test-provider to Uint types

There is a multitude of use of legacy compatibility methods, migrate
them in one go.

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

Fixup warnings in openflowplugin-extension-onf

There are a few warnings about raw types and static methods. Fix
them up, improving BundleAddMessageConverter implementation.

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

Migrate test-common user of deprecated methods

We are using some deprecated Uint- and List-related methods, migrate
to their replacements.

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

Migrate openflowplugin-extension-onf to use Uint types

There are a few references to old methods in the tests, migrate
them. We do not migrate production code yet, so we have
cross-testing.

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

Migrate test-extension to use Uint types

There are a few users of legacy methods, migrate them.

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

Mark ConntrackConvertor methods static

There are two utility methods which can be static, mark them as
such to improve clarity.

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

Migrate openflowplugin-extension-nicira to use Uint types

There are a few references to old methods in the tests, migrate
them. We do not migrate production code yet, so we have
cross-testing.

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

Migrate openflowplugin-extension-eric to use Uint types

There are a few references to old methods in the tests, migrate
them. Since we are in the area, also clean up Mockito/Assert
references.

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

Remove unused Joiner/Splitter

MatchUtil is no longer using these fields, remove them.

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

Migrate openflowjava-extension-nicira tests to Uint types

This migrates tests to use Uint types, fixing warnings there while
keeping production code unmodified.

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

Fix raw types in openflowjava-extension-nicira test

We have a few warnings here, fix them up.

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

Merge "Clean up EricExtensionsRegistratorTest"

Merge "Use Uint types in openflowjava-extension-eric"

Merge "Fix raw type warnings in openflowjava-extension-eric test"

Merge "Remove LLDPDiscoveryUtils.macToString()"

Merge "Migrate topology-lldp-discovery to use Uint types"

Merge "Migrate table-miss-enforcer to use Uint types"

Merge "Cleanup ReconciliationServiceImpl"

Merge "Migrate notification-supplier to use Uint types"

Merge "Fix various warnings"

Merge "Use Assert.assertThrows()"

Merge "Migrate Mockito methods"

Merge "Migrate fr-sync to use Uint types"

Merge "Fix use of raw types in fr-sync"

Merge "Migrate forwardingrules-manager to uint types"

Merge "Added RFC 8040 URL, JSON payloads in flow examples"

Clean up EricExtensionsRegistratorTest

We want to use static imports of Mockito methods, plus we want to
use them from non-deprecated locations (ArgumentMatchers).

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

Use Uint types in openflowjava-extension-eric

A few tests are referencing old type-widened data types, fix that
up.

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

Fix raw type warnings in openflowjava-extension-eric test

We should be using proper type parameters to retain type safety.

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

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>

Added RFC 8040 URL, JSON payloads in flow examples

used sphinx code-tabs

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

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"