plainolneesh commented on a change in pull request #338:
URL: https://github.com/apache/fluo-muchos/pull/338#discussion_r468579221



##########
File path: README.md
##########
@@ -227,6 +227,8 @@ to this swarm. When this is set, docker will be installed 
on all nodes of the cl
 recommended that the swarm manager is specified on a worker node as it runs 
docker containers. Check
 out [Portainer] if you want to run a management UI for your swarm cluster.
 
+7. `elkserver` - Sets up the Elasticsearch, Logstash, and Kibana stack. This 
allows logging data to be search, analyzed, and visualized in real time. 

Review comment:
       I do not think there's a dependency for docker at all for this. The 
issue with docker was originally presented because while testing my changes, I 
would have to run setup multiple times. When you did this (running setup 
repeatedly), an error occurred with docker.  I believe I tested running the 
`elkserver` service without `swarmmanger` on previously and I didn't have an 
issue. 




----------------------------------------------------------------
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.

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


Reply via email to