167 Commits (8542f16fa932c5992743a66c6f1f28caeb5d6f7c)

Author	SHA1	Message	Date
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
Dag Wieers	417accf88d	Skip parameters that are being deprecated (#35319) ... * Skip parameters that are being deprecated So in most cases when parameters are deprecated, you may not want to advertise them in the documentation. One reason for this is because these parameters were not introduced within Ansible but predate upstreaming them. So this change avoids reporting parameters that are not documented and are deprecated. It's only a small subset of the existing E322 errors. * Add support for E323 with these changes * Unsure about the other issues now * Add better error message with E321 * Restore some erroneously remove files * Readd false positives to ignore.txt	7 years ago
Colins-Git	7fac5cd7d2	Support for creation and modification of bridge and bridge slaves for nmcli (#34071) ... Fixes: #31737	7 years ago
Dag Wieers	46e53f90a7	ACI: Rename aci_intf_policy_* to aci_interface_policy_* (#35170) ... * ACI: Rename aci_intf_policy_* to aci_interface_policy_* * Modify documentation accordingly * Fix validity ignore file	7 years ago
Dag Wieers	fa5adabcbf	aci_rest: Additional notes and simplify examples (#35091) ... Add additional notes, improve examples	7 years ago
Tim Rupp	ca8982f96c	Fixes some bugs and things (#35122) ... Removes deprecations and fixes bugs.	7 years ago
Matt Martz	f9c2c9570d	Clean up imports to prevent issues with mutable data being used in modules (#35112) ... * Clean up imports to prevent issues with mutable data being used in modules * Remove un-needed mock	7 years ago
Dag Wieers	2fd5a99989	ACI: Fix validate-module tests E322 and E323 (#35052)	7 years ago
John R Barker	9a32f5c889	Add missing extends_documentation_fragment (#35013) ... NXOS modules need more work, though that will all be in module_docs_fragments/nxos.py	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
Tim Rupp	c548ab0f18	Refactors and deprecation removals (#34830) ... This patch is primarily a refactor to make the validate-modules arg-spec no longer generate a traceback. It additionally includes removal of deprecated code in the virtual server module.	7 years ago
Tim Rupp	a10aee0fc3	Adds refactors for more f5 modules (#34824) ... The main patch is to remove the traceback generating code. There are other small fixes that were made in addition to doing that. * Removed re-def of cleanup_tokens. * Changed parameter args to be keywords. * Changed imports to include new module_util locations. * Imports also include developing (sideband) module_util locations. * Changed to using F5Client and plain AnsibleModule to prevent tracebacks caused by missing libraries. * Removed init and update methods from most Parameter classes (optimization) as its now included in module_utils. * Changed module and module param references to take into account the new self.module arg. * Minor bug fixes made during this refactor.	7 years ago
Tim Rupp	0e4e7de000	Various F5 related fixes for traceback raising (#34820) ... The main purpose of this patch is to do the refactor that supports replacing tracebacks with fail_json. Additionally, the following was done. * Removed re-def of cleanup_tokens. * Changed parameter args to be keywords. * Changed imports to include new module_util locations. * Imports also include developing (sideband) module_util locations. * Changed to using F5Client and plain AnsibleModule to prevent tracebacks caused by missing libraries. * Removed init and update methods from most Parameter classes (optimization) as its now included in module_utils. * Changed module and module param references to take into account the new self.module arg. * Minor bug fixes made during this refactor.	7 years ago
Tim Rupp	eace686044	Various fixes and changes for F5 (#34817) ... Primarily, this patch contains refactors to remove tracebacks that are generated when libraries are missing. There is also, * Removed re-def of cleanup_tokens. * Changed parameter args to be keywords. * Changed imports to include new module_util locations. * Imports also include developing (sideband) module_util locations. * Changed to using F5Client and plain AnsibleModule to prevent tracebacks caused by missing libraries. * Removed init and update methods from most Parameter classes (optimization) as its now included in module_utils. * Changed module and module param references to take into account the new self.module arg. Minor bug fixes made during this refactor.	7 years ago
Tim Rupp	44a5b3abb4	Various fixes (#34815) ... This patch was primarily an effort to reduce traceback errors for work that sivel was doing. Part of (and in some cases in addition to) that, the following was done. * Removed re-def of cleanup_tokens. * Changed parameter args to be keywords. * Changed imports to include new module_util locations. * Imports also include developing (sideband) module_util locations. * Changed to using F5Client and plain AnsibleModule to prevent tracebacks caused by missing libraries. * Removed init and update methods from most Parameter classes (optimization) as its now included in module_utils. * Changed module and module param references to take into account the new self.module arg. * Minor bug fixes made during this refactor.	7 years ago
Matt Martz	dcc05093db	Validate modules arg spec fixes (#34477) ... * Update validate-modules arg_spec introspection to be faster, by only mocking the imports we explicitly list * The use of types.MethodType in redhat_subscription wasn't py3 compatible, use partial instead * Remove argument_spec import hacks, make them errors, we can ignore them with ansible-test * Enable the --arg-spec flag for validate-modules	7 years ago
Matt Clay	2d4a43bb0a	Add file+rule ignore support to validate-modules. (#34757) ... * Clean up code. * Add file+rule ignore support to validate-modules.	7 years ago