-
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 module for managing LDAP authentication for E-Series #42356
Conversation
|
||
return domain | ||
|
||
def is_embedded(self): |
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.
Could this be moved to the common fragment? I don't think it's a must-fix for this PR.
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 that's a good idea, but let's get this one merged first. I'll add the common code after all of the other changes to the common code in netapp utils file.
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.
@lmprice Reminder that this may need moving to common code
The test
The test
The test
The test
The test
|
lib/ansible/module_utils/netapp.py
Outdated
@@ -35,6 +35,7 @@ | |||
from ansible.module_utils.six.moves.urllib.error import HTTPError | |||
from ansible.module_utils.urls import open_url | |||
from ansible.module_utils.api import basic_auth_argument_spec | |||
from ansible.release import __version__ as ansible_version |
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.
Use this instead:
from ansible.module_utils.ansible_release import __version__
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.
Got it. I was guessing on the best place to pull that information.
shipit |
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
shipit |
@lmprice this PR contains the following merge commits: Please rebase your branch to remove these commits. |
Can you please rebase this by following http://docs.ansible.com/ansible/devel/dev_guide/developing_rebasing.html Thanks in advance |
The test
|
The test
|
This is an expected failure. Waiting on #44659 to merge. |
The test
|
This module will allow users to manage LDAP authentication domains for accessing E-Series systems.
The test
|
This module will allow users to manage LDAP authentication domains for
accessing E-Series systems.
SUMMARY
This is a new module to allow an owner of an E-Series system to manage the LDAP authentication domains that can be utilized for logging into the system.
ISSUE TYPE
COMPONENT NAME
netapp_e_ldap
ANSIBLE VERSION
ADDITIONAL INFORMATION