Details

    • 5847

    Description

      dkms is a cross distro mechanism for the building and maintenance of out-of-tree kernel modules.
      (http://linux.dell.com/dkms/) The attached patch adds dkms support to the debian/ubuntu packaging infrastructure.

      The current debian/ubuntu kernel module package uses module-assistant, which produces a deb which is tied to a specific kernel version, requiring the user to manually rebuild packages when the kernel is changed.

      The dkms package allows modules for multiple kernel versions to be packaged together into a single deb. Once installed, the package contains triggers so that when a new kernel is installed for which no pre-built lustre modules exists, dkms will automatically build and install them. This reduces maintenance overhead on client machines.

      dkms also works on redhat and sles; it should be possible to fold dkms support into the rpm build process.

      Attachments

        1. diff
          10 kB
        2. dkms.patch
          0.6 kB
        3. lustre.spec
          9 kB

        Issue Links

          Activity

            [LU-1032] Add dkms support for kernel modules

            Oleg Drokin (oleg.drokin@intel.com) merged in patch https://review.whamcloud.com/26358/
            Subject: LU-1032 build: fix typo in lustre-dkms.spec changelog
            Project: fs/lustre-release
            Branch: master
            Current Patch Set:
            Commit: 74fb19ec4b12e9fb416c2d8fbe98825bfdd05846

            gerrit Gerrit Updater added a comment - Oleg Drokin (oleg.drokin@intel.com) merged in patch https://review.whamcloud.com/26358/ Subject: LU-1032 build: fix typo in lustre-dkms.spec changelog Project: fs/lustre-release Branch: master Current Patch Set: Commit: 74fb19ec4b12e9fb416c2d8fbe98825bfdd05846

            Andreas Dilger (andreas.dilger@intel.com) uploaded a new patch: https://review.whamcloud.com/26358
            Subject: LU-1032 build: fix typo in lustre-dkms.spec changelog
            Project: fs/lustre-release
            Branch: master
            Current Patch Set: 1
            Commit: 166a49ce2a1d7b5a4491a33e3ebac0aef72006b0

            gerrit Gerrit Updater added a comment - Andreas Dilger (andreas.dilger@intel.com) uploaded a new patch: https://review.whamcloud.com/26358 Subject: LU-1032 build: fix typo in lustre-dkms.spec changelog Project: fs/lustre-release Branch: master Current Patch Set: 1 Commit: 166a49ce2a1d7b5a4491a33e3ebac0aef72006b0
            gerrit Gerrit Updater added a comment - - edited

            -Andreas Dilger (andreas.dilger@intel.com) uploaded a new patch: https://review.whamcloud.com/26356-
            Subject: LU-1032 build: fix typo in lustre-dkms.spec changelog
            Project: fs/lustre-release
            Branch: master
            Current Patch Set: 1
            Commit: c7b3fc6ea51887183efbbfda106fb6f804a29c89

            gerrit Gerrit Updater added a comment - - edited -Andreas Dilger (andreas.dilger@intel.com) uploaded a new patch: https://review.whamcloud.com/26356- Subject: LU-1032 build: fix typo in lustre-dkms.spec changelog Project: fs/lustre-release Branch: master Current Patch Set: 1 Commit: c7b3fc6ea51887183efbbfda106fb6f804a29c89

            Oleg Drokin (oleg.drokin@intel.com) merged in patch http://review.whamcloud.com/12347/
            Subject: LU-1032 build: DKMS RPM for Lustre Client modules
            Project: fs/lustre-release
            Branch: master
            Current Patch Set:
            Commit: 549c57e77b7f3a4cae5a7381d612a499c2ca3dcc

            gerrit Gerrit Updater added a comment - Oleg Drokin (oleg.drokin@intel.com) merged in patch http://review.whamcloud.com/12347/ Subject: LU-1032 build: DKMS RPM for Lustre Client modules Project: fs/lustre-release Branch: master Current Patch Set: Commit: 549c57e77b7f3a4cae5a7381d612a499c2ca3dcc
            pjones Peter Jones added a comment -

            Bruno I think that it would be better to create a new ticket that is linked to this one

            pjones Peter Jones added a comment - Bruno I think that it would be better to create a new ticket that is linked to this one

            I am adding this comment in this closed/fixed JIRA since it has been the main ticket to track issues around the Lustre modules DKMS RPMs creation, so patch to implement the necessary stuff to allow for a Lustre Client (only) modules DKMS RPM is at http://review.whamcloud.com/12347.

            bfaccini Bruno Faccini (Inactive) added a comment - I am adding this comment in this closed/fixed JIRA since it has been the main ticket to track issues around the Lustre modules DKMS RPMs creation, so patch to implement the necessary stuff to allow for a Lustre Client (only) modules DKMS RPM is at http://review.whamcloud.com/12347 .

            We've sorted the DKMS issues on the ZFS side and have updated the ZFS stable repositories before dkms-2.2.0.3-2 made it in to EPEL. Only Fedora was impacted for a few hours but Lustre may need a similar fix to that in 2276 above.

            behlendorf Brian Behlendorf added a comment - We've sorted the DKMS issues on the ZFS side and have updated the ZFS stable repositories before dkms-2.2.0.3-2 made it in to EPEL. Only Fedora was impacted for a few hours but Lustre may need a similar fix to that in 2276 above.

            We should be able to sort this out on our side. It would be best for everyone if we relied on the version of DKMS shipped by the distributions. We're sorting out a reasonable fix now, there's a proposed patch in https://github.com/zfsonlinux/zfs/pull/2776. You'll likely need to make a similar tweak for Lustre.

            behlendorf Brian Behlendorf added a comment - We should be able to sort this out on our side. It would be best for everyone if we relied on the version of DKMS shipped by the distributions. We're sorting out a reasonable fix now, there's a proposed patch in https://github.com/zfsonlinux/zfs/pull/2776 . You'll likely need to make a similar tweak for Lustre.
            brian Brian Murrell (Inactive) added a comment - - edited

            Ugh. So dkms-2.2.0.3-28.git.7c3e7c5.fc20.noarch doesn't work either now? Is there a version that is known to work? Do we know of a patch to dkms-2.2.0.3-28.git.7c3e7c5.fc20.noarch that will make it work?

            I suppose we could just apply the reversion of http://linux.dell.com/cgi-bin/cgit.cgi/dkms.git/commit/dkms?id=2ea43f6108558849125cc1d66902d6992ee3fe39 and take advantage of the bug they fixed until this is more properly fixed.

            brian Brian Murrell (Inactive) added a comment - - edited Ugh. So dkms-2.2.0.3-28.git.7c3e7c5.fc20.noarch doesn't work either now? Is there a version that is known to work? Do we know of a patch to dkms-2.2.0.3-28.git.7c3e7c5.fc20.noarch that will make it work? I suppose we could just apply the reversion of http://linux.dell.com/cgi-bin/cgit.cgi/dkms.git/commit/dkms?id=2ea43f6108558849125cc1d66902d6992ee3fe39 and take advantage of the bug they fixed until this is more properly fixed.

            Interesting. OK, thanks for letting me know of getting it fixed. Unfortunately, it seems to have another problem as of today. There was recently an upstream patch merged which broke things by removing the builds directory. Here's the new ticket https://bugzilla.redhat.com/show_bug.cgi?id=1151123

            behlendorf Brian Behlendorf added a comment - Interesting. OK, thanks for letting me know of getting it fixed. Unfortunately, it seems to have another problem as of today. There was recently an upstream patch merged which broke things by removing the builds directory. Here's the new ticket https://bugzilla.redhat.com/show_bug.cgi?id=1151123

            People

              bfaccini Bruno Faccini (Inactive)
              gmpc@sanger.ac.uk Guy Coates
              Votes:
              0 Vote for this issue
              Watchers:
              20 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: