-
Notifications
You must be signed in to change notification settings - Fork 61
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
Portal AD connection #45
Conversation
@spitzerr Each Portal needs to restart, that bit is missing. Everything else looks good. |
@shailesh91 Added the restart. Even it was not necessary on our testing. Ready to merge :-) Fixes #42 |
@spitzerr I need to validate for the Portal HA (When AD is configured it should be also checked for not($Join)) and the case wherein AD connection is made post-deployment (From the looks of it Restart Only happens on the primary portal and not on the Secondary). |
Issue - Need to find a way to restart the secondary machine when AD Config happens as a post-deployment step. |
Modules/ArcGIS/ArcGISUtility.psm1
Outdated
@@ -125,8 +129,12 @@ function Wait-ForServiceToReachDesiredState | |||
if ($Attempts++ -gt 0) { # to skip the message for first attempt | |||
Write-Verbose "Checking state of Service '$ServiceName'. Attempt # $Attempts" | |||
} | |||
if($MachineName -ne $null -or $MachineName -ne ""){ | |||
$Service = Get-Service -Name $ServiceName -ErrorAction Ignore -ComputerName $MachineName |
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.
Issue -This query fails when fetching Service status from a remote machine since the local system account of the machine making the request doesn't have necessary permissions.
@spitzerr Confirmed internally, Portal restarts are not necessary. Hence, removing it. Can you check if this looks good? |
@shailesh91 Looks very good to me. On our site it`s working fine! |
@spitzerr yes will merge it by the start of next week. |
Connect Portal with Active directory as user store and enable automatic account creation