-
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
win_domain modules: ensure Netlogon service is still running after promotion #43703
Conversation
# with ansible_winrm_transport=basic so we just display a warning if | ||
# this fails. | ||
try { | ||
Start-Service -Name Netlogon |
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 wonder if its worth 'treading lightly' and only calling 'Start-Service' if Netlogon isn't currently running? Would potentially avoid an unneeded warning if using Basic auth?
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.
Probably not, if the service is already running it shouldn't be doing anything. If an error comes up then it's probably a sign of a bigger issue and we should know about it.
…omotion (ansible#43703) (cherry picked from commit 7b1cc11)
Backport PR to stable-2.6 #43913 |
* devel: (30 commits) Prevent data being truncated over persistent connection socket (#43885) Fix eos_command integration test failures (#43922) Update iosxr cliconf plugin (#43837) win_domain modules: ensure Netlogon service is still running after promotion (#43703) openvswitch_db : Handle column value conversion and idempotency in no_key case (#43869) Fix typo Fix spelling of ansbile to ansible (#43898) added platform guides for NOS and VOSS (#43854) Fix download URL for yum integration test. New module for managing EMC VNX Block storage (#42945) Docker integration tests: factorize setup (#42306) VMware: datastore selection (#35812) Remove unnecessary features from cli_command (#43829) [doc] import_role: mention version from which behavior changed and fix some typos (#43843) Add source interface and use-vrf features (#43418) Fix unreferenced msg from vmware_host (#43872) set supports_generate_diff to False vyos (#43873) add group_by_os_family in azure dynamic inventory (#40702) ansible-test: Create public key creating Windows targets (#43760) azure_rm_loadbalancer_facts.py: list() takes at least 2 arguments fix (#29046) (#29050) ...
SUMMARY
When promoting a server to a domain controller, Ansible is unable to authenticate to the server unless it is manually rebooted, the
Netlogon
service is started manually, orbasic
auth is used. This change tries to start theNetlogon
service manually before returning back to the host so subsequent calls likewin_reboot
can reboot the host.Fixes #39235
ISSUE TYPE
COMPONENT NAME
win_domain
win_domain_controller
ANSIBLE VERSION