pppoe: T1318: migrate user-id and password nodes under an authentication node

2 files changed, 25 insertions, 14 deletions

diff --git a/src/conf_mode/interfaces-pppoe.py b/src/conf_mode/interfaces-pppoe.py
index 49902b989..d2147fa1f 100755
--- a/src/conf_mode/interfaces-pppoe.py
+++ b/src/conf_mode/interfaces-pppoe.py
@@ -78,9 +78,9 @@ replacedefaultroute
 {% endif %}
 mtu {{ mtu }}
 mru {{ mtu }}
-user "{{ user_id }}"
-password "{{ password }}"
-{% if 'auto' in name_server -%}
+user "{{ auth_username }}"
+password "{{ auth_password }}"
+{% if name_server -%}
 usepeerdns
 {% endif %}
 {% if ipv6_enable -%}
@@ -91,6 +91,8 @@ usepeerdns
 
 default_config_data = {
     'access_concentrator': '',
+    'auth_username': '',
+    'auth_password': '',
     'on_demand': False,
     'default_route': 'auto',
     'deleted': False,
@@ -103,10 +105,8 @@ default_config_data = {
     'local_address': '',
     'mtu': '1492',
     'name_server': 'auto',
-    'password': '',
     'remote_address': '',
     'service_name': '',
-    'user_id': '',
     'source_interface': ''
 }
 
@@ -137,6 +137,14 @@ def get_config():
     if conf.exists(['access-concentrator']):
         pppoe['access_concentrator'] = conf.return_values(['access-concentrator'])
 
+    # Authentication name supplied to PPPoE server
+    if conf.exists(['authentication', 'user']):
+        pppoe['auth_username'] = conf.return_value(['authentication', 'user'])
+
+    # Password for authenticating local machine to PPPoE server
+    if conf.exists(['authentication', 'password']):
+        pppoe['auth_password'] = conf.return_value(['authentication', 'password'])
+
     # Access concentrator name (only connect to this concentrator)
     if conf.exists(['connect-on-demand']):
         pppoe['on_demand'] = True
@@ -181,11 +189,7 @@ def get_config():
     if conf.exists(['name-server']):
         pppoe['name_server'] = conf.return_value(['name-server'])
 
-    # Password for authenticating local machine to PPPoE server
-    if conf.exists(['password']):
-        pppoe['password'] = conf.return_value(['password'])
-
-    # IPv4 address of local end of the PPPoE link
+    # IPv4 address for remote end of PPPoE session
     if conf.exists(['remote-address']):
         pppoe['remote_address'] = conf.return_value(['remote-address'])
 
@@ -193,10 +197,6 @@ def get_config():
     if conf.exists(['service-name']):
         pppoe['service_name'] = conf.return_value(['service-name'])
 
-    # Authentication name supplied to PPPoE server
-    if conf.exists(['user-id']):
-        pppoe['user_id'] = conf.return_value(['user-id'])
-
     return pppoe
 
 def verify(pppoe):
diff --git a/src/migration-scripts/interfaces/4-to-5 b/src/migration-scripts/interfaces/4-to-5
index 0683861f2..19e0352c8 100755
--- a/src/migration-scripts/interfaces/4-to-5
+++ b/src/migration-scripts/interfaces/4-to-5
@@ -19,6 +19,17 @@ def migrate_dialer(config, tree, intf):
         # parts
         config.copy(tree + [pppoe], pppoe_base)
 
+        # Migrate user-id and password nodes under an 'authentication'
+        # node
+        if config.exists(pppoe_base + ['user-id']):
+            user = config.return_value(pppoe_base + ['user-id'])
+            config.set(pppoe_base + ['authentication', 'user'], value=user)
+            config.delete(pppoe_base + ['user-id'])
+
+        if config.exists(pppoe_base + ['password']):
+            pwd = config.return_value(pppoe_base + ['password'])
+            config.set(pppoe_base + ['authentication', 'password'], value=pwd)
+            config.delete(pppoe_base + ['password'])
 
         # remove enable-ipv6 node and rather place it under ipv6 node
         if config.exists(pppoe_base + ['enable-ipv6']):