[LU-12671] replay-vbr accesses non-existent mdd.sync_permission setting

• Type: Bug
• Resolution: Fixed
• Priority: Minor
• Fix Version/s: Lustre 2.13.0, Lustre 2.12.4
• Affects Version/s: Lustre 2.13.0, Lustre 2.12.3
• Severity: 3
• Assignee: James A Simmons
• Reporter: Oleg Drokin

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

[Gerrit Updater]

Oleg Drokin ([e-mail]) 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

[Gerrit Updater]

Minh Diep ([e-mail]) 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

[Peter Jones]

Landed for 2.13

[Gerrit Updater]

Oleg Drokin ([e-mail]) 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

[Gerrit Updater]

James Simmons ([e-mail]) 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

[James A Simmons]

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.

[Gerrit Updater]

Oleg Drokin ([e-mail]) 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

[Oleg Drokin]

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