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 refactored bigip_ssl_certificate #212

Closed
37 tasks done
caphrim007 opened this issue May 19, 2017 · 3 comments
Closed
37 tasks done

upstream refactored bigip_ssl_certificate #212

caphrim007 opened this issue May 19, 2017 · 3 comments
Assignees
Labels
upstream Issues for modules that need to be upstreamed due to bugfix, enhancement, or new module

Comments

@caphrim007
Copy link
Contributor

caphrim007 commented May 19, 2017

ISSUE TYPE
  • Upstream request
COMPONENT NAME
ANSIBLE VERSION

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.box)
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.4.
  • 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.4.

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 tox -e flake -- library/MODULE_NAME.py
  • Unit test code passes tox -e flake -- 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

(this must be done inside of Ansible's source tree)

  • Module code in Ansible repo passes ansible-test units --tox --python 2.6 MODULE_NAME
  • Module code in Ansible repo passes ansible-test units --tox --python 2.7 MODULE_NAME
  • Module code in Ansible repo passes ansible-test units --tox --python 3.5 MODULE_NAME
  • Module code in Ansible repo passes ansible-test units --tox --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
  • Module PR has been mentioned as a comment for the next Networking Team meeting here Network Working Group Meeting Agenda ansible/community#110
SUMMARY
@caphrim007 caphrim007 created this issue from a note in Ansible 2.4 release (Todo) May 19, 2017
@caphrim007 caphrim007 self-assigned this May 19, 2017
@caphrim007 caphrim007 added the upstream Issues for modules that need to be upstreamed due to bugfix, enhancement, or new module label May 19, 2017
@caphrim007 caphrim007 moved this from Todo to In Progress in Ansible 2.4 release May 30, 2017
@caphrim007
Copy link
Contributor Author

this module is ready for upstreaming

@caphrim007
Copy link
Contributor Author

a PR for this has been opened here ansible/ansible#25882

@caphrim007
Copy link
Contributor Author

this is now merged

@caphrim007 caphrim007 moved this from In Progress to Done in Ansible 2.4 release Jun 28, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
upstream Issues for modules that need to be upstreamed due to bugfix, enhancement, or new module
Projects
No open projects
Development

No branches or pull requests

1 participant