Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

upstream bigip_wait module #388

Closed
26 of 45 tasks
caphrim007 opened this issue Oct 17, 2017 · 2 comments
Closed
26 of 45 tasks

upstream bigip_wait module #388

caphrim007 opened this issue Oct 17, 2017 · 2 comments
Assignees
Labels
new-module Issues that require the creation of a new Ansible module

Comments

@caphrim007
Copy link
Contributor

caphrim007 commented Oct 17, 2017

ISSUE TYPE
  • Upstream request
COMPONENT NAME

bigip_wait

ANSIBLE VERSION
ansible 2.5.0
  config file = None
  configured module search path = [u'/root/.ansible/plugins/modules', u'/usr/share/ansible/plugins/modules']
  ansible python module location = /usr/local/lib/python2.7/site-packages/ansible
  executable location = /usr/local/bin/ansible
  python version = 2.7.14 (default, Sep 18 2017, 20:47:33) [GCC 4.9.2]
PYTHON VERSION

BIGIP VERSION
  • 12.0.0 (BIGIP-12.0.0.0.0.606)
  • 12.1.0 (BIGIP-12.1.0.0.0.1434)
  • 12.1.0-hf1 (BIGIP-12.1.0.1.0.1447-HF1)
  • 12.1.0-hf2 (BIGIP-12.1.0.2.0.1468-HF2)
  • 12.1.1 (BIGIP-12.1.1.0.0.184)
  • 12.1.1-hf1 (BIGIP-12.1.1.1.0.196-HF1)
  • 12.1.1-hf2 (BIGIP-12.1.1.2.0.204-HF2)
  • 12.1.2 (BIGIP-12.1.2.0.0.249)
  • 12.1.2-hf1 (BIGIP-12.1.2.1.0.264-HF1)
  • 13.0.0 (BIGIP-13.0.0.0.0.1645)
  • 13.0.0-hf1 (BIGIP-13.0.0.1.0.1668-HF1)
Python VERSION
  • 2.7.14
  • 3.5.4
  • 3.6.2
CHECKLIST

Module code related

  • Module code has EXAMPLES
  • Module code has DOCUMENTATION
  • Module code has RETURN
  • Module code has ANSIBLE_METADATA
  • Module code conforms to coding standards (v3). This includes a Parameters class, ModuleManager class, and ArgumentSpec class.
  • Module includes GPL3 license at top of the file and F5 Networks copyright.
  • Module imports are not from *
  • If this module is new to Ansible, the version_added key of the DOCUMENTATION variable is set to 2.5.
  • If this module is not new to Ansible, but you have added accepted parameters to it, the version_added key for each of those new parameters is 2.5.

Integration test related

  • Playbook exists to do functional tests
  • Role directory with functional tests exists
  • Module code has been tested on major releases post 12.x. If a major release cannot be supported, document "why" in the notes section of the module
    DOCUMENTATION

Unit test related

  • Unit tests in tests/unit for the product exist
  • TestParameters unit test class exists and has content
  • TestManager unit test class (or *Managers if required) exists and has content

QC related

  • Module code passes pycodestyle library/MODULE_NAME.py
  • Unit test code passes pycodestyle test/unit/PRODUCT/test_MODULE_NAME.py

Docs related

  • Docs exist and build correctly (make docs)
  • Docs have been merged to f5-ansible so they can be rebuilt by ReadTheDocs.

Upstream vendor related, with f5-sdk

This must be done inside of Ansible's source tree. (cd local/ansible)

  • Module code in Ansible repo passes ansible-test units --python 2.7 MODULE_NAME
  • Module code in Ansible repo passes ansible-test units --python 3.5 MODULE_NAME
  • Module code in Ansible repo passes ansible-test units --python 3.6 MODULE_NAME
  • Module code in Ansible repo passes ansible-test sanity --test validate-modules
  • Module code in Ansible repo passes ansible-test sanity --test pep8

Upstream vendor related, without f5-sdk

This must be done inside of Ansible's source tree. (cd local/ansible)

  • Module code in Ansible repo passes ansible-test sanity --test import --python 2.7
  • Module code in Ansible repo passes ansible-test sanity --test import --python 3.5
  • Module code in Ansible repo passes ansible-test sanity --test import --python 3.6
  • Module code in Ansible repo passes nosetests test/units/modules/network/f5/test_MODULE_NAME.py using venv made with requirements.bare.txt

Upstream vendor related, misc

Upstream vendor, sanity related

  • Command bash test/ansible/sanity/integration-test-idempotent-names.sh passes
  • Command ./test/ansible/sanity/short-description-ends-with-period.py passes
SUMMARY
@caphrim007 caphrim007 added new-module Issues that require the creation of a new Ansible module upstream Issues for modules that need to be upstreamed due to bugfix, enhancement, or new module labels Oct 17, 2017
@caphrim007 caphrim007 self-assigned this Oct 17, 2017
@caphrim007
Copy link
Contributor Author

PR created here ansible/ansible#31846

@caphrim007
Copy link
Contributor Author

this is now merged

@caphrim007 caphrim007 changed the title upstream wait_for_bigip module upstream bigip_wait module Oct 18, 2017
@caphrim007 caphrim007 removed the upstream Issues for modules that need to be upstreamed due to bugfix, enhancement, or new module label Nov 2, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
new-module Issues that require the creation of a new Ansible module
Projects
None yet
Development

No branches or pull requests

1 participant