syntaxe : utiliser une variable dans une cmdlet

Version imprimable

Bonsoir,

je souhaite paramétrer l'utilisation d'une cmdlet. Voici mon essai, qui ne marche pas :
Code:

1 2 3 4 #construction de la variable $ADgroup #appel de la cmdlet Add-ADGroupMember "CN=$ADgroup,OU=Access Management,OU=Groups,DC=gad,DC=schneider-electric,DC=com" -server gad.schneider-electric.com -Member $data
Ma façon d'introduire la variable $ADgroup n'est pas bonne.

Citation:

At C:\Users\sesa27384\Documents\aalaurent\boulot\powershell\scripts\PopulateADGroup3.ps1:71 char:23
+ Add-ADGroupMember "CN=$ADgroup,OU=Access Management,OU=Groups,DC=gad,DC=schne ...
+ ~~~~~~~~~~~
Unexpected token 'CN=$ADgroup' in expression or statement.

Quelle serait une bonne façon de faire ?

L'implémentation de ta variable semble bonne
Code:

1 2 3 $ADgroup = "toto" Write-Host "CN=$ADgroup,OU=Access Management,OU=Groups,DC=gad,DC=schneider-electric,DC=com" CN=toto,OU=Access Management,OU=Groups,DC=gad,DC=schneider-electric,DC=com
La commande Add-ADGroupMember semble ne pas supporter ta syntaxe et attend un objet de type "Group" ou "User" retourné via "Get-ADGroup" et "Get-ADUser"

Issu de https://technet.microsoft.com/en-us/.../ee617210.aspx
Code:

1 2 3 $User = Get-ADUser -Identity "CN=Chew David,OU=UserAccounts,DC=NORTHAMERICA,DC=FABRIKAM,DC=COM" -Server "northamerica.fabrikam.com" $Group = Get-ADGroup -Identity "CN=AccountLeads,OU=UserAccounts,DC=EUROPE,DC=FABRIKAM,DC=COM" -Server "europe.fabrikam.com" Add-ADGroupMember -Identity $Group -Member $User -Server "europe.fabrikam.com"

29/04/2017, 18h30
laurentSc

Bonjour
J'ai bien vu ta réponse vendredi matin mais ne pourrai tester que mardi.
Et petite difficulté pressentie : la variable $ADgroup est générée par l'utilisateur qui choisit dans une liste déroulante donc ça va être dur de la générer par une des 2 fonctions que tu indiques...
30/04/2017, 10h02
ericlm128

Peut être avec cette syntaxe alors

Code:

Get-ADGroup -filter {name -eq $ADgroup}
30/04/2017, 18h13
laurentSc

Ok et si je fais ça ; après je fais quoi ?
01/05/2017, 09h53
ericlm128
Pour répondre à la question du post, je ferais comme cela
Code:

1 2 $Group = Get-ADGroup -filter {name -eq $ADgroup} Add-ADGroupMember -Identity $Group -Member $data -Server "gad.schneider-electric.com"
Attention, cette commande suppose que $data soit aussi du type attendu par la commande "Add-ADGroupMember".
01/05/2017, 12h01
laurentSc

Merci j'essaye demain et te dis

Ca y est ; non sans mal, j'ai fini par réussir à essayer et apparemment, ça ne va pas.

Mon script complet :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
Write-Host "Hello, World"
 
#vider le contenu des 2 fichiers, s'ils existent
#$strFileName="C:\Users\SESA27384\Desktop\AD\SESAOK.csv"
if (test-path "C:\Users\SESA27384\Desktop\AD\SESAOK.csv") {clear-content C:\Users\SESA27384\Desktop\AD\SESAOK.csv}
#$strFileName="C:\Users\SESA27384\Desktop\AD\SESANOK.csv"
if (test-path "C:\Users\SESA27384\Desktop\AD\SESANOK.csv") {clear-content C:\Users\SESA27384\Desktop\AD\SESANOK.csv}
 
#création des 2 tables pour les logs (à faire avant le foreach)
$sesaNOK=@()
$sesaOK=@()
 
#import du fichier csv à une seule colonne
[System.Reflection.Assembly]::LoadWithPartialName("System.Windows.Forms")
 
$objForm = New-Object System.Windows.Forms.OpenFileDialog
$objForm.InitialDirectory = "c:\"
$objForm.Title = "Selectionner un fichier :"
$objForm.FilterIndex = 3
$Show = $objForm.ShowDialog()
 
If ($Show -eq "Cancel")
{
"Annulé par l'utilisateur"
}
Else
{
write-host $objForm.FileName
}
$path=$objForm.FileName
 
