Update .gitreview for stable/nitrogen

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

Updated git submodules

Project: coe master d89fc18468bd6f782c58c706c9dc2bc147378b71

Fix Add node json string

Fix the Add node function json string

Change-Id: I82024a21a12a73899aa6cf570078b2ebe02a788f
Signed-off-by: Mohamed ElSerngawy <m.elserngawy@gmail.com>

Updated git submodules

Project: genius master 7dd573c4707dfe4ff10a31303fd9a1e43d88e556

FutureRpcResults

This is https://git.opendaylight.org/gerrit/#/c/53998/ generalized!

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

Updated git submodules

Project: releng/builder master 21ccb05794a5743bc749d55c37b062984e37f7d3

Merge "Exclude mininet-2.6 for Ubuntu 14.04"

Exclude mininet-2.6 for Ubuntu 14.04

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

Updated git submodules

Project: releng/builder master 7a142a53d81e6dd1c7f1192e9cb3813409529dfa

Fix taglist getting the incorrect tags

We need to escape the $ so that the git rev-parse runs inside of the
submodule rather than before entering the submodule causing current
taglists to pull autorelease.

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

Updated git submodules

Project: releng/builder master de8f9f1afd3111dd70d68ba85341b0cdd6043bc3

Handle yangtools nitrogen on non-standard branch

YangTools uses v1.2.x as it's nitrogen contribution so we need to handle
the difference in branch naming.

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

Updated git submodules

Project: sfc master f369718d1af29e16f6fb32485ae5db6706cf5fc0

Fix findbugs/checkstyle/sonar violations in sfc-ovs

- Field isn't final but should be
- Method invokes inefficient new String(String) constructor
- Load of known null value
- Parameter must be non-null but is marked as nullable
- Exception handlers should preserve the original exception
- Utility classes should not have public constructors
- String literals should not be duplicated
- String.valueOf() should not be appended to a String
- Local variable and method parameter names should comply with a naming convention
- Collapsible "if" statements should be merged
- Method parameters, caught exceptions and foreach variables should not be reassigned

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

Merge "Remove archives/ path from PATCH_URL"

Updated git submodules

Project: sfc master a700674542a48604833de9a23b72bb76b1cecc5a

Refactor SbRestAbstractDataListener

Refactored SbRestAbstractDataListener and derived classes to pass
parameters into the ctor instead of via setters. This makes it cleaner
so class fields can be final and avoids null checks elsewhere.

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

Updated git submodules

Project: sfc master 97da7fda3a36f9c01f617d46ab87270fc324546f

Add an API to handle ServiceNode entities

Add an API to handle ServiceNode entities. This will be firstly used by
the SFC CLI.

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

Remove archives/ path from PATCH_URL

The patches file is now in the root of the logs directory.

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

Updated git submodules

Project: releng/builder master 0f8d771a26e6c9341edca8f844db48a6264be595

Merge "Trigger RPM/deb builds when changing pkg logic"

Trigger RPM/deb builds when changing pkg logic

Adds triggers to test RPM/deb builds when files related to package
building change. Also adds Gerrit keywords to trigger same builds.

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

Updated git submodules

Project: integration/test master 40659a054760ee11b6a8b188c03bc10fc3ad9743

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 JJB patch in releng/builder:
  https://git.opendaylight.org/gerrit/#/c/63427/

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

Updated git submodules

Project: sfc master b227575c7fe06838831255e0e3c445a3f44b0540

Refactor SbRestAbstractTask

- Change fields to private
- Remove setters and pass params to ctor so field can be final and
  avoid null checks elsewhere
- Expose a method to add to the restUriList

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

Updated git submodules

Project: netvirt master e2590a94c3d310b8eaeb99db3f8b9bf7e19c1ac9

Bug 8953 - IllegalArgumentException: vrfEntry is missing mandatory
descendant origin

Change-Id: I0978d55f09d0e9b31aa2b74e5cfac68a757c29da
Signed-off-by: eupakir <kiran.n.upadhyaya@ericsson.com>

Updated git submodules

Project: infrautils master 73355081d90192386bf6decf9aa8c30efa1614c7

Drop StringUtil

It isn’t used anywhere outside infrautils, and doesn’t bring anything
useful to the table.

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

Updated git submodules

Project: infrautils master 98d2a78fecea7ff25866b2aa08c538529b40d513

Pull Model For DiagStatus

Change-Id: I22c1b09f11de41bb10aa3bc91ec778f0ebdce69f
Signed-off-by: Faseela K <faseela.k@ericsson.com>

