Skip to content

Commit

Permalink
helm: 'bpf.ctTcpMax' and 'bpf.ctAnyMax' need to be strings, not integers
Browse files Browse the repository at this point in the history
Signed-off-by: Maxime VISONNEAU <[email protected]>
  • Loading branch information
mvisonneau authored and joestringer committed Dec 15, 2020
1 parent 5b911b8 commit 6d3143f
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 3 deletions.
2 changes: 1 addition & 1 deletion install/kubernetes/cilium/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -59,7 +59,7 @@ contributors across the globe, there is almost always someone available to help.
| azure.enabled | bool | `false` | Enable Azure integration |
| bandwidthManager | bool | `true` | Optimize TCP and UDP workloads and enable rate-limiting traffic from individual Pods with EDT (Earliest Departure Time) through the "kubernetes.io/egress-bandwidth" Pod annotation. |
| bpf.clockProbe | bool | `false` | |
| bpf.lbMapMax | int | `65536` | Configure the maximum number of entries in the TCP connection tracking table. ctTcpMax: 524288 -- Configure the maximum number of entries for the non-TCP connection tracking table. ctAnyMax: 262144 -- Configure the maximum number of service entries in the load balancer maps. |
| bpf.lbMapMax | int | `65536` | Configure the maximum number of entries in the TCP connection tracking table. ctTcpMax: '524288' -- Configure the maximum number of entries for the non-TCP connection tracking table. ctAnyMax: '262144' -- Configure the maximum number of service entries in the load balancer maps. |
| bpf.monitorAggregation | string | `"medium"` | Configure auto-sizing for all BPF maps based on available memory. ref: https://docs.cilium.io/en/v1.9/concepts/ebpf/maps/#ebpf-maps -- Configure the level of aggregation for monitor notifications. Valid options are none, low, medium, maximum |
| bpf.monitorFlags | string | `"all"` | Configure which TCP flags trigger notifications when seen for the first time in a connection. |
| bpf.monitorInterval | string | `"5s"` | Configure the typical time between monitor notifications for active connections. |
Expand Down
4 changes: 2 additions & 2 deletions install/kubernetes/cilium/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -193,11 +193,11 @@ bpf:

# -- Configure the maximum number of entries in the TCP connection tracking
# table.
# ctTcpMax: 524288
# ctTcpMax: '524288'

# -- Configure the maximum number of entries for the non-TCP connection
# tracking table.
# ctAnyMax: 262144
# ctAnyMax: '262144'

# -- Configure the maximum number of service entries in the
# load balancer maps.
Expand Down

0 comments on commit 6d3143f

Please sign in to comment.