Skip to content
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.

Commit facea89

Browse files
committedMar 25, 2025·
Fix sriov MFT dependency installation for Debian
1 parent 72270c9 commit facea89

File tree

3 files changed

+28
-4
lines changed

3 files changed

+28
-4
lines changed
 
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,5 @@
1+
---
2+
features:
3+
- |
4+
Updates the sriov role to correctly install MFT dependencies when on a
5+
Debian system.

‎roles/sriov/defaults/main.yml

+10-1
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,7 @@ sriov_mft_url: https://www.mellanox.com/downloads/MFT/mft-4.23.0-104-x86_64-rpm.
77

88
sriov_mft_tarball: "{{ sriov_mft_url | urlsplit('path') | basename }}"
99

10-
sriov_os_pkgs:
10+
sriov_os_pkgs_rhel:
1111
- gcc
1212
- rpm-build
1313
# -E- Could not find lspci, you may need to install "pciutils" package
@@ -17,6 +17,15 @@ sriov_os_pkgs:
1717
- kernel-devel
1818
- make
1919

20+
sriov_os_pkgs_deb:
21+
- gcc
22+
# -E- Could not find lspci, you may need to install "pciutils" package
23+
- pciutils
24+
# NOTE(wszumski): Assumes running latest kernel. We could include package version with
25+
# output of uname -r, but the packages aren't always available.
26+
- linux-headers-generic
27+
- make
28+
2029
sriov_restart_handler: reboot
2130

2231
sriov_numvfs: 8

‎roles/sriov/tasks/mlxconfig.yml

+13-3
Original file line numberDiff line numberDiff line change
@@ -1,9 +1,19 @@
11
---
2-
- name: Install MFT dependencies
2+
- name: Install MFT dependencies (RHEL)
33
ansible.builtin.yum:
4-
name: "{{ sriov_os_pkgs }}"
4+
name: "{{ sriov_os_pkgs_rhel }}"
55
become: true
6-
when: sriov_install_enabled
6+
when:
7+
- sriov_install_enabled
8+
- ansible_facts.os_family == 'RedHat'
9+
10+
- name: Install MFT dependencies (Debian)
11+
ansible.builtin.apt:
12+
name: "{{ sriov_os_pkgs_deb }}"
13+
become: true
14+
when:
15+
- sriov_install_enabled
16+
- ansible_facts.os_family == 'Debian'
717

818
- name: Make working directory
919
ansible.builtin.file:

0 commit comments

Comments
 (0)
Please sign in to comment.