Adding na_ontap_disks.py Module

[carchi8py]

SUMMARY

Adding na_ontap_disks.py Module

ISSUE TYPE

• New Module Pull Request

COMPONENT NAME

• na_ontap_disks.py

ANSIBLE VERSION

bash-3.2# ansible --version
ansible 2.7.0.dev0 (sf_commonfiles caf71e24dd) last updated 2018/08/06 10:46:57 (GMT -700)
  config file = None
  configured module search path = [u'/Users/chrisarchibald/.ansible/plugins/modules', u'/usr/share/ansible/plugins/modules']
  ansible python module location = /private/tmp/ansible/lib/ansible
  executable location = /private/tmp/ansible/bin/ansible
  python version = 2.7.12 (default, Oct 11 2016, 05:20:59) [GCC 4.2.1 Compatible Apple LLVM 8.0.0 (clang-800.0.38)]

ADDITIONAL INFORMATION

[ansibot]

@amit0701 @hulquest @lmprice @ndswartz @schmots1

As a maintainer of a module in the same namespace this new module has been submitted to, your vote counts for shipits. Please review this module and add shipit if you would like to see it merged.

click here for bot help

[ansibot]

cc @amit0701 @hulquest @lmprice @ndswartz @schmots1
click here for bot help

[schmots1]

shipit

[gundalow]

This doesn't make sense to me (though I'm not a NetApp user)

• Used only with auto or all parameter - What defines auto or all?
• Is this meant to be the name of a node? does node mean disk, cluster, ...?

[schmots1]

shipit

[gundalow]

How would you unassign, set node: to empty, null?
Would be worth adding an example for that.

If it's not possible then I think short_description may need updating

[schmots1]

Updated. Will change short description back for a later release when we modify how much this module does.