Updated git submodules

Project: releng/builder master 883f4927d94a0b216f8cfc7db3ff5c6e4499071e

Add odl-jolokia to bgp-ingest job

It is required for reset and new suite using
jolokia.

Change-Id: Iaa160c62bfb9c6eed5feeab7b85917d261275cff
Signed-off-by: Peter Gubka <pgubka@cisco.com>

Updated git submodules

Project: integration/test master bcae39cf1222c30eb3bd0245299a02e78603d8e5

Mark TODO as comment

Change-Id: I8cff5d74db32f15a4133dd26d13ccc2c108be276
Signed-off-by: Peter Gubka <pgubka@cisco.com>

Updated git submodules

Project: netvirt stable/carbon ef8c8b711e4a2a59e80224de181e91e17196a3a0

Bug 8743 : Fix NAT Service to safely use vpn-to-dpn list.

Description : Its been observed the vpn-to-dpn list been not updated
properly resulting in missing dpn-state with active state. As a result,
installation of default route rule been missing resulting in SNAT/DNAT
failure.

Changes are done to make use of vpn footprint service to update the
vpn-to-dpn list whenever floating-ip and external fixed ips are added
and deleted.

Change-Id: Ic55fc5364ef45fbbe7ce63a9feef51bf39332541
Signed-off-by: Vivekanandan Narasimhan <n.vivekanandan@ericsson.com>
Signed-off-by: cgowdru <chetan.arakere@altencalsoftlabs.com>

Updated git submodules

Project: netconf stable/carbon 32621d57701de889651097ce5c1bfc3bb5f43ce9

BUG-8697: select correct transformer for schemaless netconf mounts

Getting or editing the configuration of a schemaless-mounted netconf device
from the code of an ODL application results in a NullPointerException.

This patch fixes the NetconfBaseOps constructor so that the RPC structure
transformer dedicated to schemaless mount points is actually selected.

Change-Id: Ib0b2d1c9910613f4354bdcb4f1c56e9be36a95b2
Signed-off-by: Gwenael Lambrouin <gwenael.lambrouin@b-com.com>

Updated git submodules

Project: releng/builder master c91130bc5339b0ca04884e78a884cd8bcbc1b7af

Enable trigger with no parameters

If a build trigger does not pass any parameters we need to set this to
true or the parameterized build trigger will skip the trigger.

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

Updated git submodules

Project: integration/test master 3d722df36f8e922fe79d30afc1516346723474ae

Do not call http status 500 a validation error

+ Variables table in TemplaterRequests sorted and better commented.
+ Comments pointing to 400 usually being better code to expect.
+ Utils resource now contains keywords for both 400 and 500.
+ Comments pointing to TemplatedRequests instead of Utils.

Change-Id: I0b44cc69f84d7d882f643d6cbd7535489e1a2709
Signed-off-by: Vratko Polak <vrpolak@cisco.com>

Updated git submodules

Project: releng/builder master 80e8eaec420426f39113edf5c493996bb6fa9193

Merge "Trigger rel-notes job from version-bump job"

Trigger rel-notes job from version-bump job

Release notes jobs is only useful with the properties file
updated done as post release activity. Therefore runing this job
at the end of AR job is not very helpful. Move this trigger
to AR version bump job.

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

Updated git submodules

Project: integration/test master d0409f16ff1815489c3feecd0f226645f7e580f6

Add new test for multiple switch connection

Change-Id: I167bf6c0c5ab48102f53ac46f69725063a111844
Signed-off-by: Len <lgiersch@brocade.com>

Updated git submodules

Project: integration/packaging master 6fcb94e430838a91e4336df38e5f69a166935d98

Add Boron SR4 example RPM repo config

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

Updated git submodules

Project: releng/builder master 34c0a4decedded19d47c841436b2f1be623a88a3

Change OFP UI feature to REST feature

dlux project is not active anymore (maintenance mode).

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

Updated git submodules

Project: integration/test master 5a5cc289dc6de4ed420ba4a06f43a16bf2e75d74

Changes predefined flows

- openflowplugin changes the port described as INPORT to IN_PORT

Change-Id: Ibc0011d5e3cbde75a8d1e6fb24d7eccd37b4b94e
Signed-off-by: Jozef Bacigal <jozef.bacigal@pantheon.tech>

Updated git submodules

Project: integration/packaging master a6917d19500734494f897afb751f2835383b21bf

Various minor docs fixes, link additions

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

Updated git submodules

Project: releng/builder master b8a9171a21356cc75ed35252b87b60e89857fea5

Merge "Create odlparent 2.0.x maintenance jobs"

Create odlparent 2.0.x maintenance jobs

With odlparent releasing out of simrel we need to create new maintenance
jobs for odlparent 2.0.x. This change does the following:

- Renames nitrogen jobs to master
- Add new odlparent 2.0.x jobs
- Remove {project-name}-integration-{stream} jobs
  (odlparent does not use them as they do not have any upstream)
- Configure distribution_branch for both master and 2.0.x to master

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

Updated git submodules

Project: integration/test master 013e27b9764602c3b5ff9a37c04cac54be70eb36

Bug 8629: Do not remove replica in listener suites

Current Carbon implementation of cluster-wide listener
relies on local replica being available.
When the replica is shut down (or otherwise deactivated),
listener stops getting data change notifications.

This makes the suite perform explicit leader movement,
so every member still has its local replica.
As we can now control which member becomes new leader,
there is one more test case.

Also, the suite was renamed to better match what happens during test.

Change-Id: I3cccd2581e6a4612badd7b53a600da2e1d0f1794
Signed-off-by: Vratko Polak <vrpolak@cisco.com>

Updated git submodules

Project: integration/test master 11827b2bab19e12ffa5f1d970198f6044aa75560

Remove nic vtnrenderer csit

per this email, vtnrenderer csit is no longer needed:
https://lists.opendaylight.org/pipermail/nic-dev/2017-June/001713.html

ideally, this patch will be merged after this patch in releng/builder:
https://git.opendaylight.org/gerrit/#/c/59435/

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

Updated git submodules

Project: genius stable/carbon 208655aa7da2c58008f06b824b9aa528c4b50a2e

Genius wide blueprint xml path change: /OSGI-INF/ -> /org/opendaylight/

Also have removed fake "availability=optional"

Change-Id: Ia4c21bbba98f01d938c97990f2a02dd9996a8490
Signed-off-by: Abhinav Gupta <abhinav.gupta@ericsson.com>
(cherry picked from commit 7e715d89baf71179084fbad83c11886c32ba480d)

Updated git submodules

Project: releng/builder master aff9e44f33a568fadc9c697c78e6a9d571e0bac8

Auto Update CSIT Jobs to run

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

Updated git submodules

Project: releng/builder master e0bd0b45eb352a3ae8b14e477087bf9188681889

Remove nic vtnrenderer csit

per this email there is no longer a need to keep the vtnrenderer
csit:
https://lists.opendaylight.org/pipermail/nic-dev/2017-June/001713.html

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

Updated git submodules

Project: integration/test master 5f53fc0f287f26c07905f1e7efd239a0e75d68b0

Fix HWVTEP CSIT openstack commands

Fix syntax in "openstack network create" command and
"openstack security group rule create" command.

Change-Id: Ib2f692c0a2422bd85afbab4786599c309e6103f2
Signed-off-by: Victor Pickard <vpickard@redhat.com>
Signed-off-by: Jamo Luhrsen <jluhrsen@redhat.com>

Updated git submodules

Project: releng/builder master 877d0e69d47d2d6ee515ef319f7e286f0ca4b36d

Merge "Execute Openstack Client from Robot VM"

Execute Openstack Client from Robot VM

- Install Openstack client
- Create the environment variables required to execute the
  Openstack client.

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

Updated git submodules

Project: releng/builder master d10b27d582d4f84a84ab7c0a0a8d9abd34afc51f

Merge "Bump release tag for next Boron release"

Bump release tag for next Boron release

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

Updated git submodules

Project: netconf stable/carbon 788891384d75bf9b9d113117e8a374cdb5f2440c

Merge "Fixing NPE caused when rpc definition does not include "input" or "output"" into stable/carbon

Fixing NPE caused when rpc definition does not include "input" or "output"

Change-Id: I90cd8da7abfbb24015a87c1b2a7cf7dc3f047b4a
Signed-off-by: Srini Seetharaman <srini.seetharaman@gmail.com>

Updated git submodules

Project: netvirt stable/carbon 8b517bd581abed2b0c083e1caeb4d5ec225dfad9

Move fibmanager-shell blueprint.xml from OSGI-INF/blueprint to org/opendaylight/blueprint

This may (?) fix the SFT failing to find the DataBroker, as discussed on follow-up to
https://lists.opendaylight.org/pipermail/netvirt-dev/2017-June/004770.html; and if
this does fix that SFT problem, then this was an indirect impact of
https://git.opendaylight.org/gerrit/#/c/54885/.

This also reverts both https://git.opendaylight.org/gerrit/#/c/59105/ private field,
and the availability="optional" from https://git.opendaylight.org/gerrit/#/c/59051/

Change-Id: I856c0f4311a02a5380512496619aee74cf70ee2e
Signed-off-by: Michael Vorburger <vorburger@redhat.com>
(cherry picked from commit b0a5086f16acf9d9bd5a38eea1a0715625744aa0)

Updated git submodules

Project: integration/test master d86b9852a3db706d878ac6fbd7b6ecfe6ec03896

Use shutdown(-prefix)-shard-replica in ddb testing

Change-Id: I79fae058c9980e661702c562a9a629580e5224a2
Signed-off-by: Peter Gubka <pgubka@cisco.com>

Updated git submodules

Project: integration/test master 971b82aed16a2992bd5a5fd8f9b17d4979e4ba5c

Add shutdown(-prefix)-shard_replica rpc into lib

Change-Id: I3d3508cb0063287138d804fe7c9a1731abc9b393
Signed-off-by: Peter Gubka <pgubka@cisco.com>

Updated git submodules

Project: netvirt stable/carbon 88673d3013e6fa99cdab2f6bb1218d82cd98f49e

Netvirt wide blueprint xml path change: /OSGI-INF/ -> /org/opendaylight/

Also have removed fake "availability=optional"

Change-Id: Iae7caa155eeffa7c831822ac600bbc6283a7e2fc
Signed-off-by: Abhinav Gupta <abhinav.gupta@ericsson.com>

Updated git submodules

Project: releng/builder master 4cd26a1e974d6346b73380a171ca9559dcc6b9a8

Add kuryr.conf as part of the log collection

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

Updated git submodules

Project: integration/test master f12358fd0327983c15a646e43fecf45cce4b924d

IPv6: support in basic vpn_service tests

Add 03_basic_ipv6_vpnservice.robot to test L3VPN with IPv6 subnets and
extra routes.

Add 03_basic_ipv6_vpnservice.robot in
netvirt-1node-openstack.txt test plan, to launch it automatically on a
regular basis.

Change-Id: I5a9245b2723b46234fcb2f9baf66e7f95057dfab
Signed-off-by: Valentina Krasnobaeva <valentina.krasnobaeva@6wind.com>
Acked-by: Philippe Guibert <philippe.guibert@6wind.com>

Updated git submodules

Project: netvirt stable/carbon 70391261d0be69edc11649d375faa7d69a945036

Bug 8179 - Data validation failed for path bgp/networks

Issue:

Data validation failed exception during withdrawing prefix from BGP.

Analysis:

The issue occured while trying to delete an element from the
bgp/networks which was not in the desired format. The DPNid is
used as rd in case of extra routes with internal VPN. The prefixes in
the internal VPN are not be advertised to the BGP. The condition for
this already exists(rd is the vpnName here). For extra routes, DPNIDs
are used as rds for ECMP and they should also be advertised. This was
already handled but there was no handling to avoid the withdrawal from
the BGP in case of the removal.

Fix:

Condition has been added to avoid calling withdrawPrefix() for the extra
routes removal in case of the internal VPN.

Note: Also some other errors in logging have been fixed.

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

Add a new section for AAA CLI

Add a new section with all the CLI commands supported by
the AAA project via the Karaf console.

Change-Id: Ide73f850345d00a8f5d9477db4bf402ee1a9a018
Signed-off-by: David Suarez <david.suarez.fuentes@ericsson.com>
(cherry picked from commit 8b61b30042963215c9e837753abfbf35734529db)

Updated git submodules

Project: releng/builder master 6ea5a2357c4b407fb5d7e3c599ba3f1d5a19bfb4

Merge "Disable Karaf 4 SFT only"

Disable Karaf 4 SFT only

Since change I235eafb777d62a95ca8ffb8010ef95754618421e was merged we now
have an option to disable only the Karaf 4 SFT so let's do that so that
we can have Karaf 3 SFT running again in Carbon.

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

Updated git submodules

Project: netvirt stable/carbon 0174759f6ec5428cc5cd07393ec48afdcfb8468d

Bug 7824: Handling CSIT Exception under Elan module

Change-Id: I2f34d558dc6887324cf5edef6551e1ee6451f816
Signed-off-by: epgoraj <p.govinda.rajulu@ericsson.com>

Updated git submodules

Project: netvirt stable/carbon 06dae48ecb55cbe109ad894993ee1815af500332

Bug 8710:  Bgp afi related bug fixes

Change-Id: Ie62474af854b2b4db5f53c57fa7e79499e163b21
Signed-off-by: SivaKumar Perumalla <sivakumar.perumalla@ericsson.com>

Updated git submodules

Project: integration/test master 7ab8ae8c2d692d145425fff08d7b6aa3fe760bd5

Enlarge http_timeout in entity.robot

Change based on https://bugs.opendaylight.org/show_bug.cgi?id=8596#c3

Change-Id: I8c60403381778dbe59f4c26a3fb91cb507fef81e
Signed-off-by: Peter Gubka <pgubka@cisco.com>

Updated git submodules

Project: integration/test master 78ab9580aa553fe4531a40787d0d236931e143dc

OpenstackOperations: fixes due to openstack CLI

Due to migration to Openstack CLI in OpenStackOperations.robot and
DevstackUtils.robot, introduced in the following commit
02eb3418af8de2831ad8a5aa1d119746fffd1bac ("Migrate to Openstack CLI"),
appropriate fix is needed for  "openstack security group rule create"
command and formatted output is needed for "openstack router list",
"openstack port list" commands.

Change-Id: I9831e3909c38c7a70389ba29754526a2f675111f
Signed-off-by: Valentina Krasnobaeva <valentina.krasnobaeva@6wind.com>

Updated git submodules

Project: releng/builder master 2b0894555df9673b13e43bce6448c8f261250b64

Merge "Add Gate job for netvirt-hwvtep"

Add Gate job for netvirt-hwvtep

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

Updated git submodules

Project: netvirt stable/carbon 12ffded5a8c887b0fe46409ee6efd750f578e73c

VrfEntryListener refactoring using chain of responsibility pattern

This is a first review where BGP Route handling is separated into a new
handler class BgpRouteVrfEntryHandler.

All common methods are put into a new superclass i.e VrfEntryHandler. Each
of the subclasses like BgPRouteVrfEntryHandler will extend the superclass.
This is done to avoid circular depency on VrfEntryListener.

As part of chain of responsibility pattern , VrfEntryListener will pass on
the Vrf object to the first handler in a chain based on the RouteOrigin
type. Once the first handler completes processing the Vrf obj , it will
pass on the same to next handler.

In the current review BgpRouterVrfEntryHandler will process Routes of type
"RouteOrigin.BGP". BGP routes need not be processed by any other handler ,
hence next handler is not defined in the chain.

In the subsequent patches , when new handlers like
LocalRouteVrfEntryHandler, SubnetRouteVrfEntryHandler etc will have to
pass on the Vrf obj to next handler which will be
ImportExportVrfEntryHandler.

Change-Id:  I538085bfcda7733b5f3ba5ffe49d541619ac40ec
Signed-off-by: ehvkand <hanamantagoud.v.kandagal@ericsson.com>

Updated git submodules

Project: releng/builder master 2092343f7f4927c9b6a7170acdd4b14fa488f61c

Remove tcpmd5 project jobs

This project has been archived per TSC vote.
https://meetings.opendaylight.org/opendaylight-meeting/2017/tsc/opendaylight-meeting-tsc.2017-06-15-17.00.html

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

Updated git submodules

Project: ovsdb stable/carbon 063f6d0421017d29e573340be78dc8027e3ffd5b

BUG 8529: Deleted vlantag and trunks present in operational

Change-Id: Iab1c46173127f74af4b81a80a5dfba4f7d29f936
Signed-off-by: Vishal Thapar <vishal.thapar@ericsson.com>
(cherry picked from commit eb53e326463a4838c404eb4533c801a1010ca6dc)

Updated git submodules

Project: integration/test master f01446175c8707dfc48e0b9d13710b90c58c8930

Fix Mininet Start multiple connections

With latest mininet VM changes Ubuntu 16.04 OVS 2.5.2
it is not possible to start mininet with no controller
and OVS options at the same time.

In this patch we get rid of unnecessary OVS options.

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

Updated git submodules

Project: netvirt stable/carbon 5c7a7b61a82b7597566e1785df2c9e557b06764a

Spec update for DHCP Neutron Port Allocation

+ Updated the pipeline/yang changes
+ DHCP request handling added .
+ Introduce controller-dhcp-mode flag in DHCP Service.

Change-Id: I633571012d6f3ff8a07e0c8e50da9ab05c106c55
Signed-off-by: Achuth Maniyedath <achuth.m@altencalsoftlabs.com>

Updated git submodules

Project: odlparent stable/carbon 6166f35dd9864ac6ffe27ed7e25619fd7cce7247

Add method to skip karaf4 featureTest

Allow us to skip karaf4 featureTest seperately from karaf3 featureTest.

Change-Id: I235eafb777d62a95ca8ffb8010ef95754618421e
Signed-off-by: Thanh Ha <thanh.ha@linuxfoundation.org>
Bump akka to 2.4.18

http://akka.io/news/2017/05/02/akka-2.4.18-released.html
https://github.com/real-logic/Agrona/releases/tag/0.9.5
https://github.com/real-logic/Aeron/releases/tag/1.2.5

Change-Id: I2a19b3b2b1d35dba28e729938d6c579d55930681
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
(cherry picked from commit 7980f99e6874247df63fe8bed836cf1d0ce4ca96)

Bump scala to 2.11.11

Scala-2.11.11 is the last release in the 2.11 train, update to it.

Change-Id: I2d053d10b35b55efff0359f4303a66e8099e810b
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
BUG-4219: fix karaf CLI reload

Controller's blueprint extenstion is pulling in an optional
dependency of pax-logging, causing it to reload, effectively
restarting everything. Wire osgi-service before starting up
to eliminate this.

Change-Id: Idf0170d805e98362e7f3560db7f166c68a18d19d
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
Bump versions by x.y.(z+1)

Change-Id: I1b83ec0785b8bad1c894c71d08060b697b967148
Signed-off-by: jenkins-releng <jenkins-releng@opendaylight.org>
Introduce pax-cdi-api into dependencyManagement

This enables @Inject @OsgiService instead of <reference> in XML; see
https://wiki.opendaylight.org/view/BestPractices/DI_Guidelines#.40OsgiService_.26_.40OsgiServiceProvider

see c/56723 for full details re. why it is <optional>true

Change-Id: I1aa5f314cb561dbbd82cbd34c4812e0e1a803a4b
Signed-off-by: Michael Vorburger <vorburger@redhat.com>
(cherry picked from commit bf832961de4bf40135fc102e39b16851783f01f9)

skip.karaf.featureTest in single-feature-parent

Historically, the skip.karaf.featureTest property could be used to
skip SingleFeatureTest; that was lost in the migration to Karaf 4.
This patch restores the functionality in single-feature-parent.

Change-Id: Id6ed63129d7b9d7284356ca665152b321fc9ca16
Signed-off-by: Stephen Kitt <skitt@redhat.com>
Fix pax-web's commons-collections

pax-web-4.3.0 can still pull in commons-collections-3.2.1, fix that.

Change-Id: Id0e0a678fcc54b45a864b69aed8ba7d1345bb756
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
Align pax-web and karaf standard features

Karaf's standard feature uses ow2/asm-all 5.0.4, while pax-web
is installing 5.0.2. Patch pax-web to use the same version, saving
around 230k.

Change-Id: Ifb29d1b3d3a54d4d55818795289b3eb0e357a826
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
Bug 7891: Place provided features to featuresBoot

Specifically, org.apache.sshd/sshd-core/0.14.0 is excluded,
thus "ssh" feature has to be always installed.

Without this, distribution4-check with odl-netconf-mdsal
in integration fails at installing boot features, because
org.opendaylight.netconf.netty-util depends on sshd-core bundle,
which is only activated by "ssh" feature
(as any ODL feature drops that dependency,
because of exclusions in feature parents).

Change-Id: Ic1d3b19981fccca9c320a5079aaf9e2eadf09397
Signed-off-by: Vratko Polak <vrpolak@cisco.com>
Karaf 4: allow SFT configurations to be combined

We need to allow children of single-feature-parent to reconfigure the
generator plugin, while still ensuring that the list of excluded
artifacts is honoured.

Change-Id: Ifa284e6e10c5182a68d4a3457446903ef9d1ccb8
Signed-off-by: Stephen Kitt <skitt@redhat.com>
Bug 7900: make feature repos aggregate features

Instead of redefining features and producing standalone feature
repositories, feature-repo-parent can produce features which list the
individual feature repositories produced by single-feature-parent.
This allows integration/distribution's integration-index to build much
faster (by rendering SFT moot there).

SFT now tests individual features in single-feature-test instead of
testing the full repository. This makes the -Psft profile useless so
it is removed. It also means that odl-bundles-test needs to be handled
specially, since it is used in features4-test: it can't use
single-feature-parent, otherwise it introduces a dependency loop. This
patch introduces a new, internal parent,
untested-single-feature-parent, which is used by odl-bundles-test and
single-feature-parent: untested- takes care of the actual feature
construction, and single-feature-parent adds the SFT tests.

(cherry picked from commit bd4340895f8ec544393f2ec75f0cb5cfd25eae1f)

Change-Id: Ib0de6ce1e6488944bcbe0aa7e84496676f6db19a
Signed-off-by: Stephen Kitt <skitt@redhat.com>
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
Bug 8261: prevent TestBundleDiag from throwing NPE

There is a stack trace demonstrating this behavior in bug 8261.  In
short, TestBundleDiag is occasionally throwing NPE because a null
Bundle is returned from serviceReference.getBundle().  This change just
adds some null sanity checking in TestBundleDiag and
ServiceReferenceUtil.

Also, in ServiceReferenceUtil, a local variable is extracted just in
case the object is mutated between calls.  This at least gives a
consistent snapshot for usingBundles over the course of the method
call.

Change-Id: I334b0850be4a2045f9eeb260f5de4af542896dc1
Signed-off-by: Ryan Goulding <ryandgoulding@gmail.com>
(cherry picked from commit 667f362239f037fb01b670dbb20340ab2798ca1e)

Bug 7926: exclude OSGi bundles from features

We need to avoid having OSGi core bundles pulled in by features (they
cause bundle refreshes which break Pax Exam tests). This patch marks
the OSGi bundles as provided, and excludes them from generated
features.

Change-Id: I66f519e4d498f956cb3fa087d98d30a64b3316f4
Signed-off-by: Stephen Kitt <skitt@redhat.com>
Relax Checkstyle RightCurly rule for LITERAL_WHILE

In the Checkstyle upgrade from 6.16 to 7.6.1 done in parallel in other
changes, I've come across a number of "violations" because it did not
like "... } while (condition)" in one line.

This indeed seems to be what our current Checkstyle rule would mandate,
but because of what looks like a bug in our currently used version 6.16
was never enforced.  As we upgrade, they seem to have fixed that, so
either we add a \n after } before while during the upgrade in the
projects which are already enforcing Checkstyle - or we just relax this
rule - personally I think it's ugly on two lines, and thus suggest this.

See http://checkstyle.sourceforge.net/config_blocks.html#RightCurly

I have not actually really tested this, just seems above right.

Change-Id: I21e20d91b854340a636c4fbbe5a6038caf0bee57
Signed-off-by: Michael Vorburger <vorburger@redhat.com>
(cherry picked from commit 3d622f95319336f5a5b9587b924bd6bac98982d7)

BUG-8208: remove prereq from jdbc/jetty features

It seems that dependencies have to be completely converged
and prerequisite=true does not play nice with optional imports,
leading to half-baked installation.

This patch drops dependency/prerequisite flags, which will need
to be revisited once we have this thing working.

Change-Id: I9340e7dff6566b793689406e8029b546ec349302
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
Revert "Bug 8208: revert to the Karaf 4.0.7 runtime"

This reverts commit dacab8c7e49bc18578ceb8f088333550cfe389ab.

Change-Id: I6c7808b1ef130287a52c6d24c20c951f7d5dcd53
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
BUG-8202: make war a prerequisite

This seems to help getting the container up and about in local testing,
but it may break something else.

Change-Id: I0cadae70de7ef39001fccfaa33982de3293cc130
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
Bug 8208: revert to the Karaf 4.0.7 runtime

We keep the 4.0.9 Maven plugin to get the new generator features, but
switch back to the 4.0.7 runtime to avoid the resolver issues.

Change-Id: Ifebe6ab8155a86c2e9e8c80315ae345a7f033a79
Signed-off-by: Stephen Kitt <skitt@redhat.com>
Remove akka-distributed-data-experimental

This module is causing trouble with org.objectweb.asm and is not
used anywhere anymore. Remove it.

Change-Id: I0dcb1d73533f2a7ef1999093840c0c93a4f1450d
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
Update .gitreview to stable/carbon

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

Updated git submodules

Project: netconf stable/carbon e51312f71f5c2e964bc660b5ffb6f5089777ef51

Remove UriInfo from JSONRestconfService API methods

https://git.opendaylight.org/gerrit/#/c/48369/ added a UriInfo parameter
to several JSONRestconfService API methods in Carbon. However this was an
undocumented API change which will break downstream users. Further,
there is no available implementation of UriInfo so each user would have
to provide an implementation which would pretty much render the API
unusable. We do not need to expose UriInfo in this API so I've remove the
parameters.

We'll need to cherry-pick to stable/carbon for SR1 to minimize breakage.

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

Updated git submodules

Project: integration/test master 57ffdb8b0bb58c5392d0d7989a9d8e9bf794fdd9

Refactor suite to use RemoteBash, clean Utils

Refactor suite to use RemoteBash lib and clean Utils lib.
Move Write_Bare_Ctrl_C and Write_Bare_Ctrl_D from Utils to RemoteBash.

Change-Id: I3519f85aea8e47df07c9761eb5e079818ac7ae3a
Signed-off-by: Andrej Kilvady <akilvady@cisco.com>

Updated git submodules

Project: integration/test master 672d276e8a08326b7c3c5b4c7e6ea7ee2df7c648

Change longevity job duration from 24h to 23h

It is because releng contains a script which may kill VMs with a day
granularity.

Change-Id: Ice44f01246792f5e49318b8205499eb5979e1a79
Signed-off-by: Peter Gubka <pgubka@cisco.com>

Updated git submodules

Project: netconf stable/carbon 557006cd35f1b540bc0dc15e0b800e0687b73934

Un-deprecate JSONRestconfService(Impl)

Patch https://git.opendaylight.org/gerrit/#/c/56391/ deprecated
JSONRestconfService and JSONRestconfServiceImpl however no reason
was given in the patch message (in fact the patch was about fixing
checkstyle violations) nor were any comments added to the classes
as to why they're deprecated.

There really shouldn't be any reason to deprecate these classes
as they are useful and are used in applications. Therefore I remove
the @Deprecated annotation.

Change-Id: I335c8e8665cc2b489e4d3b690d8cac7dbd157b4f
Signed-off-by: Tom Pantelis <tompantelis@gmail.com>
BUG-8085: create missing parent augmentation node

Augmentation nodes do not exist in serialized form
(e.g. in edit-config message), but are required by DataTree-based
DOMDataBroker implementations, so should be created if data from
augment is present.

This patch creates missing augment nodes by issuing merge on augmentation
before put operation (based on current behaviour for ListSchemaNodes).

Change-Id: If657ae96e914fc46617099042a833c7d4d5883b7
Signed-off-by: Marek Gradzki <mgradzki@cisco.com>
BUG 8566 direct writes to ordered list fail

Incorrect handling of direct writes into a mapEntry which tries
to merge an empty parent list instead of an exists check but doesnt
correctly handle the possibility of an OrderedMapNode as a parent.

Change-Id: Ia2c75cc50e6ff4c9c7c9be36846c08cc85cff491
Signed-off-by: Tomas Cere <tcere@cisco.com>
Merge "Bug 7933: NPE when posting using XML" into stable/carbon

Merge changes Ieb46b050,I6bff6467 into stable/carbon

* changes:
  Bug 8533: Not possible to invoke RPC on mount points with new Restconf
  Bug 7933: NPE when posting using XML

Merge "Bug 8455: Yang Patch response is not having the error details" into stable/carbon

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

Change-Id: Iccde43caf965f4f6d147cc96027f8585125e6229
Signed-off-by: jenkins-releng <jenkins-releng@opendaylight.org>
Bug 8533: Not possible to invoke RPC on mount points with new Restconf

- put identifer of RPC into InstanceIdentiferContext
instead of mount point identifer which leads to fail
in unit test

- when working with mount point use mount point's
schema context to find RPC module, previously used
controller's schema context leads to NPE

- repaired imports in RestconfApplication to use
new implementation

Change-Id: Ieb46b050e881c89ea58bb9f521be69554733d575
Signed-off-by: Ivan Hrasko <ivan.hrasko@pantheon.tech>
Bug 7933: NPE when posting using XML

PUT operation:
- fix issue when putting malformed XML body did not
fail with appropriate status.

Change-Id: I6bff6467f4b99bed9adf1ac39def54822899761e
Signed-off-by: matus.kubica <matus.kubica@pantheon.tech>
Signed-off-by: Ivan Hrasko <ivan.hrasko@pantheon.tech>
Bug 7933: NPE when posting using XML

- NPE was thrown because leaf value was not parsed,
thus data value was null
- fixed by adding condition to parse leaf nodes
- log warning when unknown schema node type was
not parsed
- fixed in old and new restconf

Change-Id: I13c8e419e35999980548e09f6963b0d407a45b31
Signed-off-by: Ivan Hrasko <ivan.hrasko@pantheon.tech>
Bug 8455: Yang Patch response is not having the error details

- add error-info to error message

Change-Id: I0d637871969ff91850749228e92bda2055c71eef
Signed-off-by: Ivan Hrasko <ivan.hrasko@pantheon.tech>
Bug 8421 - Revert "Resolve exception from registerClusterSingletonService"

This change causes, that whole topology is closed, when
cluster singleton service registration fails, which is incorrect.

This reverts commit 25e450c382211e0b80d873f7697c7b9807be54af.

Change-Id: I37053d4fda7830ef53dc61fcbda3d149f11bbdbe
Signed-off-by: Andrej Mak <andrej.mak@pantheon.tech>
Merge "Bug 8074 - actor name [test-device] is not unique!" into stable/carbon

Bug 8074 - actor name [test-device] is not unique!

When creating a new actor with same name as an old one, it might fail
becuase old actor might not be completely gone at this point.

This creates a slaveActor which will never be closed, but instead
every time that mountpoint is closed or master is changed, it would
update NetconfNodeActor data and keep the same actor.

Only time that actor is closed is when node was deleted. Or slave
became a master

The name of the actor is removed - letting akka to handle that -
log the akka created name.

Change-Id: I6d10db7b8902048632e336fb5a9701a797b363c6
Signed-off-by: miroslav.kovac <miroslav.kovac@pantheon.tech>
Resolve exception from registerClusterSingletonService

registerClusterSingletonService return RuntimeException in
case of problems with registration and client should implement
strategy to resolve this issue

Change-Id: If6350969f1d1c2917c78850f4c66629b52137b0b
Signed-off-by: miroslav.kovac <miroslav.kovac@pantheon.tech>
(cherry picked from commit dbcd66681f76ef4175917d1620b9066b86f2a267)

Bug 8405: Add close check to NetconfDevice

Since schema resolution runs in its own thread, it is possible,
that handleSalInitializationSuccess is called when
NetconfDeviceCommunicator was closed meanwhile. Add check to
prevent this.

Change-Id: If93d32b26f0b98c4c0d47fdd65fdb5104db20bc5
Signed-off-by: Andrej Mak <andrej.mak@pantheon.tech>
Merge "Bug 8364: Set bundle context when module reloads" into stable/carbon

Merge "Bug 8361 - NetconfConnector cannot be created due to NullPointerException" into stable/carbon

Bug 8364: Set bundle context when module reloads

Fix for bug Bug 8214 handles waiting for AAA, when Restconf
is installed for the first time. Override handleChangedClass
to do the same thing for reinstall case. Also close old
module to release bound ports and started threads.

Change-Id: I00f5f330fca39bb63f66517dddd8d1371ae353e8
Signed-off-by: Andrej Mak <andrej.mak@pantheon.tech>
Bug 8361 - NetconfConnector cannot be created due to NullPointerException

- Adds Nullpointer checks in RemoteDeviceConnecterImpl
- If Yang module capabilities are missing its override flag is set to false
- If Non module capabilities are missing its override flag is set to false

Change-Id: I25d172e94a1013d79c0c7db37c930c89ad2ed94f
Signed-off-by: Martin Mihálek <mamihale@cisco.com>
Bug 7891: Add ssh feature to netconf-netty-util

Odl-netconf-netty-util needs sshd-core bunde to work.
It is provided by karaf. However, if odl-netconf-netty-util
is placed to featuresBoot, ssh provided by karaf isn't
started, so ResolutionException is thrown. To prevent this
add ssh feature to odl-netconf-netty-util.

Change-Id: Iaf5ecddaa726ea892191b5cf47095c1df3b096f6
Signed-off-by: Andrej Mak <andrej.mak@pantheon.tech>
Bug 8311 - Apidoc: Incomprehensible 500 id model is wrong
Bug 8266 - Apidoc explorer is broken after installing Boron SR3

Depending on path if it is relative or absolute module has to be
resolved different ways. If it is relative path it will have to
be resolved through DataSchemaNote which returns namespace and
revision of module in which node is situated. If it is absolute
path we need TypeDefinition which returns a namespace and
revision of original module.

Change-Id: I522ea66383ba1d65e6674d5e8ff67ec31e019303
Signed-off-by: miroslav.kovac <miroslav.kovac@pantheon.tech>
Bug 8084 - FilterContentValidator.getKeyValues creates invalid YII key values

Change-Id: Ie6c65c9cb005f9f1be85e04a8cf643e48e07bb94
Signed-off-by: miroslav.kovac <miroslav.kovac@pantheon.tech>
Bug 8032 - Initialization in sal failed, disconnecting from device

MountpointService is initialized with blueprint and therefore it
should never throw null pointer exception anymore.

All the binding and dom broker are deleted because they are
not needed any more.

Change-Id: I9aa9064752043819bc73043f7b667b4ea2c0fd0a
Signed-off-by: miroslav.kovac <miroslav.kovac@pantheon.tech>
Bug 8289 - 409 in cluster restperfclient test

To ensure message ordering we need ReadWriteTransactionActor. This
way we are sure that all reads and writes will be executed in
the order that they are sent.

Change-Id: I7a76f6b4d9e6e348ec0d58abe24e56dfeae66f24
Signed-off-by: miroslav.kovac <miroslav.kovac@pantheon.tech>
Merge "Bug 8197: Deregister schema sources on actor stop" into stable/carbon

Bug 8197: Deregister schema sources on actor stop

NetconfNodeActor registers schema source provider
to schema registry. When mountpoint is removed, this
registration should be removed too. If it isn't,
following issue can happen:

1. Create mountpoint for device1
2. Master actor for device1 is registered as mod-1.yang provider
3. Delete device1
4. Create mountpoint for device2
5. Master actor for device2 is registered as mod-1.yang provider
6. Register slave -
   schemaContextFactory.createSchemaContext(sourceIdentifiers)
   is called
7. Since dead device1 master is still registered as provider,
   ask in ProxyYangTextSourceProvider timeouts
8. After timeout device2 master is queried
9. Device 2 slave mountpoint registered

This delays slave mountpoint registration.

Change-Id: I060c8b1988ba7b54f9a93d7eb37adb5c5e48b23b
Signed-off-by: Andrej Mak <andrej.mak@pantheon.tech>
Fix odl-netconf-ssh single feature test

Add dependency on odl-netconf-tcp and odl-netconf-util modules so we
will see their features repositories generated in odl-netconf-ssh
feature.xml file.

Also set dependency="true" atribute for both dependency features when
overriding generated feature.xml file.

Change-Id: I290f63d36961c1eb9cac06d48dfeb2bfee4dace4
Signed-off-by: Jakub Morvay <jmorvay@cisco.com>
(cherry picked from commit 855980926d53f158e8d3e76eb0fbe9e13661fb43)

Merge "Bug 8214: Consume AAAService" into stable/carbon

Bug 8181 - Add karaf4 feat to artifacts

Change-Id: Ied3f487b8292b41b2a863bedc2ccf2aefaea662d
Signed-off-by: Ivan Hrasko <ivan.hrasko@pantheon.tech>
Bug 8214: Consume AAAService

Consume AAAService ensuring the bundles are available from the BundleContext
prior to trying to load the rest of RESTCONF.

Change-Id: If3e081712d475e65799b805ce0cdcd6f946c93f2
Signed-off-by: Ryan Goulding <ryandgoulding@gmail.com>
Refactor netconf clustered topology tests

- split long tests into smaller
- remove duplicated code

Change-Id: I64913380fd0768394b430971ef5ff118d56b7b6b
Signed-off-by: Andrej Mak <andrej.mak@pantheon.tech>
Bug 8152: Add way to configure idle timeout

Clustered apps should be configurable via datastore
to allow config distribution to all nodes according to
https://wiki.opendaylight.org/view/Using_Blueprint#Using_the_Datastore
Add such configuration for clustered topology.

Change-Id: I1a318590c75c6c95da48cb1e0521994f0b83dfe4
Signed-off-by: Andrej Mak <andrej.mak@pantheon.tech>
Bug 8152: Transaction is already opened

This issue happens, when for some reason transaction
submit or cancel message isn't delivered to master
node. With current implementation, only one device
transaction can be opened at the time, so submit or
cancel delivery failure will lock device forever.

To prevent this, this patch introduces write trancaction
idle timeout. Write transaction actor will be stopped
and its device transaction cancelled, when no message
is received for given time. Cancellation unlocks device,
so mountpouint is usable again.

Change-Id: I37bef30038cf6fd10fa5149a3fa949540ac16eab
Signed-off-by: Andrej Mak <andrej.mak@pantheon.tech>
(cherry picked from commit 95dc1455a7303eac56c755d01a37ca1f203543c0)

Merge changes I220c5a0e,I9f5dca23,Ibf5628f6 into stable/carbon

* changes:
  Make EditConfig methods static
  Eliminate ReadWriteTransaction proliferation
  BUG-5581: batch read check

Change handling of netconf cluster transactions

Own actor is created on master for every transaction.
Proxy transactions use this actor for mount point access.
Creation of proxy transaction initiates sending of message
to the master node actor. Master node actor starts
transaciton actor if needed.
Both master and slaves communicate with device via
proxy transactions.

For write transaction, submit and cancel cause actor stop.
There can be single instance of read actor, which can be
reused for all transactions.

Change-Id: Iea39d794e8f2026b76f049409a718b6d94f41649
Signed-off-by: Andrej Mak <andrej.mak@pantheon.tech>
(cherry picked from commit 446738525190ad92736f1a8868e3263737276aff)

Make EditConfig methods static

Some of these do not refer to object state and hence we can
make them static.

Change-Id: I220c5a0ed9340bc9cdd8f2ce7541568a2eaa7613
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
Eliminate ReadWriteTransaction proliferation

Many of the internal methods really need only a WriteTransaction,
hence adjust the type to make it more clear what a method can
actually do.

Change-Id: I9f5dca23c5174f2668f269fbf40e811e7046862d
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
BUG-5581: batch read check

When posting data we are executing potentially a lot of reads,
which can hurt if the backend is not residing on the node which
is executing the operations.

Fix this by issuing read requests in a batch and then collecting
the results, which means we will amortize request latency.

Change-Id: Ibf5628f6fdf56b7845b4a4623255ff4d417c4546
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
Merge "Bug-8187: call home throwing exception on startup" into stable/carbon

Merge "Close read-only transactions" into stable/carbon

Close read-only transactions

As noted in API, read-only transactions are resources and must
be closed just as any other external resource.

Change-Id: I0f2d888a31edf242bc48410e6f2d4dfeed43a6b3
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
Migrate to karaf4 for the local karaf distribution

Several folks have put in incredible effort to make karaf4 a reality in ODL.
Kudos to those folks for this great effort.  This patch officially converts
netconf's local karaf distribution to use odlparent's karaf4-parent artifact.

The karaf4 implementation will still be accessible at the following well-known
path:

> karaf/target/assembly

Change-Id: I00a1e5b93f5c57447e95a714d40e0635b5747393
Signed-off-by: Ryan Goulding <ryandgoulding@gmail.com>
(cherry picked from commit cb106fb29817db6c9e39ae6f47a3c3276d8e5e92)

BUG-7868: perform checks before starting modifications

The codepath for makeNormalPost() performs sub-optimal emulation
of create().

This patch changes the logic to first check for presence and then
perform modifications. This saves cycles as continuous modify/read
cycles incur heavy penalty on DataTree and also allows batching
of modification requests.

Change-Id: Ic8e18ada094334948170f2a147da806db7db1a16
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
Bug 8115: Change URI decoding from ISO-8859-1 to UTF-8

Change-Id: I3bcc7cae197aa267a43c8598f54050e987712c9a
Signed-off-by: lbeles <lbeles@frinx.io>
Signed-off-by: Ivan Hrasko <ivan.hrasko@pantheon.tech>
Bug-8187: call home throwing exception on startup

Remove distracting exception in log.
Restore ability to change the call home server port.
Move to using blueprint for configuration instead of
using ad hoc logic.

Change-Id: I273156ba13237b1ed8efc0e6ff9861b1aa5feb3b
Signed-off-by: allan clarke <clarkea@brocade.com>
(cherry picked from commit a03f9c965b43c9d7b78a78ab9e56935a5da07a88)

Update .gitreview to stable/carbon

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

Updated git submodules

Project: netvirt stable/carbon 230a53309eab12da49567a4e26604d9336a7d50e

Bug 8441, 8630 - LB groups not deleted after extra route deletion

Issue :

When an extra route is completely removed from all the DPNs, the
loadbalancing groups are not updated properly.

Analysis:

There was a "NullpointerException" occuring in the
setupLoadbalancingGroup() method.

Fix:

Changes have been made to avoid the NPE. Also, changes were
made in resolveadjacency() method to populate the adjacencylist
correctly.

This patch also fixes the bug 8630
(CSIT Sporadic failures - arp learning suite
 - MIP not in /config/odl-fib:fibEntries/)
as well which resulted due to the exceptions
caused as mentioned above.

Also it reverts some part of the review that landed here:
https://git.opendaylight.org/gerrit/#/c/59081/

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

Updated git submodules

Project: releng/builder master 09646d0a818928a78c1aa778477722e2a46d8484

Merge "Use pipdeptree instead of pip freeze"

Use pipdeptree instead of pip freeze

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

Updated git submodules

Project: releng/builder master ad5661e545b79100e1dd220ab319423df06a07ad

Do not run javadoc:aggregate in bgpcep

Other projects are not using it either, so let's keep
things consistent at least for nitrogen.

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

Updated git submodules

Project: ovsdb stable/carbon 45209f5ca7598b34d7451214ace2883fcfd4b185

bug 8257 handling back to back ucast mac updates

Making the ucast mac remove command participate in
dependency resolution workflow.

Change-Id: Icc27763e3fdca46d12be89218d4f17758a7e8d93
Signed-off-by: K.V Suneelu Verma <k.v.suneelu.verma@ericsson.com>

Updated git submodules

Project: ovsdb stable/carbon 737f29793e916d3e5819ad3ec823ce85bb98e4f7

bug 8257 handling back to back logical switches

Making logical switches update/remove participate in
dependency resolution workflow.

When logical switch delete requests comes  it is held until
the previous logical switch add request finishes and vice versa.

Change-Id: I5d158759a977f7a0f66dcafd297c4d2f06fa3fd4
Signed-off-by: K.V Suneelu Verma <k.v.suneelu.verma@ericsson.com>

Updated git submodules

Project: ovsdb stable/carbon 4b0913ac93fe903e5400bcc1d1cf7abde6e44155

bug 8257 handling back to back mcast mac updates

Making the remove mcast mac participate in depenency resolution
flow.

Put the delete event on hold while the previous add/update on
the same key is still in transit.

Delete mcast mac event resumes after we receive the
 response for the previous create.

Mark the state of the mcast mac key in transit while we execute its delete/add/update.

Change-Id: I0b15d82dad346bd2c35c0368b4d7f63dc3bfa09d
Signed-off-by: K.V Suneelu Verma <k.v.suneelu.verma@ericsson.com>
(cherry picked from commit 1c48d4d9765a0fabc1abce8e90ab263580a40870)

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

Change-Id: I4b054c308bd3f13c37b4a7905c2d8de1efe22220
Signed-off-by: jenkins-releng <jenkins-releng@opendaylight.org>
bug 7985 Using UncaughtExceptionHandler in TransactionInvokerImpl

when a particular transaction command fails , it is killing the only
worker processing the transaction commands queue.

when the exection is caught log the exception and resubmit the worker

if 10 commands are to be exectuded
while executing 5th command some exception happens
then the execution resumes from 6th command ignoring the 5th command.

Again if 6th command also fails because of some exception,
then execution resumes from 7th command onwards that way it wont result in
infinite loops.

This patch does not try to address transaction failures, only addressing any
exceptions in the commands.

for transaction chain failures, let us say all 10 commands are executed
successfully and submitted to transaction chain, now we wait for
success/failure of each such transaction asynchronously.
Now our pending tx list reads as 1,2,3,4,5,6,7,8,9,10

we may get success response for 1,2,3,6,7 and failure for 5th.
Then pending tx list reads as 4,5,8,9,10.

then the transactions will resume from 8th onwards. Note 4th is not resumed.
This behavior may not be desired as we are trying to resubmit 8th one which
already got executed.
If the transactions are idempotent then it should not be an issue at all to
reexecute the succeded ones.
Now if the 8th transaction is about deleting an entry ,
in its previous run it already deleted it, so this run will fail.

