-
-
Notifications
You must be signed in to change notification settings - Fork 20
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add a requirement and installation doc for K3s #1624
base: main
Are you sure you want to change the base?
Conversation
@mrjones-plip and @esthermmoturi , |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This image is too small to be useful and I can't tell what it is. If you'd like to keep it, please move it to the new location under production 4.x k8s directory, make it higher resolution and ensure it's applicable for how to install CHT Core on k3s (and not a generic k3s architecture diagram).
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for checking in @ernestoteo ! Nice to see some forward momentum on the ticket \o/
Feedback is:
-
per the parent ticket:
this ticket does not mean adding steps for setting up k3s from scratch
Unfortunately this means much of the content you've written needs to be remove 😢 . Replacing this with an intro sentence or two with something like "Please ensure your k3s cluster is set up and meets our requirements", but being sure to mention any extra configurations that
deploy
script needs or however you plan on pushing CHT Core to the cluster. -
The requirements page was edited in this PR to massively reduce the hardware requirements - I suspect they're the bare minimum requirements of k3s, not of CHT. Please revert to what it is currently.
-
the image you've added needs some fixes - see my other comment
-
Please move your content to be the
_index.md
page inside the 4.x production k8s folder:
@mrjones-plip , are we going to encourage people to use K3s for local/dev hosting or is it just for production hosting? |
For sure no k3s for dev hosting - it's just way too complicated when you don't need most of what it offers. current docs recommend docker helper which is a best practice and has everything you need for app development. Docker is used for smaller production hosting too! |
Thanks @mrjones-plip for clarifying. @ernestoteo , you can incorporate the requested changes. Once the PR is ready you can convert it from 'draft' to 'ready for review'. |
@mrjones-plip, from your comment, I should only document how to install K3s and what its component are ? |
@ernestoteo - nope - do not document how to install k3s. That is well covered in their docs. What we're looking for is the happy path installation for CHT Core on an existing k3s deployment. You can look at the current production docker docs for a good example of what we hope to write for k3s. This might include: What specifics steps do you need to take? Can you use the This means you're going to literally deploy k3s as the prerequisite and then literally deploy the CHT to it. Document the steps, tear it all down and then follow your own docs to make sure they work! I'm happy to jump on a call if you still have more questions on what to do next |
@ernestoteo - there's a good example of what we're looking for with CHT Sync and their steps to deploy to k3s. Possibly reach out to @witash who did a lot of work on k3s & CHT Sync. |
#1602