[LU-18249] RPM: Migrate ofed (ofa_kernel modules) requires to MOFED o2iblnd module package

• Type: Bug
• Resolution: Fixed
• Priority: Minor
• Fix Version/s: Lustre 2.17.0
• Severity: 3
• Assignee: Shaun Tancheff
• Reporter: Shaun Tancheff

Description

The current RPM packaging build for MOFED places an external dependency requirement on kmod-mlnx-ofa_kernel / mlnx-ofa_kernel-kmp for all the lustre packages.

In the case of user-space packages there should be no direct dependency on the 3rd party OFED, it should only apply to the actual kernel module package that requires MOFED.

This enables the use case where build servers build with an external MOFED and in-kernel users can install the distributed rpms and choose the in-kernel package with out hitting a MOFED requires.

Comments

[Peter Jones]

Merged for 2.17

[Gerrit Updater]

"Oleg Drokin <[e-mail]>" merged in patch https://review.whamcloud.com/c/fs/lustre-release/+/56446/
Subject: LU-18249 o2iblnd: external mofed driver requires
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: d3e400b086379e46635b0e0057b86b3be0828431

[Shaun Tancheff]

Only the package with the kernel module(s) should have any MOFED dependencies, otherwise you end up with unable to install the user space package with dkms (if you built the user space with MOFED).

In the case of multiple_lnds then only the o2iblnd package will depend on MOFED. This offers the most flexibility as the user can then install dkms or the lnd(s) that they need.

It would be nice to have for 2.16 since we are shifting to dkms internally though it can be worked around by not building the main user package with MOFED so it is not critical for us.

For those looking to ship with multiple_lnds binary rpms then this would be critical, imo.

[Andreas Dilger]

Should even the main Lustre package depend on MOFED, or should that only be the ko2iblnd package? It definitely shouldn't be required if only using TCP.

Is this significant enough to be flagged for 2.16 inclusion?

[Gerrit Updater]

"Shaun Tancheff <[e-mail]>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/56446
Subject: LU-18249 o2iblnd: external mofed driver requires
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 7745cd81451710b7f390180e4684845ed2305ee5