Now for the 4th one we may get a success callback or failure callback.
If we get failure callback , then again 8,9,10 will be reexecuted.

This patch is not addressing this scenario , can be taken up in later
patches with the right design.

Note the difference between command execution and transaction result.
This patch is addressing exceptions in command execution.

Change-Id: I93bba958784637a4752b860d23354506f0bd98bb
Signed-off-by: K.V Suneelu Verma <k.v.suneelu.verma@ericsson.com>
bug 7599 improving perf of ucast mac learning

when the device is connected all its macs are learnt in one update.
node.merge is called for each local ucast and remote ucast mac.
Now it is called only once.

when a mac update comes its physical locator need not be part of the
same update. If the locator is not available in the same update get
it from the cache ( which got filled from previous locators update msg).

Change-Id: Idbcf0b881b8ac90526f400c5d17d56a825f0a611
Signed-off-by: K.V Suneelu Verma <k.v.suneelu.verma@ericsson.com>
BUG 8280: TerminationPoint reconciliation fails

TerminationPointConfigReconciliationTask is incorrectly setting up
BridgeOperationalState to be empty. This ends up triggering reconciliation
even for ports that are already present in switch.

When we try to add ports that are already present we get constraint
violation in transaction and as per OVSDB RFC it causes all operations in
that transaction to abort. This means that the ports that actually needede
to be reconciled never get added back.

Change-Id: I48819f46b6419c15a8ba85541b06551b950a3055
Signed-off-by: Vishal Thapar <vishal.thapar@ericsson.com>
Add missing <name> fields for pom.xml files

This is used by autorelease scripts to automatically parse which project
is failing a build and report to the mailing list automatically. We need
names in the format:

    ODL :: <groupId> :: <artifactId>

This patch formats in the same format as found in the startup archetypes
patch found here: https://git.opendaylight.org/gerrit/52522

Change-Id: Ib3621b8a5f2aa1e0b50aa0968b0b728081993772
Signed-off-by: Anil Belur <abelur@linuxfoundation.org>
BUG 6692 - Fix checkstyle problems not detected by the current version

This change is required for overall move to new Checkstyle version, see
https://git.opendaylight.org/gerrit/#/q/topic:bumpCheckstyle

Change-Id: If08990eaf8b849e6d826322eb1b7301996bc86c1
Signed-off-by: David <david.suarez.fuentes@ericsson.com>
bug 7781 update vlan bindings and tunnel ips

When physical switch update arrives write it in op datastore.

Change-Id: I46ca4f70b59961eeebbc17c0073fe40f852278b5
Signed-off-by: K.V Suneelu Verma <k.v.suneelu.verma@ericsson.com>
Don't ignore .gitignore

Change-Id: I0153a81206b7b61e806c43eebdd8169aec0c9e20
Signed-off-by: Stephen Kitt <skitt@redhat.com>
Bug 7619: correct the southbound artifact

The feature repository should be southbound-features4, not features4.

Change-Id: I03250768f0371929f61eb79f6eca32752e16afe7
Signed-off-by: Stephen Kitt <skitt@redhat.com>
Update .gitreview to stable/carbon

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

Updated git submodules

Project: netvirt stable/carbon 7b5cdf963d51295b7588a762279908113216918e

Fix null Optional instantiations

Optional type variables must never be null.

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

Updated git submodules

Project: netvirt stable/carbon 1393b174d768978479c2bcc8f49811c2d7ca2fde

Restore VRF imported routes case

In createLocalFibEntry, localNextHopInfo is always null. It used to be
initialised in what became the forEach() loop; this patch restores a
standard foreach loop and tracks whether local next hop info has
actually been seen.

Change-Id: I9dc7a041a1010f7f85003bd225d75f2d03a7d076
Signed-off-by: Stephen Kitt <skitt@redhat.com>
Signed-off-by: Sam Hague <shague@redhat.com>

Updated git submodules

Project: integration/test master 31fb94bf9f599c61cc683f2d5d5493bc314942f3

Enable some ipv6 tempest.scenario cases

Change-Id: If16b2367fcaa9a2a4ee61db3e31095e4f2ebe3b6
Signed-off-by: mor.zvi <mor.zvi@hpe.com>
Signed-off-by: Jamo Luhrsen <jluhrsen@redhat.com>

Updated git submodules

Project: releng/builder master c4b23edec73340b9d8d3d28e9d31828afa480e35

Fix bundle URL dectection script

Both jenkins and nexus use similar URL structure:

*/artifact/version/artifact-version*.zip

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

Updated git submodules

Project: netvirt stable/carbon 86dfa540619d83734380428909ef52403e585bfb

Bug 8706: Restore null-check in createLocalFibEntry

checkCreateLocalFibEntry handles null gracefully, but dereferencing
localNextHopInfoLocal in the method call renders that useless.

Note that just following the forEach(), localNextHopInfo is always
null which suggests something got lost in the transformation...

Change-Id: If15cc4a881c2a1468c84a4e0cebb2a940465f33b
Signed-off-by: Stephen Kitt <skitt@redhat.com>
Signed-off-by: Sam Hague <shague@redhat.com>
(cherry picked from commit b53e6bdf8ab17e4f96d49d607c3eae0a7332b73c)

Updated git submodules

Project: netvirt stable/carbon 1fff16efb7fbfad3133caa90a42a17cef2fb6813

Minor code clean-up

Replace vrfEntry.getDestPrefix with localNextHopIP in createLocalFibEntry

Change-Id: I7f891ce9a692723829c749736e7f5d89011822d9
Signed-off-by: Janki Chhatbar <jchhatba@redhat.com>

Updated git submodules

Project: netvirt stable/carbon 60de1892898780c4047e634ba4c86c20129931ea

Bug 8657: Fix for failure seen with IPv6 extra-route

Currently when programming an IPv6 extra-route, we see an
exception in Netvirt and the corresponding table-21 flow
is not programmed. This patch addreses this issue.

Change-Id: I5d8192dd4d095ecbb8f0c733a795953a34c48487
Signed-off-by: Sridhar Gaddam <sgaddam@redhat.com>

Updated git submodules

Project: netvirt stable/carbon 9be663c4608df02ec9c8e570af6195e399d4525e

Bug 8552 - [l3vpn]Stale flow entries not getting removed even when all the
VM config are deleted

FIB handling was being interrupted by and NPE, which led to stale remote
table-21 flows. This is because the GroupEntityBuilder has been modified
with respect to the way it adds BucketListInfo.
It tries to addALL the elements from the input List of buckets now, and
hence a null param causes NPE. This has been fixed.

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

Updated git submodules

Project: integration/test master d302a86c44b63aa6a4110ea7af2ae0721a759e78

OpenStackOperations.robot: format output

Make an output of "openstack router create" formatted, it makes test,
which are using OpenStackOperations.robot, more stable.

Change-Id: I70809cd9da2a7719728db05cc71e6e4717c3ca00
Signed-off-by: Valentina Krasnobaeva <valentina.krasnobaeva@6wind.com>

Updated git submodules

Project: netvirt stable/carbon d24cb415e506edd3f94f6a00d907935f13eb1168

Use optional for dataBroker

Change-Id: Ifa83b22b5868ad024557b8a9c6ca9297c24a636a
Signed-off-by: Sam Hague <shague@redhat.com>
(cherry picked from commit 0e1d866844a1876cab7d47db8f5d66ba1b5e7816)

Updated git submodules

Project: integration/test master 3c90bb44094929ed6000ce09deed4d10fdfaf239

IPv6: refactor 01_basic_vpnservice.robot

Unify variables names and move some of them to netvirt/Variables.robot
lib for reusing it in future Ipv6 L3VPN basic tests.

Move some keywords and checks to OpenStackOperations.robot and
VpnOperations.robot libs as well, for having an ability to reuse it
future IPv6 L3VPN basic tests.

Change-Id: I812bbfb9d619d46e20fd05747212ab32c3933eb8
Signed-off-by: Valentina Krasnobaeva <valentina.krasnobaeva@6wind.com>

Updated git submodules

Project: netvirt stable/carbon f927129623950729507d14e360c7ca627d797d72

Fix SingleFeatureTest breakage on fibmanager-shell bundle

Looks like the build breakage happened via landing:
https://git.opendaylight.org/gerrit/#/c/58554/

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

Updated git submodules

Project: genius stable/carbon 660687dcc0960ce31eb2368740871f782d3706a3

Merge "Bug 8146:BFD tunnel monitoring interval update fix" into stable/carbon

Bug 8146:BFD tunnel monitoring interval update fix

Added support for dumping more caches
Caches were not getting cleared correctly. Fix to remove tunnels from Cache whenTEPs are deleted.

Change-Id: I4bbc5e203cf90f81ba9b9b70818c4972d05ef2c6
Signed-off-by: Hema Gopalakrishnan <hema.gopalkrishnan@ericsson.com>

Updated git submodules

Project: integration/packaging master d5911bb92cc6033b350c3dc6071f859bf42dcc99

Major docs update

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

Updated git submodules

Project: releng/builder master 63e9993ad921bad51c005e410942287368308229

Merge "Add OVSDB TLS connection tests in AAA project"

Add OVSDB TLS connection tests in AAA project

Feature odl-ovsdb-southbound-impl-ui added to features to install.
Added tools_system_image: Ubuntu 14.04 - mininet-ovs-25 - 20170210-0300.

