[Fix] models

napalm/base/base.py

@@ -343,7 +343,7 @@ def get_interfaces(self) -> Dict[str, models.InterfaceDict]:
          * is_enabled (True/False)
          * description (string)
          * last_flapped (float in seconds)
-         * speed (int in Mbit)
+         * speed (float in Mbit)
          * MTU (in Bytes)
          * mac_address (string)
 
@@ -356,7 +356,7 @@ def get_interfaces(self) -> Dict[str, models.InterfaceDict]:
                 'is_enabled': False,
                 'description': '',
                 'last_flapped': -1.0,
-                'speed': 1000,
+                'speed': 1000.0,
                 'mtu': 1500,
                 'mac_address': 'FA:16:3E:57:33:61',
                 },
@@ -366,7 +366,7 @@ def get_interfaces(self) -> Dict[str, models.InterfaceDict]:
                 'is_enabled': True,
                 'description': 'foo',
                 'last_flapped': 1429978575.1554043,
-                'speed': 1000,
+                'speed': 1000.0,
                 'mtu': 1500,
                 'mac_address': 'FA:16:3E:57:33:62',
                 },
@@ -376,7 +376,7 @@ def get_interfaces(self) -> Dict[str, models.InterfaceDict]:
                 'is_enabled': True,
                 'description': 'bla',
                 'last_flapped': 1429978575.1555667,
-                'speed': 1000,
+                'speed': 1000.0,
                 'mtu': 1500,
                 'mac_address': 'FA:16:3E:57:33:63',
                 },
@@ -386,7 +386,7 @@ def get_interfaces(self) -> Dict[str, models.InterfaceDict]:
                 'is_enabled': True,
                 'description': 'bar',
                 'last_flapped': -1.0,
-                'speed': 1000,
+                'speed': 1000.0,
                 'mtu': 1500,
                 'mac_address': 'FA:16:3E:57:33:64',
                 }
@@ -1383,7 +1383,7 @@ def ping(
             OR
 
             {
-                'error': 'unknown host 8.8.8.8.8'
+                'error': 'unknown host 8.8.8.8'
             }
 
         """
@@ -1503,7 +1503,7 @@ def traceroute(
             OR
 
             {
-                'error': 'unknown host 8.8.8.8.8'
+                'error': 'unknown host 8.8.8.8'
             }
         """
         raise NotImplementedError

napalm/base/models.py

@@ -12,7 +12,7 @@
     "FactsDict",
     {
         "os_version": str,
-        "uptime": int,
+        "uptime": float,
         "interface_list": List,
         "vendor": str,
         "serial_number": str,

napalm/eos/eos.py

@@ -544,7 +544,7 @@ def get_facts(self):
             "model": version["modelName"],
             "serial_number": version["serialNumber"],
             "os_version": version["internalVersion"],
-            "uptime": int(uptime),
+            "uptime": float(uptime),
             "interface_list": interfaces,
         }
 

napalm/ios/ios.py

@@ -1043,7 +1043,7 @@ def get_facts(self):
             interface_list.append(interface)
 
         return {
-            "uptime": uptime,
+            "uptime": float(uptime),
             "vendor": vendor,
             "os_version": str(os_version),
             "serial_number": str(serial_number),

napalm/iosxr/iosxr.py

@@ -158,7 +158,7 @@ def get_facts(self):
             "vendor": "Cisco",
             "os_version": "",
             "hostname": "",
-            "uptime": -1,
+            "uptime": -1.0,
             "serial_number": "",
             "fqdn": "",
             "model": "",
@@ -185,7 +185,7 @@ def get_facts(self):
             str, napalm.base.helpers.find_txt(system_time_tree, "Hostname")
         )
         uptime = napalm.base.helpers.convert(
-            int, napalm.base.helpers.find_txt(system_time_tree, "Uptime"), -1
+            float, napalm.base.helpers.find_txt(system_time_tree, "Uptime"), -1.0
         )
         serial = napalm.base.helpers.convert(
             str, napalm.base.helpers.find_txt(platform_attr_tree, "SerialNumber")

napalm/iosxr_netconf/iosxr_netconf.py

@@ -335,7 +335,7 @@ def get_facts(self):
             "vendor": "Cisco",
             "os_version": "",
             "hostname": "",
-            "uptime": -1,
+            "uptime": -1.0,
             "serial_number": "",
             "fqdn": "",
             "model": "",
@@ -362,15 +362,15 @@ def get_facts(self):
 
         # Retrieves uptime
         uptime = napalm.base.helpers.convert(
-            int,
+            float,
             self._find_txt(
                 facts_rpc_reply_etree,
                 ".//suo:system-time/\
             suo:uptime/suo:uptime",
                 default="",
                 namespaces=C.NS,
             ),
-            -1,
+            -1.0,
         )
 
         # Retrieves interfaces name

napalm/junos/junos.py

@@ -412,7 +412,7 @@ def get_facts(self):
             "os_version": str(output["version"]),
             "hostname": str(output["hostname"]),
             "fqdn": str(output["fqdn"]),
-            "uptime": uptime,
+            "uptime": float(uptime),
             "interface_list": interface_list,
         }
 

napalm/nxos/nxos.py

@@ -966,7 +966,7 @@ def get_facts(self) -> models.FactsDict:
         uptime += uptime_mins * 60
         uptime += uptime_secs
 
-        facts["uptime"] = uptime
+        facts["uptime"] = float(uptime)
 
         iface_cmd = "show interface"
         interfaces_out = self._send_command(iface_cmd)

napalm/nxos_ssh/nxos_ssh.py

@@ -662,7 +662,7 @@ def get_facts(self):
             interface_list.append(helpers.canonical_interface_name(interface))
 
         return {
-            "uptime": int(uptime),
+            "uptime": float(uptime),
             "vendor": vendor,
             "os_version": str(os_version),
             "serial_number": str(serial_number),