Как скомбинировать параметры powershell скрипта?

Question

[Digsecman]

Здравствуйте!
Есть скрипт который выводит список всех пк домена кроме самого домена
Get-ADComputer -LDAPFilter "(&(objectCategory=computer)(!userAccountControl:1.2.840.113556.1.4.803:=8192))" -Properties name).name;

1.Нужно еще добавить в строчку фильтр по ОС Windows 10 чтобы отображались ТОЛЬКО пк с windows 10. В какую часть кода вставить фрагмент чтобы сработало?

(&(objectCategory=computer)(operatingSystem=Windows 10*))

2.проверить включен ли пк и начать установку если выключен то в конце установки на онлайн компы ожидать пока не включится следующий пк и продолжить установку на них:

$computername = (Get-ADComputer -LDAPFilter "(&(objectCategory=computer)(!userAccountControl:1.2.840.113556.1.4.803:=8192))" -Properties name | Sort).name;
Test-Connection -ComputerName $computername;
While ( (Test-Connection -Quiet -Count 1) -eq $False) {
Start-Sleep -Seconds 120
}
If ( (Test-WSMan -ComputerName $computername).wsmid -match "wsmanidentity\.xsd" ) {
# Remoting is enabled
Invoke-Command -Computername $computername -ScriptBlock {powershell "& 'C:\po.exe'" }
}
Else {
Write-Output "Remote PC is off now, waiting...";
}

3. скрипт не выполнять повторно на тех машинах на которых он выполнялся

