You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@apisix.apache.org by GitBox <gi...@apache.org> on 2022/12/19 09:08:47 UTC

[GitHub] [apisix] navendu-pottekkat commented on a diff in pull request #8536: docs: add missing `kubectl exec` step in the installation guide.

navendu-pottekkat commented on code in PR #8536:
URL: https://github.com/apache/apisix/pull/8536#discussion_r1051960354


##########
docs/en/latest/installation-guide.md:
##########
@@ -216,6 +216,11 @@ brew services start etcd
 
 ### Configuring APISIX
 
+If you install APISIX via Docker, please exec into the apisix pod first:
+```shell
+kubectl exec -it <name-of-apisix-pod> -n <namespace-running-apisix> -- sh
+```
+

Review Comment:
   Can you highlight this as a note? You can use the admonition `:::note`.
   
   Please add spaces around the code blocks (between ```shell and the text). There should be a failing lint check otherwise.
   
   If you install APISIX via Docker, you need to exec into the "apisix" container to run the `apisix` command:
   
   ```
   command to exec into Docker container from host machine and running the apisix command
   ```
   
   In Kubernetes, you can do this by running:
   
   ```
   existing command mentioned in the PR
   ```



-- 
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: notifications-unsubscribe@apisix.apache.org

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