Opened 10 years ago

Closed 2 years ago

#4520 closed enhancement (fixed)

Tracker's "Low Space" warning has bizarre criteria.

Reported by: dustin howett Owned by: waddlesplash
Priority: normal Milestone: Unscheduled
Component: Applications/Tracker Version: R1/alpha1
Keywords: Cc:
Blocked By: Blocking:
Has a Patch: no Platform: All

Description

I notice, looking through the code, that Tracker only warns that a volume is running critically low on space if it's got <20 MB free AND that <20MB is <10% of the disk's capacity. I stumbled across this when I was looking at a volume (ext2) with 0.1% free disk space (about 100MB out of 100GB), and I considered this to be a very low disk space condition.

I've attached a (very simple) patch to rectify the situation (make the criteria a hard 9%-or-lower (<10%)), but might not fully understand the reasoning behind the original criteria.

Attachments (1)

tracker_lowspace.patch (568 bytes) - added by dustin howett 10 years ago.
Aforementioned patch.

Download all attachments as: .zip

Change History (4)

Changed 10 years ago by dustin howett

Attachment: tracker_lowspace.patch added

Aforementioned patch.

comment:1 Changed 10 years ago by axeld

It's been some time since I wrote this code, and the files were a lot smaller back then. The reason I limited that to 20 MB was that 20 MB was a lot of space 8 years ago - why should you be warned if you have that much space left? :-)

I am still a bit torn on this issue, though, as a percentage really doesn't make that much sense. If you have a 1 TB partition, it would already warn you when you still have 100 GB left - and that's still a lot of space these days. And that would make this warning pretty much meaningless - it should only warn you if the space is actually tight. 20 MB might not be adequate, anymore, though.

comment:2 Changed 4 years ago by waddlesplash

Milestone: R1Unscheduled
Owner: changed from axeld to waddlesplash
Status: newassigned

comment:3 Changed 2 years ago by waddlesplash

Resolution: fixed
Status: assignedclosed

Fixed in hrev50896. I went with 5%.

Note: See TracTickets for help on using tickets.