[lxc-devel] [lxd/master] network: Don't require ipt_checksum

[stgraber on Github]

A non-text attachment was scrubbed...
Name: not available
Type: text/x-mailbox
Size: 397 bytes
Desc: not available
URL: <http://lists.linuxcontainers.org/pipermail/lxc-devel/attachments/20170917/cd27c06f/attachment.bin>
-------------- next part --------------
From a81eacbf79b4febdcaabf5606b6554ef1315f16e Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?St=C3=A9phane=20Graber?= <stgraber at ubuntu.com>
Date: Sun, 17 Sep 2017 16:56:20 -0700
Subject: [PATCH] network: Don't require ipt_checksum
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

This should fix things for Solus users.

Signed-off-by: Stéphane Graber <stgraber at ubuntu.com>
---
 lxd/networks.go | 7 ++-----
 1 file changed, 2 insertions(+), 5 deletions(-)

diff --git a/lxd/networks.go b/lxd/networks.go
index a34e6ba27..939869556 100644
--- a/lxd/networks.go
+++ b/lxd/networks.go
@@ -735,11 +735,8 @@ func (n *network) Start() error {
 			}
 		}
 
-		// Workaround for broken DHCP clients
-		err = networkIptablesPrepend("ipv4", n.name, "mangle", "POSTROUTING", "-o", n.name, "-p", "udp", "--dport", "68", "-j", "CHECKSUM", "--checksum-fill")
-		if err != nil {
-			return err
-		}
+		// Attempt a workaround for broken DHCP clients
+		networkIptablesPrepend("ipv4", n.name, "mangle", "POSTROUTING", "-o", n.name, "-p", "udp", "--dport", "68", "-j", "CHECKSUM", "--checksum-fill")
 
 		// Allow forwarding
 		if n.config["bridge.mode"] == "fan" || n.config["ipv4.routing"] == "" || shared.IsTrue(n.config["ipv4.routing"]) {