Merge pull request #94 from chendo/master

Allow overriding MTU for DIND
buildkite · Sep 6, 2022 · 5c2b995 · 5c2b995
2 parents aa03745 + 712f607
commit 5c2b995
Show file tree

Hide file tree

Showing 4 changed files with 6 additions and 1 deletion.
diff --git a/stable/agent/Chart.yaml b/stable/agent/Chart.yaml
@@ -1,7 +1,7 @@
 apiVersion: v1
 description: Buildkite Agent Chart
 name: agent
-version: 0.5.5
+version: 0.5.6
 appVersion: 3.25.0
 icon: https://buildkite.com/_next/static/assets/assets/images/brand-assets/buildkite-logo-portrait-on-light-61fc0230.png
 keywords:

diff --git a/stable/agent/README.md b/stable/agent/README.md
@@ -92,6 +92,7 @@ Parameter | Description | Default
 `dind.enabled` | Enable preconfigured Docker-in-Docker (DinD) pod configuration | `false`
 `dind.image` | Image to use for Docker-in-Docker (DinD) pod container | `docker:19.03-dind`
 `dind.port` | Port Docker-in-Docker (DinD) daemon listens on as REST request proxy | `2375`
+`dind.mtu` | The MTU used for Docker-inDocker (DinD) daemon. Must be lower than pod networking interface | `1500`
 `dind.resources` | Pod resource requests & limits for dind sidecar (if enabled) | `{}`
 `dind.volumeMounts` | Extra volumeMounts configuration | `nil`
 `terminationGracePeriodSeconds` | Duration in seconds the pod needs to terminate gracefully | `30`

diff --git a/stable/agent/templates/deployment.yaml b/stable/agent/templates/deployment.yaml
@@ -130,6 +130,9 @@ spec:
 {{- if .Values.dind.enabled }}
         - name: dind
           image: {{ .Values.dind.image | default "docker:19.03-dind" }}
+          args:
+            - "--mtu"
+            - "{{ .Values.dind.mtu | default "1500" }}"
           securityContext:
             privileged: true
           env:

diff --git a/stable/agent/values.yaml b/stable/agent/values.yaml
@@ -177,6 +177,7 @@ dind:
   image: docker:19.03-dind
   port: 2375
   resources: {}
+  mtu: 1500
   # We usually recommend not to specify default resources and to leave this as a conscious
   # choice for the user. This also increases chances charts run on environments with little
   # resources, such as Minikube. If you do want to specify resources, uncomment the following