Details

    • Bug
    • Resolution: Fixed
    • Blocker
    • Lustre 2.8.0
    • Lustre 2.8.0
    • None
    • zfs, dne
    • 3
    • 9223372036854775807

    Description

      sanityn/91 with few zfs-backed MDTs may fail unexpectedly due to destroy vs. attr_get race.

      Attachments

        Activity

          [LU-7713] sanityn/91 fails with zfs
          pjones Peter Jones added a comment -

          Landed for 2.8

          pjones Peter Jones added a comment - Landed for 2.8

          Oleg Drokin (oleg.drokin@intel.com) merged in patch http://review.whamcloud.com/18155/
          Subject: LU-7713 osd: osd-zfs should serialize destroy vs. others
          Project: fs/lustre-release
          Branch: master
          Current Patch Set:
          Commit: f9ca359284357d145819beb08b316e932f7a3060

          gerrit Gerrit Updater added a comment - Oleg Drokin (oleg.drokin@intel.com) merged in patch http://review.whamcloud.com/18155/ Subject: LU-7713 osd: osd-zfs should serialize destroy vs. others Project: fs/lustre-release Branch: master Current Patch Set: Commit: f9ca359284357d145819beb08b316e932f7a3060

          The last patch was abandoned, the current patch is http://review.whamcloud.com/18155

          adilger Andreas Dilger added a comment - The last patch was abandoned, the current patch is http://review.whamcloud.com/18155
          bzzz Alex Zhuravlev added a comment - please, try this version http://review.whamcloud.com/#/c/18281/

          Alex Zhuravlev (alexey.zhuravlev@intel.com) uploaded a new patch: http://review.whamcloud.com/18281
          Subject: LU-7713 osd: osd-zfs should serialize destroy vs. others
          Project: fs/lustre-release
          Branch: master
          Current Patch Set: 1
          Commit: 0427e3ae7a8d356e17916596e812d140271dc554

          gerrit Gerrit Updater added a comment - Alex Zhuravlev (alexey.zhuravlev@intel.com) uploaded a new patch: http://review.whamcloud.com/18281 Subject: LU-7713 osd: osd-zfs should serialize destroy vs. others Project: fs/lustre-release Branch: master Current Patch Set: 1 Commit: 0427e3ae7a8d356e17916596e812d140271dc554

          My apologies for the spam. The problem occur only if I try to use your patch together with http://review.whamcloud.com/#/c/16838/42. Di will push a update for his patch. I hope this didn't cause unnecessary work.

          heckes Frank Heckes (Inactive) added a comment - My apologies for the spam. The problem occur only if I try to use your patch together with http://review.whamcloud.com/#/c/16838/42 . Di will push a update for his patch. I hope this didn't cause unnecessary work.
          bzzz Alex Zhuravlev added a comment - - edited

          hmm, the patch itself doesn't report this.. I'm gonna try locally.
          update: it applies and builds.. I've refreshed it to address Andreas's comments.

          bzzz Alex Zhuravlev added a comment - - edited hmm, the patch itself doesn't report this.. I'm gonna try locally. update: it applies and builds.. I've refreshed it to address Andreas's comments.

          Hi Alex: We have a problem documented in LU-5565 pointing to this change. Could please upload a rebased version for http://review.whamcloud.com/#/c/18155/
          I tried to build on tip of master (commit bb32a147e35f3339832e41e045de68f304ce18cc), but patches don't merge anymore.
          This might help avoid reformatting soaked FS. Many thanks in advance.

          heckes Frank Heckes (Inactive) added a comment - Hi Alex: We have a problem documented in LU-5565 pointing to this change. Could please upload a rebased version for http://review.whamcloud.com/#/c/18155/ I tried to build on tip of master (commit bb32a147e35f3339832e41e045de68f304ce18cc), but patches don't merge anymore. This might help avoid reformatting soaked FS. Many thanks in advance.

          Assigning to Alex as there is already a patch in flight.

          jgmitter Joseph Gmitter (Inactive) added a comment - Assigning to Alex as there is already a patch in flight.

          Alex Zhuravlev (alexey.zhuravlev@intel.com) uploaded a new patch: http://review.whamcloud.com/18155
          Subject: LU-7713 osd: osd-zfs should serialize destroy vs. others
          Project: fs/lustre-release
          Branch: master
          Current Patch Set: 1
          Commit: c7524fc7a5261f68167ec7258945506e219610dd

          gerrit Gerrit Updater added a comment - Alex Zhuravlev (alexey.zhuravlev@intel.com) uploaded a new patch: http://review.whamcloud.com/18155 Subject: LU-7713 osd: osd-zfs should serialize destroy vs. others Project: fs/lustre-release Branch: master Current Patch Set: 1 Commit: c7524fc7a5261f68167ec7258945506e219610dd

          People

            bzzz Alex Zhuravlev
            bzzz Alex Zhuravlev
            Votes:
            0 Vote for this issue
            Watchers:
            6 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: