[LU-15887] add always_except() to test-framework Created: 25/May/22 Updated: 29/Oct/22 |
|
| Status: | In Progress |
| Project: | Lustre |
| Component/s: | None |
| Affects Version/s: | None |
| Fix Version/s: | None |
| Type: | Bug | Priority: | Minor |
| Reporter: | John Hammond | Assignee: | John Hammond |
| Resolution: | Unresolved | Votes: | 0 |
| Labels: | None | ||
| Severity: | 3 |
| Rank (Obsolete): | 9223372036854775807 |
| Description |
|
The way we add and remove tests from ALWAYS_EXCEPT is really senseless and encourages merge conflicts and porting mistakes. Instead of always just adding the next test to be excepted to the end of the list it would be better if we excepted groups of tests one issue per line and kept things sorted by test number. Using a function to manage ALWAYS_EXCEPT will also help avoid problems like So instead of a miniature IQ test like: -# bug number: LU-8411 LU-9054 LU-14921 EX-4334 -ALWAYS_EXCEPT+=" 407 312 101f 101k 428" +# bug number: LU-8411 LU-9054 LU-14921 ATM-2415 EX-4334 +ALWAYS_EXCEPT+=" 407 312 101f 101k 400a 428" We could just do: +always_except ATM-2415 400a |
| Comments |
| Comment by Gerrit Updater [ 25/May/22 ] |
|
"John L. Hammond <jhammond@whamcloud.com>" uploaded a new patch: https://review.whamcloud.com/47452 |
| Comment by Gerrit Updater [ 30/May/22 ] |
|
"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/47452/ |
| Comment by Gerrit Updater [ 13/Jun/22 ] |
|
"John L. Hammond <jhammond@whamcloud.com>" uploaded a new patch: https://review.whamcloud.com/47614 |