[LU-10210] conf-sanity test 67 missing path to lustre_routes_conversion Created: 08/Nov/17  Updated: 15/Mar/18  Resolved: 15/Mar/18

Status: Resolved
Project: Lustre
Component/s: None
Affects Version/s: Lustre 2.11.0
Fix Version/s: Lustre 2.11.0

Type: Bug Priority: Minor
Reporter: Oleg Drokin Assignee: Sonia Sharma (Inactive)
Resolution: Fixed Votes: 0
Labels: None

Severity: 3
Rank (Obsolete): 9223372036854775807

 Description   

When running out of build tree conf-sanity test 67 cannot find lustre_routes_conversion script because it's not in PATH.
Need to do something similar to how LUSTRE_RMMOD is implemented.

 Comments   
Comment by Oleg Drokin [ 09/Nov/17 ]

Sonia, it would be great if you take a look at this

Comment by Gerrit Updater [ 01/Dec/17 ]

Sonia Sharma (sonia.sharma@intel.com) uploaded a new patch: https://review.whamcloud.com/30335
Subject: LU-10210 tests: Add lustre_routes_conversion script in PATH
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 50ca834d2db83ddbf2ec18504310a20640c164a8

Comment by Gerrit Updater [ 04/Jan/18 ]

Oleg Drokin (oleg.drokin@intel.com) merged in patch https://review.whamcloud.com/30335/
Subject: LU-10210 tests: Add lustre_routes_conversion script in PATH
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: 144a76262155a6b2f866746c980459729db73544

Comment by Peter Jones [ 04/Jan/18 ]

Landed for 2.11

Comment by Oleg Drokin [ 14/Jan/18 ]

The current patch does not appear to work. Did you test it on a in-tree configuration I wonder?

== conf-sanity test 67: test routes conversion and configuration ===================================== 04:09:49 (1515920989)
conf-sanity.sh: line 4785: /tmp/legacy_lnet_config: Permission denied
 conf-sanity test_67: @@@@@@ FAIL: routes conversion test failed
  Trace dump:
  = ./../tests/test-framework.sh:5335:error()
  = conf-sanity.sh:4800:test_67()
  = ./../tests/test-framework.sh:5611:run_one()
  = ./../tests/test-framework.sh:5650:run_one_logged()
  = ./../tests/test-framework.sh:5497:run_test()
  = conf-sanity.sh:4805:main()
Dumping lctl log to /tmp/test_logs/1515916510/conf-sanity.test_67.*.1515920989.log
Dumping logs only on local client.
Resetting fail_loc on all nodes...done.
FAIL 67 (0s)

I think the problem is here:

        $LUSTRE_ROUTES_CONVERSION $legacy $new > /dev/null

The LUSTRE_ROUTES_CONVERSION variable is empty so it tries to execute the config file giving us this error.

Comment by Sonia Sharma (Inactive) [ 18/Jan/18 ]

Doesn't this test out of build tree configuration?
https://testing.hpdd.intel.com/test_sets/05c7bea6-dbb3-11e7-a066-52540065bddc

Comment by Jian Yu [ 03/Feb/18 ]

There was a typo in the patch: "LUSTRE_ROUTES_COVERSION" should be "LUSTRE_ROUTES_CONVERSION" in test-framework.sh.

Comment by Gerrit Updater [ 05/Feb/18 ]

Sonia Sharma (sonia.sharma@intel.com) uploaded a new patch: https://review.whamcloud.com/31173
Subject: LU-10210 tests: Add lustre_routes_conversion script in PATH
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 46c7c26596b9f8e6e5ae3641f98bd0ba56235866

Comment by Gerrit Updater [ 15/Mar/18 ]

Oleg Drokin (oleg.drokin@intel.com) merged in patch https://review.whamcloud.com/31173/
Subject: LU-10210 tests: Add lustre_routes_conversion script in PATH
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: 9a526209ad030a2c099d564e81a28d9dc0034a72

Comment by Peter Jones [ 15/Mar/18 ]

Landed for 2.11

Generated at Sat Feb 10 02:33:02 UTC 2024 using Jira 9.4.14#940014-sha1:734e6822bbf0d45eff9af51f82432957f73aa32c.