Adding AD users to security group using a csv file using powershell on windows server 2008 r2

Instead of using Add-ADGroupMember, I use Add-ADPrincipalGroupMembership -Identity $instloginID -MemberOf $instGroup. It works well for me.

I'm a little confused about why you are putting the information in an array. I am not a pro; have only been running posh about 2.5 years. So my question is as much for my benefit as it is for yours. Does the array make it run faster? I plug each part of my user definition into its own variable and do it that way. The code looks a lot more straight forward and possibly easier to maintain that way.

My code to create home directories:

function Create-HomeDirs ($Synonym, $Number2Make, $studhomedir) 
{
###################################################################################
# This function creates student home directories. It also assigns permissions.    #
###################################################################################

    for ($i=1; $i -le $Number2Make; $i++)                           # Create homedirs from synonym                                       
        {
        $NewUser = ($Synonym + $i.ToString("00"))                   # Pad last two digits with zeroes so you get xxxxx01 instead of xxxxx1 
        $HomeDir = "$studhomedir\$NewUser"
        $Principal= "domain\$NewUser"

        write-host "HomeDir Being Created = $HomeDir"
        New-Item ($HomeDir) –Type Directory

        }   #end for
#
# The piece that assigns permissions sometimes fails due to sync problems. 
# This is why I put the "read-host" command in the code, to slow it down.
#  
$x = (read-host "`nReady to do permisisons? Press <Enter> to continue.")
   for ($i=1; $i -le $Number2Make; $i++)                           # Add permissions to homedirs                                       
     {
        $NewUser = ($Synonym + $i.ToString("00"))                   # Pad last two digits with zeroes so you get xxxxx01 instead of xxxxx1 
        $HomeDir = "$studhomedir\$NewUser"
        $Principal= "domain\$NewUser"
        write-host "newuser = $newuser. Homedir = $homeDir. Principal = $principal"
        write-host "HomeDir Permissions being created = $HomeDir"

        $Rights = [System.Security.AccessControl.FileSystemRights]"FullControl"                      # This line and next 3 put security settings in variables 
        $InheritanceFlag = [System.Security.AccessControl.InheritanceFlags]"ContainerInherit, ObjectInherit"
        $PropagationFlag = [System.Security.AccessControl.PropagationFlags]::None 
        $objType =[System.Security.AccessControl.AccessControlType]::Allow 

        $objACE=New-Object System.Security.AccessControl.FileSystemAccessRule($Principal, $Rights, $InheritanceFlag, $PropagationFlag, $objType) 
        $objACL = Get-ACL $HomeDir                                                        # Get existing ACL for home directory

        $objace

        if ($objACe)
           {
           $objACL.AddAccessRule($objACE)                                                    # Add ACE to this ACL

           Set-ACL $HomeDir $objACL                                                          # Put modified ACL back on home directory
           }
        else
           {write-host "objACL appears to be empty, line 359"}
     }   #end for

} # end function Create-HomeDirs

Hey guys I figured it out I guess I won't fail after all :)

#Importing the Users
Import-CSV C:\Users\Administrator\Desktop\users.csv | foreach-object {New-ADUser -Name  $_.Name -GivenName $_.FirstName -Surname $_.LastName -Enabled $True -PasswordNeverExpires $True -AccountPassword (ConvertTo-SecureString $_.Password -AsPlainText -Force) -ChangePasswordAtLogon $False -Department $_.Department -EmailAddress $_.Email -OfficePhone $_.Phone -Path $_.Path -SamAccountName $_.SamAccountName -Title $_.Title -UserPrincipalName $_.UPN}

#Adding the Users to the Groups
Import-CSV C:\Users\Administrator\Desktop\Powershell\usergroups.csv | ForEach-Object {
$SAM = $_.SAM
$Group = $_.Group
$Groups = $Group.split(" ")
foreach($l in $Groups){
Add-ADGroupMember -Identity $l -Member $SAM
}
}

It's looking for the group identity, which was omitted from the group array and needs to inserted as a $_. variable.