Improve InitialFlowWriter

Pick up the path only after checking the modification path.

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

Merge "add comment of ipAddress in addAddress function"

Merge "add the comment for ipAddress addAddress function"

Migrate flow writers

Use Uint types and BindingMap instead of widened types and lists. Also
fix hybrid mode, which would not work due to 'order' being set to zero
for both actions.

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

Migrate InventoryReaderTest

- use MockitoJUnitRunner
- use BindingMap.of() instead of lists
- use ArgumentMatchers.any()
- improve test assertions

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

Merge "Do not use JdkFutureAdapters"

add comment of ipAddress in addAddress function

Change-Id: Ieb49f4a1c2d7e51ff2ca01394443652cf3ae4bb8
Signed-off-by: wenjie yan <yanwenjie911@126.com>

add the comment for ipAddress addAddress function

Change-Id: I3102b3f8f58f86693697bd49576c3c8456da6e98
Signed-off-by: wenjie yan <yanwenjie911@126.com>

Do not use JdkFutureAdapters

RPCs are already returning ListenableFuture, there is no point in
bringing JdkFutureAdapters to the party.

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

Migrate loopremover to use maps

Loopremover is only using single-element lists where it should be using
a map. Switch to using BindingMap.of(element), which does the right
thing.

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

Use Uint types in loopremover

We have unnecessary round-trip through widened types, eliminate it in
favor of using Uint8/Uint16.

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

Remove trailing comments

The // at the end of the line was an attempt to get Eclipse auto-format
to behave with fluent builders. SonarCloud does not like them for
obvious reasons. Remove this clutter.

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

Modernize TopologyLinkDataChangeHandlerTest a bit

Use List.of() instead of Collections.singletonList(). Also use
ArgumentMatchers.anyList() to fix a warning.

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

Fix warnings in NetworkGraphImplTest

Use diamond notation to eliminate a warnings about rendundant type
specification.

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

Migrate use of boolean getters

isFoo() is a legacy construct, superseded by the regular getFoo()
pattern. Migrate callers.

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

Update upstreams to Silicon SR4

Bump upstreams to:
- odlparent-8.1.9
- mdsal-7.0.14
- controller-3.0.16
- netconf-1.13.8
- openflowplugin-0.12.4

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

Remove unneeded dependencies

- org.osgi.core is not used anywhere
- slf4j-api is already declared in odlparent
- slf4j-simple is already declared in odlparent for tests
- junit/mockito are already declared iin odlparent for tests

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

Update upstreams to Silicon GA

Bump upstreams to:
- odlparent-8.1.1
- mdsal-7.0.6
- controller-3.0.7
- netconf-1.13.1
- openflowplugin-0.12.0

Also fix a javadoc warning and update tests to correctly set
connector/node ID, so that the corresponding key can be formed.

Change-Id: Ic1da101145ca844e3f63c3b787bf845ab379ce50
Signed-off-by: David Arjona <darjona@yahoo.com>
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Fix InventoryReaderTest

The Aluminium patch slipped past a raw type and removed the second
address. Fix that up.

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

Bump upstreams to Aluminium SR4

Dump upstreams to:
- odlparent-7.0.10
- mdsal-6.0.10
- controller-2.0.8
- netconf-1.12.4
- openflowplugin-0.11.4

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

Update to work with Aluminium GA

This updates pointers to Aluminum versions, adding porting changes.

We disable the odl-l2switch-ui feature, as we do not have dlux
available. It should be pretty easy to add back.

Places where we need to change logic due to Binding mapping having been
changed receive a bit more substantial changes in that:
- we use try-with-resources blocks for read transactions
- dispatch logic is updated to reduce unnecessary nesting
- a few places are also updated to use Uint types

Change-Id: Id93b64a0b14de61ca1b0a455d386ce70c49f5a5f
Signed-off-by: Matt McCormack <mmccorm1@andrew.cmu.edu>
Signed-off-by: David Arjona <darjona@yahoo.com>
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Bump versions to Sodium SR4

Update upstream version to Sodium SR4. This requires solving the
long-standing modeling problem where we have overlapping fields
between two cases.

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

Update to work on Sodium SR1

This updates pointers to Sodium SR1 versions, adding porting
changes.

We disable the odl-l2switch-ui feature, as we do not have dlux
available. It should be pretty easy to add back.

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

Bump to odlparent 3.1.0 and yangtools 2.0.3

Change-Id: Ia006d67867ccd242efba6bd811bd21e258e7df9f
Signed-off-by: Stephen Kitt <skitt@redhat.com>

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

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

Merge "Fix FindBugs violations and enable enforcement"

L2SWITCH-97: export packages that are not exported in the jung jars

Change-Id: I77d72bb326b9a49251c536e8bf77997aa96565ee
Signed-off-by: Evan Zeller <evanrzeller@gmail.com>

Fix FindBugs violations and enable enforcement

- Method call passes null for non-null parameter
- Method ignores exceptional return value
- Boxing/unboxing to parse a primitive
- Redundant nullcheck of value known to be non-null
- Constructor invokes Thread.start()
- Load of known null value

Change-Id: I3cf3c48306c1ade906fe51b5c0492f5a669a2d43
Signed-off-by: Tom Pantelis <tompantelis@gmail.com>

Fix checkstyle violations

Change-Id: I07087441e114e802d932ab71261719139efd125f
Signed-off-by: Tom Pantelis <tompantelis@gmail.com>

POM cleanup

- derived l2switch-parent from binding-parent to paick up yangtools
  definitions.

- removed yangtools.version and other properties no longer needed in
  l2switch-parent pom.

- removed unnecesrary yang-maven-plugin definitions.

Change-Id: I0a0b07f72e5a53fc09f411d32af8af6ceeade6a4
Signed-off-by: Tom Pantelis <tompantelis@gmail.com>

Bump yangtools to 2.0.1

Change-Id: Ifb6a4988ae2aeed2c4ee1bb53272b61d12dbf814
Signed-off-by: Luis Gomez <ecelgp@gmail.com>

Bump to yangtools-2.0.0 and odlparent-3.0.2

Change-Id: I558e42e33dee324570789b70d196dcb032331766
Signed-off-by: Tom Pantelis <tompantelis@gmail.com>

Bump net.sf.jung version

Use the version from upstream, not the old, repackaged one.

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

Bump l2switch to use yangtools 1.2.0

Jira: releng-485

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

Bump odlparent 2.0.4 to 2.0.5

Change-Id: Ifca4b21143336fb91f91ea2b0d800749101eea17
Signed-off-by: Stephen Kitt <skitt@redhat.com>

Convert DataChangeListeners to DataTreeChangeListeners

The DataChangeListener API has been deprecated for a couple releases
with possible removal in Oxygen.

Change-Id: Ide7e2b975dedccc4afefb40e421b92bf33f69272
Signed-off-by: Tom Pantelis <tompantelis@gmail.com>

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

Change-Id: I3496637ee771662e2f0b109d649c9917277d3c5b
Signed-off-by: Thanh Ha <thanh.ha@linuxfoundation.org>

Bump odlparent 2.0.2 to 2.0.4

Change-Id: Iec13f2a58b023fe4faa7173a6133a6db14bd40ea
Signed-off-by: Stephen Kitt <skitt@redhat.com>

Bump all Odlparent dependencies to 2.0.2

Change-Id: Ia0094f59379ffd3612cfff06aa80f7c8c35d5e28
Signed-off-by: Vratko Polak <vrpolak@cisco.com>

Fix features4 -> features

Part of Karaf 4 Migration is a recommendation to rename features4
artifacts to features.

Change-Id: I01b52d826df46b52422372b579ff6ff0c7b6230e
Signed-off-by: Thanh Ha <thanh.ha@linuxfoundation.org>

Remove JMXGenerator from poms

l2switch no longer uses the config subsystem so doesn't need to
reference the JMXGenerator. Also cleaned up other unused CSS references.

Change-Id: I117643536c747342a4f5f364c8e9db9e83f32e91
Signed-off-by: Tom Pantelis <tompantelis@gmail.com>

Fix karaf pom to use features4-l2switch