#Ouvre une fenêtre.
[reflection.assembly]::LoadWithPartialName("System.Windows.Forms")
$form1 = New-Object Windows.Forms.Form
$form1.text = "Contrôles"             
$form1.Size = New-Object System.Drawing.Size(300,200)
#Liste déroulante (ComboBox).
$liste1 = New-Object System.Windows.Forms.Combobox
$liste1.Location = New-Object Drawing.Point 20,20
$liste1.Size = New-Object System.Drawing.Size(250,30)
$liste1.DropDownStyle = "DropDownList"
$liste1.Items.AddRange(("GAD-SU-NextGenPDM_NonPROD","GAD-SU-NextGenPDM_PrePROD","GAD-SU-MediaReviewUATGroup","GAD-SU-MediaReviewUserGroup"))
$liste1.SelectedIndex = 0
#Attache le contrôle à la fenêtre
$form1.controls.add($liste1)
#Affiche le tout.
$form1.ShowDialog()
#Affiche la sélection.
write-host "ComboBox = " $liste1.Text
#Fin.
$ADgroup=$liste1.Text #on affecte le choix de la liste déroulante dans la variable $ADgroup
 
$csv = Import-Csv -path $path
foreach($line in $csv)
{ 
    $columnvalue = $line.name
 
	$data = @()
	$domains = "gad.schneider-electric.com","eur.gad.schneider-electric.com","apa.gad.schneider-electric.com","nam.gad.schneider-electric.com","gmea.gad.schneider-electric.com"
	foreach($domain in $domains)
	{
		$data += Get-ADUser -filter "((EmployeeID -like '$columnvalue') -OR (name -like '$columnvalue')) -AND (enabled -eq '$true') " -Server $domain 
	}
	[bool]$data
	if ($data) { Write-Host "Insert $columnvalue"
 
 
 
    $Group = Get-ADGroup -filter {name -eq $ADgroup}
    Add-ADGroupMember -Identity $Group -Member $data -Server "gad.schneider-electric.com"
 
	$sesaOK +=$line   #si $data existe, on a inséré le SESA et ajoût de la ligne (donc le SESA) ds la variable $sesaOK
        # ces 4 lignes, pour mémoire : 
        #foreach ($User in $data)    #on explore la variable $data{
        # Write-Host $user.SamAccountName   #pour chq ligne, on sélectionne la propriété SamAccountName (qui contient le SESA)
        # $New=$User|Select-Object SamAccountName   #construire un objet avec une seule propriété
        # $New | Export-csv -path C:\Users\SESA27384\Desktop\AD\SESAOK.csv -Append   #export de cet objet
        #}
}
if (!$data) {Write-Host "Entry Not Found $columnvalue" 
                    $sesaNOK +=$line     #SESA pas inséré donc ajoût de la ligne (donc le SESA) ds la variable $sesaNOK}
} 
#export des 2 variables ds les 2 fichiers de log
$sesaNOK | Export-csv -path C:\Users\SESA27384\Desktop\AD\SESANOK.csv
 
$sesaOK  | Export-csv -path C:\Users\SESA27384\Desktop\AD\SESAOK.csv

Le message d'erreur :

Citation:

Add-ADGroupMember : Cannot validate argument on parameter 'Identity'. The argument is null. Provide a valid value for the argument,
and then try running the command again.
At C:\Users\sesa27384\Documents\aalaurent\boulot\powershell\scripts\PopulateADGroupetape2-3.ps1:70 char:33
+ Add-ADGroupMember -Identity $Group -Member $data -Server "gad.schneider-elec ...
+ ~~~~~~
+ CategoryInfo : InvalidData: (:) [Add-ADGroupMember], ParameterBindingValidationException
+ FullyQualifiedErrorId : ParameterArgumentValidationError,Microsoft.ActiveDirectory.Management.Commands.AddADGroupMember

Un Write-Host confirme ce message : la variable $Group est bien vide ...

02/05/2017, 18h10
ericlm128

Je ne vois pas le "Write-Host $Group", ou encore "$Group".
Par contre cette ligne affiche correctement ton groupe ? write-host "ComboBox = " $liste1.Text

Tu devrais essayer simplement en ligne de commande dans l'interpréteur PowerShell la ligne "Get-ADGroup -filter {name -eq "mongroup"}" pour voir si cela fonctionne.
Sinon je ferai des tests sur mon AD, en journée.
03/05/2017, 16h24
ericlm128

Après test, la commande "Get-ADGroup -filter {name -eq "mongroup"}" fonctionne correctement sur mon AD.

Désolé mais pas eu de temps avant maintenant à consacrer au sujet.

D'abord, voici le script complet avec le Write-Host :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
Write-Host "Hello, World"
 
#vider le contenu des 2 fichiers, s'ils existent
#$strFileName="C:\Users\SESA27384\Desktop\AD\SESAOK.csv"
if (test-path "C:\Users\SESA27384\Desktop\AD\SESAOK.csv") {clear-content C:\Users\SESA27384\Desktop\AD\SESAOK.csv}
#$strFileName="C:\Users\SESA27384\Desktop\AD\SESANOK.csv"
if (test-path "C:\Users\SESA27384\Desktop\AD\SESANOK.csv") {clear-content C:\Users\SESA27384\Desktop\AD\SESANOK.csv}
 
#création des 2 tables pour les logs (à faire avant le foreach)
$sesaNOK=@()
$sesaOK=@()
 
#import du fichier csv à une seule colonne
[System.Reflection.Assembly]::LoadWithPartialName("System.Windows.Forms")
 
$objForm = New-Object System.Windows.Forms.OpenFileDialog
$objForm.InitialDirectory = "c:\"
$objForm.Title = "Selectionner un fichier :"
$objForm.FilterIndex = 3
$Show = $objForm.ShowDialog()
 
If ($Show -eq "Cancel")
{
"Annulé par l'utilisateur"
}
Else
{
write-host $objForm.FileName
}
$path=$objForm.FileName
 
#Ouvre une fenêtre.
[reflection.assembly]::LoadWithPartialName("System.Windows.Forms")
$form1 = New-Object Windows.Forms.Form
$form1.text = "Contrôles"             
$form1.Size = New-Object System.Drawing.Size(300,200)
#Liste déroulante (ComboBox).
$liste1 = New-Object System.Windows.Forms.Combobox
$liste1.Location = New-Object Drawing.Point 20,20
$liste1.Size = New-Object System.Drawing.Size(250,30)
$liste1.DropDownStyle = "DropDownList"
$liste1.Items.AddRange(("GAD-SU-NextGenPDM_NonPROD","GAD-SU-NextGenPDM_PrePROD","GAD-SU-MediaReviewUATGroup","GAD-SU-MediaReviewUserGroup"))
$liste1.SelectedIndex = 0
#Attache le contrôle à la fenêtre
$form1.controls.add($liste1)
#Affiche le tout.
$form1.ShowDialog()
#Affiche la sélection.
write-host "ComboBox = " $liste1.Text
#Fin.
$ADgroup=$liste1.Text #on affecte le choix de la liste déroulante dans la variable $ADgroup
 
$csv = Import-Csv -path $path
foreach($line in $csv)
{ 
    $columnvalue = $line.name
 
	$data = @()
	$domains = "gad.schneider-electric.com","eur.gad.schneider-electric.com","apa.gad.schneider-electric.com","nam.gad.schneider-electric.com","gmea.gad.schneider-electric.com"
	foreach($domain in $domains)
	{
		$data += Get-ADUser -filter "((EmployeeID -like '$columnvalue') -OR (name -like '$columnvalue')) -AND (enabled -eq '$true') " -Server $domain 
	}
	[bool]$data
	if ($data) { Write-Host "Insert $columnvalue"
 
 
 
    $Group = Get-ADGroup -filter {name -eq $ADgroup}
    Write-Host "Group=$Group"
    Add-ADGroupMember -Identity $Group -Member $data -Server "gad.schneider-electric.com"
 
	$sesaOK +=$line   #si $data existe, on a inséré le SESA et ajoût de la ligne (donc le SESA) ds la variable $sesaOK
        # ces 4 lignes, pour mémoire : 
        #foreach ($User in $data)    #on explore la variable $data{
        # Write-Host $user.SamAccountName   #pour chq ligne, on sélectionne la propriété SamAccountName (qui contient le SESA)
        # $New=$User|Select-Object SamAccountName   #construire un objet avec une seule propriété
        # $New | Export-csv -path C:\Users\SESA27384\Desktop\AD\SESAOK.csv -Append   #export de cet objet
        #}
}
if (!$data) {Write-Host "Entry Not Found $columnvalue" 
                    $sesaNOK +=$line     #SESA pas inséré donc ajoût de la ligne (donc le SESA) ds la variable $sesaNOK
                    }
} 
#export des 2 variables ds les 2 fichiers de log
$sesaNOK | Export-csv -path C:\Users\SESA27384\Desktop\AD\SESANOK.csv
 
$sesaOK  | Export-csv -path C:\Users\SESA27384\Desktop\AD\SESAOK.csv

Sinon, l'instruction write-host "ComboBox = " $liste1.Text affiche correctement le groupe sélectionné.

Enfin la ligne "Get-ADGroup -filter {name -eq "mongroup"}" dans l'interpréteur de commande ne fonctionne que si je retire des guillemets : "Get-ADGroup -filter {name -eq mongroup}"...

04/05/2017, 12h31
ericlm128

Ok, c'est étrange pour ma part je doit mettre les guillements autour du nom de mon groupe

Fonctionne

Code:

Get-ADGroup -filter {name -eq "mongroup"}

Fonctionne pas

Code:

Get-ADGroup -filter {name -eq mongroup}

Get-ADGroup : Erreur lors de l'analyse de la requête: «name -eq mongroup» Message d'erreur: «syntax error» à la position: «10».

Finalement pareil, mais j'avais pas compris ici les guillemets de début et de fin : "Get-ADGroup -filter {name -eq "mongroup"}" et si je les recopie pas dans l'interpréteur de commande ;) : Get-ADGroup -filter {name -eq "mongroup"}, c'est comme pour toi.

