[LU-16694] cleanup test-framework.sh, test directory Created: 31/Mar/23 Updated: 11/Dec/23 |
|
| Status: | Reopened |
| Project: | Lustre |
| Component/s: | None |
| Affects Version/s: | None |
| Fix Version/s: | Lustre 2.16.0 |
| Type: | Improvement | Priority: | Minor |
| Reporter: | Tim Day | Assignee: | Tim Day |
| Resolution: | Unresolved | Votes: | 0 |
| Labels: | None | ||
| Issue Links: |
|
||||||||
| Rank (Obsolete): | 9223372036854775807 | ||||||||
| Description |
|
There's some functions in test-framework.sh and scripts in lustre/tests/ that seem to un-needed. This ticket is for tracking a couple cleanup patches. |
| Comments |
| Comment by Gerrit Updater [ 31/Mar/23 ] |
|
"Timothy Day <timday@amazon.com>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/50491 |
| Comment by Andreas Dilger [ 01/Apr/23 ] |
|
One thing that would be nice to get rid of is the Perl implementation of socketclient and socketserver in sanity test_54a. Those are the only commands in the lustre-tests RPM that are using Perl. |
| Comment by Tim Day [ 03/Apr/23 ] |
|
Were you thinking re-implementing those in python? I looked into doing the domain socket test using just shell, but I think I'd need to use netcat or something similar to do that. |
| Comment by Gerrit Updater [ 03/Apr/23 ] |
|
"Timothy Day <timday@amazon.com>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/50513 |
| Comment by Gerrit Updater [ 18/Apr/23 ] |
|
"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/c/fs/lustre-release/+/50513/ |
| Comment by Gerrit Updater [ 22/Apr/23 ] |
|
"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/c/fs/lustre-release/+/50491/ |
| Comment by Peter Jones [ 22/Apr/23 ] |
|
Landed for 2.16 |
| Comment by Tim Day [ 24/Apr/23 ] |
|
There's a couple more patches I have for this. |
| Comment by Gerrit Updater [ 24/Apr/23 ] |
|
"Timothy Day <timday@amazon.com>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/50720 |
| Comment by Gerrit Updater [ 23/May/23 ] |
|
"Timothy Day <timday@amazon.com>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/51097 |
| Comment by Gerrit Updater [ 09/Jun/23 ] |
|
"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/c/fs/lustre-release/+/50720/ |
| Comment by Gerrit Updater [ 20/Jun/23 ] |
|
"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/c/fs/lustre-release/+/51097/ |
| Comment by Gerrit Updater [ 22/Sep/23 ] |
|
"Timothy Day <timday@amazon.com>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/52478 |
| Comment by Gerrit Updater [ 11/Dec/23 ] |
|
"Timothy Day <timday@amazon.com>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/53410 |