Fix for BUG 402: Subnet creation with NeutronAPIService sometimes fails
with a 409 error

Correct typo and bring hostRoutes out as own init block

Change-Id: I7f6f83cb6a63aa81a057b40c377ac49132a9fcb9
Signed-off-by: Ryan Moats <rmoats@us.ibm.com>

[maven-release-plugin] prepare for next development iteration

[maven-release-plugin] prepare release master-tagforprepareonly-controller-bulk-release-2-1

Migrate bundles' configuration mgmt to ConfigurationService

- Change current logic where read/save config is completely managed by each bundle,
  with moving it to configuration manager. This decoupling will allow configuration
  manager to decide how to persist the bundles' configurations to disk, withouth
  necessarely depend on java serialization as today controlelr does.
  Also now file names can be static and each bundle does not need to track the container
  context in the file name (which was lame). This allow controller to erase container
  startup files on container removal, which was not possible today.
- Global and in container configuration services to expose
  two APIs for the other bundles to retrieve and save their config
- Per container startup files organized in directories (directory name = container name)
- Container startup directory removed on container removal, and created on container creation if not present

Change-Id: I58c283eb796cf2a60466a97162428a8c2ae478ae
Signed-off-by: Alessandro Boch <aboch@cisco.com>

The config now is loaded only from the coordinator. The previous way of loading
the config only if empty had some side-effects.

Change-Id: I4f1819c60b491a36e1c14a78ec376e2dfcee8583
Signed-off-by: Asad Ahmed <asaahmed@cisco.com>

Migrate all artifact versions to dependencyManagement

- Migrate all the artifacts to use dependencyManagement to centralize
the version changes in controller and make sure the dry-run for
release could actually rewrite those.

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

Fixing a NullPointerException in Neutron Interface (Bug-323)

Ideally, this condition should never occur. But there seems to be a consistency issue in the ML2 driver
which could potentially result in this condition. While we are addressing the root-cause in ODL ML2
driver, we should also patch our side of the interface as well to avoid failing for such conditions when
it is harmless for a few usecases.

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

Config save support for networkconfiguration.neutron

With the ODL ML2 Integration, the REST APIs are uni-directional, hence a simple controller restart
results in synchronization issues between Openstack/CloudStack ML2 and ODL.

The solution that was agreed with Kyle, Ryan, Hugo & others is to add the config save support on the
ODL side which will effectively remove the inconsistency issues between Openstack / Cloudstack and ODL.

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

Update to new version of ADSAL components and its dependencies proposed by jenkins-controller-version-changes-8

By splitting the effort into 2 phases, it is more managable and easy to test.
After this commit, the ADSAL components and all the NSFs are uptodate with the dependency convergence.

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

Added DELETE on cors.allowed.methods for a few missing nb bundles.
Also added toString() method to a few Neutron backend classes.

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

Convert neutron implementation classes to unix line delimiters

Replace original windows line delimiters.

Change-Id: Icbae06e51f774e788e82683f65fd1d9641102be0
Signed-off-by: Ryan Moats <rmoats@us.ibm.com>

Convert neutron service base classes to unix line delimiters.

Replace original windows line delimiters in base classes.

Change-Id: I5d50ddf17604a640028bb40a0e8e8129bed67111
Signed-off-by: Ryan Moats <rmoats@us.ibm.com>

Fix for incorrect request assumption in Update Neutron Port op

Neutron Port update code incorrectly assumed that request body
would always include fixed_ip information.  Bracket fixed_ip
adjustment code with proper test.

Change-Id: Icee6b594869e5cba74c0ef339fc0fe7a4d7c671a
Signed-off-by: Ryan Moats <rmoats@us.ibm.com>

Refactor to remove empty catch exception.

Initializing a subnet object included an empty catch exception.
Refactor to return a boolean from this method so that exception
is reflected to calling methods, which can then take appropriate
action.

Change-Id: Ia6f3188064677de9035508994d864373915db0f1
Signed-off-by: Ryan Moats <rmoats@us.ibm.com>

Add checkstyle rule to prevent empty statements (;)

Change-Id: Id8b34fde64988326cf6cab494019f99d2d61cbc5
Signed-off-by: Alissa Bonas <abonas@redhat.com>

Refactor if/else with dead branch to explain corner case better

Change allocating address from subnet pool path to split the pool
in the main branch rather than the else and explain what happens
if the pool as a single address with a comment.

Change-Id: I3e910aecc2b251da419fc5d8055c2a9ccd975563
Signed-off-by: Ryan Moats <rmoats@us.ibm.com>

Clean all unused and redundant imports in controller.
Enable verification of unused and redundant imports via checkstyle.
Rename checkstyle configuration file because it no longer checks only
for tabs.

Change-Id: Id94a92fbc8a79aba01a4177aec3d80a2b3394508
Signed-off-by: Alissa Bonas <abonas@redhat.com>

Reworked pom files

Each pom (artifact) declares it's dependencies (internal and external)
No dependencies declared at parent POM level, only dependency Management.

Also moved the <modules> from distribution/opendaylight to the root pom.xml
distribution/opendaylight now builds entirely from dependencies

I have intentionally left some things in here I think are wrong,
because the goal of this commit is to reproduce the existing build
(which it does, I've checked).  Subsequent commits will clean up other things.

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

Add missing test methods to IRouterAware and clean up unused imports.

Realized that initial Neutron IRouterAware was missing test methods
for interface attach and detach operation.  In addition, removed
unused imports from other classes.

Change-Id: I8547ad7e7437d7215151f843744b56b2e0d93197
Signed-off-by: Ryan Moats <rmoats@us.ibm.com>

Add curly braces to for and if statements.

Change-Id: I1a681f0de1fe3f2b442d71612b4f15171df48ea2
Signed-off-by: Alissa Bonas <abonas@redhat.com>

[maven-release-plugin] prepare for next development iteration

[maven-release-plugin] prepare release releasepom-0.1.0

Add SCM section to all the poms

- For the release process we need SCM section to all the poms, some
are still missing it so adding them.

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

Initial push of Neutron interface

Two bundles included in this:
-networkconfig.neutron (contains JAXB annotated classes, *CRUD
interfaces with an implementation [for storing objects], and
Aware interfaces [for other bundles to register services])
-networkconfig.neutron.northbound (contains the JAXRS annotated
classes along with JAXB annotated request specific classes)

Commit amended to refactor .internal package to separate
implementation bundle and package

Commit re-amended to clean up issues in commons/opendaylight
and distribution/opendaylight poms

Commit re-re-amended to fix conflicts on rebase (I hope)

Getting close to just abandoning the change and starting over

Commit re-re-re-amended to fix parent issues in pom

amend to remove leaked CRs in distribution pom

- Cleanup of some duplicates artifact and of commons-net in modules
dependencies, anyway they get in via the commons.opendaylight

Change-Id: I15724c55e843237c010dfaec1e1e523560174f51
Signed-off-by: Ryan Moats <rmoats@us.ibm.com>

Initial empty repository