summaryrefslogtreecommitdiffstats
path: root/build/hv-collector-analysis
diff options
context:
space:
mode:
authorFilip Krzywka <filip.krzywka@nokia.com>2019-01-24 11:21:26 +0100
committerFilip Krzywka <filip.krzywka@nokia.com>2019-01-29 11:24:31 +0000
commite7204cbcf6af61856330cffc541b6f5c78476a09 (patch)
tree7c5ed27887698b028901d4d2168e65d44df8633c /build/hv-collector-analysis
parent40993732b302ce43ae1dbedbda44cc0113e9b6f2 (diff)
Correct totalConnections metric
In previous implementation performed healthcheck on container was counted as client connection, because metric was notified for every TCP channel opened. This was making this metric less useful (to avoid saying "useless" ;) ). - refactored NettyTcpServer trying to extract functions with logic not so strictly related to HV-VES behavior. This also resolves discussions in https://gerrit.onap.org/r/#/c/76274/ - some renames and methods splitting was made in attempt to make code more readable - hv-ves should not count connections from either "127.0.0.1" or "localhost" to his `totalConnections` metric - removed redundant logging by adding new methods to Logger Change-Id: I5f10dac8dac82eafd3b0de92a7ec43f2c23b8c16 Issue-ID: DCAEGEN2-1119 Signed-off-by: Filip Krzywka <filip.krzywka@nokia.com>
Diffstat (limited to 'build/hv-collector-analysis')
-rw-r--r--build/hv-collector-analysis/src/main/resources/onap-detekt-config.yml2
1 files changed, 1 insertions, 1 deletions
diff --git a/build/hv-collector-analysis/src/main/resources/onap-detekt-config.yml b/build/hv-collector-analysis/src/main/resources/onap-detekt-config.yml
index 36f092dd..f8d8b1ee 100644
--- a/build/hv-collector-analysis/src/main/resources/onap-detekt-config.yml
+++ b/build/hv-collector-analysis/src/main/resources/onap-detekt-config.yml
@@ -116,7 +116,7 @@ complexity:
thresholdInObjects: 11
thresholdInEnums: 11
ignoreDeprecated: false
- ignorePrivate: true
+ ignorePrivate: false
empty-blocks:
active: true