Update git submodules

* Update docs/submodules/infrautils from branch 'master'
  to 7c6fe03471c36a69e46277220faf359d545279dd
  - add metrics-api to artifacts POM

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

Update git submodules

* Update docs/submodules/infrautils from branch 'master'
  to 12a7ea4638bd36636d2c60afa3cd87c0a7a0060e
  - fix parent POM infrautils:findbugs version

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

Update git submodules

* Update docs/submodules/netconf from branch 'master'
  to 1aaec97aa24f5dfe62fd3e6523e651db883ab51f
  - Remove dependency on controller config-util

    netconf uses a few classes from config-util, including
    DocumentedException, XmlElement, XmlUtil, Capability etc
    so copy them to netconf-api. config-util will be removed
    later.

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

Update git submodules

* Update docs/submodules/aaa from branch 'master'
  to d0d869008d7c7bb8d08a77051a660fb087134a6b
  - Add Karaf build profile

    Project local Karaf distros are handy for devs to test their work
    however is unneeded by autorelease builds and should not be released
    as part of the Simultanious Release. Add a profile that is active by
    default so that default behaviour is unchanged however allows the
    autorelease project to disable building this module.

    Change-Id: If26f62fd722bedce8d39d3dfe673064441fd1d36
Signed-off-by: Jamo Luhrsen <jluhrsen@redhat.com>
    (cherry picked from commit 320971a7892e4540bc5d253cf9a2f8117b61e2ce)

Update git submodules

* Update docs/submodules/coe from branch 'master'
  to 763b146f863288a46fdb24c2e8ac2b0623db168d
  - Remove obsolete Maven Site configuration

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

Update git submodules

* Update docs/submodules/ovsdb from branch 'master'
  to 3d808b6b636b063adb17d2567b9d5b13c9ff9e54
  - Ensure bundles are present in a single feature

    The following bundles are currently included in multiple features:
    * southbound-api
    * utils.config
    * utils.mdsal-utils
    * utils.southbound-utils

    This patch ensures that the relevant feature is used wherever
    appropriate, so that each of these bundles is only included in a
    single feature.

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

Update git submodules

* Update docs/submodules/genius from branch 'master'
  to a40a3ffa4e556b80e5873a5c9b413f9c09eb5842
  - AlivenessMonitorUtils: wrap top-level in transactions

    This uses transactions throughout based on top-level operations,
    propagating the transaction through the call chain to ensure
    consistent views of the MD-SAL.

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

Update git submodules

* Update docs/submodules/genius from branch 'master'
  to 062a1a1238635c67d2039f71f641a9fc453b7470
  - Remove MDSALDataStoreUtils

    Since there are very few users left, remove the class entirely
    (replacing it with SingleTransactionDataBroker).

    This needs Ia012cc825a9eb7768de91c1882f76127d44c3595 to be merged
    first.

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

Update git submodules

* Update docs/submodules/infrautils from branch 'master'
  to 48ce6423820376e39a7de5b4244d26fc20ace16f
  - Enhancing logs for invokeRemoteJMXOperation

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

Merge "Replace RC0 with “code freeze” milestone"

Update git submodules

* Update docs/submodules/odlparent from branch 'master'
  to 9608cc9dabebd1b14984c9c7750601518416f60b
  - Clean up Apache Commons features

    We were missing the dependency-tracking files, and commons-text
    depends on commons-lang3 which we need to represent in the feature.

    Change-Id: I0ef952688f9589f2385ef919bed2e3d1410d831d
Signed-off-by: Stephen Kitt <skitt@redhat.com>
  - Update dependency-check

    The jackson dependencies have been replaced by jackson-bom.

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

Update git submodules

* Update docs/submodules/bgpcep from branch 'master'
  to af558543ebb1f50391661059a4016f3a305bccc8
  - BGPCEP-578: Add additional path config

    to afis/safis peer group

    Change-Id: I2a4776d7566d79fd15c799c0c3d7613c01638e9e
Signed-off-by: Claudio D. Gasparini <claudio.gasparini@pantheon.tech>

Update git submodules

* Update docs/submodules/bgpcep from branch 'master'
  to 115500137a782a71191cbf7b2d2fa83fd593e235
  - Fix missing return under AbstractBestPathSelector

    Change-Id: I0156a2be5a7772ecdbd55de5f5308092656b38eb
Signed-off-by: Claudio D. Gasparini <claudio.gasparini@pantheon.tech>

Update git submodules

* Update docs/submodules/odlparent from branch 'master'
  to f4d7935494d0e9952f57f6e0c520928931c9d21a
  - remove ovsdb config from custom.properties

    Change-Id: Ic3f4762c8f92ebcc958a80cec01fbda5749544af
    Issue-ID: NETVIRT-434
    Issue-ID: ODLPARENT-34
Signed-off-by: Sam Hague <shague@redhat.com>

Update git submodules

* Update docs/submodules/netconf from branch 'master'
  to 34187b5b4efeb1232d80e285857449dbc1c9ada2
  - Remove the config-netconf-connector

    The config system is being removed in Flourine.

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

Update git submodules

* Update docs/submodules/genius from branch 'master'
  to 52dde879f951f841321bcf26cfdd845b33ac249f
  - NETVIRT-430 Ping responder on tenant network and FIP-FIP traffic between
    vm in same n/w in same compute is not working in stateful SG mode.

    Added Ct Clear action

    Depends-on: Change-Id: I163ed99d5af8d3643749ac2a278f5c8170d7ae29
    Change-Id: If86203e73a5ec516ff487f3f5c33fbd7dbfd12e1
Signed-off-by: Aswin Suryanarayanan <asuryana@redhat.com>

Update git submodules

* Update docs/submodules/openflowplugin from branch 'master'
  to 68675ac5113a2a737e72182806b72c2717155d77
  - Merge "OPNFLWPLUG-993 ct_clear is not supported in Openflowplugin"
  - OPNFLWPLUG-993 ct_clear is not supported in Openflowplugin

    Added ct_clear Action in nicira extension

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

Update git submodules

* Update docs/submodules/netconf from branch 'master'
  to 1c2c99dd2be77e78114c5b0eff0275734e329ec2
  - Merge "Remove unused controller config dependencies"
  - Remove unused controller config dependencies

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

Update git submodules

* Update docs/submodules/genius from branch 'master'
  to 2215da0e7fb9b295e9152091acfa5ae180524c9d
  - add requireNonNull to SingleTransactionDataBroker constructor

    thought of this while seeing this which is actually broken:

      @Inject DataBroker dataBroker;
      S.T.D.B. db = new SingleTransactionDataBroker(dataBroker);

    which cannot work with Dependency Injection (Guice or otherwise), and
    it's better to have the null check here in the constructor already, not
    fail with NPE later when one of the (non-static) methods of it is used.

    see Change-Id: I63d0a93da481003b701c5a1843cece7d50cf51d1

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

Update git submodules