Change-Id: I9dcda99017d3e78b414f038fc0d33c50fae55b3e
Signed-off-by: Donald Hunter <donaldh@cisco.com>

Move to odlparent 2.0.0

Change-Id: I3e26e79bf535e46787360248b3bd871473656526
Signed-off-by: Colin Dixon <colin@colindixon.com>

Migrate to odlparent 1.9.0

Change-Id: I267c493e256dad7965b5ed62a735c0f8676eb9fe
Signed-off-by: Thanh Ha <thanh.ha@linuxfoundation.org>

Migrate to odlparent 1.8.0-Carbon

Per request of odlparent project we are downgrading all Nitrogen
projects to use the released odlparent 1.8.0-Carbon to allow for the
odlparent project to start performing semver style releases.

Change-Id: Ic69a19d5ee7077f0e9d3c8d666759a79398796ad
Jira: RELENG-159
RT: 41406
Signed-off-by: Thanh Ha <thanh.ha@linuxfoundation.org>

BUG-8568: remove reference to DataBrokerService

This class was removed, update javadoc.

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

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

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

Bug 7624: add Karaf 4 features

Change-Id: I78d34fd067f6a829baf3a75c1a0080a76716ced2
Signed-off-by: Stephen Kitt <skitt@redhat.com>

BUG-6859 - Binding generator v1 refactoring

Fix compilation failure after binding generator move

Change-Id: Idc1ffceba0604a6130d2dad3abfe701a9fc1fce0
Signed-off-by: Martin Ciglan <mciglan@cisco.com>

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

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

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

Remove bad manifestLocation from maven-bundle-plugin

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

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: I138ff0cbd9541a490ea4954c5e4d49a96fa0cacc
Signed-off-by: Robert Varga <rovarga@cisco.com>

Replace odl-dlux-core with odl-dluxapps-topology

Due DLUX split, we removed odl-dlux-topology feature from odl-dlux-core.
Because your project have dependency on odl-dlux-core, i assume, you used
Topology app.

More info:
https://wiki.opendaylight.org/view/Weather#Rename_some_odl-dlux-.2A_features_to_odl-dluxapps-.2A

Change-Id: Idc542d3eacad5c18cc5ff749a2926cb415546ef4
Signed-off-by: Daniel Malachovsky <dmalacho@cisco.com>

Bug 5452 - Adding comments that details the configuration.

Using this patch meaningful comments are added that explains
true / false setting for proactive mode configurations.

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

Using incremental numbers for initial flow can easily conflict with the flows installed through config data store.
To make a simple fix, this patch adds L2switch prefix with the incremental flow-id

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

Bug 6655 - arphandler unable to flood arp packet

Change-Id: I637edb6c9b5a975f8e7c041ae5beb8c1a794046d
Signed-off-by: Ye xu <ye.xu1@zte.com.cn>

Bug 6278: Switch to use odlparent's karaf-parent

Change-Id: I3bd6a301d628fc63b1f2e768cbcaf83f67869e95
Signed-off-by: Geng Xingyuan <geng.xingyuan@zte.com.cn>

Bump versions by 0.1.0 for next dev cycle

Change-Id: I1c1db748f54d545c2c57eba24612d743e9fc171c
Signed-off-by: Thanh Ha <thanh.ha@linuxfoundation.org>

Bug 6199 - Add icmp decoder to packethandler

Change-Id: I71f7c32c6cc4b2a357e9d264656a6e102c54e6d2
Signed-off-by: Yakir Dorani <yakir.dorani@hpe.com>

Upgrade ietf-{inet,yang}-types to 2013-07-15

Change-Id: Idaacfe3c173e3cd1a43a34aedaf72aa901cd8ab6
Signed-off-by: Lorand Jakab <lojakab@cisco.com>

Add .checkstyle to gitignore

Change-Id: Ia6ace14aabe8f4c61bfdac167d1386d41b5a3df5
Signed-off-by: Alexis de Talhouët <adetalhouet@inocybe.com>

Add blueprint wiring for packet handler

Change-Id: Ie19e31858533d892098152d86ff75c4d40c6a9d7
Signed-off-by: Alexis de Talhouët <adetalhouet@inocybe.com>

