From 680a6d2ad323e699098a7474a57df0299e4dd848 Mon Sep 17 00:00:00 2001 From: Jo-Philipp Wich Date: Sat, 28 Jul 2018 14:09:34 +0200 Subject: [PATCH] luci-proto-relay: propagate up state, do not forward netifd errors Due to historical reasons, the relayd daemon configuration resides in the form of a fake "config interface" section in /etc/config/network without actually registering a protocol handler. This causes netifd to emit an "INVALID_PROTO" error for the interface which is technically correct, but confusing for LuCI users. This situation needs to be resolved upstream by either moving relayd configuration out of the network config, or by converting the relayd service into a proper protocol handler. Until this happens, do not report any netifd errors since we're not operating on an actual interface. While we're at it, also propagate the protocol up state from the fake virtual device to ensure that LuCI displays the relay bridge as "up" when all relayed interfaces are up as well. Signed-off-by: Jo-Philipp Wich (cherry picked from commit aae187dedf726fe609ede8260f8f8343d843d046) --- .../luasrc/model/network/proto_relay.lua | 9 +++++++++ 1 file changed, 9 insertions(+) diff --git a/protocols/luci-proto-relay/luasrc/model/network/proto_relay.lua b/protocols/luci-proto-relay/luasrc/model/network/proto_relay.lua index a2dabf9af..3b811d44d 100644 --- a/protocols/luci-proto-relay/luasrc/model/network/proto_relay.lua +++ b/protocols/luci-proto-relay/luasrc/model/network/proto_relay.lua @@ -32,6 +32,11 @@ function proto.is_virtual(self) return true end +function proto.is_up(self) + local iface = self:get_interface() + return iface and iface:is_up() or false +end + function proto.get_interface(self) return device(self.sid, self) end @@ -80,6 +85,10 @@ function proto.uptime(self) return upt end +function proto.errors(self) + return nil +end + function device.__init__(self, ifname, network) self.ifname = ifname -- 2.25.1