Merge "Marking of FWaaS classes and interfaces with as deprecated."

Merge topic 'jaxb_junit_testing'

* changes:
  Add JAXB JUnit testing for Neutron Floating IPs
  Add Junit testing of JAXB for Neutron FWaaS objects

Merge "Keep test numbering consistent"

Add JAXB JUnit testing for Neutron Floating IPs

Add JAXB Junit testing for Neutron Floating IPs, update FloatingIP
POJO, Northbound and Yang Model to add in missing attributes in each.

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

Keep test numbering consistent

Also fix spacing on the pom file.

Change-Id: I3fad4059121a9d50112afd99bbe9a56951e11a33
Signed-off-by: Sam Hague <shague@redhat.com>

Merge "Catch only the exceptions thrown by SubnetUtils"

Add Junit testing of JAXB for Neutron FWaaS objects

Correct some type mistakes in NeutronFirewallPolicy and
NeutronFirewallRule and add Junit testing of JAXB annotations
for NeutronFirewall objects

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

Catch only the exceptions thrown by SubnetUtils

Change-Id: Ie9b0abc63dce0c789198571afed9010043f839e4
Signed-off-by: Sam Hague <shague@redhat.com>

Merge "Fixed copyright messages Signed-off-by: Kiran Sreenivasa <kkoushik@brocade.com> Change-Id: Id1aa34710166f8c659f113fd4831141271363cac Signed-off-by: Kiran Sreenivasa <kkoushik@brocade.com>"

Merge "Cleaning up style checks in spi part 1"

Fixed copyright messages
Signed-off-by: Kiran Sreenivasa <kkoushik@brocade.com>
Change-Id: Id1aa34710166f8c659f113fd4831141271363cac
Signed-off-by: Kiran Sreenivasa <kkoushik@brocade.com>

Cleaning up style checks in spi part 1

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

Catch only the exceptions thrown by InetAddress.getByName

Change-Id: Idda91acd51a5c256a4094ca42b049aa171d2aafa
Signed-off-by: Sam Hague <shague@redhat.com>

Merge "neutron-spi JUnit Test Cases"

Merge "Initial code for Bug 2540"

neutron-spi JUnit Test Cases

Adding JUnit Test Cases for non-rest related methods.
(Includes V6 tests and NeutronSubnet code in support of same)

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

Fix Bug 2290

Ensure that isIPInUse, getLowAddr, allocateIP, releaseIP, gatewayIP_Pool_overlap and
initDefaults are all V6 friendly.

This is a port of gerrit: https://git.opendaylight.org/gerrit/#/c/12516/

Change-Id: I28c09691f02d339fc271a7eb62fa808e52e4b8b7
Signed-off-by: Flavio Fernandes <ffernand@redhat.com>
Also-By: Ryan Moats <rmoats@us.ibm.com>

Initial code for Bug 2540

Add allowed address pairs to Port class

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

Merge "Candidate Yang models for openstack neutron in OpenDaylight Controller"

Candidate Yang models for openstack neutron in OpenDaylight Controller

This patch is moving the neutron models pushed to controller repo through following gerrit
https://git.opendaylight.org/gerrit/#/c/13970/1

Scope of this patch is to define yang models in OpenDaylight controller for openstack neutron
APIs, so that we can enable the mechanism to store the neutron resource data in md-sal data
store for other applications to consume.
Patch 2: Address comments from reviewer
Patch 3: Removed whitespaces
Patch 4: Split models into 4 files. Fixed all reviewer comments(except one).
Patch 5: Fixed a build error(removed neutron.yang)
Patch 6: Fixed one remaining reviewer comment(move network-type to identity)
Patch 7: Fixed many comments(including tabs, moving provider to separate file, naming issues, re-org etc.
Patch 8: Added neutron-L3.yang module and fixed some more review comments
Patch 9: Fixed more comments from Ryan. Added augments in provider and L3 module.
Patch 10: Added neutron-metering.yang
Patch 11: Added neutron-secgroups.yang
Patch 12: Fixed uuid issue with Ed's patch + moved network defns into networks module
Patch 13: Fixed Ryan's latest comments(remove trailing space & Fix description)
Patch 14: Fixed Ryan's comments about Security group rules
Change-Id: I1fbc79e6e9856fa7cd18dd92d138e2b84a6d6a3c
Signed-off-by: Anil Vishnoi <vishnoianil@gmail.com>
Signed-off-by: Kiran Sreenivasa <kkoushik@brocade.com>

Merge "Add missing toString methods"

Merge "fixed potential NPE"

Merge "Dummy provider for neutron testing"

Dummy provider for neutron testing

classes implementing a dummy provider for testing neutron
northbound API

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

fixed potential NPE

removed duplicity of subnet existence check

Change-Id: I32cbc227bd7ce8c1122651815897c94f41ecee07
Signed-off-by: Martin Sunal <msunal@cisco.com>

Add missing toString methods

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

Add <name> field for Sonar sorting

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

Marking of FWaaS classes and interfaces with as deprecated.

Neutron FWaaS is still experimental in Kilo and so should be marked
as being deprecated.

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

Merge "Removed duplicity"

Fix for moxy load order and missing dependency.

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

Switch to using features-parent

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

Merge "Added a local karaf distribution for testing purposes"

Added a local karaf distribution for testing purposes

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

Removed duplicity

Change-Id: I9b4cf0232eed1ad3c038c2c8629da12cefbe0de1
Signed-off-by: Martin Sunal <msunal@cisco.com>

Add jacoco plugin to parent pom xml

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

Move to using parent pom structure

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

Fix scm tags to use new neutron project info

Change-Id: I93b8fccbb3c877580cd142e1fe8b774210a1efc8
Signed-off-by: Sam Hague <shague@redhat.com>

Point Jetty to the actual correct init class

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

Fixed package directory issues.

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

Add project's own artifacts as feature deps

This is needed so that dependent projects retrieve the appropriate jar
files.  It also allows building karaf features without having built the
rest of the project locally, with an empty ~/.m2/repositories/

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

Fix features so it actually runs SingleFeatureTest and passes

- Fixed features file so it actually runs.
- Fixed northbound-api to not include the neutron bundle which doesn't exit. Otherwise the test will fails with an unresolved constraint.

Change-Id: If651d14429b1e9fdf537142de79695df35b0a8ad
Signed-off-by: Sam Hague <shague@redhat.com>

Update features.xml with new neutron artifact names

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

Rebase of 15072 to NN repository for bug 2691

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

Fix for Bug 2120

When starting a new pool, allocate the next address as the start

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

Personal nit: -all is a terrible terrible meme.  This patch removes it.

As a matter of personal belief, I think that -all is pure evil.

*But*... you need -implementation and -northbound currently to
have a clean running instance of the neutron service.

Because I *also* believe you should be able to load a single feature
for a single thing, I have brought these two personal beliefs together
by renaming odl-neutron-all to odl-neutron-service.

Feel free to disagree with my taste ;)

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

Rooted out internal use of org.opendaylight.controller groupId

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

Rename neutron artifacts

Neutron artifacts renamed:
controller.networkconfig.neutron -> neutron.neutron-spi
controller.networkconfig.neutron.implementation -> neutron.transcriber
controller.networkconfig.neutron.northbound -> neutron.northbound-api

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

Merge "add .gitreview"

Merge "Add new groupId to pom.xml files"

add .gitreview

Change-Id: I4822e3e64c0d857867c4a8b066af7fd3fdb106f2
Signed-off-by: Flavio Fernandes <ffernand@redhat.com>

add .gitignore

Change-Id: I96be63c8c63ec6ce4543680c3e681abf5763330c
Signed-off-by: Flavio Fernandes <ffernand@redhat.com>

Add new groupId to pom.xml files

Before we can merge we need to set group Ids correctly

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

Update northbound annotations

Update annotiations of Northbound calls to cover ServiceUnavailable
errors (503 status codes)

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

Bring features/neutron into the same parent dir as everything else

This will ease breaking the code out to the neutron northbound project

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

Neutron to return ServiceUnavailable if no providers registered

opNFV interwork revealed that neutron returns 200 in error cases
where no SB providers are registered and if there is an error getting
the list of SB providers.  Close both of these holes by returning
Service Unavailable Status

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

BUG-2626: activate enunciate only on jdk-1.7

The plugin is not supported with JDK8. Create two parent pom.xmls, one
of which defines the activation on JDK7 only and the second one
centralizes the AD-SAL definitions for it.

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

BUG-2626: Do not override enunciate version

The overall version is 1.28, so do not downgrade it.

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

Bug 2684 - Fixing support for Neutron binding extended attributes

Change-Id: If2a309af55555253d508428d47b25c1b1d4ea796
Signed-off-by: Wojciech Dec <wdec@cisco.com>

neutron now works with jetty

This patch makes the feature neutron-northbound work with
jetty.

It also moves the adsal to port 8282, and the stock
karaf jetty to port 8080 (in additional to 8181).

GET, POST, and PUT for neutron network has been
tested:

GET:

curl -u admin:admin http://127.0.0.1:8080/controller/nb/v2/neutron/networks

POST:

curl -u admin:admin -H "Content-Type: application/json" -d @network.post http://127.0.0.1:8080/controller/nb/v2/neutron/networks

PUT:

curl -u admin:admin -H "Content-Type: application/json" -d @network2.post -X PUT  http://127.0.0.1:8080/controller/nb/v2/neutron/networks/d6220bbb-35f3-48ab-8eae-69c60aef3544

where:

cat network.post
{"network": {"status": "ACTIVE", "subnets": [], "name": "net-dedicated", "provider:physical_network": null, "admin_state_up": true, "tenant_id": "aa902936679e4ea29bfe1158e3450a13", "provider:network_type": "gre", "router:external": false, "shared": false, "id": "d6220bbb-35f3-48ab-8eae-69c60aef3544", "provider:segmentation_id": 1}}

cat network2.post
{"network": {"subnets": [], "name": "net-dedicated", "provider:physical_network": null, "admin_state_up": true, "provider:network_type": "gre", "router:external": true, "shared": false, "provider:segmentation_id": 1}}

This patch should not be merged before 2/1/2015 to allow time for discussion.

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

Make neutron a simple osgi app

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

Bug 2372: Removing duplicate call in loadBalancerPoolNorthbound to loadBalancerPoolInterface.addNeutronLoadBalancerPool(singleton);

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

Move adsal into its own subdirectory.

Moved various adsal components into an opendaylight/adsal subdir,
gave it a parent pom, and changed the root parent pom to have the
opendaylight/adsal as a module.

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

Fix for Bug 2290.

Wrap existing IPv4 code with version test.
Add new IPv6 code with version test.

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

Incrementing versions by 0.1.0 for post-Helium master branch

* The only version not incrmented is aaa.version, which is left at
  0.1.0-SNAPSHOT since they depend on controller and can't update yet.
* To break the cyclic dependency, this patch temporarily stops running
  the netconf-connector and restconf feature tests.
* Also fixing a bug causing Errors like "Choice has two nodes case with
  same qnames".

A second patch (to be run after AAA increments their versions to
0.2.0-SNAPSHOT) wiil update aaa.version and re-enable these tests.

Change-Id: I70a3987d32504e798c5ed05a67a0a1174338844b
Signed-off-by: Colin Dixon <colin@colindixon.com>
Signed-off-by: Robert Varga <rovarga@cisco.com>

Bug 2026: Fix for Neutron REST API NullPointer exception on POST Port when no fisxed-ips are present

Change-Id: I80e6435add3e0a07f28dbdf37911cd2bc8b263d0
Signed-off-by: Vina Ermagan <vermagan@cisco.com>

Bug 1852 - Setting the right type for the XMLelement members in NeutronLoadBalancerPool

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

Bug 1673 - Adding pool_id element to the NeutronLoadBalancerPoolMember and associated get/set functions. Note that the pool_id is not part of the request, but the URI.

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

Add getID() to NeutronRouter_Interface

Change-Id: Ie0afb1b2d09521913dc3877be6f92000db0d6cdd
Signed-off-by: Flavio Fernandes <ffernand@redhat.com>

Add toString() to Neutron_IPs

Change-Id: I6b9f8af8e1b65073a6a1fab3fe6eeb46c5c0cfcb
Signed-off-by: Flavio Fernandes <ffernand@redhat.com>

Fixing incorrect case for interface method neutronLoadBalancerPoolMemberDeleted

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

Add additional fields to Neutron Router

This adds additional fields to the Neutron Router object that can assist
with providing Distributed L3

Change-Id: I71dabe323a55a9028e54b1b46e923a7e4614f46d
Signed-off-by: Dave Tucker <djt@redhat.com>

Bug #1532 Neutron Port bindings missing Security Group List

Neutron Port calls are returning empty SecGroup lists. The
bindings are there, just not the return.

Change-Id: I882f0921af35e690672ee3a728cc2c029c4739fa
Signed-off-by: Brent Salisbury <brent.salisbury@gmail.com>

Neutron LBaaS v2.0 API support

Patch#2:
-Changed Pool Memebers to its own CRUD interface.

Patch#1
-Added LBaaS support per the v2.0 API Doc
http://docs.openstack.org/api/openstack-network/2.0/content/

-There are four root API members as defined by the API spec:
Load Balancer, LB Pools, LB Listeners and Health Monitors.

-Members are nested underneath LB Pools. The URL API path
is defined in the patch but needs to be wired into the binding.
That work wouldnt change the API just the implementation.

-The patch is getting huge so unloading for review. LBaaS
will have a lot of moving parts o0. The postman collection
for testing this is at:
[ovsdb.git] / resources / openstack
/ Neutron-v2.0-API-Examples.json.postman_collection

