Fixes #6

Removed legacy fack option and explicitly state kernal 4+ dependency in settings.
Stono · Jul 27, 2020 · 4fcf38e · 4fcf38e
1 parent f4555f2
commit 4fcf38e
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 1 deletion.
diff --git a/helmfile/charts/kconmon/templates/agent/daemonset.yaml b/helmfile/charts/kconmon/templates/agent/daemonset.yaml
@@ -61,7 +61,6 @@ spec:
           sysctl -w net.ipv4.tcp_max_syn_backlog=3240000
           sysctl -w net.ipv4.tcp_timestamps=0
           sysctl -w net.ipv4.tcp_sack=0
-          sysctl -w net.ipv4.tcp_fack=0
         imagePullPolicy: IfNotPresent
         image: "{{ required "Please specify the docker image" .Values.docker.image }}:{{ .Values.docker.tag | default .Chart.Version }}"
         securityContext:

diff --git a/helmfile/charts/kconmon/values.yaml b/helmfile/charts/kconmon/values.yaml
@@ -2,6 +2,7 @@ docker:
   image: stono/kconmon
 
 # Should we run an initContainer that enables core tcp connection setting tweaks
+# Note: This requires kernel 4+
 enableTcpTweaks: true
 
 config: