From fbb82a0875dacc8200d3f663c3286104d0a1afbf Mon Sep 17 00:00:00 2001
From: GomathiselviS <gomathiselvi@gmail.com>
Date: Fri, 24 Jul 2020 16:16:03 -0400
Subject: [vyos] Fix sanity issues (#58)

[vyos] Fix sanity issues

Reviewed-by: https://github.com/apps/ansible-zuul
---
 plugins/modules/vyos_interface.py | 77 ++++++++++++++++++++++++++++++++++++++-
 1 file changed, 75 insertions(+), 2 deletions(-)

(limited to 'plugins/modules/vyos_interface.py')

diff --git a/plugins/modules/vyos_interface.py b/plugins/modules/vyos_interface.py
index be844c59..839e5595 100644
--- a/plugins/modules/vyos_interface.py
+++ b/plugins/modules/vyos_interface.py
@@ -41,24 +41,28 @@ options:
   name:
     description:
     - Name of the Interface.
-    required: true
+    type: str
   description:
     description:
     - Description of Interface.
+    type: str
   enabled:
     description:
     - Interface link status.
     type: bool
+    default: True
   speed:
     description:
     - Interface link speed.
+    type: str
   mtu:
     description:
     - Maximum size of transmit packet.
+    type: int
   duplex:
     description:
     - Interface link status.
-    default: auto
+    type: str
     choices:
     - full
     - half
@@ -69,24 +73,93 @@ options:
       device. This wait is applicable for operational state argument which are I(state)
       with values C(up)/C(down) and I(neighbors).
     default: 10
+    type: int
   neighbors:
     description:
     - Check the operational state of given interface C(name) for LLDP neighbor.
     - The following suboptions are available.
+    type: list
+    elements: dict
     suboptions:
       host:
         description:
         - LLDP neighbor host for given interface C(name).
+        type: str
       port:
         description:
         - LLDP neighbor port to which given interface C(name) is connected.
+        type: str
   aggregate:
     description: List of Interfaces definitions.
+    type: list
+    elements: dict
+    suboptions:
+      name:
+        description:
+        - Name of the Interface.
+        required: true
+        type: str
+      description:
+        description:
+        - Description of Interface.
+        type: str
+      enabled:
+        description:
+        - Interface link status.
+        type: bool
+      speed:
+        description:
+        - Interface link speed.
+        type: str
+      mtu:
+        description:
+        - Maximum size of transmit packet.
+        type: int
+      duplex:
+        description:
+        - Interface link status.
+        type: str
+        choices:
+        - full
+        - half
+        - auto
+      delay:
+        description:
+        - Time in seconds to wait before checking for the operational state on remote
+          device. This wait is applicable for operational state argument which are I(state)
+          with values C(up)/C(down) and I(neighbors).
+        type: int
+      neighbors:
+        description:
+        - Check the operational state of given interface C(name) for LLDP neighbor.
+        - The following suboptions are available.
+        type: list
+        elements: dict
+        suboptions:
+          host:
+            description:
+            - LLDP neighbor host for given interface C(name).
+            type: str
+          port:
+            description:
+            - LLDP neighbor port to which given interface C(name) is connected.
+            type: str
+      state:
+        description:
+        - State of the Interface configuration, C(up) means present and operationally
+          up and C(down) means present and operationally C(down)
+        type: str
+        choices:
+        - present
+        - absent
+        - up
+        - down
   state:
     description:
     - State of the Interface configuration, C(up) means present and operationally
       up and C(down) means present and operationally C(down)
     default: present
+    type: str
     choices:
     - present
     - absent
-- 
cgit v1.2.3