Add of-config Nitrogen release notes

Change-Id: I05ce872bf7dae1c1aaf2866379a2d29bad61d3f3
Signed-off-by: Wsx25289 <10200860@zte.com.cn>

Merge "Update TTP release notes for Nitrogen"

Merge "Update Cardinal Nitrogen release notes"

Update TTP release notes for Nitrogen

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

Updated git submodules

Project: genius master 9b10915a8a9955f27550067a6fb402c5d4a3b489

Remove un-used InterfaceManagerRpcService IMdsalApiManager & methods

* buildInstructions()
* makeTerminatingServiceFlow()
* makeLFIBFlow()
* getFlowRef()

and instance field IMdsalApiManager mdsalMgr.

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

Updated git submodules

Project: infrautils master d83cefef8c54d9b2066906f8caf22195e61dd015

Cleanup some methods not used

Let's remove some methods not even used within this package. Fix some
issues related to checkstyle as well.

Some methods still used might be eventually replaced by "standard" ones
in later patches.

Change-Id: I74f3cb47ff01f3daa3ee5bcc84de18fc86c84f47
Signed-off-by: David Suarez <david.suarez.fuentes@gmail.com>

Updated git submodules

Project: integration/test master 6b96e5dbd8d7ea3b77e2f3fae8c06a671943f4c2

Add a WUKS to netconf connector check

The check for netconf connector is not able to finish in time,
though the subsequent test passed. Adding the check.

Change-Id: Ib6f0ece50643ebab7e8e32dc6023941ce6d2c4e2
Signed-off-by: Atul Gosain <atul.gosain@gmail.com>

Update Cardinal Nitrogen release notes

Change-Id: I791b1eb294ea1a324e91da7e67f1f9f8b91cdf0b
Signed-off-by: Subodh <subodh.roy@tcs.com>

Updated git submodules

Project: sfc master 266e23c41603e42f18d2663e3cf6e3dc06cc00dd

Fix findbugs/sonar violations in sfc-provider

- Exception handlers should preserve the original exception
- Null value is guaranteed to be dereferenced
- Possible null pointer dereference
- Method parameters should not be reassigned
- Collapsible "if" statements should be merged
- Method ignores exceptional return value
- Comparison of String objects using == or !=
- Consider using Locale parameterized version of invoked method
- Field should be package protected (malicious code vulnerability)
- Write to static field from instance method
- Inefficient use of keySet iterator instead of entrySet iterator
- Load of known null value
- Redundant nullcheck of value known to be non-null
- Useless object created
- Unused local variables should be removed
- Comments should not be located at the end of lines of code

Also enabled findbugs enforcement.

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

Merge "nitrogen release notes for unimgr project"

Updated git submodules

Project: sfc master 0fcd57a9b1f90016465cc55e035d61b42fc029b7

Fix findbugs/sonar violations in sfc-genius

Enabled findbugs enforcement. Several violations are
invalid so I had to supress them.

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

Updated git submodules

Project: genius master 852f343731c81d3b0da15bf96517ca3776f6ba83

Enforce findbugs maven plugin for module arputil-api

Change-Id: I0a0e3137468cb1509e905bb6a5f7134cd016fb3e
Signed-off-by: David Suarez <david.suarez.fuentes@gmail.com>

nitrogen release notes for unimgr project

Change-Id: I7b00d7781e065c987d9d05da3af7df0453ec761f
Signed-off-by: Bartosz Michalik <Bartosz.Michalik@amartus.com>

Updated git submodules

Project: releng/builder master 91c1e1491a67d0ac97d92d8fd1da3c78dc5fa66d

Merge "Auto Update CSIT Jobs to run for boron"

Auto Update CSIT Jobs to run for boron

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

Updated git submodules

Project: releng/builder master 95385859250fe1fd0e9c1e988c51d0c2233f404c

Merge "Auto Update CSIT Jobs to run for oxygen"

Auto Update CSIT Jobs to run for oxygen

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

Updated git submodules

Project: releng/builder master fe05fcfb0a58d879645debfe771a1b2f676968a9

Merge "Auto Update CSIT Jobs to run for nitrogen"

Auto Update CSIT Jobs to run for nitrogen

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

Updated git submodules

Project: releng/builder master aad5dc53130950c5ef4d2f61777f81388c550b46

Auto Update CSIT Jobs to run for carbon

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

Updated git submodules

Project: netvirt master edbe08e2378dbd34e5c219d37bef2d860f0b2a9d

bgpmanager thrift upgrade to 0.9.3

Change-Id: I5154fc5b3522313b30d647b68ed37dfd9e02f9ca
Signed-off-by: Siva Kumar Perumalla <sivakumar.perumalla@ericsson.com>

Updated git submodules

Project: netvirt master 39ab6bb866020a3bfc987c6cf69ae09a2be0f415

Bug 8963 - Option to configure EVPN address family

evpn can  be configured as neighbor from karaf CLI with command bgp-nbr
In the bgp-nbr command, for supported address families, vpnv4 is added as
it was missing.

Change-Id: I9d8854c9a7a05f1a48836f0ae88c7934b31a14be
Signed-off-by: Yugandhar Reddy Kaku <yugandhar.reddy.kaku@ericsson.com>

Updated git submodules

Project: genius master 05092c01759968cf1a4c75d3e4d8db2a40461446

Lower log level for non errors

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

Updated git submodules

Project: netvirt master 0f1503f59962293a5bcdbe1e2a6adfc9b8afa1a6

Bug 9196 - Maxpath value should be between 1 to 64 in BGP multipath

Adding a check to ensure maxpath value for BGP multipath is in the range 1
to 64

Change-Id: I03786726053672d04b8ef552cb16cc45a9c4a840
Signed-off-by: Gowri R <gowri.r@ericsson.com>

Updated git submodules

Project: releng/builder master 55d64d2b6a5fac58e5bbb4e9516f5846a8df560e

Remove SNMP CSIT

the job(s) for SNMP CSIT have never worked, and have been ignored
for several (if not all) releases. it's a waste of time and
resources to have them around.

see the status pages for Nitrogen, Carbon, Boron
and Beryllium for history:
  https://docs.google.com/spreadsheets/d/1MYyGLFWN2RzUkJl8XMzXQ-3zWuOrUCQpIS6ORbmf4_U/edit#gid=421626541
  https://docs.google.com/spreadsheets/d/1VcB12FBiFV4GAEHZSspHBNxKI_9XugJp-6Qbbw20Omk/edit#gid=1226505712
  https://docs.google.com/spreadsheets/d/1zImtd764e-hOgJAxoJKl85fxHCPu2agLfqsBtf13zQY/edit#gid=1037896712
  https://docs.google.com/spreadsheets/d/1hci5TMUPyB6PX8Al-fwfVqvs5SQVa2wZLja_7rKWN6o/edit#gid=843418255

partner patch in integration/test:
  https://git.opendaylight.org/gerrit/63428

Change-Id: I3588ddb5890651ff90e5c5e03082ab16950124a1
Signed-off-by: Jamo Luhrsen <jluhrsen@redhat.com>

Updated git submodules

Project: releng/builder master eafe65d034ba20f98e2b49743f852c786938ab2f

Don't skip votes for puppet-full tests

Realized that this config was the culprit for why Jenkins was running
but not voting for some jobs.

Change-Id: Iff49ea0156d2332368ae4e0a9b854f1226c5ae46
Signed-off-by: Daniel Farrell <dfarrell@redhat.com>

Updated git submodules

Project: integration/packaging master ebe438535f0694fac3fe63f8727b53f6fc63f2a7

Refactor packaging Vagrant env

Change-Id: Ib4a3dd05b681a9dae92e8ffb28823ff89344889e
Signed-off-by: Daniel Farrell <dfarrell@redhat.com>
Common Vagrantfile for local testing

A single vagrant file makes sense post refactoring than two vagrantfiles
in each deb and rpm directories. The commit also updates debian box to
the new stable 'stretch' version.

Change-Id: I62543f02ebd768540593a78d5ce644187b3c42a8
Signed-off-by: Alok Anand <alok4nand@gmail.com>

Updated git submodules

Project: genius master b7e0cf0842f8247707c477b2eb51e3bfc0231da0

Mechanical: ItmTunnelAggregationHelper Eclipse Auto-Save clean-up

just so that this won't clutter
https://git.opendaylight.org/gerrit/#/c/63372/5/itm/itm-impl/src/main/java/org/opendaylight/genius/itm/confighelpers/ItmTunnelAggregationHelper.java@306

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

Updated git submodules

Project: netvirt master a991d06713483f33b5a37ee2788de947efa03878

BUG 8930 - delete Op VPN interface when deleting external network

Op VPNInterafce is removed in VrfEntryListener when the last
adjacency for the VPNInterface is removed.
The problem is VPN interface for external network doesn't
have any adjacency nor VrfEntry.
As a result, when the external Vpn interface is removed
the Op VpnInstance is not removed as it should.

Fix: Delete the Op VPN interface in VpnInterfaceManager
if the VPN interface to be deleted doesn't have any adjacency

Change-Id: Ice4b9fcb0103c6aff25053cc5b3db3b01311be06
Signed-off-by: Vinh Nguyen <vinh.nguyen@hcl.com>

Updated git submodules

Project: netvirt master 062f957a7cf83f7a5918b7e842782dacaf865c97

Lower log levels for non error's

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

Updated git submodules

Project: netvirt master a13bc037ff558d61e33b28dabd767671a4504321

elanmanager: clean up Futures collections

This patch replaces lists of Futures with singleton lists where
possible. It also removes instances of passing such lists around,
using return values instead.

This patch also cleans up the job classes generally: instance fields
are final where possible, unused fields and parameters are removed.

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

Updated git submodules

Project: netvirt master f35f3a38e5ff9862d34dc529d3119091e233ce53

Bug 9193 - In conntrack SNAT , flows are programmed twice on a router
g/w set.

Check is added in RouterDpnChangeListener to check if the port added is
th pseudo router interface and prevent the notification.

Fixed an exception caused by not creating VNI pool.

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

Updated git submodules

Project: integration/test master a4e1e8a412f915d33dcfdcfce583e33c41c445c5

Get compute node hostname from hypervisor list

Change-Id: I756e90602490f2c96d2eb41e8b4319c618cd7880
Signed-off-by: Jamo Luhrsen <jluhrsen@redhat.com>

Updated git submodules

Project: genius master 865ad959b30006fbc59575397fbba09d4087e8cd

Fix problems detected by the FindBugs plugin

- FindBugs enforcement.

Change-Id: Ibfb739a1e74c64a85bafc49b0b1d0d61e5005b04
Signed-off-by: David Suarez <david.suarez.fuentes@gmail.com>

Updated git submodules

Project: genius master 04e1bb4aed2cb712d5c29a4e5835a2d2cac52924

InterfaceManagerRpcService include exception cause in RPC result

and LOG.error SOME (but not all) of such problems (logging is as before)

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

Updated git submodules

Project: genius master 0dfaa335dfbf99893f432ac6effd2c93cd035df7

Add a basic framework to create DataTreeChangeListeners

Add a basic framework to create DataTreeChangeListeners.

Change-Id: Iaf9ddea9bf312ba7659a57f1d63d5d93fa949938
Signed-off-by: David Suarez <david.suarez.fuentes@ericsson.com>
Signed-off-by: David Suarez <david.suarez.fuentes@gmail.com>

Updated git submodules

Project: genius master a9ba3461f8b9923c9933b8d27649d91430ff0ac1

InterfaceManagerConfigurationTest using TestableQueues awaitEmpty

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

Updated git submodules

Project: genius master 4737e872a897f3f366c013f7a5d9a5b6617515f5

Clean up MDSALManager.removeGroup()

Removing a group only needs a DPN and group identifier; deprecate the
other forms, add a simple variant of removeGroup() and remove
duplicates.

This patch also moves the test package so that we avoid needing to
make methods public.

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

Updated git submodules

Project: genius master 1c3556fb583a961f7ccd24fe8e6892bf38031f16

Cleanup config-parent usage

Change-Id: Ic51def71c73210ed21969238538449b90e45b005
Signed-off-by: Vishal Thapar <vishal.thapar@ericsson.com>

Updated git submodules

Project: netvirt master a883688e2176af0969a0dd2f6387a6a8465452dd

IfMgr clean-up

* Avoid calling overridable methods in the constructor.
* Initialise fields in the declaration.
* Remove unused parameters.
* Remove null checks after new, new never returns null.
* Remove unnecessary return statements.
* Remove end-of-method null assignments for local variables.
* Use Map methods to simplify processing.
* Fix a couple of invalid null-handling constructs.

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

Updated git submodules

Project: netvirt master 0520c3c10436047d61337bcfa0f7bdd7584beae7

Restrict NeutronvpnUtils.read

As a first step towards removing NeutronvpuUtils.read, replace all
external uses with SingleTransactionDataBroker.syncReadOptional and
adjust the exception handling accordingly (which also allows us to
clean up a number of disabled Checkstyle warnings).

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

Updated git submodules

Project: netvirt master 0280ea39a7e3b41d1275b1f845183566aec8b661

bug-9190: NullPointerException at getIsExternal

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

Updated git submodules

Project: netvirt master a4793f9d20883403af79765b9d59dfa1062fe121

BUG 8969 - Fix Exeption when clearing external router GW

When clearing the router GW, the flow entry for router GW
on the NAPT primary switch is also removed.
Exception occurs when we try to delete the router GW
but there is no NAPT primary switch.

Fix: check for existence of NAPT primary switch
before deleting the router GW flow on the switch.
Also downgrade ERROR to DEBUG log level when primary
NAPT switch can not be found since this could
happen when no internal network attaches to the router yet.

Change-Id: I7ebc0d60183954044aeb8a4e8f062b4d227f6c76
Signed-off-by: Vinh Nguyen <vinh.nguyen@hcl.com>

Updated git submodules

Project: genius master e31093689dcd1852d907129a2e35119676d3920d

Migrating IDManager test cases to use GuiceRules

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

Updated git submodules

Project: infrautils master 155ddd811b61df8d6512bc142b7f4ab8d8eee839

LoggingRejectedExecutionHandler

mv from org.opendaylight.genius.infra.LoggingRejectedExecutionHandler

it's currently used by the genius data tree change listeners, but really
belongs here - especially with the upcoming new listeners in genius
https://git.opendaylight.org/gerrit/#/c/59177/, where the idea will be
(I expect) to, later/soon, use a central ExecutorService (perhaps
offered by infrautils?) which would this here instead of there.

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

Updated git submodules

