Add sensible defaults for metadata required for Maven Central deployment

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

Checkstyle now also covers src/test/java & src/test|main/resources

But license headers are not enforced in src/test/java - not because that
wouldn't be nice, but just because it is way too much work to fix up
everything - I've tried, and then realized that's not realistic.

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

Checkstyle new rule ConstantName

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

Checkstyle IllegalThrows with ignoreOverriddenMethods

The default is true, and for good reason; it was stupid of me to
originally have used ignoreOverriddenMethods = false.

This is because when you @Override a method, you cannot do anything
anymore about its throws clause - you have to comply with what you are
overriding.

Based on doing some real world Checkstyle clean up work and noticing
that this was wrong.

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

Add NoCleanIfFailure option to SingleFeatureTest

This will as shown much clearer errors in case of SFT failures; e.g.
"java.lang.IllegalStateException: Can't install
feature odl-integration-compatible-with-all/0.5.0-SNAPSHOT:
Could not start bundle mvn:org.opendaylight.snmp4sdn/snmp4sdn/0.4.0-SNAPSHOT
in feature(s) odl-snmp4sdn-snmp4sdn-0.4.0-SNAPSHOT: The bundle
"org.opendaylight.snmp4sdn_0.4.0.SNAPSHOT [504]" could not be resolved.
Reason: Missing Constraint: Import-Package: org.opendaylight.controller.sal.packet; version="[0.7.0,1.0.0)""

as shown on https://lists.opendaylight.org/pipermail/odlparent-dev/2016-July/000467.html

Change-Id: I5b5df8807e87c76975598a0f3311010b781227c6
Signed-off-by: Yakir Dorani <yakir.dorani@hpe.com>

Option to performance profile SingleFeatureTest

with Java Mission Control FlightRecorder

via -Dkaraf.featureTest.profile=true

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

Quick (-Pq) should skip running tests, but not building them

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

Intro. Google Truth into dependencyManagement

see http://google.github.io/truth/.  This seems neater than Hamcrest in
particular for tests on Iterables (Collections).  For example, in
Hamcrest:

assertThat(Arrays.asList("foo", "bar"), contains("foo", "baz")); // OK

assertThat(Arrays.asList("foo", "bar"), contains(Arrays.asList("foo",
"bar"))); // NOK! :-( because contains only works for vararg not List

whereas with Google Truth:

assertThat(Arrays.asList("foo", "bar")).
containsExactlyElementsIn(Arrays.asList("foo", "baz"));

==> AssertionError: Not true that <[foo, bar]> contains exactly <[foo,
baz]>. It is missing <[baz]> and has unexpected items <[bar]>

also see http://google.github.io/truth/comparison

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

Increase enforced Maven minimal version from 3.1.1 to 3.2.5 (no upper)

as discussed on [odlparent-dev] thread "Re: Introducing PMD Copy/Paste
Detector (CPD)"

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

Introduce PMD Copy/Paste Detector (CPD)

Take II - is using <excludeRoots><excludeRoot> better? (We tried this
before and had to revert, despite it passing on Jenkins; very strange.)

Like Checkstyle, this by default only reports to the console during
(local..) builds, but does not enforce it - builds will NOT fail with
this. Then projects ready to "opt-in" for CPD enforcement can use:
<properties> <pmd.cpd.fail>false</pmd.cpd.fail> </properties>

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

Checkstyle java.nio.charset.StandardCharsets instead of com.google.common.base.Charsets

http://blog2.vorburger.ch/2016/06/how-to-use-checkstyle-to-enforce-using.html

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

Revert "Introduce PMD Copy/Paste Detector (CPD)"

This reverts commit 68afa051f302f5f67aedd42bf1ab2f4cc20d75c7.
It breaks build in yangtools: checkstyle fails to parse
configuration and cannot assign String to File[].

Change-Id: I5f9025c1dd79c19f10e8c17a56e270d17cbda043
Signed-off-by: Robert Varga <nite@hq.sk>

Upgrade Felix dependencies

* Dependency Manager 3.1.0 -> 4.3.0
* Dependency Manager Shell 3.0.1 -> 4.0.4
* File Install 3.1.6 -> 3.5.4
* Web Console 4.2.0 -> 4.2.14

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

Introduce PMD Copy/Paste Detector (CPD)

Like Checkstyle, this by default only reports to the console during
(local..) builds, but does not enforce it - builds will NOT fail with
this. Then projects ready to "opt-in" for CPD enforcement can use:

  <properties>
    <pmd.cpd.fail>false</pmd.cpd.fail>
  </properties>

BTW: The "minimumTokens" threshold may take some tuning - let's start
with these approx. 10 line rule, and then adjust it as we see how this
plays out in practice on a number of projects.

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

Mark slf4j-api as provided

Since Karaf provides its own SLF4J encapsulation, it might make sense
to simply identify slf4j-api as provided.

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

akka-testkit should be scoped to test

This is a testing framework for testing actor systems,
it should never be carried in production.

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

Checkstyle remove duplicate UpperEll (minor)

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

Introduce hamcrest-library, going beyond hamcrest-core, to the stars..

http://www.vogella.com/tutorials/Hamcrest/article.html
https://code.google.com/archive/p/hamcrest/wikis/Tutorial.wiki

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

Relax Checkstyle for JavaDoc, not mandatory for annotations

Because in at least one instance a project (ovsdb/library;
https://git.opendaylight.org/gerrit/#/c/40967/) has adopted Checkstyle
99% but then disabled enforcement because of lack of JavaDoc in a few
annotation types.  This is the wrong way to go about this IMHO; I'd
rather we relax this arbitrary requirement (that even annotations do not
have to have JavaDoc, like everything else), and be able to enforce
Checkstyle in more projects.

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

Intro. Xtend dependencyManagement and Maven plugin

Xtend <http://xtend-lang.org> is already used in ODL's MDSAL project,
and I'd like to start using it in other projects as well (for writing
test code, for now).  In order to keep the version of Xtend used
uniform, and to keep child pom.xml shorter re. the xtend-maven-plugin
configuration, it would thus make sense to have this in the ODL parent.

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

Add guava-testlib to dependencyManagement

Guava Testlib has a neat EqualsTester(), and other test helpers.

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

Add skip for pmd & cpd to Q

Even though PMD / CPD aren't actually used yet, I'm planning to propose
gradually introducing it, and having this already in Q will help not
forgetting it, and people not complaining.  It does not hurt having this
in early.

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

Checkstyle rulez for Exception Handling Anti-Patterns

* http://checkstyle.sourceforge.net/config_coding.html#IllegalCatch
* http://checkstyle.sourceforge.net/config_coding.html#IllegalThrows
* http://checkstyle.sourceforge.net/config_design.html#MutableException

and TBD:
* http://sevntu-checkstyle.github.io/sevntu.checkstyle/apidocs/com/github/sevntu/checkstyle/checks/coding/AvoidHidingCauseExceptionCheck.html
* http://sevntu-checkstyle.github.io/sevntu.checkstyle/apidocs/com/github/sevntu/checkstyle/checks/coding/EitherLogOrThrowCheck.html

Includes fixing "Catching 'Exception' is not allowed" in
SingleFeatureTest.

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

Checkstyle enforce no redundant modifiers

see
http://checkstyle.sourceforge.net/config_modifier.html#RedundantModifier

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

Increase SingleFeatureTest timeout from 5 to 10 min.

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

SingleFeatureTest PerRepoTestRunner logging

Clarified; description includes feature name which is what you want to
show here, instead of just features.xml. System.out exceptionally used
because, whatever I did, I could never see that log otherwise..

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

maven-enforcer-plugin requireJavaVersion 1.8.0

but we should probably move this to odlparent-lite, not just odlparent,
so that it already works for <packaging>pom roots.

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

Checkstyle System.out violations only in Java not in TXT etc.

In Java sources, still do not permit them even commented out.

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

Bump akka to 2.4.7

The regression introduced in 2.4.6 has been fixed,
bump akka to latest.

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

Upgrade Maven plugins

(maven-surefire-plugin is left out because 2.19.1 breaks our tests.)

build-helper-maven-plugin 1.9.1 -> 1.10
 * 1.10: http://www.mojohaus.org/build-helper-maven-plugin/github-report.html

maven-bundle-plugin 3.0.0 -> 3.0.1
 * http://www.mail-archive.com/users%40felix.apache.org/msg16914.html

maven-clean-plugin 2.6.1 -> 3.0.0
 * 3.0.0: http://mail-archives.apache.org/mod_mbox/maven-announce/201510.mbox/%3C20151023060827.136D6188B5%40minotaur.apache.org%3E

maven-compiler-plugin 3.3 -> 3.5.1
 * 3.5.1: http://mail-archives.apache.org/mod_mbox/maven-announce/201602.mbox/%3CCAK2FUK3FfTSZSeQybK_4KXhFbtRNZ1uo6wxEbfnnyy5MCh1DsQ%40mail.gmail.com%3E
 * 3.5: http://mail-archives.apache.org/mod_mbox/maven-announce/201601.mbox/%3CCAK2FUK1kNz9di_SAUDBO2VjD3XLEL%3D0XLJkLkg68Q7x4ChJh-w%40mail.gmail.com%3E

