Update git submodules

* Update docs/submodules/integration/test from branch 'master'
  to 1cff2e15406854e64b4f2b65a2e77e550d170651
  - Ensure everything fits on screen

    Depending on the window size, the controller names and number and
    names of shards, we might run off the end of the screen, which aborts
    curses in a rather inelegant fashion. We reduce the column widths
    until everything fits.

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

Update git submodules

* Update docs/submodules/releng/builder from branch 'master'
  to 61e4ca3a03caf2425e0bc02b4337f8324ca9bd2c
  - Fix OF scale test

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

Update git submodules

* Update docs/submodules/integration/test from branch 'master'
  to cfe6bba264c670bbe02dc92f03667b2e4b9b37a0
  - Fix bug in ip -> hypervisor retrieval

    the bug was being hit when the ip address of the compute node
    in question matched the substring of the ip address of another
    compute node in the hypervisor list output.

    example:
    10.30.170.5 was matching on 10.30.170.5 -and- 10.30.170.56.

    This was probably not happening often, but it did happen here:
    https://logs.opendaylight.org/releng/vex-yul-odl-jenkins-1/netvirt-csit-1node-openstack-ocata-upstream-stateful-carbon/245/log_full.html.gz#s1-s8-t7-k3-k3-k4

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

Update git submodules

* Update docs/submodules/releng/builder from branch 'master'
  to 8da9f0bdc88c77738033c51abb951e0b3295648e
  - Merge "Create RAM disk on ODL VM /tmp folder"
  - Create RAM disk on ODL VM /tmp folder

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

Update git submodules

* Update docs/submodules/releng/builder from branch 'master'
  to f9913c593c902def3683edf75781309eb5b6267c
  - Update HEAT scripts to use local gerrit mirror

    With Gerrit no longer living in the same cloud as the CI infrastructure
    a localized mirror has been put in place to speed up git operations as
    much as possible. Since instance bring-ups should be as fast as we can
    possibly make them we need to point at this localized mirror for the
    spin-up scripts.

    Change-Id: I7bbeaf36bd7ccd5f5c9d45707d3c1f8a8507ee5f
Signed-off-by: Andrew Grimberg <agrimberg@linuxfoundation.org>

Update git submodules

* Update docs/submodules/integration/test from branch 'master'
  to 3a805b211800941b4250effa90fa82cc3ad9142b
  - React quickly to quit requests

    This patch checks keypresses in both loops inside the main loop,
    ensuring that the monitor quits soon after the user presses 'q'.

    Change-Id: I5889d1958e6c3887cde715feac4bc3cb87b67f34
Signed-off-by: Stephen Kitt <skitt@redhat.com>
Signed-off-by: Jamo Luhrsen <jluhrsen@redhat.com>

Update git submodules

* Update docs/submodules/integration/test from branch 'master'
  to 64f2a2226c27ba9b10650d08aa02913276509984
  - Support starting up with a partial cluster

    This patch reworks the monitor so that it can start without a working
    cluster:
    * controller and shard information retrieval is done in the main loop;
    * unknown controllers are displayed using their IP address and port
      (and the “controller” column has its own size);
    * whenever information becomes available (controller name, shards) it
      is added to the working set.

    Change-Id: Ic50aac9b835e4aabeb5edccb588835d0823e2e47
Signed-off-by: Stephen Kitt <skitt@redhat.com>
Signed-off-by: Jamo Luhrsen <jluhrsen@redhat.com>

Update git submodules

* Update docs/submodules/integration/test from branch 'master'
  to 85c3c53c576057e2859ba88b02c2290d563d097d
  - Avoid tracebacks on initial probe

    rest_get is called to retrieve shard information twice, once without a
    try block, and again in a try block. When a controller is down, the
    first one fails, so the try block is useless (and the user ends up
    with an obscure error message).

    This patch removes the first, useless call.

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

Update git submodules

* Update docs/submodules/integration/test from branch 'master'
  to c22f4802df965143d8bd72ae25b0c763f51a8bde
  - Track controller names in the main controller map

    Instead of building a separate array of controller names, track them
    as an additional property in the map (this will simplify handling
    controllers which are down at startup).

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

Update git submodules

* Update docs/submodules/integration/test from branch 'master'
  to 8b8c573c4688be682727d3264ad37ee5ff556754
  - pcepuser update

    On Oxygen there is new session-state output
    Reworked Test Cases to compare more specific output

    Change-Id: Id55de439a3d8ae657325bc02fcd0bdf4ab7c415e
Signed-off-by: Tomas Markovic <tomas.markovic@pantheon.tech>

Update git submodules

* Update docs/submodules/integration/packaging from branch 'master'
  to 1bcdcce400e92361d4b29ab277e92bf7f6d7e755
  - Refactor ansible-opendaylight docs

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

Update git submodules

* Update docs/submodules/releng/builder from branch 'master'
  to ef1e7804bd0754c42bf8525d3fbdb04080bd18a5
  - Add more build timeout in bgpcep

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

Update git submodules

* Update docs/submodules/integration/test from branch 'master'
  to abdf8e95fc1881db5ec164183b8a74cc738fa096
  - bgp flowspec test fix

    Unifying number of connection retries across tests
    Reconfiguring exabgp config file

    Change-Id: Ibfc511006634e28de368082f675503bca863e9be
Signed-off-by: Tomas Markovic <tomas.markovic@pantheon.tech>

Update git submodules

* Update docs/submodules/integration/test from branch 'master'
  to 09a10b345d45095d02e02976b864b4d0837c371e
  - Increase json http timeout

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

Update git submodules

* Update docs/submodules/integration/test from branch 'master'
  to 3fbb0e18faa83c3adc4316b6410943bc28351ab9
  - s/CFEATURES/CONTROLLERFEATURES

    looks like this patch [0] is using CONTROLLERFEATURES on
    the pybot command line. I don't see anywhere where it
    gets copied to CFEATURES

    [0] https://git.opendaylight.org/gerrit/#/c/66534

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

Update git submodules

* Update docs/submodules/releng/builder from branch 'master'
  to aa09d0d9c1737f2fcb3dc20c4b60f607f806a500
  - Merge "Increase build timeout"
  - Increase build timeout

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

Update git submodules

* Update docs/submodules/releng/builder from branch 'master'
  to be9f17c9002262eb53dbf225b720e253126bb118
  - Remove accidental shague in job name

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

Update git submodules

* Update docs/submodules/releng/builder from branch 'master'
  to 315b03903de6eb50a6e99b13ce068c25ca194ea6
  - Increase build timeout for docs

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

Update git submodules

* Update docs/submodules/integration/test from branch 'master'
  to 764603158cbba71d7b99f3c23633b6f5c5b07bae
  - increase template timeout

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

Update git submodules

* Update docs/submodules/releng/builder from branch 'master'
  to 01c8cb541b68ce31c95a276513772aea03510ff8
  - Merge "Increase build timeouts for genuis"
  - Increase build timeouts for genuis

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

Update git submodules

* Update docs/submodules/releng/builder from branch 'master'
  to f489e4544e201659f9d07e294e7f86d689b1797b
  - Increase build timeout

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

Update git submodules

* Update docs/submodules/releng/builder from branch 'master'
  to a04c68217dfce1fe07371d581ea7dbcc96b810d4
  - Merge "Remove unused os compute nodes"
  - Remove unused os compute nodes

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

Update git submodules

* Update docs/submodules/integration/test from branch 'master'
  to 4634d5bdc188c5ef1edc72cc185ebf61ec8912ae
  - cleanup connections

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

Update git submodules

* Update docs/submodules/integration/packaging from branch 'master'
  to a80f11a3ef3070d98a49d0aec97e3b1873e005e8
  - Adds Ansible opendaylight RTD docs

    Imports most content from
    https://git.opendaylight.org/gerrit/#/admin/projects/integration/packaging/ansible-opendaylight

    Also:
      * removes typos
      * adds example use cases for role variables

    Change-Id: Ie4ff329c7fb5ba14ca0f6246b13bf0848f8636d5
    Resolves: INTPAK-44
Signed-off-by: Dipankar Jain <bunny_jain@ymail.com>

Update git submodules

* Update docs/submodules/releng/builder from branch 'master'
  to 9e475a3437c3cb888155e509f6c552e1dc9b4e17
  - Increase vm perf

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

Update git submodules

* Update docs/submodules/integration/test from branch 'master'
  to f98e958da3a334baa3d533195dea49310370e07b
  - increase tempest timeout

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

Update git submodules

* Update docs/submodules/releng/builder from branch 'master'
  to 94c375fc80a9590586b9f8af51e56ec1ce8733cd
  - Move CSIT perf job to 4c ODL image

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

Update git submodules

* Update docs/submodules/releng/builder from branch 'master'
  to 18acbcbb8006ada369c956f1674851a5dcc35fc8
  - Merge "Bump up vm images"
  - Bump up vm images

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

Update git submodules

* Update docs/submodules/releng/builder from branch 'master'
  to 48101e4c93aba0e17159174020ed2da0a03d560a
  - Merge "Refactor Is Feature Installed"
  - Refactor Is Feature Installed

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

Update git submodules

* Update docs/submodules/integration/test from branch 'master'
  to 9af2e4ffccd55d6307f298dc857bd3dbdd0c6ab3
  - Refactor Is Feature Installed

    Removed the need to query the odl system by
    using the CONTROLLERFEATURES parameter.

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

Update git submodules

* Update docs/submodules/releng/builder from branch 'master'
  to 471ab9f350f01c388809343a3c1aade4b34ce829
  - Change deafult ODL image to 2c

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

Update git submodules

* Update docs/submodules/releng/builder from branch 'master'
  to 276241a10ff27d65dc3d1e5c6b1883d2562ffae0
  - Add more build timeout in bgpcep

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

Update git submodules

* Update docs/submodules/integration/test from branch 'master'
  to 7fc0a191fd8f0e8b5a2bd78dc686d6bc5c2a6cd0
  - Cleanup unused netvirt suites

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

Update git submodules

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

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

Update git submodules

* Update docs/submodules/releng/builder from branch 'master'
  to 66d9d9f7c8d5fadc451faf7051dbfa25b801acee
  - Merge "Bump all java-builders to 2 core machines"
  - Bump all java-builders to 2 core machines

    This should hopefully get us back to the same build times as we had
    in the old cloud.

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

Update git submodules

* Update docs/submodules/releng/builder from branch 'master'
  to 6a53dc8aa789fcffdb94302d209f84b0ce776df7
  - Merge "Remove TSDR from Oxygen CSIT blacklist"
  - Remove TSDR from Oxygen CSIT blacklist

    Change-Id: I95624a429df6f45b52a847dcd5c5858c3fdb4e24
Signed-off-by: AbdulRahman AlHamali <a.alhamali93@gmail.com>

Update git submodules

* Update docs/submodules/integration/test from branch 'master'
  to 7780cb34c49165d3d49485a6c10a14cd20e08501
  - Increase Get Vm IPs from 120s to 180s

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

Update git submodules

* Update docs/submodules/releng/builder from branch 'master'
  to 7a3af21a0b8d0c01800dd27a65c891a7f53fe4d4
  - Merge "Update cloud image list docs"
  - Update cloud image list docs

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

Update git submodules

* Update docs/submodules/releng/builder from branch 'master'
  to 566849d2c2b088a970f137645c16284c01843fcf
  - Fix auto-update image docs script

    Should pick up the new Vexxhost images now.

    LF-Issue: RELENG-724
    Change-Id: Ic0f898e9f84c7adc4bdc2346c52a1169d5b4705a
Signed-off-by: Thanh Ha <thanh.ha@linuxfoundation.org>

Update git submodules

* Update docs/submodules/releng/builder from branch 'master'
  to 65f6f5fc89f0cc9b3fbab7e23c6c2539f5cd9aec
  - Add Vexxhost flavors docs

    Include Vexxhost performance node flavor docs since they do not
    document this on their own website.

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

Update git submodules

* Update docs/submodules/integration/packaging from branch 'master'
  to 206866932247285e67bf6c785cacc3596854c98a
  - Add docs about new Nexus-hosted RPM CD repos

    Also update versions/links for all other repo docs, fix typos.

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

Update git submodules

* Update docs/submodules/releng/builder from branch 'master'
  to a5bf92f8ca094c484c0fb59a34ffd84af0c98410
  - Fix orphaned nodes script

    Looks like unintended changes from
    4d3119e3d3dbd7743517854174a5f6e75ad59c67 snuck in somehow.

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

Update git submodules

* Update docs/submodules/integration/test from branch 'master'
  to da6cdbb5fb739935b2592aa3ec1f5c696872493a
  - bgp_functional_route_ref fix

    Addressing variety in order in cli output.
    Devided template into multiple sections.

    Change-Id: I9e2cde0dd82fe87d6f2ce6fa1ff349ce1644e90c
Signed-off-by: Tomas Markovic <tomas.markovic@pantheon.tech>

Update git submodules

* Update docs/submodules/releng/builder from branch 'master'
  to aff41e45235e9bf2b3302953d432a50f7c57f117
  - Fix indent

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

Update git submodules

* Update docs/submodules/releng/builder from branch 'master'
  to d6629c77790f1317c1545612f19d4d71da79a1eb
  - Merge "Continue to delete instead of abandon"
  - Continue to delete instead of abandon

    The abandon API is not supported by Vexxhost so keep trying deletes
    and see if it eventually works out.

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

Update git submodules

* Update docs/submodules/releng/builder from branch 'master'
  to 281c30cfd0704a42eb19850ff04594e3312d1c8b
  - Merge "Print the name of the stack being deleted"
  - Print the name of the stack being deleted

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

Update git submodules

* Update docs/submodules/releng/builder from branch 'master'
  to e6600be8695cb9e21b47fabfc487d5b57a7f2420
  - Merge "Add a job to cleanup orphaned nodes"
  - Add a job to cleanup orphaned nodes

    This job assumes that releng minions have a 'prd' prefix and
    sandbox has a 'snd' prefix. These minions are Jenkins managed
    minions and are not Heat managed minions.

    The script get's a list of jobs with the prd or snd prefix and then
    checks against Jenkins to see if the nodes are attached to Jenkins
    if they are not then issue a delete call as it is likely Jenkins
    lost track of them.

    Issue: RELENG-724
    Change-Id: I624102ace279d49d2696d287cfea9e4e89ba7a20
Signed-off-by: Thanh Ha <thanh.ha@linuxfoundation.org>

Update git submodules

* Update docs/submodules/releng/builder from branch 'master'
  to a4b421f369e7828af6c63da4c502bf1780ffb99a
  - Merge "Remove unnecessary jq call in stale-stacks"
  - Remove unnecessary jq call in stale-stacks

    jq is not actually necessary to run our query. By getting values
    instead of json data we can remove the unnecessary complexity of
    this part of the script.

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

Update git submodules

* Update docs/submodules/releng/builder from branch 'master'
  to 0d07d7fdb70be0afc0174403bc378958cbebc5b1
  - Create volume using Cinder

    This switches the volume creation to be done using Heat and
    attached via Nova instead of letting Nova create it right away.

    The reason behind this is that it will probably be much cleaner
    for clean-up and Heat is much more 'patient' in order to get
    the resource created.

    Change-Id: Ifc70acb967a8a2f4007aa3b020232258b4032aec
Signed-off-by: Mohammed Naser <mnaser@vexxhost.com>

Update git submodules

* Update docs/submodules/releng/builder from branch 'master'
  to 2c18f845ed77f3fbbc022aaec4858f796f996087
  - Use right pid for jstack collection

    The jstack collection fails in post-carbon distros.
    Starting with karaf 4 there are two processes for karaf
    where the existing code expected one.

    Grep was reworked to use the main process of the
    karaf shell to return a single pid. This works for
    carbon also since that is the only process found.

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

Update git submodules

* Update docs/submodules/releng/builder from branch 'master'
  to 31bdba3657dc73f68fe1c8c63c01351cf2bbbcf8
  - Merge "Update cloud image list docs"
  - Update cloud image list docs

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

Update git submodules

* Update docs/submodules/releng/builder from branch 'master'
  to adf066a0a9e6202a0728ffb17222c316376df095
  - Merge "Change default RPM repo, CentOS CBS to ODL Nexus"
  - Change default RPM repo, CentOS CBS to ODL Nexus

    Default to new fully-automated CD repos, away from manual CBS.

    Also move to Nitrogen from Carbon while changing upgrade test URLs.

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

Update git submodules

* Update docs/submodules/releng/builder from branch 'master'
  to ef61b21a692157a0eae2e65a44258c168d56044f
  - Modify mtu values in ml2 conf

    smaller mtu values are needed in the new cloud
    provider

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

Update git submodules

* Update docs/submodules/releng/builder from branch 'master'
  to f3a144b749f560590f2fc350dd13c5a45e3834fe
  - Merge "Do not use domain part of hostname"
  - Do not use domain part of hostname

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

Update git submodules

* Update docs/submodules/releng/builder from branch 'master'
  to 3169d1b21c03a816cdd3527a88f2507483eafbfd
  - Use 4x16 VM for control nodes

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

Update git submodules

* Update docs/submodules/releng/builder from branch 'master'
  to bdb97e6d9e5282e298d082b7d298c86a72dde72b
  - Merge "Fix dep tests to work with both Karaf 3/4"
  - Fix dep tests to work with both Karaf 3/4

    Karaf 4 distros built on the CentOS CBS (should be clean env) end up
    with a requirement listing /usr/bin/env whereas Karaf 3 distros don't.

    I don't think these simple requirements are functional, but this gets
    the test passing so it can catch actually-harmful requirements if they
    slip in.

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

Update git submodules

* Update docs/submodules/releng/builder from branch 'master'
  to 6b9683246b616633445ecf6136369b9f0d9dbd9e
  - Revert "Consolidate a single openstack-cron job"

    This actually does not work as expected so reverting for now.
    Will need to explore this again at a later point.

    This reverts commit 893b5ff40571682200424fce68bada556db4cece.

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

Update git submodules

* Update docs/submodules/releng/builder from branch 'master'
  to b9f6fab05577d1a42e9697383eada89a53b35d25
  - Bump builder jjb job timeouts to 30

    The new cloud servers have less CPUs so less actual parallel
    can happen despite having a worker # of 4. Job updates are taking
    longer now.

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

Update git submodules

* Update docs/submodules/releng/builder from branch 'master'
  to 893b5ff40571682200424fce68bada556db4cece
  - Consolidate a single openstack-cron job

    Rather than wasting VMs to run a bunch of separate OpenStack
    maintenance jobs, consolidate into a single cron job which
    runs at our shortest needed interval.

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

Update git submodules

* Update docs/submodules/releng/builder from branch 'master'
  to f63efd2b7c5e453af225a64b158e03e5a9bdbc28
  - Merge "Fix broken log collections"
  - Fix broken log collections

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

Update git submodules

* Update docs/submodules/releng/builder from branch 'master'
  to 94dbade4c5d33fda5ce90eaceb527fc3c4b197e6
  - Merge changes from topic 'RELENG-704'

    * changes:
      Use protected to prevent cleanup script rming img
      Update cleanup scripts to work in Vex
      Update Orchestration scripts to work with Vex
      Update mininet and docker builders
      Update java-builder build-nodes for new cloud
      Update packer config to work in Vexxhost

  - Use protected to prevent cleanup script rming img

    With a pure OpenStack cloud we can use the "protected" setting to
    mark an image and prevent the cleanup script from removing in use
    images.

    Needs a new lftools release with the following patch
    https://gerrit.linuxfoundation.org/infra/7887

    Issue: RELENG-705
    Change-Id: I1096c2e7724494f9e24db5f31ee396eaea024fdc
Signed-off-by: Thanh Ha <thanh.ha@linuxfoundation.org>
  - Update cleanup scripts to work in Vex

    This patch depends on a new lftools release with
    https://gerrit.linuxfoundation.org/infra/7887

    This is needed so that we can filter images with the
    ci_managed=yes metadata field and also hide protected images from the
    search query as we only want to delete images that are managed by
    changes which are not protected.

    Issue: RELENG-705
    Change-Id: Id8724035ed470f20418d6bbf181802592ffe18d0
Signed-off-by: Thanh Ha <thanh.ha@linuxfoundation.org>
  - Update Orchestration scripts to work with Vex

    The Heat scripts as well as cleanup scripts all need to be recoded to
    work with Vexxhost.

    Issue: RELENG-705
    Issue: RELENG-707
    Change-Id: I5557e8fc7e760fb49da99eb62074b15039554bd5
Signed-off-by: Thanh Ha <thanh.ha@linuxfoundation.org>
  - Update mininet and docker builders

    The new cloud provider has different flavours than what we were using
    so need to update the flavours to work in the new cloud.

    Issue: RELENG-707
    Change-Id: I42617eba8c780fbf7152874937b9305b3e04b4b0
Signed-off-by: Thanh Ha <thanh.ha@linuxfoundation.org>
  - Update java-builder build-nodes for new cloud

    The new cloud provider has different flavours than what we were using
    so need to update the flavours to work in the new cloud.

    Issue: RELENG-707
    Change-Id: I86ba856d71c429c4310054614cd22e516bbd90c3
Signed-off-by: Thanh Ha <thanh.ha@linuxfoundation.org>
  - Update packer config to work in Vexxhost

    Improved the templates to support SOCKS 5 proxy configuration
    via the `ssh_proxy_host` configuration that is optional. This
    can be configured in the vars file.

    Additionally we are now prefixing image names with "ZZCI - " so that
    you can immediately tell from the name that the image is managed by
    our automated systems as well as having them sorted last. This should
    make it easier to manage images in the WebUI.

    Update packer version from 0.12.2 to 1.1.3 so that we can use the
    ssh_proxy_host parameter.

    Issue: RELENG-706
    Change-Id: I3b7200c6a917a961f9da38e5eeea95347ceed4a7
Signed-off-by: Thanh Ha <thanh.ha@linuxfoundation.org>

Update git submodules

* Update docs/submodules/releng/builder from branch 'master'
  to 591c0f4ac4524e773f3a841d4e406708d5a480af
  - Merge "Do RPM tests in RPM build job to gate publish"
  - Do RPM tests in RPM build job to gate publish

    We currently only run our RPM tests after the RPM build job finishes.
    This isn't good because by the time we've tested the RPM it has already
    been pushed to Nexus and is being consumed by downstreams. Instead we
    should fail the RPM build job if the tests fail and not publish.

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

Update git submodules

* Update docs/submodules/releng/builder from branch 'master'
  to 733e81e8225fc136517a3ac64b034641fe1551a6
  - Merge "Do RPM dep tests in snapshot build, minor cleanup"
  - Do RPM dep tests in snapshot build, minor cleanup

    Verify RPM dependencies are as expected during snapshot builds using the
    logic already included in normal builds. Also add verification that
    there's exactly one RPM at the path we expect so we don't claim there
    are dependency problems if there are actually RPM building problems (0
    RPMs) or something strange (>1 RPM). Also minor output cleanup.

Fixes: INTPAK-113
    Change-Id: Iac3a765924fffe7506f8cd6fbb0944bf9839f55a
Signed-off-by: Daniel Farrell <dfarrell@redhat.com>
Signed-off-by: Jamo Luhrsen <jluhrsen@redhat.com>

Update git submodules

* Update docs/submodules/releng/builder from branch 'master'
  to 719530a083b34930493c3f8ac81b8ff113c51a48
  - Update cloud image list docs

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

Update git submodules

* Update docs/submodules/releng/builder from branch 'master'
  to 99c81a81f8573f87affa9e1a68f92b83556480ce
  - Merge "Always keep copy of RPM in build out dir"
  - Always keep copy of RPM in build out dir

    The RelEng and sandbox silos need to behave differently with regard to
    uploading artifacts to Nexus (sandbox shouldn't, RelEng should).
    However, moving the artifacts in RelEng but not in the sandbox was
    causing the new dependency tests to work in the sandbox but not in
    RelEng because the artifact no longer existed at the path we expected.
    Instead of moving the artifacts, copy them.

    We needed to mv and not cp initially because we were still hosting on
    Jenkins and after a cp it found both old and new, but we no longer have
    that logic.

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

Update git submodules

* Update docs/submodules/integration/test from branch 'master'
  to fbb8033e310d966c22c7ae33ea3489058582485a
  - Copy dhcp files

    Depends-on: https://git.opendaylight.org/gerrit/66364

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

Update git submodules

* Update docs/submodules/releng/builder from branch 'master'
  to 84cafc4f0ce3196c5e4c4cc462ccecca975e8ca0
  - Merge "Copy dhcp files"
  - Copy dhcp files

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

Update git submodules

* Update docs/submodules/releng/builder from branch 'master'
  to 9ecdc90241405fc99e9da6ea9a23f86a2d295efc
  - Merge "Enable packer job for Ubuntu 16.04 GBP"
  - Enable packer job for Ubuntu 16.04 GBP

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

Update git submodules

* Update docs/submodules/releng/builder from branch 'master'
  to e740257ed26c2a5fe09ec69e7b2d89efb8392304
  - Update netconf jobs to use Ubuntu 16.04 GBP image

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

Update git submodules

* Update docs/submodules/integration/test from branch 'master'
  to a6fa4d75af4ccfcd28cf11cb96622b6aa510ce75
  - Add [Documentation] to openstack Suite Teardown kw

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

Update git submodules

* Update docs/submodules/releng/builder from branch 'master'
  to 8b643d16ee7e7638ed12e3b9ee9b2c94e4af1552
  - Fix HEAT userdata scripts

    The migration of the Gerrit server to a different environment has caused
    the startup scripts to fail. The issue is that the new location does not
    allow a ping. Instead, we will use the host command to do a DNS query
    since that tests for network connetivity just as well.

    Change-Id: I83755e7fca0afda7e8d71fa9598dc73b7db94853
Signed-off-by: Andrew Grimberg <agrimberg@linuxfoundation.org>

Updated git submodules

Project: releng/builder master 5a2c1a1d8fd167ac3e96fdc5f13a44edd59536a9

Merge "Revert "Use eval command to append mail options""

Revert "Use eval command to append mail options"

This reverts commit cbf45c195d170f6703b53e2bde7a9f97c96998b2.

Using eval for dynamic code expansion is not a best practice.
Ref: http://mywiki.wooledge.org/BashFAQ/048

The script also has portability issues which works well on Ubuntu 16.04
but not CentOS7 (due to diffirences in the version of bash) also
making the code less readable and difficult to maintain, therefore
reverting change.

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

Updated git submodules

Project: openflowplugin stable/carbon f5f97a55b0244b22d2f0e166d6aa0b1f2304c48c

Rename addDeleteOperationTotTxChain => addDeleteOperationToTxChain

to drop the 't' which looks like an obvious typo in this method

Change-Id: I0c7648f13bc1ea2312fa8fd31f8e9238260e731e
Signed-off-by: Michael Vorburger <vorburger@redhat.com>
Signed-off-by: Anil Vishnoi <vishnoianil@gmail.com>

Updated git submodules

Project: openflowplugin stable/carbon e82c78a419f644d1c774e25cc212678b6f41da6f

Merge "Lower logging level of FRM reconciliation process" into stable/carbon

Lower logging level of FRM reconciliation process

Change most of logging to match logging of OpenFlowPlugin sal services
(add and deletion and failures are logged on debug levels) to remove
unnecessary clutter from logs.

Change-Id: I6ae82c8a5125cf2ef35f4b2039f5d31d3f341948
Signed-off-by: Tomas Slusny <tomas.slusny@pantheon.tech>
(cherry picked from commit 1c45db329f7b7906b787b1ec066f88229ed57dc3)

Updated git submodules

Project: openflowplugin stable/carbon ea49d9c2d612ac0c5f1e79937e4f7df7edbcbea0

Merge "Add Docs for Openflowplugin" into stable/carbon

Add Docs for Openflowplugin

This patch just initializes sphinx for Openflowplugin

Change-Id: Ie7d0e2004d48e7ddacaf32ab18d0f94f5bc40c27
Signed-off-by: D Arunprakash <d.arunprakash@ericsson.com>
(cherry picked from commit ab322c98dcbc4df11382cf2e25dd65e93148314d)

Updated git submodules

Project: releng/builder master 2361d81beaa15731061872fe92821a213fe1cfb5

Increase wait for rabbitmq

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

Updated git submodules

Project: integration/test master 9e933e821f18ebed0ca54ba6ffca6e48ff936407

Add OpenStack grenade

Change-Id: I19d64b38dacd99c2b98a60260a5a4ba102b6255a
Signed-off-by: Sam Hague <shague@redhat.com>
Signed-off-by: Jamo Luhrsen <jluhrsen@redhat.com>

Updated git submodules

Project: netvirt stable/carbon 6116ca4f90a835edb5f440b13b7734db38e18429

NETVIRT-1023 - OptimisticLockFailedException: infrautils.jobcoordinator-impl

https://jira.opendaylight.org/browse/NETVIRT-1023

Looks like my patch below introduced a new exception.
 https://git.opendaylight.org/gerrit/#/c/65726/

Looks like only the TZ should be deleted, which will also delete
the VTEP. So, just delete the TZ.

Note: This exception only showed up in CSIT runs, did not see
any exceptions in local testing.

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

Updated git submodules

Project: releng/builder master be2c59333b8daf28d073c6a0e593590631150aa4

Auto Update CSIT Jobs to run for oxygen

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

Updated git submodules

Project: releng/builder master 8712701b1c971a668e71f84788b90a660e43b0fd

Merge "add csit stream for p4plugin"

add csit stream for p4plugin

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

Updated git submodules

Project: integration/test master ecf6176d4ab0bd097134ad3098dd2c927d9046e8

add project p4plugin system test job

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

Updated git submodules

Project: genius stable/carbon 08b088a54351d2b6cef16157e8c332df4c7f167b

GENIUS-102: interface manager: use transaction manager

Use the transaction manager to manage all write transactions.

Change-Id: If224c5193e4e9f4c12ebcaa1c31864646aafd026
Signed-off-by: Stephen Kitt <skitt@redhat.com>
(cherry picked from commit 12703b9cc6ce567154676eaf7388c4a2cc044f05)
(cherry picked from commit a04a0299da2709e07ec79737264d19ec4be11313)

Updated git submodules

Project: integration/packaging master 65eefb65b55b0136ab085d5c9b81a2d024955882

Fix typo s/match/matched

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

Updated git submodules

Project: releng/builder master 417035461954a2a53ae3c2c3037042cc92592f75

Merge "Add test for RPM requirements"

Add test for RPM requirements

After building an RPM, check its requirements against a master copy
from an RPM built on the CentOS Community Build System. This should
catch problems like INTPAK-10 in the future.

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

Updated git submodules

Project: integration/packaging master 9a0bfe3d4a185c9dc18c6336e5726c69419341f9

Add docs about RPM build publishing Nexus/logs

Add docs about how RPMs are published from build jobs running in
production Jenkins (Nexus) and sandbox Jenkins (optinoal logs archive).

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

Merge "Update BMP test tool argument doc" into stable/carbon

Updated git submodules

Project: integration/packaging master 4b1fdd40fc10f6bbb9d904919ce8ddf9398cf243

Blacklist false RPM dependencies

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

Updated git submodules

Project: integration/packaging master 0fb5b94da219365fb02c67651d57639eeed40884

Fix pkg unit tests, URL update

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

Update BMP test tool argument doc

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

Updated git submodules

Project: integration/test master 6c58b61fb1ef7bf98c635760d8f22ae98a5b9a24

Migrate Variables.py to Variables.robot

per our long lived task:
https://jira.opendaylight.org/projects/INTTEST/issues/INTTEST-31

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

Updated git submodules

Project: integration/test master 45530bc46e4da4655fdca00ff1172f7e18f98401

Stabilize netconf test

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

Updated git submodules

Project: integration/test master bb05669a1e2f1bc4e123a8ae5fa674b123a42538

bgp_functional_route test update

verify odl stats using restconf bgp operational-state
verify odl stats using odl-bgpcep-bgp-cli

Change-Id: I6d76b0cef0e832b7444b789906092ca6bc49e425
Signed-off-by: Tomas Markovic <tomas.markovic@pantheon.tech>

Updated git submodules

Project: releng/builder master ad3143474a1fff2f70d4a4068451488dd9cc0357

Merge "Use eval command to append mail options"

Use eval command to append mail options

Use eval to append mail options with the notify script. When the eval
command is used without escaping the variables causes wordsplitting.

Ref: http://mywiki.wooledge.org/BashFAQ/050

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

Updated git submodules

Project: releng/builder master 545a57351081c06b1d6cc576a9ed64cbde3c96d0

Merge "Make robot report logs better"

Make robot report logs better

- use more specific name for each pybot iteration which
  will help when looking at results via jenkins robot
  plugin

- start creating the full single log file again. We
  stopped doing this because our small 2G VMs could not
  handle the large log files. But, we are now using 4G
  VMs and it should be fine

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

Updated git submodules

Project: integration/test master 5b9e37523c30ae627f697d54cf66359c56376352

Change return code checker to be more clear

Should Be True    '${rc}' == '0'

  is more clear (to me at least) than

Should Not Be True    ${rc}

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

Updated git submodules

Project: integration/test master 31d2963f8fe13f2dcfb0410e6e78ee82c9245845

Do not restart/cleanup OVS in init code

Do not restart/cleanup OVS on compute node during
L2GW CSIT init.

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

Updated git submodules

Project: integration/test master e9237072333be7e4c11eca415f363e468eba7e97

vpnservice-ipv6: fix calls in neutron port

When calling neutron port creation, the calls are internal and do not
refer to external libraries.

Change-Id: Ib8322a065cad3bd6682326d2a1e542b15a13b183
Signed-off-by: Philippe Guibert <philippe.guibert@6wind.com>

Updated git submodules

Project: releng/builder master d2728da8c27b3fa350c24308ff364f2a469596c7

Merge "Configure tep-ip for genius auto tunnels"

Configure tep-ip for genius auto tunnels

Config tep-ip for control and compute nodes
as needed for genius auto tunnels to work.

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

Updated git submodules

Project: releng/builder master 832dc2bdf5c7012d8be8758b57ceb458ec8c9855

Revert "Use eval command to append mail options"

The original commit introduces a regression in our ability to send
emails to the correct mailing lists.

This reverts commit ea26afb7d635a615547490e05a7aef2d9bcda265.

Issue: RELENG-70
Change-Id: If493d40462d12edba2913971ba145a3d2b4a75e9
Signed-off-by: Thanh Ha <thanh.ha@linuxfoundation.org>

Updated git submodules

Project: openflowplugin stable/carbon d9d4102ab215b42ed4630e2d2ad9539cd114e860

Fix possible transaction leak in StatisticsGatheringUtils

see https://jira.opendaylight.org/projects/OPNFLWPLUG/issues/OPNFLWPLUG-961;
5x transactions opened here which were not closed

Change-Id: I1d9983a257881a7618943f76a6780a14c4e97e82
Signed-off-by: Michael Vorburger <vorburger@redhat.com>
bug-7826: Data validation failed for group-id

OPNFLWPLUG-858
Identified the root cause for issue TransactionCommitFailed for group during statistic collection.

Change-Id: I8fe639e6102629fb903689114c617e76a61d68db
Signed-off-by: sunilkumarms <sunil.g.kumar@ericsson.com>
OPNFLWPLUG-930 Inconsistent flow IDs between flows
 in config and operational data stores

Change-Id: I4ee8787e163a6ab64a4574f0dc906c7c91a7d0b2
Signed-off-by: Anil Vishnoi <vishnoianil@gmail.com>
Bump versions by x.y.(z+1)

Change-Id: I765b74e91d07f2b32ef5eb679ea04943844ba4e0
Signed-off-by: jenkins-releng <jenkins-releng@opendaylight.org>
Merge "BUG-9223:Remove hardcoded value of lldp interval" into stable/carbon

Merge "This patch implements ct-mark support in nicira extensions." into stable/carbon

BUG-9223:Remove hardcoded value of lldp interval

 Cherry-pick changes from stable/boron

  Addressed comments

Change-Id: Id4fe81e27663cf89af8d95be577826988deeb7fe
Signed-off-by: Jalpa Modasiya <jmodasiy@luminanetworks.com>
Fix possible NPE on ContextChainHolderImpl

- deviceConnected method

Change-Id: I7cbd8c3bb2a4d92ecdaa5fc541e01e3d8ac738b6
Signed-off-by: Jozef Bacigal <jozef.bacigal@pantheon.tech>
Signed-off-by: Anil Vishnoi <vishnoianil@gmail.com>
This patch implements ct-mark support in nicira extensions.

Specifically:
- support for ct-mark MATCH, including mask.
- support for ct-mark ACTION (set-field), but not including mask

This patch supports implementing ACL spec:
https://git.opendaylight.org/gerrit/#/c/60580

Change-Id: Ib828da39b4470e7172fe7028e8d3123fb184c742
Signed-off-by: Bertrand <bertrand.low@hcl.com>
Merge "Sort bucket actions" into stable/carbon

Do not try to close context with null deviceInfo

Do not try to lookup contextChain with null deviceInfo to prevent
NullPointerException.

Resolves: bug 9217

Change-Id: I8f5d860317375e20a7df9bb8ab5031af62c43936
Signed-off-by: Tomas Slusny <tomas.slusny@pantheon.tech>
Sort bucket actions

Sort bucket actions by order when serializing them in
GroupMessageSerializer.

Resolves: bug 9216

Change-Id: Icc5ad3f8b533a9f660e0248efed1b158c52336cf
Signed-off-by: Tomas Slusny <tomas.slusny@pantheon.tech>
Use HashedWheelTimer instead of item scheduler

Use HashedWheelTimer instead of items scheduler for scheduling the SLAVE
role sending to device and move this logic to device context, where role
management is done.

See also: bug 9089

Change-Id: Ia180b3d7704bd2096bfa0bee88c2e87da333cfaa
Signed-off-by: Tomas Slusny <tomas.slusny@pantheon.tech>
Merge "Solves issue with two connections from one device." into stable/carbon

Terminate SLAVE task before sending role change

- Terminate SLAVE task in device context when role change is requested and
not after role change is finished.
- Do not send notification about role failed to set to mastership
service in case future was cancelled
- Do not retry sending role to device 42 times

See also: bug 9089

Change-Id: Ia784d3b682b8991fe7a471bbfc7462aebfa6d4ef
Signed-off-by: Tomas Slusny <tomas.slusny@pantheon.tech>
Fix transaction manager closing.

- the event onTransactionFailed we properly closing chain and creating
  a new chain
- changed writeOnlyTransaction to readWriteTransaction
- moved transaction manager to the common module
- topology manager using transaction manager instead of creating a
  transaction chain for itself
- added control of unfinished transaction on close

See also: Bug-9038

Change-Id: Idadbb4ed0f4c61e7f80da5e2dbedbd80dece118e
Signed-off-by: Jozef Bacigal <jozef.bacigal@pantheon.tech>
Fix TransactionChainManager IllegalStateException

- Make txChainFactory volatile
- Close failed chain only if is current

See also: bug 9038

Change-Id: Ie897c5356addaef3f180a96296f5dd38950f2fb3
Signed-off-by: miroslav.macko <miroslav.macko@pantheon.tech>
Solves issue with two connections from one device.

See also:Bug 8723

Change-Id: I1e07eb590f285f9d6c1326b94ef5b159242c0757
Signed-off-by: Jozef Bacigal <jozef.bacigal@pantheon.tech>
Fix context state comparison

Fix comparison of context state and atomic reference of context state.

See also: bug 8668

Change-Id: Ie001d913982dc1331d1bb418e3fc8c7208e4c52d
Signed-off-by: Tomas Slusny <tomas.slusny@pantheon.tech>
Bug 8882 - With conntrack SNAT communication with PNF fails

Added support for load metadata action.

Change-Id: Ia1b0454a4aeccd9a04cecb59b50f516264bddb3e
Signed-off-by: Aswin Suryanarayanan <asuryana@redhat.com>
Close CSS registration in separate thread

Instead of sending initialization of CSS in separate thread,
do closing of registration in separate thread and only in case
we are closing the registration during initialization, to prevent
CSS freezes. Also, when closing CSS service, add checks for connection
state to prevent closing context when connection is already RIP.

See also: bug 8805

Change-Id: I0759f7913937c897f41899a87b699989cdfe4abc
Signed-off-by: Tomas Slusny <tomas.slusny@pantheon.tech>
Fix context chain initialization and SLAVE change

- Merge ContextChain and LifecycleService
- Let ClusterSingletonService handle stopping of mastership services in
closeServiceInstance method
- Store cluster singleton initialization future and process it
synchronously in case we are closing
- Cancel device initialization future if we got timeout
- Handle initialization chain order in ContextChain instead of storing
each next phase in contexts
- Remove SLAVE task from scheduler in case we acquired SLAVE role
already
- Prevent adding exactly same auxiliary connection twice to list of
auxiliary connections in ContextChain
- When SLAVE role is obtained from cluster singleton service, notify
MastershipChangeListener about it
- Log exceptions from handshake as warning instead of error (as only
problem is that device gets disconnected and reconnected, what is not
critical)
- In case context chain is still closing and we receive new connection,
drop the connection and let context chain finish closing

See also: bug 8805

Change-Id: If14da19899087018c5e6cc42d195af43c1e89bfd
Signed-off-by: Tomas Slusny <tomas.slusny@pantheon.tech>
Merge "Fix match extensions deserialization" into stable/carbon

Merge "Improve flow collection" into stable/carbon

Fix match extensions deserialization

Fix transformation of match extensions for different match types during
deserialization so they will not be lost during this process.

Resolves: bug 8836

Change-Id: If089a72759a26b764479e58ea112b3a1a062c90e
Signed-off-by: Tomas Slusny <tomas.slusny@pantheon.tech>
Bump versions by x.y.(z+1)

Change-Id: I0576e16aabc016896b51ec4538b453e66da709c9
Signed-off-by: jenkins-releng <jenkins-releng@opendaylight.org>
Improve flow collection

- Delete all known flows synchronously
- Add boolean isTransactionsEnabled to TxFacade to determine if it was
initialized and initial submit was called
- Improve deletion of flows during on the fly statistics gathering (now,
  each gathering have 3 states, INITIALIZATION, WORKING and TERMINATION
  and behaves according to that)
- Fix possibility of statistics collection freeze when error occurs
  during OF Java to OF Plugin model translation while gathering
  statistics on the fly

Resolves: bug 7501

Change-Id: I74843d4536215c192ac43964634502430da1aa9a
Signed-off-by: Tomas Slusny <tomas.slusny@pantheon.tech>
Solves compilation error

- due openflow java changes plugin run in compilation errors

Resolves bug:8788

Change-Id: I41f964260c216fe3ee417c9d874cb58177f4596d
Signed-off-by: Jozef Bacigal <jozef.bacigal@pantheon.tech>
Fix NPE in ConnectionContextImpl

- When we receive port status message, in some cases nodeId can be null,
  so when logging, get nodeId safely and in case it is null, just log
  "null"
- When we are closing outbound queue, log value of nodeId or null in
case nodeId is null

Resolves: bug 8762

Change-Id: Ifda277fd298f7d30fd5e94f07a94e67813971029
Signed-off-by: Tomas Slusny <tomas.slusny@pantheon.tech>
Revert "Remove lifecycle service from tx chain manager"

This reverts commit 5bbffc61f90c85d6360a73c5eab35e203f6f20fd.

Change-Id: I793f7448803519e0dcdd8c5446e555fd8d6dfa15
Signed-off-by: Jozef Bacigal <jozef.bacigal@pantheon.tech>
Remove lifecycle service from tx chain manager

Resolves: Bug8748

Change-Id: I6e9f1d7dbdfbdcca5cd52c368985ba7d37310a74
Signed-off-by: Jozef Bacigal <jozef.bacigal@pantheon.tech>
Merge "Report (TCP) port number for switches" into stable/carbon

Fix context chain closing

- Close and create EOS registration correctly when closing context
chain holder and creating context chain holder.
- Cancel running scheduled task on context chain holder termination
- Always remove device from DS when we was not able to propagate roles,
not only when we have this device already in context chain map
- Add toleration for adding context chains to queue
- Wait for registration of RPC services before fully granting mastership
for context chain
- Close transaction chain manager on device context close
- Remove unnecessary auxiliary connection gathering in device context
(because this was moved to context chain)
- Fix closing of TxChain when DeviceContext is closed when activating tx
chain manager but not enabling submit transaction (do not try to submit
data)
- Unregister RPC registrations on RpcContext close
- Do not close ClusterSingletonServiceRegistration during
ClusterSingletonService initialization, to prevent freeze of
ClusterSingletonServiceGroup, send it to separate thread
- Prevent propagating of connection closed event twice (add checks for
RIP connection state)
- Fix ignored tests
- Remove getter for table statistics boolean from DeviceContext and just
send it to DeviceInitializer
- Close all RPC contexts when RPC manager is closed
- Do not reschedule statistics one more time when context chain is
closed

Resolves: bug 8668

Change-Id: Ifa8c492486c17a73eccd760fc610a435ef6a42ed
Signed-off-by: Tomas Slusny <tomas.slusny@pantheon.tech>
Report (TCP) port number for switches

Write also port number to datastore for OpenFlow switches.

Resolves: bug 2188

Change-Id: I15054b55907b4f15b1567b21f5fa2d35ad416579
Signed-off-by: Tomas Slusny <tomas.slusny@pantheon.tech>
Fix "stale" state after controller disconnected.

See also: Bug 8411

Change-Id: If5bf24f34a87ef0a7d5514b1c655a42508889724
Signed-off-by: Jozef Bacigal <jozef.bacigal@pantheon.tech>
Ensure that statistics gathering is terminated

Store current statistics gathering future for cancellation instead of
settable future, so we can cancel it when we need to (there is no point
in cancelling settable future other than preventing more statistics
running, but current in-progress statistics gathering will still try
to finish).
Cancel statistics gathering also when become slave.

Resolves: bug 8635

Change-Id: If221a34901b08f8923ae5321a51abd00f4b047b1
Signed-off-by: Tomas Slusny <tomas.slusny@pantheon.tech>
Merge "Fix connection when slave role request is unsupported" into stable/carbon

Fix connection when slave role request is unsupported

- Allow the switch connection when OFPRRFC_UNSUP is received
- Set feature exception for getGenerationIdFromDevice RPC error
- Add unit test

Resolves: bug 7664

Change-Id: If8d7d045fb516f55346333cc7e60d0ccb2135efd
Signed-off-by: miroslav.macko <miroslav.macko@pantheon.tech>
Fix port update

- Update yang models. Add grouping common port with mask.
- Use config mask in the PortConvertor and PortMessageSerializer.
- Update unit tests.

Resolves: bug 4747

Change-Id: If558fea34f86c026ca45ba2853107c2cd82c7f50
Signed-off-by: miroslav.macko <miroslav.macko@pantheon.tech>
(cherry picked from commit 8d18002e59e2b7e7fca95e52d2cb4a75d1023117)

Merge "Fix 'INPORT' keyword in port field" into stable/carbon

Merge "Fix onSwitchIdleEvent echo request-reply xid" into stable/carbon

Fix 'INPORT' keyword in port field

- also fix NPE in serialization by using wrong port number

Enumeration from opeflow java yang files is correct used keyword 'IN_PORT'.
Openflowplugin with the wrong translation (getString instead of getName) used the
keyword 'INPORT'. This fix use the correct name from enumeration but allow use
the legacy 'INPORT' keyword for next release.
Also prevent to throw NPE by using wrong keyword in port field.

Change-Id: If8b3d301d73169fd158a546a233928ab0c65d1c3
Fix: Bug 2095
Signed-off-by: Jozef Bacigal <jozef.bacigal@pantheon.tech>
Merge "Bug 8253: Set-Field can not accept vendor extension fields." into stable/carbon

Fix onSwitchIdleEvent echo request-reply xid

Solves: Bug-4422

Change-Id: I879e9e6aaf560da486767a68a7e2f015db5a7d95
Signed-off-by: Jozef Bacigal <jozef.bacigal@pantheon.tech>
Merge "Write port statistics when needed" into stable/carbon

BUG-8607: Replace logger and log by LOG

Replace logger and log by LOG to follow the
OpenDaylight recommendations [1].

[1]
https://wiki.opendaylight.org/view/BestPractices/Logging_Best_Practices

Change-Id: I77ef621927ba05417736d24c18c6659c7f504917
Signed-off-by: David Suarez <david.suarez.fuentes@ericsson.com>
Make create-parents in bulk-o-matic not required

To make bulk-o-matic input working like before, make create-parents
to default to true to use original functionality with same input
as it was before patch that introduced this new flag.

Change-Id: I89ee7d299f0ade62bd6859209c537528e5728aa7
Signed-off-by: Tomas Slusny <tomas.slusny@pantheon.tech>
(cherry picked from commit d13de0fa783ab11e33a58417007be0d855be84ae)

Merge "Bug 8293: Add table writer to bulk-o-matic" into stable/carbon

Merge "Bug 8647: remove odl-dlux-core as artifact POM" into stable/carbon

Merge "HeaderDeserializer and HeaderSerializer for codecs" into stable/carbon

Merge "Close OpenFlowPluginProvider during shutdown" into stable/carbon

Bug 8647: remove odl-dlux-core as artifact POM

odl-dlux-core is used in dependency management as an artifact POM, but
it isn’t. Removing the declaration is fine since the actual dlux
dependencies specify the version to use.

Change-Id: I83598feb8175d674616a3a3f25c0d52ca1739664
Signed-off-by: Stephen Kitt <skitt@redhat.com>
BUG 7332: Cleanup queue after switch disconnect

In context of Helium Plugin, ingress queue in
ConnectionConductorImpl is not closed, which results in the memory
leak.

Change-Id: I3e69721114f887f8b71c22224a2a834d69a64a43
Signed-off-by: Vijay Daniel <vijay.daniel@ericsson.com>
Merge "Optimize port status and hello message handling" into stable/carbon

Close OpenFlowPluginProvider during shutdown

- Call close method on OpenFlowPluginProvider during shutdown
- Close all created managers, thread pool and timer during shutdown
- Remove unused notification service
- Remove warning when unknown property is loaded on start (because of
  recent FRM changes)

Resolves: bug 8598

Change-Id: Ib5a4f84ea1fa0c957b90ef216346eb85aec81ad0
Signed-off-by: Tomas Slusny <tomas.slusny@pantheon.tech>
Stop reschedule stat. after device disconnected

Resolves: Bug-7940

Change-Id: Ida42676ec5bf6f06fb47837c0532878976eed442
Signed-off-by: Jozef Bacigal <jozef.bacigal@pantheon.tech>
HeaderDeserializer and HeaderSerializer for codecs

Implement HeaderDeserializer and HeaderSerializer for nicira
match codecs.

Resolves: bug 4862

Change-Id: Ia3dbbf622460ecfa3b6a2c48e94e432294a08c0e
Signed-off-by: Tomas Slusny <tomas.slusny@pantheon.tech>
Write port statistics when needed

When we receive port statistics reply, sometimes in OF 1.0 we do not
have parent node written to datastore, so now when OpenFlow version is
1.0 then write those port statistics with parents if needed.
Replace gathering missing port statuses with PORT_DESC with registry
that will collect missed port statuses when on MASTER.

Resolves: bug 8239, bug 6908

Change-Id: I47468cb37fa59a68cfa89efeb7e5c4191e07039d
Signed-off-by: Tomas Slusny <tomas.slusny@pantheon.tech>
(cherry picked from commit acf2b5630ca4460bac2343bccc72f37da12f6e0d)

Add Nicira extension support for matching IPv6 Src/Dst

This patch implements the necessary support for programming a reg_move action
(i.e., move:NXM_NX_IPV6_SRC[]->NXM_NX_IPV6_DST[]) with IPv6 source/destination
headers. It will be used while constructing OVS flows that auto-respond to ping6
requests addressed to a router-interface.

Change-Id: I64757f55ed1d30a12217c4a5aadaa3f56572ab0e
Signed-off-by: Sridhar Gaddam <sgaddam@redhat.com>
(cherry picked from commit 7d00489cd0b6014b58b68fc84c533f6d33fc8dc1)

Fix auxiliary connections

See also: Bug-6755

Change-Id: If4964c1a0f07bfdd8104c7254f4b4484fd42d81e
Signed-off-by: Jozef Bacigal <jozef.bacigal@pantheon.tech>
Optimize port status and hello message handling

- Reduce number of created threads and close used threads.

Change-Id: Ib37268be5e7982c64b95da9abffa6db78ef4edc1
Signed-off-by: miroslav.macko <miroslav.macko@pantheon.tech>
Merge "Fix logging of exception in HandshakeListenerImpl" into stable/carbon

Merge "Fix no reserved xid" into stable/carbon

Merge "Bug 8497 - Provide config knob to disable the Forwarding Rule Manager reconciliation" into stable/carbon

Merge "Add missing convertors for OF1.0 actions" into stable/carbon

Bug 8497 - Provide config knob to disable the Forwarding Rule Manager reconciliation

Resolves: Bug 8497, Bug 7957

Change-Id: Ib798d9282f019e0c8dd520c15d8794678ee9af3d
Signed-off-by: Anil Vishnoi <vishnoianil@gmail.com>
Merge "Remove unused method" into stable/carbon

Merge "Bug 8535: Fix IPv6 OXMHeader Mask issue" into stable/carbon

Merge "Though shall not use org.eclipse.tycho's osgi.. you don't even need it!" into stable/carbon

Merge "Adding Reason Attribute to FlowCapableNodeConnector" into stable/carbon

Add missing convertors for OF1.0 actions

- SetDlSrc
- SetDlDst
- SetNwSrc
- SetNwDst
- SetNwTos
- SetTpSrc
- SetTpDst
- SetVlanId
- SetVlanPcp

Resolves: bug 8527

Change-Id: I9c7c97804d077e1089fe92b3465c826a43da6a33
Signed-off-by: Tomas Slusny <tomas.slusny@pantheon.tech>
Bug 8253: Set-Field can not accept vendor extension fields.

The various Set-Field actions are identified by their field type
which defined in OpenFlow Specification. It will be failed if we
pass a vendor extension field to Set-Field, that means Set-Field
can not accept those vendor extension fields. There are products
that hope Set-Field can accept those vendor extension fields, so
it will make sense to make Set-Field extensible.

Change-Id: Id25456d2b9008ec260ce81d3a459bdc471a69e49
Signed-off-by: Li guosheng <li.guosheng6@zte.com.cn>
(cherry picked from commit 4b3a659ef96aaca99c0fbd17ccdf9411c2e12da3)

Bug 8535: Fix IPv6 OXMHeader Mask issue

When programming an OVS flow to set an ipv6_src/dest as
an action field (i.e., set_field:2001::1->ipv6_src), the
current implementation is not checking the Mask and is
including the Mask (even for a full ipv6 address) in the
OXM header. Because of this, switch is rejecting the flow
with the following error.

"decode error: OFPBAC_BAD_SET_MASK. OXM header 0:32768:26:1:32
includes mask but masked OXMs are not allowed here"

This patch addresses this issue by checking the prefix and
including the prefix only if applicable.

Change-Id: Ieffdc7f0e74e9418156171d85a28c24dd336527c
Signed-off-by: Sridhar Gaddam <sgaddam@redhat.com>
(cherry picked from commit 85285353da4c397a9b4ad0e67ba137cddb0c4740)

Bump versions by x.y.(z+1)

Change-Id: Ifa2f0fe59824706e9bc78b3fab1b2830335b8bee
Signed-off-by: jenkins-releng <jenkins-releng@opendaylight.org>
Adding Reason Attribute to FlowCapableNodeConnector

-The reason attribute is added to the FlowCapableNodeConnector Yang
Modelling.
-The PortStatus contains reason when notification comes from Switch such
as ADD, UPDATE, DELETE.
-This reason flied will help in distinguishing between different
PortStatusMessages
-Also changed TestCase File to include Reason for test cases

Change-Id: If61fc1ba0afbe56cbf5382ab3997c67166989819
Signed-off-by: GauravBhagwani <gaurav.bhagwani@ericsson.com>
(cherry picked from commit 9af18614bfef2d765b61564096afe2c8a360bbb5)

Though shall not use org.eclipse.tycho's osgi.. you don't even need it!

see https://git.opendaylight.org/gerrit/#/c/51592/
see https://git.opendaylight.org/gerrit/#/c/51520/

There are 2 remaining use of org.eclipse.tycho in openflowplugin's
drop-test and test-provider, something related to console commands...
could we ask you if you would mind to change that to use
the Karaf Console API instead?  Or delete those? ;)

