← Back to team overview

holland-coredev team mailing list archive

[Bug 1162011] [NEW] failing on estimation is stupid

 

Public bug reported:

holland should not try to estimate the size of a backup and then fail.
estimation may be useful for information purposes, but we should no
longer fail on "oh no i may not have enough space".

Rather, holland should have a watchdog thread that monitors space and
aborts cleanly if we exceed some configurable threshold.  I imagine this
working like pt-stalk's --disk-bytes-free, --disk-pct-free options, with
perhaps somewhat larger defaults.

The actual implementation would periodically look at os.statvfs and then
if we have exceeded the threshold, send a SIGTERM to the process group
(so we can automatically terminate background commands) and then raise
an appropriate BackupError in a signal handler.

** Affects: holland-backup
     Importance: Undecided
         Status: New

-- 
You received this bug notification because you are a member of Holland
Build, which is subscribed to holland-backup.
https://bugs.launchpad.net/bugs/1162011

Title:
  failing on estimation is stupid

Status in Holland Backup Framework:
  New

Bug description:
  holland should not try to estimate the size of a backup and then fail.
  estimation may be useful for information purposes, but we should no
  longer fail on "oh no i may not have enough space".

  Rather, holland should have a watchdog thread that monitors space and
  aborts cleanly if we exceed some configurable threshold.  I imagine
  this working like pt-stalk's --disk-bytes-free, --disk-pct-free
  options, with perhaps somewhat larger defaults.

  The actual implementation would periodically look at os.statvfs and
  then if we have exceeded the threshold, send a SIGTERM to the process
  group (so we can automatically terminate background commands) and then
  raise an appropriate BackupError in a signal handler.

To manage notifications about this bug go to:
https://bugs.launchpad.net/holland-backup/+bug/1162011/+subscriptions


Follow ups

References