Reviews needed for 99 new modules in Ansible Extras

71 views
Skip to first unread message

Greg DeKoenigsberg

unread,
Jun 16, 2015, 5:08:53 PM6/16/15
to ansible...@googlegroups.com, ansible-devel
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

--
Greg DeKoenigsberg
Ansible Community Guy

Find out why SD Times named Ansible
their #1 Company to Watch in 2015:
http://sdtimes.com/companies-watch-2015/

Barry Kaplan

unread,
Jun 16, 2015, 8:59:47 PM6/16/15
to ansible...@googlegroups.com, ansibl...@googlegroups.com
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?

Brian Coca

unread,
Jun 17, 2015, 8:48:44 AM6/17/15
to Barry Kaplan, ansible...@googlegroups.com, ansibl...@googlegroups.com
modules cannot depend on roles, it should always be libraries (or
versions of python).



--
Brian Coca
Reply all
Reply to author
Forward
0 new messages