Fix Checkstyle violations (formatting only, no real changes)

All of changes made here are purely "formatting", and none change code
behaviour in any way.  This should thus be safe to merge ASAP.

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

Merge "Add two node IT" into stable/boron

Add two node IT

Change-Id: Idd5aa1cf2be711420d96938769a293a1822b7407
Signed-off-by: Sam Hague <shague@redhat.com>
fix checkstyle for NetvirtIT

Change-Id: I51517f242d063ab2b68be692a96cf7c7b24bfec8
Signed-off-by: Sam Hague <shague@redhat.com>

Fix broken legacy NetvirtIT

Change-Id: I60cd9814190a162302b6bad15348010f63c9324b
Signed-off-by: Sam Hague <shague@redhat.com>
(cherry picked from commit 599b209edc6e04315cfb6275799a217390b1019c)

Merge "fix garp packet out creation" into stable/boron

Merge "Fix bugs related to wrong exception handling" into stable/boron

fix garp packet out creation

Change-Id: I4dd681f5f8a42070c49654dd6a27587046892ef0
Signed-off-by: Shlomi <shlomi.alfasi@hpe.com>

Fix bugs related to wrong exception handling

see discussion in thread "Re: [netvirt-dev] Exception Handling in
elanmanager-impl" on netvirt-dev@lists.opendaylight.org.  Checkstyle
raised these, and is right.

This resolves all Checkstyle related violations related to IllegalCatch.

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

Merge "elanmanager CLI console commands impl should never catch Exception (II)" into stable/boron

Merge "Fixes Bug 6514" into stable/boron

Merge "Bug 6532: Fix ACL IPv6 VM to VM communication on same network" into stable/boron

elanmanager CLI console commands impl should never catch Exception (II)

Thank You to Checkstyle for pointing these out.  Please see
https://wiki.opendaylight.org/view/BestPractices/Coding_Guidelines#IllegalCatch
for background & details.

This finishes up & completes had started in a previous Gerrit.

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

Merge "Removing Blind imports across the module" into stable/boron

Fixes Bug 6514

The AclInterface cache was getting cleaned on a interface state change.
Now it will be cleaned up only on port delete.

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

Merge "Fix NPE in FloatingIpGarpHandler.sendGarpOnInterface" into stable/boron

Bug 6532: Fix ACL IPv6 VM to VM communication on same network

This patch programs the necessary flows in the ACL (Stateful version)
to allow IPv6 VM to VM communication on the same network.

Change-Id: I1fa4bf5210c1ebfe73658800796f2c53ec88441f
Signed-off-by: Sridhar Gaddam <sgaddam@redhat.com>

Merge "Gratitious ARP handling added in ARP cache feature of L3VPN" into stable/boron

Merge "VpnPortIpToPort Listener for ELAN" into stable/boron

Fix NPE in FloatingIpGarpHandler.sendGarpOnInterface

Change-Id: I03ecf452559649391ffee5b3597ef9cf95479224
Signed-off-by: Ravit Peretz <ravit.peretz@hpe.com>

Move odl-netvirt-openstack-it to IT bundle

Change-Id: I6041e27ee2ef8ec213b2301981c9a15364175448
Signed-off-by: Sam Hague <shague@redhat.com>
Signed-off-by: Stephen Kitt <skitt@redhat.com>

Removing Blind imports across the module

This commit removes blind imports across the netvirt module.

Change-Id: I3b6d70cd6e83384eaf54f2a2f97495ae47df9e49
Signed-off-by: Suraj Ranjan <suraj.ranjan@ericsson.com>

Gratitious ARP handling added in ARP cache feature of L3VPN

As a part of the ARP cache feature of L3VPN, a timer runs once the ARP of an enterprise host is learned and is cleared as soon as this timer expires. When this host sends ARP packets(GARP for instance),the ARP cache timeout has to be reset again. Handling for it has been added.

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

VpnPortIpToPort Listener for ELAN

* Listens to VpnPortIpToPortListener changes
* Add SMAC and DMAC flows
* Update ELAN operational DS

