Details

    • Bug
    • Resolution: Fixed
    • Minor
    • Lustre 2.8.0
    • Lustre 2.8.0
    • 3
    • 17729

    Description

      This issue was created by maloo for wangdi <di.wang@intel.com>

      This issue relates to the following test suite run: https://testing.hpdd.intel.com/test_sets/9eb20534-c1f5-11e4-9e14-5254006e85c2.

      The sub-test test_17n failed with the following error:

      CMD: shadow-49vm3 /usr/sbin/lctl get_param -n version
      striped dir -i1 -c2 /mnt/lustre/d17n.sanity/remote_dir_0
      total: 10 creates in 0.05 seconds: 216.72 creates/second
      /mnt/lustre/d17n.sanity/remote_dir_0/f4 migrate failed -1
      /mnt/lustre/d17n.sanity/remote_dir_0/f2 migrate failed -1
      /mnt/lustre/d17n.sanity/remote_dir_0/f8 migrate failed -1
      /mnt/lustre/d17n.sanity/remote_dir_0/f6 migrate failed -1
      /mnt/lustre/d17n.sanity/remote_dir_0/f0 migrate failed -1
      mv: cannot migrate '/mnt/lustre/d17n.sanity/remote_dir_0' to MDT0001: Operation not permitted
       sanity test_17n: @@@@@@ FAIL: migrate remote dir error 0 
        Trace dump:
        = /usr/lib64/lustre/tests/test-framework.sh:4675:error_noexit()
        = /usr/lib64/lustre/tests/test-framework.sh:4706:error()
        = /usr/lib64/lustre/tests/sanity.sh:684:test_17n()
        = /usr/lib64/lustre/tests/test-framework.sh:4968:run_one()
        = /usr/lib64/lustre/tests/test-framework.sh:5005:run_one_logged()
        = /usr/lib64/lustre/tests/test-framework.sh:4855:run_test()
        = /usr/lib64/lustre/tests/sanity.sh:695:main()
      Dumping lctl log to /logdir/test_logs/2015-03-03/lustre-reviews-el6_6-x86_64--review-dne-part-1--2_8_1__30753__-69949316400520-144537/sanity.test_17n.*.1425396593.log
      CMD: shadow-49vm3,shadow-49vm4,shadow-49vm5,shadow-49vm6.shadow.whamcloud.com,shadow-49vm7 /usr/sbin/lctl dk > /logdir/test_logs/2015-03-03/lustre-reviews-el6_6-x86_64--review-dne-part-1--2_8_1__30753__-69949316400520-144537/sanity.test_17n.debug_log.\$(hostname -s).1425396593.log;
               dmesg > /logdir/test_logs/2015-03-03/lustre-reviews-el6_6-x86_64--review-dne-part-1--2_8_1__30753__-69949316400520-144537/sanity.test_17n.dmesg.\$(hostname -s).1425396593.log
      migrate remote dir error 0
      

      Info required for matching: sanity 17n

      Attachments

        Issue Links

          Activity

            [LU-6330] sanity test_17n:migrate failed -1
            di.wang Di Wang added a comment -

            Thanks Lai, I will fix the test script.

            di.wang Di Wang added a comment - Thanks Lai, I will fix the test script.
            di.wang Di Wang added a comment -

            Ah, I think we can not migrate striped dir yet. that is why we saw these -1 error.

            di.wang Di Wang added a comment - Ah, I think we can not migrate striped dir yet. that is why we saw these -1 error.
            di.wang Di Wang added a comment -

            Sorry for late response. Ah, hmm, I thought LU-6263 already fixed it. let me check

            di.wang Di Wang added a comment - Sorry for late response. Ah, hmm, I thought LU-6263 already fixed it. let me check
            laisiyao Lai Siyao added a comment - - edited

            This failure can be traced back to http://review.whamcloud.com/#/c/12282/21/lustre/tests/sanity.sh line 679:
            test_mkdir -p $DIR/$tdir/remote_dir_$i

            this directory used to be created by mkdir, but now by test_mkdir, which will create stripe directory by default, while later 17n tries to migrate file under this directory from MDT1 to MDT0, and mdt_reint_migrate_internal() checks whether parent is remote, if not, it will refuse to migrate. IMHO this check is wrong for striped directory, Di, what's your opinion?

            laisiyao Lai Siyao added a comment - - edited This failure can be traced back to http://review.whamcloud.com/#/c/12282/21/lustre/tests/sanity.sh line 679: test_mkdir -p $DIR/$tdir/remote_dir_$i this directory used to be created by mkdir, but now by test_mkdir, which will create stripe directory by default, while later 17n tries to migrate file under this directory from MDT1 to MDT0, and mdt_reint_migrate_internal() checks whether parent is remote, if not, it will refuse to migrate. IMHO this check is wrong for striped directory, Di, what's your opinion?

            People

              laisiyao Lai Siyao
              maloo Maloo
              Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: