[LU-13478] LNet: peer update adjustment on discovery toggle

• Type: Bug
• Resolution: Fixed
• Priority: Minor
• Fix Version/s: Lustre 2.14.0
• Severity: 3
• Assignee: Amir Shehata (Inactive)
• Reporter: Amir Shehata (Inactive)

Description

Do not delete a non-MR peer when it first connects. We should only delete a peer from our local database iff we've already discovered it and it's state is changing from discovery on to discovery off.

I think it is better to do a push to the peers only when discovery is toggled from on to off. This lets the peers know to clear their representation of the node. When it attempts to connect to it after, it'll get the correct list of NIDs.

However in the reverse case; discovery going from off to on, the PUSH can be sent on an interface which is not recorded on the far side. The push is dropped in this case. By not pushing when we go from off to on we avoid this scenario and allow the peer to rediscover when it commences communication.

Comments

[Peter Jones]

Landed for 2.14

[Gerrit Updater]

Oleg Drokin ([e-mail]) merged in patch https://review.whamcloud.com/38321/
Subject: LU-13478 lnet: handle discovery off properly
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: adae4295b62b1074f5c3c45543c586282394b1be

[Gerrit Updater]

Amir Shehata ([e-mail]) uploaded a new patch: https://review.whamcloud.com/38321
Subject: LU-13478 lnet: handle discovery off properly
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 0aba7078644af2f8379a9dee58f4c8827263683c