From 5aa7cc4e73097b59981a1a2ffa16152717d511f0 Mon Sep 17 00:00:00 2001
From: Yingge He <157551214+yinggeh@users.noreply.github.com>
Date: Fri, 12 Apr 2024 18:04:54 -0700
Subject: [PATCH] Update L0_logging to reflect error when trying to update
 log_file (#7112)

---
 qa/L0_logging/logging_endpoint_test.py | 27 +++++++-------------------
 1 file changed, 7 insertions(+), 20 deletions(-)

diff --git a/qa/L0_logging/logging_endpoint_test.py b/qa/L0_logging/logging_endpoint_test.py
index f48f857f66..4b50231189 100755
--- a/qa/L0_logging/logging_endpoint_test.py
+++ b/qa/L0_logging/logging_endpoint_test.py
@@ -227,34 +227,21 @@ def test_grpc_update_settings(self):
         triton_client = grpcclient.InferenceServerClient("localhost:8001")
 
         log_settings_1 = {
+            "log_file": "log_file.log",
             "log_info": True,
             "log_warning": True,
             "log_error": True,
             "log_verbose_level": 0,
             "log_format": "default",
         }
-        expected_log_settings_1 = grpcclient.service_pb2.LogSettingsResponse()
-        json_format.Parse(
-            json.dumps(
-                {
-                    "settings": {
-                        "log_file": {"stringParam": ""},
-                        "log_info": {"boolParam": True},
-                        "log_warning": {"boolParam": True},
-                        "log_error": {"boolParam": True},
-                        "log_verbose_level": {"uint32Param": 0},
-                        "log_format": {"stringParam": "default"},
-                    }
-                }
-            ),
-            expected_log_settings_1,
+        expected_log_settings_1 = (
+            "log file location can not be updated through network protocol"
         )
 
-        self.assertEqual(
-            expected_log_settings_1,
-            triton_client.update_log_settings(settings=log_settings_1),
-            "Unexpected updated log settings",
-        )
+        with self.assertRaisesRegex(
+            InferenceServerException, expected_log_settings_1
+        ) as e:
+            triton_client.update_log_settings(settings=log_settings_1)
 
         log_settings_2 = {
             "log_info": False,