I'll like to block use of dependency tycho in odlparent ASAP.

Change-Id: Ifccbd7f6ec463487e83aa546701cdffd1c7accb4
Signed-off-by: Michael Vorburger <vorburger@redhat.com>
Merge "Add timeout for device initialization" into stable/carbon

Fix logging of exception in HandshakeListenerImpl

Exception was not logged, so in case initial processing failed and
thrown exception, there was no exception actually shown in log. To
properly log exception, it needs to be either passed as second parameter
or inserted into message via logger string interpolation ({}).

Change-Id: Ic76f6d20588e34837994b50509603d994009b705
Signed-off-by: Tomas Slusny <tomas.slusny@pantheon.tech>
Merge "Remove device without master from DS." into stable/carbon

Merge "Bug 8408 - Deserialization exception in logs when NAT flows are added." into stable/carbon

Add timeout for device initialization

Sometimes during device initialization, switch starts ignoring all
messages from plugin, and so we ended up with Future that waited
indefinitely for switch response until Karaf was restarted.
Add timeout that will prevent happening of above issue.

See also: bug 8401

Change-Id: I90c6c8b53f6a6e879b6d9f93c259a2cbf09f71d3
Signed-off-by: Tomas Slusny <tomas.slusny@pantheon.tech>
Remove device without master from DS.

See also: Bug-6459

Change-Id: I6d27681b60a75df173a49742885c75c16d691b26
Signed-off-by: Jozef Bacigal <jozef.bacigal@pantheon.tech>
Bug 8408 - Deserialization exception in logs when NAT flows are added.

The code is modified to fix the issues with multiple CT actions.

The logic skip padded bit is added during Deserialization

Change-Id: Ic5574f0a877228d966ca92eacfa22c751fa14b3c
Signed-off-by: Aswin Suryanarayanan <asuryana@redhat.com>
Fix queue and xid issue

Change-Id: Idef52a9243d9b2b9ac7028386422f0acc580c07d
Signed-off-by: Jozef Bacigal <jozef.bacigal@pantheon.tech>
Fix no reserved xid

Change-Id: I153df3f1d3df895e2780a77dd6ed04e43bbc9599
Signed-off-by: Jozef Bacigal <jozef.bacigal@pantheon.tech>
Remove unused method

Change-Id: I02e3d48f2ccca528995b18cb159700ab08449862
Signed-off-by: Jozef Bacigal <jozef.bacigal@pantheon.tech>
Bug 8293: Add table writer to bulk-o-matic

* Table writer is introduced for Pre-Leader Role test in which
  table is added first and flows next. We want the ability to
  create tables separately and not to be created automatically
  on addition of flows.
