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

Patch LU-11077 breaks single node testing

Details

    • Bug
    • Resolution: Duplicate
    • Minor
    • None
    • Lustre 2.17.0
    • None
    • Testing out of the source tree with Lustre not installed onto the testing node.
    • 3
    • 9223372036854775807

    Description

      Testing from the source tree is now broken.  This is due to mount.lustre not being  a binary in /sbin and its linked to liblustreapi.so which in the case of testing  in the source tree only doesn't work since its not installed.

      Attachments

        Issue Links

          duplicates

          Improvement - An improvement or enhancement to an existing feature or task. LU-11077 Client-specific tunable parameter configuration

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

          Activity

            [LU-18491] Patch LU-11077 breaks single node testing
            adilger Andreas Dilger added a comment -

            Alex or James, is there a way to statically link liblustreapi.a to the mount.lustre binary? I would think that --disable-shared would statically link all libraries, which isn't necessary, only the ones that are not installed when running from the build tree.

            Alternately, if there was a way to detect that this build was not part of RPM or Deb packaging...

            adilger Andreas Dilger added a comment - Alex or James, is there a way to statically link liblustreapi.a to the mount.lustre binary? I would think that --disable-shared would statically link all libraries, which isn't necessary, only the ones that are not installed when running from the build tree. Alternately, if there was a way to detect that this build was not part of RPM or Deb packaging...
            bzzz Alex Zhuravlev added a comment -

            I use --disable-shared to configure to work this problem around.

            bzzz Alex Zhuravlev added a comment - I use --disable-shared to configure to work this problem around.
            simmonsja James A Simmons added a comment -

            Oh I thought it landed. Sorry I was on vacation so I had the wrong impression. 

            simmonsja James A Simmons added a comment - Oh I thought it landed. Sorry I was on vacation so I had the wrong impression. 
            adilger Andreas Dilger added a comment -

            Are you talking about https://review.whamcloud.com/55414 ("LU-11077 utils: mount with client-only params")? That patch hasn't landed yet, and you've marked it -1 so opening a ticket on it doesn't really help.

            adilger Andreas Dilger added a comment - Are you talking about https://review.whamcloud.com/55414 (" LU-11077 utils: mount with client-only params ")? That patch hasn't landed yet, and you've marked it -1 so opening a ticket on it doesn't really help.
            pjones Peter Jones added a comment -

            The patch you are referring to is in master-next but not landed. Your -1 on the patch in Gerrit will suffice to prevent this merging.

            pjones Peter Jones added a comment - The patch you are referring to is in master-next but not landed. Your -1 on the patch in Gerrit will suffice to prevent this merging.

            People

              fdilger Fred Dilger
              simmonsja James A Simmons
              Votes:
              0 Vote for this issue
              Watchers:
              4 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: