docs: 3383/docker-k8s-helm-updates#3436
Conversation
✅ Deploy Preview for brilliant-pasca-3e80ec ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
🚀 Performance Test ResultsTest Configuration:
Test Metrics:
📜 Logs |
There was a problem hiding this comment.
I can't verify the technical pieces, so I only commented on minor things. I did have some thoughts, though. Let me know what you think.
- For the two sections that have steps appearing in the page's table of contents, perhaps we can exclude those so the TOC isn't so long. Similarly, the subsections in Common issues could be excluded if you think it makes sense.
TOC has been condensed.
- I don't know if this is a good idea or not, but the commands within the Common issues steps are formatted with inline code. If the command is static, and is something the user might want to copy/paste, then consider code blocks. If a command is something where the user would need to replace a piece with their own values, then maybe we leave as is.
Initially, I was struggling with making the code blocks line up correctly, but it turned out I just needed two extra tabs :)
Temporary removed configurations to values.yaml as the DevOps team works on an umbrella Helm chart.
melissahenderson
left a comment
There was a problem hiding this comment.
One minor suggestion, take it or leave it 😁
| This guide explains how to deploy Rafiki using Helm charts on a Kubernetes cluster. Helm is a package manager for Kubernetes that allows you to define, install, and upgrade complex Kubernetes applications through Helm charts. | ||
|
|
||
| To run Rafiki in your production environment you must have the following software and tools installed: | ||
| ## Overview of Rafiki architecture |
There was a problem hiding this comment.
I suggest removing this heading. The "On this page" TOC shows Overview, then Overview of Rafiki architecture. The two sections are short enough that I think the automatically inserted "Overview" heading is enough.
Updates to Docker Compose and Helm-Kubernetes deployment pages that include