Skip to content

Commit

Permalink
handle innobackup.log file
Browse files Browse the repository at this point in the history
  • Loading branch information
nqb committed Jun 30, 2022
1 parent baaf67f commit 7ec9bed
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 1 deletion.
1 change: 1 addition & 0 deletions lib/pf/file_paths.pm
Original file line number Diff line number Diff line change
Expand Up @@ -369,6 +369,7 @@ $connectors_config_file = catdir($conf_dir,"connectors.conf");
mysql-probe.log galera-autofix.log haproxy_portal.log
haproxy_db.log haproxy_admin.log proxysql.log firewall.log
pfconnector-client.log pfconnector-server.log keepalived.log
innobackup.log
);

@stored_config_files = (
Expand Down
3 changes: 2 additions & 1 deletion lib/pf/pfcmd/checkup.pm
Original file line number Diff line number Diff line change
Expand Up @@ -712,10 +712,11 @@ sub permissions {
add_problem( $FATAL, "pfcmd needs setuid and setgid bit set to run properly. Fix with chmod ug+s $bin_dir/pfcmd" );
}

# log owner must be pf otherwise apache or pf daemons won't start
foreach my $log_file (@log_files) {
# if log doesn't exist it is created correctly so no need to complain
next if (!-f $log_file);
# file not managed by rsyslog, permissions are different
next if ($log_file eq "/usr/local/pf/logs/innobackup.log");

add_problem( $FATAL, "$log_file must be owned by group pf. Check rsyslog configuration" )
unless (getgrgid((stat($log_file))[5]) eq 'pf');
Expand Down

0 comments on commit 7ec9bed

Please sign in to comment.