maven-jar-plugin 2.6 -> 3.0.0 (pending
https://git.opendaylight.org/gerrit/40175)
 * 3.0.0: http://mail-archives.apache.org/mod_mbox/maven-announce/201605.mbox/%3C20160515194703.8B01819730%40minotaur.apache.org%3E

maven-project-info-reports-plugin 2.8.1 -> 2.9
 * 2.9: http://mail-archives.apache.org/mod_mbox/maven-announce/201603.mbox/%3C56DC1611.1050409%40apache.org%3E

maven-release-plugin 2.5.2 -> 2.5.3
 * http://mail-archives.apache.org/mod_mbox/maven-announce/201510.mbox/%3CCALhtWkezhk9ykpxzx-1T7qiyt7VZ88Nn1R%2BsSz9iK%2Buv3r%2B4aA%40mail.gmail.com%3E

maven-resources-plugin 2.7 -> 3.0.1
 * 3.0.1: http://mail-archives.apache.org/mod_mbox/maven-announce/201606.mbox/%3C20160608173951.D512D19E18%40minotaur.apache.org%3E
 * 3.0.0: http://mail-archives.apache.org/mod_mbox/maven-announce/201605.mbox/%3C20160521184859.A223F18263%40minotaur.apache.org%3E

maven-shade-plugin 2.4.1 -> 2.4.3
 * 2.4.3: http://mail-archives.apache.org/mod_mbox/maven-announce/201601.mbox/%3CCAKuVzBS2BeJoxXi3Yotbbyy5kGJemVSSCNr3%2BYW3ytz51G2wbQ%40mail.gmail.com%3E
 * 2.4.2: http://mail-archives.apache.org/mod_mbox/maven-announce/201510.mbox/%3C20151029055337.CB8AC18CD5%40minotaur.apache.org%3E

maven-site-plugin 3.4 -> 3.5.1
 * 3.5.1: http://mail-archives.apache.org/mod_mbox/maven-announce/201604.mbox/%3C7199325.eTch8dpi9I%40herve-desktop%3E
 * 3.5: http://mail-archives.apache.org/mod_mbox/maven-announce/201602.mbox/%3C1518487.5YI36Fm8fR%40herve-desktop%3E

maven-source-plugin 2.4 -> 3.0.0
 * 3.0.0: http://mail-archives.apache.org/mod_mbox/maven-announce/201602.mbox/%3C20160218044420.1FAD3185C6%40minotaur.apache.org%3E

propertymavenplugin 1.0-alpha-2 -> 1.0.0
 * http://www.mojohaus.org/properties-maven-plugin/github-report.html

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

Update plugin property info

checkstyle.runtime.version, gmaven.plugin.version,
jar.plugin.version, and maven.release.version aren't used outside
odlparent, remove them.

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

Change Checkstyle allowedAbbreviationLength from 3 to 4 ("managerUUID")

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

Fix Checkstyle warnings, and make them errors for the future

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

Add apache xerces declaration

Dealing with Xerces is fun, except when we pull in wrong version of
xml-apis. As turns out, 1.4.01 is actually newer than 2.0.2 (based
on search.maven.org). Adjust the xml-apis version and declare xerces
dependency.

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

Add mvn install -DaddInstallRepositoryPath=.../karaf/system

http://blog2.vorburger.ch/2016/06/maven-install-into-additional.html

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

Checkstyle Import order as per original ODL convention on Wiki

see discussion on dev@lists.opendaylight.org email thread "Import
ordering: Going Wiki or Checkstyle way?"

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

Use ${project.remoteProjectRepositories} in karaf-plugin

This Change is inspired by https://issues.jboss.org/browse/JBASMP-40

Without this, https://git.opendaylight.org/gerrit/#/c/40432/1 fails,
as karaf-plugin picks repositories defined for itself,
and not for the project it is supposed to handle.

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

Checkstyle JavaDoc presence not enforced, but if present, check style

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

Revert "Bump maven-failsafe-plugin version from 2.18.1 to 2.19.1, to solve NoClassDefFoundError"

This reverts commit fb0e070b9c331b4487ee476673f5104e0d4f12d9.

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

Bump maven-failsafe-plugin version from 2.18.1 to 2.19.1, to solve NoClassDefFoundError

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

[INFO] ------------------------------------------------------------------------
[INFO] Reactor Summary:
[INFO]
[INFO] archetypes-parent .................................. SUCCESS [  2.431 s]
[INFO] odl-model-project .................................. SUCCESS [  9.970 s]
[INFO] opendaylight-configfile-archetype .................. SUCCESS [  3.365 s]
[INFO] distribution-karaf-archetype ....................... SUCCESS [ 24.224 s]
[INFO] opendaylight-karaf-features-archetype .............. SUCCESS [02:12 min]
[INFO] opendaylight-startup-archetype ..................... FAILURE [05:31 min]
[INFO] ------------------------------------------------------------------------
[INFO] BUILD FAILURE
[INFO] ------------------------------------------------------------------------
[INFO] Total time: 08:23 min
[INFO] Finished at: 2016-06-14T14:25:26+02:00
[INFO] Final Memory: 50M/1025M
[INFO] ------------------------------------------------------------------------
[ERROR] Failed to execute goal org.apache.maven.plugins:maven-failsafe-plugin:2.18.1:integration-test (failsafe-integration-tests) on project opendaylight-startup-archetype: Execution failsafe-integration-tests of goal org.apache.maven.plugins:maven-failsafe-plugin:2.18.1:integration-test failed: There was an error in the forked process
[ERROR] java.lang.NoClassDefFoundError: projects/basic/project/basic/it/target/test-classes/it/pkg/it/BasicIT (wrong name: it/pkg/it/BasicIT)
[ERROR] at java.lang.ClassLoader.defineClass1(Native Method)
[ERROR] at java.lang.ClassLoader.defineClass(ClassLoader.java:763)
[ERROR] at java.security.SecureClassLoader.defineClass(SecureClassLoader.java:142)
[ERROR] at java.net.URLClassLoader.defineClass(URLClassLoader.java:467)
[ERROR] at java.net.URLClassLoader.access$100(URLClassLoader.java:73)
[ERROR] at java.net.URLClassLoader$1.run(URLClassLoader.java:368)
[ERROR] at java.net.URLClassLoader$1.run(URLClassLoader.java:362)
[ERROR] at java.security.AccessController.doPrivileged(Native Method)
[ERROR] at java.net.URLClassLoader.findClass(URLClassLoader.java:361)
[ERROR] at java.lang.ClassLoader.loadClass(ClassLoader.java:424)
[ERROR] at sun.misc.Launcher$AppClassLoader.loadClass(Launcher.java:331)
[ERROR] at java.lang.ClassLoader.loadClass(ClassLoader.java:357)
[ERROR] at org.apache.maven.surefire.util.DefaultScanResult.loadClass(DefaultScanResult.java:131)
[ERROR] at org.apache.maven.surefire.util.DefaultScanResult.applyFilter(DefaultScanResult.java:95)
[ERROR] at org.apache.maven.surefire.junit.JUnit3Provider.scanClassPath(JUnit3Provider.java:137)
[ERROR] at org.apache.maven.surefire.junit.JUnit3Provider.invoke(JUnit3Provider.java:88)
[ERROR] at org.apache.maven.surefire.booter.ForkedBooter.invokeProviderInSameClassLoader(ForkedBooter.java:203)
[ERROR] at org.apache.maven.surefire.booter.ForkedBooter.runSuitesInProcess(ForkedBooter.java:155)
[ERROR] at org.apache.maven.surefire.booter.ForkedBooter.main(ForkedBooter.java:103)
[ERROR] -> [Help 1]

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

Introduce "mvn -Pq install" to just build JAR, but no tests, QA etc.

@see #also http://memory-alpha.wikia.com/wiki/Q ;)

Bumped maven-source-plugin from 2.4 to 3.0.0 just because
maven.source.skip appears to be broken with 2.4 (it gen. the
*-sources.jar anyway) but with 3.0.0 it's correctly "Skipping source per
configuration."

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

Upgrade Apache Shiro 1.2.4 to 1.2.5

Fixes include the following:
[SHIRO-443] - SessionValidationScheduler created multiple times, enabling it is not thread safe
[SHIRO-462] - Authentication exceptions are swallowed
[SHIRO-467] - Authentication exception gets swallowed
[SHIRO-550] - Randomize default remember me cipher
[SHIRO-504] - Java 8 support
[SHIRO-516] - Explicitly specify the version of aspectjtools to avoid build warning
[SHIRO-562] - WildcardPermission calls String.trim() twice in setParts()

Release notes are located here:

https://issues.apache.org/jira/secure/ReleaseNote.jspa?projectId=12310950&version=12332994

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

Revert "Upgrade Netty 4.0.37.Final -> 4.1.1.Final"

This reverts commit 34a2b5c5d7d09af6dad600f97332e05c65526022. Merged accidentally.

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

Upgrade Netty 4.0.37.Final -> 4.1.1.Final

See also https://lists.opendaylight.org/pipermail/odlparent-dev/2016-April/000269.html

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

Fix use of BAD BAD BAD BAD static class initializer code block

This clarifies a problem we run into when doing the PAX Exam upgrade.
Fixing this will give us a much clearer error message the next time
anything related to this goes wrong.

Change-Id: I296d7475bf6fbb6390f0c08dafabd42fa1a9891d
Signed-off-by: Michael Vorburger <vorburger@redhat.com>
Signed-off-by: Robert Varga <rovarga@cisco.com>

Upgrade Netty 4.0.36.Final -> 4.0.37.Final

Fixes CVE-2016-4970, among other thing, see [1] for more.

[1] http://netty.io/news/2016/06/07/4-0-37-Final.html

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

Move 3 Checkstyle from controller/checkstyle to odlparent

These used to be in
controller/opendaylight/commons/checkstyle/src/main/resources/controller/checkstyle.xml,
but should better be here in odlparent odl_checks.xml:

* http://checkstyle.sourceforge.net/config_misc.html#UpperEll
* http://checkstyle.sourceforge.net/config_coding.html#EmptyStatement
* http://checkstyle.sourceforge.net/config_coding.html#EqualsHashCode

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

Move Checkstyle module MethodName up in XML.

There must be order. Always.

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

Revert "Upgrade Pax Exam 4.4.0 -> 4.9.1"

This reverts commit 223a8cf3cb62c7873f3ba3de15728be06ae39010. It breaks
the features tests in yangtools.

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

Merge "Upgrade Pax Exam 4.4.0 -> 4.9.1"

Revert "Upgrade Jackson 2.3.2 -> 2.7.4"

This reverts commit 2061aa51982fa82d3414d8f03729819b785ff296, because it
broke sal-rest-docgen (as well as SFC according to skitt).

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

Upgrade Pax Exam 4.4.0 -> 4.9.1

This brings many bug-fixes, and a few improvements:
* support for OSGi 6
* addition of an embedded Karaf container

One important caveat is that the pax-exam POMs no longer pull in
org.osgi:org.osgi.core.

For details, see
https://ops4j1.jira.com/wiki/display/PAXEXAM4/Pax+Exam

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

Merge "Move Checkstyle rules not related to slfj4 logging from yangtools"

Merge ".gitignore .factorypath created by m2e-apt"

.gitignore .factorypath created by m2e-apt

https://github.com/vorburger/opendaylight-eclipse-setup now bundles
https://github.com/jbosstools/m2e-apt, which creates .factorypath files
in projects using "active" (APT) Java annotations.

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

Move Checkstyle rules not related to slfj4 logging from yangtools

All of these used to be in yangtools' checkstyle-logging.xml, but should
better be here in odlparent odl_checks.xml.  Further notes FTR:

* Indentation was slightly different in yangtools, safer to keep as-is
* RedundantImport is new (from neither), and seems sensible
* AvoidStarImport was just moved

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

Revert "Bump akka to 2.4.6"

This reverts commit be2a37ddcfbd86651c73c809a4b3096555df396d. There
is a regression in akka-2.4.5 and akka-2.4.6 which renders controller's
Remote RPC Broker unusable.

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

Merge "Bump akka to 2.4.6"

Upgrade moxy to version 2.6.2

Upgrade moxy from version 2.5.2 to 2.6.2.
Wait to merge this until all downstream consumers add a dependency
on javax.validation.

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

Merge "Upgrade Jackson 2.3.2 -> 2.7.4"

Merge "Allow license header to be glue to "package" statement"

Bump akka to 2.4.6

Includes a couple of fixes.

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

Merge "Bump Checkstyle Maven plugin version from 2.16 to 2.17"

Allow license header to be glue to "package" statement

This looks nicer, saves 1 line screen space - and seems to be what much
of the ODL code base already does, including e.g. the startup-archetype!

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

Upgrade Jackson 2.3.2 -> 2.7.4

Many improvements and bug-fixes; see
https://github.com/FasterXML/jackson/wiki/Jackson-Release-2.7.4 and
linked pages for details. Some highlights:
* map entries are simplified
  (https://github.com/FasterXML/jackson/wiki/Jackson-Release-2.5)
* JSR-303 bean validation annotations are supported
* "True"/"False" are accepted for boolean JSON representations
* Protobuf-encoded data is handled
  (https://github.com/FasterXML/jackson/wiki/Jackson-Release-2.6)
* helper objects can be injected from the OSGi registry

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

Merge "Upgrade org.ow2.asm 5.0.4 -> 5.1"

Upgrade org.ow2.asm 5.0.4 -> 5.1

Main changes:
- Remapping...Adapter classes are now deprecated, replaced with
  ...Remapper classes
- Add support for method reference on an interface

Bug fixes:
- 317567: SerialVersionUIDAdder issues
- 317578: incorrect max code size check in MethodWriter getSize()
- 317579: Signature Process in SignatureRemapper is Broken
- 317581: Import-Package headers should denote version by setting the
          version attribute
- 317586: VerifyError caused by LocalVariablesSorter
- 317602: Asmifier does not recognize null value for TypePath
- 317604: Bug in AdviceAdapter.visitFieldInsn()
- 317607: Calling add or set on an InsnListIterator throws NPE when at
          end of list
- 317606: Frequent use of StringBuffer instead of StringBuilder is
          hurting performance

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

Upgrade slf4j 1.7.12 -> 1.7.21, logback 1.1.3 -> 1.1.7

These upgrades need to happen in sync since logback 1.1.4 requires
SLF4J 1.7.16 or later.

For details of the changes, see
* http://logback.qos.ch/news.html
* http://www.slf4j.org/news.html

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

Bump Checkstyle Maven plugin version from 2.16 to 2.17

This change has two pre-requisites which need to merge first:
  1. separate Checkstyle Runtime Core VS. Maven plugin versions
  2. upgrade Checkstyle Runtime Core from 6.2 to 6.16

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

Merge "Bump Checkstyle version from 6.2 to 6.16"

Upgrade Karaf 3.0.3 -> 3.0.6

Note that OSGi 6.0.0 needs Karaf 4.0.1.

Release notes:
* 3.0.4: https://issues.apache.org/jira/secure/ReleaseNote.jspa?projectId=12311140&version=12329179
* 3.0.5: https://issues.apache.org/jira/secure/ReleaseNote.jspa?projectId=12311140&version=12332380
* 3.0.6: https://issues.apache.org/jira/secure/ReleaseNote.jspa?projectId=12311140&version=12333641

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

Merge "Refine Checkstyle rules: optional JavaDoc on c'str & getter/setter"

Merge "Refine Checkstyle rules: allowedAbbreviationLength 3 instead 1"

Merge "Upgrade resteasy jaxrs-api 3.0.4 -> 3.0.12"

Merge "Upgrade commons-net 3.4 -> 3.5"

Merge "Refine Checkstyle rules: single letter parameter OK"

Bump Checkstyle version from 6.2 to 6.16

We must stick to v6.16 instead of latest v6.17/18 in order to get
consistent results in-IDE (Eclipse) and on CLI; as the Eclipse
Checkstyle
plugin is still on 6.16 (they'll upgrade eventually); please see
https://sourceforge.net/p/eclipse-cs/feature-requests/158/ and
http://stackoverflow.com/questions/37126270/maven-checkstyle-and-eclipse-checkstyle-report-similar-but-different-results-usi

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

Separate the Checkstyle Runtime Core VS. Maven plugin versions

This change does not actually upgrade the version of either of them, the
maven-checkstyle-plugin v2.16 is the one currently already used, and the
6.2 runtime likewise (but until this change it was implicit & transitive
from the Maven plugin).

This is because then we can get more recent Checkstyle versions faster
and Checkstyle Maven plugin version upgrades are independant of the
Checkstyle runtime version and we don't (automatically) upgrade the
Checkstyle runtime version when upgrading the Checkstyle Maven plugin
version; see
https://maven.apache.org/plugins/maven-checkstyle-plugin/history.html
and
https://maven.apache.org/plugins/maven-checkstyle-plugin/examples/upgrading-checkstyle.html

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

Refine Checkstyle rules: allowedAbbreviationLength 3 instead 1

as in e.g. getYangAST() - that's just fine, and actually much better &
clearer than getYangAst() IMHO.

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

Refine Checkstyle rules: single letter parameter OK

as in e.g. logError(Exception e) - that's just fine

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

Refine Checkstyle rules: optional JavaDoc on c'str & getter/setter

See also
http://checkstyle.sourceforge.net/config_javadoc.html#JavadocMethod and
http://stackoverflow.com/questions/9955255/checkstyle-no-javadoc-for-getter-and-setter-only-works-for-getters
re. getter/setter

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

Clean up more properties

Now that centinel is updated, and since persistence won't release with
Boron (and is still on odlparent 1.6.0-SNAPSHOT), remove the remaining
obsolete dependency properties:
* guava.version
* org.json.version
* jettison.version
* jsr305.api.version
* junit.version
* slf4j.version

To handle the removal of ${guava.version}, hard-code the provided
feature version; this allows us to cleanly provide both versions 18
and 19, in preparation for migration to the latter. Add both versions
to the linked Javadocs.

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

Upgrade commons-net 3.4 -> 3.5

This is a bug-fix release:
http://www.apache.org/dist/commons/net/RELEASE-NOTES.txt

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

Merge "Improved Eclipse IDE experience: m2e's maven-dependency-plugin "

removed checkstyle m2e ignore lifecycle-mapping

This enables the Checkstyle checks to run in Eclipse, now that we have
the appropriate M2E connector plugin m2e-code-quality which configures
things as they should be, thanks to
https://github.com/vorburger/opendaylight-eclipse-setup

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

Remove obsolete properties

Now that the patches to didm and tsdr have been merged, along with the
Jersey clean-up patch to centinel, jersey.version and mockito.version
are no longer used anywhere; this patch removes them.

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

Merge "Checkstyle clean-up"

Upgrade Jetty 8.1.14.v20131031 -> 8.1.19.v20160209

We're sticking to the 8 series for now because iotdm uses
ContentExchange which has been removed in Jetty 9. This patch adds
jetty-client and jetty-server so that iotdm can pull them in from
odlparent (simplifying later upgrades), jetty-servlet-tester for aaa
and jetty-webapp for defense4all.

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

Upgrade resteasy jaxrs-api 3.0.4 -> 3.0.12

The release notes are available on the upstream tracker:
* 3.0.5: https://issues.jboss.org/secure/ReleaseNote.jspa?version=12322452&styleName=Html&projectId=12310560
* 3.0.6: https://issues.jboss.org/secure/ReleaseNote.jspa?version=12323117&styleName=Html&projectId=12310560
* 3.0.7: https://issues.jboss.org/secure/ReleaseNote.jspa?version=12323844&styleName=Html&projectId=12310560
* 3.0.8: https://issues.jboss.org/secure/ReleaseNote.jspa?version=12323980&styleName=Html&projectId=12310560
* 3.0.9: https://issues.jboss.org/secure/ReleaseNote.jspa?version=12324625&styleName=Html&projectId=12310560
* 3.0.10: https://issues.jboss.org/secure/ReleaseNote.jspa?version=12325657&styleName=Html&projectId=12310560
* 3.0.11: https://issues.jboss.org/secure/ReleaseNote.jspa?version=12325993&styleName=Html&projectId=12310560
* 3.0.12: https://issues.jboss.org/secure/ReleaseNote.jspa?version=12327705&styleName=Html&projectId=12310560

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

Checkstyle clean-up

Let's ensure odlparent adheres to its own Checkstyle rules...

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

Update property uses

The following properties are no longer used in ODL, remove them:
* jackson.version
* javassist.version
* logback.version

Update the usage information and removal patches for the remaining
dependency properties.

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

Merge "Upgrade lmax disruptor 3.3.2 -> 3.3.4"

Revert "Minor upgrade of Jersey from 1.17 to 1.19.1"

This reverts commit 9d92f22d7d3641ff20da942500c96adf5725753e.

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

Minor upgrade of Jersey from 1.17 to 1.19.1

Since a major upgrade of Jersey and JAX-RS might not have enough support
or contributors, we should look at least to upgrade to the latest 1.X
version.  This patch bumps Jersey from 1.17 to 1.19.1.

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

Merge "Bug 5787 Upgrade for Jersey from 2.8 to 2.22.2, the latest version"

Merge "Fix for improved Eclipse IDE setup: m2e's no longer explodes on import"

Bug 5787 Upgrade for Jersey from 2.8 to 2.22.2, the latest version

Upgrades jersey dependencies in the root pom to utilize the latest
version of jersey; from 2.8 to 2.22.2.

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

removing <version> as previously declared in <pluginManagement>

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

Fix for improved Eclipse IDE setup: m2e's no longer explodes on import

Eclipse setup experience improvement #2.  More to come - much more.

java.lang.IllegalArgumentException: Missing parameter for
pluginExecutionFilter. groupId, artifactId, versionRange and goals must
be specificed, but found: groupId = 'null'
artifactId = 'maven-dependency-plugin'
versionRange = '[0.0,)'
goals = '[resolve]'
  at
org.eclipse.m2e.core.internal.lifecyclemapping.model.PluginExecutionFilter.checkAllFieldsSet(PluginExecutionFilter.java:320)

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

Merge "Fix for improved Eclipse IDE setup: m2e Checkstyle ConversionException"