-
Notifications
You must be signed in to change notification settings - Fork 23.8k
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
snow: Add new parameter 'host' #66184
Conversation
Added support for specifying custom service now domain. Fixes: ansible#62018 Signed-off-by: Abhijeet Kasurde <akasurde@redhat.com>
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 don't use this so can't try it manually. would be greate if someone did
shipit
@garbled1 @n3pjk @jmcleroy @theemptyone Can you please test this PR and let me know if it works for you? Thanks in advance. |
Hello, I tested this in #41320 (comment) It looks OK. But I am mostly interested in the inventory part of service now. Is there something else you would like me to test? I would like to test it without disabling CA verification so that's something I'll have to look into, taking suggestions how to do that, maybe even without updating the system CA bundle? |
Migrated to ServiceNowITOM/servicenow-ansible#5 |
SUMMARY
Added support for specifying custom service now domain.
Fixes: #62018
Signed-off-by: Abhijeet Kasurde akasurde@redhat.com
ISSUE TYPE
COMPONENT NAME
changelogs/fragments/62018_snow_host.yml
lib/ansible/module_utils/service_now.py
lib/ansible/modules/notification/snow_record.py
lib/ansible/modules/notification/snow_record_find.py
lib/ansible/plugins/doc_fragments/service_now.py