Skip to content

major: Set MTU for docker globally to 1400 #476

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
venkatamutyala merged 1 commit into from
Apr 26, 2026
Merged

major: Set MTU for docker globally to 1400 #476

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
1 change: 1 addition & 0 deletions developer-setup.sh
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -22,6 +22,7 @@ echo "Installing other requirements now"

curl -fsSL https://get.docker.com -o get-docker.sh
sudo sh get-docker.sh
sudo mkdir -p /etc/docker && echo '{"mtu": 1400}' | sudo tee /etc/docker/daemon.json > /dev/null && sudo systemctl restart docker
Copy link

Copilot AI Apr 26, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This command overwrites /etc/docker/daemon.json unconditionally. If the host already has Docker daemon configuration (e.g., registry-mirrors, insecure-registries, log-driver, etc.), this will clobber it and may break Docker on reruns. Consider preserving existing JSON and only setting/overriding the "mtu" key (e.g., read existing file if present and merge), or gate the write behind a check when the file doesn’t exist.

Suggested change
sudo mkdir -p /etc/docker && echo '{"mtu": 1400}' | sudo tee /etc/docker/daemon.json > /dev/null && sudo systemctl restart docker
sudo mkdir -p /etc/docker
TMP_DOCKER_DAEMON_JSON=$(mktemp)
if sudo test -s /etc/docker/daemon.json; then
sudo python3 -c 'import json; import sys; p = sys.argv[1]; data = json.load(open(p)); data["mtu"] = 1400; json.dump(data, sys.stdout)' /etc/docker/daemon.json | sudo tee "$TMP_DOCKER_DAEMON_JSON" > /dev/null
else
echo '{"mtu": 1400}' | sudo tee "$TMP_DOCKER_DAEMON_JSON" > /dev/null
fi
sudo mv "$TMP_DOCKER_DAEMON_JSON" /etc/docker/daemon.json
sudo systemctl restart docker

Copilot uses AI. Check for mistakes.
sudo apt-get update
sudo apt install tmux jq figlet qemu-guest-agent autossh ripgrep -y
sudo apt-get clean
Expand Down
Loading