-
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
[DPE-1935][DPE-1754] Postgresql logs #223
Conversation
Codecov Report
@@ Coverage Diff @@
## main #223 +/- ##
==========================================
+ Coverage 80.64% 80.65% +0.01%
==========================================
Files 9 9
Lines 1855 1856 +1
Branches 303 303
==========================================
+ Hits 1496 1497 +1
Misses 309 309
Partials 50 50
|
* Make sure snap daemon's home exist * Move snap_daemon home creation to on_install * Assert dir creation
@@ -3033,7 +3033,7 @@ | |||
"style": "dark", | |||
"tags": [ | |||
"postgres", | |||
"db", |
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.
Typo
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.
Mostly copied from the spec.
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.
Tnx!
* Postgresql logs * Set published revision. * [DPE-2495] Ensure that snap_daemon home exists (canonical#220) * Make sure snap daemon's home exist * Move snap_daemon home creation to on_install * Assert dir creation * Log rotation config * Update snap * Typo in dashboard * Fix tls tests * Update test_tls.py
Issue
Postgresql's logs are not split into a separate location
Solution
Split the Postgresql logs and add the log rotation configuration from the spec.