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 made changes -
            Status Original: Resolved [ 5 ] New: Closed [ 6 ]
            yujian Jian Yu made changes -
            Resolution New: Fixed [ 1 ]
            Status Original: Open [ 1 ] New: Resolved [ 5 ]
            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
            adilger Andreas Dilger made changes -
            Labels Original: patch New: MB patch
            adilger Andreas Dilger made changes -
            Priority Original: Major [ 3 ] New: Critical [ 2 ]
            pjones Peter Jones made changes -
            Comment [ Frank

            I think that that capability is limited to internal only (default settings). I'll happily adjust it on your behalf if you let me know what you would like it to be

            Peter ]
            pjones Peter Jones made changes -
            Comment [ Heh. I thought that this was some kind of test to spot the typos but then I realized Andreas beat me to it to correct them :-) ]
            pjones Peter Jones made changes -
            Labels New: patch
            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!

            People

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

              Dates

                Created:
                Updated:
                Resolved: