Opened 12 years ago

Closed 12 years ago

#19 closed defect (fixed)

bbackupd artificial block check prevents setting low or zero soft-limit

Reported by: Gary Niemcewicz Owned by: chris
Priority: normal Milestone: 0.11
Component: bbackupd Version: 0.10
Keywords: soft-limit hard-limit threshold exceeded storage soft hard limit Cc:

Description

The "threshold for uploading new stuff" bbackupd artificial block check requires bbstored to maintain at least soft-limit plus 1/3 of soft/hard limit delta to attempt a backup, regardless of the (potentially smaller) new backup size. If the soft limit is set low (to retain few old/deleted blocks) or set to zero (to discard all old/deleted blocks on every housekeeping run), this check causes bbackupd to abandon a backup, even though there might be plenty of free blocks on a server. Attached patch changes the check in question to investigate a hard limit only.

Attachments (2)

bbackupd-soft-limit-check.patch (1.1 KB) - added by Gary Niemcewicz 12 years ago.
bbackupd-soft-limit-check.2.patch (1.1 KB) - added by Gary Niemcewicz 12 years ago.

Download all attachments as: .zip

Change History (7)

Changed 12 years ago by Gary Niemcewicz

Changed 12 years ago by Gary Niemcewicz

comment:2 Changed 12 years ago by chris

(In [1659]) Apply Gary's patch from ticket #19 to respect the server's hard limit rather than the soft limit. This allows setting the soft limit to zero, so that housekeeping will remove all old versions of all files (refs #19)

comment:3 Changed 12 years ago by chris

(In [1663]) New win32 native build (1662) with the following fixes and new features:

  • Log progress of large file backups at trace level ([1589], [1590])
  • Notify (log) during full file uploads ([1594])
  • Improve error messages in bbackupquery when comparing or getting a file fails with an exception ([1597], [1600])
  • Improve error reporting in bbackupquery ([1638])
  • Initialise logging framework and use for banner in bbackupquery ([1601], [1602])
  • Include LICENSE.txt in binary package, but not Nick's old ReadMe?.txt ([1655])
  • Save unused locations across bbackupd restarts, so that they will be deleted when their time is up, thanks to Gary ([1658], refs #18)
  • Respect server hard limits rather than soft limits, to allow use of zero as a soft limit, thanks to Gary ([1659], [1662], refs #19)
  • Fix restore to a directory under the root, with or without drive letter (c:\test\ or \test\), thanks to Mikael Syska ([1660], [1661])

comment:4 Changed 12 years ago by chris

(In [1754]) Apply Gary's patch from ticket #19 to respect the server's hard limit rather than the soft limit. This allows setting the soft limit to zero, so that housekeeping will remove all old versions of all files. (refs #19, refs #3, merges [1659])

comment:5 Changed 12 years ago by chris

Resolution: fixed
Status: assignedclosed

Fixed by Gary's patch, thanks!

Note: See TracTickets for help on using tickets.