Skip to content

fix: update log rotation location and frequency #62

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

Merged
merged 2 commits into from
Jul 16, 2021

Conversation

darora
Copy link
Contributor

@darora darora commented Jul 16, 2021

No description provided.

darora added 2 commits July 16, 2021 14:27
This helps avoid situation where postgres logs would fill up the disk
under heavy usage conditions. Technically, it can still happen, but
should be exceedingly rare outside of abusive conditions.
@darora darora requested review from inian and dragarcia July 16, 2021 06:33
@darora darora changed the title fix: update log rotation location fix: update log rotation location and frequency Jul 16, 2021
@@ -1,10 +1,9 @@
/var/lib/postgresql/12/main/pg_log/postgresql.csv {
daily
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

the daily directive is a no-op when used with size, so I'm removing it

/var/lib/postgresql/12/main/pg_log/postgresql.csv {
daily
size 50M
/var/lib/postgresql/data/pg_log/postgresql.log {
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@dragarcia fyi it'll be broken on the pg13 ami w/o this

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

sure thing, let me rebuild and update the images

@darora darora merged commit 979db2d into feature/supabasePostgresV1.0 Jul 16, 2021
@darora darora deleted the da/fix-logrotation branch July 16, 2021 08:01
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