Code cleanup: use collection interfaces in declarations

Replace HashMap with Map and HashSet with Set in
variable declarations

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

Merge "Clean up collapsable if statements"

Code cleanup: remove unused local variables

In addition, in NeutronPort class, remove potential NPE
by reordering comparison with static strings.

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

Clean up collapsable if statements

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

Clean up minor code issues

Remove unused imports and get public/final ordering
correct.

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

Merge "Clean code issues in NeutronSubnetIPAllocationPool"

Clean code issues in NeutronSubnetIPAllocationPool

Address the following issues:
1. Rename the class itself (contains underscore)
2. Rename methods that contain underscore
3. Ensure caught exceptions are logged

Also change dependent classes impacted by 1 and 2

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

Clean issues in NeutronUtil and PaginateRequestFactory

Fix the following code issues in these two classes:
1. Declare them final.
2. Correct method names
3. Don't assign values to parameters

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

Merge "build failure of neutron"

Add more neutron network IT cases

This patch is a cadidate for cherry picking to stable/lithium.
However, it changes the semantics of the canNetworkBeUpdated
part of the INetworkAware interface and so before we change
that, we need to communicate with other projects.

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

Merge "Remove commented code blocks"

Merge "Hide constructor of utility classes"

Merge "Add braces on else blocks"

Merge "Fix field naming in NeutronVPNServiceInterface"

Merge "Clean up unnecessary object allocation"

Merge "Clean code in NeutronPort class"

Merge "Fix mac_regex constant name and type"

Merge "Fix inefficient string buffering "

Remove commented code blocks

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

Hide constructor of utility classes

From squid:
"Utility classes, which are a collection of static members,
are not meant to be instantiated. Even abstract utility
classes, which can be extended, should not have public constructors.

Java adds an implicit public constructor to every class
which does not define at least one explicitly. Hence,
at least one non-public constructor should be defined."

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

Add braces on else blocks

Found two else blocks without braces...

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

Fix field naming in NeutronVPNServiceInterface

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

Clean up unnecessary object allocation

NeutronSecurityRuleInterface used new Integer to instantiate
objects where Integer.valueOf is more friendly

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

Clean code in NeutronPort class

Remove two instances of local variables hiding
class variables and fix a type definition as an
implementation rather than an interface.

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

Fix mac_regex constant name and type

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

Fix inefficient string buffering

Don't combine arguments in NeutronSubnet_IPAllocationPool

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

Remove unused field from NeutronTranscriberProvider

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

build failure of neutron

Building with -DskipITs or -DskipTests leads to a failure when
/neutron/integration/test/target/site does not exist.

Change toplevel antrun move directives to not fail on error
as this error is not critical

Change-Id: If64c53ada8414d82c9fd389d0d754bb219764e12
Signed-off-by: Isaku Yamahata <isaku.yamahata@intel.com>

Merge "Consolidate duplicated code from transcriber classes"

Merge "Remove @deprecation from FWaaS classes"

Remove @deprecation from FWaaS classes

FWaaS is not going away in Liberty, so related classes need to hang
around.

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

Consolidate duplicated code from transcriber classes

The AbstractNeutronInterface allows us to consolidate the overwrite
method that was part of multiple classes in transcriber

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

Change NeutronCRUDInterfaces class from static

This change is intended to make it easier to scale when we
start adding extensions by not requiring code to call static
methods in NeutronCRUDInterfaces

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

Create custom checkstyle rules for NN

There are some rules in ODL parent checkstyle that
are annoying - this patch allows us to set our own
for some of them

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

Adapting reporting approach to work with maven goal order

Because maven goals are set to "clean install", that means
the top level project has to move the IT reports from the
integration test project to itself.

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

Fix code duplication in report to sonar

The patch that got IT reporting working had the nasty
side effect of doubling the LoC reported by sonar
for the module. This patch rearranges things to
fix that and report IT results for each of the modules,
rather than from the IT module.

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

Merge "neutron-secgroups.yang: update model for security group"

neutron-secgroups.yang: update model for security group

- neutron security group accepts protocol as integer.
  It's defined as uint8.
- neutron security group defines port as uint16 since
  its port number of UDP/TCP.
  There is no point to define it as uint32.

Change-Id: I6fe01623ad07800a663fbd871dcfb030b9eef098
Signed-off-by: Isaku Yamahata <isaku.yamahata@intel.com>

Yet another attempt at IT reporting to Sonar

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

Remove model classes from sonar reporting

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

Try to fix IT reporting to sonar via parameter set

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

Try #2 to bring IT to sonar

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

Merge "Bringing IT back on line with reporting to sonar"

Merge "pyang: missing key in neutron port attribute"

Bringing IT back on line with reporting to sonar

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

Change ut and surefire configuration back to default

Also remove it via failsafe for now

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

pyang: missing key in neutron port attribute

This patch fixes the following error by pyang.
ip version is missing for extra dhcp attribute and add key statement.

> neutron.yang:42: error: the list at "neutron.yang:42 (at neutron-ports.yang:54)" needs at least one key because it is used as config

Change-Id: Ie2ae3661799fc2b7179f028c4a586a1593e23c69
Signed-off-by: Isaku Yamahata <isaku.yamahata@intel.com>

Another try at getting reporting to sonar

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

Use default outputDirectory for jacoco

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

Changes to see if we can get results reported to sonar

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

Merge "Augment Bug 3812 test with second subnet delete"

Merge "Fix NPEs in dependent port code called by network delete"

Merge "pyang: add missing key for list statement"

Merge "pyang: eliminate warning by pyang validation"

