Skip to content

Commit

Permalink
chore: don't add default max qpairs per controller
Browse files Browse the repository at this point in the history
Limiting the qpairs to a low value means we cannot then allocate more for the rebuilds.
  • Loading branch information
tiagolobocastro committed Feb 18, 2022
1 parent f4a2475 commit e493695
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 6 deletions.
3 changes: 0 additions & 3 deletions chart/templates/mayastor-daemonset.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -36,9 +36,6 @@ spec:
env:
- name: RUST_LOG
value: info,mayastor={{ .Values.mayastorLogLevel }}
- name: NVMF_TCP_MAX_QPAIRS_PER_CTRL
# Current recommendation is to set this value to be the number of cores provided to mayastor (see -l argument) plus 1.
value: "{{ add .Values.mayastorCpuCount 1 }}"
- name: NVMF_TCP_MAX_QUEUE_DEPTH
value: "32"
- name: MY_NODE_NAME
Expand Down
3 changes: 0 additions & 3 deletions deploy/mayastor-daemonset.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -38,9 +38,6 @@ spec:
env:
- name: RUST_LOG
value: info,mayastor=info
- name: NVMF_TCP_MAX_QPAIRS_PER_CTRL
# Current recommendation is to set this value to be the number of cores provided to mayastor (see -l argument) plus 1.
value: "2"
- name: NVMF_TCP_MAX_QUEUE_DEPTH
value: "32"
- name: MY_NODE_NAME
Expand Down

0 comments on commit e493695

Please sign in to comment.