From fb5284fb6da3d47d0657bd7a885087fcb94ea762 Mon Sep 17 00:00:00 2001
From: Edoardo Putti <edoardo.putti@gmail.com>
Date: Mon, 24 Jul 2017 15:43:50 +0200
Subject: [PATCH] [airos] updated ntpclient converter to use ntp schema from
 openwrt

---
 netjsonconfig/backends/airos/converters.py | 37 ++++++++++++++--------
 1 file changed, 24 insertions(+), 13 deletions(-)

diff --git a/netjsonconfig/backends/airos/converters.py b/netjsonconfig/backends/airos/converters.py
index b229e8ebd..055b0ce48 100644
--- a/netjsonconfig/backends/airos/converters.py
+++ b/netjsonconfig/backends/airos/converters.py
@@ -349,22 +349,33 @@ def to_intermediate(self):
 
 
 class Ntpclient(AirOsConverter):
-    netjson_key = 'ntp_servers'
+    netjson_key = 'ntp'
+
+    default_servers = [
+        "0.openwrt.pool.ntp.org",
+        "1.openwrt.pool.ntp.org",
+        "2.openwrt.pool.ntp.org",
+        "3.openwrt.pool.ntp.org",
+    ]
+
+    def ntp_status(self, ntp):
+        if ntp.get('enabled', True):
+            return 'enabled'
+        else:
+            return 'disabled'
 
     def to_intermediate(self):
         result = []
-        temp = []
-        original = get_copy(self.netjson, self.netjson_key, [])
-        if original:
-            for ntp in original:
-                temp.append({
-                    'server': ntp,
-                    'status': 'enabled',
-                })
-            result.append(temp)
-            result.append({'status': 'enabled'})
-        else:
-            result.append({'status': 'disabled'})
+        servers = []
+        original = get_copy(self.netjson, self.netjson_key, {})
+        result.append({'status': self.ntp_status(original)})
+
+        for ntp in original.get('servers', self.default_servers):
+            servers.append({
+                'server': ntp,
+                'status': 'enabled',
+            })
+        result.append(servers)
         return (('ntpclient', result),)