Project: genius master c958691e32f3f8e58f52d6b4b22ea7dd6c6eae52

Removed unnecessary/unused CSS config yang remnants

Removed unnecessary/unused config yang imports and service type
identities.

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

Updated git submodules

Project: netconf master f9a0c88a73fe4ba7577c1d072251b8f286bbdd62

Merge "Bug 8988 - Check for empty payload properly"

Bug 8988 - Check for empty payload properly

Do not use InputStream#avalaible method to check for empty payload's
input streams in our implementations of MessageBodyReader interface.
InputStream#avalaible method returns an estimate of the number of bytes
that can be read from input stream without blocking. We cannot be sure
the stream is actually empty, even if this method returns 0.

Wrap the payload's input stream in PushbackInputStream and try to read
and unread from it instead.

Change-Id: I43b3a8d837f3dc4bc59d7cc2db29ffa06786844f
Signed-off-by: Jakub Morvay <jmorvay@cisco.com>

Updated git submodules

Project: netconf master 7d5a5e624a2acdf98565738a36248389907d1ab3

Enable public key based authentication for netconf

Change-Id: Icca97924515537b670688767d41ddd4bd6e1bbf6
Signed-off-by: Atul Gosain <agosain@brocade.com>
BUG 8839: Revert "Make netconf utilize encrypted passwords only"

This reverts commit 1867bf17d781f21473dabcde2ad3f85b9b1defc6.

Change-Id: Ia5d97c118d805bdeff9e47f466a9ec64b58c03ca
Signed-off-by: Tomas Cere <tcere@cisco.com>
Adjust tests for yangtools changes

yang-maven-plugin now adds revision to the packaged yang file name.
This is introduced in https://git.opendaylight.org/gerrit/#/c/62706/.
Adjust NetconfMappingTest and NetconfMDSalMappingTest to that.

https://git.opendaylight.org/gerrit/#/c/62731/ enforces revisions to be
in valid yyyy-MM-dd format. Adjust Restconf tests to this change.

Change-Id: I00372ee98b15d3194c8a8567a041f24f161d7276
Signed-off-by: Jakub Morvay <jmorvay@cisco.com>

Updated git submodules

Project: integration/test master e12ba90cd27577c4c6c70ee54f7d599b5f6777ba

Default BUNDLEFOLDER var to /opt/opendaylight

this currently will have zero effect on upstream ODL CSIT,
because we always pass this variable on the pybot command
line.

However, some other CSIT consumers (e.g. OPNFV) are seeing
lots of warnings [0] because they are not *always* passing
BUNDLEFOLDER in their robot argument list. This will remove
that error for them, while giving us a decent default of
/opt/opendaylight which is where the rpm package installs
to.

[0] https://git.opendaylight.org/gerrit/63293

Change-Id: Ie32d50cd8f42b05378949439b1b1976348032e59
Signed-off-by: Jamo Luhrsen <jluhrsen@redhat.com>

Updated git submodules

Project: netvirt master a48e9f1da21e75bedeb62214534b9c77c8efb43c

Bug 9105: close removeElanInterface transaction

Cancel the interface transaction if we return early from
removeElanInterface. Initialise the flow transaction after the early
return so we don’t need to handle it specially.

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

Updated git submodules

Project: netvirt master 75996fbe0f11fe3bd7ff766011a93d06be54344d

ElanUtils clean-up: ElanL2GatewayUtils

This patch initialises ElanL2GatewayUtils using BluePrint. This
involves adding static variants to a small number of methods in order
to break the dependency cycle involving ElanInstanceManager,
ElanInterfaceManager, ElanUtils and ElanL2GatewayUtils.

Change-Id: Icf17e0fb7683b871b8246b148d905e51bad4e598
Signed-off-by: Stephen Kitt <skitt@redhat.com>
ElanUtils clean-up: L2GatewayConnectionUtils

This turns L2GatewayConnectionUtils into a self-carrying bean, instead
of relying on ElanUtils to initialise it.

Change-Id: I2c5c8330e57a46048651a1f6b656fb34b099afe3
Signed-off-by: Stephen Kitt <skitt@redhat.com>
ElanUtils clean-up: remove unnecessary references

A number of classes reference ElanUtils instances even though they no
longer need them, or they call static methods via an instance. This
patch cleans them up.

Change-Id: I29bf1454e7a56ac474ef9198a48114b0b0447aea
Signed-off-by: Stephen Kitt <skitt@redhat.com>
ElanUtils clean-up: ElanL2Gateway{Multicast,}Utils

Extract various utility methods to ElanDmacUtils, ElanEtreeUtils and
ElanItmUtils, so that ElanL2GatewayMulticastUtils’ dependency on
ElanL2GatewayUtils (which needs to be post-filled after
initialisation) can be removed.

Change-Id: Iec71e32ea61ff1789537bcb1e7e35fa3253f8363
Signed-off-by: Stephen Kitt <skitt@redhat.com>
ElanUtils clean-up: more ElanL2GatewayMulticastUtils

