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

[WIP] patch for discussion #174 #175

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Conversation

mikecaat
Copy link
Contributor

@mikecaat mikecaat commented May 14, 2021

This is the patch for #174 discussion. We need to discuss how to handle the issue.

mikecaat added 3 commits May 14, 2021 13:27
Although "log_directory" parameter's default value is changed in
PostgreSQL version 10, pg_rman didn't change it.

It leads following main issues if the "log_directory" is not specified.

1. Server logs are always backuped even if "--with-serverlog" is not
specified because pg_rman fails to exclude the "PGDATA/log" directory
from backup target lists.

2. pg_rman's "--keep-srvlog-files" and "--keep-srvlog-days" options are
invalid because SRVLOG_PATH is wrong.
pg_rman supports to backup server logs with "--with-serverlog" option
and backups them to the BACKUP_PATH directory.

But, it didn't support to restore the backuped server logs. So, this
patch supports to restore them from full backup and incremental backup.
@mikecaat mikecaat changed the title [WIP] related to #174 [WIP] patch for discussion #174 May 14, 2021
@MoonInsung
Copy link
Contributor

Dear @mikecaat .

Thank you for responding to this issue.
I noticed a problem with the backup and restoration of server log files,
and it seems to be a critical issue depending on the user who uses it.
First, I think the problem occurred because I didn't check that the default folder name was changed by the PostgreSQL 10.
So, I need to think a little more about dealing with the restore and backup to the server log.
So please wait a little bit. Sorry.
##I want to be processed before May 20th.

Best regards.
Moon.

@mikecaat
Copy link
Contributor Author

Dear @MoonInsung

Thanks for researching!

BTW, I think it's better to discuss this issue on the issue page, not on this page because some users may check only issue pages.

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.

2 participants