aboutsummaryrefslogtreecommitdiffstats
path: root/appc-adapters/appc-chef-adapter/appc-chef-adapter-bundle/src/main
AgeCommit message (Collapse)AuthorFilesLines
2019-09-17Sonar fix- APPC-1715Arundathi Patil1-4/+5
Fixed multiple sonar issues across this file Issue-ID: APPC-1715 Change-Id: I5a7df4bde164b8ec3c8f91f07646d0630d52fa59 Signed-off-by: Arundathi Patil <arundpil@in.ibm.com>
2019-08-19APPC-1681- remove the thrown exception 'org.json.JSONException' which is a ↵Thugutla Sailakshmi1-1/+1
runtime exception. remove the thrown exception Issue-ID: APPC-1681 Change-Id: I940a5dd42bfe7a52b39be81a016cf4cf47805cae Signed-off-by: Thugutla Sailakshmi <tsaila10@in.ibm.com>
2019-07-12Sonar Fix - Removed unused Variable Utils.javaSudarshan Kumar1-1/+0
Sonar Fix - Removed unused Variable Utils.java Issue-ID: APPC-1629 Change-Id: I557c065ff835834f5eed5eb80bd7136e182217f5 Signed-off-by: Sudarshan Kumar <sudars19@in.ibm.com>
2019-04-02Used constants to avoid dup of literalsJegadeesh Babu1-5/+7
Sonar fix Issue-ID: APPC-1562 Change-Id: Ic9d14b32f4deac975b46154a2ac8e4664ec11c52 Signed-off-by: Jegadeesh Babu <jegabab1@in.ibm.com>
2019-04-02Reordered the modifier as per java standardJegadeesh Babu1-2/+4
Sonar fix Issue-ID: APPC-1563 Change-Id: I9a4b82ca67b5c7f5184457b95c347089de728658 Signed-off-by: Jegadeesh Babu <jegabab1@in.ibm.com>
2019-03-27Used constants in the place of dup literalsJegadeesh Babu1-4/+5
Sonar fix Issue-ID: APPC-1557 Change-Id: I4b52b9febabd0fae8aa098004aeba0ef9a085710 Signed-off-by: Jegadeesh Babu <jegabab1@in.ibm.com>
2019-03-18Replaced hardcoded value with constant definedJegadeesh Babu1-2/+4
Sonar fix Issue-ID: APPC-1539 Change-Id: Id3891e50bdf89799d1310b37c80ca920d7bdc02c Signed-off-by: Jegadeesh Babu <jegabab1@in.ibm.com>
2019-03-12Added loggerJegadeesh Babu1-0/+4
Sonar fix Issue-ID: APPC-1527 Change-Id: Ic2708d67b6f2740ddabe834e56b09c11389b8894 Signed-off-by: Jegadeesh Babu <jegabab1@in.ibm.com>
2019-01-03Fixed sonar issue in Utils.javaezhil1-7/+2
Change-Id: I5edf48443c10f08200b0ea376194a9a8fc6dc29a Issue-ID: APPC-1302 Signed-off-by: ezhil <ezhrajam@in.ibm.com>
2018-12-18Sonar Fix: ChefResponse.javaArundathi Patil1-4/+6
Fixed sonar issues/code-smells across this file. Issue-ID: APPC-1279 Change-Id: I50d5de26d69928037882686452aecd3deb6edbec Signed-off-by: Arundathi Patil <arundpil@in.ibm.com>
2018-11-28Sonar Fix: ChefApiClientFactory.javaArundathi Patil1-3/+3
Fixed sonar issues/code-smells across this file Issue-ID: APPC-1253 Change-Id: Id1060c34f0b99e906dba618d9ea0aebca0fc1beb Signed-off-by: Arundathi Patil <arundpil@in.ibm.com>
2018-06-21chef adapter changes for pushjob status nodescheckModaboina, Kusumakumari (km583p)1-75/+103
Issue-ID: APPC-1012 Change-Id: Iac03825b092f62a8d3747a62303f7869bc8280e8 Signed-off-by: Modaboina, Kusumakumari (km583p) <km583p@us.att.com>
2018-06-13More updates to lic header in appc propertiesJohn McClung1-2/+1
License header fixes to reflect 2018 changes in various appc-adapters, appc-client, appc-common, appc-config, appc-dg and appc-directed-graph java and properties files. Change-Id: Idbe000f175e11ba42f6e83689a2b2d3d353be008 Issue-ID: APPC-996 Signed-off-by: John McClung <jm1364@att.com>
2018-06-01Continue removing ECOMP trademark from lic header.John McClung3-6/+3
Remove ECOMP trademark license header in miscellaneous files. Change-Id: I76e386462402cb959de377b56997ecf7986ffd6b Issue-ID: APPC-931 Signed-off-by: John McClung <jm1364@att.com>
2018-04-27Fixes for Chef Adapter bundleWonnell, Skip (kw5258)5-51/+77
Programatically configure values for truststore and password when initializing the HttpClient. Update post and put methods to set entity Add organization into request URI Restore log messages to original values Issue-ID: APPC-868 Change-Id: Ia0fdf13fcb1e3ad91a4d2c5ce704523562191379 Signed-off-by: Wonnell, Skip (kw5258) <kw5258@att.com>
2018-04-25ChefAdapterImpl junitsTomasz Gwozdecki1-5/+7
-Added junit tests for fetchResults method -Support for Michal Kabaj as part of APPC-437 Change-Id: I3219a5ec0ffd530338ce364f608a9706d5a92c03 Issue-ID: APPC-858 Signed-off-by: Tomasz Gwozdecki <tomasz.gwozdecki@nokia.com>
2018-03-26ChefAdapterImpl junitsMichal Kabaj1-4/+0
-Add new JUnits to ChefAdapterImplVNFCOperationsTest -remove unused JSONException catch clause from ChefAdapterImpl, when there is no jackson lib used Change-Id: I96b8d6155ca13087b697a19e1fcbba2ee82a8291 Issue-ID: APPC-437 Signed-off-by: Michal Kabaj <michal.kabaj@nokia.com>
2018-03-23ChefAdapterImpl cleanupMichal Kabaj2-301/+24
- Switched usage of RequestContext class for wrapping the SvcLogicContext to using the SvcLogicContext directly - Removed unused RequestContext class Change-Id: I7376e3d6f5602887cf5a25f1f90ff45805c620b2 Issue-ID: APPC-437 Signed-off-by: Michal Kabaj <michal.kabaj@nokia.com>
2018-03-22ChefAdapterImpl junitsMichal Kabaj1-29/+25
- Added new Junit class ChefAdapterImplVNFCOperationsTest - Added unit tests for method vnfcEnvironment - Small refactor to for exception handling in vnfcEnvironment method to improve readability and avoid catching and rethrowing same exception Change-Id: I58a43390ff45598f6d17da48eed3620667008cb7 Issue-ID: APPC-437 Signed-off-by: Michal Kabaj <michal.kabaj@nokia.com>
2018-03-20ChefAdapterImpl JunitsMichal Kabaj1-8/+6
- Added new Unit tests for ChefAdapterImpl checkPushJob method to class ChefAdapterImplJobPusherTest - Added param check in implementation method - Small refactor of nested conditions for improved readability Change-Id: Id65b897535874eea6aafda66b7ac227832446cf4 Issue-ID: APPC-437 Signed-off-by: Michal Kabaj <michal.kabaj@nokia.com>
2018-03-13ChefAdapterImpl JunitsMichal Kabaj3-49/+23
-Added new Unit tests for ChefAdapterImpl#trigger method -Refactored ChefAdapterFactory to allow mock injection -Added new create() method to ChefApiClientFactory which does not set any Chef specific headers for the purpose of trigger method -Cleaned up the trigger method - simplified usage, removed useless doFailure method and inlined its logic and unified HttClient usage -> HttpClient is not used directly but through existing ChefApiClient -removed redundant TestChefAdapterImpl tests which have already been rewritten Change-Id: I830d203877224aa2c5e1b0dff39d1171f1f2f0ef Issue-ID: APPC-437 Signed-off-by: Michal Kabaj <michal.kabaj@nokia.com>
2018-03-12Decouple ChefApiClientImpl from header creationMichal Kabaj3-20/+41
Introduce new FunctionalInterface HttpHeaderFactory to allow customization of http header providers when creating ChefApiClientImpl using ChefApiClientFactory This allows for better encapsulation of header specific values such as: userId,organizations,pemPath. Change-Id: Iec2c7cff8e664a18e15d7e973920152fdb158c98 Issue-ID: APPC-437 Signed-off-by: Michal Kabaj <michal.kabaj@nokia.com>
2018-03-09ChefAdapterImpl JUnitsMichal Kabaj5-29/+87
-Added unit tests for chefXXX methods - extracted new test class -Moved privateKeyCheck method to new class PrivateKeyChecker + JUnits -New Factory class ChefAdapterFactory for ChefAdapterImpl construction -Fixed potential bug with wrong String being assigned in iniSvcLogic method Change-Id: I27badfd01bfaa807b8ecb9b4a4c13e7f026e34af Issue-ID: APPC-437 Signed-off-by: Michal Kabaj <michal.kabaj@nokia.com>
2018-03-06ChefAdapterImpl JUnitsMichal Kabaj1-8/+2
-Added unit testcases for retrieveData method - extracted new test class -Added unit test for combineStrings method Change-Id: I8adcc9d2dc17e4ebf305fc2a8139095d5841a445 Issue-ID: APPC-437 Signed-off-by: Michal Kabaj <michal.kabaj@nokia.com>
2018-03-01ChefAdapterImpl JUnitsMichal Kabaj1-36/+3
-Added new ChefAdapterImplTest class for new ChefAdapterImpl JUnits -Added test for nodeObjectBuilder method - JSON Format is now verified using JSONObject -Unified ChefAdapterImpl construction - removed unused constructors Change-Id: I63f96bac0129b5f13d34efd3a173784e0b1d8a5d Issue-ID: APPC-437 Signed-off-by: Michal Kabaj <michal.kabaj@nokia.com>
2018-02-23ChefApiClient Package Reorganization and cleanupMichal Kabaj13-328/+37
-Reorganize ChefApiClient classes according to their usage, construction and implementation. -Introduce api and impl packages. -Extract interface from ChefApiClient impl -Remove unused old chefapi.* classes Change-Id: Ic1ddbdcfd0e1bbc0830b923da5aac5b68ac44a2f Issue-ID: APPC-437 Signed-off-by: Michal Kabaj <michal.kabaj@nokia.com>
2018-02-19ChefApiClient JUnitsMichal Kabaj7-102/+409
Add(fix) JUnits for ChefApiClient + refactor. Junits work on mocked HttpClient instead of attempting to make real calls. All ChefApiClient methods are now tested. Improve readability and refactored the ChefApiClient flow (simplified). Heavily improved SRP in both ChefApiClient and Api request creation. Change-Id: I997490bf269e7d78b741baf7f8d5d7bf64f6b99a Issue-ID: APPC-437 Signed-off-by: Michal Kabaj <michal.kabaj@nokia.com>
2018-02-09ChefAdapterImpl sonar fixesJakub Dudycz1-70/+69
Change-Id: I3e1a496e18a4b9e6e9dc954bc16fb4af991fd5d3 Issue-ID: APPC-579 Signed-off-by: Jakub Dudycz <jakub.dudycz@nokia.com>
2018-02-07Removing old name referencesPatrick Brady2-2/+2
Removing references to bvc, att Change-Id: I794861fe40b96c922f6e5c011b2fe5d2c4bc515b Signed-off-by: Patrick Brady <pb071s@att.com> Issue-ID: APPC-292
2018-02-07ChefActivator JUnitsMichal Kabaj3-42/+22
Add new JUnits for ChefActivator class, minor refactor + cleanup. - Remove redundant getAdapterName() method from ChefAdapter to unify adapter name constant definition Change-Id: I483d34aaa0f4e76a4360b179f1a60cc1263ec9b7 Issue-ID: APPC-437 Signed-off-by: Michal Kabaj <michal.kabaj@nokia.com>
2018-01-25Minor and major fixes in ChefAdapterImplJakub Dudycz2-147/+150
Actually did some other refactor too Change-Id: I4b622018baa2ed130a5904dbc824fc01e82c8a60 Issue-ID: APPC-508 Signed-off-by: Jakub Dudycz <jakub.dudycz@nokia.com>
2018-01-25ChefActivator fixesJakub Dudycz1-9/+8
Change-Id: Iddff91573fa5f7cc99171ca0b782415e77560154 Issue-ID: APPC-514 Signed-off-by: Jakub Dudycz <jakub.dudycz@nokia.com>
2018-01-25Fix loging in Utilsburdziak1-5/+8
Change-Id: Ibe5250e426bfc3387cbe886be4048f18dbe45c13 Issue-ID: APPC-491 Signed-off-by: burdziak <olaf.burdziakowski@nokia.com>
2018-01-18Reformat and clean up ChefAdapterImpl.javaSkip Wonnell1-426/+345
Fix some Sonar issues Update code to ONAP java style Issue-ID: APPC-419 Change-Id: I0c6b7b684c10b3874c3ef7de40066eb6c3b6345e Signed-off-by: Skip Wonnell <skip@att.com>
2018-01-18Fix bundle name in ChefActivator.javaSkip Wonnell1-1/+1
Issue-ID: APPC-420 Change-Id: Ia55c85cec3024f4199920200e614d60c7822bbbb Signed-off-by: Skip Wonnell <skip@att.com>
2018-01-18fixed some Sonar critical issuesTaka1-2/+0
Change-Id: I9dfe73a45b43919ff7b01ab3217d957a40ac2018 Issue-ID: APPC-408 Signed-off-by: Taka <tc012c@att.com>
2018-01-09code changes for platform hardening appc adapterskusuma kumari M5-151/+569
Issue-ID: APPC-305 Change-Id: Ice60ca77ffccc409d50ef11318e841c3b1f82ce1 Signed-off-by: kusuma kumari M <km583p@att.com>
2017-12-13Fourth part of onap renamePatrick Brady1-1/+1
Some paths within the docker image need to be updated from opt/openecomp to opt/onap Change-Id: I74b32ba6906a0b7c44764485dccef6140c21afa3 Signed-off-by: Patrick Brady <pb071s@att.com> Issue-ID: APPC-13
2017-12-13Second part of onap renamePatrick Brady12-0/+0
This is the second commit of the rename. The folder structure is renamed for appc-adapters and appc-config in this commit. Change-Id: Iaa2b8c937ff1ca1b5d1178128961fb115ee65d9b Signed-off-by: Patrick Brady <pb071s@att.com> Issue-ID: APPC-13
2017-12-13First part of onap renamePatrick Brady12-95/+95
This is the first commit of the rename. This commit changes refereces to org.openecomp.appc > org.onap.appc that occur within files. The package folder structure is not changed in this commit. Change-Id: Ic95d749eb99d8a6f4f2b9ee9b06eb41c5cfa7e1c Signed-off-by: Patrick Brady <pb071s@att.com> Issue-ID: APPC-13
2017-12-07Fixed several Sonas blockerTaka1-1/+0
Change-Id: Iea0a227957092101094ac99967efdb552453363d Issue-ID: APPC-337 Signed-off-by: Taka <tc012c@att.com>
2017-09-18Add Unit Tests for appc-chef-adapterMarcus G K Williams2-17/+19
Issue-Id: APPC-181 Change-Id: Iee824ca929ea46944ac1673e709074687ad812f9 Signed-off-by: Marcus G K Williams <marcus.williams@intel.com>
2017-09-07Change to CCSDK and ODL CarbonPatrick Brady4-5/+4
Change-Id: I49a0f436e366ff4a1b99705fb6bb5e7ba82e6ecc Signed-off-by: Patrick Brady <pb071s@att.com> Issue-Id: [APPC-91] Issue-Id: [APPC-158]
2017-09-01Code clean-up in ChefAdapterImpl.javaSkip Wonnell1-116/+67
Fix multiple issues found in Sonar Analysis Issue-ID: APPC-156 Change-Id: I242c45a2eab7c4e31fb42ff9d17405aa3f210147 Signed-off-by: Skip Wonnell <skip@att.com>
2017-08-31Fix sonar blocker in ChefAdpaterSkip Wonnell1-15/+5
Issue-ID: APPC-156 Change-Id: Ic2cbb76bb639094386c14bd06e068c9d5ad7bd0d Signed-off-by: Skip Wonnell <skip@att.com>
2017-08-30ChefAdapter code clean-upSkip Wonnell11-1036/+769
Issue-ID: APPC-90 Change-Id: I582d6b0c81d5a89938fff6038dfcc221ccaf19c1 Signed-off-by: Skip Wonnell <kw5258@att.com>
2017-08-02Merging in bug fixesPatrick Brady1-49/+82
Change-Id: I85da90b9ba0d03149f42035a820c76ef0285e413 Signed-off-by: Patrick Brady <pb071s@att.com> Issue: APPC-100
2017-07-27Fix resource leak in ChefAdapterImpl.javaSkip Wonnell1-5/+4
Convert to Closeable versions of HttpClient and HttpResponse and instantiate both inside a TWR. Change-Id: I37ac34bab545592f51fb054068065fa1c00f7224 Signed-off-by: Skip Wonnell <kw5258@att.com> Issue: APPC-94
2017-07-20[APPC-93] Resource leak in appc.adapter.chef.chefclient.UtilsSkip Wonnell1-8/+2
Place instantiation of FileReader and BufferedReader inside a TWR Change-Id: Ibc0fffa76e536bdccca92731dc7ebc878faf67a1 Signed-off-by: Skip Wonnell <kw5258@att.com>
2017-07-20Applying license changes to all filesPatrick Brady12-36/+60
[APPC-87] Fix Maven License Plugin Change-Id: I23a95b6c1020618d05f17c63c72152cf86e4d4a0 Signed-off-by: Patrick Brady <pb071s@att.com>