Updated git submodules

Project: sfc master 44d5b18c2623b9e2a74130969fc4951c5421641f

Fix findbugs/sonar violations in sfc-sb-rest

- Method ignores exceptional return value
- Consider using Locale parameterized version of invoked method
- Field isn't final but should be (malicious code vulnerability)
- Method uses the same code for two switch clauses
- Redundant nullcheck of value known to be non-null
- Unread public/protected field  (SbRestAbstractTask)
- Field not initialized in constructor but dereferenced without null check

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

Updated git submodules

Project: netvirt master a6356e962973f7ea3c47cb15a443506c0f388e6e

lower log levels for non-errors

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

Updated git submodules

Project: netvirt master 3cf0fc60774d9bdd73b865cde2f6fdb85aed6738

ClearBgpCli reads from socket to send/receive from bgpd
(some previous commit modifed to read from session parameters)

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

Updated git submodules

Project: netconf master 383ff321e0ae4dcfc45a6dc682ede45f49232d9e

Minor cleanup of blueprint config

The path for private file was left here due to which the netconf module
looks for the private key file and shows error in the log which is a false alarm.
Cleaning up the configuration

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

Updated git submodules

Project: infrautils master ffdb00f87a200c403c147bfd905e97252007c7be

Enable FindBugs for diagstatus

it found no real bugs, but two very minor clean ups; but should help to
ensure that no new bugs (detectable by FindBugs) get introduced later

and remove a dependency we had to yangtools - infrautils should not (and
FindBugs did not like that equals() implementation - EQ_UNUSUAL)

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

Updated git submodules

Project: genius master 460418a1da8d2a2fbbf546724340e3309d4480bd

Add @PreDestroy (and final) to AbstractDataTreeChangeListener's close

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

Updated git submodules

Project: infrautils master 02c83722e55a17aeefc29026b4ab56abafd6bb63

Minor: Remove AutoCloseable from DiagStatusService API

This pattern is wrong - it's the implementation that is closeable. The
service API is not - as a consumer of interface DiagStatusService you do
not care about this - you'll never close() it.

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

Updated git submodules

Project: infrautils master f14c7bb3735970613ca0c50c8d6951c7c64f5a0f

New ready order FunctionalityReady with *Notifier, to help order startup

This came out of some thought I've given
https://git.opendaylight.org/gerrit/#/c/61323/

This is just the API; if there are no strong objections to this
approach, I'd like to get this merged in, and the propose the actual
FunctionalityReadyNotifierImpl body in a new follow-up change.

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

Merge changes

Merge "Link to Nitrogen release notes Wiki page"

Updated git submodules

Project: integration/test master c79e328375ccaa0ebdd7faf5943f48a87aba0af9

Add timeout/retry to L2GW CSIT TC05

Add timeout and retry to Verify Ovs Tunnel in test case
TC05. This was failing in 3 node tests, as it takes a
little longer for config to be applied in in cluster.

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

Updated git submodules

Project: infrautils master 71b07293cd421d4750f4548970ddbdfa33f1ad20

Remove a "throws NullPointerException" which Sonar flags as Major

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

Updated git submodules

Project: infrautils master f0c7d0739f238efa0b76aa5aeec28818d1833f91

Drop CompareUtil

It isn’t used anywhere outside infrautils, and can be replaced by
Objects.

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

Updated git submodules

Project: integration/test master 3ce1ba4af1e2bac4d1adb177f48e2c43f49376d9

Fix for Genius CSIT failure on Oxygen

Description : This patch has keyword Get Tunnel modifed with
code to get tunnel interface name between dpns

Change-Id: I6bac03d6d3c244f4ef40b2d5ad1e8f0f046eeeaf
Signed-off-by: vidya sathyanarayanachar <s.vidya990@gmail.com>

Updated git submodules

Project: integration/test master 005966b19cfd55ee543ec68b0e46170c0718d646

Bug 8999: Add entity isolation netconf suite

Added a suite based on entity.robot which
will verify that a 3 node netconf cluster recovers after
isolation and rejoin of the entity owner.

Change-Id: I2a5510338dfa323c71d3ce4820036b4bfe0342a2
Signed-off-by: Richard Hill <rhill@frinx.io>
Signed-off-by: Vratko Polak <vrpolak@cisco.com>

Updated git submodules

Project: genius master 26c605f8f1c9f87d5e3041f9494edbebdae1a208

Add gpe option to itm-config.xml

https://git.opendaylight.org/gerrit/60769 missed adding
the option into the config.xml.

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

Link to Nitrogen release notes Wiki page

and fixed missing dot in infrautils release notes.

A more detailed lower level technical What's New
update will be provided in a presentation at the
Developer Design Forum in October.

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

Updated git submodules

Project: sfc master 08e38497dc82ebfdc7041b2dc50a261f8916e842

BUG9093 Also attempt to use plain Vxlan tunnel

Give preference to VxlanGpe tunnel type in logical SFF but use standard
Vxlan tunnel if not present. This is because GPE extension might be
globally enabled as a Genius ITM configuration option so that specific
VxlanGpe tunnel types are not used.

Change-Id: I2540ada637b67276977601a6b2dfa418defe872a
Signed-off-by: Jaime Caamaño Ruiz <jaime.caamano.ruiz@ericsson.com>
(cherry picked from commit c4907b6221e1c3919129eb72d920d85f0d2567e1)

Updated git submodules

Project: genius master 6fc2df52686f392cd1ae950938a44dc41c71c692

Improve the Genius Listeners Framework

Improve the Listeners Framework created in [1] to ease registration
and un-registration processes for entities.

[1] https://git.opendaylight.org/gerrit/#/c/59177/

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

Updated git submodules

Project: netconf master 202b4d062c1064fc061009d5725b8a102fcc13c4

RPC for netconf node addition. Supports encrypt option for password encryption.

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

Updated git submodules

Project: infrautils master 580b57b7f454908e3cf7dae914459201ebc3999e

Add odl-guava-22 dependencies where necessary

Features which use Guava should use the odl-guava-22 feature instead
of bundling the library themselves.

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

Updated git submodules

Project: netvirt master 6a6dd6e51234a177665a8590ab698f215100aabc

bgpmanager: change API of bgpmanager to add VRF IPv4 or IPv6

This change adds a change in IBgpManager API, since it adds afi and safi
parameters to VRF creation and deletion. This information is propagated
to BGP so as to create and delete VPN context tables in function of the
AFI and SAFI settings.
Instead of passing a layerType value, an AddressFamily type is used.
AddressFamily is mapped to internal afi/safi values.
The values are appended or removed to the VRF objects of BGP Manager.
The VRF object is then written to the config DS.
Subsequently, the Quagga will be called upon the various changes in the VRF object.
If the object is added or updated, the Quagga will be called with the appropriate
afi/safi values. If the object is removed, all the afi/safi values are removed from
the Quagga. If the object is added, the afi/safi values are added to the quagga.

Change-Id: Ife754ce561f69dbb67fcdf74308b5153cbde53e6
Signed-off-by: Valentina Krasnobaeva <valentina.krasnobaeva@6wind.com>
Signed-off-by: Noel de Prandieres <prandieres@6wind.com>
Signed-off-by: Philippe Guibert <philippe.guibert@6wind.com>
neutronvpn: create ipv4 or ipv6 context

Add ipv4Family and ipv6Family attributes to VPN instance. Add
IpVersionChoice class, which allows to set values for these attributes
in dependency of Ethertype of added/removed to VPN subnet.

Change-Id: If5ea1a0a29d31388d7898b9e6e53bdc4aecb1cbe
Signed-off-by: Valentina Krasnobaeva <valentina.krasnobaeva@6wind.com>
Signed-off-by: Philippe Guibert <philippe.guibert@6wind.com>
Signed-off-by: Noel de Prandieres <prandieres@6wind.com>

Update release note index files

Include release notes of participating nitrogen projects only

Change-Id: If12413172c5f06faa09c22827d87c07636434039
Signed-off-by: Kit Lou <klou.external@gmail.com>

Merge "Update mdsal release notes for Nitrogen"

Merge "Update eman release notes for Nitrogen"

Merge changes

Merge "Update Distribution documents for Nitrogen"

Merge "Add of-config Nitrogen release notes"

Update eman release notes for Nitrogen

Change-Id: I03d518efe396e5773310cbc045f360dbb6b5bff2
Signed-off-by: fsandoval <francisrsandoval@gmail.com>

Merge "Update snmp release notes for Nitrogen"

Update mdsal release notes for Nitrogen

- Links updated
- Reformatted to 80 columns
- Reworded EOL section

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

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 snmp release notes for Nitrogen

Change-Id: Ib9a6e5acd941f0ceae7d0b0b7599931b3324b814
Signed-off-by: fsandoval <francisrsandoval@gmail.com>

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>