[LU-203] parallel-scale test_statahead: FAIL: Failed to mount lustre on /tmp/mntpt/lustre0

• Status: Closed
• Project: Lustre
• Affects Version/s: Lustre 2.1.0, Lustre 1.8.6
• Fix Version/s: Lustre 2.1.0, Lustre 1.8.6
• Type: Bug
• Priority: Minor
• Reporter: Jian Yu
• Assignee: Jian Yu
• Resolution: Fixed
• Severity: 3

Description

parallel-scale test_statahead failed as follows:

Mounting 5 lustre clients starts on client-8-ib,client-9-ib 
Starting client client-8-ib,client-9-ib: -o client-16-ib@o2ib:/lustre /tmp/mntpt/lustre0
client-8-ib: Usage: mount -V                 : print version
client-8-ib:        mount -h                 : print this help
client-8-ib:        mount                    : list mounted filesystems
client-8-ib:        mount -l                 : idem, including volume labels

https://maloo.whamcloud.com/test_sets/5e650626-61e5-11e0-a2b4-52540025f9af

This is caused by a bug in the zconf_mount_clients():

    local clients=$1
    local mnt=$2
    local OPTIONS=${3:-$MOUNTOPT}

The above codes should be:

    local clients=$1
    local mnt=$2
    shift 2
    local OPTIONS=${@:-$MOUNTOPT}

Comments

[Jian Yu]

Patch was pushed to Gerrit for review in: http://review.whamcloud.com/428

[Brian Murrell (Inactive)]

Or should the caller be quoting what is sent as argument #3 so that all of the options are given to zconf_mount_clients() in the 3rd argument?

Either solution will work of course. I just think having a finite and predictable number of arguments makes extending the function's arguments easier in the future.

[Jian Yu]

Or should the caller be quoting what is sent as argument #3 so that all of the options are given to zconf_mount_clients() in the 3rd argument?

Yes, this makes sense.

The updated patch was in http://review.whamcloud.com/430.

[Brian Murrell (Inactive)]

Yu Jian,

When you update a patch, be sure to leave the Change-Id: in the commit unchanged so that each revision of the patch does not create a new changeset. That way a changeset has an audit trail that reflects the comments in the related JIRA issue.

[Jian Yu]

When you update a patch, be sure to leave the Change-Id: in the commit unchanged so that each revision of the patch does not create a new changeset. That way a changeset has an audit trail that reflects the comments in the related JIRA issue.

Thanks Brian for the useful instruction. I know how to do next time when I need push a revised patch.

[Build Master (Inactive)]

Integrated in lustre-master » x86_64,server,el5,inkernel #40
LU-203 pass client mount options to zconf_mount_clients() correctly

Oleg Drokin : 5872f585d91d277ffc27e703c1564c361f3a8a23
Files :

• lustre/tests/parallel-scale.sh

[Build Master (Inactive)]

Integrated in lustre-master » x86_64,client,ubuntu1004,inkernel #40
LU-203 pass client mount options to zconf_mount_clients() correctly

Oleg Drokin : 5872f585d91d277ffc27e703c1564c361f3a8a23
Files :

• lustre/tests/parallel-scale.sh

[Build Master (Inactive)]

Integrated in lustre-master » x86_64,client,el5,inkernel #40
LU-203 pass client mount options to zconf_mount_clients() correctly

Oleg Drokin : 5872f585d91d277ffc27e703c1564c361f3a8a23
Files :

• lustre/tests/parallel-scale.sh

[Build Master (Inactive)]

Integrated in lustre-master » i686,client,el6,inkernel #40
LU-203 pass client mount options to zconf_mount_clients() correctly

Oleg Drokin : 5872f585d91d277ffc27e703c1564c361f3a8a23
Files :

• lustre/tests/parallel-scale.sh

[Build Master (Inactive)]

Integrated in lustre-master » x86_64,server,el6,inkernel #40
LU-203 pass client mount options to zconf_mount_clients() correctly

Oleg Drokin : 5872f585d91d277ffc27e703c1564c361f3a8a23
Files :

• lustre/tests/parallel-scale.sh

[Build Master (Inactive)]

Integrated in lustre-master » x86_64,server,el5,ofa #40
LU-203 pass client mount options to zconf_mount_clients() correctly

Oleg Drokin : 5872f585d91d277ffc27e703c1564c361f3a8a23
Files :

• lustre/tests/parallel-scale.sh

[Build Master (Inactive)]

Integrated in lustre-master » x86_64,client,el6,inkernel #40
LU-203 pass client mount options to zconf_mount_clients() correctly

Oleg Drokin : 5872f585d91d277ffc27e703c1564c361f3a8a23
Files :

• lustre/tests/parallel-scale.sh

[Build Master (Inactive)]

Integrated in lustre-master » x86_64,client,ubuntu1004,ofa #40
LU-203 pass client mount options to zconf_mount_clients() correctly

Oleg Drokin : 5872f585d91d277ffc27e703c1564c361f3a8a23
Files :

• lustre/tests/parallel-scale.sh

[Build Master (Inactive)]

Integrated in lustre-master » x86_64,client,el5,ofa #40
LU-203 pass client mount options to zconf_mount_clients() correctly

