Skip to content

checkDiskSpace: df по пути вместо Storage::getAllHdd — фикс для syste…#55

Merged
boffart merged 1 commit into
masterfrom
develop
Apr 24, 2026
Merged

checkDiskSpace: df по пути вместо Storage::getAllHdd — фикс для syste…#55
boffart merged 1 commit into
masterfrom
develop

Conversation

@boffart

@boffart boffart commented Apr 24, 2026

Copy link
Copy Markdown
Contributor

…m-disk

На system-disk инсталляциях (один физический vda с разделами vda2=/offload, vda3=/cf, vda4=/storage/usbdisk1) Storage::getAllHdd возвращает диск с mounted=/offload (первый раздел), а цикл искал запись, у которой mounted начинается на /storage/usbdisk — не находил ничего, freeSpace оставался 0, и бекап всегда падал с 'not enough free disk space', хотя места много.

  • getFreeSpaceOnMountPoint: статический, возвращает int, использует df -m -P (резолвит подкаталог в точку монтирования) с timeout 3 для защиты от зависания на stale FUSE/FTP/SFTP маунте.
  • checkDiskSpace: удалены ветки remote/Docker/host — все три вычисляли одно и то же, но только remote работал корректно. Убран больше не нужный getDockerHdd.
  • unpackUploadedImgConf: такая же проверка (Storage::getAllHdd + цикл) дедуплицирована через тот же хелпер — баг system-disk был и там.

…m-disk

На system-disk инсталляциях (один физический vda с разделами vda2=/offload,
vda3=/cf, vda4=/storage/usbdisk1) Storage::getAllHdd возвращает диск с
mounted=/offload (первый раздел), а цикл искал запись, у которой mounted
начинается на /storage/usbdisk — не находил ничего, freeSpace оставался 0,
и бекап всегда падал с 'not enough free disk space', хотя места много.

- getFreeSpaceOnMountPoint: статический, возвращает int, использует
  df -m -P <path> (резолвит подкаталог в точку монтирования) с timeout 3
  для защиты от зависания на stale FUSE/FTP/SFTP маунте.
- checkDiskSpace: удалены ветки remote/Docker/host — все три вычисляли
  одно и то же, но только remote работал корректно. Убран больше не
  нужный getDockerHdd.
- unpackUploadedImgConf: такая же проверка (Storage::getAllHdd + цикл)
  дедуплицирована через тот же хелпер — баг system-disk был и там.
@boffart boffart merged commit 55583d3 into master Apr 24, 2026
1 of 3 checks passed
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.

1 participant