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

lfs migrate sends file to stdout

Details

    • Bug
    • Resolution: Fixed
    • Critical
    • Lustre 2.7.0
    • Lustre 2.7.0
    • 3
    • 16439

    Description

      The following test will not migrate the files, but sends them to stdout (eg. the display).

      ./llmount.sh
cd /mnt/lustre
cp /bin/ls .
 ~/lustre-cleanup/lustre/utils/lfs migrate -o 0 ls

      The problem is in llapi_file_open_param(), where rc is set to 1 by llapi_search_ost(), and then never reset.

      Patch will follow.

      (I can't believe that didn't break any test!)

      Attachments

        Issue Links

          is related to

          New Feature - A new feature of the product, which has yet to be developed. LU-4665 utils: lfs setstripe to specify OSTs

          • Critical - Crashes, loss of data, severe memory leak.
          • Resolved

          Activity

            [LU-5878] lfs migrate sends file to stdout
            yujian Jian Yu added a comment -

            Patch was landed on master branch.

            yujian Jian Yu added a comment - Patch was landed on master branch.
            gerrit Gerrit Updater added a comment -

            Oleg Drokin (oleg.drokin@intel.com) merged in patch http://review.whamcloud.com/12601/
            Subject: LU-5878 lfs: migrate file to its proper destination
            Project: fs/lustre-release
            Branch: master
            Current Patch Set:
            Commit: 01bc529ccffdb21d33ab6a36ed64fe96ef9ac55b

            gerrit Gerrit Updater added a comment - Oleg Drokin (oleg.drokin@intel.com) merged in patch http://review.whamcloud.com/12601/ Subject: LU-5878 lfs: migrate file to its proper destination Project: fs/lustre-release Branch: master Current Patch Set: Commit: 01bc529ccffdb21d33ab6a36ed64fe96ef9ac55b
            jlevi Jodi Levi (Inactive) added a comment -

            Yu Jian,
            Would you be able to keep an eye on this patch?
            Thank you!

            jlevi Jodi Levi (Inactive) added a comment - Yu Jian, Would you be able to keep an eye on this patch? Thank you!
            adilger Andreas Dilger added a comment -

            The llapi_file_open_param() function was added in patch http://review.whamcloud.com/9383

            adilger Andreas Dilger added a comment - The llapi_file_open_param() function was added in patch http://review.whamcloud.com/9383
            fzago Frank Zago (Inactive) added a comment -

            http://review.whamcloud.com/12601

            fzago Frank Zago (Inactive) added a comment - http://review.whamcloud.com/12601

            People

              yujian Jian Yu
              fzago Frank Zago (Inactive)
              Votes:
              0 Vote for this issue
              Watchers:
              7 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: