Skip to content
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

Documentation #193

Draft
wants to merge 4 commits into
base: main
Choose a base branch
from
Draft

Conversation

imane-ess
Copy link

These commits contain a few additions to the documentation that are intended to further clarify Rancher-Desktop's functioning to new users. A glossary is included, as well as diagrams of the functional architecture and networking layer.

@Nino-K
Copy link
Member

Nino-K commented Apr 18, 2023

@imane-ess I approved your PR to run in CI as per our conversation on Slack, I noticed it's in draft mode, is there still more to do? If so, ping us when it's all ready for review.

@imane-ess imane-ess marked this pull request as ready for review April 21, 2023 01:11
@imane-ess
Copy link
Author

@imane-ess I approved your PR to run in CI as per our conversation on Slack, I noticed it's in draft mode, is there still more to do? If so, ping us when it's all ready for review.

Thank you. I have marked the PR as ready for review.

@sunilarjun sunilarjun self-assigned this Apr 21, 2023
@sunilarjun
Copy link
Contributor

Hello @imane-ess! Would you please update your branch to grab the latest version of the docs and commit these changes against the latest? I'll continue reviewing once the update has been made. Thanks for your work!

@sunilarjun
Copy link
Contributor

Hello again @imane-ess! After further review, could you please split this PR into two PR's: one that can be made against the Rancher Desktop docs repository that includes the glossary changes, and another made against the development documentation as a Rancher Desktop application PR which includes the diagrams.

We want to split into two PR's as the diagrams could be beneficial to a development audience in understanding the application's design in more detail, and we currently have a user oriented design schema on the architecture page with overlap with your diagram. With having a developer audience in mind, could you please update the diagrams to contain a bit more detail for each layer (for example expanding on k3s/Traefik components, perhaps how the ingress controller is handled, how data volume is handled).

Thanks again for your work and please don't hesitate to reach out with questions.

@gaktive gaktive marked this pull request as draft June 5, 2023 18:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants