-
Notifications
You must be signed in to change notification settings - Fork 23.7k
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
Add new module mlnxos_facts for retrieving facts of MLNX-OS Mellanox network devices #34287
Conversation
network devices Signed-off-by: Samer Deeb <samerd@mellanox.com>
""" | ||
|
||
EXAMPLES = """ | ||
# Note: examples below use the following provider dict to handle |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Are you using provider:
?
# Note: examples below use the following provider dict to handle | ||
# transport and authentication to the node. | ||
--- | ||
# Collect all facts from the device |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please change the comment to a name:
, e.g.
- name: Collect all facts from the device
mlnxos_facts:
gather_subset: all
Best practice is to specify a name:
in Ansible. We find people often copy examples when writing playbooks.
Signed-off-by: Samer Deeb <samerd@mellanox.com>
@samerd Hi, do you think this is ready for final review and merge? |
@gundalow : yes, please review and merge if it is OK. |
Signed-off-by: Samer Deeb samerd@mellanox.com
SUMMARY
Add module mlnxos_facts for retrieving facts of MLNX-OS Mellanox switches
the module manages the facts: version, interfaces and modules.
ISSUE TYPE
COMPONENT NAME
modules/network/mlnxos/mlnxos_facts
ANSIBLE VERSION
ADDITIONAL INFORMATION