Uploaded image for project: 'Lustre'
  1. Lustre
  2. LU-10199

sanity test_56xb: hard links individually migrated (12 != 11)

Details

    • Bug
    • Resolution: Fixed
    • Major
    • Lustre 2.11.0, Lustre 2.10.4
    • Lustre 2.11.0
    • None
    • 3
    • 9223372036854775807

    Description

      This issue was created by maloo for Bob Glossman <bob.glossman@intel.com>

      This issue relates to the following test suite run: https://testing.whamcloud.com/test_sets/740cdf26-c31d-11e7-88ab-52540065bddc.

      The sub-test test_56xb failed with the following error:

      hard links individually migrated (12 != 11)

      strongly suspect this is due to the very recently landed "LU-6051 utils: allow lfs_migrate to handle hard links"
      test 56xb is brand new, only introduced in this patch.
      I'm wondering if this test has ever been passed on any SLES version.

      Info required for matching: sanity 56xb

      Attachments

        Activity

          [LU-10199] sanity test_56xb: hard links individually migrated (12 != 11)
          gerrit Gerrit Updater added a comment -

          John L. Hammond (john.hammond@intel.com) merged in patch https://review.whamcloud.com/31775/
          Subject: LU-10199 utils: strip // from fid2path
          Project: fs/lustre-release
          Branch: b2_10
          Current Patch Set:
          Commit: a2b9a5dabf5a6a66192cc15beb03fba8bde167af

          gerrit Gerrit Updater added a comment - John L. Hammond (john.hammond@intel.com) merged in patch https://review.whamcloud.com/31775/ Subject: LU-10199 utils: strip // from fid2path Project: fs/lustre-release Branch: b2_10 Current Patch Set: Commit: a2b9a5dabf5a6a66192cc15beb03fba8bde167af
          gerrit Gerrit Updater added a comment -

          Bob Glossman (bob.glossman@intel.com) uploaded a new patch: https://review.whamcloud.com/31775
          Subject: LU-10199 utils: strip // from fid2path
          Project: fs/lustre-release
          Branch: b2_10
          Current Patch Set: 1
          Commit: 2fa1e1bf7a7e6f9c60fd130dfcb59b168f9b5de2

          gerrit Gerrit Updater added a comment - Bob Glossman (bob.glossman@intel.com) uploaded a new patch: https://review.whamcloud.com/31775 Subject: LU-10199 utils: strip // from fid2path Project: fs/lustre-release Branch: b2_10 Current Patch Set: 1 Commit: 2fa1e1bf7a7e6f9c60fd130dfcb59b168f9b5de2
          adilger Andreas Dilger added a comment -

          Sorry, I was looking at an old branch that still showed 56xb as being in ALWAYS_EXCEPT.

          adilger Andreas Dilger added a comment - Sorry, I was looking at an old branch that still showed 56xb as being in ALWAYS_EXCEPT.
          bogl Bob Glossman (Inactive) added a comment -

          this test was fixed on master (probably) by https://review.whamcloud.com/30078 and then reenabled by https://review.whamcloud.com/30616.

          bogl Bob Glossman (Inactive) added a comment - this test was fixed on master (probably) by https://review.whamcloud.com/30078 and then reenabled by https://review.whamcloud.com/30616 .
          adilger Andreas Dilger added a comment - - edited

          This problem is not fixed, only the test is in ALWAYS_EXCEPT on master.

          As a general rule, any ticket with the always_except label should not be closed. This label should be removed when the patch that re-enables the test is landed. I've often found, as in this case, that a ticket is marked resolved, but the reason the test is no longer failing is because the test has been disabled.

          adilger Andreas Dilger added a comment - - edited This problem is not fixed, only the test is in ALWAYS_EXCEPT on master. As a general rule, any ticket with the always_except label should not be closed. This label should be removed when the patch that re-enables the test is landed. I've often found, as in this case, that a ticket is marked resolved, but the reason the test is no longer failing is because the test has been disabled.
          bogl Bob Glossman (Inactive) added a comment - - edited

          more seen on b2_10:
          https://testing.hpdd.intel.com/test_sets/62af78a4-2fa8-11e8-b3c6-52540065bddc
          https://testing.hpdd.intel.com/test_sets/9ff156a6-2fa8-11e8-b3c6-52540065bddc
          https://testing.hpdd.intel.com/test_sets/62af78a4-2fa8-11e8-b3c6-52540065bddc
          https://testing.hpdd.intel.com/test_sets/f0f35630-2fda-11e8-b3c6-52540065bddc
          https://testing.hpdd.intel.com/test_sets/f58874a0-306b-11e8-b74b-52540065bddc

          bogl Bob Glossman (Inactive) added a comment - - edited more seen on b2_10: https://testing.hpdd.intel.com/test_sets/62af78a4-2fa8-11e8-b3c6-52540065bddc https://testing.hpdd.intel.com/test_sets/9ff156a6-2fa8-11e8-b3c6-52540065bddc https://testing.hpdd.intel.com/test_sets/62af78a4-2fa8-11e8-b3c6-52540065bddc https://testing.hpdd.intel.com/test_sets/f0f35630-2fda-11e8-b3c6-52540065bddc https://testing.hpdd.intel.com/test_sets/f58874a0-306b-11e8-b74b-52540065bddc
          gerrit Gerrit Updater added a comment -

          Oleg Drokin (oleg.drokin@intel.com) merged in patch https://review.whamcloud.com/30616/
          Subject: LU-10199 tests: Re-enable sanity test 56xb
          Project: fs/lustre-release
          Branch: master
          Current Patch Set:
          Commit: d90d9cd45e977d1672272786cb74d4941b6604a2

          gerrit Gerrit Updater added a comment - Oleg Drokin (oleg.drokin@intel.com) merged in patch https://review.whamcloud.com/30616/ Subject: LU-10199 tests: Re-enable sanity test 56xb Project: fs/lustre-release Branch: master Current Patch Set: Commit: d90d9cd45e977d1672272786cb74d4941b6604a2
          adilger Andreas Dilger added a comment -

          The patch to re-enable this test was not landed, and it is still being skipped.

          adilger Andreas Dilger added a comment - The patch to re-enable this test was not landed, and it is still being skipped.
          gerrit Gerrit Updater added a comment -

          Steve Guminski (stephenx.guminski@intel.com) uploaded a new patch: https://review.whamcloud.com/30616
          Subject: LU-10199 tests: Re-enable sanity test 56xb
          Project: fs/lustre-release
          Branch: master
          Current Patch Set: 1
          Commit: 14c03c1e43f62fa79fc38288834ebc0a5df40cf2

          gerrit Gerrit Updater added a comment - Steve Guminski (stephenx.guminski@intel.com) uploaded a new patch: https://review.whamcloud.com/30616 Subject: LU-10199 tests: Re-enable sanity test 56xb Project: fs/lustre-release Branch: master Current Patch Set: 1 Commit: 14c03c1e43f62fa79fc38288834ebc0a5df40cf2
          pjones Peter Jones added a comment -

          Landed for 2.11

          pjones Peter Jones added a comment - Landed for 2.11

          People

            sguminsx Steve Guminski (Inactive)
            maloo Maloo
            Votes:
            0 Vote for this issue
            Watchers:
            4 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: