[LU-12227] Lustre init script does not check if ZFS devices are already mounted

• Status: Resolved
• Project: Lustre
• Affects Version/s: Lustre 2.12.0, Lustre 2.10.7
• Fix Version/s: Lustre 2.13.0
• Type: Bug
• Priority: Minor
• Reporter: Aurelien Degremont (Inactive)
• Assignee: Aurelien Degremont (Inactive)
• Resolution: Fixed
• Severity: 3

Description

Init script lustre installed in /etc/init.d/lustre when called with start argument does check if devices are already started, but it skips this test for ZFS.

When calling service lustre start on a system with some targets already started, the script will start the missing ones but will return a non-zero exit code.

If using systemd, it will see this non-zero exit code as an error and will stop and start all devices.

 

# mount -t lustre -l
ostPOOL0/ost0 on /mnt/lustre/local/lustre-OST0000 type lustre (ro)
ostPOOL0/ost1 on /mnt/lustre/local/lustre-OST0001 type lustre (ro)
ostPOOL0/ost2 on /mnt/lustre/local/lustre-OST0002 type lustre (ro)

# umount /mnt/lustre/local/lustre-OST0001

# service --skip-redirect lustre start; echo $?
Mounting ostPOOL0/ost0 on /mnt/lustre/local/lustre-OST0000
mount.lustre: according to /etc/mtab ostPOOL0/ost0 is already mounted on /mnt/lustre/local/lustre-OST0000
Mounting ostPOOL0/ost1 on /mnt/lustre/local/lustre-OST0001
Mounting ostPOOL0/ost2 on /mnt/lustre/local/lustre-OST0002
mount.lustre: according to /etc/mtab ostPOOL0/ost2 is already mounted on /mnt/lustre/local/lustre-OST0002
2

I will propose a patch to enable mount and label checks for all backend types, including zfs.

With the patch:

# umount /mnt/lustre/local/lustre-OST0001

# service --skip-redirect lustre start; echo $?
lustre-OST0000 is already mounted
lustre-OST0002 is already mounted
Mounting ostPOOL0/ost1 on /mnt/lustre/local/lustre-OST0001
0

Comments

[Gerrit Updater]

Aurelien Degremont ([e-mail]) uploaded a new patch: https://review.whamcloud.com/34766
Subject: LU-12227 scripts: check for mounted ZFS devices too
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 95b36bd524701d54339b9c9a30f5e30624599a44

[Gerrit Updater]

Oleg Drokin ([e-mail]) merged in patch https://review.whamcloud.com/34766/
Subject: LU-12227 scripts: check for mounted ZFS devices too
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: 34bc379f7425f364535e608d03f424a64bd1a04e

[Peter Jones]

Landed for 2.13