[LU-7667] conf-sanity test_52 fails due to $TMP/files already existing Created: 14/Jan/16 Updated: 23/May/16 Resolved: 23/May/16 |
|
| Status: | Closed |
| Project: | Lustre |
| Component/s: | None |
| Affects Version/s: | Lustre 2.8.0 |
| Fix Version/s: | Lustre 2.9.0 |
| Type: | Bug | Priority: | Minor |
| Reporter: | James Nunez (Inactive) | Assignee: | James Nunez (Inactive) |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | easy, tests | ||
| Severity: | 3 |
| Rank (Obsolete): | 9223372036854775807 |
| Description |
|
conf-sanity test 52 fails with 'Unable to create directory' Logs are at sanity test 51d writes the output of ‘lfs getstripe’ for all files in $DIR/$tdir to a file $TMP/files and does not remove the file. When conf-sanity runs after sanity, conf-sanity test 52 tries to create a directory at $TMP/files and copy files into it. $TMP/files already exists and, thus, conf-sanity test 52 fails. There are at least two issues here. We should use a unique file or directory name based on the test being run and we should clean up $TMP after using it. |
| Comments |
| Comment by Gerrit Updater [ 02/Feb/16 ] |
|
James Nunez (james.a.nunez@intel.com) uploaded a new patch: http://review.whamcloud.com/18261 |
| Comment by Gerrit Updater [ 21/May/16 ] |
|
Oleg Drokin (oleg.drokin@intel.com) merged in patch http://review.whamcloud.com/18261/ |
| Comment by James Nunez (Inactive) [ 23/May/16 ] |
|
Patch landed to master. |