Fix NPEs in dependent port code called by network delete

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

Specifically log deletes via dummyprovider

Fixes the fact that the original dummyprovider code
didn't log deletes well.

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

Augment Bug 3812 test with second subnet delete

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

pyang: add missing key for list statement

This patch fixes following pyang errors

> neutron.yang:38: error: the list at "neutron.yang:38 (at neutron-L3.yang:155)" needs at least one key because it is used as config
> neutron.yang:40: error: the list at "neutron.yang:40 (at neutron-metering.yang:80)" needs at least one key because it is used as config
> neutron.yang:41: error: the list at "neutron.yang:41 (at neutron-metering.yang:90)" needs at least one key because it is used as config
> neutron.yang:42: error: the list at "neutron.yang:42 (at neutron-ports.yang:33)" needs at least one key because it is used as config

Change-Id: Iebfd6a0233a37eac539613c7f3e6cf765f9a0cee
Signed-off-by: Isaku Yamahata <isaku.yamahata@intel.com>

pyang: eliminate warning by pyang validation

Eliminate those warnings
> warning: imported module ietf-inet-types not used
> warning: imported module ietf-inet-types not used

Change-Id: Ic8fdf3a60b74a1738f8654d26eed96b23c89b441
Signed-off-by: Isaku Yamahata <isaku.yamahata@intel.com>

Fix selected neutron Tempest test

Added IT code to test and address issues found by
PortsIpV6TestJSON.test_create_port_in_allowed_allocation_pools

This also fixes bugs 2540, 2541, 2542, 2543, 2544, and 2550

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

Pay down technical debt: remove trailing comments

Squid includes a rule that Comments should not be located at the
end of lines of code to ensure readability. The main idea behind
this rule is that in order to be really readable, trailing comments
would have to be property written and formatted (correct alignment,
no interference with the visual structure of the code, not too long
to be visible) but most often, automatic code formatters would not
handle this correctly: the code would end up less readable.

Therefore, comments are far better placed on the previous empty
line of code, where they will always be visible and properly
formatted.

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

Pay down technical debt: close remaining minor items

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

Merge "pyang: fix import error"

Merge "pyang: warning unused imported module"

pyang: warning unused imported module

This patch eliminates the pyang warning like
warning: imported module neutron-xxx not used

Change-Id: Ia9b52addf8d477dcf1d3796f365957eb5164faf2
Signed-off-by: Isaku Yamahata <isaku.yamahata@intel.com>

pyang: fix import error

pyang complains
> error: module "neutron-networks" not found in search path

From rfc6020.txt
> 5.2.  File Layout
>
>    YANG modules and submodules are typically stored in files, one module
>    or submodule per file.  The name of the file SHOULD be of the form:
>
>      module-or-submodule-name ['@' revision-date] ( '.yang' / '.yin' )

Since it says "SHOULD", the file name convention is not mandatory.
But there isn't any good reason to have a file name
of neutron-network.yang (without lasting "s").

Change-Id: Ic6c98382fbbc5e073d5fc18e4a3dccdb4569face
Signed-off-by: Isaku Yamahata <isaku.yamahata@intel.com>

Correct handling of SGRs and SGs

Change-Id: I562d37a242d824809ed0cfdb16bbba2c1e91fb95
Signed-off-by: Ed Warnicke <hagbard@gmail.com>

Merge "Technical debt paydown: clean up repeated string constants"

Merge "Fix SecGrp and SecGrpRule."

Fix SecGrp and SecGrpRule.

I can now get SecGrp and SecGrpRules from:
https://review.openstack.org/#/c/178325/

Change-Id: Ieaf9aefda432908f8af38c180d6eb31a1eea15e0
Signed-off-by: Ed Warnicke <hagbard@gmail.com>
(cherry picked from commit 3e3b80adc65d712daa6811beaf72eb6dad893694)

Clean up IT and address Bug 3812

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

Technical debt paydown: clean up repeated string constants

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

Run jacoco in UT VM and report to sonar

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

Merge "Fix logger constant name and remove unused logger constants"

Bug 3533: Fix for CorsFilter error

Change-Id: If898855ab97dd0bcdb913e7cc01cfde04b31b522
Signed-off-by: Ed Warnicke <hagbard@gmail.com>
(cherry picked from commit 2394212e811e99bfd057cd197adc35a10fda2faa)

Try #2 to get jacoco to report integration results

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

Fix logger constant name and remove unused logger constants

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

Merge "Remove redundant modifiers from neutron-spi interfaces"

Merge "Remove redundant modifiers from northbound interfaces"

trying different jacoco configuration

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

Remove redundant modifiers from northbound interfaces

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

Remove redundant modifiers from neutron-spi interfaces

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

Remove magic numbers from northbound-api

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

Merge "Remove magic numbers from transcriber"

Merge "Remove magic numbers from dummyprovider"

Merge "Remove magic numbers from neutron-spi classes"

Merge "Fix if/else/for/while/do statements that lack braces"

Cherry pick of 22633 to master

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

Remove magic numbers from dummyprovider

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

Remove magic numbers from neutron-spi classes

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

Remove magic numbers from transcriber

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

Fix if/else/for/while/do statements that lack braces

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

Replace printStackTrace with logger

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

Tests and Code for Bug 3014

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

Merge "Add Metering Create Tests for Bug 2546"

Merge "Test code for Bug #2545"

Bug 3622: Allow empty (non-null) getGatewayPortId and tenantId values for router

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

Add Metering Create Tests for Bug 2546

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

Test code for Bug #2545

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

Merge "Integrate IT via jacoco"