Details
-
Improvement
-
Resolution: Fixed
-
Major
-
None
-
9223372036854775807
Description
osp_pre_status=-ENOSPC is used to skip OST from object allocation. The error was set when OST available space is less than 0.1% of total OST size. This value is not configurable. For many cases 0.1% is too low, because append works for existing files. So, procfs files was added for configuration.
Attachments
Issue Links
Activity
Andreas Dilger (andreas.dilger@intel.com) merged in patch http://review.whamcloud.com/17617/
Subject: Revert "LU-6910 osp: add procfs values for OST reserved size"
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: acc918d76856ff14306c543c74e6ceef3865bcbc
Andreas Dilger (andreas.dilger@intel.com) uploaded a new patch: http://review.whamcloud.com/17617
Subject: Revert "LU-6910 osp: add procfs values for OST reserved size"
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 266fd420fb049d23f3c85b99ff56b9f920148c0a
This also fixed a couple of bugs in the handling of nearly full OSTs.
Oleg Drokin (oleg.drokin@intel.com) merged in patch http://review.whamcloud.com/15731/
Subject: LU-6910 osp: add procfs values for OST reserved size
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: 0585b0fb5895a24f07ca32e830d1fa72b75f4f2b
Alexander, could you please resubmit your patch once you have fixed the two test failures (
LU-7550andLU-7552. To run those tests multiple times to get a good confidence about the fix, you should use:To run a bunch of tests and two sets in parallel so it doesn't take so long.
There was also a code late review comment internally that returning -ENOSPC from ->statfs() is probably not the right thing, because this may cause unexpected errors if the caller just wants to know the space usage. Instead, it should set OS_STATE_ENOSPC or OS_STATE_ENOINO in the os_state field and the caller should check that. Having two separate flags also avoids the need to guess what the -ENOSPC error is for.