[LU-12280] Provide option to disable soft quota expiration Created: 09/May/19 Updated: 20/Feb/20 Resolved: 20/Feb/20 |
|
| Status: | Resolved |
| Project: | Lustre |
| Component/s: | None |
| Affects Version/s: | None |
| Fix Version/s: | Lustre 2.14.0 |
| Type: | Improvement | Priority: | Minor |
| Reporter: | Cameron Harr | Assignee: | Hongchao Zhang |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | llnl | ||
| Environment: |
Lustre 2.8+ |
||
| Issue Links: |
|
||||||||
| Rank (Obsolete): | 9223372036854775807 | ||||||||
| Description |
|
Currently, when you specify a soft quota, you also set a "grace period" after which the soft quota expires and becomes a hard quota. We wish to have an option to still enable soft quotas but prevent the soft quota from becoming a hard block if the usage is below the hard quota levels. In other words, the user would still be notified the soft quota has been reached, but they wouldn't ever be blocked from writing unless they hit the hard quota limits. I can envision a couple similar approaches to this: you could have an option that allows an inifinite grace period (e.g. lfs setquota -t -1 ...), or you could have an option that prevents the soft quota from ever becoming a hard quota. |
| Comments |
| Comment by Andreas Dilger [ 10/May/19 ] |
|
According to the lfs setquota usage message and lfs-setquota.1 man page (added in commit v2_10_50_0-114-gd2e3bff) the maximum quota grace period is 2^48 - 1 seconds. It probably makes sense to just map -1 to 2^48 - 1 inside lfs_setquota_times() to make this easier to use. That is about 9M years, so probably long enough... The lfs_setquota_times() code checks in that code use UINT_MAX as the maximum value, and internally it is stored in a 64-bit value, so I'm not totally sure where the 2^48 second limit comes from, but in any case this is long enough. |
| Comment by Peter Jones [ 10/May/19 ] |
|
Hongchao Interested in your thoughts on this one Peter |
| Comment by Cameron Harr [ 21/Jun/19 ] |
|
Any further thought on this? |
| Comment by Gerrit Updater [ 01/Sep/19 ] |
|
Hongchao Zhang (hongchao@whamcloud.com) uploaded a new patch: https://review.whamcloud.com/36017 |
| Comment by Gerrit Updater [ 20/Feb/20 ] |
|
Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/36017/ |
| Comment by Peter Jones [ 20/Feb/20 ] |
|
Landed for 2.14 |