[openwhisk] branch master updated: Update wrong error message "action does not exist". (#5293)

[style95]

This is an automated email from the ASF dual-hosted git repository.

style95 pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/openwhisk.git


The following commit(s) were added to refs/heads/master by this push:
     new 9bc5e0434 Update wrong error message "action does not exist". (#5293)
9bc5e0434 is described below

commit 9bc5e043444d115cec0d79ac47d7002baa522924
Author: Dominic Kim <styl...@apache.org>
AuthorDate: Mon Jul 25 21:36:35 2022 +0900

    Update wrong error message "action does not exist". (#5293)
---
 .../core/containerpool/v2/FunctionPullingContainerProxy.scala      | 7 +++----
 1 file changed, 3 insertions(+), 4 deletions(-)

diff --git 
a/core/invoker/src/main/scala/org/apache/openwhisk/core/containerpool/v2/FunctionPullingContainerProxy.scala
 
b/core/invoker/src/main/scala/org/apache/openwhisk/core/containerpool/v2/FunctionPullingContainerProxy.scala
index 6ad2ba844..d290a6bde 100644
--- 
a/core/invoker/src/main/scala/org/apache/openwhisk/core/containerpool/v2/FunctionPullingContainerProxy.scala
+++ 
b/core/invoker/src/main/scala/org/apache/openwhisk/core/containerpool/v2/FunctionPullingContainerProxy.scala
@@ -905,9 +905,8 @@ class FunctionPullingContainerProxy(
                 logging.error(this, s"An unknown DB connection error occurred 
while fetching an action: $e.")
                 
ExecutionResponse.whiskError(Messages.actionFetchErrorWhileInvoking)
             }
-            logging.error(
-              this,
-              s"Error to fetch action ${msg.action} for msg 
${msg.activationId}, error is ${t.getMessage}")
+            val errMsg = s"Error to fetch action ${msg.action} for msg 
${msg.activationId}, error is ${t.getMessage}"
+            logging.error(this, errMsg)
 
             val context = UserContext(msg.user)
             val activation = generateFallbackActivation(action, msg, response)
@@ -921,7 +920,7 @@ class FunctionPullingContainerProxy(
             storeActivation(msg.transid, activation, msg.blocking, context)
 
             // in case action is removed container proxy should be terminated
-            Future.failed(new IllegalStateException("action does not exist"))
+            Future.failed(new IllegalStateException(errMsg))
         }
     } else {
       // Iff the current namespace is blacklisted, an active-ack is only 
produced to keep the loadbalancer protocol