[LU-6913] conf-sanity test_30b defect

• Status: Resolved
• Project: Lustre
• Affects Version/s: Lustre 2.8.0
• Fix Version/s: Lustre 2.8.0
• Type: Bug
• Priority: Minor
• Reporter: Elena Gryaznova
• Assignee: Jian Yu
• Resolution: Fixed
• Epic/Theme: patch, test
• Severity: 3

Description

conf-sanity test_30b takes into account only "fake nid" and does not take into account a failover_nid configured for "--failnode".
This leads test always fails on the setup with ost1failover_HOST != ost1_HOST.
Example of failure :
configuration used :

ost1_HOST=fre812
ost1failover_HOST=fre811

test fails with :

Updated after 2s: wanted '192.168.108.32@tcp' got '192.168.108.32@tcp'
failover_nids: [ 192.168.108.12@tcp, 192.168.108.11@tcp, 192.168.108.32@tcp ]
should have 2 failover nids: 3
 conf-sanity test_30b: @@@@@@ FAIL: Failover nid not added

NIDCOUNT is equal 2 only if ost1failover_HOST is default (equal to ost1_HOST). In this case failover_nids expected to contain OSTNID and NEW :

failover_nids: [ <OSTNID>, <NEW> ]

For case ost1failover_HOST != ost1_HOST NIDCOUNT is equal 3 because failover_nids also contains failover_nid configured for failover node (192.168.108.11):

failover_nids: [ <OSTNID>, <ostfailover nid>, <NEW> ]
failover_nids: [ 192.168.108.12@tcp, 192.168.108.11@tcp, 192.168.108.32@tcp ]'

Comments

[Gerrit Updater]

Elena Gryaznova ([e-mail]) uploaded a new patch: http://review.whamcloud.com/15772
Subject: LU-6913 test: fix conf-sanity test_30b defect
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 126e64940e76cf0608b12d144e0f3b2f6591e6d1

[Gerrit Updater]

Oleg Drokin ([e-mail]) merged in patch http://review.whamcloud.com/15772/
Subject: LU-6913 test: fix conf-sanity test_30b defect
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: 8b0b0892da03dc5de5cd4025b9d31c9217b1e6f1

[Joseph Gmitter (Inactive)]

Landed for 2.8.