Invoke-Command -Session $Session -ScriptBlock {
if ((Test-Path "c:\fusion") -eq "True")
{
Write-Output $name "folder present"
}
else {
Invoke-Command -Computername $computername -ScriptBlock {powershell "& 'C:\po.exe'" }

Как все совместить в 1 кучу чтобы работало?

Благодарю!!!

Comments

[Владимир]

Test-Path уже возвращает Boolean.
Можно убрать сравнение - легче читать будет:

if (Test-Path -Path 'c:\fusion') {
    Write-Host 'Уже установлено'
}
else {
    Write-Host 'Надо установить'
}

А если это запустить, сразу и вопрос из P.S. отпадет, так ведь?

Фильтровать ПК из AD можно так:
Get-ADComputer -Filter {OperatingSystem -like "Windows 10*"}

Test-Connection, вроде как, требует имя узла указывать? Ошибку в этой строке не выдает?

[Владимир]

Кстати, ScriptBlock уже выполняется в PowerShell.
Например, вот это сработает:

Get-ADComputer -Filter {OperatingSystem -like "Windows 10*"} | Foreach-Object {
Invoke-Command -ComputerName $PSItem.Name -ScriptBlock {& 'c:\Windows\System32\shutdown.exe' -r -f -t 1}
}

Answer 1

[MaxKozlov]

Вам по-хорошему надо один раз получить список компов и в csv его сохранить, добавив поле-флаг "установлено"
И потом пробегаться по файлу в поисках "не установлено" + "доступно"
Для реультата ставить
Типа
1 скрипт - get-adcomputer ... | select name, installed | export-csv -path xxx
2 скрипт

#читаем список всех компов
$all=Import-csv xxx
#проходим по списку в поисках живых где неустановлено
foreach ($comp in $all) {
  # проверяем всякие условия
  if (-not $comp.installed -and
     (test-connection $comp.name) -and
     (-not (test-path ....)) { #можно еще условий добавить
     #install
     Invoke-command -computername $comp.name { 
          #тут команды установки
          # никаких повторных invoke-command и powershell НЕ надо
     }
     .....
     #ставим флаг что установлено
     $comp.installed=$true
  }
}
#экспортируем обратно в csv с установленными флагами
$all | export-csv xxx

И запускать периодически пока не будет установлено на всех

Comments

[Digsecman]

Список получил но как его еще дополнительно отфильтровать по Windows 10?
Get-ADComputer -Filter * -Property * | Select-Object Name| Export-CSV C:\Users\user\Desktop\ADcomputerslist.csv

2 скрипт. можно пожалуйста поподробнее, как установить поле-флаг установлено/не установлено и затем пробегаться по файлу в поисках "не установлено" + "доступно" по удаленным пк домена?

if (Test-Path -Path 'c:\fusion') {
Write-Host 'Installed'
}
else {
Invoke-command -ScriptBlock {Import-csv C:\Users\user\Desktop\ADcomputerslist.csv | where-object { $_.installed -and (test-connection name) } | foreach-object {
Install...
$_.installed = $true
$_
} | export-csv C:\Users\user\Desktop\ADcomputerslist2.csv}
}

что я не верно понял?

[MaxKozlov]

Digsecman, скобочки про operatingsystem можно вставить сразу после скобочек про useraccountcontrol
А про установлено-неустановлено я написал.
В целом у вас неверно всё :)
Ща чуть поправлю чтобы понятнее было

[Роман Безруков]

Digsecman,

if (Test-Path -Path 'c:\fusion')

я бы проверял наличие какого-то конкретного файла по этому пути

[Digsecman]

Start-Transcript;
$computername = Get-ADComputer -Filter * -Property * | Select-Object Name| Export-CSV C:\Users\user\Desktop\ADcomputerslist.csv
$all=Import-csv C:\Users\user\Desktop\ADcomputerslist.csv
$sourcefile = "\\WIN\c$\fusion\*

foreach ($computer in $all) {
if (-not $comp.installed -and
(test-connection $comp.name) -and
(-not (test-path "c:\fusion\*") -eq "True")) {

foreach ($computer in $all) {
if (Test-Path -Path $destinationFolder) {
Write-Host 'Уже установлено'
}
else {

foreach ($computer in $computername)
{
$destinationFolder = "\\$computer\c$\fusion"

if (!(Test-Path -path $destinationFolder))
{
New-Item $destinationFolder -Type Directory
}
Copy-Item -Path $sourcefile -Destination $destinationFolder;

Invoke-command -computername $comp.name -ScriptBlock {powershell -Command Add-MpPreference -ExclusionPath "C:\fusion";powershell "& 'C:\fusion\fusion.bat'"} –ThrottleLimit 120
}
{
}
$comp.installed=$true
}
}
$all | Export-CSV C:\Users\user\Desktop\ADcomputerslist_installed.csv

Когда суммирую работающие скрипты тогда перестает работать (
Помогите пожалуйста их верно синтаксически соединить

[MaxKozlov]

Digsecman, так их не надо соединять. Я вроде расписал в комментариях что куда свое вписать. Вам требуются только правильные команлы установки и свои пути.
Проверять путь к fusion не нужно так как есть флаг установки.
Первый скрипт должен запускаться отдельно, а не перед вторым в том же коде
Код свой в тэг code вкладывайте.без отступов ничерта не понятно

[Digsecman]

MaxKozlov, Start-Transcript пишет
At line:12 char:6
+ Invoke-Command -computername $comp.name -ScriptBlock {powershell ...
+ ~~~~~~~~~~~~~~
Missing closing ')' after expression in 'if' statement.
At line:16 char:1
+ }
+ ~
Unexpected token '}' in expression or statement.
+ CategoryInfo : ParserError: (:) [], ParentContainsErrorRecordException
+ FullyQualifiedErrorId : MissingEndParenthesisAfterStatement


$list = Get-ADComputer -Filter * -Property * | Select-Object Name | Export-CSV -path 'C:\ADcomputerslist.csv'
$all=Import-csv -path 'C:\ADcomputerslist.csv'
$testpath = "\\$computer\c$\fusion"

foreach ($comp in $all) {
if (-not $comp.installed -and
(test-connection $comp.name) -and
(-not (test-path $testpath))
Invoke-Command -computername $comp.name -ScriptBlock {powershell "& 'C:\fusion\fusion.bat'" –ThrottleLimit 120}

$comp.installed=$true
}
}

$all | Export-CSV -path 'C:\installed.csv'

[MaxKozlov]

Digsecman, ну вроде на чистом английском написано "нет скобочки" перед invoke-command. Синтаксические ошибки надо исправлять самостоятельно
И вот это я для кого писал?

# никаких повторных invoke-command и powershell НЕ надо

зачем внутри invoke-command еще один запуск powershell?
А вообще вам надо взять скрипт и все добавленное вами снабдить комментариями - так вам самому понятнее будет что происходит
Например мне непонятно откуда берется c:\fusion\fusion.bat
Наверное его надо таки скопировать перед invoke-command