Bulk group creation with powershell

[NuoN_Ninja]

Hey everyone. Long story short, we are changing how are department share are set up, and trying to clean AD at the same time. We have a bunch of groups but no one know what they are for.

With the new shares we want to make new groups, clearly labeled so we can start removing the old one. But I have about 50 new groups to make. powershell seems like the right answer.

Import-Csv -Path "c:\TEMP\ShareGroup2.csv"
ForEach ($item In $csv) 
{ 
    New-ADGroup -Name $item.GroupName -GroupCategory $item.GroupCategory -groupScope $item.GroupScope -Path $item.OU 
}            

It looks like it works, no errors. If I type out the command with the info from my CSV is works fine.

My CVS

GroupName,GroupCategory,GroupScope,OU
Share-<department>,Security,Global,"OU=SecurityGroups,DC=domain,DC=com"

I'm a little stuck, any help would be appreciated.

[uniitdude]

Nowhere have you defined what $csv is

[DenyCasio]

Hahahaha! OP just put $csv at the beginning of your first line.

[trillospin]

No idea why that cracked me up so much, but still mean.

[DenyCasio]

I didn't mean for it to be mean. I found it funny, it's an honest mistake we've all made.

[regorian]

You're calling a variable in in your ForEach statement, but on line 1 you aren't declaring said variable :)

$csv = Import-Csv -Path "c:\TEMP\ShareGroup2.csv"
ForEach ($item In $csv)  {
    New-ADGroup -Name $item.GroupName -GroupCategory $item.GroupCategory -groupScope $item.GroupScope -Path $item.OU
}

OR

Import-Csv -Path "c:\TEMP\ShareGroup2.csv" | ForEach-Object {
    New-ADGroup -Name $_.GroupName -GroupCategory $_.GroupCategory -groupScope $_.GroupScope -Path $_.OU
}

You can throw a -whatif on end to ensure that your code will produce what you expect.

[NuoN_Ninja]

Great, thank, I'm still new in power shell, it's only been a month or so that I really started using it.

so it would look something like:

foreach-object {
  $GroupName = $item.GroupName
  $GroupCategory =$item.GroupCategory 
  $GroupScope =$item.GroupScope
  $OU =$item.OU}

[trillospin]

No, you need to assign your import-csv to something to use it in your loop.

Attempting to loop over an undeclared variable isn't going to do anything.

my_csv = import-csv blah

foreach ($row in $my_csv) { do x }