Skip to content

[WIP] patch for discussion #174 #175

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

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.

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants