Remove OpenVINO from mem leak whitelist (triton-inference-server#4897)

* Fix indentation * Remove OpenVINO from mem leak whitelist
Spartee · Sep 20, 2022 · 7936f06 · 7936f06
1 parent c8e1384
commit 7936f06
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 9 deletions.
diff --git a/qa/L0_client_memory_growth/test.sh b/qa/L0_client_memory_growth/test.sh
@@ -135,8 +135,8 @@ for PROTOCOL in http grpc; do
         else
             python3 ../common/check_valgrind_log.py -f $LEAKCHECK_LOG
             if [ $? -ne 0 ]; then
-            echo -e "\n***\n*** Memory leak detected\n***"
-            RET=1
+                echo -e "\n***\n*** Memory leak detected\n***"
+                RET=1
             fi
 
             set +e

diff --git a/qa/L0_client_valgrind/test.sh b/qa/L0_client_valgrind/test.sh
@@ -87,8 +87,8 @@ for PROTOCOL in http grpc; do
         else
             python3 ../common/check_valgrind_log.py -f $LEAKCHECK_LOG
             if [ $? -ne 0 ]; then
-            echo -e "\n***\n*** Memory leak detected\n***"
-            RET=1
+                echo -e "\n***\n*** Memory leak detected\n***"
+                RET=1
             fi
         fi
     done

diff --git a/qa/common/check_valgrind_log.py b/qa/common/check_valgrind_log.py
@@ -35,14 +35,10 @@
 #     -> tensorflow::LibHDFS::LoadAndBind()::{lambda(char const*, void**)#1}::operator()(char const*, void**)
 #     -> tensorflow::internal::LoadLibrary
 #     -> dlerror
-#   * InferenceEngine::getAvailableCoresTypes() leak in OpenVINO could be due to
-#     https://github.com/openvinotoolkit/openvino/issues/6593. Should be fixed
-#     after version 2021.4(https://github.com/openvinotoolkit/openvino/issues/6593#issuecomment-878093502).
-#     Remove this after migrating to OpenVINO to 2022.1: DLIS-4055
 
 LEAK_WHITE_LIST = [
     'cnmem', 'tensorflow::NewSession', 'dl-init', 'dl-open', 'dlerror',
-    'libtorch', 'InferenceEngine::getAvailableCoresTypes()'
+    'libtorch'
 ]