The following pull request was submitted through Github.
It can be accessed and reviewed at: https://github.com/lxc/lxd/pull/7508

This e-mail was sent by the LXC bot, direct replies will not reach the author
unless they happen to be subscribed to this list.

=== Description (from pull-request) ===
Warn if setting failed due to host missing VLAN filtering feature.

Signed-off-by: Thomas Parrott <thomas.parr...@canonical.com>
From 6f459040405e9936fad49f33669284f98c034c43 Mon Sep 17 00:00:00 2001
From: Thomas Parrott <thomas.parr...@canonical.com>
Date: Wed, 10 Jun 2020 09:57:51 +0100
Subject: [PATCH] lxd/network: Make setting bridge VLAN filtering & default
 PVID optional

Warn if setting failed.

Signed-off-by: Thomas Parrott <thomas.parr...@canonical.com>
---
 lxd/network/network.go | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/lxd/network/network.go b/lxd/network/network.go
index d8819d0a3b..2c5d9abad3 100644
--- a/lxd/network/network.go
+++ b/lxd/network/network.go
@@ -287,13 +287,13 @@ func (n *Network) setup(oldConfig map[string]string) 
error {
        if n.config["bridge.driver"] != "openvswitch" {
                err = BridgeVLANFilterSetStatus(n.name, "1")
                if err != nil {
-                       return err
+                       logger.Warnf("%v", err)
                }
 
                // Set the default PVID for new ports to 1.
                err = BridgeVLANSetDefaultPVID(n.name, "1")
                if err != nil {
-                       return err
+                       logger.Warnf("%v", err)
                }
        }
 
_______________________________________________
lxc-devel mailing list
lxc-devel@lists.linuxcontainers.org
http://lists.linuxcontainers.org/listinfo/lxc-devel

Reply via email to