Update commons-lang3 3.5 → 3.6

Release notes:
http://www.apache.org/dist/commons/lang/RELEASE-NOTES.txt

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

Remove <prerequisites><maven> avoid mvn 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

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

Remove @Deprecated bundles4test TestBundleDiag

as infrautils.ready does not use this anymore, now

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

Bump Checkstyle from 6.16 to 7.6.1

This (likely) requires some tweaks in projects enforcing CS;
needs testing using https://wiki.opendaylight.org/view/EvolvingCheckstyle

Motivated by having had to fix
https://git.opendaylight.org/gerrit/#/c/54978/

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

Fix aggregator-parent relativePath

Fixes aggregator-parent's incorrect relative path to odlparent-lite.

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

Bug 4290 Add missing org.osgi.service.event

Because apparently we need this, always (without it
opendaylight-karaf4-empty does not start anymore, following c/58053).

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

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>

Remove remote repositories

Our ODL Karaf distributions should never download artifacts from
maven.org or repository.springsource.com (we prepare an offline Maven
repo under the system/ directory in our Karaf distribution during
build).

This also adds the missing Apache Aries Quiesce API to system/ to fix
another problem; this may always have gotten downloaded at runtime, but
we never noticed this problem before removing the remote repos (above),
re. "Could not find artifact
org.apache.aries.quiesce:org.apache.aries.quiesce.api:jar:1.0.0"

I've noticed this while trying to work offline on the plane to SFO for
the DDF! ;)

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

Relax Checkstyle RightCurly rule for LITERAL_DO

The change [1] try to avoid checkstyle forcing this
"...} while (condition)" statement to be split in two lines by
removing the LITERAL_WHILE from odl_checkstyle, but the correct
token should have been LITERAL_DO.

LITERAL_WHILE avoids forcing this [2]:
 while(line !=null)
 { // We do not want this
   process(line);
   line = in.readLine();
 }

LITERAL_DO avoids forcing the following [3]:
 do
 { // We do not want this
   x = rand.nextInt(10);
 } // neither this
 while(x < 5);

[1] https://git.opendaylight.org/gerrit/#/c/55264/
[2] http://checkstyle.sourceforge.net/apidocs/com/puppycrawl/tools/checkstyle/api/TokenTypes.html#LITERAL_WHILE
[3] http://checkstyle.sourceforge.net/apidocs/com/puppycrawl/tools/checkstyle/api/TokenTypes.html#LITERAL_DO

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

Better gitignore rule for vim tmp files

Old one just caught the first backup, this catches additional.

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

Bug 8415 related: Make TestBundleDiag a lib

This is the alternative to c/56848 requested by Robert.

We're changing this so that we can consume bundles-test-lib in the
infrautils.ready implementation as a service for both Karaf v3 and v4.

As a (VERY nice!) side effect, this allows us to remove the duplicated
Java code we had between bundles-test and bundles4-test.  That was,
originally, intended to only be like it still is for a short while, but
with the Karaf 4 situation being what it is, this code duplication is
still there - and the respective code has started diverging a bit; so
unifying this in the new bundles-test-lib is great.

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

Pin requests to <2.16

The new requests library pulls in a chardet dependency that is in
conflict with coala packages. Pin the version of requests to ensure
we're pulling in a supported version.

Copied change by Thanh.

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

Pin coala to 0.11

coala-bears 0.11 just released 5 hrs from this patch author date. It
breaks the build because we were pinning coala. It seems coala and
coala-bears versions must match exactly so pin both versions for good
measure.

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

Pin coala version to <0.11

I seems the new 0.11 version breaks us due to dependency conflicts.
Let's pin the version and migrate when we have time to troubleshoot.

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

Bug 8176: Bump Pax Exam (and Pax URL) versions

to solve NoRouteToHostException on Fedora-based Linux

see https://ops4j1.jira.com/browse/PAXEXAM-740, and
http://blog2.vorburger.ch/2017/04/java-local-rmi-using-jdks-inetaddress.html

I've locally verified this does indeed fix the original problem.

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

blueprint-maven-plugin file move made conditional

This makes the move of the generated blueprint XML file conditional,
because we should not unconditionally hard code the move of the
blueprint XML files generated by the blueprint-maven-plugin.

This move from OSGI-INF/blueprint/autowire.xml to
org/opendaylight/blueprint should instead be conditional on the presence
of hand written XML in src/main/resources/org/opendaylight/blueprint
(which, typically, uses the ODL extensions).

This is because it should still be possible to use the @Singleton and
@Inject annotations with the XML generation by blueprint-maven-plugin in
bundles which do not depend on the MDSAL (e.g. in infrautils; I actually
have this case & problem in the real world).

Note that the gen. autowire.xml never uses the ODL extensions (unless we
do more work in the future to generate ODL extensions from annotations;
in which case the move should be made conditional on the XML content
actually containing/sing ODL extensions, not location).

Of course, normal ODL instance will always have the ODL controller mdsal
blueprint integration active.  But e.g. in ITs (e.g. in infrautils) it
seems "silly" to have to depend on the full mdsal JUST to get the
blueprint integration working JUST because we hard-coded and enforced
the move here.

This change has been tested by running a "mvn clean package" on e.g.
genius/interfacemanager/interfacemanager-impl, which is a typical ODL
bundle with a src/main/resources/org/opendaylight/blueprint/*.xml with
xmlns:odl="http://opendaylight.org/xmlns/blueprint/v1.0.0” as well as
some @Singleton in *.java, and verified that in the resulting JAR as
expected there are x2 XML in org/opendaylight/blueprint, namely the
handwritten say interfacemanager.xml as well as the generated
autowire.xml - both with and without this change; so no regressions.
Likewise, IFF there is no src/main/resources/org/opendaylight/blueprint,
then (only) the autowire.xml stays in OSGI-INF/blueprint (that's new,
and the goal of this change).

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

Bug 8483: Fix maven-dependency-plugin to v3.0.1

In the (vain?) hope that this may prevent NPEs; see bug.

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

Introduce new aggregator-parent POM

Just to avoid having that <skip>true for maven-deploy-plugin and
maven-install-plugin copy/pasted all over ODL for each *-aggregator.

This also makes changes as appropriate to use it in odlparent itself.

The intention is to start shifting to using this in other projects too.

Having a new parent artifact for all aggregators MAY also come in handy
when I start looking in earnest at our JavaDoc generation.

Q & TODO: Some, but not all, of the typical *-aggregator projects also
have a maven-site profile with maven-javadoc-plugin - should we move
that here as well? I'm hesitant because that sometimes has
<inherited>false .. which I'm not very familiar with - it would not do
anything with <inherited>false in a <parent>, would it? So do we include
it in aggregator-parent, but without <inherited>false ?  Or forget about
it and just start with a simple aggregator-parent as is proposed here
for now (and refine JavaDoc related stuff later) ?

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

Bug 8415 related: TestBundleDiag API improved

required for https://git.opendaylight.org/gerrit/#/c/56749/

The implicit throws org.awaitility.core.ConditionTimeoutException and
java.lang.AssertionError were not great.. an AssertionError is special,
and more importantly Awaitility's ConditionTimeoutException is an
implementation detail.

Awaitility is actually bundled as a private OSGi embedded JAR-in-JAR,
which is a bit like shaded fat über JARs with relocation in non-OSGi
Java built by the maven-shade-plugin or maven-assembly-plugin; so it was
wrong to throw an exception from it to a consumer of this utility.

Therefore, this introduces a new SystemStateFailureException instead.

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

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>

javax.inject dependencyManagement <optional>true

Because at production run-time in Karaf we do not actually need these
annotations - they are only consumed by the blueprint-maven-plugin to
generate XML.  (In component tests with Guice we do need them at
runtime, but then infrautils.inject.guice.testutils enforces
<optional>false via c/56724.)

Thanks to FELIX-2809, resolution:=optional actually gets added to the
OSGi MANIFEST.MF by the maven-bundle-plugin if in the POM a dependency
is marked as <optional>true.

NB that even though javax.inject.Inject & Co. have
@Retention(RetentionPolicy.RUNTIME), this does work - because in Java
annotations can be absent at runtime when loading classes (contrary to
other linked classes; annotations are handled specially, see JLS).

This should be safe & backwards compatible to change, because it just
means we don't have to have javax.inject at runtime anymore, but for
those existing features which include a wrap:mvn:javax.inject already,
it normally should not cause any disruption either.

see also https://ops4j1.jira.com/browse/PAXCDI-230

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

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>

Add pax-url-link & pax-exam-link-assembly

into dependencyManagement of odlparent's POM.

I'm experimenting with using http://veithen.github.io/alta/ to use Pax
URL's link: protocol and (eventually) completely do away with the mvn:
URLs from pax-url-aether (which is a stupid approach, not by us/ODL but
originally by Pax Exam).

Merging this isn't urgent yet, as I can just put the version into the
POM of where I'm experimenting with this, but ultimately it would be
good to have this in odlparent; just thought I'd raise it early.

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

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>
(cherry picked from commit 60cadf07b66747fd5e9823edcb8ef61b1f33b804)

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>

Use odl4-guava in odl4-akka-leveldb

Our Karaf 4 features should only use other Karaf 4 features.
odl4-akka-leveldb-0.7 ended up pulling odl-guava-18 instead of
odl4-guava-18... This patch fixes that, and changes odl4-guava-18 to a
dependency in odl4-akka-leveldb-0.7.

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

Update dependencies for Scala 2.11.11

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

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>

Simplify feature dependencies

Our Karaf 4 features include a number of dependencies which are also
transitive dependencies, and which don’t constitute the primary
API/runtime we’re interested in providing. This patch removes these
features (the generated features are identical).

Once we remove the corresponding Karaf 3 features we’ll also be able
to remove some of these dependencies from dependency management
altogether.

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

Allow logging feature dependency changes

This adds two properties controlling the Karaf feature generator’s
tracking features.

Setting checkDependencyChange to true enables checks and logs: when
features are generated, a simplified form  saved in src/main/history;
builds where this file is already present compare newly-generated
features against them, and log any failures. Any detected changes in
features are logged as a warning, with the detail of the changed
feature contents. It would be interesting to parse this in the builds
and send emails when changes are detected.

Setting failOnDependencyChange fails the build if a dependency change
is detected in the features.

This patch also enables both these flags in odlparent features where
it’s useful. This ensures that any changes to those features are
caught.

Change-Id: If9a1ebb2b218413f78a1bc28bfeb20d39b956675
Signed-off-by: Stephen Kitt <skitt@redhat.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>

Revert "Bug 8208: revert to the Karaf 4.0.7 rt"

This reverts commit dacab8c7e49bc18578ceb8f088333550cfe389ab.

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

Clean up lambdas

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

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.

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

Replace explicit types with <>

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

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>

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>

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>

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>

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>
(cherry picked from commit e9f860ea8c2c9a01b4e9e4f12e8ece04e62e91df)

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>

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>

Bump versions by x.(y+1).z for next dev cycle

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

Bug 4219: upgrade to Karaf 4.0.9

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

BUG-8117: fix jetty/pax-web configuration mismatch

Karaf 4 uses pax-web as the jetty configuration bridge, which
compares the jetty configuration with its directives. If a port
or address mismatch is found, this leads to duplicate ports being
used and hence preventing successful jetty startup.

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

Drop karaf-maven-plugin

We can’t use it within odlparent anyway, and the release of Karaf
4.0.9 means we don’t need it.

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

Add .tox/ to .gitignore

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

Bug 8175: Improve SFT TestBundleDiag under Karaf 4

This does NOT yet completely fix all problems of the SingleFeatureTest
TestBundleDiag under Karaf 4 (which was in really bad shape!), most
importantly there is still something somewhere in Pax Exam that I
haven't understood yet which seems to just "swallow" all exceptions
thrown by SingleFeatureTest's @Test installFeature() but at least now:

1. there is something in the log now for failures at least
   thanks to a (try/catch/log) WORKAROUND (!)

2. LinkageError fixed (see details below)

3. IllegalStateException: BundleContext is no longer valid fixed (?)

This changes the setup by just embedding bundles4-test and it's
dependencies into features4-test, instead of programmatically installing
the odl-bundles-test feature (which is removed in this change) or the
features4-test bundle.

This simplifies installation, avoid problems with the wrap: protocol,
and should resolve weird problems we've seen (only!) in the distribution
jobs, which seem to have had something to do with a custom local Maven
repo.

IMHO it would be *REALLY* good to practice proper TDD for SFT (with
TestBundleDiag) now, and have some automated negative testing of broken
features and bundles (incl. BP) in odlparent...

Bug: 7981 related later proper solution
Change-Id: I516c0f6c4e9ee85f0c753cb4b58a0b53fc22c7ea
Signed-off-by: Michael Vorburger <vorburger@redhat.com>
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Bug 8100: Karaf 4 SingleFeatureTest TestBundleDiag blacklist genius

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

added -XX JVM options to be able to analyze OOMs in SingleFeatureTest

  -XX:+HeapDumpOnOutOfMemoryError
  -XX:OnOutOfMemoryError="kill -3 %p"

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

Add Git Commit Coala Bear

This enforces line lengths in commit messages.

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

Add cli property to un-skip git-commit-id skip flag

Change-Id: I88340ab60e3e922fd3c79fe01f6b60471eec55e7
Signed-off-by: Sam Hague <shague@redhat.com>
Signed-off-by: Michael Vorburger <vorburger@redhat.com>

Improve whitespace negligibly

The motivation is to run verify on the current code,
but no harm merging this.

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

Revert "Bug 7588: Stop blacklisting odl-hbaseclient from SFT"

This reverts commit 297793d5f9faa091323e07f601e96641df9fe5ff.

Before reverting this revert, make sure Tsdr verify (or merge) works,
as opposed to distribution-check or multipatch.

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

Fix checkstyle FinalClass issue

Fix the FinalClass checkstyle violation described in this pending
change:
https://git.opendaylight.org/gerrit/#/c/43324/

Move the constructor to be the first method.

The same fix solved in this patch:
https://git.opendaylight.org/gerrit/#/c/53250/

Change-Id: I1986199ee2b761e1b73cac31f2f1630f57e73b64
Signed-off-by: David <david.suarez.fuentes@ericsson.com>

Fix checkstyle FinalClass issue

Fix the FinalClass checkstyle violation described in this pending
change:
https://git.opendaylight.org/gerrit/#/c/43324/

Move the constructor to be the first method.

Change-Id: I475ab18c233655ebfb0df72ddf2dad8e5a8d8bdd
Signed-off-by: David <david.suarez.fuentes@ericsson.com>

Bug 7588: Stop blacklisting odl-hbaseclient from SFT

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

Bug 7895: Fix java version regex in Karaf scripts

This only fixes Karaf 3 scripts.
Any idea how to reliably create .patch files for Karaf 4 scripts?

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

Minor (very) clean up of SFT related code I bumped into

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

Upgrade commons-net 3.5 → 3.6

Release notes:
http://www.apache.org/dist/commons/net/RELEASE-NOTES.txt

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

Remove invalid code in Karaf 3 instance script

If jvm version is lower than 1.8.0, the checkJvmVersion
function will exit. As a result, check jvm version code
is invalid in subsequent setupDefaults function.

Karaf 4 patchs files are not updated.

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

Do not package protobuff-java

Change-Id: I3261557e16b338beb7f95710f895118d2e057587
Signed-off-by: Robert Varga <rovarga@cisco.com>

Replace exit with die function in Karaf script.

die function is a unified error handling function, which was used
anyplace except checkJvmVersion funciton in Karaf script.

Change-Id: I2d22e0967c4fb4c55feffe830f22cb7d02017540
Signed-off-by: Ding Rui <ding.rui@zte.com.cn>

Remove invalid code in Karaf scripts

If jvm version is lower than 1.8.0, the checkJvmVersion function will
exit. As a result, check jvm version code is invalid in subsequent
setupDefaults function.

Change-Id: I0cda1473326e6c90792fa6452c23a0a1dcd89719
Signed-off-by: Ding Rui <ding.rui@zte.com.cn>

SingleFeatureTest (SFT) TestBundleDiag with full diag during wait loop

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

Checkstyle: no space inside parentheses

Add the ParenPad check to odl_checks.xml, which makes sure there is no
space after a left parenthesis and before a right parenthesis.

Change-Id: I2b843fa150b80b14043147377339b9c633e68b91
Signed-off-by: Lorand Jakab <lojakab@cisco.com>
Signed-off-by: Robert Varga <rovarga@cisco.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/52522

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

Add akka-persistence-tck dependency

Change-Id: I208c169bd59da87d9e573b3de5e8617441f26c38
Signed-off-by: Tom Pantelis <tpanteli@brocade.com>

Bump bouncycastle dependencies from 1.55 to 1.56

Comprehensive release notes can be found here:

https://www.bouncycastle.org/releasenotes.html

This release fixes the following:

*    It is now possible to configure the provider to only import keys for specific named curves.
*    Work has been done to improve the "constant time" behaviour of the RSA padding mechanisms.
*    The GCM ciphers in the JCE and lightweight API will now fail if an attempt is made to use
     them for encryption after doFinal or without changing the IV.
*    The constructor for IESParameterSpec that allows the use of cipher without a nonce has been
     deleted. See also details for CVE-2016-1000344, CVE-2016-1000352.
*    Strict encoding enforcement has been introduced for ASN1Integer.
*    CVE-2016-1000338: DSA does not fully validate ASN.1 encoding of signature on verification.
     It is possible to inject extra elements in the sequence making up the signature and still
     have it validate, which in some cases may allow the introduction of "invisible" data into a
     signed structure.
*    CVE-2016-1000339: AESFastEngine has a side channel leak if table accesses can be observed.
     The use of lookup large static lookup tables in AESFastEngine means that where data accesses
     by the CPU can be observed, it is possible to gain information about the key used to
     initialize the cipher. We now recommend not using AESFastEngine where this might be a concern.
     The BC provider is now using AESEngine by default.
*    CVE-2016-1000340: Static ECDH vulnerable to carry propagation bug. Carry propagation bugs in
     the implementation of squaring for several raw math classes have been fixed
     (org.bouncycastle.math.raw.Nat???). These classes are used by our custom elliptic curve
     implementations (org.bouncycastle.math.ec.custom.**), so there was the possibility of rare
     (in general usage) spurious calculations for elliptic curve scalar multiplications. Such
     errors would have been detected with high probability by the output validation for our scalar
     multipliers.
*    CVE-2016-1000341: DSA signature generation vulnerable to timing attack. Where timings can be
     closely observed for the generation of signatures, the lack of blinding in 1.55 or earlier,
     may allow an attacker to gain information about the signatures k value and ultimately the
     private value as well.
*    CVE-2016-1000342: ECDSA does not fully validate ASN.1 encoding of signature on verification.
     It is possible to inject extra elements in the sequence making up the signature and still have
     it validate, which in some cases may allow the introduction of "invisible" data into a signed
     structure.
*    CVE-2016-1000343: DSA key pair generator generates a weak private key if used with default
     values. If the JCA key pair generator is not explicitly initialised with DSA parameters,
     1.55 and earlier generates a private value assuming a 1024 bit key size. In earlier releases
     this can be dealt with by explicitly passing parameters to the key pair generator.
*    CVE-2016-1000344: DHIES allows the use of unsafe ECB mode. This algorithm is now removed from
     the provider.
*    CVE-2016-1000345: DHIES/ECIES CBC mode vulnerable to padding oracle attack. For BC 1.55 and
     older, in an environment where timings can be easily observed, it is possible with enough
     observations to identify when the decryption is failing due to padding.
*    CVE-2016-1000346: Other party DH public key not fully validated. This can cause issues as
     invalid keys can be used to reveal details about the other party's private key where static
     Diffie-Hellman is in use. As of this release the key parameters are checked on agreement
     calculation.
*    CVE-2016-1000352: ECIES allows the use of unsafe ECB mode. This algorithm is now removed
     from the provider.

This upgraded provider provides a great number of security fixes.  We should consider moving to
the updated version swiftly.

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

Generator: add plugin configuration for prereq/deps

This adds two lists of features as configuration elements in
karaf-maven-plugin, dependencyFeatures and prerequisiteFeatures;
features named there (and defined elsewhere, in the POM) will be
flagged as dependencies and prerequisites respectively.

Bug: 7752
Bug: 7753
Change-Id: Ice35279822f513945df08f70880edcc74395da33
Signed-off-by: Stephen Kitt <skitt@redhat.com>

BUG-7813: remove spring from base distribution

We do not use spring anywhere in the project, hence we can safely
remove it from the distribution, cutting it down by 35MB.

Change-Id: I07e6c8f0996d0d9cc9b0d6c50195a9fbb9f448b7
Signed-off-by: Robert Varga <rovarga@cisco.com>

Add syslog log4j appender entry, disabled by default

Add correct formatting for enabling syslog appender for ODL.  By default,
the appender is disabled.

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

Forbid use of mockito-all via a new maven-enforcer-plugin rule

mockito-all is bad, because it includes other libs (non-shaded), they
never intended it to be used from system with transitive dependency
management such as Maven; it can cause issues incl. fix
NoSuchMethodError: org.hamcrest.Matcher ...

see
http://maven.apache.org/enforcer/enforcer-rules/bannedDependencies.html

We'll remove mockito-all from dependencyManagement a little later.

Bug 7662 related
Change-Id: Id0da73a875cbb6de481782db1ccaadd6ac54ea6f
Signed-off-by: Michael Vorburger <vorburger@redhat.com>

Bump akka to 2.4.17

http://akka.io/news/2017/02/10/akka-2.4.17-released.html

Change-Id: Ib0de05829fe1c9f581ec89fab7e0b22f69dd5910
Signed-off-by: Robert Varga <rovarga@cisco.com>

Generator: merge existing dependency information

This patch allows the generator to merge dependency information from
feature.xml stubs and POMs ("dependency" and "prerequisite" flags).
This allows version information to be pulled from the POMs and still
allow the flags to be set.

Bug: 7752
Bug: 7753
Change-Id: I2b5147dc1636b4c9a3ec3b77654c9994702c7877
Signed-off-by: Stephen Kitt <skitt@redhat.com>

Skip site generation for karaf-maven-plugin

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

Add Maven site information

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

Import the Karaf Maven plugin

This is a temporary measure to enable quick iteration on new features
in the descriptor generator. New features there will as far as
possible be submitted upstream.

Having this as a plugin in odlparent will allow us to drop the
4.0.8-odl artifacts and should resolve the Maven site plugin issues.

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

BUG-7767: upgrade Jetty to 9.2.19

This bumps the version of jetty we are pulling in by our proxy
feature, which should result in an all-round upgrade.

Change-Id: I766a6489ce467590b03744f086a761ec4535132e
Signed-off-by: Robert Varga <rovarga@cisco.com>

Bump netty to 4.1.8

See [0] for release notes.

While at it, also update API link from 4.0.x to 4.1.x.

[0] http://netty.io/news/2017/01/30/4-0-44-Final-4-1-8-Final.html

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

Karaf 4: remove the war feature

In Karaf 4, the war feature ends up pulling in Jetty 9.2, which we
aren't ready for yet.

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

Jetty 9.2 upgrade: add h2 to dependency management

This allows the Jetty upgrade to be driven from odlparent, without
requiring synchronised commits to odlparent and aaa (once aaa starts
using this dependency management).

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

Checkstyle: no space inside parentheses

Fix violations of ParenPad rule so it can be enabled at a later point.

Change-Id: I17968e48ca25b09be9dea7dac355612b81c7e606
Signed-off-by: Lorand Jakab <lojakab@cisco.com>
Signed-off-by: Robert Varga <rovarga@cisco.com>

Revert "Checkstyle: no space inside parentheses"

This reverts commit 28204b1caf363e7f7ae8d184ad305cf958a24608.

Breaks controller, we certainly do not need any more breakage at this point.

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

Downgrade javax.mail version

Installing both jdbc and jetty-8 features causes a conflict on
which version of javax.mail to use, as they pull in 1.4.4 and 1.4.5
respectively.

Resolve this by downgrading the version in jetty-8.

Change-Id: Icfa2fe3d5f87b7fc97405f94fe7bf33113178a75
Signed-off-by: Robert Varga <rovarga@cisco.com>

Remove unneeded groupId

Maven plugins have an implied groupId, there is no need to be
explicit.

Change-Id: I6c0d3aa532be7c1c3255e2a1cc5e927a96bb66c4
Signed-off-by: Robert Varga <rovarga@cisco.com>

Add version declaration for maven-remote-resources-plugin

Fixes a warning about undeclared version.

Change-Id: I47118df48ee7f3b7d0d09c090228a46d70acbd68
Signed-off-by: Robert Varga <rovarga@cisco.com>

Pull in karaf's enterprise repository for jdbc

This seems to be causing downstream grief, while passing SFT here.
Pull in the repository explicitly.

Change-Id: I963fa67fe7470f155bae459ba2b2cdf8a2f874fa
Signed-off-by: Robert Varga <rovarga@cisco.com>

Checkstyle: no space inside parentheses

Add the ParenPad check to odl_checks.xml, which makes sure there is no
space after a left parenthesis and before a right parenthesis. Fix the
violations in odlparent.

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

Add odl-karaf-feat-jetty and odl-karaf-feat-feature

Jetty is yet another feature we are using, provide a proxy for it.
Also note that we should be upgrading it.

feature service is needed by config-adaptor in controller, so add
that as well.

Change-Id: Idb5049ca822f9fb2a7383e79ca04d40c37eaa8eb
Signed-off-by: Robert Varga <rovarga@cisco.com>

Add single features wrapping karaf features

Auto-generated features have no way of referring to core
karaf features, hence we need to wrap them. This provides
jdbc and war needed for aaa project.

Change-Id: Ia3a9cc3a926b6f7d00a001f604adda9cae551851
Signed-off-by: Robert Varga <rovarga@cisco.com>

Karaf 4: make odl-akka-leveldb-0.7 use the Guava feature

The LevelDB feature ends up pulling in Guava, so make it use our Guava
feature explicitly.

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

Karaf 4: exclude artifacts provided by our distribution

This is a first pass, covering all the controller features'
dependencies which are (now) provided by our Karaf distribution: Aries
Blueprint, Aries Quiesce, jline, sshd, and the core Karaf JAAS and
shell bundles.

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

Revert "FindBugs' full :annotation instead of :jsr305 artifact sub-set"

This reverts commit 4647724a79fbf7db440716f93f1067d17ecd8dda.

It breaks yangtools' and bgpcep's SFT, I suspect it breaks others
as well.

Change-Id: I880c3835958edac14a608fb8f7346c2274298c83
Signed-off-by: Robert Varga <rovarga@cisco.com>

FindBugs' full :annotation instead of :jsr305 artifact sub-set

This will require that all downstream projects first switch their
<dependency><groupId>com.google.code.findbugs from <artifactId>jsr305 to
<artifactId>annotations in all pom.xml.  They will NOT have to adapt any
code, because the actual Java types are exactly the same (just from a
different Maven artefact).

see also discussion in https://git.opendaylight.org/gerrit/#/c/46914/

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

SFT: avoid testing aggregator features

Feature repositories built by the Karaf Maven plugin contain an
aggregator feature which pulls in all the features in the relevant
POM. It's pointless to test this, and very expensive, so we now skip
them: the rule is that, in repositories with more than one feature,
any feature with the same name as the repository is skipped.
(Repositories containing a single feature are tested, so that single
features can be checked in -Psft mode.)

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

Use equinox instead of felix for SFT

Default felix has some problem with attaching optional imports,
breaking downstream features. Switch to using equinox, which is
actually what we package.

Change-Id: I6813f2a5e3d9343fb59ec52d12e0367ca07f71b9
Signed-off-by: Robert Varga <rovarga@cisco.com>

Bump pax-exam to 4.10.0

Just an usual upgrade.

Change-Id: Ie642793bb1388a1bb700a9e0859c301f86567b2a
Signed-off-by: Robert Varga <rovarga@cisco.com>

Fix karaf's aries-blueprint feature

quiesce.api is something we are using and since blueprint-core has
it as an optional dependency, we end up refreshing blueprint when
we install it. That leads to pax-exam thinking the test during SFT
has ended and proceeds to the next test -- wreaking havoc in process
of doing so.

Patch the upstream feature file so quiesce.api is installed
unconditionally, so we do not have to install it from a feature,
hence we won't trigger blueprint refresh.

Change-Id: I14ad8b70d1dc8a961944da9087a54242ba017195
Signed-off-by: Robert Varga <rovarga@cisco.com>