Hi,

Are you using version 1.3 of the cmdlets per chance?

I have had a similar problem with a script that works in about 200Mb on version 1.22 but kills my system with version 1.3. after using about 1.7Gb!

It seems like the get-qaduser does not clean up the object memory (finalization step?) when you are running it through a loop.

Try it with a version previous to 1.3 and see if it improves the memory management.

Gareth

hello,

if anybody is interested - here is the final script i use now:

$Date = Get-Date

$LogFile = "C:\PSScripts\Logs\UserProfileMaintenance-DeleteProfilesOlder.txt"
$LogFileDaily = "C:\PSScripts\Logs\UserProfileMaintenance-" + $Date.DayOfWeek + ".txt"

$emailFrom = "profileadmin@domain.local"
$smtpServer = "smtp-relay.domain.local"

# Check if file already exists and delete before appending new data
if( Test-Path $LogFile) {Remove-Item $LogFile -force}
if( Test-Path $LogFileDaily) {Remove-Item $LogFileDaily -force}

Get-ChildItem \\profiles\Userprofiles$ | where {$_.psiscontainer} | ForEach-Object {
 $Deleted = $False

 If ((Get-User ("DOMAIN\" + $_.Name.Replace(".V2","")) -ErrorAction SilentlyContinue) -ne $Null) {
  $UAC = (([ADSI]("LDAP://" + (Get-User ("DOMAIN\" + $_.Name.Replace(".V2",""))).DistinguishedName)).UserAccountControl).ToString()
  $Disabled = $False
  Switch ($UAC) {
   {($UAC -bor 2) -eq $UAC} {$Disabled = $True}
  }
  
  If ($Disabled -eq $True) {
   Write-Host "\\profiles\Userprofiles$\$_ Deleted - Account Disabled!"
   Add-Content -path $LogFile -value "\\profiles\Userprofiles$\$_ Deleted - Account Disabled!"
   Remove-Item $_.FullName -Force -Recurse -WhatIf
   $Deleted = $True
  }
 }
 Else {
  Write-Host "\\profiles\Userprofiles$\$_ Deleted - Account not found!"
  Add-Content -path $LogFile -value "\\profiles\Userprofiles$\$_ Deleted - Account not found!"
  Remove-Item $_.FullName -Force -Recurse -WhatIf
  $Deleted = $True
 }
 
 if (($_.LastWriteTime -lt $Date.AddDays(-60)) -and ($Deleted -eq $False)) {
  Write-Host "\\profiles\Userprofiles$\$_ Deleted - Profile older than 60 Days!"
  Add-Content -path $LogFile -value "\\profiles\Userprofiles$\$_ Deleted - Profile older than 60 Days!"
  Remove-Item $_.FullName -Force -Recurse -WhatIf
  $Deleted = $True
 }
 
 if (($_.name -match '!|_') -and ($Deleted -eq $False)) {
  Write-Host "\\profiles\Userprofiles$\$_ Deleted - Renamed Profile!"
  Add-Content -path $LogFile -value "\\profiles\Userprofiles$\$_ Deleted - Renamed Profile!"
  Remove-Item $_.FullName -Force -Recurse -WhatIf
  $Deleted = $True
 }
 
 If ($Deleted -eq $False) {
  $Size = (Get-ChildItem $_.FullName -Recurse -Force | Measure-Object Length -Sum).Sum /1MB
  Add-Content -path $LogFileDaily -value ("$($_.FullName);" + $Size.ToString("#.##"))
 
  If ($Size -gt 150) {
   $subject = "Userprofil Warning " + $_.Name.Replace(".V2","")
   $emailTo = $_.Name.Replace(".V2","") + "@domain.local"
   $body = "bla bla bla " + $Size.ToString("#.##") + " bla bla bla"
   $smtp = new-object Net.Mail.SmtpClient($smtpServer)
   $smtp.Send($emailFrom, $emailTo, $subject, $body)
  }
 }
}

i didn´t used your script exactly because:

.) it is a litte bit more complex - i don´t want to check the size of a profile that i delete and sending an e-mail to a disabled user doesn´t make sense
.) only do it once - checking a deleted folder does not make sense...
.) when you format the size string directly you can´t compare it
.) i forgot that win7 adds a .v2 at the end of the profile name - this i added here
.) without the get-qaduser it is a little bit more code at the moment, but runs MUCH better until quest fixes this huge memory leak.

brgds Deas