From 65e32243b832d297c7670f3538e41db65bdf9698 Mon Sep 17 00:00:00 2001 From: Ted Date: Sat, 11 Feb 2017 07:41:24 -0800 Subject: [PATCH] =?UTF-8?q?Make=20access=5Ftoken=20type=20str,=20remove=20?= =?UTF-8?q?alias,=20and=20make=20validate=5Fcerts=20def=E2=80=A6=20(#20811?= =?UTF-8?q?)?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit * Make access_token type str, remove alias, and make validate_certs default value true * Remove extra white space and add end of file newline * Remove comma * Try removing that whitespace again --- .../modules/network/bigswitch/bigmon_chain.py | 13 +++++-------- 1 file changed, 5 insertions(+), 8 deletions(-) diff --git a/lib/ansible/modules/network/bigswitch/bigmon_chain.py b/lib/ansible/modules/network/bigswitch/bigmon_chain.py index 6004578afa..35c7b897d7 100755 --- a/lib/ansible/modules/network/bigswitch/bigmon_chain.py +++ b/lib/ansible/modules/network/bigswitch/bigmon_chain.py @@ -2,7 +2,7 @@ # -*- coding: utf-8 -*- # Ansible module to manage Big Monitoring Fabric service chains -# (c) 2016, Ted Elhourani , +# (c) 2016, Ted Elhourani # # This file is part of Ansible # @@ -53,11 +53,7 @@ options: choices: [true, false] access_token: description: - - Bigmon access token. - required: false - -notes: - - An environment variable can be used, BIGSWITCH_ACCESS_TOKEN. + - Bigmon access token. If this isn't set the the environment variable C(BIGSWITCH_ACCESS_TOKEN) is used. ''' @@ -67,6 +63,7 @@ EXAMPLES = ''' name: MyChain controller: '{{ inventory_hostname }}' state: present + validate_certs: false ''' @@ -144,8 +141,8 @@ def main(): name=dict(type='str', required=True), controller=dict(type='str', required=True), state=dict(choices=['present', 'absent'], default='present'), - validate_certs=dict(type='bool', default='False'), - access_token=dict(aliases=['BIGSWITCH_ACCESS_TOKEN'], no_log=True) + validate_certs=dict(type='bool', default='True'), + access_token=dict(type='str', no_log=True) ) )