[openvpn] Differentiate server between manual, routed and bridged

Author: nemesifier

netjsonconfig/backends/openvpn/openvpn.py

@@ -74,6 +74,9 @@ def auto_client(self, host, server, ca_path=None, ca_contents=None,
             c['proto'] = 'tcp-client'
         else:
             c['proto'] = 'udp'
+        # determine if pull must be True
+        if 'server' in server or 'server_bridge' in server:
+            c['pull'] = True
         # tls_client
         if 'tls_server' not in server or not server['tls_server']:
             c['tls_client'] = False

netjsonconfig/backends/openvpn/renderers.py

@@ -20,6 +20,9 @@ def _transform_vpn(self, vpn):
         config = deepcopy(vpn)
         skip_keys = ['script_security', 'remote']
         delete_keys = []
+        # allow server_bridge to be empty and still rendered
+        if config.get('server_bridge') == '':
+            config['server_bridge'] = True
         for key, value in config.items():
             if key in skip_keys:
                 continue