[LU-5863] Can't mount MGS due to label being less than 8 characters long.

• Status: Resolved
• Project: Lustre
• Affects Version/s: Lustre 2.7.0
• Fix Version/s: Lustre 2.7.0
• Type: Bug
• Priority: Critical
• Reporter: James A Simmons
• Assignee: James Nunez (Inactive)
• Resolution: Fixed
• Labels: HB, patch
• Severity: 3

Description

The patch for LU-5383 fixed a out bound for ldd_svname. It test to see if ldd_svname is at least 8 characters in length but the function in lustre_disk.h server_make_name() sets ldd_svname for the mgs to MGS which is to short. This can prevent the MGS from mounting.

Issue Links

• is duplicated by: LU-5876 MGS volumes with short names do not m...

Comments

[James A Simmons]

Patch to fix this is at http://review.whamcloud.com/#/c/12564. I think it shows up for previously formatted file systems.

[James Nunez (Inactive)]

Thanks for the patch, James.

[Cliff White (Inactive)]

This bug has stopped all testing on the stable filesystem, as the stable filesystem (formatted with older Lustre ) can no longer be mounted.
arg[4] = /dev/mapper/iws11_4
arg[5] = /mnt/mgs
source = /dev/mapper/iws11_4 (/dev/mapper/iws11_4), target = /mnt/mgs
options = rw
checking for existing Lustre data: found
Reading CONFIGS/mountdata
mount.lustre: invalid name 'MGS'

I really don't want to nuke the stable side, so a fix would be appreciated

[Cliff White (Inactive)]

And, I can't make a new clean filesystem, either. Hyperion is down
MGSNID="192.168.120.5@o2ib"
MGSDEV="/dev/mapper/iws10_4"
MDSDEV="/dev/mapper/iws10_1"
mkfs.lustre --reformat --mgs --device-size=$((512 * 1048576)) --fsname lustre $MGSDEV
mkfs.lustre --reformat --mdt --index=0 --mkfsoptions='-i 4096' --mgsnid=$MGSNID --device-size=$((2048 * 1048576)) --fsname lustre $MDSDEV

mkdir /mnt/mgs
mkdir /mnt/mds

mount -t lustre $MGSDEV /mnt/mgs
mount -t lustre $MDSDEV /mnt/mds
~
Fails with same error

[Cliff White (Inactive)]

And i have confirmed that we do NOT test internally with a separate MGS - so this was a complete escape.

[James Nunez (Inactive)]

Landed to master (pre-2.7)

[Andreas Dilger]

The patch fixes the core of the problem, but it doesn't add a test. Clearly, we need a conf-sanity test that formats a separate MGS and mounts it, since we can't possibly have had such a test if this bug slipped through.

[Jian Yu]

Hi James Simmons,
Are you going to add a regression test case into conf-sanity.sh or would you like me to do this?

[James A Simmons]

Could you finish it off. I'm working on a few other tickets.

[Jian Yu]

Sure. I'll do.

[Jian Yu]

Hi Andreas,

Clearly, we need a conf-sanity test that formats a separate MGS and mounts it, since we can't possibly have had such a test if this bug slipped through.

I found that conf-sanity test 21d is a basic test case that starting a separate MGS. However, autotest system always uses the configuration of combined MGT and MDT, and then test 21d has been always skipped. Should we enhance autotest system to add a configuration of separated MGT and MDT, and run a test session with this configuration?
For now and this ticket, I'll just add a new test case that formats $fs2mds_DEV as a separate MGT and mounts it.

[Patrick Farrell (Inactive)]

In case it influences the decision to know what people are doing in the real world, Cray is increasingly using separate MGS and MDS volumes for ease of backup and recovery.
Also for DNE - it's convenient to have MDS volumes all identical.

[James A Simmons]

Same here. ORNL has run separate MGS/MDS servers for years. If I remember right running separate MGS/MDS is required for IR to work properly.

[Gerrit Updater]

Jian Yu ([e-mail]) uploaded a new patch: http://review.whamcloud.com/13391
Subject: LU-5863 tests: add a separate MGS/MDS test case into conf-sanity
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 41231258e22cb35555e17a8b94d80c7bfdc5522d

[Jian Yu]

Thank you Patrick and James for the information. The work to enhance autotest system is tracked in TEI-91. For this ticket, I pushed the above patch to add a test case into conf-sanity.sh to verify separate MGS/MDS. The new test case will not be skipped under autotest combined MGS/MDS configuration.

[Gerrit Updater]

Oleg Drokin ([e-mail]) merged in patch http://review.whamcloud.com/13391/
Subject: LU-5863 tests: add a separate MGS/MDS test case into conf-sanity
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: 589d93f8b01f0a923c0f6ebb64c7ad9935fb05da

[Peter Jones]

Landed for 2.7