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

build: Re-add the lustre-devel package.

Details

    • Bug
    • Resolution: Fixed
    • Major
    • Lustre 2.14.0
    • None
    • None
    • 3
    • 9223372036854775807

    Description

      This re-adds the lustre-devel package.

      Attachments

        Issue Links

          Activity

            [LU-9215] build: Re-add the lustre-devel package.

            Andreas I can do that. The only reason I brought this up is that people earlier in this ticket asked for it. Also patch https://review.whamcloud.com/#/c/25997 seemed to be heading into that direction instead of handling the user land devel package handling. Cory those *.pc inspired me to develop the patch that just landed 

            simmonsja James A Simmons added a comment - Andreas I can do that. The only reason I brought this up is that people earlier in this ticket asked for it. Also patch https://review.whamcloud.com/#/c/25997  seemed to be heading into that direction instead of handling the user land devel package handling. Cory those *.pc inspired me to develop the patch that just landed 
            spitzcor Cory Spitz added a comment - - edited simmonsja , FYI: https://github.com/Cray/lustre/blob/cray-2.12-int/cray-obs/cray-lnet.pc and https://github.com/Cray/lustre/blob/cray-2.12-int/cray-obs/cray-lustre-api-devel.pc

            James, that could go under a separate ticket? My understanding is that would be something quite different - building out-of-tree kernel modules, rather than the patch which just landed, which is for userspace tools that need access to the interface/protocol structures and constants and such?

            adilger Andreas Dilger added a comment - James, that could go under a separate ticket? My understanding is that would be something quite different - building out-of-tree kernel modules, rather than the patch which just landed, which is for userspace tools that need access to the interface/protocol structures and constants and such?

            Actually one more patch. People want the ability to build against external LND drivers.

            simmonsja James A Simmons added a comment - Actually one more patch. People want the ability to build against external LND drivers.
            pjones Peter Jones added a comment -

            Aa true man of the people Landed for 2.14

            pjones Peter Jones added a comment - Aa true man of the people Landed for 2.14

            Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/40728/
            Subject: LU-9215 build: Re-add the lustre-devel package
            Project: fs/lustre-release
            Branch: master
            Current Patch Set:
            Commit: 16af4e5ed634942cf6d8a847832684c5542a956c

            gerrit Gerrit Updater added a comment - Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/40728/ Subject: LU-9215 build: Re-add the lustre-devel package Project: fs/lustre-release Branch: master Current Patch Set: Commit: 16af4e5ed634942cf6d8a847832684c5542a956c

            By popular demand I added back lustre-devel for user land handling. As for what people are requesting that is kmod-lustre-devel rpm that you want. 

            simmonsja James A Simmons added a comment - By popular demand I added back lustre-devel for user land handling. As for what people are requesting that is kmod-lustre-devel rpm that you want. 

            James Simmons (jsimmons@infradead.org) uploaded a new patch: https://review.whamcloud.com/40728
            Subject: LU-9215 build: Re-add the lustre-devel package
            Project: fs/lustre-release
            Branch: master
            Current Patch Set: 1
            Commit: 1d71647c5ce75d7d7dbd464e218764b7314b7811

            gerrit Gerrit Updater added a comment - James Simmons (jsimmons@infradead.org) uploaded a new patch: https://review.whamcloud.com/40728 Subject: LU-9215 build: Re-add the lustre-devel package Project: fs/lustre-release Branch: master Current Patch Set: 1 Commit: 1d71647c5ce75d7d7dbd464e218764b7314b7811

            I don't think there is any question about the kernel or Lustre APIs being kept stable,, but that doesn't prevent modules from having conditional compilation as we already do with Lustre today.

            I don't object in principle to our-of-tree modules being built, but agree that "lustre-devel" is maybe not a good package name unless it also includes both the in-kernel and userspace development headers.

            adilger Andreas Dilger added a comment - I don't think there is any question about the kernel or Lustre APIs being kept stable,, but that doesn't prevent modules from having conditional compilation as we already do with Lustre today. I don't object in principle to our-of-tree modules being built, but agree that "lustre-devel" is maybe not a good package name unless it also includes both the in-kernel and userspace development headers.

            That is not how devel rpms work. They normally contain only static libraries and user land headers. Your ptl4lnd driver is a kernel driver so its going to always be dependent on the lustre kernel code  Their is no guarantee that the internal kernel apis will remain stable. In fact more changes to the LNet core are in the pipe line. 

            simmonsja James A Simmons added a comment - That is not how devel rpms work. They normally contain only static libraries and user land headers. Your ptl4lnd driver is a kernel driver so its going to always be dependent on the lustre kernel code  Their is no guarantee that the internal kernel apis will remain stable. In fact more changes to the LNet core are in the pipe line. 

            People

              simmonsja James A Simmons
              parinay parinay v kondekar (Inactive)
              Votes:
              0 Vote for this issue
              Watchers:
              12 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: