[LU-11558] t32_verify_quota() not being called by t32_test()

• Type: Bug
• Status: Open
• Priority: Minor
• Resolution: Unresolved
• Affects Version/s: Lustre 2.11.0, Lustre 2.12.0
• Severity: 3
• Assignee: Wang Shilong
• Reporter: Andreas Dilger

Description

Since patch https://review.whamcloud.com/28020 "LU-3285 test: add Data-on-MDT tests and fixes" landed as commit v2_10_53_0-39-ga7625cd2f3 the call to t32_verify_quota() from conf-sanity.sh t32_test() has been removed, and (I'd guess) that we are no longer verifying that quota is still working correctly after an upgrade.

It is unclear why this check was removed. It needs to be returned back to the code so that we resume testing that quota is working after an upgrade from an old disk format.

It looks like the t32_verify_quota() function itself was broken by patch https://review.whamcloud.com/30087 "LU-7004 tests: move from lctl conf_param to lctl set_param -P" which recently landed, because that is calling "set_persistent_param_and_check $node" when it should be calling "set_persistent_param_and_check $facet" (though t32_verify_quota() is not passed a facet, so I'm not yet sure what needs to be done to fix it).

Issue Links

• is related to: LU-11556 conf-sanity test 32b crashes on MDT umount with “ASSERTION( atomic_read(&d->ld_ref) == 0 ) failed: Refcount is 1”
• is related to: LU-3285 Data on MDT