Add blueprint wiring for l2switch-main

Change-Id: I98069ed77d697b51592458e999549a4f22a59c7f
Signed-off-by: Alexis de Talhouët <adetalhouet@inocybe.com>

Add blueprint wiring for loop remover

Change-Id: Id6cd32afccdabbab7ad6790a9f1f541fc4b1db33
Signed-off-by: Alexis de Talhouët <adetalhouet@inocybe.com>

Add blueprint wiring for host tracker

Change-Id: Ia4543ca5d2bdd36c347499b1798cac2baab4ce44
Signed-off-by: Alexis de Talhouët <adetalhouet@inocybe.com>

Add blueprint wiring for arp handler

Change-Id: I9db51c05cb0992f449ba3a79907bf872a8568c40
Signed-off-by: Alexis de Talhouët <adetalhouet@inocybe.com>

Add blueprint wiring for address tracker

Change-Id: I97f48556b3649def457e527b36b1596d2fab56ff
Signed-off-by: Alexis de Talhouët <adetalhouet@inocybe.com>

Use nexusproxy from odlparent

Change-Id: Iac6a3cdd0075c7807dce72fb41bd411b907e94ee
Signed-off-by: Thanh Ha <thanh.ha@linuxfoundation.org>

BUG 5816 - Expired hosts never comeback after timing out

Appropriate configurations in l2switch needs to be made for expired hosts
to be seen back in the network. This patch adds comments where to modify
the configurable parameters for expire hosts to be seen when they are
back in network.

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

Merge "BUG 5012 - adding configurable support for timing out hosts ( in interval of seconds )"

Merge "Bug 5768 - Changing L2switch to depend on openflow-li features This way l2switch can be added to final integration distribution as final distribution can run only one version of open flow."

Merge "Bug 5452 - stable / beryllium - Adding comments that details the xml configuration proactive mode setting."

Merge "Bug 2501 Reactive mode -default L2Switch timeout values"

BUG 5012 - adding configurable support for timing out hosts ( in interval of seconds )

This adds two new configuration parameters (host-purge-age and
host-purge-interval) in the host tracker config yang and initial config
xml file as well as the backing logic for them.

This allows for hosts and associated links to be removed from the
operational store after a specified time since the last observations.

Author:    Sai MarapaReddy <sai.marapareddy@gmail.com>
Author:    Colin Dixon <colin@colindixon.com>

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

Bug 5452 - stable / beryllium - Adding comments that details the xml
configuration proactive mode setting.

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

Bug 5629 - Cleaning up Yang files
Unused imports are being removed from yang files.

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

Bug 2501 Reactive mode -default L2Switch timeout values

The current Default L2Switch Idle (1800secs) and hard (3600seconds) timeouts
are unrealistic for a real network. As a result of this unwanted
flows are stuck are for an unimaginable amount of time. With this
commit default values will be 300 / 600 secs respectively.

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

Derive distribution/karaf pom from karaf-parent

This picks up the recent changes to the karaf distro from a common
place.

Change-Id: Ide0aecfa99c662984087c5f5a1155d70ba3f6063
Signed-off-by: Tom Pantelis <tpanteli@brocade.com>

Bug 5768 - Changing L2switch to depend on openflow-li features
This way l2switch can be added to final integration distribution as
final distribution can run only one version of open flow.

Change-Id: I86cc0775fd21cee025b84bd212da0962c3f5ca73
Signed-off-by: Sai MarapaReddy <sai.marapareddy@gmail.com>
Author:    Amit Mandke <ammandke@cisco.com>
Author:    Sai MarapaReddy <sai.marapareddy@gmail.com>

Bug 5640 AddressTracker depends on PacketHandler

AddressTracker doesn't directly depend on ARPHandler, so it can be replaced
with direct dependency of packethandler.

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

Merge "Bug-5644: Fix checkstyle invocation for l2switch builds - fixed parent pom file to fix invocation of checkstyle during build process - fixed checkstyle warnings reported during build (incorrect indentation, import order, logger init, etc.)"

