summaryrefslogtreecommitdiffstats
path: root/adapters/mso-openstack-adapters/src/main/java
AgeCommit message (Collapse)AuthorFilesLines
2019-09-30Merge "Added null check to server.getImage() to prevent crash at getId() call."Lukasz Muszkieta1-5/+7
2019-09-26Added null check to server.getImage() to preventOleksandr Moliavko1-5/+7
crash at getId() call. Issue-ID: SO-1841 Signed-off-by: Oleksandr Moliavko <o.moliavko@samsung.com> Change-Id: I6ecaf66649e9df971003268b7272f677c5394083
2019-09-10Integrate Logging LibraryPlummer, Brittany6-55/+95
Updated SO to use ONAP provided filters and constants to meet logging standards. Removed redundant classes Updated constant references Issue-ID: SO-2301 Signed-off-by: Benjamin, Max (mb388a) <mb388a@att.com> Change-Id: Ia4e3c2b1a4dcb3881aa34d39885c8b2782880d64
2019-09-10Merge "Created external task utils in a common location"Steve Smokowski5-105/+24
2019-09-10General sonar fixesArindam Mondal2-18/+17
+)Format logger ++) Remove toString() +++) Extract nested IF Issue-ID: SO-2226 Change-Id: Iff28408584b04ffd03b6aca0f647d0af9090a9c2 Signed-off-by: arind.mondal <arind.mondal@samsung.com>
2019-09-08Created external task utils in a common locationBoslet, Cory5-105/+24
Created external task utils in a common location then refactored external task to use these common utils Updated junit errors and failures in openstack Removed env param and updated unit test with class Remove env file from parm in unit test class Match argu to match class abstract to con. removed implementation within anonymous class Issue-ID: SO-2291 Signed-off-by: Benjamin, Max (mb388a) <mb388a@att.com> Change-Id: I47c2850d750cbfe7ef67e2396fcb46db43e3cb97
2019-08-29sonar fix-remove unused importThugutla sailakshmi1-1/+0
sonar fix-remove unused import Issue-ID: SO-2174 Change-Id: I5c13bbf27aa0f03b448fca5c2cea7329512aff01 Signed-off-by: Thugutla sailakshmi <tsaila10@in.ibm.com>
2019-08-22Merge "Moved setNeutronNetworkId() call inside null check to prevent crash ↵Lukasz Muszkieta1-1/+1
at outputs.get() call"
2019-08-22Merge "Moved null check for outputs so that get() method call won't result ↵Lukasz Muszkieta1-4/+4
in crash."
2019-08-21Added null check for rootNode to prevent crash atOleksandr Moliavko1-13/+18
for loop. Issue-ID: SO-1841 Signed-off-by: Oleksandr Moliavko <o.moliavko@samsung.com> Change-Id: I5ec16f3d73982c89e2bf5b14758274f1d4dd7740
2019-08-19Added null check for heat to remove staticOleksandr Moliavko1-1/+5
analyzer warning Issue-ID: SO-1841 Signed-off-by: Oleksandr Moliavko <o.moliavko@samsung.com> Change-Id: I71eb14adcc1227af4470853e6e1acdb631a7efe8
2019-08-16Removed unused CSAR class.Oleksandr Moliavko1-189/+0
Issue-ID: SO-1841 Signed-off-by: Oleksandr Moliavko <o.moliavko@samsung.com> Change-Id: I0efbb38cb34b0b36d2659ebf36f6cd2d109e5deb
2019-08-16Replaced string concatenations in a loop withOleksandr Moliavko1-4/+9
StringBuffer appends to remove static analyzer warning about inefficient string operations Issue-ID: SO-1841 Signed-off-by: Oleksandr Moliavko <o.moliavko@samsung.com> Change-Id: I89e0e5b599a8263882b0f91fb0f9191734a04ec1
2019-08-13Merge "Removed duplicate use of networkType field in String.format() call to ↵Lukasz Muszkieta1-2/+2
prevent static analyzer warning"
2019-08-13Removed duplicate use of networkType field inOleksandr Moliavko1-2/+2
String.format() call to prevent static analyzer warning Issue-ID: SO-1841 Signed-off-by: Oleksandr Moliavko <o.moliavko@samsung.com> Change-Id: I1e3ea352aac9a88172b683d267b2f3c804bb362b
2019-08-09Moved setNeutronNetworkId() call inside null checkOleksandr Moliavko1-1/+1
to prevent crash at outputs.get() call Issue-ID: SO-1841 Signed-off-by: Oleksandr Moliavko <o.moliavko@samsung.com> Change-Id: I51ad7a6de93c88ce6743234b943f9abcbc065a70
2019-08-09Moved null check for outputs so that get() methodOleksandr Moliavko1-4/+4
call won't result in crash. Issue-ID: SO-1841 Signed-off-by: Oleksandr Moliavko <o.moliavko@samsung.com> Change-Id: I23deef70ec7efec22cd31489666341a3b53c9899
2019-08-08fix sonar issuesarada prasad sahoo1-15/+16
fix sonar issues Issue-ID: SO-1811 Signed-off-by: sarada prasad sahoo <sarada.prasad.sahoo@huawei.com> Change-Id: Iba1f585de7465da5649b60e6990a737936656ac7
2019-08-08Merge "Replace duplicated String literals"Seshu Kumar M3-20/+22
2019-08-08Shift string literal to left side of equals(avoid null pointer ex)sindhu36722-2/+2
Avoid null pointer exception mso-openstack-adapters classes Issue-ID: SO-2187 Signed-off-by: sindhu3672 <arcot.sindhuri@huawei.com> Change-Id: Ie7099318a79162aed26ec3a8e4a013cece367736
2019-08-08Replace duplicated String literalssindhu36723-20/+22
Replace duplicated String literals mso-openstack-adapters files Issue-ID: SO-2187 Signed-off-by: sindhu3672 <arcot.sindhuri@huawei.com> Change-Id: I3089f407ffd01afbccaada81c2aef290299dfe32
2019-08-02General sonar fixesArindam Mondal2-2/+3
+) Add logger to log exception ++) Format logger Issue-ID: SO-2187 Change-Id: Id77271340070a61e7ce82822321d7e5b8b03f759 Signed-off-by: arind.mondal <arind.mondal@samsung.com>
2019-07-30Merge "Added null check to heatStack.getOutputs() to prevent crash at get() ↵Seshu Kumar M1-10/+13
call; removed redundant null check to simplify the code"
2019-07-30Added null check to heatStack.getOutputs()Oleksandr Moliavko1-10/+13
to prevent crash at get() call; removed redundant null check to simplify the code Issue-ID: SO-1841 Signed-off-by: Oleksandr Moliavko <o.moliavko@samsung.com> Change-Id: If9d8dcba3008cd7ebe60c17c08c71a07a6df9df4
2019-07-30Merge "Replaced constructor call with valueOf() call to remove static ↵Seshu Kumar M1-1/+1
analyzer warning about inefficient use of constructor"
2019-07-30Merge "Sonar fix: HealthStackAudit.java"Seshu Kumar M1-1/+2
2019-07-30Replaced constructor call with valueOf() call toOleksandr Moliavko1-1/+1
remove static analyzer warning about inefficient use of constructor Issue-ID: SO-1841 Signed-off-by: Oleksandr Moliavko <o.moliavko@samsung.com> Change-Id: I5b6285aa8708b57bb54a90d38130a21b07c2b2c5
2019-07-25Add SpringClientFilter in ValetClient to autologMcblain, Thomas1-6/+6
Add SpringClientFilter in ValetClient to autolog msgs Modify to use our RestTemplateConfig for RestTemplate Use Provider for thread-safe RestTemplate autowire Issue-ID: SO-2160 Signed-off-by: Benjamin, Max (mb388a) <mb388a@att.com> Change-Id: I02d1995ebd6d2c93477008feb13193d3c0b3cd05
2019-07-23Major Sonar fixesr.bogacki1-31/+16
Major Sonar fixes in different classes: -Removed unnecessary code. -Removed unused variables. -Removed unnecessary imports. -Simplified diamond operators. -Improved String comparisons in equals. Issue-ID: SO-2102 Signed-off-by: Robert Bogacki <r.bogacki@samsung.com> Change-Id: I279c63f8e7161d6e33d5d6a28b938c0c64e90f1f
2019-07-22Merge "Added servicename to MDC so that it gets logged"Steve Smokowski5-4/+20
2019-07-19Added servicename to MDC so that it gets loggedBoslet, Cory5-4/+20
Added servicename to MDC so that it gets logged and added enter and exit markers. Issue-ID: SO-2122 Signed-off-by: Benjamin, Max (mb388a) <mb388a@att.com> Change-Id: I63ff5e5de21d851c989ffa5db2051755751d9ead
2019-07-19Sonar fix: HealthStackAudit.javaArundathi Patil1-1/+2
Fixed sonar issues across this file Issue-ID: SO-2137 Change-Id: I7650e1746bc3da5149310e6eb4b53f2ee98e3d9f Signed-off-by: Arundathi Patil <arundpil@in.ibm.com>
2019-07-18Merge changes I5e72e1c6,I5e1cb624Steve Smokowski1-8/+16
* changes: Added null check for function param pFqdns to prevent crash at for loop. Added check for null ptr before dereferencing at gBBInput.getCustomer().getServiceSubscription() - to prevent static analyzer warning
2019-07-17add query stack data and populate table stepBoslet, Cory7-15/+271
Need to check if audit is enabled first in the delete record task Updates to audit delete redeisgn and data to table Various updates to logging, audit object, added tag to query, updated tag value. Updated write method to make sure an entry for this heat stack doesnt already exist to avoid duplicate entrys Updated unit test to account for get query Added so that id is not hidden from query results Updated the was delete is done plus minor updates. Updated delete method to use resttemplate instead of bowman client Combined the write and get stack data task into the external tasks. Removed the task to delete the stack data from db Updates to junit request id so its not a random id for test Updated service tasks in att BB to call external task Update to be a real uuid format due to failing unit test Updated fields to protected and updated unit test Issue-ID: SO-2111 Signed-off-by: Benjamin, Max (mb388a) <mb388a@us.att.com> Change-Id: I8f479a046cf837403f60c330ed6ad8bc08a9c82f
2019-07-12Added null check for function param pFqdns toOleksandr Moliavko1-8/+16
prevent crash at for loop. Issue-ID: SO-1841 Signed-off-by: Oleksandr Moliavko <o.moliavko@samsung.com> Change-Id: I5e72e1c6d74c5f197479062348090ab2b87fafae
2019-07-09Sonar fix in CreateInventoryTask.javaArundathi Patil1-4/+8
Fixed multiple sonar issues in this file Issue-ID: SO-2098 Change-Id: I7da9ffae8ee11d8475445b5263bc7cfb78834a9c Signed-off-by: Arundathi Patil <arundpil@in.ibm.com>
2019-07-02Added null check to prevent call of getVnfResources() methodOleksandr Moliavko1-2/+4
for null vf object Issue-ID: SO-1841 Signed-off-by: Oleksandr Moliavko <o.moliavko@samsung.com> Change-Id: I0c2ea5de5ed9ab55ad78f58f3a68241e7cb2d064
2019-07-02Added null check for heatStack to prevent crash atOleksandr Moliavko1-1/+1
getCanonicalName() call Issue-ID: SO-1841 Signed-off-by: Oleksandr Moliavko <o.moliavko@samsung.com> Change-Id: Icf49b7db336002ad56769b221ec61101fac04fd5
2019-06-24Sonar fixes in MsoVnfAdapterAsyncImplr.bogacki1-5/+3
General fixes and improvements according to the Sonar analysis. -Removed unnecessary code. Issue-ID: SO-1220 Signed-off-by: Robert Bogacki <r.bogacki@samsung.com> Change-Id: I0f9ab1fe490975e4eefa31afd5bf09616f4de3f2
2019-06-18Merge "Changes related to eviction of connections"Steve Smokowski1-0/+77
2019-06-18Fixed critical Sonar bugsr.bogacki1-4/+7
Fixed critical bugs according to the Sonar analysis. -Fixed possible NPE. -Fixed wrong comparison. -Removed unused variables. Issue-ID: SO-2025 Signed-off-by: Robert Bogacki <r.bogacki@samsung.com> Change-Id: I5e2e5413c8421eb7ed1db21de6d888a50e1db689
2019-06-17Changes related to eviction of connectionsBenjamin, Max (mb388a)1-0/+77
These are changes related to ASDC controller for evicting database connection pool when ip address changes Included other appllications such as requestDB, CatalogDB, API Handler, BPMN, OpenstackAdapter etc Change-Id: I9a96ea12fb0c10643a204a58d55360bebba326fe Issue-ID: SO-2018 Signed-off-by: Benjamin, Max (mb388a) <mb388a@us.att.com>
2019-06-17Merge "Fix sonar issues in mso-openstack-adapters"Steve Smokowski1-79/+62
2019-06-17Improvements in MsoVnfAdapterImplr.bogacki1-67/+62
Improvements in MsoVnfAdapterImpl according to the Sonar analyses. -Removed unnecessary code. -Fixed imports. -Extracted constants. Issue-ID: SO-2011 Signed-off-by: Robert Bogacki <r.bogacki@samsung.com> Change-Id: I9dc8406fe0cdea74b08ebb013a68f970da647b6c
2019-06-14update multi stage code to accurately skip bbsBenjamin, Max17-0/+340
moved around classes to packages that make more sense updated license headers in java files converted MultiStage logic to FlowManipulator moved tests to new package location and fixed issues allow for custom listeners before flowsToExecute moved validation classes to listener.validation Change-Id: Ic5416812ef9c5611fe19bf9b6aa316373b110ec6 Issue-ID: SO-2021 Signed-off-by: Benjamin, Max (mb388a) <mb388a@us.att.com>
2019-06-10replaced String.repeat with static final stringsBenjamin, Max9-165/+155
replaced String.repeat with static final strings Change-Id: Ife14fde70e8f648881a03e9343ae8faf9cb87e2e Issue-ID: SO-1999 Signed-off-by: Benjamin, Max (mb388a) <mb388a@us.att.com>
2019-06-03Cleaned up duplicated literalsrob.bog7-128/+143
Cleaned up Sonar code smells related to the duplicated literals in loggers. Issue-ID: SO-1932 Signed-off-by: Robert Bogacki <r.bogacki@samsung.com> Change-Id: I58c48cec0012c42b7b892ae76c24c6a087c60c7e
2019-05-31enhanced vnf adapter errog log.Sangalang, Felix3-6/+15
enhanced vnf adapter error log for heatEnvironment when null. removed the junit tests because of no assertions coverage. enhanced vnf adapter errog log. Change-Id: I80d3070ce203c4c8339f0a0e7df6f3731d0a16c2 Issue-ID: SO-1967 Signed-off-by: Benjamin, Max (mb388a) <mb388a@us.att.com>
2019-05-31Merge "converted lock time from hard coded value"Steve Smokowski2-5/+9
2019-05-31Corrected enum formatting by pluginBenjamin, Max (mb388a)1-3/+9
enums are now single item per line Change-Id: I6330ea4a9093071457dd47b95b6ed3bb7f8312e7 Issue-ID: SO-1970 Signed-off-by: Benjamin, Max (mb388a) <mb388a@us.att.com>