Reviews needed for 99 new modules in Ansible Extras

Archives Ansible Project
1

We currently have 99 new modules in Ansible Extras that need review.
You can see the list here:

https://github.com/ansible/ansible-modules-extras/labels/new_module

If you are a module author yourself (official list:
https://github.com/ansible/ansible-modules-extras/blob/devel/REVIEWERS.md),
please take the time to review one of these modules. The process is
simple:

1. Go to the pull request URL and leave a note that you’re in the
process of reviewing the module.
2. Run the module to test its basic functionality.
3. Review the module for its adherence to the module guidelines.
(http://docs.ansible.com/developing_modules.html#module-checklist).
4. Add any necessary feedback to the pull request.
5. Once the module looks good, give it a +1!

Any module with two +1s from official module reviewers will be
approved for inclusion in Ansible Extras.

Thanks for your help in improving the PR merge process. If you have
any feedback, comments or questions, please don't hesitate to ask.

--g

2

This is an impressive set of extensions.

A question, when a modules has in its docs, eg

requirements: [datadog]

That implies the need for ‘pip install datadog’? Not a requirement for a datadog role
that includes the python library?

3

modules cannot depend on roles, it should always be libraries (or
versions of python).

4

What are the test required if I want to add Module in Cloud Module Directory. and what is the process to review module? Which of these tests are required among Units, Integration, Sanity and Compile/

5

Hi Roshan,

Sanity and Compile are must. Depending on modules units and integration are optional, not sure about Cloud module.

Thanks
Naval

6

Hi Naval,

I am working on 1 PR request which has stale_ci tag and got “All checks have passed” through shippable Devops, what does that mean?

I have done all sanity and compile test locally. What should I do now to merge it to Ansible Directory?
PR #44256

Please help…Thanks in advance

Roshan

7

Hi Roshan,

Get familiar with ansible bot,

click here for bot help
put bot_status in comment.

bot_status will give you detail about files in your commit and who are maintainers of those files.

Once you are done with review comment,
put ready_for_review in comment.

I will recommend you to join ansible-devel group irc-chat, you can get answer there.

–Naval