Oleg Drokin : 5872f585d91d277ffc27e703c1564c361f3a8a23
Files :

• lustre/tests/parallel-scale.sh

[Build Master (Inactive)]

Integrated in lustre-master » i686,server,el5,inkernel #40
LU-203 pass client mount options to zconf_mount_clients() correctly

Oleg Drokin : 5872f585d91d277ffc27e703c1564c361f3a8a23
Files :

• lustre/tests/parallel-scale.sh

[Build Master (Inactive)]

Integrated in lustre-master » i686,client,el5,ofa #40
LU-203 pass client mount options to zconf_mount_clients() correctly

Oleg Drokin : 5872f585d91d277ffc27e703c1564c361f3a8a23
Files :

• lustre/tests/parallel-scale.sh

[Build Master (Inactive)]

Integrated in lustre-master » i686,server,el5,ofa #40
LU-203 pass client mount options to zconf_mount_clients() correctly

Oleg Drokin : 5872f585d91d277ffc27e703c1564c361f3a8a23
Files :

• lustre/tests/parallel-scale.sh

[Build Master (Inactive)]

Integrated in lustre-master » i686,client,el5,inkernel #40
LU-203 pass client mount options to zconf_mount_clients() correctly

Oleg Drokin : 5872f585d91d277ffc27e703c1564c361f3a8a23
Files :

• lustre/tests/parallel-scale.sh

[Jian Yu]

Patch for b1_8: http://review.whamcloud.com/461
It's the same as what was pushed to master branch.

[Build Master (Inactive)]

Integrated in lustre-b1_8 » x86_64,server,el5,inkernel #20
LU-203 pass client mount options to zconf_mount_clients() correctly

Johann Lombardi : d46a1ca866adbd5bf7f33c6b514dba8ff227b636
Files :

• lustre/tests/parallel-scale.sh

[Build Master (Inactive)]

Integrated in lustre-b1_8 » x86_64,client,ubuntu1004,inkernel #20
LU-203 pass client mount options to zconf_mount_clients() correctly

Johann Lombardi : d46a1ca866adbd5bf7f33c6b514dba8ff227b636
Files :

• lustre/tests/parallel-scale.sh

[Build Master (Inactive)]

Integrated in lustre-b1_8 » x86_64,client,el6,inkernel #20
LU-203 pass client mount options to zconf_mount_clients() correctly

Johann Lombardi : d46a1ca866adbd5bf7f33c6b514dba8ff227b636
Files :

• lustre/tests/parallel-scale.sh

[Build Master (Inactive)]

Integrated in lustre-b1_8 » i686,server,el5,ofa #20
LU-203 pass client mount options to zconf_mount_clients() correctly

Johann Lombardi : d46a1ca866adbd5bf7f33c6b514dba8ff227b636
Files :

• lustre/tests/parallel-scale.sh

[Build Master (Inactive)]

Integrated in lustre-b1_8 » x86_64,client,el5,inkernel #20
LU-203 pass client mount options to zconf_mount_clients() correctly

Johann Lombardi : d46a1ca866adbd5bf7f33c6b514dba8ff227b636
Files :

• lustre/tests/parallel-scale.sh

[Build Master (Inactive)]

Integrated in lustre-b1_8 » x86_64,client,el5,ofa #20
LU-203 pass client mount options to zconf_mount_clients() correctly

Johann Lombardi : d46a1ca866adbd5bf7f33c6b514dba8ff227b636
Files :

• lustre/tests/parallel-scale.sh

[Build Master (Inactive)]

Integrated in lustre-b1_8 » i686,client,el6,inkernel #20
LU-203 pass client mount options to zconf_mount_clients() correctly

Johann Lombardi : d46a1ca866adbd5bf7f33c6b514dba8ff227b636
Files :

• lustre/tests/parallel-scale.sh

[Build Master (Inactive)]

Integrated in lustre-b1_8 » i686,client,el5,ofa #20
LU-203 pass client mount options to zconf_mount_clients() correctly

Johann Lombardi : d46a1ca866adbd5bf7f33c6b514dba8ff227b636
Files :

• lustre/tests/parallel-scale.sh

[Build Master (Inactive)]

Integrated in lustre-b1_8 » i686,client,el5,inkernel #20
LU-203 pass client mount options to zconf_mount_clients() correctly

Johann Lombardi : d46a1ca866adbd5bf7f33c6b514dba8ff227b636
Files :

• lustre/tests/parallel-scale.sh

[Build Master (Inactive)]

Integrated in lustre-b1_8 » i686,server,el5,inkernel #20
LU-203 pass client mount options to zconf_mount_clients() correctly

Johann Lombardi : d46a1ca866adbd5bf7f33c6b514dba8ff227b636
Files :

• lustre/tests/parallel-scale.sh

[Build Master (Inactive)]

Integrated in lustre-b1_8 » x86_64,server,el5,ofa #20
LU-203 pass client mount options to zconf_mount_clients() correctly

Johann Lombardi : d46a1ca866adbd5bf7f33c6b514dba8ff227b636
Files :

• lustre/tests/parallel-scale.sh

[Jian Yu]

The patches have been pushed to both b1_8 and master branches in fs/lustre-release repo.