[LU-12671] replay-vbr accesses non-existent mdd.sync_permission setting Created: 19/Aug/19  Updated: 03/Jan/20  Resolved: 27/Aug/19

Status: Resolved
Project: Lustre
Component/s: None
Affects Version/s: Lustre 2.13.0, Lustre 2.12.3
Fix Version/s: Lustre 2.13.0, Lustre 2.12.4

Type: Bug Priority: Minor
Reporter: Oleg Drokin Assignee: James A Simmons
Resolution: Fixed Votes: 0
Labels: None

Severity: 3
Rank (Obsolete): 9223372036854775807

 Description   

Looking around my logs noticed this:

== replay-vbr test 1b: open (O_CREAT) checks version of parent ======================================= 17:17:44 (1566163064)
Starting client: oleg230-client.virtnet:  -o user_xattr,flock oleg230-server@tcp:/lustre /mnt/lustre2
mount.lustre: according to /etc/mtab oleg230-server@tcp:/lustre is already mounted on /mnt/lustre2
oleg230-server: error: set_param: param_path 'mdd/lustre-MDT0000/sync_permission': No such file or directory
pdsh@oleg230-client: oleg230-server: ssh exited with exit code 2

Some digging around reveals the the setting is actually named

LUSTRE_RW_ATTR(sync_perm);

not sure what's up with the double mount attempt, probably a separate issue.

 Comments   
Comment by Oleg Drokin [ 19/Aug/19 ]

Ok, it was broken by https://review.whamcloud.com/33632

technically this is an api change, but it's been part of 2.12 release already so I am not sure if we want to fix the naming or the scripts.

I'll add the script fixing for now

Comment by Gerrit Updater [ 19/Aug/19 ]

Oleg Drokin (green@whamcloud.com) uploaded a new patch: https://review.whamcloud.com/35826
Subject: LU-12671 tests: Update mdd/sync_permissions to sync_perm
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 102758949e919c510de65002336a334511dfa557

Comment by James A Simmons [ 19/Aug/19 ]

The patch 33632 landed at the start of 2.13 development cycle. Its not in the 2.12 branch. I just looked. We can rename the sysfs file name to its proper name.

Comment by Gerrit Updater [ 21/Aug/19 ]

James Simmons (jsimmons@infradead.org) uploaded a new patch: https://review.whamcloud.com/35851
Subject: LU-12671 tests: Update mdd/sync_permissions to sync_perm
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: bda5b02c381e434311d4407afbc05b9a9817cad5

Comment by Gerrit Updater [ 27/Aug/19 ]

Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/35851/
Subject: LU-12671 mdd: rename mdd/sync_perm to sync_permissions
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: 55a7e2dcecaf482c40840840db2b0b795bad2bb9

Comment by Peter Jones [ 27/Aug/19 ]

Landed for 2.13

Comment by Gerrit Updater [ 13/Dec/19 ]

Minh Diep (mdiep@whamcloud.com) uploaded a new patch: https://review.whamcloud.com/37006
Subject: LU-12671 mdd: rename mdd/sync_perm to sync_permissions
Project: fs/lustre-release
Branch: b2_12
Current Patch Set: 1
Commit: 9619cc063e90851c534b03fc7f31d8fa697f6fd5

Comment by Gerrit Updater [ 03/Jan/20 ]

Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/37006/
Subject: LU-12671 mdd: rename mdd/sync_perm to sync_permissions
Project: fs/lustre-release
Branch: b2_12
Current Patch Set:
Commit: a14908990314e32e69cc898cf974304ec55ecca7

Generated at Sat Feb 10 02:54:38 UTC 2024 using Jira 9.4.14#940014-sha1:734e6822bbf0d45eff9af51f82432957f73aa32c.