Skip to content

Commit

Permalink
fix: account for builtin nvme-tcp module in initContainer
Browse files Browse the repository at this point in the history
 - If sysfs check pass - driver is available as either loaded already or builtin
 - If sysfs check fails - driver isn't builtin but can be possibly loaded

Signed-off-by: Diwakar Sharma <[email protected]>
  • Loading branch information
dsharma-dc committed Jan 20, 2025
1 parent 9ea6869 commit 3dac134
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion chart/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -364,7 +364,7 @@ csi:
enabled: false
containers:
- name: nvme-tcp-probe
command: ['sh', '-c', 'trap "exit 1" TERM; until $(lsmod | grep nvme_tcp &>/dev/null); do [ -z "$WARNED" ] && echo "nvme_tcp module not loaded..."; WARNED=1; sleep 60; done;']
command: ['sh', '-c', 'trap "exit 1" TERM; until [ -d /sys/module/nvme_tcp ]; do [ -z "$WARNED" ] && echo "nvme_tcp module not loaded..."; WARNED=1; sleep 60; done;']

io_engine:
# -- Log level for the io-engine service
Expand Down

0 comments on commit 3dac134

Please sign in to comment.