Change-Id: I6acb234e7f246a98c57a9cab97af405c028b708d
Signed-off-by: Brent Salisbury <brent.salisbury@gmail.com>

OS Neutron FWaaS api/jaxrs and associated bindings

Patch#2:
-Removed and cleaned up some comments.
-Fixed null check on Firewall Rules

Patch#1:
The Neutron v2.0 API document can be viewed at:
http://docs.openstack.org/api/openstack-network/2.0/content/

Sample API calls that can be imported into Postman
are located in the OVSDB directory at:
Directory: [ovsdb] / resources / openstack /
File: Neutron-v2.0-API-Examples.json.postman_collection

Change-Id: If37a0dd702aa02db9e94811c9a414718d25f42a7
Signed-off-by: Brent Salisbury <brent.salisbury@gmail.com>

Fixed for bug 1168 : Issue while update subnet

Change-Id: I3460f77f331c2fdf474bf8beb41bdf1332201a16
Signed-off-by: faizan <md.faizan@tcs.com>

Refactor Neutron Pagination Code to support Network, Port and Subnet

- Move pagination logic to PaginatedRequestFactory
- Add INeutronObject interface to bound Generics
- Add INeutronRequest interface to enable pagination to be generic

Patchset 4:
- Fix typos in @XMLElement annotation on the 'links' attributes

Change-Id: I7fb6c800268847a36eccd2bc402580cb48c60c46
Signed-off-by: Dave Tucker <djt@redhat.com>

Fixed for bug : 1171 - issue while creating subnet

Change-Id: Ibb102b6c1ec6d0ca70da3c279448998c02612a46
Signed-off-by: faizan <md.faizan@tcs.com>

Registered SecGroup/Rules in netconfig.Neutron Activator

This patch initiates the Sec CRUD interfaces.

Change-Id: Idf5ac503bbc04f1377bf89afa037e5e9d763d35d
Signed-off-by: Brent Salisbury <brent.salisbury@gmail.com>

Neutron API v2.0 bindings and APIs for security groups/rules.

-This is to enhance the ODL/OpenStack ML2 integration
work to include port security services.

https://wiki.openstack.org/wiki/Neutron/APIv2-specification

Change-Id: Ice5c2fd12381732ab1cbb2394fd67ffed1dcf62c
Signed-off-by: Brent Salisbury <brent.salisbury@gmail.com>

Unify maven-bundle-plugin version at 2.4.0

This removes superfluous version declarations, fixing about 70 eclipse
warnings. At the same time it bumps the version 2.4.0, to unify it -- we
were using both 2.3.7 and 2.4.0 at the same time.

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

Adding auto-sorting plugin to all Controller via parent/pom.xml

leveraging google's sortpom plugin, all controller poms are sorted
at build time as per the opendaylight/commons/parent/pom.xml configuration.

Sorted lists of properties, dependencies, and plugins will increase pom readability
and help prevent duplicate entry errors.

This first submission is huge as I changed one pom.xml and all are then sorted.
Only if the sortpom plugin is reconfigured will all pom's again be changed.

Note that if any pom.xml is pushed without having been through a build ... folks are
going to see pom.xml files showup as modified unintentionally.

Change-Id: I8553f38b9f0e5f6d27177ad6b9b9c35a4b43d55a
Signed-off-by: jameshall03885 <ghall@brocade.com>

Fix bug 666 - Neutron Subnet Creation Error

When a neutron subnet is created on the Northbound API, containing the
new "ipv6_address_mode" or "ipv6_ra_mode" fields with "null" values
Jersey will fail to deserialise the object correctly and throw an HTTP
500 error with a NullPointerException.

This commit adds the above fields to the NeutronSubnet object and
permits these fields to be nillable.

Change-Id: I225581b57a1698a5e5c17f91b117fba556b70976
Signed-off-by: Dave Tucker <dave.j.tucker@hp.com>

Fix for Bug #533

Ensure start of Allocation Pool is set properly in all cases

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

update the pom files to enable the nexus repository path to be configured by properties such that if can be overriden via the command line to support deployment to alternative locations

Change-Id: Ie96244a1a2a3f6b2f3760f366899017a44aa381d
Signed-off-by: David K. Bainbridge <dbainbri@ciena.com>

minor pom cleanup to eliminate duplicate group ID specification when the group ID is the same as the parent pom, just removes some warnings from eclipse, should not be any functional difference

Change-Id: I396126408fff54d7120d3ae94b2a5c447d14f8d2
Signed-off-by: David K. Bainbridge <dbainbri@ciena.com>

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>