184 Commits (c3c86a02483ce6007c1e68fc396722b1f45c24b7)

Author	SHA1	Message	Date
Troy Murray	15ce7c5bab	change OS X to macOS (#41294) ... * change OS X to macOS <!--- Your description here --> +label: docsite_pr * Update all Mac OS X references to be macOS * Drop extra Mac	6 years ago
Andrea Tartaglia	a342538aba	Add shell out checks (#41545) ... * Added error codes for shell_out checks * Added ignore lines for allowed Modules * Added shell out checks * Fixed pep8 * Updated regex to only match subprocess.Popen * Added failing modules to ignore.txt * Wrong postgresql module in ignore.txt * Removed bigip from ignore.txt	6 years ago
Abhijeet Kasurde	27b85e732d	Argument spec must be dict/hash (#40858) ... validate-modules should fail when argument is not dict/hash. Signed-off-by: Abhijeet Kasurde <akasurde@redhat.com>	6 years ago
Matt Clay	70c475da6c	Implement new changelog generator.	7 years ago
Sam Doran	8746d9ef87	Add guidelines on developing lookup plugins (#38681)	7 years ago
Matt Clay	8d39515914	Remove tab check from validate-modules. ... It is redundant with the pycodestyle W191 "indentation contains tabs" check.	7 years ago
Simon	fb6ae90f60	Memset: initial module, integration tests, module_utils and supporting documentation. (#40071) ... * * Memset: * module_utils and associated documentation. * module to manage DNS zones. * integration tests to run against Memset's API. * * memset.py: * remove import of requests from memset.py in favour of internal Ansible modules. * import necessary Ansible modules for request and response handling. * create a response object in order to retain a stable interface to memset_api_call from existing modules. * rework memset_api_call to remove requests. * memset_zone.py: * improve short description to be more verbose. * ensure all imports directly follow documentation. * remove any reference to requests. * correct keyerror carried over from elsewhere. * remove dependency on requests from documentation string * disable integration tests until we have a cleanup method available	7 years ago
Jon Dufresne	1d640182c6	Update all pypi.python.org URLs to pypi.org (#38988) ... For details on the new PyPI, see the blog post: https://pythoninsider.blogspot.ca/2018/04/new-pypi-launched-legacy-pypi-shutting.html	7 years ago
Ken Celenza	e476089bb9	spacing issue (#39884)	7 years ago
Matt Martz	1663b64e18	Allow subspec defaults to be processed when the parent argument is not supplied (#38967) ... * Allow subspec defaults to be processed when the parent argument is not supplied * Allow this to be configurable via apply_defaults on the parent * Document attributes of arguments in argument_spec * Switch manageiq_connection to use apply_defaults * add choices to api_version in argument_spec	7 years ago
Kevin Breit	c8d287fece	Meraki organization module (#38773) ... * Initial commit Query an organization within Meraki. No support is in place for managing or creating yet * Change output_level method and make the state parameter required. * Implemented listing all organizations - Updated documentation - Parse results and return all organizations - Parse results and return specified organization * Framework for creating an organization - Documentation example for organization creation - Framework exists for creating organizations, pending PR 36809 - Created functions for HTTP calls - Renamed from dashboard.meraki.com to api.meraki.com - Added required_if for state * Remove absent state - Meraki API does not support deleting an organization so absent is removed - Updated documentation to call it state instead of status * Small change to documentation * Support all parameters associated to organization - Added all parameters needed for all organization actions. - None of the added ones work at this time. - Added documentation for clone. * Integration test for meraki_organization module * Rename module to meraki for porting to module utility * Meraki documentation fragment - Created initial documentation fragment for Meraki modules * Add meraki module utility to branch. Formerly was on a separate branch. * CRU support for Meraki organization module * CRU is supported for Meraki organizations * There is no DELETE function for organizations in the API * This code is very messy and needs cleanup * Create and Update actions don't show status as updated, must fix * Added Meraki module utility to module utility documentation list * Added support for organization cloning * Renamed use_ssl to use_https * Removed define_method() * Removed is_org() * Added is_org_valid() which does all org sanity checks * Fixes for ansibot - Changed default of use_proxy from true to false - Removed some commented out code - Updated documentation * Changes for ansibot - Removed requirement for state parameter. I may readd this. - Updated formatting diff --git a/lib/ansible/module_utils/network/meraki/meraki.py b/lib/ansible/module_utils/network/meraki/meraki.py index 3acd3d1038..395ac7c4b4 100644 --- a/lib/ansible/module_utils/network/meraki/meraki.py +++ b/lib/ansible/module_utils/network/meraki/meraki.py @@ -42,7 +42,7 @@ def meraki_argument_spec(): return dict(auth_key=dict(type='str', no_log=True, fallback=(env_fallback, ['MERAKI_KEY'])), host=dict(type='str', default='api.meraki.com'), name=dict(type='str'), - state=dict(type='str', choices=['present', 'absent', 'query'], required=True), + state=dict(type='str', choices=['present', 'absent', 'query']), use_proxy=dict(type='bool', default=False), use_https=dict(type='bool', default=True), validate_certs=dict(type='bool', default=True), diff --git a/lib/ansible/modules/network/meraki/meraki_organization.py b/lib/ansible/modules/network/meraki/meraki_organization.py index 923d969366..3789be91d6 100644 --- a/lib/ansible/modules/network/meraki/meraki_organization.py +++ b/lib/ansible/modules/network/meraki/meraki_organization.py @@ -20,11 +20,9 @@ short_description: Manage organizations in the Meraki cloud version_added: "2.6" description: - Allows for creation, management, and visibility into organizations within Meraki - notes: - More information about the Meraki API can be found at U(https://dashboard.meraki.com/api_docs). - Some of the options are likely only used for developers within Meraki - options: name: description: @@ -32,21 +30,18 @@ options: - If C(clone) is specified, C(name) is the name of the new organization. state: description: - - Create or query organizations - choices: ['query', 'present'] + - Create or modify an organization + choices: ['present', 'query'] clone: description: - Organization to clone to a new organization. - type: string org_name: description: - Name of organization. - Used when C(name) should refer to another object. - type: string org_id: description: - ID of organization - author: - Kevin Breit (@kbreit) extends_documentation_fragment: meraki @@ -86,7 +81,6 @@ RETURN = ''' response: description: Data returned from Meraki dashboard. type: dict - state: query returned: info ''' @@ -103,6 +97,7 @@ def main(): argument_spec = meraki_argument_spec() argument_spec.update(clone=dict(type='str'), + state=dict(type='str', choices=['present', 'query']), ) @@ -125,11 +120,9 @@ def main(): meraki.function = 'organizations' meraki.params['follow_redirects'] = 'all' - meraki.required_if=[ - ['state', 'present', ['name']], - ['clone', ['name']], - # ['vpn_PublicIP', ['name']], - ] + meraki.required_if = [['state', 'present', ['name']], + ['clone', ['name']], + ] create_urls = {'organizations': '/organizations', } @@ -162,23 +155,16 @@ def main(): - - # method = None - # org_id = None - - - # meraki.fail_json(msg=meraki.is_org_valid(meraki.get_orgs(), org_name='AnsibleTestOrg')) - if meraki.params['state'] == 'query': - if meraki.params['name'] is None: # Query all organizations, no matter what - orgs = meraki.get_orgs() - meraki.result['organization'] = orgs - elif meraki.params['name'] is not None: # Query by organization name - module.warn('All matching organizations will be returned, even if there are duplicate named organizations') - orgs = meraki.get_orgs() - for o in orgs: - if o['name'] == meraki.params['name']: - meraki.result['organization'] = o + if meraki.params['name'] is None: # Query all organizations, no matter what + orgs = meraki.get_orgs() + meraki.result['organization'] = orgs + elif meraki.params['name'] is not None: # Query by organization name + module.warn('All matching organizations will be returned, even if there are duplicate named organizations') + orgs = meraki.get_orgs() + for o in orgs: + if o['name'] == meraki.params['name']: + meraki.result['organization'] = o elif meraki.params['state'] == 'present': if meraki.params['clone'] is not None: # Cloning payload = {'name': meraki.params['name']} @@ -193,7 +179,10 @@ def main(): payload = {'name': meraki.params['name'], 'id': meraki.params['org_id'], } - meraki.result['response'] = json.loads(meraki.request(meraki.construct_path('update', org_id=meraki.params['org_id']), payload=json.dumps(payload), method='PUT')) + meraki.result['response'] = json.loads(meraki.request(meraki.construct_path('update', + org_id=meraki.params['org_id']), + payload=json.dumps(payload), + method='PUT')) diff --git a/lib/ansible/utils/module_docs_fragments/meraki.py b/lib/ansible/utils/module_docs_fragments/meraki.py index e268d02e68..3569d83b99 100644 --- a/lib/ansible/utils/module_docs_fragments/meraki.py +++ b/lib/ansible/utils/module_docs_fragments/meraki.py @@ -35,6 +35,7 @@ options: description: - Set amount of debug output during module execution choices: ['normal', 'debug'] + default: 'normal' timeout: description: - Time to timeout for HTTP requests. diff --git a/test/integration/targets/meraki_organization/aliases b/test/integration/targets/meraki_organization/aliases new file mode 100644 index 0000000000..ad7ccf7ada --- /dev/null +++ b/test/integration/targets/meraki_organization/aliases @@ -0,0 +1 @@ +unsupported * Formatting fix * Minor updates due to testing - Made state required again - Improved formatting for happier PEP8 - request() now sets instance method * Fix reporting of the result * Enhance idempotency checks - Remove merging functionality as the proposed should be used - Do check and reverse check to look for differences * Rewrote and added additional integration tests. This isn't done. * Updated is_update_required method: - Original and proposed data is passed to method - Added ignored_keys list so it can be skipped if needed * Changes per comments from dag - Optionally assign function on class instantiation - URLs now have {} for substitution method - Move auth_key check to module utility - Remove is_new and get_existing - Minor changes to documentation * Enhancements for future modules and organization - Rewrote construct_path method for simplicity - Increased support for network functionality to be committed * Changes based on Dag feedback and to debug problems * Minor fixes for validitation testing * Small changes for dag and Ansibot - Changed how auth_key is processed - Removed some commented lines - Updated documentation fragment, but that may get reverted * Remove blank line and comment * Improvements for testing and code simplification - Added network integration tests - Modified error handling in request() - More testing to come on this - Rewrote construct_path again. Very simple now. * Remove trailing whitespace * Small changes based on dag's response * Removed certain sections from exit_json and fail_json as they're old	7 years ago
Alicia Cozine	c8a9b411bc	Last docs link fixes (#39391) ... * should not need <>, but fails without * adds anchor to keywords page, uses it on plugins pages * fixes envvar link errors * harmonize file name and ref name as python_3 * removes undefined-lable from ignore list	7 years ago
Abhijeet Kasurde	9f1254a32f	Typo fix in testing link (#39407) ... Signed-off-by: Abhijeet Kasurde <akasurde@redhat.com>	7 years ago
Alicia Cozine	4b52a54e18	Reduce warnings (#39254) ... * removes FAQ links; no entries exist for linked config settings * fixes various anchors and links * addresses abadger comments, thanks * marks orphan pages, avoids TOC errors * adds links for remote_tmp setting to FAQ	7 years ago
Matt Clay	7975c8fab0	Improve integration-aliases docs. (#39087)	7 years ago
Alicia Cozine	8f83061e70	corrects links to 'list_of' pages (#39161)	7 years ago
Matt Clay	c262dbfd30	Use https for links to ansible.com domains.	7 years ago
Alicia Cozine	bbfd6c6ab1	Internal refs (#39094) ... * fixes community refs * fixes appendix refs * fixes scenario refs, keeps ACI guide link to devel * fixes windows refs * fixes user guide refs * fixes dev guide refs	7 years ago
Alicia Cozine	64dcc7def0	Fixes more rST errors (#39042) ... * removes include, avoids duplicate label rST warnings * reduces literal_block errors on playbooks_filters * addresses TOC errors * ref error on modules page * trying ref for list_of_database_modules	7 years ago
Toshio Kuratomi	9faf7b949e	Fix places in docs that refer to modules without namespace ... We've namespaced all plugin docs. Change the docs to reflect that	7 years ago
Matt Clay	3d57f45815	Update httptester docs.	7 years ago
Rémy Léone	d639d5dc96	Fix typo (#38652) ... Typo +label: docsite_pr	7 years ago
Matt Clay	26fa3adeab	Add docs-build sanity test.	7 years ago
Pierre Templier	1e0df01980	Small typos in docs (#38371) ... * Small typos in docs * Use same wording as in porting_guide_2.5.rst	7 years ago
ewubnhn	ea9ece8e72	Update spelling_word_choice.rst (#38113) ... Added missing format string on "Installer" label: docsite_pr	7 years ago
Sam Doran	b5b3beff83	Correct syntax highlighting in plugin dev docs (#38184) ... Use rst em dash	7 years ago
Matt Martz	ffbbb5a25b	Use arg_spec type for comparisons on default and choices (#37741) ... * Use arg_spec type for comparisons on default and choices * Further improve type casting * Make sure to capture output in more places * Individually report invalid choices * Update ignore.txt after resolving merge conflicts	7 years ago
scottb	381359a8f8	Doc build warning/broken link clean-a-palooza (#37382) ... * Doc build warning/broken link clean-a-palooza, WIP commit 1. * Fixed broken anchor * Fixing additional broken links; converting from doc to ref. * Fix anchor	7 years ago
Alexander Lex	19f92005b9	Update developing_api.rst (#37108) ... Fix for #17716 and #36110	7 years ago
Adam Miller	3fd5b0740e	allow ANSIBLE_KEEP_REMOTE_FILES for local test runner (#33357) ... * allow ANSIBLE_KEEP_REMOTE_FILES for local test runner * add ANSIBLE_KEEP_REMOTE_FILES to tox.ini, update docs * Clarify handling of environment variables.	7 years ago
John R Barker	985f09270d	Ability to link to other pages from plugin docs (#37009) ... Support relative links	7 years ago
scottb	e7f9c5cc60	Copy edits and tweaks	7 years ago
Brian Coca	cf43e0d605	updated api example ... - added many more comments - removed very obsolete 1.x refs	7 years ago
Kevin Breit	487ff1fe9b	Fix typos (#36671) ... Fix minor typos	7 years ago
Brian Coca	3eff279dd7	updates to module testing (#36043) ... * updates to module testing gives those using internal modules an alternative * Copy edit	7 years ago
Matt Martz	50adc5409b	Add several new doc<->arg_spec checks (#36247) ... * Add several new doc<->arg_spec checks. See #18183 * Update ignore.txt for validate-modules	7 years ago
David Newswanger	3f5caf659e	added support for --testcase flag in ansible-test (#36134) ... * added support for --testcase flag in ansible-test * fixed command format * added tab completion * fixed sanity issues * added documenation for --testcase * don't autocomplete when multiple modules are selected	7 years ago
scottb	373b1dcf59	Core Docs Refactor and Redesign (#36067) ... * Docs refactor as outlined in https://github.com/ansible/proposals/issues/79. Moves content into 'guides'; refactors TOC; fixes CSS; design tweaks to layout and CSS; fixes generated plugin, CLI and module docs to fix links accodingly; more. * Adding extra blank line for shippable	7 years ago
Rvgate	d5ae63c0ea	Change wording of Going Further section (#35949) ... * Change wording of Going Further section The original wording was confusing with its non-devel branch part. Made it much clearer for people to understand what to do. Also removed the git command for creating branches, I believe a develop should be aware of how basic git commands work. * Edited for concision and clarity. * Typo fix * Typo fix	7 years ago
John R Barker	c3523cdd60	Test against a single transport (#35903) ... Network integration tests can use tags to run only a specific transport	7 years ago
Joshua Smith	0c012703c1	Update developing inventory script docs. (#35639) ... * Update developing inventory script docs. * Copy edit.	7 years ago
Alicia Cozine	adf76b7d3f	Reflect new process for network integration tests (#35577) ... With `ansible-test network-integration` we don't need the top level yaml files except NXOS and Dell (for now)	7 years ago
John R Barker	a23c95023b	Module deprecation: docs, scheme and tests (#34100) ... Enforce module deprecation. After module has reached the end of it's deprecation cycle we will replace it with a docs stub. * Replace deprecated modules with docs-only sub * Use of deprecated past deprecation cycle gives meaningful message (see examples below) * Enforce documentation.deprecation dict via `schema.py` * Update `ansible-doc` and web docs to display documentation.deprecation * Document that structure in `dev_guide` * Ensure that all modules starting with `_` have a `deprecation:` block * Ensure `deprecation:` block is only used on modules that start with `_` * `removed_in` A string which represents when this module needs **deleting** * CHANGELOG.md and porting_guide_2.5.rst list removed modules as well as alternatives * CHANGELOG.md links to porting guide index To ensure that meaningful messages are given to the user if they try to use a module at the end of it's deprecation cycle we enforce the module to contain: ```python if __name__ == '__main__': removed_module() ```	7 years ago
Jordan Borean	4da19f2d6b	Added code-smell check to verify filenames are allowed with Windows (#35436)	7 years ago
Matt Clay	7abdab6c9e	Convert ansible-test compile into a sanity test.	7 years ago
Matt Clay	e0010f15e4	Add pslint sanity test. (#35303) ... * Add pslint sanity test. * Fix `no-smart-quotes` sanity test. * Add docs for `pslint` sanity test.	7 years ago
Matt Martz	3c400c5899	ANSIBLE_METADATA doesn't support YAML currently, change 315 to invalid data type (#35009)	7 years ago
Matt Martz	a352d43824	Compare arg+aliases between docs and argument_spec (#34809) ... * compare arg+aliases between docs and argument_spec * Add some special handling for the network modules provider options that also appear in the top level arg spec * Fix error code for bigip_hostname * Address merge conflicts due to changes in f5 modules * Update validate-modules ignore based off a clean execution * Address merge conflicts * Address renamed module * Address recent changes to modules * Add ignore for ucs_ip_pool * Update aci modules to get more reliable documentation comparison, but not mutating the module_utils aci_argument_spec * Update ignore.txt after recent aci updates * Add extra guard to ensure we handle provider special only for network modules * Address additional changes to modules	7 years ago
Alexander Lex	59042d79f5	Update developing_api.rst to use correct inventory format for single explicit host (#34919) ... Fix for #34918	7 years ago
Brian Coca	bbd6b8bb42	Temporary (#31677) ... * allow shells to have per host options, remote_tmp added language to shell removed module lang setting from general as plugins have it now use get to avoid bad powershell plugin more resilient tmp discovery, fall back to `pwd` add shell to docs fixed options for when frags are only options added shell set ops in t_e and fixed option frags normalize tmp dir usag4e - pass tmpdir/tmp/temp options as env var to commands, making it default for tempfile - adjusted ansiballz tmpdir - default local tempfile usage to the configured local tmp - set env temp in action add options to powershell shift temporary to internal envvar/params ensure tempdir is set if we pass var ensure basic and url use expected tempdir ensure localhost uses local tmp give /var/tmp priority, less perms issues more consistent tempfile mgmt for ansiballz made async_dir configurable better action handling, allow for finally rm tmp fixed tmp issue and no more tempdir in ballz hostvarize world readable and admin users always set shell tempdir added comment to discourage use of exception/flow control * Mostly revert expand_user as it's not quite working. This was an additional feature anyhow. Kept the use of pwd as a fallback but moved it to a second ssh connection. This is not optimal but getting that to work in a single ssh connection was part of the problem holding this up. (cherry picked from commit 395b714120522f15e4c90a346f5e8e8d79213aca) * fixed script and other action plugins ensure tmpdir deletion allow for connections that don't support new options (legacy, 3rd party) fixed tests	7 years ago