EDIT : pas possible de mettre en cause le format de la variable générée par la liste déroulante, car même chose si je la remplace par une chaîne de caractères $Group = Get-ADGroup -filter {name -eq "GAD-SU-NextGenPDM_NonPROD"}.

EDIT2 : j'ai essayé de changer la partie du script qui gère la liste déroulante mais même combat :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
Write-Host "Hello, World"
 
#vider le contenu des 2 fichiers, s'ils existent
#$strFileName="C:\Users\SESA27384\Desktop\AD\SESAOK.csv"
if (test-path "C:\Users\SESA27384\Desktop\AD\SESAOK.csv") {clear-content C:\Users\SESA27384\Desktop\AD\SESAOK.csv}
#$strFileName="C:\Users\SESA27384\Desktop\AD\SESANOK.csv"
if (test-path "C:\Users\SESA27384\Desktop\AD\SESANOK.csv") {clear-content C:\Users\SESA27384\Desktop\AD\SESANOK.csv}
 
#création des 2 tables pour les logs (à faire avant le foreach)
$sesaNOK=@()
$sesaOK=@()
 
#import du fichier csv à une seule colonne
[System.Reflection.Assembly]::LoadWithPartialName("System.Windows.Forms")
 
$objForm = New-Object System.Windows.Forms.OpenFileDialog
$objForm.InitialDirectory = "c:\"
$objForm.Title = "Selectionner un fichier :"
$objForm.FilterIndex = 3
$Show = $objForm.ShowDialog()
 
If ($Show -eq "Cancel")
{
"Annulé par l'utilisateur"
}
Else
{
write-host $objForm.FileName
}
$path=$objForm.FileName
 
[void] [System.Reflection.Assembly]::LoadWithPartialName("System.Windows.Forms")
[void] [System.Reflection.Assembly]::LoadWithPartialName("System.Drawing") 
 
$objForm = New-Object System.Windows.Forms.Form 
$objForm.Text = "Select a Computer"
$objForm.Size = New-Object System.Drawing.Size(300,200) 
$objForm.StartPosition = "CenterScreen"
 
$objForm.KeyPreview = $True
$objForm.Add_KeyDown({if ($_.KeyCode -eq "Enter") 
    {$x=$objListBox.SelectedItem;$objForm.Close()}})
$objForm.Add_KeyDown({if ($_.KeyCode -eq "Escape") 
    {$objForm.Close()}})
 
$OKButton = New-Object System.Windows.Forms.Button
$OKButton.Location = New-Object System.Drawing.Size(75,120)
$OKButton.Size = New-Object System.Drawing.Size(75,23)
$OKButton.Text = "OK"
$OKButton.Add_Click({$x=$objListBox.SelectedItem;$objForm.Close()})
$objForm.Controls.Add($OKButton)
 
$CancelButton = New-Object System.Windows.Forms.Button
$CancelButton.Location = New-Object System.Drawing.Size(150,120)
$CancelButton.Size = New-Object System.Drawing.Size(75,23)
$CancelButton.Text = "Cancel"
$CancelButton.Add_Click({$objForm.Close()})
$objForm.Controls.Add($CancelButton)
 
$objLabel = New-Object System.Windows.Forms.Label
$objLabel.Location = New-Object System.Drawing.Size(10,20) 
$objLabel.Size = New-Object System.Drawing.Size(280,20) 
$objLabel.Text = "Please select a computer:"
$objForm.Controls.Add($objLabel) 
 
$objListBox = New-Object System.Windows.Forms.ListBox 
$objListBox.Location = New-Object System.Drawing.Size(10,40) 
$objListBox.Size = New-Object System.Drawing.Size(260,20) 
$objListBox.Height = 80
 
[void] $objListBox.Items.Add("GAD-SU-MediaReviewUATGroup")
[void] $objListBox.Items.Add("GAD-SU-MediaReviewUserGroup")
[void] $objListBox.Items.Add("GAD-SU-NextGenPDM_NonPROD")
[void] $objListBox.Items.Add("NextGenPDM_PrePROD")
[void] $objListBox.Items.Add("GAD-SU-NextGenPDM_PROD")
[void] $objListBox.Items.Add("GAD-SU-TES-Crash_plan")
 
$objForm.Controls.Add($objListBox) 
 
$objForm.Topmost = $True
 
$objForm.Add_Shown({$objForm.Activate()})
[void] $objForm.ShowDialog()
 
 
$ADgroup=$objListBox.text #on affecte le choix de la liste déroulante dans la variable $ADgroup
 
$csv = Import-Csv -path $path
foreach($line in $csv)
{ 
    $columnvalue = $line.name
 
	$data = @()
	$domains = "gad.schneider-electric.com","eur.gad.schneider-electric.com","apa.gad.schneider-electric.com","nam.gad.schneider-electric.com","gmea.gad.schneider-electric.com"
	foreach($domain in $domains)
	{
		$data += Get-ADUser -filter "((EmployeeID -like '$columnvalue') -OR (name -like '$columnvalue')) -AND (enabled -eq '$true') " -Server $domain 
	}
	[bool]$data
	if ($data) { Write-Host "Insert $columnvalue"
 
 
 
    $Group = Get-ADGroup -filter {name -$ADGroup}
    Write-Host "Group=$Group /ADgroup=$ADgroup"
    Add-ADGroupMember -Identity $Group -Member $data -Server "gad.schneider-electric.com"
 
	$sesaOK +=$line   #si $data existe, on a inséré le SESA et ajoût de la ligne (donc le SESA) ds la variable $sesaOK
        # ces 4 lignes, pour mémoire : 
        #foreach ($User in $data)    #on explore la variable $data{
        # Write-Host $user.SamAccountName   #pour chq ligne, on sélectionne la propriété SamAccountName (qui contient le SESA)
        # $New=$User|Select-Object SamAccountName   #construire un objet avec une seule propriété
        # $New | Export-csv -path C:\Users\SESA27384\Desktop\AD\SESAOK.csv -Append   #export de cet objet
        #}
}
if (!$data) {Write-Host "Entry Not Found $columnvalue" 
                    $sesaNOK +=$line     #SESA pas inséré donc ajoût de la ligne (donc le SESA) ds la variable $sesaNOK
                    }
} 
#export des 2 variables ds les 2 fichiers de log
$sesaNOK | Export-csv -path C:\Users\SESA27384\Desktop\AD\SESANOK.csv
 
$sesaOK  | Export-csv -path C:\Users\SESA27384\Desktop\AD\SESAOK.csv

$Group est encore vide. Il faut donc me proposer autre chose pour la ligne 104...

EDIT3 : je viens de voir un message d'erreur :

Citation:

Get-ADGroup : Erreur lors de l’analyse de la requête: «name -$ADGroup» Message d’erreur: «Operator Not supported: » à la position: «6
».
At C:\Users\sesa27384\Documents\aalaurent\boulot\powershell\scripts\PopulateADGroupetape2-3.ps1:108 char:14

Bonsoir,

j'ai contourné le problème et finalement, je n'utilise pas de variable dans la cmdlet mais j'obtiens le comportement souhaité donc je suis satisfait. L'idée est qu'au lieu d'utiliser une variable générée par une liste déroulante dans une cmdlet, je génère cette variable par la liste déroulante, puis j'effectue un switch avec autant d'appels à la cmdlet que de valeurs possibles pour cette variable :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
#début liste déroulante 
[void] [System.Reflection.Assembly]::LoadWithPartialName("System.Windows.Forms")
[void] [System.Reflection.Assembly]::LoadWithPartialName("System.Drawing") 
 
$objForm = New-Object System.Windows.Forms.Form 
$objForm.Text = "Select a Computer"
$objForm.Size = New-Object System.Drawing.Size(300,200) 
$objForm.StartPosition = "CenterScreen"
 
$objForm.KeyPreview = $True
$objForm.Add_KeyDown({if ($_.KeyCode -eq "Enter") 
    {$x=$objListBox.SelectedItem;$objForm.Close()}})
$objForm.Add_KeyDown({if ($_.KeyCode -eq "Escape") 
    {$objForm.Close()}})
 
$OKButton = New-Object System.Windows.Forms.Button
$OKButton.Location = New-Object System.Drawing.Size(75,120)
$OKButton.Size = New-Object System.Drawing.Size(75,23)
$OKButton.Text = "OK"
$OKButton.Add_Click({$x=$objListBox.SelectedItem;$objForm.Close()})
$objForm.Controls.Add($OKButton)
 
$CancelButton = New-Object System.Windows.Forms.Button
$CancelButton.Location = New-Object System.Drawing.Size(150,120)
$CancelButton.Size = New-Object System.Drawing.Size(75,23)
$CancelButton.Text = "Cancel"
$CancelButton.Add_Click({$objForm.Close()})
$objForm.Controls.Add($CancelButton)
 
$objLabel = New-Object System.Windows.Forms.Label
$objLabel.Location = New-Object System.Drawing.Size(10,20) 
$objLabel.Size = New-Object System.Drawing.Size(280,20) 
$objLabel.Text = "Please select a computer:"
$objForm.Controls.Add($objLabel) 
 
$objListBox = New-Object System.Windows.Forms.ListBox 
$objListBox.Location = New-Object System.Drawing.Size(10,40) 
$objListBox.Size = New-Object System.Drawing.Size(260,20) 
$objListBox.Height = 80
 
 
[void] $objListBox.Items.Add("GAD-SU-MediaReviewUATGroup")
[void] $objListBox.Items.Add("GAD-SU-MediaReviewUserGroup")
[void] $objListBox.Items.Add("GAD-SU-NextGenPDM_NonPROD")
[void] $objListBox.Items.Add("NextGenPDM_PrePROD")
[void] $objListBox.Items.Add("GAD-SU-NextGenPDM_PROD")
[void] $objListBox.Items.Add("GAD-SU-TES-Crash_plan")
 
$objForm.Controls.Add($objListBox) 
 
$objForm.Topmost = $True
 
$objForm.Add_Shown({$objForm.Activate()})
[void] $objForm.ShowDialog()
 
$ADgroup=$objListBox.text #on affecte le choix de la liste déroulante dans la variable $ADgroup
#fin liste déroulante
...
switch ( $ADgroup )
    {
        "GAD-SU-MediaReviewUATGroup"  {Add-ADGroupMember -Identity "GAD-SU-MediaReviewUATGroup" -Member $data -Server "gad.schneider-electric.com";break}
        "GAD-SU-MediaReviewUserGroup" {Add-ADGroupMember -Identity "GAD-SU-MediaReviewUserGroup" -Member $data -Server "gad.schneider-electric.com";break}
        "GAD-SU-NextGenPDM_NonPROD" {Add-ADGroupMember -Identity "GAD-SU-NextGenPDM_NonPROD" -Member $data -Server "gad.schneider-electric.com";break}
        "NextGenPDM_PrePROD" {Add-ADGroupMember -Identity "NextGenPDM_PrePROD" -Member $data -Server "gad.schneider-electric.com";break} 
        "GAD-SU-NextGenPDM_PROD" {Add-ADGroupMember -Identity "GAD-SU-NextGenPDM_PROD" -Member $data -Server "gad.schneider-electric.com";break}
        "GAD-SU-TES-Crash_plan" {Add-ADGroupMember -Identity "GAD-SU-TES-Crash_plan" -Member $data -Server "gad.schneider-electric.com";break}
    }

05/05/2017, 07h49
ericlm128

Évite le Switch pour ton cas présent. Le ".text" doit fonctionner.

Le problème de ta ligne 104 est simplement qu'il manque l'opérateur "-eq".

Ta ligne corrigée :

Code:

$Group = Get-ADGroup -filter {name -eq $ADGroup}

J'y ai cru, mais je viens d'essayer et $group reste vide...

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
Write-Host "Hello, World"
 
#vider le contenu des 2 fichiers, s'ils existent
#$strFileName="C:\Users\SESA27384\Desktop\AD\SESAOK.csv"
if (test-path "C:\Users\SESA27384\Desktop\AD\SESAOK.csv") {clear-content C:\Users\SESA27384\Desktop\AD\SESAOK.csv}
#$strFileName="C:\Users\SESA27384\Desktop\AD\SESANOK.csv"
if (test-path "C:\Users\SESA27384\Desktop\AD\SESANOK.csv") {clear-content C:\Users\SESA27384\Desktop\AD\SESANOK.csv}
 
#création des 2 tables pour les logs (à faire avant le foreach)
$sesaNOK=@()
$sesaOK=@()
 
#import du fichier csv à une seule colonne
[System.Reflection.Assembly]::LoadWithPartialName("System.Windows.Forms")
 
$objForm = New-Object System.Windows.Forms.OpenFileDialog
$objForm.InitialDirectory = "c:\"
$objForm.Title = "Selectionner un fichier :"
$objForm.FilterIndex = 3
$Show = $objForm.ShowDialog()
 
If ($Show -eq "Cancel")
{
"Annulé par l'utilisateur"
}
Else
{
write-host $objForm.FileName
}
$path=$objForm.FileName
 
#Ouvre une fenêtre.
[reflection.assembly]::LoadWithPartialName("System.Windows.Forms")
$form1 = New-Object Windows.Forms.Form
$form1.text = "Contrôles"             
$form1.Size = New-Object System.Drawing.Size(300,200)
#Liste déroulante (ComboBox).
$liste1 = New-Object System.Windows.Forms.Combobox
$liste1.Location = New-Object Drawing.Point 20,20
$liste1.Size = New-Object System.Drawing.Size(250,30)
$liste1.DropDownStyle = "DropDownList"
$liste1.Items.AddRange(("GAD-SU-NextGenPDM_NonPROD","GAD-SU-NextGenPDM_PrePROD","GAD-SU-MediaReviewUATGroup","GAD-SU-MediaReviewUserGroup"))
$liste1.SelectedIndex = 0
#Attache le contrôle à la fenêtre
$form1.controls.add($liste1)
#Affiche le tout.
$form1.ShowDialog()
#Affiche la sélection.
write-host "ComboBox = " $liste1.Text
$ADgroup=$liste1.text #on affecte le choix de la liste déroulante dans la variable $ADgroup
#fin liste déroulante
#Fin.
 
$csv = Import-Csv -path $path
foreach($line in $csv)
{ 
    $columnvalue = $line.name
 
				$data = @()
				$domains = "gad.schneider-electric.com","eur.gad.schneider-electric.com","apa.gad.schneider-electric.com","nam.gad.schneider-electric.com","gmea.gad.schneider-electric.com"
				foreach($domain in $domains)
				{
					$data += Get-ADUser -filter "((EmployeeID -like '$columnvalue') -OR (name -like '$columnvalue')) -AND (enabled -eq '$true') " -Server $domain 
				}
				[bool]$data
				if ($data) { Write-Host "Insert $columnvalue"
 
 
                         $Group = Get-ADGroup -filter {name -eq $ADGroup}
                        Write-Host "Group=$Group /ADgroup=$ADgroup"
                        Add-ADGroupMember -Identity $Group -Member $data -Server "gad.schneider-electric.com"
 
 
 
 
						#Add-ADGroupMember "CN=GAD-SU-NextGenPDM_NonPROD,OU=Access Management,OU=Groups,DC=gad,DC=schneider-electric,DC=com" -server gad.schneider-electric.com -Member $data
			            $sesaOK +=$line   #si $data existe, on a inséré le SESA et ajoût de la ligne (donc le SESA) ds la variable $sesaOK
                         # ces 4 lignes, pour mémoire : 
                         #foreach ($User in $data)    #on explore la variable $data{
                          # Write-Host $user.SamAccountName   #pour chq ligne, on sélectionne la propriété SamAccountName (qui contient le SESA)
                          # $New=$User|Select-Object SamAccountName   #construire un objet avec une seule propriété
                          # $New | Export-csv -path C:\Users\SESA27384\Desktop\AD\SESAOK.csv -Append   #export de cet objet
                         #}
				}
				if (!$data) { Write-Host "Entry Not Found $columnvalue" 
                              $sesaNOK +=$line     #SESA pas inséré donc ajoût de la ligne (donc le SESA) ds la variable $sesaNOK
				}
} 
#export des 2 variables ds les 2 fichiers de log
$sesaNOK | Export-csv -path C:\Users\SESA27384\Desktop\AD\SESANOK.csv
 
$sesaOK  | Export-csv -path C:\Users\SESA27384\Desktop\AD\SESAOK.csv

Citation:

Hello, World

GAC Version Location
--- ------- --------
True v4.0.30319 C:\windows\Microsoft.Net\assembly\GAC_MSIL\System.Windows.Forms\v4.0_4.0.0.0__b77a5c561934e089\System.Window...
C:\Users\sesa27384\Desktop\AD\testunecolonne2.csv
True v4.0.30319 C:\windows\Microsoft.Net\assembly\GAC_MSIL\System.Windows.Forms\v4.0_4.0.0.0__b77a5c561934e089\System.Window...
Cancel
ComboBox = GAD-SU-NextGenPDM_NonPROD
True
Insert sesa449786
Group= /ADgroup=GAD-SU-NextGenPDM_NonPROD
Add-ADGroupMember : Cannot validate argument on parameter 'Identity'. The argument is null. Provide a valid value for the argument,
and then try running the command again.
At C:\Users\sesa27384\Documents\aalaurent\boulot\powershell\scripts\PopulateADGroupetape2.ps1:71 char:53
+ Add-ADGroupMember -Identity $Group -Member $data -Server ...
+ ~~~~~~
+ CategoryInfo : InvalidData: (:) [Add-ADGroupMember], ParameterBindingValidationException
+ FullyQualifiedErrorId : ParameterArgumentValidationError,Microsoft.ActiveDirectory.Management.Commands.AddADGroupMember
...etc

05/05/2017, 15h00
ericlm128

La vérité est ailleurs car j'ai fait le test et $ADgroup contient bien le choix de la liste déroulante.

Sinon j'ai simulé ton programme sur notre AD. Il trouve bien l'utilisateur, le groupe et l'ajoute correctement.
05/05/2017, 15h03
ericlm128

tu pourrait tenter de spécifier ton serveur AD. Mais tout ceci dépend de ton écosystème.

Code:

$Group = Get-ADGroup -filter {name -eq $ADGroup} -Server "gad.schneider-electric.com"
05/05/2017, 15h23
laurentSc

Citation:

Envoyé par ericlm128

tu pourrait tenter de spécifier ton serveur AD. Mais tout ceci dépend de ton écosystème.

Code:

$Group = Get-ADGroup -filter {name -eq $ADGroup} -Server "gad.schneider-electric.com"

:ccool: c'est une bonne solution
06/05/2017, 00h34
laurentSc

Bonsoir,
j'aimerais bien comprendre cette "magie". Pourrais-tu m'expliquer ce que fait cette ligne :
$Group = Get-ADGroup -filter {name -eq $ADGroup} -Server "gad.schneider-electric.com"
06/05/2017, 19h03
ericlm128

Get-ADGroup
Permet de récupérer un ou plusieurs groupes active directory
https://technet.microsoft.com/fr-fr/.../ee617196.aspx

Filter
Permet de spécifier des critères de recherche

Server
Permet de spécifier l'instance d'active directory

En bref :
La commande "Get-ADGroup" cherche dans l'AD du serveur "gad.schneider-electric.com" un groupe qui à pour Nom le contenu de $ADGroup. Et nous stockons le résultat dans $Group

Merci pour cette explication, et comprends-tu comment on construit la variable $data dans le script que voici :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
 
...
$csv = Import-Csv -path $path #import du fichier csv
 
foreach($line in $csv)
{ 
        $columnvalue = $line.name
 
	$data = @()
	$domains = "gad.schneider-electric.com","eur.gad.schneider-electric.com","apa.gad.schneider-electric.com","nam.gad.schneider-electric.com","gmea.gad.schneider-electric.com"
	foreach($domain in $domains)
	{
		$data += Get-ADUser -filter "((EmployeeID -like '$columnvalue') -OR (name -like '$columnvalue')) -AND (enabled -eq '$true') " -Server $domain 
	}
	[bool]$data
	if ($data) { 
            Write-Host "Insert $columnvalue"                        
 
            $Group = Get-ADGroup -filter {name -eq $ADGroup}  -Server "gad.schneider-electric.com" #génération d'une variable comprise pr la cmdlet
            Add-ADGroupMember -Identity $Group -Member $data -Server "gad.schneider-electric.com"
 
            $sesaOK +=$line   #si $data existe, on a inséré le SESA et ajoût de la ligne (donc le SESA) ds la variable $sesaOK
 
        }
        if (!$data) { Write-Host "Entry Not Found $columnvalue" 
                        $sesaNOK +=$line     #SESA pas inséré donc ajoût de la ligne (donc le SESA) ds la variable $sesaNOK
        }
}

Explication

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
$csv = Import-Csv -path $path #import du fichier csv
 
# Pour chaque ligne de l'import csv
foreach($line in $csv)
{ 
    # Déclaration de $data en type tableau
    $data = @()
 
    # cré la liste des domaines
    $domains = "gad.schneider-electric.com","eur.gad.schneider-electric.com","apa.gad.schneider-electric.com","nam.gad.schneider-electric.com","gmea.gad.schneider-electric.com"
 
    # Pour chaque domaine
    foreach($domain in $domains)
    {
        # Recherche l'utilisateur pour ce domaine avec les critères, et l'ajoute au tableau $data
	    $data += Get-ADUser -filter "((EmployeeID -like '$columnvalue') -OR (name -like '$columnvalue')) -AND (enabled -eq '$true') " -Server $domain 
    }
 
    # Aucun interet surement, peut être présent pour le débug
    [bool]$data
 
    # Si $data contient quelques chose (est égal à $true)
    if ($data)
    { 
        Write-Host "Insert $columnvalue"                        
 
        $Group = Get-ADGroup -filter {name -eq $ADGroup}  -Server "gad.schneider-electric.com" #génération d'une variable comprise pr la cmdlet
        # Ajout du ou des utilisateurs $data au groupe $Group du serveur "gad.schneider-electric.com"
        Add-ADGroupMember -Identity $Group -Member $data -Server "gad.schneider-electric.com"
 
        $sesaOK +=$line   #si $data existe, on a inséré le SESA et ajoût de la ligne (donc le SESA) ds la variable $sesaOK
 
    }
 
    # Si $data est vide (est différent de $true)
    if (!$data)
    {
        Write-Host "Entry Not Found $columnvalue" 
        $sesaNOK +=$line     #SESA pas inséré donc ajoût de la ligne (donc le SESA) ds la variable $sesaNOK
    }
}

Tu peux aussi ajouter la vérification sur l’existence du groupe AD

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
$csv = Import-Csv -path $path
$domains = "gad.schneider-electric.com","eur.gad.schneider-electric.com","apa.gad.schneider-electric.com","nam.gad.schneider-electric.com","gmea.gad.schneider-electric.com"
 
foreach($line in $csv)
{ 
    $data = @()
    foreach($domain in $domains)
    {
	    $data += Get-ADUser -filter "((EmployeeID -like '$columnvalue') -OR (name -like '$columnvalue')) -AND (enabled -eq '$true') " -Server $domain 
    }
 
    if ($data)
    { 
        Write-Host "Insert $columnvalue"
        $Group = Get-ADGroup -filter {name -eq $ADGroup}  -Server "gad.schneider-electric.com"
        if ($Group)
        {
            Add-ADGroupMember -Identity $Group -Member $data -Server "gad.schneider-electric.com"
            $sesaOK += $line
        }
        else
        {
            Write-Host "Group Not Found $columnvalue : $ADGroup"
            $sesaNOK += $line
        }
    }
    else
    {
        Write-Host "Entry Not Found $columnvalue" 
        $sesaNOK += $line
    }
}

07/05/2017, 14h24
laurentSc

Merci de nouveau pour l'explication.

En fait, je m'aperçois que j'avais déjà compris la logique de ce script, mais quand tu dis "Recherche l'utilisateur pour ce domaine avec les critères", j'aurais pas été capable de pondre cette ligne (notamment les critères).

La ligne 20, que tu juges inutile et juste pour le debug, aucune idée de son rôle. Si je la supprime, le résultat est identique alors que j'avais déjà essayé et ce n'était pas le cas...