[LU-7528] sanity test_160b test_160c tests defect: changelog user is not deregistered on test failure Created: 08/Dec/15 Updated: 06/Dec/16 Resolved: 29/Aug/16 |
|
| Status: | Resolved |
| Project: | Lustre |
| Component/s: | None |
| Affects Version/s: | Lustre 2.5.0 |
| Fix Version/s: | Lustre 2.9.0 |
| Type: | Bug | Priority: | Major |
| Reporter: | Kirtan Shetty (Inactive) | Assignee: | WC Triage |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | patch, test | ||
| Environment: |
single node, 4-node |
||
| Severity: | 3 |
| Rank (Obsolete): | 9223372036854775807 |
| Description |
|
The test 160b and 160c of sanity does not deregister the changelog users that they have registered so if tests which run "changelog_register" fails before "changelog_deregister $USER", then "changelog_deregister $USER" never gets executed and all changes (such as file creation, deletion, etc.) are still recorded. This can be the reason why tests stdouts are huge, they contain "lfs changelog lustre-MDT0000" results. Example : test_160b () {
...
USER=$(do_facet $SINGLEMDS $LCTL --device $MDT0 changelog_register -n)
...
touch $LONGNAME1 || error "create of $LONGNAME1 failed"
...
echo "deregistering $USER"
do_facet $SINGLEMDS $LCTL --device $MDT0 changelog_deregister $USER
...
}
|
| Comments |
| Comment by Gerrit Updater [ 08/Dec/15 ] |
|
kirtan.shetty (kirtan.shetty@seagate.com) uploaded a new patch: http://review.whamcloud.com/17506 |
| Comment by Gerrit Updater [ 29/Aug/16 ] |
|
Oleg Drokin (oleg.drokin@intel.com) merged in patch http://review.whamcloud.com/17506/ |
| Comment by Peter Jones [ 29/Aug/16 ] |
|
Landed for 2.9 |