Information for submitting a group of modules¶
This section discusses how to get multiple related modules into Ansible.
This document is intended for both companies wishing to add modules for their own products as well as users of 3rd party products wishing to add Ansible functionality.
It’s based on module development tips and tricks that the Ansible core team and community have accumulated.
LICENSING REQUIREMENTS Ansible enforces the following licensing requirements:
- Utilities (files in
lib/ansible/module_utils/) may have one of two licenses:
A file in
module_utilsused only for a specific vendor’s hardware, provider, or service may be licensed under GPLv3+. Adding a new file under
module_utilswith GPLv3+ needs to be approved by the core team.
module_utilsmust be licensed under BSD, so GPL-licensed third-party and Galaxy modules can use them.
If there’s doubt about the appropriate license for a file in
module_utils, the Ansible Core Team will decide during an Ansible Core Community Meeting.
- Utilities (files in
All other files shipped with Ansible, including all modules, must be licensed under the GPL license (GPLv3 or later).
Although it’s tempting to get straight into coding, there are a few things to be aware of first. This list of prerequisites is designed to help ensure that you develop high-quality modules that flow easily through the review process and get into Ansible more quickly.
New modules must be PEP 8 compliant. See PEP 8 for more information.
Starting with Ansible version 2.7, all new modules must support Python 2.7+ and Python 3.5+. If this is an issue, please contact us (see the “Speak to us” section later in this document to learn how).
Have a look at the existing modules and how they’ve been named in the All modules, especially in the same functional area (such as cloud, networking, databases).
Shared code can be placed into
Shared documentation (for example describing common arguments) can be placed in
With great power comes great responsibility: Ansible module maintainers have a duty to help keep modules up to date. As with all successful community projects, module maintainers should keep a watchful eye for reported issues and contributions.
Although not required, unit and/or integration tests are strongly recommended. Unit tests are especially valuable when external resources (such as cloud or network devices) are required. For more information see Testing Ansible and the Testing Working Group. * Starting with Ansible 2.4 all Network modules MUST have unit tests.
As you may have noticed when looking under
lib/ansible/modules/ we support up to two directories deep (but no deeper), e.g. databases/mysql. This is used to group files on disk as well as group related modules into categories and topics the Module Index, for example: Database modules.
The directory name should represent the product or OS name, not the company name.
Each module should have the above (or similar) prefix; see existing All modules for existing examples.
File and directory names are always in lower case
Words are separated with an underscore (
Module names should be in the singular, rather than plural, eg
Circulating your ideas before coding is a good way to help you set off in the right direction.
After reading the “Before you start coding” section you will hopefully have a reasonable idea of the structure of your modules.
We’ve found that writing a list of your proposed module names and a one or two line description of what they will achieve and having that reviewed by Ansible is a great way to ensure the modules fit the way people have used Ansible Modules before, and therefore make them easier to use.
Ansible has a thriving and knowledgeable community of module developers that is a great resource for getting your questions answered.
In the Ansible Community Guide you can find how to:
Subscribe to the Mailing Lists - We suggest “Ansible Development List” (for codefreeze info) and “Ansible Announce list”
#ansible-devel- We have found that IRC
#ansible-develon FreeNode’s IRC network works best for module developers so we can have an interactive dialogue.
IRC meetings - Join the various weekly IRC meetings meeting schedule and agenda page
Now that you’ve reviewed this document, you should be ready to open your first pull request.
The first PR is slightly different to the rest because it:
defines the namespace
provides a basis for detailed review that will help shape your future PRs
may include shared documentation (doc_fragments) that multiple modules require
may include shared code (module_utils) that multiple modules require
The first PR should include the following files:
lib/ansible/modules/$category/$topic/__init__.py- An empty file to initialize namespace and allow Python to import the files. Required new file
lib/ansible/modules/$category/$topic/$yourfirstmodule.py- A single module. Required new file
lib/ansible/plugins/doc_fragments/$topic.py- Code documentation, such as details regarding common arguments. Optional new file
lib/ansible/module_utils/$topic.py- Code shared between more than one module, such as common arguments. Optional new file
And that’s it.
Before pushing your PR to GitHub it’s a good idea to review the Contributing your module to Ansible again.
After publishing your PR to https://github.com/ansible/ansible, a Shippable CI test should run within a few minutes. Check the results (at the end of the PR page) to ensure that it’s passing (green). If it’s not passing, inspect each of the results. Most of the errors should be self-explanatory and are often related to badly formatted documentation (see YAML Syntax) or code that isn’t valid Python 2.6 or valid Python 3.5 (see Ansible and Python 3). If you aren’t sure what a Shippable test message means, copy it into the PR along with a comment and we will review.
If you need further advice, consider join the
#ansible-devel IRC channel (see how in the “Where to get support”).
We have a
ansibullbot helper that comments on GitHub Issues and PRs which should highlight important information.
By this point you first PR that defined the module namespace should have been merged. You can take the lessons learned from the first PR and apply it to the rest of the modules.
Raise exactly one PR per module for the remaining modules.
Over the years we’ve experimented with different sized module PRs, ranging from one module to many tens of modules, and during that time we’ve found the following:
A PR with a single file gets a higher quality review
PRs with multiple modules are harder for the creator to ensure all feedback has been applied
PRs with many modules take a lot more work to review, and tend to get passed over for easier-to-review PRs.
You can raise up to five PRs at once (5 PRs = 5 new modules) after your first PR has been merged. We’ve found this is a good batch size to keep the review process flowing.
Now that your modules are integrated there are a few bits of housekeeping to be done.
Bot Meta Update Ansibullbot so it knows who to notify if/when bugs or PRs are raised against your modules BOTMETA.yml.
If there are multiple people that can be notified, please list them. That avoids waiting on a single person who may be unavailable for any reason. Note that in BOTMETA.yml you can take ownership of an entire directory.
Review Module web docs Review the autogenerated module documentation for each of your modules, found in Module Docs to ensure they are correctly formatted. If there are any issues please fix by raising a single PR.
If the module documentation hasn’t been published live yet, please let a member of the Ansible Core Team know in the
#ansible-devel IRC channel.
We realize this may be your first use of Git or GitHub. The following guides may be of use:
Please note that in the Ansible Git Repo the main branch is called
devel rather than
master, which is used in the official GitHub documentation
After your first PR has been merged ensure you “sync your fork” with
ansible/ansible to ensure you’ve pulled in the directory structure and and shared code or documentation previously created.
As stated in the GitHub documentation, always use feature branches for your PRs, never commit directly into