Moving installDpnMacsInL2gwDevice from ElanL2GatewayUtils to
ElanInterfaceManager allows ElanL2GatewayUtils’ dependency on
ElanL2GatewayMulticastUtils to be removed.

Change-Id: I1af5c31f516f4dfe7121c16e797339b60a1d03ca
Signed-off-by: Stephen Kitt <skitt@redhat.com>
ElanUtils clean-up: ElanL2GatewayMulticastUtils

elanUtils.getInvolvedDpnsInElan is a very simple wrapper around
elanInstanceManager.getElanDPNByName, which isn’t even needed any more
since the latter never returns null. Using the latter directly in
ElanL2GatewayMulticastUtils means we no longer need ElanUtils there,
which simplifies initialisation (although not perfectly yet because of
the loop between ElanL2GatewayMulticastUtils and ElanL2GatewayUtils).

Change-Id: Ie4ef925d5e0c02e5476141430f844dc2b0a0647e
Signed-off-by: Stephen Kitt <skitt@redhat.com>
ElanUtils clean-up: make read() static

It turns out no callers use the variant of read() which relies on
ElanUtils’ own broker, so we can drop that and make read() static,
which reduces the requirement for ElanUtils instances.

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

Updated git submodules

Project: netvirt master 60c1f1547df05aef7e992305b638d18bfaf99e8b

aclservice: clean up Futures collections

This patch replaces lists of Futures with singleton lists where
possible.

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

Updated git submodules

Project: netvirt master ea33bf6ce1716882c1f3f03c6a24218a3fa4a04a

dhcpservice: clean up Futures collections

This patch replaces lists of Futures with singleton lists where
possible. It also removes instances of passing such lists around,
using return values instead.

This patch also cleans up the job classes generally: instance fields
are final where possible, unused fields and parameters are removed.

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

Updated git submodules

Project: integration/test master 5cbb1081a20b6bd7573b2ba5e675ddad6075b84f

Convert Sleep to WUKS in lispflowmapping CSIT

Part 2, 020_RPC_CRUD.robot

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

Merge "Update dluxapps release notes for Nitrogen"

Updated git submodules

Project: netvirt master fc4eb412241b2ce603ebc57e44291fdc8a1492ee

coe: clean up Futures collections

This patch replaces lists of Futures with singleton lists where
possible.

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

Updated git submodules

Project: netvirt master 0d7820fed5ffdbbb909776ac217a42f004661147

Remove un-used SynchronousEachOperationNewWriteTransaction

replaced by SingleTransactionDataBroker which is now widely adopted

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

Updated git submodules

Project: integration/test master f3036b01959596b7a70e1717c9f64f0e3d1c973b

Support https endpoints (Keystone and Neutron)

It simply replaces http://${NEUTRON}:9696 by ${NEUTRONURL} and
http://${KEYSTONE}:5000 by ${KEYSTONEURL}.

It also switches to Variables.robot and defines
${NEUTRON_SUBNETS_API} in it.

Change-Id: I76702facfbfb068160933fbba4dbb5f2cbc41d15
Signed-off-by: Cédric Ollivier <cedric.ollivier@orange.com>

Merge "Release notes for Nitrogen."

Updated git submodules

Project: integration/test master 5fc0d62dbc11b191a82617fdded9b0a189079dc8

Stabilize OF split connection test

It is possible that during unstable scenario of 2 connection
fighting each other there is no flow being pushed.

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

Updated git submodules

Project: sfc master b197b95fd96909352115197552b87ef9b4c36dbc

Fix findbugs/checkstyle/sonar violations in sfc-scf-vpp

- Method invokes inefficient new String(String) constructor
- Inefficient use of keySet iterator instead of entrySet iterator
- Should be a static inner class
- Method concatenates strings using + in a loop
- Dead store to local variable

Also enabled findbugs and checkstyle enforcement.

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

Updated git submodules

Project: sfc master 7c71514cd05c65f7bf6f92013bd4bb799e25bb4b

Fix findbugs/sonar violations in sfc-scf-openflow

- Utility classes should not have public constructors
- Remove unnecessary throws declaration in close()
- Boxing/unboxing to parse a primitive
- Dead store to local variable

Also enabled findbugs enforcement.

Change-Id: I49b6f5de493a5625c02b322d7341ec2ae3688d73
Signed-off-by: Tom Pantelis <tompantelis@gmail.com>
Fix findbugs/sonar violations in sfc_vpp_renderer

- Generic exceptions should never be thrown
- Dead store to local variable

Also enabled findbugs enforcement.

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

Release notes for Nitrogen.

Change-Id: I4ee377d08cd0c67abe94f2f33114775563d77805
Signed-off-by: Kevin <k.kershaw@cablelabs.com>

Merge "Add daexim release notes"

Update dluxapps release notes for Nitrogen

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

Merge "Update dlux release notes for Nitrogen"

Merge "Update documents for Cardinal Nitrogen"

Updated git submodules

Project: odlparent master f39ddfc271d86cdd79ad4202143c44ef70d274bb

maven-archetype-plugin pluginManagement

The new version property introduced here will be used in controller.

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

Updated git submodules

Project: odlparent master 0fbd31f74b19337d25fc3e7a6a870ddc9defee6a

Bump Checkstyle core from 7.6.1 to 8.2

and Sevntu from 1.21.0 to 1.24.2 (note the required change in groupId)

also includes removal of LeftCurly.maxLineLength = 100, which is not
supported (anymore; or -more likely- never was, probably the validation
just became stricter)

and removal of a duplicate RightCurly, which was always wrong

and required fixes to odlparent Java code showing violations now

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

Updated git submodules

Project: sfc master f4bbb9367a85ecc3f464fe383b6b554aef340ec6

Fix sonar violations in  sfc-pot-netconf-renderer

- Method parameters should not be reassigned
- Unused private fields should be removed

Also enabled findbugs enforcement.

Change-Id: I41a9dec0c59a5a5a6e0653c9788a21dcd61e955a
Signed-off-by: Tom Pantelis <tompantelis@gmail.com>
Fix findbugs/sonar violations in sfc-openflow-renderer

- Method call passes null for non-null parameter
- Method ignores exceptional return value
- Consider using Locale parameterized version of invoked method
- Field isn't final but should be
- Method invokes toString() method on a String
- Inefficient use of keySet iterator instead of entrySet iterator
- Should be a static inner class
- Private method is never called
- Method invokes inefficient Number constructor; use static valueOf instead
- Dead store to local variable
- Write to static field from instance method
- Unread public/protected field
- Field not initialized in constructor but dereferenced without null check
- Useless object created
- Collapsible "if" statements should be merged

Also enabled findbugs enforcement.

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

Updated git submodules

Project: sfc master d4f3058899e146488932f3d05302b3100edfd620

Fix findbugs/sonar violations in sfc-ios-xe-renderer

- Suspicious reference comparison of Boolean values
- Dead store to local variable
- Either log or rethrow this exception

Also enabled findbugs enforcement.

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

Updated git submodules

Project: odlparent master 64079afbd3649cb337fcb3ad24bb65bf4c5ac426

Eliminate scala.version and scala.micro.version

These properties are interfering with scala-maven-plugin, which is
using scala.version to control target versions.

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

Update dlux release notes for Nitrogen

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

Updated git submodules

Project: infrautils master dcfa282bc03aa7001b423f818bc1dc8866041b9f

Add .factorypath to .gitignore

Change-Id: I245804fb9c576a5ce6f37cd95f7b4809be7175c9
Signed-off-by: David Suarez <david.suarez.fuentes@gmail.com>

Updated git submodules

Project: infrautils master 8d9f7ebbe9b6f8709ec8b5dcb1b0b9feff27a220

JobCoordinatorImpl catch Exception instead of Throwable

after all, let's undo https://git.opendaylight.org/gerrit/#/c/60588/,
based on the latest discussion around this topic in
https://git.opendaylight.org/gerrit/#/c/60855/

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

Updated git submodules

Project: netvirt master 5edd37427dee16a6b356c9cf3e51b890b07a55f6

cleanup sync

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

Updated git submodules

Project: releng/builder master 33a187693430a85a0befa9c309cfe752aaf6c31c

Merge "Copy constraints file and cleanup log vars"

Copy constraints file and cleanup log vars

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

Updated git submodules

Project: ovsdb master 2eea33c2efaa563d8f113784ebf378fecc02c6bd

BUG 9166 - Fix Netvirt L2GW Illegal state exception

Fix exception and modify logic for comparing OVSDB
Hardware_vtep database version retrived from device
to min version specified.

Change-Id: I0b882b4a4883af80a3ea01224781e781eaefac1a
Signed-off-by: Victor Pickard <vpickard@redhat.com>

Updated git submodules

Project: releng/builder master daef992523b338772a9eb5bc7861c5b57a0a293b

Fix typo s/_/- for sfc netvirt csit

Change-Id: I1adbd111f41b57e2626472a33a7d6c6e3119c689
Signed-off-by: Jamo Luhrsen <jluhrsen@redhat.com>

Merge "OpenFlowPlugin Nitrogen Release Notes"

Merge "Add OCPplugin Nitrogen release note"

Merge "NETCONF Nitrogen release notes"

Correct typos in faas release-notes

Change-Id: I76e1b4e85c7ec0192d2f19bdb7516b58f5b010c5
Signed-off-by: xingjun <xingjun.chu@huawei.com>

Updated git submodules

Project: releng/builder master 481f117f954fce11483f96574361501b8d1f662f

Merge "Add stable/nitrogen branch for puppet-odl"

Add stable/nitrogen branch for puppet-odl

Change-Id: I801433d85f3ed610d6df3bb6867fd2ed781ea167
Signed-off-by: Daniel Farrell <dfarrell@redhat.com>