Change-Id: I70280b5b4e1560e9725015d54f2345401d66f432
Signed-off-by: eceghkl <manu.b@ericsson.com>

Bug 6515: Fix NPE for an IPv6 subnet without gateway

This patch addresses the exception in ipv6service when a IPv6
subnet is created without the gateway_ip.

Change-Id: Ibfa71e1775e15a51e717557ad080634d46d1220c
Signed-off-by: Sridhar Gaddam <sgaddam@redhat.com>

Merge "Fix boron rc1 build by removing netvirt-it-utils" into stable/boron

Fix boron rc1 build by removing netvirt-it-utils

Change-Id: I0661c19bbbfdc53ef59f1ab01f489c90e80d3c48
Signed-off-by: Sam Hague <shague@redhat.com>

BUG-6455 - due to blueprint migration, the order in which the respective
ConfigActivator.start() methods is called for Netvirt and
Netvirt-Providers is now random.

This could cause timing issues where dependencies are not yet loaded
when modules like NetVirt starts up.

We need to inject the dependency of OvsdbInventoryService for
NetvirtProviders so that NetvirtProviders has that service loaded before
it starts.

Change-Id: I431554dbd76c9cad71e6d49916a8eb9e4d456f71
Signed-off-by: Bertrand <bertrand.low@hcl.com>

Merge "Netvirt - Neutron QoS rate limit support" into stable/boron

Netvirt - Neutron QoS rate limit support

Adds Neutron QoS support to Neutronvpn.

Supports listening to neutron nb for
qos policy and bandwidth limit rule changes and configuring
corresponding ovsdb ports.

Supports:
- add, update, remove of qos policy assigned to Neutron port
- add, update and delete of bandwidth limit
  rules assigned to a Neutron port.
- add, update and delete of bandwidth limit rules
  assigend to a Neutron network.

todo:
- dscp marking rules (for port and network)

Testing procedure:
1.  Create neutron network, subnet and instance.
2.  Create neutron qos policy and rules via postman.
3.  using postman assign policy to postman, remove, add rules, etc.
4.  Result:  corresponding ovsdb port ingress policing attributes
    are configured appropriately.

v2 - rebase with master
v3 - rebase with master - blueprint changes
v4 - address review comments
     use different source for mdsalutils
     fix LOGGER class
v5 - rebase to master
v6 - rebase to master

Change-Id: Ibc9e0c8c2883eebf83d5f836a91ab8bc6c5e6469
Signed-off-by: Eric Multanen <eric.w.multanen@intel.com>

Bug 6473: Fix for CSIT SSH issue.

 * If security rules created with CIDR as 0.0.0.0/0 in TCP Custom rules
   it creates nw_dst=0.0.0.0 because of this behavior TCP packets are
   getting dropped.
 * Fix: If CIDR created with 0.0.0.0/0, nw_dst and nw_src will be
   removed from table 40 and table 90.

Change-Id: I82fdfa49d4da545619259572de8e0d2f571107b1
Signed-off-by: bharathi.m <bharathi.m@hcl.com>

Merge "Fixed the vpninstance creation" into stable/boron

Merge "Refactor aclservice to for all the acl types to inherit the same abstract class" into stable/boron

Fixed the vpninstance creation

The vpn instance was created in the NeutronVpnManager and there was a bug.
It could not create it without the external gateway info.
Should now create it after external gateway info creation

Change-Id: I203693ca69a48d06b69bb8acc9868a899371fcea
Signed-off-by: yair <yair.zinger@hpe.com>

Refactor aclservice to for all the acl types to inherit the same abstract class

Depends on:
https://git.opendaylight.org/gerrit/#/c/44453

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

Change the register that is used in acl learn actions

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

Merge "Add support for using kernel modules with docker for IT" into stable/boron

Merge "BUG-6495 BC Grp wrong for E/W VLAN provider net" into stable/boron

Add support for using kernel modules with docker for IT

This enabled support for using the kernel modules with Docker containers. Also the default seucity-group-mode is set to transparent until the proper support for stateful and stateless are included in the IT.

Use the following commands to test netvirt from the netvirt/vpnservice directory:

1. Using DockerOvs in userspace
mvn -nsu -f it/pom.xml verify -Pintegrationtest -Dovsdb.controller.address=172.17.0.1 -Dit.test=NetvirtIT#testNeutronNet -Dovsdb.userspace.enabled=yes -Dsgm=transparent

2. Using DockerOvs with kernel modules.
mvn -nsu -f it/pom.xml verify -Pintegrationtest -Dovsdb.controller.address=172.17.0.1 -Dit.test=NetvirtIT#testNeutronNet -Dsgm=transparent

3. Using an externally started Docker container (or vm)
mvn -nsu -f it/pom.xml verify -Pintegrationtest -Dovsdb.controller.address=172.17.0.1 -Dit.test=NetvirtIT#testNeutronNet -Dovsdb.userspace.enabled=yes -Dsgm=transparent -Dovsdbserver.ipaddress=127.0.0.1 -Dovsdbserver.port=6641

Use the following commands to test legacy netvirt from the netvirt/openstack directory:

mvn -nsu -f net-virt-it/pom.xml verify -Pintegrationtest -Dovsdb.controller.address=172.17.0.1 -Dit.test=NetvirtIT#testNeutronNet -Dovsdb.userspace.enabled=yes
mvn -nsu -f net-virt-it/pom.xml verify -Pintegrationtest -Dovsdb.controller.address=172.17.0.1 -Dit.test=NetvirtIT#testNeutronNet
mvn -nsu -f net-virt-it/pom.xml verify -Pintegrationtest -Dovsdb.controller.address=172.17.0.1 -Dit.test=NetvirtIT#testNeutronNet -Dovsdb.userspace.enabled=yes -Dovsdbserver.ipaddress=127.0.0.1 -Dovsdbserver.port=6641

Change-Id: I9e8fac1fa25d9e55f763f66bc87984ebab8738d3
Signed-off-by: Sam Hague <shague@redhat.com>

Merge "Fix IndexOutOfBoundsException when a port with IPv6 address is deleted" into stable/boron

Merge "Addition of BgpAlarms" into stable/boron

Fix IndexOutOfBoundsException when a port with IPv6 address is deleted

This patch avoids the following exception.
java.lang.IndexOutOfBoundsException: Index: 0, Size: 0
 at java.util.ArrayList.rangeCheck(ArrayList.java:653)[:1.8.0_101]
 at java.util.ArrayList.get(ArrayList.java:429)[:1.8.0_101]
 at *netvirt.neutronvpn.NeutronPortChangeListener.addPortToSubnets(NeutronPortChangeListener.java:552)
 at *.netvirt.neutronvpn.NeutronPortChangeListener.handleNeutronPortUpdated(NeutronPortChangeListener.java:355)
 <SNIP>

Change-Id: If9de109b22dde15da07dcd9284b8525ffba2b6fb
Signed-off-by: Sridhar Gaddam <sgaddam@redhat.com>

Merge "BUG-6501: avoid NPE if sgm isn't defined" into stable/boron

Merge "BUG 6489: Routes to InterVpnLink not advertised on BGP" into stable/boron

Addition of BgpAlarms

Change-Id: I14071695fa0ed858627b0bb1ea16e90ae2188cdc
Signed-off-by: Chintan Apte <chintan.apte@ericsson.com>

BUG-6495 BC Grp wrong for E/W VLAN provider net

As per the explanation in the bug, this group
is configured twice and one of these times is
broken for VLAN and only works for VxLan.
Easy way to solve this it to only call the second
method when the network type is VxLan. That's
what this patch does. Would be better if this
group was configured in one place, coherently.

Change-Id: Ia5d8f003d2f591b6a3d366928683c9e1a3515eb6
Signed-off-by: Josh <jhershbe@redhat.com>

Merge "Learn Security Groups bug fixes" into stable/boron

BUG-6501: avoid NPE if sgm isn't defined

System.getProperty("sgm") returns null if the system property isn't
defined, which throws a NullPointerException on the equals() call.
Inverting the equals() avoids this.

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

BUG 6489: Routes to InterVpnLink not advertised on BGP

 + Missing functionality: when a route pointing to an interVpnLink
   endpoint is added/removed to a Neutron Router, CSC must advertise
   the prefix. The nexthop(s) for the BGP are the DPNs where the
   endpoint of the InterVpnLink was instantiated

 + Moves some InterVpnLink utility methods from VpnUtil.java to
   InterVpnLinkUtil.java

 + Changes some return values to Optional.

Change-Id: I9e671b34400151099e501d26a8a7839c7730d0b7
Signed-off-by: Suraj Ranjan <suraj.ranjan@ericsson.com>

Learn Security Groups bug fixes

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

Fix bug wherein console commands would not work with Karaf ssh session

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

Bug 6477 - Fixes dissc of rtr from VPN & VPNPortFixedIpToPort

1. Fix to ensure adjacencies are deleted properly for learnt routes upon
dissociation.
2. Refactor to reduce reads/writes/builders
3. Added synchronization on interfacename for Arp and NeutronVPN while
updating vpninterface-adjacencies.
4. Fix in remove FibManager to not call leakRouteIfNeeded for non-BGP
routes
5. Renamed ArpRemoveCacheTask and ArpAddCacheTask classes to have proper name.
6. Eliminated reads upon DS deletes for VpnPortIPToPortData DS.

Change-Id: Ib6580c71cbce6451d411e1c5f030582225830b4d
Signed-off-by: Abhinav Gupta <abhinav.gupta@ericsson.com>

Merge "Copy default aclservice-config.xml to etc/...config" into stable/boron

Merge "Fix read of VpnPortipToPort to read from Operational" into stable/boron

Merge "Use node conn info remote ip for sff" into stable/boron

Copy default aclservice-config.xml to etc/...config

Change-Id: I13c3f7618830c8864a1c3cd1a91bba4c7efe7989
Signed-off-by: Sam Hague <shague@redhat.com>

Merge "Handle learn security groups" into stable/boron

Fix read of VpnPortipToPort to read from Operational

In patchset https://git.opendaylight.org/gerrit/#/c/43719/
VpnPortipToPort is written operational DS.

This commit fixes NatUtil/SubnetGwMacChangeListener to read from
the updated location.

Change-Id: I7045c9bff47ef0bde7fc3e9a9d8a3b526e54e125
Signed-off-by: Koby Aizer <koby.aizer@hpe.com>

Use node conn info remote ip for sff

Change-Id: I789e650fb465bc2f8123cd021c242d315320c906
Signed-off-by: Sam Hague <shague@redhat.com>

Handle learn security groups

Change-Id: I9d3385502a1e902ba6dd1a034927595d04b54ee8
Signed-off-by: Slava <slava.radune@hpe.com>
(cherry picked from commit ac4ba29b50a1f768016e50b61222b3b4a7eeecb7)

BUG 6487: Extra routes to IVpnLink endpoint not in FIB

 + LportTag range for InterVpnLink endpoints has changed to a range of
   [170001..270000]. Those values does not fit in a uint16 attribute

 + Also applies a bit of refactoring to reduce the duplication of code
   as much as possible

 + It also aligns the system properties names to the ones defined
   in sdnc repository

 Apart from above bug fix this commit also deals with:

 1> Fixes in metadata writing
 2> InterVpnLink Active even if VPN not exists

Change-Id: Iafe72a651153a7ca48e68748c6b7b271541e835c
Signed-off-by: Suraj Ranjan <suraj.ranjan@ericsson.com>

Bug 6486 - Add waits for IT Utils

Change-Id: I315706be5e216d23de4f4a0d037cd272b698f734
Signed-off-by: Sam Hague <shague@redhat.com>
(cherry picked from commit 313777d7c8891c4d26bb343564b3eeda32191bd3)

Allow non-default controller-dhcp-enabled at init

Currently controller-dhcp-enabled cn only be set through yang and defaults
to false. To change one must wait for bundles to come up and change
through restconf. This adds option to specif a different default at bring
up.

To set the default for controller-dhcp-enabled:

1. Change the value in dhcpservice/.../netvirt-dhcpservice.xml
2. Copy the file into the unzipped distribution before starting karaf:
  cd netvirt/vpnservice
  mkdir -p ./distribution/karaf/target/assembly/etc/opendaylight/datastore/initial/config
  cp dhcpservice/impl/src/main/resources/netvirt-dhcpservice.xml ./distribution/karaf/target/assembly/etc/opendaylight/datastore/initial/config

Change-Id: I6768e117515a4ff2c52e70961efb3aeac65c8d5f
Signed-off-by: Vishal Thapar <vishal.thapar@ericsson.com>
(cherry picked from commit d115bc161fe0d562c23ae5d041778590616ca88d)

Merge "Fixes Bug 6469" into stable/boron

Merge "code optimization for cluster environment" into stable/boron

Merge "Fixes Bug 6478" into stable/boron

BUG 6389: ServChain not applied to VMs in VPN

Service Chaining Traffic should be forwarded to the DPN connected to
the Fist VM of SC Path.

Aprt from fixing above bug, this review also deals with the following:

1> Fixes bug in InterVpnLink static route provision
2> Removes NPE protection from NeutronVpnmanager

Change-Id: Ic31e0399e5684d7512571a4d63f4ab01fd3400a3
Signed-off-by: Suraj Ranjan <suraj.ranjan@ericsson.com>

Fixes Bug 6469

CtZone is added while submitting the packet to the netfilter.
This help addressing the issue with overlapping ip address as each
elanId will be tracked separately.

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

Bug 6390 Fix ArpNotificationHandler initialization.

Change-Id: If9b755be9dab216e75c945bc9c28e9943f954e06
Signed-off-by: Achuth <achuth.maniyedath@ericsson.com>

Fixes Bug 6478

Changed the code to match the destination port for TCP and UDP in both
egress and ingress.

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

code optimization for cluster environment

1. handle to execute code only from one of the cluster node.
2. handle local cache updation in all the cluster nodes.

Change-Id: Iff2b79d4e1d96cb08883650bf9c82876e803aef2
Signed-off-by: gujele <imanilyadav@gmail.com>

Merge "Bug 6390 VpnPortIpToportData to use Oper DS" into stable/boron

Merge "Clean up a bunch of trival Checkstyle violations" into stable/boron

Merge "Blueprint fix" into stable/boron

Merge "No need to create br-ex any more. br-ex(it) ;)" into stable/boron

No need to create br-ex any more. br-ex(it) ;)

devstack no longer checks for it and we
do not use it.

Change-Id: I4f72331e321e5094ffb7b378cbee894932001dbf
Signed-off-by: Josh <jhershbe@redhat.com>

Blueprint fix

Registration of entity type "elan" missed out. This is used by L2Gw in
order to identify the owner for type "elan".

Change-Id: I583ec617529a82ee8a0db1dbd34421c29ac9f571
Signed-off-by: Kency Kurian <kency.kurian@ericsson.com>

elanmanager CLI console commands impl should never catch Exception

Thank You to Checkstyle for pointing these out.  Please see
https://wiki.opendaylight.org/view/BestPractices/Coding_Guidelines#IllegalCatch
for background & details.

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

Merge "Port cloud-servicechain to Blueprint" into stable/boron

Merge "clean isExternal NPE and string syntax" into stable/boron

Merge "add local_ip to other_config to satisfy auto-tunnel" into stable/boron

Merge "More elan Blueprint clean-up" into stable/boron

Clean up a bunch of trival Checkstyle violations

None of these changes has any real impact on code, so please merge this
ASAP, to avoid re-re-re-base.

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

clean isExternal NPE and string syntax

Change-Id: I5aed5a3bcc830a9df461e7c76fa92d0095ff796f
Signed-off-by: Sam Hague <shague@redhat.com>

add local_ip to other_config to satisfy auto-tunnel

Change-Id: I06b6688572440f2445a0a0f64f9fe6a6933ac18b
Signed-off-by: Sam Hague <shague@redhat.com>

Merge "ACL Support for IPv6 IPAM" into stable/boron

