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

Posted to commits@devlake.apache.org by GitBox <gi...@apache.org> on 2022/08/12 19:01:49 UTC

[GitHub] [incubator-devlake-website] keon94 commented on a diff in pull request #155: docs: helm doc improvements

keon94 commented on code in PR #155:
URL: https://github.com/apache/incubator-devlake-website/pull/155#discussion_r944761748


##########
docs/QuickStart/HelmSetup.md:
##########
@@ -69,7 +69,7 @@ helm install devlake . \
     --set "ingress.tlsSecretName=ssl-certificate"
 ```
 
-After deployed, visit devlake: https://devlake-0.example.com:8443, and grafana at https://devlake-0.example.com:8443/grafana

Review Comment:
   Honestly, having played with it some more, the ingress.HttpsPort setting has been inconsequential on my machine. I used both 8443 and 8444. Turns out 8443 is coincidentally the target port on the kube apiserver, and if I try hitting it directly I get a forbidden error. But 8444 maps to nothing, which leads me to believe this is not doing anything.



-- 
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: commits-unsubscribe@devlake.apache.org

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