-
Notifications
You must be signed in to change notification settings - Fork 27
[DPE-8381] Add docs tmpfs warning #1163
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
Conversation
Signed-off-by: Marcelo Henrique Neppel <[email protected]>
Signed-off-by: Marcelo Henrique Neppel <[email protected]>
a-velasco
left a comment
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.
Thank you @marcelo! Made some small suggestions.
One last tiny request: could you remove the leftover " symbol after the word latest on line 7? (another typo that already existed before this PR). For some reason, the GitHub interface doesn't let me comment on that line after expanding the file.
|
Regarding the CI errors:
|
Co-authored-by: Andreia <[email protected]>
Co-authored-by: Andreia <[email protected]>
Signed-off-by: Marcelo Henrique Neppel <[email protected]>
Sure! Thanks for noticing that. I removed that on 0b369be. I'm now updating the Juju anchors. |
Signed-off-by: Marcelo Henrique Neppel <[email protected]>
Thanks! I fixed the Juju anchors on 316720c. |
a-velasco
left a comment
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 @marceloneppel!
Issue
We're fixing the tmpfs mount and using it for the
tempstorage after a reboot on canonical/postgresql-single-kernel-library#11 and #1156.However, there is nothing documented regarding that only the charm supports the tmpfs storage provider only for the
tempstorage (and not for thearchive,data, andlogsstorage).Solution
Add a warning in the documentation.
Checklist