Merge "Neutronvpn caches require update upon Router/Network changes" into stable/boron

Merge "Fixing external network group table lifecycle" into stable/boron

Merge "Remove elanUtils instance from ElanInstanceManager" into stable/boron

More elan Blueprint clean-up

* Use static ElanUtils methods where appropriate.
* Drop ElanUtils and IInterfaceManager from
  ElanInterfaceStateChangeListener.
* Drop ElanUtils and MdsalUtils from ElanStatisticsImpl.
* Remove ElanServiceProvider.getElanUtils().

Change-Id: I00271336051cf82ba28bdc596576000ac201a51d
Signed-off-by: Stephen Kitt <skitt@redhat.com>
(cherry picked from commit 35f27a7e0cf6da670d7559d0755ede25ec8638b9)

Merge "Create VLAN trunk once for multiple VLAN networks" into stable/boron

ACL Support for IPv6 IPAM

This patch implements the following support.
1. Programs the anti-spoofing rules not to allow a VM to run an IPv6 Router.
2. Allows the necessary ICMPv6 traffic in the VM-egress pipeline.
3. Allows the necessary ICMPv6 traffic in the VM-ingress pipeline.
4. Fixes the DHCPv6 ports

Pending:
Some of the flows should be made more restrictive by taking into
consideration the MAC address of the IPv6 Router/DHCP server. This
change is required for both V4 and V6 rules and will be handled in
a separate patch.

Change-Id: I5e64d989dfa4d03ad6ce941201e32f2f2caef308
Signed-off-by: Sridhar Gaddam <sgaddam@redhat.com>

Fixing external network group table lifecycle

Subnet changes will create and remove group tables

Change-Id: I000a295f93af5f1217cc54d5ad2cbf3b3aa3ed9a
Signed-off-by: yair <yair.zinger@hpe.com>

Merge "Bug 6449 - Issues in Service Function Forwarder translation" into stable/boron

Merge "Rename JMX MBean's ObjectName from com.ericsson to org.opendaylight" into stable/boron

Neutronvpn caches require update upon Router/Network changes

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

Port cloud-servicechain to Blueprint

This also enables the feature.

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

Rename JMX MBean's ObjectName from com.ericsson to org.opendaylight

From com.ericsson.sdncp.services.status:type=SvcElanService to
org.opendaylight.netvirt.elan:type=SvcElanService

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

Fix completely broken ElanStatusMonitor JMX MBean

including a test for future non-regression of this problem:

javax.management.NotCompliantMBeanException: MBean class
org.opendaylight.netvirt.elan.statusanddiag.ElanStatusMonitorJMX does
not implement DynamicMBean, and neither follows the Standard MBean
conventions (javax.management.NotCompliantMBeanException: Class
org.opendaylight.netvirt.elan.statusanddiag.ElanStatusMonitorJMX is not
a JMX compliant Standard MBean) nor the MXBean conventions
(javax.management.NotCompliantMBeanException:
org.opendaylight.netvirt.elan.statusanddiag.ElanStatusMonitorJMX: Class
org.opendaylight.netvirt.elan.statusanddiag.ElanStatusMonitorJMX is not
a JMX compliant MXBean)
at
com.sun.jmx.mbeanserver.Introspector.checkCompliance(Introspector.java:176)
at
com.sun.jmx.interceptor.DefaultMBeanServerInterceptor.registerMBean(DefaultMBeanServerInterceptor.java:317)
at
com.sun.jmx.mbeanserver.JmxMBeanServer.registerMBean(JmxMBeanServer.java:522)
at
org.opendaylight.netvirt.elan.statusanddiag.ElanStatusMonitorJMX.registerMbean(ElanStatusMonitorJMX.java:30)
at
org.opendaylight.netvirt.elan.statusanddiag.ElanStatusMonitorJMX.init(ElanStatusMonitorJMX.java:24)

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

Create VLAN trunk once for multiple VLAN networks

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

Remove elanUtils instance from ElanInstanceManager

With a couple of clean-ups in ElanUtils, ElanInstanceManager can run
with only static method calls.

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