From bfb5df51378caffd8a57ea83125dc3d996f5c5d7 Mon Sep 17 00:00:00 2001 From: Sangyoon Jang Date: Fri, 18 Jun 2021 20:11:12 +0900 Subject: [PATCH] Fix update failure of rpk package The rpk package doesn't contain shared directory, we don't need to check free space for backup this directory. Change-Id: Ie9cd4b1743487b4ac240891bb6a56645f3941903 Signed-off-by: Sangyoon Jang --- src/common/step/backup/step_copy_backup.cc | 8 +++++++- 1 file changed, 7 insertions(+), 1 deletion(-) diff --git a/src/common/step/backup/step_copy_backup.cc b/src/common/step/backup/step_copy_backup.cc index 8a7c11d..73f8f8c 100644 --- a/src/common/step/backup/step_copy_backup.cc +++ b/src/common/step/backup/step_copy_backup.cc @@ -70,7 +70,13 @@ Step::Status StepCopyBackup::precheck() { install_path_ = context_->GetPkgPath(); backup_path_ = GetBackupPathForPackagePath(context_->GetPkgPath()); - if (!CheckFreeSpace(backup_path_, install_path_ / "shared")) { + // We only "copy" shared dir for backup. + // So if there is no shared dir, we don't need to check free space. + bf::path shared_dir = install_path_ / "shared"; + if (!bf::exists(shared_dir)) + return Status::OK; + + if (!CheckFreeSpace(backup_path_, shared_dir)) { LOG(ERROR) << "not enough space for backup"; return Step::Status::OUT_OF_SPACE; } -- 2.7.4