Updated git submodules

Project: integration/test master 147ebb212d7eb25a353b404090ce9f474e94354f

Rename yang-system-test module

 * rename yang-system-test module in Yangtools
   to odl-yang-validator

Change-Id: I67909babb2c61446327d7d7b1bae33f9029b4777
Signed-off-by: Jakub Toth <jakub.toth@pantheon.tech>

Updated git submodules

Project: releng/builder master 294ce275101f7315804557db4189f6d77557f212

Merge "Update global-jjb to v0.8.5"

Update global-jjb to v0.8.5

Pull in fix where RTD job was building in the wrong directory.

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

Updated git submodules

Project: releng/builder master 8d2423467ff887981f851e922042806734b5d45c

Merge "Exclude mininet-2.6 for Centos7"

Exclude mininet-2.6 for Centos7

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

Updated git submodules

Project: releng/builder master cd5c4fd34011d0171e69c02e706f48240e83a6aa

Update global-jjb to v0.8.4

This is to fix the docs RTD verify job not triggering on patch submit.

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

Updated git submodules

Project: sfc master 621b646e72cc17600d44ffa61a070756cdbcfc5c

Remove deprecated CheckedFuture

Remove deprecated CheckedFuture.

Change-Id: Ic2eaafd941dcda5511f2586e4e1f4c14d24eaa8e
Signed-off-by: David Suarez <david.suarez.fuentes@gmail.com>

Updated git submodules

Project: infrautils master a76c9cc096e87ab6a5308b5ef0de688c3c7a227b

LogCaptureRuleTest now shows logged errors even if test threw exception

and preserves exception thrown by test as a "suppressed", instead cause.

This makes it much clearer to find the root cause particularly in case
of errors logged by async code which cause ripple effect test failures.
For example, it took Sam a while to understand the root cause of an
https://git.opendaylight.org/gerrit/#/c/62458/ AclServiceStatefulTest
failures.  With this, it will be much clearer next time:

org.opendaylight.infrautils.testutils.LogCaptureRuleException: Expected
no error log, but: Thread terminated due to uncaught exception:
AsyncDataTreeChangeListenerBase-DataTreeChangeHandler-0
at
org.opendaylight.infrautils.testutils.LogCaptureRule$1.lambda$0(LogCaptureRule.java:76)
at java.util.Optional.ifPresent(Optional.java:159)
(...)
Suppressed: org.awaitility.core.ConditionTimeoutException: Condition
with alias 'TestableListener' didn't complete within 30000 milliseconds
because org.awaitility.core.ConditionFactory.untilAtomic Callable
expected <0> but was <1>.
at
org.awaitility.core.ConditionAwaiter.await(ConditionAwaiter.java:104)
at
(...)
... 16 more
Caused by:
org.opendaylight.yangtools.testutils.mockito.UnstubbedMethodException:
getInterfaceInfoFromConfigDataStore(java.lang.String interfaceName) is
not implemented in testInterfaceManager
at
org.opendaylight.yangtools.testutils.mockito.CallsRealOrExceptionAnswer.answer(CallsRealOrExceptionAnswer.java:127)
at
org.mockito.internal.handler.MockHandlerImpl.handle(MockHandlerImpl.java:93)
(...)

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

Updated git submodules

Project: infrautils master 3054c8ad9dfd6de54c6f002312b2b36f43a42b9d

Fix some minor issues

- Access modifiers
- Make some variables final
- Make some variables static

Change-Id: I10fd0718f88cd7e3c88cfe9d7779d5284853aeed
Signed-off-by: David Suarez <david.suarez.fuentes@gmail.com>

Updated git submodules

Project: integration/packaging master 006760865872fff10995ee33bc0a157063980615

Rename pkg helper logic vars->lib

This module started off containing only logic for getting build
variables, but now it's used as a generic packaging logic library.

Change-Id: Iaf953514a892a39d0fb801eedb41eec0d136f889
Signed-off-by: Daniel Farrell <dfarrell@redhat.com>

Merge "Update yangtools release notes - Nitrogen"

Merge "Add snmp4sdn release note for Nitrogen"

Merge "Update bier release-notes for nitrogen"

Merge "NEMO Release Notes"

Updated git submodules

Project: genius master 12305bcc050b5108b29a2979ecd4a1249e00f448

Remove redundant dependencies in pom

These dependencies are already declared in parent pom.

Change-Id: Ibc5c1b14f95b000caac208a788e2a344f2584933
Signed-off-by: David Suarez <david.suarez.fuentes@gmail.com>

Updated git submodules

Project: sfc master 0575edba31e636c24eed78eccd863ffd0e1c603d

Remove deprecated CheckedFuture

Remove deprecated CheckedFuture.

Change-Id: I32af83be89ccde82e2a510b22ebcd5dc61dd8ffc
Signed-off-by: David Suarez <david.suarez.fuentes@gmail.com>

Updated git submodules

Project: netvirt master 6ca9c509cf44de19707efd9a0ea4cc9b1618f5e8

Bug 8953: Fix exceptions raised due to PNF confused with FIP

Both FIP and discovered PNF IP VrfEntries
use an RD value of external-subnet-id and
thereby the below review resulted in FloatingIP (FIP)
processing being done for PNF incorrectly.
This situation happened vide review:
https://git.opendaylight.org/gerrit/#/c/62957/

In here we fix that confusion, by pushing a new
enumeration to clearly distinguish various type
of prefixes within the prefix-to-interface and use
the same in FibManager (VrfEntryListener) to unambiguously
handle processing for such prefixes (or VrfEntries).

Change-Id: I5143c603d96657e7db0db38a955a4a14f0413e0c
Signed-off-by: Vivekanandan Narasimhan <n.vivekanandan@ericsson.com>

Updated git submodules

Project: netconf master 7e47f4213d56d36fd83c0de1b1e317ee105a6048

Split Restconf implementations (draft02 and RFC) - features

  * remove restconf rfc8040 implemetation from bierman02
  * add features for restconf-nb-rfc8040

Change-Id: I96110d0721c39acdc88332e748f81de07fd5663d
Signed-off-by: Jakub Toth <jakub.toth@pantheon.tech>
Split Restconf implementations (draft02 and RFC) - Application

  * add blueprint + web.xml for restconf-nb-rfc8040

Change-Id: Ifd8ea387ace5580ddd36dde59000b7cd69515272
Signed-off-by: Jakub Toth <jakub.toth@pantheon.tech>
Split Restconf implementations (draft02 and RFC) - providers tests

Change-Id: If8c26aa059508763e4730d69d9b64f594ec9e302
Signed-off-by: Jakub Toth <jakub.toth@pantheon.tech>
Split Restconf implementations (draft02 and RFC) - move providers

Change-Id: Iab075f8d91001a646783f35aa5fe1550d811d01b
Signed-off-by: Jakub Toth <jakub.toth@pantheon.tech>
Split Restconf implementations (draft02 and RFC) - tests

Change-Id: I4e20a07ed96e63a1238ade12c7fc3377853ec211
Signed-off-by: Jakub Toth <jakub.toth@pantheon.tech>
Split Restconf implementations (draft02 and RFC) - move
sal-rest-connector

  * move Patch parts, RestUtil, RestconfSchemaUtil,
    RestconfValidationUtils, IdentityValuesDTO to common module
  * move packages from bierman02 to rfc8040:
    * /restconf/base/services/ ->
        /restconf/nb/rfc8040/services/simple
    * /restconf/common/wrapper/services ->
        /restconf/nb/rfc8040/services/wrapper
    * /restconf/common/references/ ->
        /restconf/nb/rfc8040/references
    * /restconf/handlers -> /restconf/nb/rfc8040/handlers
    * /restconf/parser -> /restconf/nb/rfc8040/utils/parser
    * /restconf/restful -> /restconf/nb/rfc8040/rests
    * /restconf/utils -> /restconf/nb/rfc8040/utils

Change-Id: I8259d69ab2378a25ddcf41d0b6f14dd65afc2507
Signed-off-by: Jakub Toth <jakub.toth@pantheon.tech>
Split Restconf implementations (draft02 and RFC) - move restconf
errors to common part of restconf

Change-Id: If8786352d59e013aa8da5b22790ee125e23b771b
Signed-off-by: Jakub Toth <jakub.toth@pantheon.tech>
Split Restconf implementations (draft02 and RFC) - base api

  * create common module for both implemetations
    * move common classes
  * move base api services to restconf-nb-rfc8040

Change-Id: I31d084c5d19da515337f79562bafa520c57a087d
Signed-off-by: Jakub Toth <jakub.toth@pantheon.tech>
Split Restconf implementations (draft02 and RFC) - Rfc8040

  *move Rfc8040 to the new module restconf-nb-rfc8040

Change-Id: Ica52f23791e8326f20f8449d0e39273b7db8f5c5
Signed-off-by: Jakub Toth <jakub.toth@pantheon.tech>
Split Restconf implementations (draft02 and RFC) - Prepare modules

  * rename sal-rest-connector module to restconf-nb-bierman02
  * prepared new module restconf-nb-rfc8040

Change-Id: I401a7fc595251dfd2ae08074096a4a16707c6527
Signed-off-by: Jakub Toth <jakub.toth@pantheon.tech>

Updated git submodules

Project: integration/test master 973d719e695b96ee223e8e5e0fe2c574186b0002

Convert Sleep to WUKS in lispflowmapping CSIT

Part 1, 030_RPC_Datatypes.robot

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

Update documents for Cardinal Nitrogen

Change-Id: Ie2029fc1e2f5c92a40d77e7f61e27cccabea6824
Signed-off-by: Subodh <subodh.roy@tcs.com>

Updated git submodules

Project: integration/test master 7dcd196f3d16b32190ac2246cff599e6117b4e27

Fix typo in OF test plan

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