Minor fixes and enhancements to bigip modules (#44549)

Various cleanup and fixes in docs and iterations.
This commit is contained in:
Tim Rupp 2018-08-23 12:50:30 -04:00 committed by GitHub
parent 59e25bbe5b
commit 5a020e7170
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
4 changed files with 21 additions and 19 deletions

View file

@ -341,6 +341,7 @@ class ArgumentSpec(object):
def main(): def main():
client = None
spec = ArgumentSpec() spec = ArgumentSpec()
module = AnsibleModule( module = AnsibleModule(
@ -357,7 +358,8 @@ def main():
cleanup_tokens(client) cleanup_tokens(client)
module.exit_json(**results) module.exit_json(**results)
except F5ModuleError as ex: except F5ModuleError as ex:
cleanup_tokens(client) if client:
cleanup_tokens(client)
module.fail_json(msg=str(ex)) module.fail_json(msg=str(ex))

View file

@ -17,7 +17,7 @@ DOCUMENTATION = r'''
module: bigip_profile_udp module: bigip_profile_udp
short_description: Manage UDP profiles on a BIG-IP short_description: Manage UDP profiles on a BIG-IP
description: description:
- Manage UDP profiles on a BIG-IP. There are a variety of UDP profiles, each with their - Manage UDP profiles on a BIG-IP. Many of UDP profiles exist; each with their
own adjustments to the standard C(udp) profile. Users of this module should be aware own adjustments to the standard C(udp) profile. Users of this module should be aware
that many of the adjustable knobs have no module default. Instead, the default is that many of the adjustable knobs have no module default. Instead, the default is
assigned by the BIG-IP system itself which, in most cases, is acceptable. assigned by the BIG-IP system itself which, in most cases, is acceptable.

View file

@ -342,7 +342,7 @@ class ModuleManager(object):
# Sleep a little to let rebooting take effect # Sleep a little to let rebooting take effect
time.sleep(20) time.sleep(20)
while nops < 6: while nops < 3:
try: try:
self.client.reconnect() self.client.reconnect()
next_reboot = self._get_last_reboot() next_reboot = self._get_last_reboot()

View file

@ -78,7 +78,7 @@ author:
EXAMPLES = r''' EXAMPLES = r'''
- name: Disable the setup utility - name: Disable the setup utility
bigip_sys_global: bigip_sys_global:
gui_setup: disabled gui_setup: no
password: secret password: secret
server: lb.mydomain.com server: lb.mydomain.com
user: admin user: admin
@ -93,9 +93,9 @@ banner_text:
type: string type: string
sample: This is a corporate device. Do not touch. sample: This is a corporate device. Do not touch.
console_timeout: console_timeout:
description: > description:
The new number of seconds of inactivity before the system - The new number of seconds of inactivity before the system
logs off a user that is logged on. logs off a user that is logged on.
returned: changed returned: changed
type: int type: int
sample: 600 sample: 600
@ -103,36 +103,36 @@ gui_setup:
description: The new setting for the Setup utility. description: The new setting for the Setup utility.
returned: changed returned: changed
type: string type: string
sample: enabled sample: yes
lcd_display: lcd_display:
description: The new setting for displaying the system menu on the LCD. description: The new setting for displaying the system menu on the LCD.
returned: changed returned: changed
type: string type: string
sample: enabled sample: yes
mgmt_dhcp: mgmt_dhcp:
description: The new setting for whether the mgmt interface should DHCP or not. description: The new setting for whether the mgmt interface should DHCP or not.
returned: changed returned: changed
type: string type: string
sample: enabled sample: yes
net_reboot: net_reboot:
description: The new setting for whether the system should boot to an ISO on the network or not. description: The new setting for whether the system should boot to an ISO on the network or not.
returned: changed returned: changed
type: string type: string
sample: enabled sample: yes
quiet_boot: quiet_boot:
description: > description:
The new setting for whether the system should suppress information to - The new setting for whether the system should suppress information to
the console during boot or not. the console during boot or not.
returned: changed returned: changed
type: string type: string
sample: enabled sample: yes
security_banner: security_banner:
description: > description:
The new setting for whether the system should display an advisory message - The new setting for whether the system should display an advisory message
on the login screen or not. on the login screen or not.
returned: changed returned: changed
type: string type: string
sample: enabled sample: yes
''' '''
from ansible.module_utils.basic import AnsibleModule from ansible.module_utils.basic import AnsibleModule