* Modify flow-test rpc to accomodate a boolean: create-parents.
  This allows us to prevent addition of flows in case the tables
  are missing.
* Fix import ordering

Change-Id: Ic3fe4764631d9cbc7550ad3d7d926debb3ca6977
Signed-off-by: Faiz Ilahi Kothari <faiz.ilahi.k.kothari@ericsson.com>
(cherry picked from commit 2f8d414a2b42bee7504c6fd78536bc6fd1049b83)

Handle missing converters for match extensions

Prevent throwing NullPointerException when converter for match
entry extension is missing, instead throw warning and allow process
to continue (like it was before single-layer-serialization).

See also: bug 8401

Change-Id: If006eef401da4b9bf61911c183e76876fac0dfcf
Signed-off-by: Tomas Slusny <tomas.slusny@pantheon.tech>
Fix no match serialization

This patch ensures that even when match in flow is null, at least empty
match is sent to device.

Change-Id: If77c59b889e45698da4b96f098088cdbdf4698ee
Signed-off-by: Tomas Slusny <tomas.slusny@pantheon.tech>
Prevent changing single-layer-serialization on fly

Prevent changing use-single-layer-serialization config
parameter on the fly, allow it only during initialization.

Resolves: bug 8278

Change-Id: Idfd2f7aa99fdf330bc32615431b4378209c83fda
Signed-off-by: Tomas Slusny <tomas.slusny@pantheon.tech>
Improve property-based configuration

- Add OpenFlowPluginConfigurationService. This service can control
  updating of OpenFlowPlugin configuration (skip-table-features etc.)
  in initialization phase and also on the fly.
- Comment out default settings in org.opendaylight.openflowplugin.cfg
  because with all default values set in that file, we are ignoring
  values specified in openflow-provider-config.yang.
- Add support for missing configuration values from
  openflow-provider-config.yang to OpenFlowPluginConfigurationService
- Remove changeable configuration values that was passed to some
  OpenFlowPlugin class implementations to both constructors and setters
  from constructors to prevent code duplicity and to simplify code a bit

Now, when feature is installed,
default values are taken from openflow-provider-config.yang, then we
check if we have org.opendaylight.openflowplugin.cfg file in karaf/etc/
(blueprint will do this for us) and try to read it as map. Then we will
pass this map to update method, that will iterate over this map and
parse it's keys and values and calls appropriate update methods in
OpenflowPluginProviderImpl based on key
(f.e skip-table-features will call
OpenflowPluginProviderImpl#updateSkipTableFeatures).

When org.opendaylight.openflowplugin.cfg file is changed, we receive
notification about it in form of
OpenFlowPluginConfigurationService#update method call, and we will
repeat same procedure as during initialization (parse map passed as
argument to this method and update configuration).

Resolves: bug 7218

Change-Id: Ida986f7dfffe55a90ddb6f354f0addf56c3c4b90
Signed-off-by: Tomas Slusny <tomas.slusny@pantheon.tech>
(cherry picked from commit f1cc4de7dee9e22c231a90cf517b03bc5071ad35)

Ignore early port status msg and process it later

When port status message arrives before we got MASTER role,
ignore it. Then, after receiving mastership collect PORT_DESC statistics
again to make sure we do not missed any port status messages in process

Resolves: bug 6908

Change-Id: I01285ed4880533f48bf90fa7c6a4ca2c7d3a210f
Signed-off-by: Tomas Slusny <tomas.slusny@pantheon.tech>
(cherry picked from commit 426392ea39557e58c0cec51eae3bc91a0bebdf6f)

Merge "Bug 8223: Fixed incorrect enable-flow-removed-notification check." into stable/carbon

Bug 8223: Fixed incorrect enable-flow-removed-notification check.

flow-removed notification needs to be published if
DeviceManager.isFlowRemovedNotificationOn() returns true.

Change-Id: I10c025f11e14670303b70bf31d3a9cefa5a676ec
Signed-off-by: Shigeru Yasuda <s-yasuda@da.jp.nec.com>
Bug 8217: Set error information into direct statistics RPC result.

Change-Id: I3fdf51b94bf95afefa47dddea37a82b8f139af95
Signed-off-by: Shigeru Yasuda <s-yasuda@da.jp.nec.com>
Update .gitreview to stable/carbon

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