gluon-client-bridge, gluon-mesh-batman-adv-core: switch roles of br-client and local-node interfaces

MAC and IP addresses are switched. This makes the gluon-client-bridge
package more useful for different routing protocols that don't need a
unique address on the client bridge.

As a side effect, gluon-radvd is now using the next-node address, which had
been considered before, but was dismissed to avoid having gluon-radvd
depend on gluon-next-node and gluon-mesh-batman-adv. This will be useful
for announcing default routes via gluon-radvd.

One downside is that this introduces a minor dependency on batman-adv in
gluon-respondd: the hotplug script that checked for the client interface
before will now check for local-node. This doesn't really matter: for mesh
protocols without a local-node interface, the check will do nothing (which
makes sense, as there is no interface to bind to for mesh-wide respondd).

package/gluon-alfred/Makefile

@@ -12,7 +12,7 @@ include $(GLUONDIR)/include/package.mk
 define Package/gluon-alfred
   SECTION:=gluon
   CATEGORY:=Gluon
-  DEPENDS:=+gluon-core +gluon-respondd +gluon-neighbour-info +micrond +alfred
+  DEPENDS:=+gluon-core +gluon-respondd +gluon-neighbour-info gluon-mesh-batman-adv +micrond +alfred
   TITLE:=Configure alfred
 endef
 

package/gluon-alfred/luasrc/lib/gluon/upgrade/500-enable-alfred

@@ -7,7 +7,7 @@ local c = uci.cursor()
 c:delete('alfred', 'alfred')
 c:section('alfred', 'alfred', 'alfred',
 	  {
-		  interface = 'br-client',
+		  interface = 'local-node',
 		  mode = 'slave',
 		  batmanif = 'bat0',
 		  start_vis = '1',

package/gluon-client-bridge/check_site.lua

@@ -1,3 +1,12 @@
+need_string_match('next_node.mac', '^%x[02468aAcCeE]:%x%x:%x%x:%x%x:%x%x:%x%x$')
+
+if need_string_match('next_node.ip4', '^%d+.%d+.%d+.%d+$', false) then
+  need_string_match('prefix4', '^%d+.%d+.%d+.%d+/%d+$')
+end
+
+need_string_match('next_node.ip6', '^[%x:]+$', false)
+
+
 for _, config in ipairs({'wifi24', 'wifi5'}) do
    if need_table(config .. '.ap', nil, false) then
       need_string(config .. '.ap.ssid')

package/gluon-client-bridge/luasrc/lib/gluon/upgrade/300-gluon-client-bridge-network

@@ -1,17 +1,40 @@
 #!/usr/bin/lua
 
+local site = require 'gluon.site_config'
 local sysconfig = require 'gluon.sysconfig'
 
+local ip = require 'luci.ip'
 local lutil = require 'luci.util'
 local uci = require('luci.model.uci').cursor()
 
 
+local ip4, netmask, ip6
+
+if site.next_node.ip4 then
+	ip4 = site.next_node.ip4
+	netmask = ip.IPv4(site.prefix4):mask():string()
+end
+
+if site.next_node.ip6 then
+	ip6 = site.next_node.ip6 .. '/128'
+end
+
 uci:section('network', 'interface', 'client',
 	{
 		type = 'bridge',
+		proto = 'static',
+		macaddr = site.next_node.mac,
+		ipaddr = ip4,
+		netmask = netmask,
+		ip6addr = ip6,
 	}
 )
 
+uci:delete('network', 'client', 'reqprefix')
+uci:delete('network', 'client', 'peerdns')
+uci:delete('network', 'client', 'sourcefilter')
+
+
 local ifname = uci:get('network', 'client', 'ifname')
 
 if type(ifname) == 'string' then
@@ -27,7 +50,20 @@ if sysconfig.lan_ifname and not ifname and not uci:get_bool('network', 'mesh_lan
 	end
 end
 
-
-uci:set('network', 'client', 'macaddr', sysconfig.primary_mac)
-
 uci:save('network')
+
+
+local dnsmasq = uci:get_first('dhcp', 'dnsmasq')
+uci:set('dhcp', dnsmasq, 'boguspriv', 0)
+uci:set('dhcp', dnsmasq, 'localise_queries', 0)
+uci:set('dhcp', dnsmasq, 'rebind_protection', 0)
+
+uci:delete('dhcp', 'client')
+uci:section('dhcp', 'dhcp', 'client',
+	{
+		interface = 'client',
+		ignore = 1,
+	}
+)
+
+uci:save('dhcp')

package/gluon-mesh-batman-adv-core/check_site.lua

@@ -23,11 +23,3 @@ need_boolean('mesh_on_lan', false)
 if need_table('mesh', nil, false) and  need_table('mesh.batman_adv', nil, false) then
   need_number('mesh.batman_adv.gw_sel_class', false)
 end
-
-need_string_match('next_node.mac', '^%x[02468aAcCeE]:%x%x:%x%x:%x%x:%x%x:%x%x$')
-
-if need_string_match('next_node.ip4', '^%d+.%d+.%d+.%d+$', false) then
-  need_string_match('prefix4', '^%d+.%d+.%d+.%d+/%d+$')
-end
-
-need_string_match('next_node.ip6', '^[%x:]+$', false)

package/gluon-mesh-batman-adv-core/files/lib/gluon/respondd/client.dev

@@ -1 +1 @@
-client
+local_node

package/gluon-mesh-batman-adv-core/luasrc/lib/gluon/upgrade/310-gluon-mesh-batman-adv-core-mesh

@@ -1,7 +1,6 @@
 #!/usr/bin/lua
 
 local sysconfig = require 'gluon.sysconfig'
-local sysctl = require 'gluon.sysctl'
 local site = require 'gluon.site_config'
 
 local uci = require('luci.model.uci').cursor()
@@ -24,18 +23,6 @@ uci:section('batman-adv', 'mesh', 'bat0',
 )
 uci:save('batman-adv')
 
-
-uci:add_to_set('network', 'client', 'ifname', 'bat0')
-
-uci:set('network', 'client', 'proto', 'dhcpv6')
-uci:set('network', 'client', 'reqprefix', 'no')
-uci:delete('network', 'client', 'igmp_snooping')
-uci:set('network', 'client', 'robustness', 3)
-uci:set('network', 'client', 'query_interval', 2000)
-uci:set('network', 'client', 'query_response_interval', 500)
-uci:set('network', 'client', 'peerdns', 1)
-uci:set('network', 'client', 'sourcefilter', 0)
-
 uci:delete('network', 'bat0')
 uci:section('network', 'interface', 'bat0',
 	{
@@ -47,56 +34,6 @@ uci:section('network', 'interface', 'bat0',
 	}
 )
 
-uci:delete('network', 'client_lan')
+uci:add_to_set('network', 'client', 'ifname', 'bat0')
-if sysconfig.lan_ifname then
-	uci:section('network', 'interface', 'client_lan',
-		    {
-			    unicast_flood = 0,
-		    }
-	)
-	uci:set('network', 'client_lan', 'ifname', sysconfig.lan_ifname)
-end
 
 uci:save('network')
-
-
-uci:delete('firewall', 'client')
-uci:section('firewall', 'zone', 'client',
-	    {
-		    name = 'client',
-		    network = {'client'},
-		    input = 'ACCEPT',
-		    output = 'ACCEPT',
-		    forward = 'REJECT',
-	    }
-)
-
-uci:section('firewall', 'rule', 'client_dns',
-	  {
-		  name = 'client_dns',
-		  src = 'client',
-		  dest_port = '53',
-		  target = 'REJECT',
-	  }
-)
-
-uci:save('firewall')
-
-
-local dnsmasq = uci:get_first('dhcp', 'dnsmasq')
-uci:set('dhcp', dnsmasq, 'boguspriv', 0)
-uci:set('dhcp', dnsmasq, 'localise_queries', 0)
-uci:set('dhcp', dnsmasq, 'rebind_protection', 0)
-
-uci:delete('dhcp', 'client')
-uci:section('dhcp', 'dhcp', 'client',
-	    {
-		    interface = 'client',
-		    ignore = 1,
-	    }
-)
-
-uci:save('dhcp')
-
-
-sysctl.set('net.ipv6.conf.br-client.forwarding', 0)

package/gluon-mesh-batman-adv-core/luasrc/lib/gluon/upgrade/320-gluon-mesh-batman-adv-core-client-bridge

@@ -0,0 +1,45 @@
+#!/usr/bin/lua
+
+local sysconfig = require 'gluon.sysconfig'
+local sysctl = require 'gluon.sysctl'
+
+local uci = require('luci.model.uci').cursor()
+
+uci:section('network', 'interface', 'client',
+	{
+		robustness = 3,
+		query_interval = 2000,
+		query_response_interval = 500,
+	}
+)
+uci:delete('network', 'client', 'igmp_snooping')
+
+uci:delete('network', 'client_lan')
+if sysconfig.lan_ifname then
+	uci:section('network', 'interface', 'client_lan',
+		{
+			unicast_flood = 0,
+		}
+	)
+	uci:set('network', 'client_lan', 'ifname', sysconfig.lan_ifname)
+end
+
+uci:save('network')
+
+
+uci:delete('firewall', 'client')
+uci:section('firewall', 'zone', 'client',
+	{
+		name = 'client',
+		network = {'client'},
+		input = 'ACCEPT',
+		output = 'ACCEPT',
+		forward = 'REJECT',
+	}
+)
+
+uci:delete('firewall', 'client_dns')
+
+uci:save('firewall')
+
+sysctl.set('net.ipv6.conf.br-client.forwarding')

package/gluon-mesh-batman-adv-core/luasrc/lib/gluon/upgrade/330-gluon-mesh-batman-adv-core-local-node

@@ -0,0 +1,77 @@
+#!/usr/bin/lua
+
+local site = require 'gluon.site_config'
+local sysconfig = require 'gluon.sysconfig'
+local sysctl = require 'gluon.sysctl'
+
+local uci = require('luci.model.uci').cursor()
+
+
+uci:delete('network', 'local_node_dev')
+uci:section('network', 'device', 'local_node_dev',
+	{
+		name = 'local-node',
+		ifname = 'br-client',
+		type = 'macvlan',
+		macaddr = sysconfig.primary_mac,
+	}
+)
+
+uci:delete('network', 'local_node')
+uci:section('network', 'interface', 'local_node',
+	{
+		ifname = 'local-node',
+		proto = 'dhcpv6',
+		reqprefix = 'no',
+		peerdns = 1,
+		sourcefilter = 0,
+	}
+)
+
+uci:delete('network', 'local_node_route6')
+uci:section('network', 'route6', 'local_node_route6',
+	{
+		interface = 'local-node',
+		target = site.prefix6,
+		gateway = '::',
+	}
+)
+
+uci:save('network')
+
+
+uci:delete('firewall', 'local_node')
+uci:section('firewall', 'zone', 'local_node',
+	{
+		name = 'local_node',
+		network = {'local_node'},
+		input = 'ACCEPT',
+		output = 'ACCEPT',
+		forward = 'REJECT',
+	}
+)
+
+uci:section('firewall', 'rule', 'local_node_dns',
+	{
+		name = 'local_node_dns',
+		src = 'local_node',
+		dest_port = '53',
+		target = 'REJECT',
+	}
+)
+
+uci:save('firewall')
+
+
+uci:delete('dhcp', 'local_node')
+uci:section('dhcp', 'dhcp', 'local_node',
+	{
+		interface = 'local_node',
+		ignore = 1,
+	}
+)
+
+uci:save('dhcp')
+
+
+sysctl.set('net.ipv6.conf.local_node.forwarding', 0)

package/gluon-mesh-batman-adv-core/luasrc/lib/gluon/upgrade/400-next-node

@@ -1,63 +0,0 @@
-#!/usr/bin/lua
-
-local site = require 'gluon.site_config'
-local uci = require 'luci.model.uci'
-local ip = require 'luci.ip'
-
-local c = uci.cursor()
-
-
-c:delete('network', 'local_node_dev')
-c:section('network', 'device', 'local_node_dev',
-	  {
-		  name = 'local-node',
-		  ifname = 'br-client',
-		  type = 'macvlan',
-		  macaddr = site.next_node.mac,
-	  }
-)
-
-local ip4, netmask, ip6
-
-if site.next_node.ip4 then
-	ip4 = site.next_node.ip4
-	netmask = ip.IPv4(site.prefix4):mask():string()
-end
-
-if site.next_node.ip6 then
-	ip6 = site.next_node.ip6 .. '/128'
-end
-
-c:delete('network', 'local_node')
-c:section('network', 'interface', 'local_node',
-	  {
-		  ifname = 'local-node',
-		  proto = 'static',
-		  ipaddr = ip4,
-		  netmask = netmask,
-		  ip6addr = ip6,
-	  }
-)
-
-c:delete('network', 'local_node_route6')
-c:section('network', 'route6', 'local_node_route6',
-	  {
-		  interface = 'client',
-		  target = site.prefix6,
-		  gateway = '::',
-	  }
-)
-
-c:save('network')
-
-c:delete('firewall', 'local_node')
-c:section('firewall', 'zone', 'local_node',
-	  {
-		  name = 'local_node',
-		  network = {'local_node'},
-		  input = 'ACCEPT',
-		  output = 'ACCEPT',
-		  forward = 'REJECT',
-	  }
-)
-c:save('firewall')

package/gluon-mesh-batman-adv-core/src/respondd.c

@@ -85,7 +85,7 @@ static struct json_object * get_addresses(void) {
 			   &flags, ifname) != 18)
 			continue;
 
-		if (strcmp(ifname, "br-client"))
+		if (strcmp(ifname, "local-node"))
 			continue;
 
 		if (flags & (IFA_F_TENTATIVE|IFA_F_DEPRECATED))