Code Review / cps.git / commit
? search:
summary | shortlog | log | commit | commitdiff | review | tree
(parent: 37e328e) | patch
CPS NCMP: Resolved high cardinality of prometheus metrics for dmi service url 18/138318/13
authorsourabh_sourabh <sourabh.sourabh@est.tech>
Thu, 20 Jun 2024 16:48:47 +0000 (17:48 +0100)
committersourabh_sourabh <sourabh.sourabh@est.tech>
Wed, 3 Jul 2024 15:52:49 +0000 (16:52 +0100)
commitd6f0977a25cf0896227ae6cc5abd7866af80e237
treeece0f193d5fcefdb18b34da406ef8b252485c369tree | snapshot
parent37e328eeaa3e79416e70f348da79cff3ad2e547bcommit | diff
 CPS NCMP: Resolved high cardinality of prometheus metrics for dmi service url

 - Used autoconfigured web client builder for http_client_requests_* prometheus metrics.
 - Refactored dmi service url builder to create url template and its
   variables.
 - Web client is modified to use uri(urlTemplate, urlvars) version.
 - Deleted InvalidDmiResourceUrlException that no longer needed.
 - Used DmiServiceUrlBuilder to build dmi health check service url.
 - Created a new pkg url.builder into utils to have all related classes
   and record.

Issue-ID: CPS-2121
Change-Id: Id67e0f0d4e640bb8f9eea0b6c2db1dba3468e1d7
Signed-off-by: sourabh_sourabh <sourabh.sourabh@est.tech>
19 files changed:
cps-ncmp-rest/src/main/java/org/onap/cps/ncmp/rest/controller/NetworkCmProxyRestExceptionHandler.java diff | blob | history
cps-ncmp-service/src/main/java/org/onap/cps/ncmp/api/impl/client/DmiRestClient.java diff | blob | history
cps-ncmp-service/src/main/java/org/onap/cps/ncmp/api/impl/config/DmiWebClientConfiguration.java diff | blob | history
cps-ncmp-service/src/main/java/org/onap/cps/ncmp/api/impl/exception/InvalidDmiResourceUrlException.java [deleted file] blob | history
cps-ncmp-service/src/main/java/org/onap/cps/ncmp/api/impl/utils/DmiServiceUrlBuilder.java [deleted file] blob | history
cps-ncmp-service/src/main/java/org/onap/cps/ncmp/api/impl/utils/url/builder/DmiServiceUrlTemplateBuilder.java [new file with mode: 0644] blob
cps-ncmp-service/src/main/java/org/onap/cps/ncmp/api/impl/utils/url/builder/UrlTemplateParameters.java [new file with mode: 0644] blob
cps-ncmp-service/src/main/java/org/onap/cps/ncmp/impl/data/DmiDataOperations.java diff | blob | history
cps-ncmp-service/src/main/java/org/onap/cps/ncmp/impl/datajobs/DmiSubJobRequestHandler.java diff | blob | history
cps-ncmp-service/src/main/java/org/onap/cps/ncmp/impl/inventory/sync/DmiModelOperations.java diff | blob | history
cps-ncmp-service/src/main/java/org/onap/cps/ncmp/impl/inventory/trustlevel/DmiPluginTrustLevelWatchDog.java diff | blob | history
cps-ncmp-service/src/test/groovy/org/onap/cps/ncmp/api/impl/client/DmiRestClientSpec.groovy diff | blob | history
cps-ncmp-service/src/test/groovy/org/onap/cps/ncmp/api/impl/config/DmiWebClientConfigurationSpec.groovy diff | blob | history
cps-ncmp-service/src/test/groovy/org/onap/cps/ncmp/api/impl/utils/DmiServiceUrlBuilderSpec.groovy [deleted file] blob | history
cps-ncmp-service/src/test/groovy/org/onap/cps/ncmp/api/impl/utils/url/builder/DmiServiceUrlTemplateBuilderSpec.groovy [new file with mode: 0644] blob
cps-ncmp-service/src/test/groovy/org/onap/cps/ncmp/impl/DmiOperationsBaseSpec.groovy diff | blob | history
cps-ncmp-service/src/test/groovy/org/onap/cps/ncmp/impl/data/DmiDataOperationsSpec.groovy diff | blob | history
cps-ncmp-service/src/test/groovy/org/onap/cps/ncmp/impl/inventory/sync/DmiModelOperationsSpec.groovy diff | blob | history
cps-ncmp-service/src/test/groovy/org/onap/cps/ncmp/impl/inventory/trustlevel/DmiPluginTrustLevelWatchDogSpec.groovy diff | blob | history
"cps"