Enable checkstyle by default

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

Validate license headers in code files

Uses checkstyle to scan the code for files that do not conform to the
license header format described on the wiki at:

https://wiki.opendaylight.org/view/ODL_Coding_Guidelines#C.2FC.2B.2B.2FJava

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

Fix indents to 4 spaces

While the Google Java Style guidelines recommends 2 spaces. Most
projects on ODL actually today follow the 4 space rule for indents. If
there is agreement I think we should make 4 spaces an exception for ODL
code style instead of following Google's guidelines here.

https://google-styleguide.googlecode.com/svn/trunk/javaguide.html#s4.2-block-indentation

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

Merge "Bug 2761: Disables external snapshot repositories for feature-test"

Bug 2761: Disables external snapshot repositories for feature-test

Pax URL and Karaf by default always search for new version of snapshots
in all snapshots repository, even if that snapshots does not belong to that
repository and maven is invoked even with -nsu (no snapshot update)
or offline mode.

This is also true for OpenDaylight snapshot artefacts - pax url tries
to resolve them from third-party repositories, even if they are not present
there - this increases time which takes for features to install.

For more complex projects this actually means several HTTP GETs for each
snapshot bundle referenced, even if the bundle is already present
in local maven repository.

In order to speed-up installation and remove unnecessary network traffic,
which fails for obvious reasons, external snapshot repositories are
removed.

BGPCEP feture test (all snapshots downloaded, running with mvn clean install -o)
takes 2:16 minutes with this patch. Originally it took 12 minutes and hundreds of
 HTTP GET calls to snapshot repositories were observed.

Change-Id: I939cbfcab86eddc1ba7663e78c37bb57f2d8f0f5
Signed-off-by: Tony Tkacik <ttkacik@cisco.com>

Ignore Eclipse warning on enforcer plugin

Eclipse shows the following warning 'maven-enforcer-plugin (goal
"enforce") is ignored by m2e' for all POM files.  This commit silences
that warning, since there's not much we can do about it (building in the
command line, or, at the latests, Jenkins will run the enforcer plugin).

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

Merge "Completely remove references to xtend"

Merge "BUG-2680: downgrade to netty 4.0.24"

BUG-2680: downgrade to netty 4.0.24

Netty 4.0.25 has introduced an OSGi regression, as reported in
https://github.com/netty/netty/issues/3302. Go back one version, waiting
for 4.0.26.

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

Completely remove references to xtend

The problem we have had before was that the dependencyManagement
declaration overrode the normal version when importing dependencies,
throwing feature tests off.

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

Added the xtend version declaration and set 2.7.3.

Because yangtools project uses the version 2.7.3 and other project does
not use xtend at all, this patch set 2.7.3.

Change-Id: Ied6cbbd49fc8e8db156af1f11958a5731a074ea7
Signed-off-by: Hideyuki Tai <Hideyuki.Tai@necam.com>

Merge "Use scope=provided for OSGi bundles"

Merge "Unify maven jar plugin versions"

Merge "Add dependency plugin version declaration"

Merge "Unify compiler version"

Merge "Unify maven release plugin version"

Merge "Remove xtend version declaration"

Merge "Use Maven Enforcer plugin to require Maven version"

Merge "BUG-2680: bump netty to 4.0.25"

Merge "BUG-1173: bump guava version to 18.0"

Remove xtend version declaration

This version is no longer used, as yangtools uses a different version
and other projects do not use xtend at all.

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

Unify maven release plugin version

Instead of defining two properties with differing versions, keep the
version in one and reference it in the other.

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

Unify compiler version

We have had two defines for this particular plugin. Unify them.

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

Add dependency plugin version declaration

Some downstream projects already define this. Provide a common
definition.

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

Unify maven jar plugin versions

Make sure we use precisely one version.

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

Use scope=provided for OSGi bundles

OSGi run-time services are always expected to be provided by the
container. Mark them as such by default.

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

Merge "Add pluginRepositories section to help resolving Maven plugins"

Merge "Added .project and .classpath in .gitignore. These are configuration files from eclipse and change from system to system."

Use Maven Enforcer plugin to require Maven version

Using the <prerequisites> configuration in pom files is deprecated and
it is recommended to use the maven-enforcer-plugin to enforce these
rules instead.

See:
http://jira.codehaus.org/browse/MNG-5297
http://jira.codehaus.org/browse/MNG-4840

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

