Skip to content

Commit

Permalink
Merge openebs#1105
Browse files Browse the repository at this point in the history
1105: chore: don't add default max qpairs per controller r=tiagolobocastro a=tiagolobocastro

Limiting the qpairs to a low value means we cannot then allocate more for the rebuilds.

Co-authored-by: Tiago Castro <[email protected]>
  • Loading branch information
mayastor-bors and tiagolobocastro committed Feb 18, 2022
2 parents f4a2475 + e493695 commit 7225aac
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 7225aac

Please sign in to comment.