-
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
Define netapp_e_mgmt_interface storage module. #42519
Conversation
c09b184
to
8d577da
Compare
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.
shipit
8d577da
to
46d7e9f
Compare
46d7e9f
to
6b706f4
Compare
The test
|
91018a4
to
c5e7087
Compare
c5e7087
to
18fdf00
Compare
The netapp_e_mgmt_interface module provides management configuration for e-series storage array interface points including remote ssh access, NTP and DNS services. This patch also includes integration and unit tests.
18fdf00
to
b59cff3
Compare
The test
The test
|
The test
|
that are present in the system. | ||
- The name represents the port number (typically from left to right on the controller), | ||
beginning with a value of 1. | ||
- Mutually exclusive with I(channel). |
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.
You can add the following to argument_spec
mutually_exclusive=(['name', 'channel'],),
- The port to modify the configuration for. | ||
- The channel represents the port number (typically from left to right on the controller), | ||
beginning with a value of 1. | ||
- Mutually exclusive with I(name). |
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.
Maybe worth adding code to check this and call fail_json
if it isn't allowed
description: | ||
- Enable ssh access to the controller for debug purposes. | ||
- This is a controller-level setting. | ||
- Older controllers may enable rlogin or telnet instead of ssh. |
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.
So this modules will do...?
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.
This is a bit confusing if you're not familiar with the E-Series systems. This option will enable/disable telnet or ssh, depending on the age/model of the E-Series system that's being managed. Perhaps it's worded poorly.
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.
Much better, thanks
SUMMARY
The netapp_e_mgmt_interface module provides management configuration for
e-series storage array interface points including remote ssh access, NTP
and DNS services.
This patch also includes integration and unit tests.
ISSUE TYPE
COMPONENT NAME
netapp_e_mgmt_interface
ANSIBLE VERSION
ADDITIONAL INFORMATION