add list of users into Global Security Group

Hi

I have a Global Security group that I need to import over 100 users and I tried this. Where am I going wrong with this?

import-module activedirectory

$list =import-csv "C:\test\export\userlist.csv"

forEach ($user in $list)

{

add-ADGroupMember -identity mytestgroup  -member $list

}
kuzumAsked:
Who is Participating?

Improve company productivity with a Business Account.Sign Up

x
 
oBdAConnect With a Mentor Commented:
The attachment is not a csv (and, sorry, I'm not opening downloaded Office documents).
But if you didn't change the 'SamAccountName', then it seems to be a real csv, in which case you need to the property you want to use. Could be like this:
import-module activedirectory
$list =import-csv "C:\test\export\userlist.csv"
forEach ($user in $list) {
	add-ADGroupMember -identity mytestgroup  -member $user.SamAccountName
}

Open in new window

0
 
oBdACommented:
Maybe stating the way-too-obvious here, but you're using "$List" as member to add, not "$User".
0
 
oBdACommented:
On closer inspection, you're using the import incorrectly as well.
Either your csv is a real csv with a header row, in which case you need to use the property (column) of the rows you imported, for example "-member $User.DistinguishedName", or you're using a flat file without header, in which case you need to use "Get-Content" instead of "Import-Csv".
0
Worried about phishing attacks?

90% of attacks start with a phish. It’s critical that IT admins and MSSPs have the right security in place to protect their end users from these phishing attacks. Check out our latest feature brief for tips and tricks to keep your employees off a hackers line!

 
kuzumAuthor Commented:
hmm, thanks. I changed it to get-content and error changed. I also attached the csv file? did it need to be .txt?
Add-ADGroupMember : Cannot find an object with identity: 'SamAccountName' under: 'DC=xx,DC=xx,DC=local'.
At line:9 char:18
+ add-ADGroupMember <<<<  -identity mytestGroup -member $user
    + CategoryInfo          : ObjectNotFound: (SamAccountName:ADPrincipal) [Add-ADGroupMember], ADIdentityNotFoundException
    + FullyQualifiedErrorId : SetADGroupMember.ValidateMembersParameter,Microsoft.ActiveDirectory.Management.Commands.AddADGroupMember
doc10.docx
0
 
Dustin SaundersDirector of OperationsCommented:
@kazum-

Yes, generally you'd want to either paste the contents of the CSV into a code block or attach a .txt or .csv file (office documents can execute macros when downloaded so most experts won't open them.)  You can also use a screenshot (png, jpg, etc)

To echo oBdA:

when you use this line:

forEach ($user in $list)

Open in new window

You're creating each $user as a row in the CSV.  So you use . and then the column name to reference the field.

For example, if your CSV is:
First,Last,FavoriteColor
John,Test,Blue

Open in new window


This code would do the following:
$list = import-csv "C:\test\export\userlist.csv"
foreach ($user in $list) {
$output = $user.First + " " + $user.Last + "'s favorite color is " + $user.FavoriteColor + "."
Write-Host $output
}

Open in new window

Would write "John Test's favorite color is Blue."
0
 
kuzumAuthor Commented:
thank you for your excellent support gents
0
Question has a verified solution.

Are you are experiencing a similar issue? Get a personalized answer when you ask a related question.

Have a better answer? Share it in a comment.

All Courses

From novice to tech pro — start learning today.