You are viewing a plain text version of this content. The canonical link for it is here.

Posted to issues@openwhisk.apache.org by "style95 (via GitHub)" <gi...@apache.org> on 2023/02/01 04:26:34 UTC

[GitHub] [openwhisk] style95 commented on a diff in pull request #5375: remove zookeeper config requirement

style95 commented on code in PR #5375:
URL: https://github.com/apache/openwhisk/pull/5375#discussion_r1092752011


##########
core/invoker/src/main/scala/org/apache/openwhisk/core/invoker/Invoker.scala:
##########
@@ -91,7 +91,6 @@ object Invoker {
     Map(servicePort -> 8080.toString) ++
       ExecManifest.requiredProperties ++
       kafkaHosts ++
-      zookeeperHosts ++

Review Comment:
   Without this configuration, a k8s deployment can still take advantage of the zookeeper?
   I think we need to handle the zookeeper dependency in the k8s deployment first.
   



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscribe@openwhisk.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org