mirror of
git://sourceware.org/git/lvm2.git
synced 2024-12-21 13:34:40 +03:00
archiver: fix reporting for check_current_backup
It's getting a bit more complex here. Basic idea behind is - check_current_backup() should not log error when a user is using a read-only filesystem, so e.g. vgscan will not report any error when it tries to take missing backup. We still have cases when error could be reported though, e.g. the backup this would be a symbolic link, but these are rather misconfiguration and unexpected case.
This commit is contained in:
parent
8b16efd17c
commit
748b8158b5
@ -121,19 +121,20 @@ static int _archive(struct volume_group *vg, int compulsory)
|
||||
}
|
||||
|
||||
if (!dm_create_dir(vg->cmd->archive_params->dir)) {
|
||||
/* FIXME: !compulsory logs error here */
|
||||
log_error("Cannot create archiving directory %s.",
|
||||
vg->cmd->archive_params->dir);
|
||||
return compulsory ? 0 : 1;
|
||||
if (compulsory)
|
||||
return_0;
|
||||
return 1;
|
||||
}
|
||||
|
||||
/* Trap a read-only file system */
|
||||
if ((access(vg->cmd->archive_params->dir, R_OK | W_OK | X_OK) == -1) &&
|
||||
(errno == EROFS)) {
|
||||
/* FIXME: !compulsory logs error here */
|
||||
log_error("Cannot archive volume group metadata for %s to read-only filesystem.",
|
||||
vg->name);
|
||||
return compulsory ? 0 : 1;
|
||||
if (compulsory) {
|
||||
log_error("Cannot archive volume group metadata for %s to read-only filesystem.",
|
||||
vg->name);
|
||||
return 0;
|
||||
}
|
||||
return 1;
|
||||
}
|
||||
|
||||
log_verbose("Archiving volume group \"%s\" metadata (seqno %u).", vg->name,
|
||||
@ -249,8 +250,8 @@ int backup_locally(struct volume_group *vg)
|
||||
/* Trap a read-only file system */
|
||||
if ((access(vg->cmd->backup_params->dir, R_OK | W_OK | X_OK) == -1) &&
|
||||
(errno == EROFS)) {
|
||||
log_warn("WARNING: Cannot backup of volume group %s metadata to read-only fs.",
|
||||
vg->name);
|
||||
/* Will take a backup next time when FS is writable */
|
||||
log_debug("Skipping backup of volume group on read-only filesystem.");
|
||||
return 0;
|
||||
}
|
||||
|
||||
@ -493,6 +494,9 @@ int backup_to_file(const char *file, const char *desc, struct volume_group *vg)
|
||||
|
||||
/*
|
||||
* Update backup (and archive) if they're out-of-date or don't exist.
|
||||
*
|
||||
* This function is not supposed to log_error
|
||||
* when the filesystem with archive/backup dir is read-only.
|
||||
*/
|
||||
void check_current_backup(struct volume_group *vg)
|
||||
{
|
||||
|
Loading…
Reference in New Issue
Block a user