Merge "Sonar Fix - LeanCloudConfiguration.java - Move variable to comply with Java Code Conventions"

Merge "Sonar Fix - HttpsAuthClient.java"

Merge changes from topics "VID-580", "VID-579", "VID-578"

* changes:
  unused import
  Move the variable to comply with JCC.
  semicolon at the end of this statement.

unused import

The imports part of a file should be handled by the Integrated Development Environment (IDE), not manually by the developer.
Unused and useless imports should not occur if that is the case.
Leaving them in reduces the code's readability, since their presence can be confusing.

Issue-ID: VID-580
Change-Id: I3ea680af37477b564d02fcc9b58956b3958d184d
Signed-off-by: anushadasari <danush10@in.ibm.com>

Move the variable to comply with JCC.

According to the Java Code Conventions as defined by Oracle, the members of a class or interface declaration should appear in the following order in the source files:
Class and instance variables
Constructors
Methods

Issue-ID: VID-579
Change-Id: Ie87b550a30df06eb2498565b719bd52b11d0c3f9
Signed-off-by: anushadasari <danush10@in.ibm.com>

semicolon at the end of this statement.

In JavaScript, the semicolon (;) is optional as a statement separator, but omitting semicolons can be confusing, and lead to unexpected results because a semicolon is implicitly inserted at the end of each line.

Change-Id: Ia09e1bb1894da379f68cc39e83f955e66b957511
Issue-ID: VID-578
Signed-off-by: anushadasari <danush10@in.ibm.com>

adding update about mounted certificates in rst file

Issue-ID: VID-494
Change-Id: I732a426975247448f05539d1a95c0b6c66ccf5a7
Signed-off-by: Bartosz Gardziejewski <bartosz.gardziejewski@nokia.com>

Sonar Fix - LeanCloudConfiguration.java - Move variable to comply with Java Code Conventions

Fixed sonar issus across the file

Issue-ID: VID-547
Change-Id: I3362cf34c3db506ab3359669045169fe761743ea
Signed-off-by: dinesha <dineshgs@in.ibm.com>

Skip log format validation for a week

Add annotation SkipTestUntil and TestNg transformer so we can add
annotation for skip test till some date (aka time bomb)
Use it in log format tests
Issue-ID: VID-378

Signed-off-by: Eylon Malin <eylon.malin@intl.att.com>
Change-Id: Ib4b5d17d73d360fd2554b5ffc0373790d22d9e4e
Signed-off-by: Eylon Malin <eylon.malin@intl.att.com>

Merge "Move this variable to comply with Java Code Conventions"

Merge "Move this variable to comply with Java Code Conventions"

Sonar Fix - HttpsAuthClient.java

Fixed Sonar issues accross the file

Issue-ID: VID-547
Change-Id: I2b344b48fec4e153cb47154ec49998aa1c267a95
Signed-off-by: dinesha <dineshgs@in.ibm.com>

Merge "moving certs from docker image to mounted catalog"

Merge "Bump VID version to 5.0.1"

Correct link in info.yaml removal entry

Change-Id: I112a9130260627e96b01ff31a67b0c7aaa2fee14
Issue-ID: VID-502
Signed-off-by: Ittay Stern <ittay.stern@att.com>

Bump VID version to 5.0.1

Issue-ID: VID-494

Change-Id: I0564a6ac162b82211733525133c6e1dadf1541c0
Signed-off-by: Ittay Stern <ittay.stern@att.com>

Add a semicolon at the end of this statement

In JavaScript, the semicolon (;) is optional as a statement separator, but omitting semicolons can be confusing, and lead to unexpected results because a semicolon is implicitly inserted at the end of each line.

Issue-ID: VID-568
Change-Id: Id89534f4fbd326ebcbe12857bce7e2d69bf73dfd
Signed-off-by: anushadasari <danush10@in.ibm.com>

Move this variable to comply with Java Code Conventions

move the variable to comply with java code convenrions

Issue-ID: VID-561
Change-Id: I11a671d8be20429c3e53fd98d54330b982564f93
Signed-off-by: Thugutla sailakshmi <tsaila10@in.ibm.com>

tests - rename csar file names for alignment with code

Issue-ID: VID-378
Signed-off-by: Eylon Malin <eylon.malin@intl.att.com>
Change-Id: I5313de573f8077e3ddcea565c0d056c2cb5857a5

Merge "Change out-of-the-box config to work on dev's environment"

Merge "Upgrade pom files"

catch any Throwable when failed to register to simulator

Issue-ID: VID-378
Signed-off-by: Eylon Malin <eylon.malin@intl.att.com>
Change-Id: I553f8bf1fa657ce792ef690db95471de6f3658b3

Upgrade pom files

versions:use-latest-releases

Issue-ID: VID-512

Change-Id: I3bdc7ff06717776e4f15316cca623e8025066495
Signed-off-by: Einat Vinouze <einat.vinouze@intl.att.com>
Signed-off-by: Ittay Stern <ittay.stern@att.com>

Sonar Fix - ResourceType.java- Added field 'final' to the static variable

Fixed Sonar issues across the file

Issue-ID: VID-547
Change-Id: I1daa62cf2e30f5b0dbe142c7ea183f93fd7fb48f
Signed-off-by: dinesha <dineshgs@in.ibm.com>

Merge "return this expression instead of assigning it to the temporary variable"

Merge "Remove declaration of thrown exception"

Merge "Remove useless trailing whitespace"

Merge "Move this variable to comply with Java Code Conventions"

Remove declaration of thrown exception

Remove the declaration of thrown exception 'org.onap.vid.aai.exceptions.HttpClientBuilderException' which is a runtime exception.

Issue-ID: VID-559
Change-Id: I8fe8c29e3aa1d4b0b35522931057efa260c60206
Signed-off-by: anushadasari <danush10@in.ibm.com>

Remove useless trailing whitespace

Trailing whitespaces are simply useless and should not stay in code. They may generate noise when comparing different versions of the same file.

Issue-ID: VID-565
Change-Id: If5647e94ea897bf42761a689d036a7229b34a500
Signed-off-by: anushadasari <danush10@in.ibm.com>

return this expression instead of assigning it to the temporary variable

Issue-ID: VID-561
Change-Id: I64e50083b9b0fe40fa0ba32723e30adb0dbce08e
Signed-off-by: Thugutla Sailakshmi <tsaila10@in.ibm.com>

Move this variable to comply with Java Code Conventions

Issue-ID: VID-561
Change-Id: I1c59d863eb93537177e84a3a10fed0862189191c
Signed-off-by: Thugutla sailakshmi <tsaila10@in.ibm.com>

Move this variable to comply with Java Code Conventions

Issue-ID: VID-561
Change-Id: I7d3382d43a0484dbadea4d3285a235a628d84ffc
Signed-off-by: Thugutla sailakshmi <tsaila10@in.ibm.com>

Merge "set VID to use internally UTC timezone"

Merge "return MsoResponseWrapper with null body in case of null response"

Change out-of-the-box config to work on dev's environment

1) Fill-up system.properties's missing fields, and same for
   system_template.properties.

   This modifies some SO endpoints, most significantly --
   mso.restapi.vnf.instance is now
   /serviceInstantiation/v7/serviceInstances/<service_instance_id>/vnfs
   instead of
   /serviceInstances/v6/<service_instance_id>/vnfs

2) Configure endpoint's to a local simulator's location:
   * ecomp_rest_url in portal.properties
   * mso.server.url, aai.server.url in system.properties
   * asdc.client.rest.host in asdc.properties

3) Added always-on getSessionSlotCheckInterval in simulator.

Issue-ID: VID-564

Change-Id: Icc7bdc83ed65df3415a4b0f36f98291775909f49
Signed-off-by: Ittay Stern <ittay.stern@att.com>

Merge "use epsdk 2.5.0 in VID"

return MsoResponseWrapper with null body in case of null response

Issue-ID: VID-378
Signed-off-by: Eylon Malin <eylon.malin@intl.att.com>
Change-Id: Ia05237264557634f18ed868904cc68350a312fc1

set VID to use internally UTC timezone

Issue-ID: VID-378
Signed-off-by: Eylon Malin <eylon.malin@intl.att.com>
Change-Id: I4b062d4a7ec68e564c46a094a75739dc9ad0355b

fix test getServiceModelALaCarteInstantiation

Issue-ID: VID-378
Signed-off-by: Eylon Malin <eylon.malin@intl.att.com>
Change-Id: I0d98886837cc67164b77f3407323ac45e3e126a4

Move this variable to comply with Java Code Conventions

Move this variable to comply with Java Code Conventions

Issue-ID: VID-561
Change-Id: I9c40a6165ece22bbfda29e17cdb8572a2b7d11e8
Signed-off-by: Thugutla sailakshmi <tsaila10@in.ibm.com>

moving certs from docker image to mounted catalog

Removing certificates from dockerfile and adding mounted volume in docker-compose, this is connected with oom commit https://gerrit.onap.org/r/c/oom/+/92888

Issue-ID: VID-494
Change-Id: Ibf52542344ecf79fb947858329f37018731d2270
Signed-off-by: Bartosz Gardziejewski <bartosz.gardziejewski@nokia.com>

Merge "Resolving VlanTagSubInterfaceTest#createSubInterface_validPopupDataAndUI beforeTest"

Resolving VlanTagSubInterfaceTest#createSubInterface_validPopupDataAndUI beforeTest

Issue-ID: VID-566
Signed-off-by: RachelF <rachel.fishbein@intl.att.com>
Change-Id: I623835324593d1de3db08824dffe4c167264174b

Sonar Fix - Group.java

Fixed Sonar issues accross the file

Issue-ID: VID-547
Change-Id: I128b69d6ddcf8b1198ffd42530d664c92dd13cc7
Signed-off-by: dinesha <dineshgs@in.ibm.com>

support more resource types in ServiceModelInflator

Issue-ID: VID-378
Signed-off-by: Eylon Malin <eylon.malin@intl.att.com>
Change-Id: I80ce2b81202ea08491f65fc47390f826b2b02d20

Merge "add default exception handling to VidRestrictedBaseController"

add default exception handling to VidRestrictedBaseController

Issue-ID: VID-378
Signed-off-by: Eylon Malin <eylon.malin@intl.att.com>
Change-Id: Ie5bac35975257ad092c75e8e7e7dfd8fefa0363e

use epsdk 2.5.0 in VID

Issue-ID: VID-378
Signed-off-by: Eylon Malin <eylon.malin@intl.att.com>
Change-Id: Id483174b283a5f1eb6160982a446ccb223606c9e

Remove the declaration of thrown exception

Remove the declaration of thrown exception 'org.onap.vid.aai.ExceptionWithRequestInfo' which is a runtime exception.

Issue-ID: VID-559
Change-Id: I275d7323d153c70fa515af4a0417fe0910ee451d
Signed-off-by: anushadasari <danush10@in.ibm.com>

Remove Unused Import

The imports part of a file should be handled by the Integrated Development Environment (IDE), not manually by the developer.
Unused and useless imports should not occur if that is the case.
Leaving them in reduces the code's readability, since their presence can be confusing.

Issue-ID: VID-563
Change-Id: Id88ce00cca08b58e0ac4beb60026c9dae5dd892e
Signed-off-by: anushadasari <danush10@in.ibm.com>

At most one statement is allowed per line

At most one statement is allowed per line, but 2 statements were found
on this line.
For better readability, do not put more than one statement on a single line.

Issue-ID: VID-562
Change-Id: Ibdae3413af96d49ce7a39e53d3af611e237cc16b
Signed-off-by: anushadasari <danush10@in.ibm.com>

Merge "removing unused ViewLog Controller and jsp"

Merge "Use lcpRegionOptionId as value in resume"

removing unused ViewLog Controller and jsp

Issue-ID: VID-378
Signed-off-by: Eylon Malin <eylon.malin@intl.att.com>
Change-Id: I7d5cdd0091e2c67adc617894386a58a89046880e

fix getNetworkListTest UT

Issue-ID: VID-378
Signed-off-by: Eylon Malin <eylon.malin@intl.att.com>
Change-Id: Id28c5b3062f45c217f0bfc93e6d2a7c6333d15c4

Merge changes Iede12655,I73fcc78e

* changes:
  add endpoints for getVpnList and getActiveNetworkList
  use cache for retrieving all subscribers list

Use lcpRegionOptionId as value in resume

Issue-ID: VID-482
Signed-off-by: Rina Roi <rina.roi@intl.att.com>
Change-Id: Ibe617ba739db8e6217362a8009dedc9bd3465239

add endpoints for getVpnList and getActiveNetworkList

Issue-ID: VID-378
Signed-off-by: Eylon Malin <eylon.malin@intl.att.com>
Change-Id: Iede126556df337b1ab70dd76c9452f4b51e11376

use cache for retrieving all subscribers list

Issue-ID: VID-378
Signed-off-by: Eylon Malin <eylon.malin@intl.att.com>
Change-Id: I73fcc78ebbd8ee01349ecefa924c20014ca33907

Sonar Critical: Make dateFormat an instance variable

Change-Id: I1f275c2b4b814afc7c50e02e756312ce6923efe0
Issue-ID: VID-535
Signed-off-by: nandkumar <np00602065@techmahindra.com>

fix tests by adding missing preset of get subscribers from AAI

Issue-ID: VID-378
Signed-off-by: Eylon Malin <eylon.malin@intl.att.com>
Change-Id: I6dc101b76089c32b8b9d2912a070b64bf01514ec

