dann frazier

Merge ~dannf/+git/maas-preseeds:hidon-re-enable-mofed-and-warn into ~ce-hyperscale/+git/maas-preseeds:main

  1. Git
  2. lp:~dannf/+git/maas-preseeds
  3. hidon-re-enable-mofed-and-warn
  4. Merge into main
Proposed by dann frazier
Status: Merged
Merged at revision: cb5e8aa0d5620cc841f91b651f5d408715327011
Proposed branch: ~dannf/+git/maas-preseeds:hidon-re-enable-mofed-and-warn
Merge into: ~ce-hyperscale/+git/maas-preseeds:main
Diff against target: 19 lines (+6/-2)
1 file modified
late.sh (+6/-2)
Reviewer Review Type Date Requested Status
Ian May (community) Approve
CE Hyperscale Pending
Review via email: mp+460445@code.launchpad.net
To post a comment you must log in.
Revision history for this message
Ian May (ian-may) wrote :

LGTM!

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1diff --git a/late.sh b/late.sh
2index d4a2f68..0d875de 100755
3--- a/late.sh
4+++ b/late.sh
5@@ -58,8 +58,12 @@ install_gpu_drivers() {
6
7 install_mellanox_ofed() {
8 if [ "$(dmidecode -s system-product-name)" = "DGXH100" ]; then
9- # Disable MOFED installation until certification is complete
10- return 0
11+ touch /home/ubuntu/JEFF-THIS-HAS-MOFED-INSTALLED-DONT-IPERF
12+ touch /root/JEFF-THIS-HAS-MOFED-INSTALLED-DONT-IPERF
13+ # We'll need to redisable this later for cert testing, so
14+ # just leave it commented out. See:
15+ # https://warthogs.atlassian.net/browse/NVDGX-675
16+ #return 0
17 fi
18 local MLNX_REPO="https://linux.mellanox.com/public/repo/mlnx_ofed"
19 # NOTE!!

Subscribers

People subscribed via source and target branches

to all changes: