[LU-8912] nodemap: error: nodemap_add_range: nodemap ranges must be contiguous

• Type: Bug
• Resolution: Fixed
• Priority: Minor
• Fix Version/s: Lustre 2.11.0, Lustre 2.10.4
• Affects Version/s: Lustre 2.9.0
• Environment: CentOS 7.2, Lustre-2.8.60
• Severity: 3
• Assignee: Kit Westneat (Inactive)
• Reporter: Stephane Thiell

Description

Hi, we noticed the following minor issue when trying to add our IB network 10.210.32.0/20 to nodemap:

$ lctl nodemap_add_range --name sherlock --range 10.210.[32-47].[0-255]@o2ib3
error: nodemap_add_range: nodemap ranges must be contiguous
$ cat /proc/fs/lustre/nodemap/sherlock/ranges
[

]

The workaround is either to add 16 separate entries (10.210.32.[0-255], 10.210.33.[0-255] etc.), or a single larger range (whenever possible).

Thanks,
Stephan

Comments

[Gerrit Updater]

John L. Hammond ([e-mail]) merged in patch https://review.whamcloud.com/31521/
Subject: LU-8912 nodemap: fix contiguous range support
Project: fs/lustre-release
Branch: b2_10
Current Patch Set:
Commit: 92d562457b947f02891e7c76b265bde4f9ed23c2

[Gerrit Updater]

Minh Diep ([e-mail]) uploaded a new patch: https://review.whamcloud.com/31521
Subject: LU-8912 nodemap: fix contiguous range support
Project: fs/lustre-release
Branch: b2_10
Current Patch Set: 1
Commit: 1f9c0431a29e2f0a8cac90eef9ae0dfa18b02cae

[Peter Jones]

Landed for 2.11

[Gerrit Updater]

Oleg Drokin ([e-mail]) merged in patch https://review.whamcloud.com/24397/
Subject: LU-8912 nodemap: fix contiguous range support
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: eac95a6587af4061b36b35051e191ca27bc6b1a3

[Minh Diep]

kit.westneat, please rebase the patch manually. thanks

[Gerrit Updater]

Kit Westneat ([e-mail]) uploaded a new patch: https://review.whamcloud.com/24397
Subject: LU-8912 nodemap: fix contiguous range support
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: f9bf47f8590cdd77a7b896eea7e83e91782f04bd

[Kit Westneat (Inactive)]

I tend to agree with you, but on our nodemap call, Intel voiced a strong preference for supporting the full LNET range expression support. I've prepared a bug fix patch to address your immediate issue.

The idea behind the full LNET range support is that instead of requiring non-contiguous ranges, if a non-contiguous range is given to lctl, multiple contiguous ranges will be created in the backend. I'll create an improvement ticket to track the work on that.