Sonar Fix- RoleGenaratorServiceImpl.java- Define a constant instead of duplicating this literal

Fixed Sonar issues accross the file

Issue-ID: VID-547
Change-Id: I91dc81916fc1c7843e0f5b723cdf47153e682592
Signed-off-by: dinesha <dineshgs@in.ibm.com>

Merge "Sonar Fix- CommandParentData.java- Move this variable to comply with Java Code Conventions"

Sonar Fix- CommandParentData.java- Move this variable to comply with Java Code Conventions

Fixed Sonar issues accross the file

Issue-ID: VID-547
Change-Id: I104df866f1e4e8477c954942ae69fe647028e621
Signed-off-by: dinesha <dineshgs@in.ibm.com>

Sonar Fix- ProbeController.java- Reorder the modifiers to comply with the Java Language Specification

Fixed Sonar issues accross the file

Issue-ID: VID-547
Change-Id: I6240ae0bc339ba43b55b8a66609935bc63705c1d
Signed-off-by: dinesha <dineshgs@in.ibm.com>

Merge "handle errors in change management and tenant isolation"

handle errors in change management and tenant isolation

use more tolerance way for parsing responses from MSO so VID can handle
error responses with html body
Issue-ID: VID-378
Signed-off-by: Eylon Malin <eylon.malin@intl.att.com>
Change-Id: I72741ec1bf636a1200e8ecbd684ab1f3b2ab332a
Signed-off-by: Ittay Stern <ittay.stern@att.com>
Signed-off-by: Eylon Malin <eylon.malin@intl.att.com>

Upgrade Sonar and JaCoCo plugins to latest

Issue-ID: VID-425

Change-Id: Icb74dd406411e10ea48b9bee0a94e594db7308aa
Signed-off-by: Ittay Stern <ittay.stern@att.com>

Merge "Sonar Fix- ErrorReportController.java"

Sonar Fix- ErrorReportController.java

Fixed Sonar Issues accross the file

Issue-ID: VID-547
Change-Id: I915b2d28050a864db5c2a2e9b0ccce4061f46cfd
Signed-off-by: dinesha <dineshgs@in.ibm.com>

Sonar Fix- VnfDetails.java

Fixed Sonar Issues accross the file

Issue-ID: VID-547
Change-Id: I263ce5e172b68da20d537824794a3fae4455fde4
Signed-off-by: dinesha <dineshgs@in.ibm.com>

Sonar Fix- AaiClientInterface.java-Remove this unused import

Fixed sonar issues accross the file

Issue-ID: VID-547
Change-Id: I3305f201e404d0e4b94f8a9eba428b03812592ed
Signed-off-by: dinesha <dineshgs@in.ibm.com>

change management return 400 handling NPE

return 400 internal status when exception is thrown during extracting serviceInstanceId
Issue-ID: VID-378

Signed-off-by: Eylon Malin <eylon.malin@intl.att.com>
Change-Id: I97f740de0a700f503d115033980f8e8366e806e7

Remove web-filter for incoming scheduler requests

authentication can be handled in a different way like using AAF
(see 48b0c6ee)

Issue-ID: VID-378
Signed-off-by: Eylon Malin <eylon.malin@intl.att.com>
Change-Id: I68c664bc6a20ce4d8722558e00985b7638fed04b

align backend get aicZone response fields name with the frontend expectation

Issue-ID: VID-378
Signed-off-by: Eylon Malin <eylon.malin@intl.att.com>
Change-Id: Ib43b5c75ce0c841aec6e4c42bef78354ddf30493
Signed-off-by: Eylon Malin <eylon.malin@intl.att.com>

Merge "Resolving some tests in CreatePortMirroringConfigurationTest by merging instantiate.html ecomp+onap"

Resolving some tests in CreatePortMirroringConfigurationTest by merging instantiate.html ecomp+onap

Issue-ID: VID-546
Signed-off-by: RachelF <rachel.fishbein@intl.att.com>
Change-Id: Icf4ffd6c2eb26cb5cf5842f9d6bfb153431d136b

Resolving testAssociatePnf

Issue-ID: VID-543
Change-Id: Ib7d12d648f399096bf8d71b98249ff4994a156fc
Signed-off-by: RachelF <rachel.fishbein@intl.att.com>

handling cloud owner and lcp region in create port mirroring configuration

Issue-ID: VID-482
Signed-off-by: Eylon Malin <eylon.malin@intl.att.com>
Change-Id: Id7aed0f44927b27b69287f513d24552cf2afc003

handling legacy region in vfModule resume

Issue-ID: VID-482
Signed-off-by: Eylon Malin <eylon.malin@intl.att.com>
Change-Id: If0cca27729e2d7313fd1836191a35d223e276070

Add flag for modern UI

Issue-ID: VID-544
Change-Id: Iad79353ff9f55416d87b5bef56e38dd01357003b
Signed-off-by: Rina Roi <rina.roi@intl.att.com>

fix expected cloud owner in e2e AddNetworkTest

Issue-ID: VID-482
Signed-off-by: Eylon Malin <eylon.malin@intl.att.com>
Change-Id: Ieade85fa5652f1ade8058b60ef9f9ef80af28b8c

Handling legacy region in vfModule creation

Issue-ID: VID-482

Signed-off-by: Eylon Malin <eylon.malin@intl.att.com>
Change-Id: I2c55587d86a77eb0c6c2fa410b212efb074894e2

Fix /version endpoint (fixup)

Issue-ID: VID-507

Change-Id: I178063887b699997a2efa8df06bcdd0e94b2a411
Signed-off-by: Ittay Stern <ittay.stern@att.com>

Bugfix: Revert soft-delete to restore VF-module deletion

Issue-ID: VID-503

Change-Id: I2c815288140116d0c5bd4630f2688132a033a16c
Signed-off-by: Ittay Stern <ittay.stern@att.com>

Handle A&AI Service-Tree and add tests

Issue-ID: VID-378

Change-Id: Id1b86b77732768f9b497e2fcd29ee1665381ac57
Signed-off-by: Ittay Stern <ittay.stern@att.com>

Fix /version endpoint

* Modify maven-replacer-plugin to set value in version.json
* Configure features.set.filename=onap.features.properties

Issue-ID: VID-507

Change-Id: I692ab3e9221118b836ae7249f483b5277a70bf6b
Signed-off-by: Ittay Stern <ittay.stern@att.com>

Take 'contact us' url from configuration

Issue-ID: VID-541
Signed-off-by: Rina Roi <rina.roi@intl.att.com>
Change-Id: Ia7d829c20a8689f7aab737ea83ed420b6c3448a3

Merge "Add app version to welcome page"

Add app version to welcome page

Issue-ID: VID-539
Signed-off-by: Rina Roi <rina.roi@intl.att.com>
Change-Id: I595869e5d6b0af279eb61cc18dac329f7fac094a

Undo scheduler.basic.auth configurability

It is not localized with localize_system, therefore left with a
dangling placeholder `${SCHEDULER_BASIC_AUTH}`.

(Introduced in change-id Ie88c6182f2cbf468615a4146bced531c2ad150e0,
found by Robot's CSIT).

Issue-ID: VID-533

Change-Id: I5e3d177ec76cd3295e8b04d4c0596b75733effa0
Signed-off-by: Ittay Stern <ittay.stern@att.com>

updating info.yaml

updating info.yaml to follow ONAP standards

Issue-ID: VID-502
Change-Id: I8f37b42bcba112eac9149e8c40171a233dca45af
Signed-off-by: Bartosz Gardziejewski <bartosz.gardziejewski@nokia.com>

Merge "Implant vid-app-common org.onap.vid.job (main and test)"

Implant vid-app-common org.onap.vid.job (main and test)

Issue-ID: VID-378

Change-Id: I41b0bdc2c4e3635f3f3319b1cd63cefc61912dfc
Signed-off-by: Einat Vinouze <einat.vinouze@intl.att.com>
Signed-off-by: Ittay Stern <ittay.stern@att.com>

Revert "Add app version to welcome page"

This reverts commit bfd2c1c8e5ceac2da1755d6dd232d10ebe659e63.
Reason for revert: left menu is broken
Issue-ID: VID-539

Signed-off-by: Eylon Malin <eylon.malin@intl.att.com>
Change-Id: I98674e6900dddb97cb25a1e687d6dd48b41da4c5

Remove restricted notice from TOSCA file

Issue-ID: VID-499

Change-Id: Ib63c2cec9b29530461ad1ef2313f80548697b1f6
Signed-off-by: Ittay Stern <ittay.stern@att.com>

Merge "Add app version to welcome page"

Add app version to welcome page

Issue-ID: VID-539
Signed-off-by: Rina Roi <rina.roi@intl.att.com>
Change-Id: I2d0c838ea1da947ac9dc7eb5c01ba68be9c14e1a

Merge "Patch NPE in Unirest HttpResponse::getBody when getRawBody is null"