* Update docs/submodules/genius from branch 'master'
  to 66c6fb0cdddcf00afa79c6429f2fb347dbc07e70
  - fix broken build; recent *Transaction change which removed commit()

    The {@link ReadWriteTransaction#commit()} in JavaDoc lead to an "error:
    reference not found" in maven-javadoc-plugin:3.0.0:jar (attach-javadocs)
    @ mdsalutil-api during the build.

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

Update git submodules

* Update docs/submodules/genius from branch 'master'
  to a2cfe2fe1da5ef82c892e6b09b42d972cfa94e1a
  - Remove AlivenessMonitor::read

    This can be entirely replaced by SingleTransactionDataBroker.

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

Update git submodules

* Update docs/submodules/sfc from branch 'master'
  to 787a3e1ba5918ef98299f444cb7f285ee2bf71b8
  - Remove sfc-ui module (DLUX)

    SFC's User Interface (module sfc-ui) is based on DLUX project which is
    not properly maintained, so it's been decided to removed this dependency
    from all Opendaylight projects.

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

Update git submodules

* Update docs/submodules/genius from branch 'master'
  to 0861933e2c678886dfe4d922f6b3089ab0ec53d0
  - Removed unused ACL constants

    Ensured there are no references in netvirt for these constants.

    Change-Id: I4372e2cd7733b469d0dee6f0ba0afc551eff0af5
Signed-off-by: Somashekar Byrappa <somashekar.b@altencalsoftlabs.com>

Update git submodules

* Update docs/submodules/bgpcep from branch 'master'
  to c98b6ebaaca98050c7e2bebe4709e918b15fba70
  - BGPCEP-740: Fix logic for MED comparison in BGP best path selection

    New better path should be selected only if new MED is lower than
    actual best path MED.

    Change-Id: I8bd07dc8b42e575cebcc4e1ea4b55d9c2c842f42
Signed-off-by: Ajay Lele <ajayslele@gmail.com>
Signed-off-by: Claudio D. Gasparini <claudio.gasparini@pantheon.tech>

Update git submodules

* Update docs/submodules/openflowplugin from branch 'master'
  to cafd5143c8ab9dda582964b6a30af45b6bb54e56
  - Add doc for bundle reconciliation usage

    Add documentation for bundle reconciliation usage

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

Update git submodules

* Update docs/submodules/openflowplugin from branch 'master'
  to e8ef2c36f8845a96af840377780f500f3914ec24
  - Update group add mod spec to reflect the correct details

    Update the spec to reflect the latest changes

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

Update git submodules

* Update docs/submodules/odlparent from branch 'master'
  to 160352a680826120a0c5def8ab3ef8007721faed
  - ODLPARENT-137: restore the opendaylight prompt

    In Karaf 4.1 and later, it turns out that the prompt defined in the
    branding is overridden by the shell initialisation script,
    etc/shell.init.script, which defines its own prompt. This patch adds
    our own version of that script which disables the override.

    Thanks to Guillaume Nodet for figuring out where the override was
    coming from, and to Michael Vorburger for the initial investigation!
    See https://issues.apache.org/jira/browse/KARAF-5570 for details.

    Change-Id: I22410cb0425ab5b8a3f6e7c410fbb76587f9f973
Signed-off-by: Stephen Kitt <skitt@redhat.com>
    (cherry picked from commit 732b0dc09d15b3b001fa636783dc4cdafc224b40)

Update git submodules

* Update docs/submodules/odlparent from branch 'master'
  to 64c4148935723f6688e3d7652fdb9ad3f461caa9
  - ODLPARENT-146: disable NP_NONNULL_PARAM_VIOLATION

    Guava 23.6 switched from @Nullable to @NullableDecl, which causes
    false positives in FindBugs’ ``NP_NONNULL_PARAM_VIOLATION`` rule;
    we’re disabling the rule for now.

    Change-Id: I135c90930ff7e8a2203da46e3106ed69c570f1c1
Signed-off-by: Stephen Kitt <skitt@redhat.com>
    (cherry picked from commit 82eee8769f07ec730cfbfa45dfa1ab5652bb8f07)

Update git submodules

* Update docs/submodules/netconf from branch 'master'
  to 81249f9abe3c66a4ae0a819ac3ac821565950265
  - Do not needlessly bridge promises

    Channel has an alternative writeAndFlush() method, which takes
    a promise to complete. Using it saves allocation of two objects
    and required listener synchronization.

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

Update git submodules

* Update docs/submodules/integration/packaging from branch 'master'
  to 2203f7dff90636b0d11528ae4ee56ffeda9915aa
  - Add Fluorine pkg repo config files

    Change-Id: I2f3dfa746260b2eaf854fa8b625f0169a8580949
Signed-off-by: Manuel Buil <mbuil@suse.com>

Update git submodules

* Update docs/submodules/bgpcep from branch 'master'
  to 542dfab18ce3a8af9535870ece0faf611ac5f7d2
  - BGPCEP-577: Neighbor’s local AS configurable

    Change-Id: I4dab50d048c2c61900251959d06c909c55722df5
Signed-off-by: Claudio D. Gasparini <claudio.gasparini@pantheon.tech>

Update git submodules

* Update docs/submodules/odlparent from branch 'master'
  to 34386fdfe576ca43ff12057bb907e2aa83de76c6
  - Add 3.1.0 release notes

    Change-Id: Ia77f26ac993219bcf7b90dfbad6cd254ea07d89d
Signed-off-by: Stephen Kitt <skitt@redhat.com>
    (cherry picked from commit 34b60e9b2bb82afd7c11e0609c85f61f562d11c5)

Update git submodules

* Update docs/submodules/odlparent from branch 'master'
  to 53b8819d5821e7baebac043f10f54e9fbec53a95
  - Add 3.0.3 release notes

    Change-Id: Ia19badf04234d5a03ef2e7d18ffe4db835b7a0be
Signed-off-by: Stephen Kitt <skitt@redhat.com>
    (cherry picked from commit 07de5fbcfc958ca416c1f2317ac4ed58050f6a94)

Replace RC0 with “code freeze” milestone

Given that the document related to the release schedule where
“code freeze” is the term used, the RC0 (release candidate 0)
was replaced with “code freeze” milestone.
ODL RC0 definition appears in /docs/release-process/release-plan.rst
and should be added to this document in case we choose
to stick with RC0.

Change-Id: I0dc30cc97e03cbc99ea22d251109fcc68385046d
Signed-off-by: Ariel Adam <aadam@redhat.com>

Update git submodules

* Update docs/submodules/odlparent from branch 'master'
  to dd1a46272b305c853e16f29344115f51f4e90476
  - Use SpotBugs 3.1.2, and NP_NONNULL_PARAM_VIOLATION

    This patch upgrades the SpotBugs plugin to 3.1.3 and the engine to
    3.1.2. The changelog is
    https://github.com/spotbugs/spotbugs/blob/3.1.2/CHANGELOG.md

    SpotBugs 3.1.2 supports the nullable annotations in Checker Framework
    (see
    https://github.com/spotbugs/spotbugs/commit/dd436ff56abf0273a080ad228fcf5abd8ee3a097
    for the patch), so we can re-enable NP_NONNULL_PARAM_VIOLATION when
    using SpotBugs. This involves using a separate configuration, so this
    patch splits everything out, including the report directory for
    projects which run both FindBugs and SpotBugs.

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

Update git submodules

* Update docs/submodules/genius from branch 'master'
  to 007041dfcd53de75cc0152bbc07dc1ef450c0628
  - Add transaction variant of getChildInterfaces

    In preparation for removing the current implementation of
    IInterfaceManager::getChildInterfaces, this patch adds a variant
    accepting a transaction (along with a few other related methods).

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

Update git submodules

* Update docs/submodules/netconf from branch 'master'
  to a3022c29beea055ec8e7cacaa30dd64d5a80887c
  - Merge "NETCONF-526 : edit-config payload default-operation needs to be set to default-value"
  - NETCONF-526 : edit-config payload default-operation needs to be set to default-value

    For the PUT/POST calls right now default-operation tag in the edit-config is set to
    "none". As per RFC with this value devices can reject a config operation when
    elements present in the request whose structure is not already present in the tree.
    For example a nested parent-child structure where parent structure does not yet exist
    yet in the tree may result in an error based on implementation.
    We should  leave this field out ( which will default to merge) and let the device make
    the right decision.

    Change-Id: I8b85ef072785381897a19724dc94003b8f315bd4
Signed-off-by: Balaji Varadaraju <bvaradar@luminanetworks.com>

Update git submodules

* Update docs/submodules/genius from branch 'master'
  to c852b8a17853fc0bad1d77cf642ceb3a1251fc9b
  - increase max time out in AbstractTestableListenerTest from 50ms to 500ms

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

Update git submodules

* Update docs/submodules/integration/packaging from branch 'master'
  to d2899bbe305a7dc856aa5eb965b9edad64f0f045
  - Update expired pkg unit test example URLs

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

Merge "BGPCEP-781: Add path-id to all AFI/SAFI guides"

Merge "Add neighbor local address of TCP connection."

Update git submodules

* Update docs/submodules/genius from branch 'master'
  to 3c4ad5fbe103a0ffbd4fb754fb8bde03aa899495
  - Move tepsNotHosted from Config DS to Oper DS

    As tepsNotHosted list was part of transport-zones container in itm.yang
    and stored into config DS earlier. Since, it gets populated from
    southbound only and will be available again on switch/ODL restart,
    even if the switch goes away, we do not want to track these information
    unnecessarily in the config DS. So, it is better to keep this data into
    OperDS instead of config DS.

    Change-Id: I731dfdb64ea3dc589567f4abeac3457ca0cbe395
Signed-off-by: Tarun Thakur <tarun.t@altencalsoftlabs.com>

Update git submodules

* Update docs/submodules/bgpcep from branch 'master'
  to bc62bc6768bc8f91f929f61d1736331eef0f32b3
  - BGPCEP-576: Neighbor’s local address configurable

    Change-Id: Ic4aaf8dbb408e37a0df40e320793dc1330879b06
Signed-off-by: Claudio D. Gasparini <claudio.gasparini@pantheon.tech>

Add neighbor local address of TCP connection.

Change-Id: I3bcc1cf32694c8106dd0cdeb629ccb1f5248af73
Signed-off-by: Claudio D. Gasparini <claudio.gasparini@pantheon.tech>

BGPCEP-781: Add path-id to all AFI/SAFI guides

Change-Id: Ie2e4cd08659ebba4f298f9ea59fef898908b05be
Signed-off-by: Claudio D. Gasparini <claudio.gasparini@pantheon.tech>

Update git submodules

* Update docs/submodules/bgpcep from branch 'master'
  to b009fc8a04c67b96818a624fdd13262e1c5c9ad5
  - BGPCEP-782: Fix peer-id

    by using the one send by peer.

    Change-Id: Ic18e7fac972b08dcd77a2dbaad7482295bdcd7ae
Signed-off-by: Claudio D. Gasparini <claudio.gasparini@pantheon.tech>

Update git submodules

* Update docs/submodules/bgpcep from branch 'master'
  to 1e4fd308dfc75f78ccee6887f786f091e4621fb5
  - Rework match role condition.

    Make condition support from peer role and
    to peer role evaluation at once.
    Improving performance.

    Change-Id: I2e3c69e82e17d651aca962f478901ec4aed63b88
Signed-off-by: Claudio D. Gasparini <claudio.gasparini@pantheon.tech>

Update git submodules

* Update docs/submodules/bgpcep from branch 'master'
  to 43f514bdf5ec35e8dd1389db38d844b7d70a98d1
  - BGPCEP-781: Provide Add Path support for all AFI/SAFI

    Change-Id: I0fefe5c28a8b9d4f2f0be5cbe966233b73c7081a
Signed-off-by: Claudio D. Gasparini <claudio.gasparini@pantheon.tech>

Update git submodules

* Update docs/submodules/netconf from branch 'master'
  to 4b5ea5b9dbcddd02b69f093ee22d9e7420b403d3
  - Remove unneeded lambda casts

    These callsites do not require disambiguation, remove unneeded
    casts, lowering explicit dependencies.

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

Update git submodules

* Update docs/submodules/sfc from branch 'master'
  to bf818465a0f83a244269f60bac2c110073a18196
  - Remove sal-binding-config dependency

    It's CSS-related and not needed and will be going away.

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

Update git submodules

* Update docs/submodules/bgpcep from branch 'master'
  to fbde13a4b2cfaabba5614f08245b7ea5298ed02e
  - BGPCEP-779: Fix Effective rib writer

    empty table creation. Create table route
    container only if its already not present.

    Change-Id: Ia621e8aa8058eeb14b06bc99956efe1e7c6a9f5b
Signed-off-by: Claudio D. Gasparini <claudio.gasparini@pantheon.tech>

Update git submodules

* Update docs/submodules/openflowplugin from branch 'master'
  to 68b10d99d0d9a680d96ba0160119303b149f7155
  - Fix test identities

    Identities are mapped to interfaces, not abstract classes. Fix
    manual test identities.

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

Update git submodules

* Update docs/submodules/sfc from branch 'master'
  to dbc5472bd58e25afb3d43ba00d877b7ef2bd917d
  - Migrate listener's deprecated methods to use the new ones

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

Update git submodules

* Update docs/submodules/bgpcep from branch 'master'
  to 58b94f4f93cf535d6381a4428464ee0e9aec67e7
  - Fix test identity

    Identities are mapped to interfaces, not classes.

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

Update git submodules

* Update docs/submodules/bgpcep from branch 'master'
  to c5c1e498b7edd9783858b4177b46c14ab48abaa5
  - Remove config system remnants

    The config system is being removed in Flourine so remove
    the unused config system dependencies.

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

Update git submodules

* Update docs/submodules/netconf from branch 'master'
  to 2c5eea937aa51612acd6e444b3e3359598781b38
  - Merge "Fix findbugs violations in netconf"
  - Fix findbugs violations in netconf

    - Field is a mutable collection which should be package protected
    - Unchecked/unconfirmed cast of return value from method
    - May expose internal representation by returning reference to mutable object
    - May expose internal representation by incorporating reference to mutable object
    - Field should be package protected
    - Parameter must be non-null but is marked as nullable
    - Nullcheck of value previously dereferenced
    - Inconsistent synchronization
    - Method ignores return value
    - Boxing/unboxing to parse a primitive
    - Boxed value is unboxed and then immediately reboxed
    - Unchecked/unconfirmed cast
    - Class implements same interface as superclass
    - Switch statement found where one case falls through to the next case
    - Field not initialized in constructor but dereferenced without null check
    - Reliance on default encoding
    - Private method is never called
    - Method invokes toString() method on a String
    - Possible null pointer dereference
    - Consider using Locale parameterized version of invoked method
    - Field not guarded against concurrent access
    - Method does not release lock on all exception paths
    - Inefficient use of keySet iterator instead of entrySet iterator
    - Load of known null value
    - Redundant nullcheck of value known to be non-null
    - Non-transient non-serializable instance field in serializable class
    - Class is Externalizable but doesn't define a void constructor
    - Class is Serializable, but doesn't define serialVersionUID
    - Class defines field that masks a superclass field
    - Should be a static inner class
    - Repeated conditional tests
    - Format string should use %n rather than \n
    - Method may fail to close stream
    - Unconditional wait
    - Wait not in loop
    - Method concatenates strings using + in a loop
    - Dead store to local variable
    - Possible null pointer dereference due to return value of called method
    - Useless object created

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

Update git submodules

* Update docs/submodules/netconf from branch 'master'
  to bcdbb29c7492c47ecd761d91447d9719c0a21073
  - Remove checkstyle config from child poms

    The checkstyle config is now in the parent pom.

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

Update git submodules

* Update docs/submodules/sfc from branch 'master'
  to 63aba8159175a4557955a3991e41fdaa4df39d28
  - Migrate listener's deprecated methods to use the new ones

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

Update git submodules

* Update docs/submodules/aaa from branch 'master'
  to 445ffed6cd99557dddcc3f8b457fd0051471d83c
  - AAA-143:  Remove jackson dependencies

    Other projects need to pull in jackson runtime dependencies themselves
    instead of depending on AAA.  AAA does not utilize Jackson anymore
    period.

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

Update git submodules

* Update docs/submodules/sfc from branch 'master'
  to fe8f630936bda4a95846ab073555c340c27043bb
  - Use odl-jackson-2.8

    We should use the odl-jackson-2.8 feature to fulfill our dependencies
    on Jackson.

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

BGPCEP-564: RR Multiple Cluster IDs

Change-Id: Iaa43c4a03515f8f4e6d374558b68936d0435789a
Signed-off-by: Claudio D. Gasparini <claudio.gasparini@pantheon.tech>

Update git submodules

* Update docs/submodules/bgpcep from branch 'master'
  to 7352d02a1a475a13a5e2de7174b1d867be4c5d5e
  - BGPCEP-564: Support multiple cluster IDs

    Change-Id: I7e50b3ff7e9a4299b70bcfa6e1fa24ba970a795b
Signed-off-by: Claudio D. Gasparini <claudio.gasparini@pantheon.tech>

Update git submodules

* Update docs/submodules/sfc from branch 'master'
  to 9006737c7ebdff81d18bdeef1faa370183f9e37d
  - Bump java-uuid-generator to 3.1.5

    The new version removes an explicit dependency on log4j, hence
    removing it from packaging.

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

Update git submodules

* Update docs/submodules/genius from branch 'master'
  to 27180ae1d0b6f2e9df3874136de3e4738b4d7ed8
  - Remove deprecated AsyncWriteTransaction#commit method

    This method has been deprecated for quite some time and now has a
    default implementation so can be safely removed from sub-classes.

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

Update git submodules

* Update docs/submodules/sfc from branch 'master'
  to 6e3002d199eb18d8ace7b045479f7912247cab5b
  - Migrate listener's deprecated methods to use the new ones

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

Update git submodules

* Update docs/submodules/sfc from branch 'master'
  to 96bbe52d0890c4111f3e4f99b239ec5f9285435a
  - SFC-215: Fix sfc-103 demo issues

    - Use Ubuntu 16 as Ubuntu 15 is no longer available
    - Fix ODL distribution URL path
    - Use latest ODL snapshot available instead of a fixed version
    - Removed unused and un-existent requirement 'messages'

    Change-Id: I6e9588dff79d82be3a077acf21478b43238e17a4
    JIRA: SFC-215
Signed-off-by: Jaime Caamaño Ruiz <jcaamano@suse.com>

Update git submodules

* Update docs/submodules/sfc from branch 'master'
  to dfc6c36284fa18b857955c64f89341a56625a2bf
  - Migrate listener's deprecated methods to use the new ones

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

Update git submodules

* Update docs/submodules/bgpcep from branch 'master'
  to 59baf6048ad1993a3854708a40d2d88fbea32098
  - Fix Route removal

    remove wrong check if from peer null.

    Change-Id: Ica7a80db21ee7279d9aa6071287b0c499999ed04
Signed-off-by: Claudio D. Gasparini <claudio.gasparini@pantheon.tech>

Update git submodules

* Update docs/submodules/netconf from branch 'master'
  to 932964ac7be6ff6c96ec376975fb1e4edab4ac64
  - Merge "Rename restconf-binding-parent -> restconf-parent"
  - Rename restconf-binding-parent -> restconf-parent

    This unifies the parent names across netconf project.

    Change-Id: Ic4f75b6de44d0a85e499e8a13f5cff2be570ba67
Signed-off-by: Jakub Morvay <jmorvay@frinx.io>

Update git submodules

* Update docs/submodules/bgpcep from branch 'master'
  to 4a815e73681b4fa7edf7b03bf836a9394f4ff980
  - Remove default Non add path implementation

    under Ribsupport interface, and implement it under
    new abstract class. Avoiding possible boilerplate.

    Change-Id: I1e563659df3678aea5e491806adb42adea3dea9d
Signed-off-by: Claudio D. Gasparini <claudio.gasparini@pantheon.tech>

Update git submodules

* Update docs/submodules/bgpcep from branch 'master'
  to 2b22416b6cee5b98c3f1151c19858ea7286d3346
  - BGPCEP-778: Fix peer-group reconfiguration

    by use correct peer-group name.

    Change-Id: Id68fbc1a9cd1a19aea26f0e810ec6b54560df359
Signed-off-by: Claudio D. Gasparini <claudio.gasparini@pantheon.tech>

Update git submodules

* Update docs/submodules/netconf from branch 'master'
  to 6c04356cdab705d5cf95b8a8c6ee43a77bf4a0e2
  - Fix NetconfDisconnectDeviceCommand logic error.

    The validation check of deviceIp and devicePort should be OR, not AND.

    Change-Id: I8a4ca1642d0bc6b196222ee60b6af9bf19f6330d
Signed-off-by: Li guosheng <li.guosheng6@zte.com.cn>

Update git submodules

* Update docs/submodules/netconf from branch 'master'
  to 06c4dbe076647af7d822fb8dc1d0da59d53ee141
  - Merge "Add netconf parent pom"
  - Add netconf parent pom

    This centralizes common configurations for checkstyle, findbugs etc

    Change-Id: Ic0b766644991a167b22b19966c494096f465f5d7
Signed-off-by: Tom Pantelis <tompantelis@gmail.com>
Signed-off-by: Jakub Morvay <jmorvay@frinx.io>

Update git submodules

* Update docs/submodules/openflowplugin from branch 'master'
  to f9b8745b55b324a1253c73ce8c95e7219ace0356
  - Do not repackage guava

    Guava is already provided by odl-guava-23, add a dependency on that.

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

Update git submodules

* Update docs/submodules/netconf from branch 'master'
  to d405f9a7b0487dc135f771f5ffe1f066fcde7be5
  - Merge changes Id70838c2,I6201e5b7,I4eb2d1d1

    * changes:
      Introduce AbstractNetconfOperationTest to decrease code duplication
      Introduce common parent for EditConfig and CopyConfig operations
      NETCONF-529: add support for <copy-config>

  - Introduce AbstractNetconfOperationTest to decrease code duplication

    Moves common code for NetconfOperation testing
    to AbstractNetconfOperationTest.

    Change-Id: Id70838c2b0c29cccb36c7a4e0c53519f65fd5bd7
Signed-off-by: Marek Gradzki <mgradzki@cisco.com>
  - Introduce common parent for EditConfig and CopyConfig operations

    <copy-config> and <edit-config> handling shares many common patterns.

    Common code was moved to AbstractEdit class
    (following AbstractGet naming)
    in order to reduce code duplication
    and simplify implementation of both operations.

    Change-Id: I6201e5b7fde6a307058e8ed20ac41f5622742024
Signed-off-by: Marek Gradzki <mgradzki@cisco.com>
  - NETCONF-529: add support for <copy-config>

    While <edit-config> is enough to configure any Netconf device,
    <copy-config> is more user friendly in scenarios,
    when entire configuration has to be written in single request, e.g.
    there is no need for explicit delete of nodes that were present in DS.

    This patch adds support for <copy-config> to mdsal-netconf-connector.
    Only <candidate/> is supported.

    To be included in subsequent patches:
    - move common test code to AbstractNetconfOperationTest
    - move common edit-config and copy-config code to AbstractEdit class

    Change-Id: I4eb2d1d1c82e60acc1502b01265de5ec3aab4cd8
Signed-off-by: Marek Gradzki <mgradzki@cisco.com>

Update git submodules

* Update docs/submodules/netconf from branch 'master'
  to 647e61052064693a422d48be8046272934916ac9
  - Merge "Remove deprecated AsyncWriteTransaction#commit method"
  - Remove deprecated AsyncWriteTransaction#commit method

    This method has been deprecated for quite some time and now has a
    default implementation so can be safely removed from sub-classes.

    Change-Id: I118d749f744d7253651f06927f490d8f17718832
Signed-off-by: Tom Pantelis <tompantelis@gmail.com>
Signed-off-by: Jakub Morvay <jmorvay@frinx.io>

Update git submodules

* Update docs/submodules/netconf from branch 'master'
  to 210c86a1688e72f9a92f7e58fada41231fedecc5
  - Merge "Use odl-jackson-2.8"
  - Use odl-jackson-2.8

    We should use the odl-jackson-2.8 feature to fulfill our dependencies
    on Jackson.

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

Update git submodules

* Update docs/submodules/netconf from branch 'master'
  to 51697b1224a8a84766d5139916ecc0f1889227dc
  - Merge "NETCONF-241: Switch to using Exificient"
  - NETCONF-241: Switch to using Exificient

    Exificient has a bit nicer API, it is available from maven central and
    has a friendly (MIT) license, plus the source code is on GitHub. Switch
    to using it instead of OpenEXI.

    Change-Id: I2cb7bf244361b021afb4cdc275e2442f5b778056
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
Signed-off-by: Andrej Mak <andrej.mak@pantheon.tech>

Update git submodules

* Update docs/submodules/netconf from branch 'master'
  to e3ec29995a4c541d90f201b838daac1250b01ace
  - Fix NetconfCommand test case error

    In testDisconnectDeviceCommand case, there should not appear
    connectDevice, but disconnectDevice.

    Change-Id: Ie4e980c1400612fb33300e9a9ccbff57e85c0067
Signed-off-by: Li guosheng <li.guosheng6@zte.com.cn>

Update git submodules

* Update docs/submodules/bgpcep from branch 'master'
  to dde173fdbce56d3ce3e7bdbfa6df02c933936b00
  - BGPCEP-712: Enforce checkstyle under binding-parent

    Remove from dindiong modules, given inheritance.

    Change-Id: I216d25eae5139f84663a9ae1ed43e94ab3d431ae
Signed-off-by: Claudio D. Gasparini <claudio.gasparini@pantheon.tech>

Update git submodules

* Update docs/submodules/bgpcep from branch 'master'
  to b70a14733a1433ebc1d3cb8865ff8aa7af844f64
  - BGPCEP-712: Set Checkstyle with warn violationSeverity level

    under specific modules. Until checkstyle issues
    are fixed. Therefore we can enforce on top parent
    the checkstyle for the rest.

    Change-Id: I70cb6e540e4d6e9fa07aabe0f52527896ba228d1
Signed-off-by: Claudio D. Gasparini <claudio.gasparini@pantheon.tech>

Update git submodules

* Update docs/submodules/bgpcep from branch 'master'
  to c9eba8ed16b5d19ebdd9a0d5867fdf90fb180119
  - Fix checkstyle

    for

    - data change counter module
    - parser mock module

    Change-Id: If4bda129f1602effd7deb67a96eaeda4f3924962
Signed-off-by: Claudio D. Gasparini <claudio.gasparini@pantheon.tech>

Update git submodules

* Update docs/submodules/odlparent from branch 'master'
  to 66e09e9d456c8c1f6fb5ef715a4298d18703bf40
  - Add features for Apache Commons

    This adds features for the four Apache Commons libraries we declare in
    dependency management: Codec, Lang 3, Net and Text.

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

BGPCEP-774: Add families to Peering doc

Change-Id: I27eeef5bd8a82cb070ac3dd16a55b22b0b6dcd65
Signed-off-by: Claudio D. Gasparini <claudio.gasparini@pantheon.tech>
(cherry picked from commit 2c0afedfd8d3b14d1dd961df618fe2adae009eef)

Update git submodules

* Update docs/submodules/bgpcep from branch 'master'
  to 439d02f636912be126cd8b2dc53c746d1d4b4519
  - Create Empty Route Container

    under eff rib in tables, when
    initializing the table.

    Change-Id: Ia6138acb76b594a2bfac26b5a2099bd069d7a4be
Signed-off-by: Claudio D. Gasparini <claudio.gasparini@pantheon.tech>

Update git submodules

* Update docs/submodules/bgpcep from branch 'master'
  to 0e4bbb56b257e90f1f2718910382165a988bc384
  - BGPCEP-777: Ignore ParserToSalTest until fix

    therefore autorelease-build will not be
    blocked

    Change-Id: I07df8e124c748a431735716926caf7c065a9d0ce
Signed-off-by: Claudio D. Gasparini <claudio.gasparini@pantheon.tech>

Update git submodules

* Update docs/submodules/bgpcep from branch 'master'
  to 64379e978ad394c752d53a62697dc5bbc4f523f0
  - BGPCEP-775: Install missing feature

    Openconfig rp Statement implementing(default
     of policies)

    Change-Id: Ic3d70dd960ce61c012a06f49d659582f9ca44444
Signed-off-by: Claudio D. Gasparini <claudio.gasparini@pantheon.tech>

Update git submodules

* Update docs/submodules/aaa from branch 'master'
  to 4d3a27866b118307e1b11ca05208be47736979c8
  - Remove aaa-filterchain Activator and statics

    Removed the bundle Activator in lieu of blueprint and also
    removed the static CustomFilterAdapterConfiguration instance.
    CustomFilterAdapterConfiguration was converted to an interface
    with implementation CustomFilterAdapterConfigurationImpl so it
    can be advertised as a service and consumed by aaa-shiro and
    injected into the CustomFilterAdapter.

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

Update git submodules

* Update docs/submodules/odlparent from branch 'master'
  to d8cfe538057d1ca67ffa0650efd9c7cf53cd0dac
  - Switch <issueManagement> from Bugzilla to JIRA

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

Update git submodules

* Update docs/submodules/ovsdb from branch 'master'
  to cb397afd30fffef22886e620d5ecb1b8c4e2b670
  - Check for null certManager

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

Update git submodules

* Update docs/submodules/netconf from branch 'master'
  to b439185577ad87fc5728de95212ab7780da8d8ed
  - NETCONF-530 - netconf-testtool failed on start with NullPointerException

    missing revisions in default names of YANG models

    Change-Id: I52c09699989b38ab828f99070081634dac9853ce
Signed-off-by: Jakub Tóth <jakub.toth@pantheon.tech>

Merge "BGPCEP: Match Neighbor Condition doc"

Update git submodules

* Update docs/submodules/netconf from branch 'master'
  to 859def62bc0aedbd28c96ad0e80bb07d297ae7d8
  - Remove unused test method

    This is done in preparation of Jackson removal.

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

BGPCEP: Match Neighbor Condition doc

Change-Id: I1c4a8850308d9d3f4f2c591bc6931ffc485b8205
Signed-off-by: Claudio D. Gasparini <claudio.gasparini@pantheon.tech>

Update git submodules

* Update docs/submodules/bgpcep from branch 'master'
  to ef4bcfbf8556bfe135d3e696a1ff2d5e9e19898d
  - Fix checkstyle under Concepts module

    Change-Id: If5ed38841ea0013f3e5b721f59c48d7cb778fedf
Signed-off-by: Claudio D. Gasparini <claudio.gasparini@pantheon.tech>

Update git submodules

* Update docs/submodules/netconf from branch 'master'
  to 1a25dbed0f61c4fe439784873e38a2d545dbfea0
  - Merge "Remove unused reference to Broker in NetconfDeviceSalProviderTest"
  - Remove unused reference to Broker in NetconfDeviceSalProviderTest

    The Broker et el interfaces are being deprecated so remove references.

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

Update git submodules

* Update docs/submodules/netconf from branch 'master'
  to 12965057f7cf7b833a87e3dca69799d02563143a
  - Merge "Depend on odl-aaa-shiro-act feature instead of bundle"
  - Depend on odl-aaa-shiro-act feature instead of bundle

    This is related to AAA-164, which involves exposing a feature for
    AAA activation instead of a bundle.

    JIRA: AAA-164
    Change-Id: I032e7dc8afef4b053d2162194454cde73cf85f45
Signed-off-by: Ryan Goulding <ryandgoulding@gmail.com>

Update git submodules

* Update docs/submodules/netconf from branch 'master'
  to 7d1c7469356747a7a572a0df320c3ff504df9727
  - Properly specify Jackson runtime dependencies

    Properly list feature dependencies for Jackson instead of relying on
    AAA bringing these deps in.

    Change-Id: I4857bf8498b7a64ddd12716c0a5e0b7d1d1d1928
    JIRA: NETCONF-531
Signed-off-by: Ryan Goulding <ryandgoulding@gmail.com>

Update git submodules

* Update docs/submodules/genius from branch 'master'
  to 31c5d2c07f552b29f8b9d1c9ebc07333187e4f09
  - deprecate IfmUtils' read() method

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

Update git submodules

* Update docs/submodules/genius from branch 'master'
  to a85ade55b848d9567978324e1415cd51d1fa89de
  - Use new Map methods

    Map::computeIfAbsent models the common "initialise value collection
    when missing and add to it" idiom.

    Change-Id: I6e21374a8dc3fba0b4942c40d0ab1031d65b97f8
Signed-off-by: Stephen Kitt <skitt@redhat.com>
  - Use switch instead of chained ifs

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