So far most of our scripts have been a little bereft of error checking. This has mainly been to make the easier to read. If you’ve been experimenting with the scripts, then it’s almost certain that you’ll have come across the following unhelpful error message from PowerShell:
Exception calling "SqlBackup" with "1" argument(s): "Backup failed for Server 'Server1'. " At line:1 char:1 + $Backup.SqlBackup($SQLSvr) + ~~~~~~~~~~~~~~~~~~~~~~~~~~ + CategoryInfo : NotSpecified: (:) [], MethodInvocationException + FullyQualifiedErrorId : FailedOperationException
You’ll probably also have discovered that amongst many things it can be a response to:
- Wrong Server name
- Permissions error on backup destination
- Contradictory parameters
- DB exists and you’ve not specified replace
- You’ve specified a filename that’s already in use
- Many, many, more
This means that for scripts that you want to be reliable for automation that you need to make sure you’ve got robust checking in place. Or at least a good logging system so you can go back and review the settings for each run. Common issues, and how to check for them are listed below
Missing folders, or files that already exist. PowerShell provides Test-Path
– does what it says on the tin, it tests a path to see if it exists, so you can quickly check if a path exists before writing to it, it returns $True
and $False
as appropriate. So to make sure that your backup folder exists before writing to it:
if (Test-Path "c:\mybackups\" -eq $False){ write-output "Missing folder" }
This could be modified to create the missing path if required, or if you actually wanted the file not to be there you can redirect your process.
Getting the SQL Server name wrong or the server not being available. When you build your new SQL Server object like so:
$sqlsvr = New-Object -TypeName Microsoft.SQLServer.Management.Smo.Server("WrongServer")
you might think that PowerShell would warn you if the server didn’t exist, but it doesn’t. And you might not find out till you get the error message above. To check it as soon as you create it you need to try to use it:
$sqlsvr = New-Object -TypeName Microsoft.SQLServer.Management.Smo.Server("WrongServer") try{ $sqlsvr.databases | Out-Null } catch{ Write-Host "SQL Server Not Available" }
You’ll need to check the database you’re targeting for backup or restore exists, thought for different reasons. If you’re restoring and the database exists you need to set the Restore objects Replace
property to $True
:
$DBname="fdg" try{ $sqlsvr.Databases.item($DbName) | Get-Member -ErrorAction stop } catch{[System.Exception] <# DB does not exist. Choose to abort backup attempt. Set a flag that can be checked in in the Finally block #> $DBexists = $false } finally{ if ($DBexists -ne $False){ <# DB exists, choose whether to abort restore or set .Replace = $True #> } }
A common problem when backing up or restoring is checking that the SQL Server Service account has the correct permissions to access the area. Remember that PowerShell is just submitting T-SQL to SQL Server to execute, so the account running the script may have permissions.
Checking the permissions takes a bit more code but is also possible. This assumes we have a validated SQL Server connection called $sqlsvr
, and also that the account running the script has permissions to see the permissions on the requested object:
$serviceaccount = $sqlsvr.ServiceAccount #Need to append \\ to the end of the domain so it works in -replace $domain= $env:USERDOMAIN+"\\" #Path to Folder we want to check $path = "c:\folder1\folder2" #Get Group memberships for Service Account $groups = Get-ADPrincipalGroupMembership ($ServiceAccount -ireplace $domain,"") #Build up an Array to hold all the group and account names $accounts = @() $accounts += $groups.samaccountname $accounts += $ServiceAccount -ireplace $domain,"" #Set a check for whether we have write permissions $write=0 $read=0 #Get the ACL on our specified path $perms = get-acl $path #For each ACL in array foreach ($acl in $perms.access){ #Then for each acccount/group foreach ($acc in $accounts){ #If a right is assigned to a group or account if ("ads\"+$acc -ieq $acl.IdentityReference){ #Check if it's a permission that gives us write access if (($acl.FileSystemRights -ilike "*FullControl*") -or ($acl.FileSystemRights -ilike "*Modify*")){ if($acl.AccessControlType -ne "Allow"){ #Hit a Deny, overrides any other permission, so drop out of checking $write = 0 break } #Increment our write check. Keep looping as we may not have hit a deny yet $write++ #If we have write we have read $read++ }elseif ($acl.FileSystemRights -contains "Read"){ $read++ } } } } if ($write -eq 0){ write-output "SQL Server service account $serviceaccount doesn't have write access to $path" #Take action } if ($read -eq 0){ write-output "SQL Server service account $serviceaccount doesn't have read access to $path" #Take action }
If you’re accessing your path through a share, then you also need to check share permissions as they may override your FileSystem permissions (same assumptions as filesystem example above):
$serviceaccount = $sqlsvr.ServiceAccount #Need to append \\ to the end of the domain so it works in -replace $domain= $env:USERDOMAIN+"\\" #Path to Folder we want to check $path="\\Server1\Share3$\folder1" #Split the path on \ to get the bits we want $ShareArray = $path -Split [regex]::Escape('\') #Get Group memberships for Service Account $groups = Get-ADPrincipalGroupMembership ($ServiceAccount -ireplace $domain,"") #Build up an Array to hold all the group and account names $accounts = @() $accounts += $groups.samaccountname $accounts += $ServiceAccount -ireplace $domain,"" #Set a check for whether we have write permissions $write=0 $read=0 $share=$sharearray[3].Trim() $ShareSecurity = Get-WMIObject -Class Win32_LogicalShareSecuritySetting -Filter "name='$Share'" -ComputerName $ShareArray[2] $ShareSecurityDesc = $ShareSecurity.GetSecurityDescriptor().Descriptor foreach($ace in $SD.DACL){ foreach ($acc in $accounts){ $acl = New-Object Security.AccessControl.FileSystemAccessRule($ace.Trustee.name, $ace.AccessMask, $ace.AceType) #If a right is assigned to a group or account if ($acc -ieq $acl.IdentityReference){ #Check if it's a permission that gives us write access if (($acl.FileSystemRights -ilike "*FullControl*") -or ($acl.FileSystemRights -ilike "*Modify*")){ if($acl.AccessControlType -ne "Allow"){ #Hit a Deny, overrides any other permission, so drop out of checking $write = 0 break } #Increment our write check. Keep looping as we may not have hit a deny yet $write++ #If we have write we have read $read++ }elseif ($acl.FileSystemRights -ilike "*Read*"){ $read++ } } } }
Those examples cover a number of the most common errors I see when leaving backups and restores to run on their own. You can add more to cover cases as you discover them.
And at a minimum, ensure that everything is encased in try,catch, finally blocks (making sure to add -ErrorAction Stop
to non terminating cmdlet. And the next most useful thing is to use the script()
method to keep a record of what you’re asking SQL Server to do, a lot of the time you’ll look at it and realise what you’ve done wrong.
This post is part of a series posted between 1st September 2013 and 3rd October 2013, an index for the series is available here.
Leave a Reply