search

DimensionDataResearch / ansible-modules-extras

Ansible extra modules - these modules ship with ansible
GNU General Public License v3.0
1 stars 2 forks source link

Added ability to list servers/nodes. #13

Closed aimonb closed 8 years ago

aimonb commented 8 years ago

Added ability to list servers/nodes.

jr2730 commented 8 years ago

I don't see much GET support. Is get supposed to be different than list? If not, maybe make get an alias for list? If so, I don't see any GET support.

relaxdiego commented 8 years ago

I can't think of a scenario when anyone would need a separate GET in this case. Ansible always works with a list of hosts even if the list only contains one host.

aimonb commented 8 years ago

get is alias for list.

On 9/25/16 4:11 PM, Mark Maglana wrote:

I can't think of a scenario when anyone would need a separate GET in this case. Ansible always works with a list of hosts even if the list only contains one host.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/DimensionDataResearch/ansible-modules-extras/pull/13#issuecomment-249446960, or mute the thread https://github.com/notifications/unsubscribe-auth/AADlp2_lmtjIrTx_LoAZOwGvAaETA7mvks5qtuN-gaJpZM4KF9Ku.

jr2730 commented 8 years ago

Might be better to use the AnsibleModule "alias" capability. Cause right now you have 2 different "commands" that are both treated the same. And it makes the documentation more confusing (UNLESS you somehow plan on separating the 2 commands at some point).

aimonb commented 8 years ago

As far as I know alias only works for arguments, not their values.

aimonb commented 8 years ago

I am thinking we just drop 'get'?

aimonb commented 8 years ago

'get' has been removed.