Add pluginRepositories section to help resolving Maven plugins

<pluginRepositories> are used to determine where to download plugins
used by Maven in order to build. While <repositories> is used to
download artifacts depended by a project during a build. We need both
defined in order to successfully download all artifacts using the
OpenDaylight mirror and not relying on Maven Central directly.

- Add pluginRepositories section to help resolve Maven plugins
- Change tab setting to 2 for xml
- Update Copyright year

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

Merge "Set root pom.xml <name> for Sonar"

Merge "Add maven install and deploy plugin versions."

BUG-2680: bump netty to 4.0.25

Simple version bump.

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

Add maven install and deploy plugin versions.

Some artifacts may reference these plugins in order to skip the operations.

Change-Id: I5242dfc7c868f29ff2ca596bb105888436a7e47d
Signed-off-by: Nathan Harmon <nathan.harmon@hp.com>

BUG-1173: bump guava version to 18.0

This is a follow-up patch once all the downstreams have removed their
overrides. Bumps guava from 14.0.1 to 18.0.

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

Bump pax-exam to 4.4.0

Switch to latest release train of pax-exam.

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

Harmonize pax-exam versions

Upgrades the global definition to 3.5.0 and removes the override which
was being applied.

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

Added .project and .classpath in .gitignore. These are configuration
files from eclipse and change from system to system.

Change-Id: Ie5f0a5628255e5a1e260bd27e84ccb15191f20a6
Signed-off-by: Charalampos Kaidos <ckaidos@intracom-telecom.com>

Set root pom.xml <name> for Sonar

As mentioned on the mailing list Sonar uses the <name> field of the
pom.xml that is passed to the mvn command as the name of the project in
Sonar. In most cases this is the root pom.xml file in a project. This
patch sets the name to the project shortname.

https://lists.opendaylight.org/pipermail/discuss/2014-November/004024.html

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

BUG-2159: bump karaf version used in tests

This is a partial re-application of
Icf65d10734c760bf8ef7f3048e3dadf7c801cd5e and mirrors the change already
made to yangtools (I67d7cd3fb3f7e243fab3cf7f8c36ff0ec3d91040).

We bump the version of karaf used during testing only (not the plugin),
so feature tests are able to succeed with Java 8. This patch is needed
to enable safe migration of downstream projects from yangtools-hosted
feature builder to the one provided in odlparent.

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

Revert "BUG-2261: bump karaf to 3.0.2"

This reverts commit 21d17e3e235979642ee73b8d410961362afc4705.

It turns out Karaf has upgraded schema incompatibly, so we need to schedule and closely execute the upgrade.

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

Merge "Change features-parent from using src/main/feature to src/main/features"

Merge "Add ODL Checkstyle"

Change features-parent from using src/main/feature to src/main/features

Also add some lifecycle plugin stuff so Eclipse is happy :)

Change-Id: Ie5cc950e1a605890c5707506e92d0421be13c235
Signed-off-by: Ed Warnicke <eaw@cisco.com>

Add ODL Checkstyle

Adds a checkstyle resource for ODL projects to use that allows them to
take advantage of ODL code style best practices in Java. Details on ODL
Code Style Guidelines can be found on the wiki at
https://wiki.opendaylight.org/view/ODL_Coding_Guidelines

This odl_checks.xml is based on the google checkstyle found at
https://google-styleguide.googlecode.com/svn-history/r130/trunk/javaguide.html

maven-checkstyle-plugin 2.13 does not yet support some of the Google Style
checks which ODL style is heavily based on. Until this is resolved we need
to comment out unsupported checkstyle sections.
https://jira.codehaus.org/browse/MCHECKSTYLE-261

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

BUG-2261: bump karaf to 3.0.2

In order to support Java 8, we need to upgrade the Karaf version, which
fixes https://issues.apache.org/jira/browse/KARAF-2602.

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

Merge "Added version for maven-deploy-plugin in pom.xml."

Merge "Bug 1918: Set org.json.version to 20131018."

Merge "Fixed url elements in pom.xml."

Removed <sonar.profile> odlparent/pom.xml.

Daily builds on the master branch have been failing since it is setup
due to the misconfiguration on <sonar.profile> of odlparent/pom.xml.
https://jenkins.opendaylight.org/releng/view/odlparent/job/odlparent-daily-master/

Considering "Sonar way with Findbugs" profile and "Sonar way ODL"
profile, which is the default profile in sonar.opendaylight.org, are
the exactly same, <sonar.profile> is not needed to be set.
https://sonar.opendaylight.org/profiles/compare?id1=1&id2=4

Change-Id: Ida4ab5c7b97c2bab7c51e501241d23bc6ce9f5b8
Signed-off-by: Hideyuki Tai <Hideyuki.Tai@necam.com>

Added version for maven-deploy-plugin in pom.xml.

Change-Id: Id5fa84a4973049b182fb07f43f22c17f16b1bfa6
Signed-off-by: Hideyuki Tai <Hideyuki.Tai@necam.com>

Fixed url elements in pom.xml.

Change-Id: I83c481f65e04d97a3997c9faa0ee37ec10383d08
Signed-off-by: Hideyuki Tai <Hideyuki.Tai@necam.com>

Merge "Introduce features-parent"

Bug 1918: Set org.json.version to 20131018.

Previously used value was 20080701,
but that somehow led to (one symptom of) Bug 1918.
20131018 version is more recent,
and already present in /system repository.
The correct fix would also make other projects
refer to version defined here, to avoid skew.

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

Merge "Fix feature-test failing with surefire 2.18"

Fix feature-test failing with surefire 2.18

feature-test fails maven-surefire-plugin 2.18 (but works with 2.17) due
to it assumming the constructor is called only once and modifying global
state. That global state is guarded from multiple modifications with an
Error. Move the offending initialization into static initializer, thus
ensuring we invoke the function only once.

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

Introduce features-parent

This is a migrated utility parent for features artifact. Downstream
projects should refer to this artifact as their parent, fill in the few
gaps and should be locked'n'ready for generating their feature and run
the test.

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

Remove duplicate declaration of maven-bundle-plugin.

Plugin declaration in pluginManagement must be unique.

Change-Id: Icbc78cf99ff6fa545974562ef334166c2770cf68
Signed-off-by: Shigeru Yasuda <s-yasuda@da.jp.nec.com>

Merge "Fix missing <nexusproxy> variable in root pom.xml"

Merge "Fixed maven-paxexam-plugin version."

Fixed maven-paxexam-plugin version.

Latest version of maven-paxexam-plugin is 1.2.4 and not 3.0.0,
this incorrect declaration broke build in controller project.

Signed-off-by: Tony Tkacik <ttkacik@cisco.com>
Change-Id: Id430fab8dde594e418cbebe496edd1f15f4258dc

Revert xtend version bump

This reverts the version bump of xtend-maven-plugin and adds the
appropriate explanation.

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

Fix missing <nexusproxy> variable in root pom.xml

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

Update Maven plugin versions

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

Merge changes I85ada413,Idd2417d0,I45a88760,I4cef2990,Iae30f52a

* changes:
  Add depends-maven-plugin declaration
  Add default Sonar configuration
  Add maven-resources-plugin declaration
  Add maven shade plugin declaration
  Add pluginManagement section for declared plugin versions

Merge changes I7e6275ea,Ia4ee994b

* changes:
  Introduce features-test
  Introduce bundle-parent

Introduce features-test

This patch introduces the features-test from yangtools. It validates
that a project's feature is self-contained.

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

Add depends-maven-plugin declaration

Adds the plugin, along with default configuration to generate a depends
file.

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

Add default Sonar configuration

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

Add maven-resources-plugin declaration

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

Add maven shade plugin declaration

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

Add pluginManagement section for declared plugin versions

Aside the version declarations, this adds pluginManagement, so we get
warnings when versions get overridden. It also allows downstream users
not to hunt for the proper property name.

This should solve the problem of having duplicate property names, which
we are already getting.

Also flags FIXMEs for convergence of the versions which are clashing.
Hopefully these will be taken care of as the warnings are emitted, as
now users do not have to specify versions at all.

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

Introduce bundle-parent

This introduces a parent for bundle artifacts. It just makes sure
maven-bundle plugin is executed with a default configuration. It can be
reused by any artifact without having to declare the plugin in its build
section.

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

Add a template settings.xml file

This file is needed to teach maven about OpenDaylight repositories
globally. When copied to ~/.m2/settings.xml, individual pom.xmls do not
need to carry repository definitions.

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

Introduce top-level aggregator

This patch introduces a top-level aggregator and moves odlparent itself
to a descendant, thus allowing us to add more infrastructure components.

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

Merge "BUG-2159: bump jacoco version"

Merge "BUG-1386: Bump javassist to 3.18.2"

BUG-1956: bump TrieMap library

This is a follow-up patch to preserve consistency.

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

BUG-2159: bump jacoco version

0.6.2 does not work with Java 8, as it mucks around in its internals.
Version 0.7.2 seems to work okay.

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

BUG-1386: Bump javassist to 3.18.2

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

Merge "Adding version/dependency info for JSONAssert"

bumped the version for lithium
Change-Id: Ifd5dcec91669c89007bdcf3df706d1ec4f6cd2a5
Signed-off-by: Surekha Bejgam <sbejgam@cisco.com>

Adding version/dependency info for JSONAssert

This adds version information for using the JSONassert library which can be
found here: https://github.com/skyscreamer/JSONassert/

The library uses the Apache License Version 2.0 as found here:
https://github.com/skyscreamer/JSONassert/blob/master/LICENSE.txt

Change-Id: I0bc2afc5e3b1718a0f202a1c516786902fd4d372
Signed-off-by: Colin Dixon <colin@colindixon.com>

Merge "Use variable to reference nexus proxy"

Merge "BUG-1675: bump netty version to 4.0.23"

BUG-1675: bump netty version to 4.0.23

Speed and correctness fixes have gone into the releases which we should
include in Helium.

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

Use variable to reference nexus proxy

*Edited hardcoded URLs to use nexusproxy variable

Change-Id: Iaf64b855759f893fc3935b462d3aa0f77dcbbafd
Signed-off-by: Devin Avery <devin.avery@brocade.com>

Patch upgrade for Apache Tomcat from 7.0.32 to 7.0.53 to take care of ODL Karaf Northbound issues.

7.0.32 had no support for Cors and hence we added a hack in the past to pull the CorsFilter from Apache Tomcat
project and placed it as a third-party bundle in the controller project.

But the Cors support was added to Tomcat patch version 7.0.42 and hence it is time to upgrade it to the proper
and remove the hack that we added earlier.

Also, the hacked bundle is not loaded in a karaf container (never attaches itself with the Fragment Host).
Debugged the heck of that, but dont know why the hacked bundle wouldnt register.
But since upgrading is cleaner anyways, we are taking that approach.

Tested the Northbound APIs and it is all working fine.

Change-Id: I37bf577320c5257b72e9fbe85313817230507a0c
Signed-off-by: Madhu Venugopal <mavenugo@gmail.com>

Make sure we mention surefire plugin

This forces the use of 2.16 (or whatever) version of surefire plugin by
default by downstream projects, so that they do not have to repeat it.

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

Merge "Declare jacoco mavne plugin"

Adding a version for jaxrs-api in org.jboss.resteasy

Change-Id: Id265bb89e07518296df31d84d00b62448c9722f0
Signed-off-by: Colin Dixon <colin@colindixon.com>

Declare jacoco mavne plugin

This adds jacoco into dependencyManagement section, so projects pick the
global version as default and do not have to rely on the property.

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

Merge "Add explicit org.hamcrest dependency"

Merge "Add gitignore"

Merge "Add support for gitreview"

Add explicit org.hamcrest dependency

org.mockito is pulling in hamcrest-core 1.1 while junit requires 1.3.
This results in failures when the two meet, for example in yangtools.

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

Cleanup of parent pom

- Cleanup of many duplicate version
- use sort-pom to tidyfy the file and then avoid duplicates in an easier way

Change-Id: I941da53bf61c48d52a3669cc078d5d5c86e98d35
Signed-off-by: Giovanni Meo <gmeo@cisco.com>

BUG-1275: add TXW2 to dependency management

This small package contains IndentingXMLStreamWriter, which is useful
for pretty-printing XMLs which are output via XMLStreamWriter.

Companion change, Ic83e55eb6d0eeea599e909cc0dae3b145ef9cc9f, introduces
the wrapped user. The dependency may be removed at a later point if
someone creates an ODL class with equivalent functionality.

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

Add gitignore

- Add ignores for common filetypes
- OSX, Linux
- Maven and Java
- Eclipse and IntelliJ

Change-Id: Ib967d577cb636532b783dfda1e12780f99b083de
Signed-off-by: Dave Tucker <dave@dtucker.co.uk>

Add support for gitreview

Change-Id: I9357406d09afbb45322c25a7fe62613a41638dbf
Signed-off-by: Dave Tucker <dave@dtucker.co.uk>