-
Notifications
You must be signed in to change notification settings - Fork 159
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
Address PR Review Feedback: Update mkdocs Navigation and Enhance Memory Settings Doc #4169
Address PR Review Feedback: Update mkdocs Navigation and Enhance Memory Settings Doc #4169
Conversation
…ing memory and OOM troubleshooting. Added a link to the official cgroups v1 documentation. Added instructions for editing `grub.cfg` on the device. Provided details on where to find OOM (Out Of Memory) messages. Fixed minor punctuation issue in the section about EVE services hitting memory limits. Signed-off-by: Nikolay Martyanov <nikolay@zededa.com>
295be18
to
e4dd6ea
Compare
Fixed the Yetus complain 🤦🏼♂️ |
docs/mkdocs/mkdocs.yml
Outdated
@@ -226,3 +226,4 @@ nav: | |||
- UEFI: 'pkg/uefi/rpi/README.md' | |||
- WLAN: 'pkg/wlan/FIXME.md' | |||
- 'Clock Synchronization': 'docs/CLOCK-SYNCHRONIZATION.md' | |||
- 'EVE Memory Settings': 'docs/MEMORY-SETTINGS.md' |
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.
Would it make sense to create section Memory
(like we have Networking
, Security
, etc.) and put there link to this doc + memory-monitor/README.md
+ later psi-*/README.md
?
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.
Yep. Will do.
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.
Done...
Can you remind me how to build the docs?..
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.
@rene will have to remind us both :)
Included 'EVE Memory Settings' in the mkdocs navigation under a new 'Memory' section. Also add there the memory-monitor doc. Signed-off-by: Nikolay Martyanov <nikolay@zededa.com>
e4dd6ea
to
4ea2ada
Compare
Fixes from previous PR review (#4164).
Added missing 'EVE Memory Settings' to mkdocs navigation.
Enhanced MEMORY-SETTINGS.md with additional configuration instructions and OOM troubleshooting tips.
These changes address @milan-zededa's feedback from the previous PR.