-
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 a module for managing E-Series top-level settings #41010
Conversation
The test
|
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
@@ -121,24 +121,31 @@ class ModuleDocFragment(object): | |||
api_username: | |||
required: true | |||
description: | |||
- The username to authenticate with the SANtricity WebServices Proxy or embedded REST API. | |||
- The username to authenticate with the SANtricity Web Services Proxy or Embedded Web Services API. |
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.
I think this change needs removing from this PR (as it's been done in another PR which has since been merged)
Also you need to rebase to resolve the merge conflict https://docs.ansible.com/ansible/devel/dev_guide/developing_rebasing.html
@@ -1207,7 +1207,6 @@ lib/ansible/modules/storage/netapp/na_cdot_volume.py E322 | |||
lib/ansible/modules/storage/netapp/na_cdot_volume.py E324 | |||
lib/ansible/modules/storage/netapp/na_cdot_volume.py E325 | |||
lib/ansible/modules/storage/netapp/netapp_e_amg.py E322 |
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.
I think this change needs removing from this PR (as it's been done in another PR which has since been merged)
Also you need to rebase to resolve the merge conflict https://docs.ansible.com/ansible/devel/dev_guide/developing_rebasing.html
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.
Module itself looks OK, please just remove the docs_fragment and ignore.txt files from this PR and then rebase your PR I think this change needs removing from this PR (as it's been done in another PR which has since been merged)
Also you need to rebase to resolve the merge conflict https://docs.ansible.com/ansible/devel/dev_guide/developing_rebasing.html
There are multiple settings that are defined at a global level for E-Series systems, but don't necessarily fit with anything else. This module is intended to provide a place to encapsulate those.
SUMMARY
There are multiple settings that are defined at a global level for
E-Series systems, but don't necessarily fit with anything else. This
module is intended to provide a place to encapsulate those.
ISSUE TYPE
New Module Pull Request
COMPONENT NAME
netapp_e_global
ANSIBLE VERSION
ADDITIONAL INFORMATION