Merge "Update GBP release notes"

Merge "Update honeycomb/vbd release notes"

Merge "Add SXP Oxygen release notes"

Update git submodules

* Update docs/submodules/aaa from branch 'master'
  to 2555ea9f43349d3a9b03ba30532190e86f2ecfba
  - AAA-168: Remove embedded h2 dependency

    This was code inherited from a long time ago and punted around AAA
    without any real cleanup.  I am not sure why the original authors
    decided to embed the dependency rather than just import it, but this
    causes several issues.  This patch removes the embedded h2 dependency
    in favor of direct import.  While I recognize that other parts of
    ImportPackage need to be cleaned, they will be done in a subsequent
    patch since they are separate concepts than what is done here.
    In other words, expect follow-ups to continuing cleaning.

    aaa-cli-jar relied on aaa-shiro shading the com.h2database:h2 jar,
    so I instead added it as a compile time dependency for the module
    and extracted the appropriate files for the generated jar in the
    maven-shade-plugin configuration.

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

Merge "Remove XSQL from user guide"

Update git submodules

* Update docs/submodules/sfc from branch 'master'
  to ed9fa0d5692dbcb7d7d6eaabf913e5adba59135a
  - Remove unused reference to AsyncDataChangeEvent

    AsyncDataChangeEvent is deprecated and scheduled for
    removal.

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

Remove XSQL from user guide

This feature was removed in Nitrogen, remove it from documention,
too.

JIRA: DOCS-61
Change-Id: I4ed6bd5daba458fc68988b3f9879799c5d4c53ac
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Merge "(BUG DOCS-61) Fix outdated feature name for ALTO"

Update git submodules

* Update docs/submodules/infrautils from branch 'master'
  to e7fdd4ec4cb1a44c2a827dc8d1587ac8cf14a039
  - add checkIsRunning() to AbstractLifecycle

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

Update git submodules

* Update docs/submodules/bgpcep from branch 'master'
  to 271d6de3bf48fcd273e012e279e0bb2ad4f42d49
  - BGPCEP-574: Fix features

    by importing features intead of use the
    blundles.

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

Update git submodules

* Update docs/submodules/infrautils from branch 'master'
  to ffa05a18df5d675adee48953540ca4f7f1eff772
  - Clean up lambdas

    Use method references where appropriate, and expression lambdas
    instead of single-expression lambdas or functional interface
    declarations.

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

Update git submodules

* Update docs/submodules/infrautils from branch 'master'
  to 43aed7429efc4616834d3f925dcec3f4641dfe71
  - Fix a couple of thinkos in metrics Configuration

    maxThreads ends up using setThreadsWatcherIntervalMS, and
    maxThreadsMaxLogIntervalSecs ignores the provided value. This patch
    fixes these.

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

Update git submodules

* Update docs/submodules/netconf from branch 'master'
  to 30e57d308d57bf0f9bb3c9d080c7728c98b34ab2
  - Add interface SslHandlerFactory

    This factory is used by the TLS clients to create SslHandler that will
    be added to the channel pipeline when the channel is active. TLS
    versions are configurable.

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

Update git submodules

* Update docs/submodules/genius from branch 'master'
  to a383afe741575affdfda9afe14ec35e2ddc1840e
  - ifm changes for itm performance enhancements

    Change-Id: I250b75fea3ae29e8adb351693e7eeb0a596e18fe
Signed-off-by: Edw7n <n.edwin.anthony@ericsson.com>

Add SXP Oxygen release notes

Change-Id: I26c499ce9122a6dd5332f38989b5bc8864e666a2
Signed-off-by: Martin Dindoffer <mdindoff@cisco.com>

Update git submodules

* Update docs/submodules/netconf from branch 'master'
  to 3c8c893c2f85f1bb5fdc7c4c49a2267788eea539
  - Merge "Eliminate duplicate dependency"
  - Eliminate duplicate dependency

    Fixes a maven warning.

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

Update git submodules

* Update docs/submodules/netconf from branch 'master'
  to 885d26cbe6ab845352c8d853190dcec0310d0ded
  - Websocket Uri should have scheme wss if registration is https

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

Update git submodules

* Update docs/submodules/netconf from branch 'master'
  to 0855db6018489a3e97c72112ed3d023636777d2e
  - Add dependency on mdsal-binding-dom-codec

    One of the upstreams has removed it, add it back.

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

Update git submodules

* Update docs/submodules/odlparent from branch 'master'
  to 3dea650e81febc9890d70b56a6d2e39521ea552f
  - add modernizer.skip to -Pq

    I'm adding https://github.com/gaul/modernizer-maven-plugin/
    to infrautils in Id85737d789efb7ee07c8c7a34bdac39d8a96e076,
    and cannot figure out how to get Maven to merge an additional
    property in a profile, so why don't we just have this in
    odlparent already?  Some day modernizer-maven-plugin could
    be in odlparent.

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

Update git submodules

* Update docs/submodules/ovsdb from branch 'master'
  to 3e823b91f09d690c555e990b846e7ba497c8ee37
  - Fix logging configuration in IT classes

    https://git.opendaylight.org/gerrit/#/c/68992/ removed the
    logConfiguraton method AbstractConfigTestBase so fix
    resulting compilation error..

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

Update git submodules

* Update docs/submodules/bgpcep from branch 'master'
  to b3763355466a46bec4d440909caf23c3cc6007bf
  - BGPCEP-574: Rework rib config policies

    by making possible to pass only wanted
    attribute for condition match.

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

Update GBP release notes

Change-Id: Iebbddca2b6dbe86811910e85cb4f99f441c1a9af
Signed-off-by: Michal Cmarada <michal.cmarada@pantheon.tech>

Update honeycomb/vbd release notes

Change-Id: Ieacbf00a141052689d1ce73e1c57aac62fbab3eb
Signed-off-by: Michal Cmarada <michal.cmarada@pantheon.tech>

(BUG DOCS-61) Fix outdated feature name for ALTO

Change-Id: I463d966f3dda794165733367d92edc4bb7332b79
Signed-off-by: Kai Gao <godrickk@gmail.com>

Merge "Rework release-notes for Oxygen"

Update git submodules

* Update docs/submodules/infrautils from branch 'master'
  to 05e6dec061dce1899678beced0f649d17313df08
  - Enable findbugs-slf4j

    porting I60b127526ccc057d37ef900b7502f4ea84517d8a
    https://git.opendaylight.org/gerrit/#/c/66449/
    from odlparent to infrautils.

    see also https://lists.opendaylight.org/pipermail/odlparent-dev/2018-March/001774.html
    for more background about why we're not just waiting for odlparent anymore.

    in I91f21504c9a18580733a1eedd3ba379de79d37c2
    https://git.opendaylight.org/gerrit/#/c/69140/ there is something very
    similar done in project netvirt, with a slightly different
    implementation (using a SNAPSHOT odlparent, which this approach avoids).

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

Update git submodules

* Update docs/submodules/integration/test from branch 'master'
  to 85448c9d97b89989488e675b29b38ac42d8674e4
  - Deploy Openstack Using Robot Keywords

    Change-Id: I86736192bbb1d610218a81fbefaa5a4dcf6e2690
Signed-off-by: gvrangan <venkatrangang@hcl.com>

Update git submodules

* Update docs/submodules/openflowplugin from branch 'master'
  to 06455079c58e3f27fd30b865331463c0b2668392
  - Merge "OPNFLWPLUG-983 Group and flow removal stats are not reported in order"
  - OPNFLWPLUG-983 Group and flow removal stats are not reported in order

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

Update git submodules

* Update docs/submodules/integration/test from branch 'master'
  to 5374ad49290b5c25e667858f9a9e514b6c7cb958
  - Strip Change-Id from regex

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

Fix master should be Oxygen

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

Link to releng/builder rtd docs

Unlink releng/builder as a submodule of the docs project. This is
no longer necessary with lfdocs-conf.

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

Update git submodules

* Update docs/submodules/releng/builder from branch 'master'
  to d4939e4eef11d88f4934f3c131c628827e974b14
  - Merge "Remove Ocpplugin CSIT for Oxygen and Fluorine"
  - Remove Ocpplugin CSIT for Oxygen and Fluorine

    https://lists.opendaylight.org/pipermail/ocpplugin-dev/2017-November/000063.html

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

Update git submodules

* Update docs/submodules/releng/builder from branch 'master'
  to 425c4675d9f10b460ea8bf7d23fd46dfe1c1709f
  - Merge "Remove Topoprocessing CSIT for Oxygen and Fluorine"
  - Remove Topoprocessing CSIT for Oxygen and Fluorine

    https://lists.opendaylight.org/pipermail/release/2017-September/012326.html

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

Update git submodules

* Update docs/submodules/releng/builder from branch 'master'
  to 42b2fe9aa594822666a6f9a5bd62e54c483cb47f
  - Merge "Remove Unimgr CSIT for Oxygen and Fluorine"
  - Remove Unimgr CSIT for Oxygen and Fluorine

    https://lists.opendaylight.org/pipermail/release/2018-February/014001.html

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

Update git submodules

* Update docs/submodules/releng/builder from branch 'master'
  to 7544d1f19f54bafd1986d7ae977390b10b0f6c64
  - Remove NIC CSIT for Oxygen and Fluorine

    https://lists.opendaylight.org/pipermail/release/2018-February/014001.html

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

Update git submodules

* Update docs/submodules/releng/builder from branch 'master'
  to 366f605d642786b7e7183ea8bb3cc5ff601dab60
  - Fix misconfiguration of builder rtd job

    Without project-pattern it will mistakenly run against all projects.

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

Update git submodules

* Update docs/submodules/releng/builder from branch 'master'
  to 2fb5a4f438cb1c4813ef85e96a5b98491c6ad755
  - Use requirements.txt file for deps

    RTD requires a file for requirements as it does not read tox.ini

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

Use lfdocs-conf for conf.py

Standardize on lfdocs-conf for pulling in shared configuration.

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

Update git submodules

* Update docs/submodules/releng/builder from branch 'master'
  to e570ffb72d8a6d0c1e374605a8df933fd1b0d048
  - Use lfdocs-conf for documentation

    New initiative to separate out docs in RTD so that the main
    docs project does not have to load so many submodules everywhere.

    This will put releng/builder docs at the url:
    http://docs.opendaylight.org/projects/releng-builder

    Also include some reference links so that we can pull these pages
    from the OpenDaylight documentation project.

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

Update git submodules

* Update docs/submodules/integration/packaging from branch 'master'
  to a3dcf513deba33f160e23959a4963e683b2018f8
  - Add repos for opensuse

    Provide a repo for opensuse in Nitrogen and Oxygen

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

Update git submodules

* Update docs/submodules/netvirt from branch 'master'
  to ecc0f58ec76028087024df8d582e8db8a3c6461e
  - fix bad Exception logging in TunnelInterfaceStateListener

    and a bunch of other clear bugs in logging not related to exception

    We will be enforcing this locally by adopting the findbugs-slf4j
    bug-pattern proposed earlier in odlparent locally in netvirt already;
    watch Change-Id: I91f21504c9a18580733a1eedd3ba379de79d37c2.

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

Update git submodules

* Update docs/submodules/genius from branch 'master'
  to a894f43cfe6f49f68b847e61283f3a7ec05b6776
  - add JavaDoc to DataTreeEventCallbackRegistrar onAddOrUpdate

    in order to clarify null-ness expectation of the used BiFunction

    also an appropriate @Nullable (has to be org.eclipse.jdt.annotation,
    cannot be javax.annotation, because that is not a type annotation
    suitable for annotating generics)

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

Update git submodules

* Update docs/submodules/integration/packaging from branch 'master'
  to 4be7771531132329ea21f743e97cc943176e136a
  - Add Packer Carbon SR3 config file

    Change-Id: Ib6cfffac1897e937f49c66da62f53c70921ee0e8
Fixes: INTPAK-141
Signed-off-by: Daniel Farrell <dfarrell@redhat.com>
  - Add Carbon SR3 CBS RPM repo config file

    Change-Id: If4f6cee26288dcd0fdde6e85d91751a068926b1c
Fixes: INTPAK-141
Signed-off-by: Daniel Farrell <dfarrell@redhat.com>

Update git submodules

* Update docs/submodules/releng/builder from branch 'master'
  to 59d6fd23efe03d8f8dee690a99f31826bbd1a65f
  - Merge "Update example builds to test in RPM verify jobs"
  - Update example builds to test in RPM verify jobs

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

Update git submodules

* Update docs/submodules/releng/builder from branch 'master'
  to 2907183b5ce1a3c98806440b884db8fd329c1870
  - Merge "Use printf over echo for adding newlines"
  - Use printf over echo for adding newlines

    Posix recommends using printf to avoid portability issues
    with echo.

    Apparently echo is not consistent across different platforms
    and versions. Sometimes /n is a newline and other times it is
    a literal \n printed to the console.

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

Update git submodules

* Update docs/submodules/genius from branch 'master'
  to 2e27d0a011a89885827768b73a7b0735f89dab6a
  - Add HwvtepNodeHACache interface

    This is the first in a series of patches to eliminate the
    static HwvtepHACache. Added an HwvtepNodeHACache interface
    which HwvtepHACache temporarily implements until netvirt
    is migrated to use HwvtepNodeHACache in the next patch.
    The static HwvtepHACache instance is also temporarily
    advertised as the HwvtepNodeHACache service.

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

Update git submodules

* Update docs/submodules/genius from branch 'master'
  to c250f4b51fdd29ca4c9de9da1c6cb89aecc37cf8
  - Minor simplification in DataTreeEventCallbackRegistrarTest

    the point of the testAddOrUpdateAdd(), contrary to the
    testAddOrUpdateUpdate(), is that the listener will only get call once
    (for the add); this change makes that more obvious when reading the
    test.  It still passes.

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

Update git submodules

* Update docs/submodules/integration/packaging from branch 'master'
  to ba67185fadc6c777c2d16cff84b31941e7022d49
  - Add Packer Nitrogen SR2 config file

    Change-Id: I76e6cfefaaaa317af324800a6cffa4d31d764d7b
Fixes: INTPAK-142
Signed-off-by: Daniel Farrell <dfarrell@redhat.com>
  - Add Nitrogen SR1 CBS RPM repo config file

    Change-Id: I35e0a252171fffc06afe85ffa2eb6cd83cd22016
Fixes: INTPAK-142
Signed-off-by: Daniel Farrell <dfarrell@redhat.com>

Update git submodules

* Update docs/submodules/netvirt from branch 'master'
  to 9783c8726adb193308a6607b5239d286d13b343b
  - NETVIRT-1128: Add sfc back into netvirt features

    Change-Id: Ie57e2d4ade222548931ada8185a590ab01cd6c2a
    JIRA: NETVIRT-1128
Signed-off-by: Jaime Caamaño Ruiz <jcaamano@suse.com>
Signed-off-by: Sam Hague <shague@redhat.com>

Update git submodules

* Update docs/submodules/genius from branch 'master'
  to e38e3131add0fe1ffdbf2a474a04b55774184b7d
  - Fix ActionNxConntrack.NxNat problems with xtendbeans (with a test)

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

Update git submodules

* Update docs/submodules/releng/builder from branch 'master'
  to bf71b1a09ea3908eb0874a5369df8806974d444c
  - Merge "Add suse support for the snaps build"
  - Add suse support for the snaps build

    suse snaps rpm build and test happens inside a docker container
    This patch requires the following patch to be merged:

    https://git.opendaylight.org/gerrit/#/c/68114/

    It reuses most of the building and testing scripts and it avoids
    duplicating the packaging-build-rpm-snap job using the variable
    {distro} to differentiate between suse and epel.

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

Update git submodules

* Update docs/submodules/integration/packaging from branch 'master'
  to 7e66f116e37fb96d2ec8e71930f9cf8d878f559e
  - Add support for openSUSE in the RPM spec

    openSUSE support is required to run ODL in XCI-OPNFV. RPM installs in
    openSUSE fail because the dependencies listed in the spec file have
    different names. Conditionally require the right names based on OS.

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

Update git submodules

* Update docs/submodules/genius from branch 'master'
  to ba314905ae33078a0842f62f08cec37993d0d021
  - Fix ActionNxConntrack.NxCtMark problem with xtendbeans (with a test)

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

Update git submodules

* Update docs/submodules/infrautils from branch 'master'
  to 77f8cc268de2211dae33ddef06076b2d8f5444a7
  - Metrics implementation based on http://Prometheus.io

    This does not yet include proper OSGi web integration with a web IT,
    Configuration and required Karaf Feature work, but is a suitable "MVP"
    for initial demos (e.g. for the ONS ODL DDF). More TBD in future
    changes; that makes this series easier to review as well.

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

Update git submodules

* Update docs/submodules/sfc from branch 'master'
  to 72da156c501919b38e05d6b8db8815e120761963
  - SFC-213 SFC statistics dont always work

    - The OpenFlow NodeId is not available on
      the SFF, so the statistics cant work.
    - There's a race condition between when SFC
      creates the OVS bridge for an SFF and when
      we try to get the bridge DPID.

    Change-Id: I05b2d0da61f8147ffea0a995b3efef38cbf05b31
Signed-off-by: Brady Johnson <bjohnson@inocybe.com>

Update git submodules

* Update docs/submodules/openflowplugin from branch 'master'
  to a862a00411d1eadd7db500711723f9bc273580db
  - OPNFLWPLUG-984: always use ingress node for link discovery

    Using the LLDP destination (packet-in ingress) guarantees that the
    receiving controller is the owner of the device. Relying on the LLDP
    source can cause a scenario where the LLDP packet never appears in the
    entity-owner of the source device.

    Change-Id: I8ad4171cdae395d260b754b95e5d9b75138fb79b
Signed-off-by: Evan Zeller <evanrzeller@gmail.com>
    (cherry picked from commit 36722d99c3a17bb23d4299a906591f11a9847f03)

  - Enable findbugs in parent pom

    Change-Id: Ib9c64da593973b23c0bf2d3bf499adc95e16df54
Signed-off-by: Tom Pantelis <tompantelis@gmail.com>
  - Fix findbugs violations in test-provider and drop-test-karaf

    - Field only ever set to null
    - int value cast to float and then passed to Math.round
    - Possible null pointer dereference in method on exception path
    - Non-virtual method call passes null for non-null parameter
    - Comparison of String parameter using == or !=
    - Should be a static inner class
    - Private method is never called
    - int value cast to float and then passed to Math.round
    - Unread field
    - Unread field: should this field be static?
    - Unused field
    - Dead store to local variable
    - Private method is never called
    - Method uses the same code for two switch clauses
    - Write to static field from instance method
    - Field not initialized in constructor but dereferenced without null check

    Change-Id: Ib09380d626ab9162c4308810fae9db90522269cf
Signed-off-by: Tom Pantelis <tompantelis@gmail.com>
  - Fix findbugs violations in openflowplugin-impl

    - Method ignores exceptional return value
    - Check for sign of bitwise operation
    - Consider using Locale parameterized version of invoked method
    - Reliance on default encoding
    - Synchronization performed on util.concurrent instance
    - Unchecked/unconfirmed cast
    - Unchecked/unconfirmed cast of return value from method
    - Return value of method without side effect is ignored
    - Dead store to local variable
    - Method uses the same code for two branches
    - Load of known null value
    - Parameter must be non-null but is marked as nullable
    - Field not initialized in constructor but dereferenced without null check

    Change-Id: I92e00a5772038111d1ab8d9f53d52c43e3f4909d
Signed-off-by: Tom Pantelis <tompantelis@gmail.com>
  - Fix findbugs violations in openflowplugin

    - Comparison of String parameter using == or !=
    - Comparator doesn't implement Serializable
    - Field is a mutable collection
    - Incorrect lazy initialization of static field
    - Inefficient use of keySet iterator instead of entrySet iterator
    - Private method is never called
    - Unsigned right shift cast to short/byte
    - Class is final but declares protected field
    - Consider returning a zero length array rather than null

    Change-Id: I714bd4005a33a772b0b76b0c0f89f28b8517cd06
Signed-off-by: Tom Pantelis <tompantelis@gmail.com>
  - Fix findbugs violations in extensions

    - Dead store to local variable
    - Unchecked/unconfirmed cast
    - Field not initialized in constructor but dereferenced without null check
    - Method ignores return value
    - Private method is never called

    Change-Id: I7a3fe1700439b456cb7dce6d84f63a9840fd68a7
Signed-off-by: Tom Pantelis <tompantelis@gmail.com>
  - Fix findbugs violations in samples

    - Inconsistent synchronization
    - Using notify() rather than notifyAll()
    - Method ignores return value
    - Read of unwritten field
    - Field not initialized in constructor but dereferenced without null check

    Change-Id: I12e8ae3ab25cc41ba05b99e9e4c85f2606a2814d
Signed-off-by: Tom Pantelis <tompantelis@gmail.com>
  - Fix findbugs violations in liblldp

    - Method call passes null for non-null parameter
    - Class defines clone() but doesn't implement Cloneable
    - Clone method may return null
    - clone method does not call super.clone()
    - equals() method does not check for null argument
    - Reliance on default encoding
    - May expose internal representation by returning reference to mutable object
    - Field is a mutable collection which should be package protected - malicious code vulnerability
    - Field should be package protected - malicious code vulnerability
    - Inefficient use of keySet iterator instead of entrySet iterator
    - Boxing/unboxing to parse a primitive - use parseXXX method
    - Redundant nullcheck of value known to be non-null

    Change-Id: I6d551d31d618a1adfc877ac59f29d02727201df6
Signed-off-by: Tom Pantelis <tompantelis@gmail.com>
  - Fix findbugs violations in applications

    - Method ignores return value
    - Consider using Locale parameterized version of invoked method
    - Reliance on default encoding
    - Write to static field from instance method
    - Inefficient use of keySet iterator instead of entrySet iterator
    - Dead store to local variable
    - Consider returning a zero length array rather than null
    - Consider using Locale parameterized version of invoked method
    - Boxing/unboxing to parse a primitive - use parseXXX method
    - Unread field
    - Dead store to local variable
    - Non-transient non-serializable instance field in serializable class
    - Parameter must be non-null but is marked as nullable
    - Field not initialized in constructor but dereferenced without null check
    - Nullcheck of value previously dereferenced
    - Redundant nullcheck of value known to be non-null

    Change-Id: I1887f775f61fb8e3e6989113edf7faf4f9b06b34
Signed-off-by: Tom Pantelis <tompantelis@gmail.com>
  - Fix findbugs violations in openflowjava

    - Invocation of toString on an array

    - Check for sign of bitwise operation:
    Example: ((event.detail & SWT.SELECTED) > 0)

    Using bit arithmetic and then comparing with the greater than operator
    can lead to unexpected results (of course depending on the value of
    SWT.SELECTED). If SWT.SELECTED is a negative number, this is a candidate
    for a bug. Even when SWT.SELECTED is not negative, it seems good practice
    to use '!= 0' instead of '> 0'

    - Reliance on default encoding - performs a byte to String (or String to byte)
    conversion, and will assume that the default platform encoding is suitable.

    - Primitive value is boxed then unboxed to perform primitive coercion

    - Unread field

    - Result of integer multiplication cast to long - possible overflow

    - Dead store to local variable - assigns a value to a local variable, but
    the value is not read or used in any subsequent instruction.

    - Method uses the same code for two switch clauses

    - Load of known null value

    - Consider returning a zero length array rather than null

    - Redundant nullcheck of value known to be non-null

    - Field not initialized in constructor but dereferenced without null check

    - Non-transient non-serializable instance field in serializable class

    - Class implements same interface as superclass

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

Update git submodules

* Update docs/submodules/integration/test from branch 'master'
  to fbb916d2437066339d422ee6ba9f5c83071f7a34
  - NETCONF-520: Fix netconf test in oxygen

    Looks like a yang model changed during oxygen so new RPC URI and data
    for adding and deleting device are required from oxygen onwards.

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

Update git submodules

* Update docs/submodules/releng/builder from branch 'master'
  to f478710cb252dab7a91a6b7454c38f292cd6ca6e
  - Merge "Print last 10 integration/test commits in console"
  - Print last 10 integration/test commits in console

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

Update git submodules

* Update docs/submodules/releng/builder from branch 'master'
  to ea91a1744b396eddb8102a197942b2773be29318
  - Merge "fix ocata stack by looking for rabbitmq pid"
  - fix ocata stack by looking for rabbitmq pid

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

Update git submodules

* Update docs/submodules/releng/builder from branch 'master'
  to 2e75303d69c6e5123b6c296c44859acebd227ce6
  - Remove remaining newton usage

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

Update git submodules

* Update docs/submodules/sfc from branch 'master'
  to 0233b0172955ec4318b809635bb1ea774371c247
  - Refactor to change packages' names

    sfc-ios-xe-renderer package is replaced by more canonical name:
    org.opendaylight.sfc.renderers.iosxe

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

Update git submodules

* Update docs/submodules/releng/builder from branch 'master'
  to 9dddf63059dfa09bb9695ed0b9f8354bd4b3413b
  - Merge "Use Karaf test script that avoids ODLPARENT-139"
  - Use Karaf test script that avoids ODLPARENT-139

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

Update git submodules

* Update docs/submodules/releng/builder from branch 'master'
  to 2e3f8181c08b9d95435a17a1aa7a39ecbba7d3a3
  - Merge "Support branch-cutting version bump workflow"
  - Support branch-cutting version bump workflow

    Add support to the version bump job to handle version bumping
    for branch cutting activities as described in ODL processes.

    http://docs.opendaylight.org/en/latest/release-process/branch-cutting.html

    JIRA: RELENG-81
    Change-Id: I911ebe69c521ecf204a54a32a86f300aec7f22fd
Signed-off-by: Anil Belur <abelur@linuxfoundation.org>

Update git submodules

* Update docs/submodules/netconf from branch 'master'
  to ca503b8a06eed0c4a00f9a5cee549f941ef77044
  - Merge "Remove deprecated registerDataChangeListener method"
  - Remove deprecated registerDataChangeListener method

    registerDataChangeListener is scheduled for removal. The
    method now has a default implementation in the interface so
    the methods can be removed from the 2 DOMDataBroker implementations.

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

Update git submodules

* Update docs/submodules/netconf from branch 'master'
  to 2c1a8b7e484301622cbbf8cb89e3061f0e00970e
  - Enable findbugs in restconf-binding-parent

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

Update git submodules

* Update docs/submodules/netconf from branch 'master'
  to 55a92f9bfad21ea46e8cc1d4463a081d3a729924
  - Fix findbugs violations in sal-rest-docgen-maven

    - Method ignores exceptional return value
    - Method may fail to close stream on exception
    - Reliance on default encoding

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

Update git submodules

* Update docs/submodules/netconf from branch 'master'
  to 9d4f66971356d2ab3769b172e8b968ec36f894c2
  - Fix findbugs violations in sal-rest-docgen

    - Reliance on default encoding
    - Field is a mutable collection
    - Write to static field from instance method
    - Exception is caught when Exception is not thrown
    - Field not initialized in constructor but dereferenced without null check
    - Useless object created

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

Update git submodules

* Update docs/submodules/netconf from branch 'master'
  to a01dad851768102b3182033c10d860f5245d7d26
  - Fix findbugs violations in restconf-nb-rfc8040

    - Possible null pointer dereference
    - Method call passes null for non-null parameter
    - Nullcheck of value previously dereferenced
    - Consider using Locale parameterized version of invoked method
    - Reliance on default encoding
    - Inconsistent synchronization
    - Inefficient use of keySet iterator instead of entrySet iterator
    - Should be a static inner class
    - Useless assignment in return statement
    - Method uses the same code for two branches
    - Method uses the same code for two switch clauses
    - Write to static field from instance method
    - Redundant nullcheck of value known to be non-null
    - Exception is caught when Exception is not thrown
    - Useless control flow

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

Update git submodules

* Update docs/submodules/sfc from branch 'master'
  to 02f86cedc23fac7749961a34fd0790314e0e89b7
  - Refactor sfc-vpp-renderer

    - Migrate to use Blueprint
    - Use Blueprint annotations
    - Migrate listeners to use Genius' ones

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

Update git submodules

* Update docs/submodules/releng/builder from branch 'master'
  to e26824a76c42a76dae58fb23f772e856d5a85612
  - Refine the high-freq netvirt csit jobs

    the csit-high-frequency lists were outdated and using non-
    existent job names in some cases. We also want to see a
    high frequency of the queens jobs running now that we
    have seen Queens mature and we are close to releasing
    Oxygen

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

Update git submodules

* Update docs/submodules/integration/test from branch 'master'
  to a08f81f84bed6960a402daf21b3349e5803b9ef5
  - Fix L2GW test case 16

    Should be 3 ports, not 1.

    Fix a typo.

    Check port_mac[2], which is the remote OVSPORT2 MAC,
    instead of port_mac[4], which is the local HWVTEPPORT3
    MAC.

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

Update git submodules

* Update docs/submodules/releng/builder from branch 'master'
  to 7fb6a1f7d1a58393de3354881e7e2fec6a9c5620
  - Merge "Increase timeout for RPM port tests"
  - Increase timeout for RPM port tests

    Most verify-rpm jobs were failing due to ports not opening up in time.
    The previous 120 seconds seems to be slightly too short, 125 seconds
    seems to be about normal.

Fixes: INTPAK-145
    Change-Id: I53f92bc84dda6318734010b8824b6d33c3c24798
Signed-off-by: Daniel Farrell <dfarrell@redhat.com>

Update git submodules

* Update docs/submodules/aaa from branch 'master'
  to f3ade62d1c64aa52b1b6fc1309ee2c4919729c00
  - AAA-167:  Refresh test cert

    Test cert used in UT for AAA was expired causing failures.  For now,
    a 10 year cert is added to unblock the release.  Later, the tests
    will be refactored to generate the key on the fly.

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

Merge "Update branch cutting documentation"

Update git submodules

* Update docs/submodules/ovsdb from branch 'master'
  to a26df7eb62bf26db7aa6a9165127eb1d5ea0c6eb
  - Fix checkstyle violations in utils

    NotifyingDataChangeListener is fixed to be thread-safe, with
    ConcurrentHashMap-backed sets and volatile fields where necessary.

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

Update git submodules

* Update docs/submodules/releng/builder from branch 'master'
  to 378418e96c45b8f2581025fba5276c421925b32b
  - Build and test rpm-snap using docker

    As agreed with Daniel, it will be easier if we move epel-7 to docker
    too before merging:

    https://git.opendaylight.org/gerrit/#/c/68201/

    Change-Id: I81453fcd23e19bf0dfdbc3199d149ebf2dbbb0cb
Signed-off-by: Manuel Buil <mbuil@suse.com>
Signed-off-by: Daniel Farrell <dfarrell@redhat.com>

Update git submodules

* Update docs/submodules/coe from branch 'master'
  to ec9fb6a8648146e2f47cf042949c5fd5d994b6d8
  - Updating paths in rst docs

    Change-Id: I170e4fd350d6f20154507e807a3d38dbd257f935
Signed-off-by: R P Karthika <r.p.karthika@ericsson.com>

Update git submodules

* Update docs/submodules/ovsdb from branch 'master'
  to 243e905149ef4fe73b96720e52b622a171c1c71d
  - Fix checkstyle violations in schema

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

Update git submodules

* Update docs/submodules/ovsdb from branch 'master'
  to 22684d071230ac6b83d55c6a8487f16e706e5a48
  - Add binding-parent pom

    This will contain common configurations for checkstyle and
    findbugs.

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

Update git submodules

* Update docs/submodules/genius from branch 'master'
  to ed6930381e6c352ac14260d03374977ae25e2923
  - Flourine idmanager build failures

    sal-common-testutil has been removed from controller,
    as the relevant modules have moved to mdsal

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

Rework release-notes for Oxygen

Change-Id: If005b8b3c205710c46353aee20bb614c53a383aa
Signed-off-by: Scott Melton <smelton2@uccs.edu>

Update git submodules

* Update docs/submodules/netvirt from branch 'master'
  to 6dca27c3006150d0f8cd33f0cdfad68d8aaf0591
  - NETVIRT-1122 - L3 ping between pods not working

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

Update git submodules

* Update docs/submodules/integration/test from branch 'master'
  to 614b5775b52cce81c8e89434fd0e2d308b25925b
  - Add OVS dumps to Test Tear Down

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

Update git submodules

* Update docs/submodules/netvirt from branch 'master'
  to 0921991f9db5babcd8ec9868c0be73449a095a0f
  - Sync nitrogen with pike

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

Update git submodules

* Update docs/submodules/netvirt from branch 'master'
  to a380f2191dd36a7127a3ea6ff846b9cf017db830
  - Add CSIT update doc for OS version changes

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

Update git submodules

* Update docs/submodules/netvirt from branch 'master'
  to 08acef79c93100ae14da9a2b3c72a0f2d68de94d
  - oxygen updates to support matrix

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

Update git submodules

* Update docs/submodules/netvirt from branch 'master'
  to 07db9627fd8289b1c2dae5e663e481135fca8906
  - fix delete Direct with switchdev port

    Change-Id: I3165d5341770296052ded02751aabc3bee654cf3
Signed-off-by: Moshe Levi <moshele@mellanox.com>
Co-authored-by: Hamdy Khader <hamdyk@mellanox.com>

Update git submodules

* Update docs/submodules/releng/builder from branch 'master'
  to ecb4770d31a9794ab8b28cadbc23cfaa6e8fa229
  - Merge "Auto Update CSIT Jobs to run for fluorine"
  - Auto Update CSIT Jobs to run for fluorine

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

Update git submodules

* Update docs/submodules/releng/builder from branch 'master'
  to 1b2526f3d902e9c182363c06ea42444063c30f6b
  - Merge "Auto Update CSIT Jobs to run for oxygen"
  - Auto Update CSIT Jobs to run for oxygen

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

Update git submodules

* Update docs/submodules/releng/builder from branch 'master'
  to ba4f2235d00b1598bc94bcfee789ccd5b2f4155a
  - Auto Update CSIT Jobs to run for nitrogen

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

Update git submodules

* Update docs/submodules/releng/builder from branch 'master'
  to 0d5a8d815bbc7a7601ae657dfde54cf3ff909ba1
  - Merge "Trim L2GW Combinations"
  - Trim L2GW Combinations

    Keeping the same combos as Netvirt in:
    https://git.opendaylight.org/gerrit/#/c/68909/

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

Update git submodules

* Update docs/submodules/releng/builder from branch 'master'
  to ef663fe84e3695fe3ab3116fc1c8bea7505df117
  - Image for the Openstack Queens Job

    Change-Id: Id201be3096332305afa5620ae04a87381fbafdf6
Signed-off-by: gvrangan <venkatrangang@hcl.com>

Update git submodules

* Update docs/submodules/sfc from branch 'master'
  to 7198b98cb40ca478dd022e0439f4c541ccdc8962
  - Minor change in .gitignore

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

Update git submodules

* Update docs/submodules/sfc from branch 'master'
  to 8ca4d655e4e62f8cc42c345e2abad0656891a582
  - Fix sfc-104 demo issues

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

Update git submodules

* Update docs/submodules/sfc from branch 'master'
  to 3c72eab2fe4c1db9aa37933be139cc17d178f82c
  - SFC-218: Fix sfc-scf-vpp wiring

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

Update git submodules

* Update docs/submodules/netvirt from branch 'master'
  to 8a7f42cd34b2f618b4bc67e0a420f11ccdd0db0c
  - NETVIRT-1126: avoid shipping the MySQL connector

    This strips the MySQL connector from the Karaf distribution we build,
    to avoid shipping a GPL component in the distribution.

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

Update git submodules

* Update docs/submodules/netvirt from branch 'master'
  to 5947c23d71c5340c8a568ee97d69998a22349507
  - NETVIRT-1123: Remote ACL issue during cross SG rules

    Remote ACL table was not programmed when port is the first port on
    the dpn for a remote ACL in the below scenario.

    VM1(sg1) <-> VM2(sg2)

    sg1 -> allow icmp to/from sg2
    sg2 -> allow icmp to/from sg1

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

Update git submodules

* Update docs/submodules/odlparent from branch 'master'
  to ae83730319a9afe97d4a67c11049d039835c94f8
  - Bump sevntu-checkstyle to 1.27

    https://sevntu-checkstyle.github.io/sevntu.checkstyle/#1.25.0
    https://sevntu-checkstyle.github.io/sevntu.checkstyle/#1.26.0
    https://sevntu-checkstyle.github.io/sevntu.checkstyle/#1.27.0

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

Update git submodules

* Update docs/submodules/odlparent from branch 'master'
  to 41b1ab85fb5c5c14d2499080b4b5b848b830ab6f
  - Bump checkstyle dependency to 8.7

    http://checkstyle.sourceforge.net/releasenotes.html#Release_8.5
    http://checkstyle.sourceforge.net/releasenotes.html#Release_8.6
    http://checkstyle.sourceforge.net/releasenotes.html#Release_8.7

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

Update git submodules

* Update docs/submodules/odlparent from branch 'master'
  to 44cb9f54d871445bccbf8b8e8c5567dc1646cc22
  - Bump maven-checkstyle-plugin to 3.0.0

    https://mail-archives.apache.org/mod_mbox/maven-announce/201801.mbox/%3Cop.zchs68akkdkhrr%40desktop-2khsk44.mshome.net%3E

    Also removes checkstyle.version property, as noted in its TODO, and
    switches from <sourceDirectory/> to <sourceDirectories/>, as
    documented on the plugin site on
    https://maven.apache.org/plugins/maven-checkstyle-plugin/

    Change-Id: I3bffa47a300f314aaefe5181ca92fd0a6c925755
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
Signed-off-by: Stephen Kitt <skitt@redhat.com>