diff options
| author |   Vratko Polak <vrpolak@cisco.com> | 2024-08-09 14:32:58 +0200 |
|---|---|---|
| committer | Vratko Polak <vrpolak@cisco.com> | 2024-08-09 14:32:58 +0200 |
| commit | f35ee96b116bbf3afce8925fe9fcee7ea0159e38 (patch) | |
| tree | 4dbf0abc0fc5c03c88bb4dbb065bb046ca855177 | |
| parent | 8204112a23fedf6743ca490392a7cf05c6fc033f (diff) | |
fix(papi): do not check client in async
This is primarily to avoid excessive logging
in scenarios with multiple get_replies (not just .add) per session.
Change-Id: Ic5b1c295965cd8d32c71a7851ec7823568483b06
Signed-off-by: Vratko Polak <vrpolak@cisco.com>
| -rw-r--r-- | resources/libraries/python/PapiExecutor.py | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/resources/libraries/python/PapiExecutor.py b/resources/libraries/python/PapiExecutor.py index 04dc0d99e5..e685f87a10 100644 --- a/resources/libraries/python/PapiExecutor.py +++ b/resources/libraries/python/PapiExecutor.py @@ -1065,6 +1065,8 @@ class PapiSocketExecutor: The messages were already sent by .add() in this mode, local_list is used just so we know how many replies to read. + Similarly to async .add, we do not check connection status here, + thus we avoid needless logging. Beware: It is not clear what to do when socket read fails in the middle of async processing. @@ -1088,7 +1090,7 @@ class PapiSocketExecutor: :rtype: List[UserDict] :raises RuntimeError: If the replies are not all correct. """ - vpp_instance = self.get_connected_client() + vpp_instance = self.get_connected_client(check_connected=False) ret_list = list() try: for index, _ in enumerate(local_list): |