[LU-12967] sanity test 80 silently fails to get sync_on_lock_cancel parameter

• Type: Bug
• Resolution: Fixed
• Priority: Minor
• Fix Version/s: Lustre 2.14.0, Lustre 2.12.4
• Affects Version/s: Lustre 2.12.0, Lustre 2.13.0, Lustre 2.14.0
• Severity: 3
• Assignee: James Nunez (Inactive)
• Reporter: James Nunez (Inactive)

Description

There are actually two issues with sanity test_80 as see here in the suite_log to https://testing.whamcloud.com/test_sets/92b52b08-05c5-11ea-9487-52540065bddc

== sanity test 80: Page eviction is equally fast at high offsets too  ================================ 19:09:39 (1573585779)
CMD: trevis-34vm3 lctl get_param -n obdfilter.*.sync_on_lock_cancel
trevis-34vm3: error: get_param: param_path 'obdfilter/*/sync_on_lock_cancel': No such file or directory
CMD: trevis-34vm3trevis-34vm3trevis-34vm3trevis-34vm3trevis-34vm3trevis-34vm3trevis-34vm3trevis-34vm3 lctl set_param obdfilter.*.sync_on_lock_cancel=never
pdsh@trevis-34vm1: gethostbyname("trevis-34vm3trevis-34vm3trevis-34vm3trevis-34vm3trevis-34vm3trevis-34vm3trevis-34vm3trevis-34vm3") failed
1+0 records in
1+0 records out
1048576 bytes (1.0 MB) copied, 0.00475709 s, 220 MB/s
CMD: trevis-34vm3trevis-34vm3trevis-34vm3trevis-34vm3trevis-34vm3trevis-34vm3trevis-34vm3trevis-34vm3 lctl set_param obdfilter.*.sync_on_lock_cancel=
pdsh@trevis-34vm1: gethostbyname("trevis-34vm3trevis-34vm3trevis-34vm3trevis-34vm3trevis-34vm3trevis-34vm3trevis-34vm3trevis-34vm3") failed

One issue is that it looks like the obdfilter.*.sync_on_lock_cancel parameter no longer exists and the host name composed in the test is problematic.

The test does not fail when it hits these issues, but most likely, the test is not working as intended.

Issue Links

• is related to: LU-15106 deprecated obdfilter parameter warn on read
• is related to: LU-10496 sanity-dom sanity test_39k: FAIL: mtime is lost on close: 1515705427, should be 1484169395
• is related to: LU-8066 Move lustre procfs handling to sysfs and debugfs.

Comments

[Gerrit Updater]

Oleg Drokin ([e-mail]) merged in patch https://review.whamcloud.com/37038/
Subject: LU-12967 tgt: clean up sync_on_cancel references
Project: fs/lustre-release
Branch: b2_12
Current Patch Set:
Commit: 1e43d3c4fc8432bebacfa7a8a320163e3b15448d

[Gerrit Updater]

Oleg Drokin ([e-mail]) merged in patch https://review.whamcloud.com/37037/
Subject: LU-12967 ofd: restore sync_on_lock_cancel tunable
Project: fs/lustre-release
Branch: b2_12
Current Patch Set:
Commit: cdffffb73080ad1100549afbcdbb09d3ee7a1c50

[Gerrit Updater]

Minh Diep ([e-mail]) uploaded a new patch: https://review.whamcloud.com/37038
Subject: LU-12967 tgt: clean up sync_on_cancel references
Project: fs/lustre-release
Branch: b2_12
Current Patch Set: 1
Commit: 894d051d1161b6a2462fdac59df2ed4668315854

[Gerrit Updater]

Minh Diep ([e-mail]) uploaded a new patch: https://review.whamcloud.com/37037
Subject: LU-12967 ofd: restore sync_on_lock_cancel tunable
Project: fs/lustre-release
Branch: b2_12
Current Patch Set: 1
Commit: 7e3ac15524e1178674e0c3e50945215b82883b0f

[Peter Jones]

Landed for 2.14

[Gerrit Updater]

Oleg Drokin ([e-mail]) merged in patch https://review.whamcloud.com/36754/
Subject: LU-12967 tgt: clean up sync_on_cancel references
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: 52a5981be4df863088168b3ea41fac9e29ddf060

[Gerrit Updater]

Oleg Drokin ([e-mail]) merged in patch https://review.whamcloud.com/36748/
Subject: LU-12967 ofd: restore sync_on_lock_cancel tunable
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: 7df7347b7b188e7168e094304fd6d2d985f7f274