Adds the bigip_user module to Ansible

[caphrim007]

SUMMARY

The patch adds the bigip_user module to Ansible to support managing
users on an F5 BIG-IP.

ISSUE TYPE

• New Module Pull Request

COMPONENT NAME

lib/ansible/modules/network/f5/bigip_user.py

ANSIBLE VERSION

ansible 2.4.0 (feature.add-bigip-user-module 35118a6eb4) last updated 2017/05/17 13:32:10 (GMT -700)
  config file = 
  configured module search path = [u'/Users/trupp/.ansible/plugins/modules', u'/usr/share/ansible/plugins/modules']
  ansible python module location = /Users/trupp/src/trupp/ansible/lib/ansible
  executable location = /Users/trupp/src/trupp/ansible/bin/ansible
  python version = 2.7.10 (default, Feb  7 2017, 00:08:15) [GCC 4.2.1 Compatible Apple LLVM 8.0.0 (clang-800.0.34)]

ADDITIONAL INFORMATION

Unit tests are provided. Integration tests can be found here

• https://github.com/F5Networks/f5-ansible/blob/devel/test/integration/bigip_user.yaml
• https://github.com/F5Networks/f5-ansible/tree/devel/test/integration/targets/bigip_user/tasks

[ansibot]

The test ansible-test sanity --test shebang failed with the following error:

Command "test/sanity/code-smell/shebang.sh" returned exit status 1.
>>> Standard Output
./test/units/modules/network/f5/test_bigip_user.py:#!/usr/bin/python
One or more file(s) listed above have an unexpected shebang.
See test/sanity/code-smell/shebang.sh for the list of acceptable values.

The test ansible-test sanity --test validate-modules failed with the following errors:

lib/ansible/modules/network/f5/bigip_user.py:0:0: E305 DOCUMENTATION.options.partition_access.type: not a valid value for dictionary value @ data['options']['partition_access']['type']. Got 'list'
lib/ansible/modules/network/f5/bigip_user.py:0:0: E307 version_added should be 2.4. Currently 2.2

click here for bot help

[caphrim007]

@gundalow are the test files not allowed to include the shebang at the top?

[ansibot]

The test ansible-test sanity --test validate-modules failed with the following error:

lib/ansible/modules/network/f5/bigip_user.py:0:0: E101 Interpreter line is not "#!/usr/bin/python"

click here for bot help

[gundalow]

1. Regarding the requirements. I've spoken with Matt Clay. We are both happy for them to be in the main file. In the future we may add in support for splitting the requirement files out, but we don't have the need yet.

2. Regarding the shebag.
   Unit tests shouldn't be executable. They also need to run under different versions of Python as specified by ansible-test

[gundalow]

First new non-core module into 2.4 with Unit Tests, good stuff :)

units/2.6
failing with

2017-05-18 00:12:43   File "/root/src/github.com/ansible/ansible/test/runner/.tox/py26/lib/python2.6/site-packages/py/_builtin.py", line 221, in exec_
2017-05-18 00:12:43     exec2(obj, globals, locals)
2017-05-18 00:12:43   File "<string>", line 7, in exec2
2017-05-18 00:12:43   File "/root/src/github.com/ansible/ansible/test/runner/.tox/py26/lib/python2.6/site-packages/f5/bigip/cm/autodeploy/software_images.py", line 23, in <module>
2017-05-18 00:12:43     from f5.bigip.resource import PathElement
2017-05-18 00:12:43   File "/root/src/github.com/ansible/ansible/test/runner/.tox/py26/lib/python2.6/site-packages/f5/bigip/resource.py", line 722
2017-05-18 00:12:43     no_meta_dict = {k: v for k, v in self.__dict__.iteritems()

units/3.5 & units/3.6
Are failing with a variety of different errors

[gundalow]

--requirements docs updated c9e0595

[caphrim007]

@gundalow ready_for_review

[gundalow]

Coverage has been updated https://codecov.io/gh/ansible/ansible/tree/devel