Change-Id: Ie72a00a32bee59f0446606344625184fc097fcb2
Signed-off-by: Ricardo Riolobos Sanz <ricardo.riolobos.sanz@ericsson.com>
Signed-off-by: Daniel Farrell <dfarrell@redhat.com>

Updated git submodules

Project: integration/test master fbf52a16d32c4feb70263caddfec05c2dc796bc6

Add test logging in karaf console

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

Updated git submodules

Project: releng/builder master 5c48c347ec44665ae8af06020cc4825658aadb73

Merge "Autorelease Nitrogen to upload K4 distro"

Autorelease Nitrogen to upload K4 distro

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

Updated git submodules

Project: releng/builder master 1cf391d4911d3a4768db0aa6d6f254f47c963538

Merge "Do not archive dependencies.log"

Do not archive dependencies.log

We do not want to use Jenkins as an archiving service.

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

Updated git submodules

Project: integration/test master 7d43097353ba885f57a3bc1ed79249257344d9b1

netvirt: fix for openstack router set command

Due to migration to Openstack CLI in OpenStackOperations.robot and
DevstackUtils.robot, introduced in the following commit
02eb3418af8de2831ad8a5aa1d119746fffd1bac ("Migrate to Openstack CLI"),
appropriate fixes are needed for ${RT_OPTIONS} and ${RT_CLEAR}
variables, which are holding arguments of "openstack router set"
command.

Change-Id: I0b536af35e986d21aa0d1772d6c62b4e7b5eda28
Signed-off-by: Valentina Krasnobaeva <valentina.krasnobaeva@6wind.com>

Updated git submodules

Project: netvirt stable/carbon 407be04a28df618db0c2599c7b687fb70e76920c

Bug 8520 - NPE at org.opendaylight.netvirt.vpnmanager.arp.responder.ArpResponderUtil.getActions

Donot setup Arp Responder flows to External Netwrok if GW router MacAddress is Null.

Change-Id: I0a013c3d43199a56786fd7f4a788521617f3df7c
Signed-off-by: Janki Chhatbar <jchhatba@redhat.com>
Signed-off-by: cgowdru <chetan.arakere@altencalsoftlabs.com>

Updated git submodules

Project: federation master 53971fce51da881659a9c3cd6873c2fd357ef129

Add missing xml:config to federation-service-impl

see https://lists.opendaylight.org/pipermail/netvirt-dev/2017-June/004692.html

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

Updated git submodules

Project: releng/builder master 7b0ad93482e401875ec7785d4c3a6c17be4041e2

Use hardcoded karaf4 bundle-url for nitrogen

In the case of nitrogen we want to run netvirt CSIT
with the netvirt project created karaf4 distribution

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

Updated git submodules

Project: integration/test master 8f4eb759cbf9932322fb91dc77f8e79104c7cbd2

Fix issues with genius script plans

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

Updated git submodules

Project: integration/test master 369706e1243f62de32f9edd9fbc5c5f205a9c754

Rename genius scriptplans

Rename genius scriptplans to align to the changes in releng/builder

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

Updated git submodules

Project: genius stable/carbon 2552766edf56459c18249cf3888ff908a788ac7a

Merge "Listener base classes should not throw exception if close()'d" into stable/carbon

Listener base classes should not throw exception if close()'d

for AsyncClusteredDataTreeChangeListenerBase and
AsyncDataTreeChangeListenerBase, by introducing a new
ShutdownLoggingExecutorService infra util. related to -1 discussion in
https://git.opendaylight.org/gerrit/#/c/57004/

This is an alternative proposal to the ShutdownLoggingExecutorService
idea from https://git.opendaylight.org/gerrit/#/c/57156/

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

Updated git submodules

Project: releng/builder master 347dd3417f85f06c999e7e9059de4df34bf57589

Merge "Update Ubuntu 1404 with 1604 mininet-ovs-25 images"

Update Ubuntu 1404 with 1604 mininet-ovs-25 images

All the mininet images are updated with Ubuntu 16.04 as per
thread since CSIT jobs are still using the outdated 14.04 images:
https://lists.opendaylight.org/pipermail/dev/2017-March/003484.html

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

Updated git submodules

Project: releng/builder master 0ee3c05331b174e1be5d2adad43b9bdaae50ffb1

Merge "Update Ubuntu 14.04 mininet images"

Update Ubuntu 14.04 mininet images

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

Updated git submodules

Project: releng/builder master 8d12b324463c0e3357d7eb2f52e63319d0ef9068

Merge "Update CentOS7 docker images"

Update CentOS7 docker images

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

Updated git submodules

Project: releng/builder master b30e58394f7c9e1c7c354922cf2a27a83a0df15e

Merge "Update CentOS7 devstack-newton images"

Update CentOS7 devstack-newton images

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

Updated git submodules

Project: integration/test master c7cff03eb26cb49fab7f334e7094797cbda39ec7

Add new test cases to clustered reconciliation

Change-Id: Id1df778573e4cc6efcdb8d228c577648df214d96
Signed-off-by: Len <lgiersch@brocade.com>

Updated git submodules

Project: netvirt stable/carbon a0b48836df8e539a461d6778be4f68039877f152

Bug 8374: NPE observed on fib-show command

 + Havent been able to reproduce the NPE, but did a bit of
   refactoring with special care about possible NPEs

 + First of all, it's not VrfEntryListener's responsibility to
   print anything on screen. This logic's been moved to the
   fib-show command implementation.

 + I took the change to remove dependency from 'shell' to 'impl'

 + L3VPNTransportTypes defines types that are used in 'shell', so I
   moved it from 'impl' to 'api'.

 + Also moved the dependency to 'itm-api' from 'impl' to 'api'

Change-Id: Ie792c40cf61af0bcefb83be9b0d6c5fcb6fb755c
Signed-off-by: Miguel Perez <francisco.miguel.perez@ericsson.com>

Updated git submodules

Project: federation master be26d98fc6a8c2da5c38ea394038eb69ac5a04ef

Migrate to odlparent 1.9.0

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

Updated git submodules

Project: integration/test master 545a0dcdfe47066579b05f21d49b8dc6312d05ce

Add service deletions to all (BGP-related) suites

Change-Id: Ieacb594a8fc9c23eb3e9b5d684e5ba59365a011d
Signed-off-by: Andrej Kilvady <akilvady@cisco.com>

Updated git submodules

Project: integration/test master 129c121903a81cd17cccf715af879702e51e4bb6

Fix handling of nested list responses

${resp}[2] is bad, @{resp}[2] is good.
@{resp}[0][2] does not work, handled using custom keyword.
This should fix MdsalLowlevelPy response handling.

Return tuple contains time first, prefix-id second.
Prefix_id is a string derived from integer member index argument.

Change-Id: If39ebc14f641be5383ed9fc6302a4bb8b38b9336
Signed-off-by: Vratko Polak <vrpolak@cisco.com>

Updated git submodules

Project: releng/builder master 3cc7eaa3efdeb8d7cebdac87f1aab7cb3e822387

Merge "Create baseline images for Ubuntu16.04 and Centos7"

Create baseline images for Ubuntu16.04 and Centos7

Packaging and builder jobs could simply use the baseline images
and does not require any extras.

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

Updated git submodules

Project: releng/builder master dad7352c82de9817a78d2e46fea4c4372b5809db

Update cloud image list docs

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

Updated git submodules

Project: aaa stable/carbon 2a5c42ef3b8e7962f70b00e8db17d32a57e9e053

Utility service to encode/decode public/private keys of type RSA, DSA and ECDSA.
Placing it here to make a common place to access this service.

Change-Id: I614a49fb7adc8fe569fc2959a79ce4a74fe6b548
Signed-off-by: Atul Gosain <agosain@brocade.com>
Merge "Fix a parameter description in the CLI" into stable/carbon

Merge "Bug 8382: disallow client specified ids" into stable/carbon

Merge "Remove <prerequisites><maven> from pom.xml to avoid WARNING" into stable/carbon

Fix a parameter description in the CLI

Minor fix for the user's email parameter description.

Change-Id: Iae54cebbba72ce315eaca3928ee822f401c08210
Signed-off-by: David Suarez <david.suarez.fuentes@ericsson.com>
(cherry picked from commit b3f7f080bd5381d2e037281e2215cf31c5e68cfa)

Merge "Bug 8352: Explicitly set the SecurityManager" into stable/carbon

Merge "Bug 8382: Update idmtool to not utilize ids in payload" into stable/carbon

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

Change-Id: I465de156a314f75098f7999c914d9cc6f0f7c89c
Signed-off-by: jenkins-releng <jenkins-releng@opendaylight.org>
Bug 8352: Explicitly set the SecurityManager

This issue is described more in depth in this thread:

http://shiro-user.582556.n2.nabble.com/ \
Shiro-with-Wicket-on-OSGi-ServiceMix-td7580484.html

We found that sometimes the SecurityManager for Shiro
is not set otherwise

Change-Id: I69829136f999d458058f0f112240cdf29bd1edc5
Signed-off-by: Ryan Goulding <ryandgoulding@gmail.com>
(cherry picked from commit eebc2121983af108f0e2c34e241bec69dbd9c7f1)

Bug 8382: Update idmtool to not utilize ids in payload

id is an internal implementation detail, as outlined by
bug 8382.  Prior patches were submitted to enforce that.
This patch just removes specification of the id as part
of the wrapped requests.

Change-Id: If1e12b899840b2c316400846bfcaf3c91d1ec11d
Signed-off-by: Ryan Goulding <ryandgoulding@gmail.com>
(cherry picked from commit eabafff5881afbe35b0aba05b5de4f100aff7681)

Bug 8382: disallow client specified ids

ids are an implementation specific internal detail.  Thus,
AAA should endpoints should prevent users from specifying
such values.

This change fixes the behavior by returning 400 when ids
are specified.  Appropriate test cases are added.

Change-Id: I15cc66f9881347334b919dbeb6bc686694367bd6
Signed-off-by: Ryan Goulding <ryandgoulding@gmail.com>
(cherry picked from commit 2052b21a43e51c996573e3f34c54adf89cce83d6)

Remove <prerequisites><maven> from pom.xml to avoid WARNING

[WARNING] The project ... uses prerequisites which is only intended for
maven-plugin projects but not for non maven-plugin projects. For such
purposes you should use the maven-enforcer-plugin. See
https://maven.apache.org/enforcer/enforcer-rules/requireMavenVersion.html

This is already done in odlparent.

Change-Id: Iedfaeb92fba3bf392554ece1f20358060c398a00
Signed-off-by: David Suarez <david.suarez.fuentes@ericsson.com>
(cherry picked from commit db8f4eda4dba15a467f4832ce4dd3a6857e4ae0d)

KeystoneAuthRealm: reuse http client

Jersey documentation advises against building multiple clients of the
same characteristics and advertises that is perfectly safe to use the
same client over multiple threads.

Moreover, each time the SSL context is obtained, it may synchronize the
keystores from the filesystem or the database. KeystoneAuthRealm
assumes a dynamic certificate handling.

The purpose of this commit is to reuse the http client to some degree
while refreshing it rather frequently due to the dynamic certificate
handling.

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

Merge "Do not override versions defined in odlparent" into stable/carbon

Merge "Bug 8214:  Avoid throwing NPE when DataBroker is null" into stable/carbon

Do not override versions defined in odlparent

This fixes commons-codec version duplication, cutting ~250kB
from distribution package.

Change-Id: I7812e5ac8ae02de25d9359eda12e78642cf33f41
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
Bug 8214:  Avoid throwing NPE when DataBroker is null

MDSALDynamicAuthorizationFilter is instantiated by shiro when
the web container is brought up.  shiro has no knowledge of
the DataBroker, so the dependency cannot be injected in shiro.ini.

shiro.ini needs to die, and is tracked by Bug 7793.  For now,
to avoid throwing the NPE (in the case that the DataBroker hasn't
been brought up yet), just fail-close until the dataBroker is
made available (i.e., AAAShiroProvider.newInstance(DataBroker)
is called).

Change-Id: I28e2eb0780f4f7761a5e2f81d167afc5a4b10cba
Signed-off-by: Ryan Goulding <ryandgoulding@gmail.com>
Bug 8373: Set destroy method for aaa h2 store

Change-Id: I3eeec2f11240c10cc70ebb7f52560cc12da19880
Signed-off-by: Andrej Mak <andrej.mak@pantheon.tech>
(cherry picked from commit 02df05821adbfac0e13404242eb59f376fff7abb)

Bug 8214 Prevent NPE in isolation scenario

If the datastore is not readable, then fail out gracefully.
Deny access to prevent unauthorized requests from
succeeding, and report a warning to karaf.log.

Change-Id: I0b40d27b3346d8a0357f41e8c68464057d4ec1c0
Signed-off-by: Ryan Goulding <ryandgoulding@gmail.com>
(cherry picked from commit e23fb31d5fd5cd940c9300c7c8ca5d132d18523c)

Merge "Do not override shiro-core version" into stable/carbon

Fix NPE in TokenAuthRealm

If the TokenStore is not available, do not throw a NPE.

Change-Id: Ie346e89dd1dc29c705f51712e58859065b81a787
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
(cherry picked from commit d053633f7e1f5fd94dd58a877835e58b09e5a0d0)

Do not override shiro-core version

This is a duplicate dependency, with managed version being 1.3.2.
Remove the duplicate declaration.

Change-Id: I4bd5f6121e2a03a0f7e72bb9d1d9566ba0d3df23
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
Bug 8313 hard code encry serv config

Updating the config seems to be causing issues in loading.

Change-Id: Iec467c589d56eb44a87f8e68b9f45ee2263b929b
Signed-off-by: Ryan Goulding <ryandgoulding@gmail.com>
Clean read-only transactions

Read-only transactions are documented to be a resource and should
be closed as soon as they are not needed. Failing to do so
triggers warnings in tell-based protocol in CDS, as they trigger
inefficient GC-triggered cleanup.

Change-Id: Ia8468704a75fd42c105f3fba49f817d5c2e0ec30
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
Bug 8214: Expose Service Interface for AAA

Hitherto, the tie between AAA and its downstream consumers has been
nebulous.  The instantiation of javax servlets requires runtime
dependencies to be resolved.  Bug 8214 exposes a race condition
in which RESTCONF attempts to resolve AAAFilter, but the AAA
service is not yet instantiated.  To resolve this, the solution
is to:

1) expose a AAAService interface.  In this case, that is quite
easy since we already have the necessary implementation,
AAAFilter.  Thus, all that is done is the extraction of the
service interface.

2) Advertise the service to the OSGI registry via blueprint.
This is a quick 3 liner.

A corresponding change will be submitted to the downstream
consumer (RESTCONF) to depend on AAAService.

Change-Id: I5d8208ddbe817963ce596ecc34ad38ff0f760410
Signed-off-by: Ryan Goulding <ryandgoulding@gmail.com>
(cherry picked from commit 6469cab6eb7d4eec7db440183f5a3b08f27eb763)

Update .gitreview to stable/carbon

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

Updated git submodules

Project: integration/test master 4475659d1f5ef8ef9979fcb7452210a63480c4e6

Workaround for bug 8636

Abort timout will be counted from transactions init, not from
node isolation.

Change-Id: Ia699b26294b98bc874d3aab8cb3d0dac422a8809
Signed-off-by: Peter Gubka <pgubka@cisco.com>

Updated git submodules

Project: netvirt stable/carbon c59ae8ba991130ee741401f06f0a19b5253fa737

Fixes:
 - BgpAlarms: avoid un-necessary reading of neighbor list

Change-Id: I4cded3511710cdeabd2fa8abb75cb35bf47b7bf7
Signed-off-by: SivaKumar Perumalla <sivakumar.perumalla@ericsson.com>

Updated git submodules

Project: releng/builder master 7c59c6d089a37274e43017f9265eeed3a7b63ecf

Merge "Update packaging jobs to use Ubuntu 16.04 images"

Update packaging jobs to use Ubuntu 16.04 images

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

Updated git submodules

Project: releng/builder master f799368e306fba0b0d813f81b6234aa82637fe51

Merge "Remove odlparent-validate-autorelease nitrogen job"

Remove odlparent-validate-autorelease nitrogen job

odlparent is no longer part of the autorelease nitrogen release so
remove the validate-autorelease job for the project.

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

Do not build libdoc in parallel

Attempt to fix RTD build. Suspecion is we might be starting too many
parallel processes to generate the docs and getting our process killed.

Change-Id: I6e7c98ddcd5398143bc6949038527f49d450ecd3
Signed-off-by: Thanh Ha <thanh.ha@linuxfoundation.org>
(cherry picked from commit a248f1b58bad6e83be48703eb600e8038d2a4389)

Updated git submodules

Project: releng/builder master 3db1010a1acc7ec4f4cd9ca4d4171b733fe8091b

Merge "Capture logs for all packaging jobs"

Capture logs for all packaging jobs

Using publishers is recommended for jobs to refer logs
at anytime, and also would help minimizing the loads on Jenkins.

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

Updated git submodules

Project: releng/builder master b62e4a1dd9912bf4ece24dd05485b6008cb212d2

Merge "Bug 8625: Remove Nitrogen Karaf 3 jobs"

Bug 8625: Remove Nitrogen Karaf 3 jobs

All jobs related to Karaf 3 in Nitrogen should be removed,
as TSC voted on not spending any cycles on Karaf 3 in Nitrogen.

Karaf 4 is the new default for CSIT in Nitrogen,
so it is expected some jobs start failing
for projects not fully working on Karaf 4.

Change-Id: Iae504c2acf6576f15cffa7f91b4cdcb37599ddbe
Signed-off-by: Vratko Polak <vrpolak@cisco.com>

Updated git submodules

Project: releng/builder master 5091cdcddb09d7459257401594e38811eb8a9478

Call openstack directly

Rather than using $PYTHON $OPENSTACK as have acceptable virtualenv paths
now.

Also fix ShellCheck in this file.

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

Updated git submodules

Project: releng/builder master 6844309cc3150aa23fbc6ea384aba930dd137274

Fix virtualenv usage in integration

Virtualenvs are in /tmp/v/{name}.
This inclused "openstack" venv.
Add missing --upgrade to every pip install line.

Change-Id: Ifd0dc429b6e088e9d86dea9ee57d33b2363ba7eb
Signed-off-by: Vratko Polak <vrpolak@cisco.com>

Updated git submodules

Project: genius stable/carbon 9a5c5ae535e3295006382ab011c7fc8078cbf1fb

Fixing a typo in DJC log statement

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

Updated git submodules

Project: genius stable/carbon 32ab887f3b4cc377e157fecfc4cc5fb4a50b49ec

Merge "Bug 8555 - Too many log info print to the console" into stable/carbon

Bug 8555 - Too many log info print to the console

Change-Id: I19133b5e7b2768ff5762bcf5d4a97994178a893b
Signed-off-by: 肖锋钢10072566 <xiao.fenggang@zte.com.cn>