wu-sheng commented on a change in pull request #23:
URL: 
https://github.com/apache/skywalking-satellite/pull/23#discussion_r574195996



##########
File path: docs/en/setup/README.md
##########
@@ -0,0 +1,19 @@
+# Setup
+First and most important thing is, SkyWalking Satellite startup behaviours are 
driven by configs/satellite_config.yaml. Understood the setting file will help 
you to read this document.
+
+## Startup script
+The startup script is /bin/skywalking-satellite-{version}-{plateform}-amd64. 
+1. Start SkyWalking Satellite.
+```shell script
+./bin/skywalking-satellite-{version}-{plateform}-amd64 start 
--config=./configs/satellite_config.yaml

Review comment:
       This is correct and not correct at the same time. This actually is a 
dev-command for the source code local compiling startup.
   But usually, like you see in SkyWalking's main repo release, you should have 
a DIST tar, including binaries and config file in the same folder. 
   @EvanLjp Do you plan to use another PR to do this?




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