Merge "Do not deploy the karaf artifact"

Do not deploy the karaf artifact

The karaf artifact is only used for testing. It does not need to be
released to the world.

Change-Id: I45b441d9bb5fe21bf8a0b0f3dd7aa47e72117e86
Signed-off-by: Thanh Ha <thanh.ha@linuxfoundation.org>

Bug-5644: Fix checkstyle invocation for l2switch builds
- fixed parent pom file to fix invocation of checkstyle during build process
- fixed checkstyle warnings reported during build (incorrect indentation, import order, logger init, etc.)

Change-Id: I5c21b3686f98b03b7d1dda33285f819b61757d3f
Signed-off-by: Ajay L <ajayl.bro@gmail.com>

Bug-5559: L2Switch distribution job does not work correctly
- replaced hard-coded versions with properties in parent pom
- removed/fixed redundant versions in pom
- when karaf was run from distribution job, it was throwing exceptions. Fixed root-cause which was that karaf.version was set to old version. Now letting this version flow from the parent
- verified using mininet that l2switch features are working as expected when launched from distribution job

Change-Id: Ib2e24fdf1b05e5c42285c3ffe749fca6fbee3617
Signed-off-by: Ajay <ajayl.bro@gmail.com>

Bug 5620 - NPE while processing onArpPacketReceived yang notification

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

Merge "Bug: 2897 Handle packet decode and publish notification on separate thread Replace tabs with spaces in modified code"

Bug: 2897
Handle packet decode and publish notification on separate thread
Replace tabs with spaces in modified code

Change-Id: Ib8539e1e3460946084aca0941eccc5a17dc32f9c
Signed-off-by: Ajay L <ajayl.bro@gmail.com>

Bug-4019: Change input for ApplyActionsBuilder->setAction to ImmutableList

Change-Id: Ie9fa3bc2b8496508e847eddfce4c970bbdd3ed50
Signed-off-by: Ajay L <ajayl.bro@gmail.com>

Bump yangtools to 1.0.0-SNAPSHOT

Change-Id: Ic1337185f4798bd23ab57fc68b332675fdfb92af
Signed-off-by: Thanh Ha <thanh.ha@linuxfoundation.org>

Bump versions by 0.1.0 for next dev cycle

Change-Id: Ic4642c66f5f5b108fca6622b9dccbcb0b7278cbb
Signed-off-by: Thanh Ha <thanh.ha@linuxfoundation.org>

Merge "Use odlparent-lite as artifacts parent"

Merge "Use odlparent configuration for jacoco"

Merge "Explicitly set git-review branch to master"

Use odlparent-lite as artifacts parent

We created odlparent-lite to provide a minimal pom which allows projects
to publish to the correct server. Without this ${project}-artifacts may
fail to deploy to Nexus.

Change-Id: Ia3fbbfad951d9d768b5df98dccd3c8b11d4aea39
Signed-off-by: Thanh Ha <thanh.ha@linuxfoundation.org>

Use odlparent configuration for jacoco

Change-Id: I747ab854b844e71e4a6124b5a7ee7d5d2acb50a9
Signed-off-by: Thanh Ha <thanh.ha@linuxfoundation.org>

Bug4588 aaa-authn-api causing stack overflow for sfc tests

Updates to use a common logging api/framework;  helps avoid
incompatibilities with features that implement standard slf4j
logging.

Change-Id: Id275f955ca28d97e361bc0e2a67dafff907ff018
Signed-off-by: Ryan Goulding <ryandgoulding@gmail.com>

Fix the parser to use radix 16

Change-Id: I2f5d34225da76dc6af340a3a05d04fc14071ac1f
Signed-off-by: Anton Ivanov <aivanov@brocade.com>

Fix for hex parsing in broad/multicast filter

Obvious - it is a hex string, it needs a 0x in front to parse

Self-inflicted as usual - if we were keeping Macs, IPs, etc
as specified in the RFC (it says canonical form) this would
not have happened.

Change-Id: I496a4b00f4aaf2a585e7c4786261ff94475a78fd
Signed-off-by: Anton Ivanov <aivanov@brocade.com>