• Home
  • Features
  • Pricing
  • Docs
  • Announcements
  • Sign In
Info updated!

rollandf / doca-driver-build
67%
main: 66%

Build:
Build:
LAST BUILD BRANCH: dtk-dkms
DEFAULT BRANCH: main
Repo Added 07 Sep 2025 12:04PM UTC
Token hrqpudLxQibJe4WKCEdl9RR33iWuDCuKF regen
Build 110 Last
Files 16
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst

LAST BUILD ON BRANCH dtk-dkms
branch: SELECT
CHANGE BRANCH
x
Sync Branches
  • No branch selected
  • 1.2.8
  • 25.10-manifest-update
  • 26.4
  • 3.2.2
  • blacklist
  • build
  • config-file
  • contributing
  • debuginfo
  • dtk-dkms
  • dtk-go
  • dtk-poll
  • flock
  • gcc-test
  • go12411
  • go125
  • go125-net26.1
  • interactibe
  • license-fix
  • license-update
  • load
  • loader
  • logoutput
  • lsrmmod
  • main
  • os-discovry
  • preparegcc
  • publicpath
  • ready
  • rhel-stig
  • rhel-update
  • rhelstig
  • sriov
  • switchdev
  • third-party
  • udev
  • udevscheme
  • updateca
  • without-dkms
  • workflow

04 May 2026 09:56AM UTC coverage: 67.262% (+1.7%) from 65.552%
25312682005

push

github

rollandf
fix(dtk): fix DKMS support in OCP/DTK driver build path

Three bugs prevented USE_DKMS=true from working with the DTK sidecar:

1. dtk/build.go unconditionally passed --without-dkms (and --disable-kmp)
   to install.pl regardless of USE_DKMS, so the driver inventory never
   contained either DKMS source packages or pre-compiled kmod binary
   packages when DKMS was requested.

2. discoverDKMSModule in driver.go could not locate the dkms.conf file
   in OFED 26.04 packages because the installed layout uses a symlink
   pointing to a source/ subdirectory rather than placing dkms.conf
   directly at the top level of /usr/src/<module>-<version>/.

3. setupDKMS attempted to run dkms build and dkms install inside the
   main driver container, which does not carry kernel headers.  In OCP
   the DTK sidecar is the only container that has the matching kernel
   headers; running dkms build in the main container therefore fails
   unconditionally.

Fix:

- When cfg.UseDKMS is true, omit both --disable-kmp and --without-dkms
  from the install.pl invocation in the DTK sidecar.  Without those flags
  install.pl produces both a DKMS source package (registers the source
  under /usr/src/ for dkms add) and pre-compiled kmod binary packages
  (place the .ko files directly under /lib/modules/<kernel>/ without
  requiring a subsequent dkms build step).

- Refactor discoverDKMSModule into the testable discoverDKMSModuleIn
  helper that resolves symlinks and descends into source/ subdirectories
  to reliably locate dkms.conf across OFED package layouts.

- In setupDKMS, when DtkOcpDriverBuild is true, stop after dkms add
  (source registration) and skip dkms build / dkms install.  The kmod
  binary packages installed by installDriver already placed the correct
  .ko files; no recompilation is needed or possible.

Add unit tests covering discoverDKMSModuleIn path variations and the
DTK+DKMS setupDKMS skip-build-install code path.

Signed-off-by: Fred Rolland <frolland@nvidia.com>

43 of 52 new or added lines in 3 files covered. (82.69%)

1 existing line in 1 file now uncovered.

2636 of 3919 relevant lines covered (67.26%)

5.34 hits per line

Relevant lines Covered
Build:
Build:
3919 RELEVANT LINES 2636 COVERED LINES
5.34 HITS PER LINE
Source Files on main
  • Tree
  • List 16
  • Changed 3
  • Source Changed 3
  • Coverage Changed 3
Coverage ∆ File Lines Relevant Covered Missed Hits/Line

Recent builds

Builds Branch Commit Type Ran Committer Via Coverage
25312682005 dtk-dkms fix(dtk): fix DKMS support in OCP/DTK driver build path Three bugs prevented USE_DKMS=true from working with the DTK sidecar: 1. dtk/build.go unconditionally passed --without-dkms (and --disable-kmp) to install.pl regardless of USE_DKMS, so t... push 04 May 2026 09:57AM UTC rollandf github
67.26
25312060733 dtk-dkms fix(dtk): fix DKMS support in OCP/DTK driver build path Three bugs prevented USE_DKMS=true from working with the DTK sidecar: 1. dtk/build.go unconditionally passed --without-dkms (and --disable-kmp) to install.pl regardless of USE_DKMS, so t... push 04 May 2026 09:42AM UTC rollandf github
67.26
25280905521 dtk-dkms fix(dtk): fix DKMS support in OCP/DTK driver build path Three bugs prevented USE_DKMS=true from working with the DTK sidecar: 1. dtk/build.go unconditionally passed --without-dkms (and --disable-kmp) to install.pl regardless of USE_DKMS, so t... push 03 May 2026 01:49PM UTC rollandf github
67.29
25226929306 dtk-dkms fix(dtk): pass --without-dkms to install.pl only when USE_DKMS is false The DTK build script unconditionally passed --without-dkms to install.pl, so the driver inventory never contained *-dkms-* source packages regardless of the USE_DKMS setting.... push 01 May 2026 06:22PM UTC rollandf github
66.68
25179853204 dtk-dkms fix(dtk): pass --without-dkms to install.pl only when USE_DKMS is false The DTK build script unconditionally passed --without-dkms to install.pl, so the driver inventory never contained *-dkms-* source packages regardless of the USE_DKMS setting.... push 30 Apr 2026 05:32PM UTC rollandf github
65.6
25111911428 main Merge pull request #312 from Mellanox/dependabot/go_modules/entrypoint/go.uber.org/zap-1.28.0 chore: bump go.uber.org/zap from 1.27.1 to 1.28.0 in /entrypoint push 29 Apr 2026 01:33PM UTC web-flow github
65.55
24835920836 dtk-poll fix: reduce DTK build poll interval from 300s to 30s Air-gapped builds from a local RPM repository complete in ~180s, making the previous 300s initial wait unnecessarily long. Poll every 30s with 30 retries (900s max timeout) and remove the halvi... push 23 Apr 2026 12:45PM UTC rollandf github
65.55
24668398371 config-file fix: skip kernel pre-req in DTK Signed-off-by: Fred Rolland <frolland@nvidia.com> push 20 Apr 2026 01:11PM UTC rollandf github
65.51
24657682705 config-file fix: invalidate driver cache on config changes The inventory cache check only validated kernel version, driver version, and an MD5 of the cached package files. It did not account for build- affecting env vars (ENABLE_NFSRDMA, USE_DKMS, APPEND_DRI... push 20 Apr 2026 08:59AM UTC rollandf github
64.44
24469123307 config-file fix: invalidate driver cache on config changes The inventory cache check only validated kernel version, driver version, and an MD5 of the cached package files. It did not account for build- affecting env vars (ENABLE_NFSRDMA, USE_DKMS, APPEND_DRI... push 15 Apr 2026 05:40PM UTC rollandf github
64.45
See All Builds (110)

Badge your Repo: doca-driver-build

We detected this repo isn’t badged! Grab the embed code to the right, add it to your repo to show off your code coverage, and when the badge is live hit the refresh button to remove this message.

Could not find badge in README.

Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst

Refresh
  • Settings
  • Repo on GitHub
STATUS · Troubleshooting · Open an Issue · Sales · Support · CAREERS · ENTERPRISE · START FREE · SCHEDULE DEMO
ANNOUNCEMENTS · TWITTER · TOS & SLA · Supported CI Services · What's a CI service? · Automated Testing

© 2026 Coveralls, Inc