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

"lfs mirror delete" should resync file if needed

Details

    • 9223372036854775807

    Description

      If "lfs mirror delete" is called on a file to delete a specific mirror (eg. with "-p <pool>" or "--comp-id <id>", but that is the only non-stale mirror of the file, it makes sense from a usability point of view to automatically resync the stale mirror(s) and then remove the mirror as requested by the user, rather than returning an error to the user.

      Otherwise, the user will first have to understand that the stale mirrors are preventing their request from completing, then manually resync the mirror with "lfs mirror resync" before calling "lfs mirror delete" again. Since "lfs mirror delete" could do the resync directly, why bother the user to have to understand this problem and do the resync manually?

      Attachments

        Issue Links

          is related to

          Bug - A problem which impairs or prevents the functions of the product. LU-5170 lfs usability

          • Minor - Minor loss of function, or other problem where easy workaround is present.
          • Open

          Bug - A problem which impairs or prevents the functions of the product. LU-14512 prohibit extend file with stale mirror

          • Minor - Minor loss of function, or other problem where easy workaround is present.
          • Closed

          Improvement - An improvement or enhancement to an existing feature or task. LU-14554 Improve usability of "lfs mirror" commands

          • Major - Major loss of function.
          • Open
          is related to

          Bug - A problem which impairs or prevents the functions of the product. LU-18468 Md5 chksum fails on the mirror file created by dd with bs=4k after doing mirror resync and delete

          • Blocker - Blocks development and/or testing work, production could not run.
          • Resolved

          Bug - A problem which impairs or prevents the functions of the product. LU-11485 MDS allows "lfs setstripe" to mark last mirror as stale

          • Major - Major loss of function.
          • Resolved

          Improvement - An improvement or enhancement to an existing feature or task. LU-11022 FLR1.5: "lfs mirror" usability for Burst Buffer

          • Minor - Minor loss of function, or other problem where easy workaround is present.
          • Resolved

          Activity

            [LU-13720] "lfs mirror delete" should resync file if needed
            pjones Peter Jones added a comment -

            Merged for 2.17

            pjones Peter Jones added a comment - Merged for 2.17
            gerrit Gerrit Updater added a comment -

            "Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/c/fs/lustre-release/+/55978/
            Subject: LU-13720 lfs: resync file if needed before mirror_split()
            Project: fs/lustre-release
            Branch: master
            Current Patch Set:
            Commit: 81481104fdc8f52b3a36fc659f41a3b696a5bbf3

            gerrit Gerrit Updater added a comment - "Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/c/fs/lustre-release/+/55978/ Subject: LU-13720 lfs: resync file if needed before mirror_split() Project: fs/lustre-release Branch: master Current Patch Set: Commit: 81481104fdc8f52b3a36fc659f41a3b696a5bbf3
            gerrit Gerrit Updater added a comment -

            "Emoly Liu <emoly@whamcloud.com>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/55978
            Subject: LU-13720 lfs: "lfs mirror delete" should resync file if needed
            Project: fs/lustre-release
            Branch: master
            Current Patch Set: 1
            Commit: 74632be00ae7d4dd27b0d08ec48a5803777bf19d

            gerrit Gerrit Updater added a comment - "Emoly Liu <emoly@whamcloud.com>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/55978 Subject: LU-13720 lfs: "lfs mirror delete" should resync file if needed Project: fs/lustre-release Branch: master Current Patch Set: 1 Commit: 74632be00ae7d4dd27b0d08ec48a5803777bf19d

            People

              emoly.liu Emoly Liu
              adilger Andreas Dilger
              Votes:
              0 Vote for this issue
              Watchers:
              4 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: