[LU-7130] sanity.sh test 162 does not properly handle some DIR values

• Type: Bug
• Resolution: Fixed
• Priority: Minor
• Fix Version/s: Lustre 2.9.0
• Labels: patch
• Severity: 3

Epic

test

• Assignee: WC Triage
• Reporter: parinay v kondekar (Inactive)

Description

• the issue

  [root@localhost tests]# DIR=/mnt/lustre/ ONLY=162 sh sanity.sh
  ...
  
  == sanity test 162: path lookup sanity == 13:34:28 (1441872268)
  fid 0x2000032e1:0x5:0x0 resolves to path d162.sanity/d2/f162.sanity (expected d162.sanity/d2/f162.sanity)
  fid 0x2000032e1:0x12:0x0 resolves to path d162.sanity/d2/p/q/r/slink (expected d162.sanity/d2/p/q/r/slink)
  fid 0x2000032e1:0x13:0x0 resolves to path d162.sanity/d2/p/q/r/slink.wrong (expected d162.sanity/d2/p/q/r/slink.wrong)
  fid 0x2000032e1:0x5:0x0 resolves to path d162.sanity/d2/a/b/c/new_file (expected d162.sanity/d2/a/b/c/new_file)
   sanity test_162: @@@@@@ FAIL: path looked up "/mnt/lustre/d162.sanity/d2/p/q/r/hlink" instead of "/mnt/lustre//d162.sanity/d2/p/q/r/hlink" 
    Trace dump:
    = /usr/lib64/lustre/tests/test-framework.sh:4748:error_noexit()
    = /usr/lib64/lustre/tests/test-framework.sh:4779:error()
    = sanity.sh:10304:err17935()
    = sanity.sh:10693:check_path()
    = sanity.sh:10732:test_162()
    = /usr/lib64/lustre/tests/test-framework.sh:5026:run_one()
    = /usr/lib64/lustre/tests/test-framework.sh:5063:run_one_logged()
    = /usr/lib64/lustre/tests/test-framework.sh:4880:run_test()
    = sanity.sh:10747:main()
  Dumping lctl log to /tmp/test_logs/1441872232/sanity.test_162.*.1441872269.log
  Dumping logs only on local client.
  FAIL 162 (1s)
  
  ...
  == sanity test 162c: fid2path works with paths 100 or more directories deep == 13:34:30 (1441872270)
   sanity test_162c: @@@@@@ FAIL: path looked up "/mnt/lustre/d162c.sanity.local/0" instead of "/mnt/lustre//d162c.sanity.local/0" 
    Trace dump:
    = /usr/lib64/lustre/tests/test-framework.sh:4748:error_noexit()
    = /usr/lib64/lustre/tests/test-framework.sh:4779:error()
    = sanity.sh:10304:err17935()
    = sanity.sh:10693:check_path()
    = sanity.sh:10795:test_162c()
    = /usr/lib64/lustre/tests/test-framework.sh:5026:run_one()
    = /usr/lib64/lustre/tests/test-framework.sh:5063:run_one_logged()
    = /usr/lib64/lustre/tests/test-framework.sh:4886:run_test()
    = sanity.sh:10808:main()
  Dumping lctl log to /tmp/test_logs/1441872232/sanity.test_162c.*.1441872270.log
  Dumping logs only on local client.
  FAIL 162c (0s)
  

• with-patch

  DIR=/mnt/lustre/ ONLY=162 sh sanity.sh | tee /root/Downloads/mrp-334.log
  ...
  == sanity test 162: path lookup sanity == 13:54:57 (1441873497)
  fid 0x200004281:0x5:0x0 resolves to path d162.sanity/d2/f162.sanity (expected d162.sanity/d2/f162.sanity)
  fid 0x200004281:0x12:0x0 resolves to path d162.sanity/d2/p/q/r/slink (expected d162.sanity/d2/p/q/r/slink)
  fid 0x200004281:0x13:0x0 resolves to path d162.sanity/d2/p/q/r/slink.wrong (expected d162.sanity/d2/p/q/r/slink.wrong)
  fid 0x200004281:0x5:0x0 resolves to path d162.sanity/d2/a/b/c/new_file (expected d162.sanity/d2/a/b/c/new_file)
  fid 0x200004281:0x5:0x0 resolves to path /mnt/lustre/d162.sanity/d2/p/q/r/hlink (expected /mnt/lustre/d162.sanity/d2/p/q/r/hlink)
  fid 0x200004281:0x5:0x0 resolves to path d162.sanity/d2/a/b/c/new_file (expected d162.sanity/d2/a/b/c/new_file)
  Resetting fail_loc on all nodes...done.
  PASS 162 (1s)
  
  == sanity test 162c: fid2path works with paths 100 or more directories deep == 13:54:58 (1441873498)
  fid 0x200004281:0x16:0x0 resolves to path /mnt/lustre/d162c.sanity.local/0 (expected /mnt/lustre/d162c.sanity.local/0)
  fid 0x200004281:0x17:0x0 resolves to path /mnt/lustre/d162c.sanity.remote/0 (expected /mnt/lustre/d162c.sanity.remote/0)
  fid 0x200004281:0x18:0x0 resolves to path /mnt/lustre/d162c.sanity.local/0/1 (expected /mnt/lustre/d162c.sanity.local/0/1)
  fid 0x200004281:0x19:0x0 resolves to path /mnt/lustre/d162c.sanity.remote/0/1 (expected /mnt/lustre/d162c.sanity.remote/0/1)
  fid 0x200004281:0x1a:0x0 resolves to path /mnt/lustre/d162c.sanity.local/0/1/2 (expected /mnt/lustre/d162c.sanity.local/0/1/2)
  fid 0x200004281:0x1b:0x0 resolves to path /mnt/lustre/d162c.sanity.remote/0/1/2 (expected /mnt/lustre/d162c.sanity.remote/0/1/2)
  ...
  fid 0x200004281:0xe1:0x0 resolves to path /mnt/lustre/d162c.sanity.remote/0/1/2/3/4/5/6/7/8/9/10/11/12/13/14/15/16/17/18/19/20/21/22/23/24/25/26/27/28/29/30/31/32/33/34/35/36/37/38/39/40/41/42/43/44/45/46/47/48/49/50/51/52/53/54/55/56/57/58/59/60/61/62/63/64/65/66/67/68/69/70/71/72/73/74/75/76/77/78/79/80/81/82/83/84/85/86/87/88/89/90/91/92/93/94/95/96/97/98/99/100/101 (expected /mnt/lustre/d162c.sanity.remote/0/1/2/3/4/5/6/7/8/9/10/11/12/13/14/15/16/17/18/19/20/21/22/23/24/25/26/27/28/29/30/31/32/33/34/35/36/37/38/39/40/41/42/43/44/45/46/47/48/49/50/51/52/53/54/55/56/57/58/59/60/61/62/63/64/65/66/67/68/69/70/71/72/73/74/75/76/77/78/79/80/81/82/83/84/85/86/87/88/89/90/91/92/93/94/95/96/97/98/99/100/101)
  Resetting fail_loc on all nodes...done.
  PASS 162c (11s)
  

Issue Links

• is duplicated by: LU-6614 strip extra / from $DIR in sanity.sh