[LU-3445] Specifying multiple networks in NIDs does no longer work

• Type: Bug
• Resolution: Fixed
• Priority: Major
• Fix Version/s: Lustre 2.4.1, Lustre 2.5.0
• Affects Version/s: Lustre 2.4.0
• Severity: 3
• Assignee: Jian Yu
• Reporter: Oliver Mangold

Description

With older version of Lustre I was used to have IB as default network, but also to have a ethernet connection as fallback, so each NID specified for failover.node or mgs.node contained both networks, e.g.:

failover.node=10.3.0.228@o2ib,192.168.50.128@tcp

When I try this with 2.4.0 it appears the parser does not understand this syntax anymore. I get an error in syslog:

LDISKFS-fs (dm-7): Unrecognized mount option "192.168.50.128@tcp" or missing value

Issue Links

• is duplicated by: LU-3830 mount fails on targets configured with multiple mgs nids

Comments

[Aurelien Degremont (Inactive)]

Thanks Jian. For my own tracking I've just created the ticket: LU-4460.

[Jian Yu]

Thanks Aurelien for pointing this out. I'll look into these issues and figure out whether I should fix the original issue in lmd_parse(). I'll create a new Jira ticket to track the work.

[Aurelien Degremont (Inactive)]

It seems to me this bug is still there for 2 reasons:

-the patch only take care of mkfs/tunefs and so there is still an upgrade issue if mountdata contains something like failover.node=10.3.0.228@o2ib,192.168.50.128@tcp
The way to workaround this looks to be a writeconf, which has side effect. Is there some UPGRADE notes somewhere relative to this?

-the patch seems to modify this kind of string

--failnode=10.3.0.228@o2ib,192.168.50.128@tcp

into

--failnode=10.3.0.228@o2ib --failnode=10.3.0.228@o2ib

Which is not the same. The first example refers to 1 failnode, with 2 NIDS to reach it. The second one refers to 2 different failnodes with 1 NID each.

[Peter Jones]

Landed for 2.4.1 and 2.5

[Bob Glossman (Inactive)]

back port to b2_4: http://review.whamcloud.com/7344

[Oleg Drokin]

this patch cannot be cherrypicked to b2_4 due to a conflict

[Jian Yu]

Hi Oleg,

Could you please cherry-pick the patch to Lustre b2_4 branch? Thanks.

[Jian Yu]

Patch for Lustre master branch: http://review.whamcloud.com/6686