From: Patrick Canterino Date: Thu, 8 Mar 2018 14:56:10 +0000 (+0100) Subject: Added logging function (credits to Jason Wasser) and removed unnecessary output X-Git-Url: https://git.p6c8.net/dsmonrot.git/commitdiff_plain/a2f3d2a6a1f784fbed79a456e65070f12faf1626?hp=d76d903b4f6a1963cd7dbdab4bf130d5c03514a9 Added logging function (credits to Jason Wasser) and removed unnecessary output --- diff --git a/TODO.md b/TODO.md index 2326604..b725eeb 100644 --- a/TODO.md +++ b/TODO.md @@ -3,6 +3,9 @@ - [x] Collect error messages and send them via email - [x] Check if directories are created successfully - [x] ~~Check if the network drive already exists before connecting~~ ⇒ `New-PSDrive` throws an exception if drive is already connected -- [ ] Create a log file for the script -- [ ] Clean up the messages sent to the console or send them to the debug or error streams (e.g. `Write-Debug` or `Write-Error`) -- [x] Suppress output of some commands \ No newline at end of file +- [x] Create a log file for the script +- [x] Clean up the messages sent to the console or send them to the debug or error streams (e.g. `Write-Debug` or `Write-Error`) +- [ ] Rotate log files +- [x] Suppress output of some commands +- [ ] Add some comments to the source code +- [ ] Allow multiple backups for a day \ No newline at end of file diff --git a/dsmonrot.ps1 b/dsmonrot.ps1 index c0ee230..95508a1 100644 --- a/dsmonrot.ps1 +++ b/dsmonrot.ps1 @@ -13,14 +13,14 @@ # Path to backup directory [String]$backupDir = "Z:\" -# Keep backup for this amount of months (excluding the current month), +# Keep backups for this amount of months (excluding the current month), # -1 for indefinite [Int32]$keepMonths = 2 # Rotate BEFORE the beginning of a full backup (default is after a successful # full backup) # WARNING: If this option is set to $True and the full backup fails you could # have NO backup -$rotateBeforeBackup = $False +[Boolean]$rotateBeforeBackup = $False # Path to Drive Snapshot [String]$dsPath = "C:\Users\Patrick\Desktop\DSMonRot\snapshot.exe" # Path to Drive Snapshot log file (specify only the file name if you set @@ -32,8 +32,12 @@ $rotateBeforeBackup = $False [Boolean]$dsLogFileToBackup = $True # Disks to backup, see http://www.drivesnapshot.de/en/commandline.htm [String]$disksToBackup = "HD1:1" -# Path to DSMonRot log file -[String]$logFile = "C:\Users\Patrick\Desktop\DSMonRot\dsmonrot.log" +# Path to directory where DSMonRot stores the log files +# Every month a new log file is created +[String]$logDir = "C:\Users\Patrick\Desktop\DSMonRot\log" +# Keep log files for this amount of months (excluding the current month), +# 0 for indefinite (currently not available) +[Int32]$keepLogs = 1 # Map network share to this drive letter, comment out if you don't want to use it [String]$smbDrive = "Z" @@ -67,13 +71,126 @@ $rotateBeforeBackup = $False # End of config -$dsAdditionalArgs = @("--UseVSS") +# This function is originally by wasserja at https://gallery.technet.microsoft.com/scriptcenter/Write-Log-PowerShell-999c32d0 +<# +.Synopsis + Write-Log writes a message to a specified log file with the current time stamp. +.DESCRIPTION + The Write-Log function is designed to add logging capability to other scripts. + In addition to writing output and/or verbose you can write to a log file for + later debugging. +.NOTES + Created by: Jason Wasser @wasserja + Modified: 11/24/2015 09:30:19 AM + + Changelog: + * Code simplification and clarification - thanks to @juneb_get_help + * Added documentation. + * Renamed LogPath parameter to Path to keep it standard - thanks to @JeffHicks + * Revised the Force switch to work as it should - thanks to @JeffHicks + + To Do: + * Add error handling if trying to create a log file in a inaccessible location. + * Add ability to write $Message to $Verbose or $Error pipelines to eliminate + duplicates. +.PARAMETER Message + Message is the content that you wish to add to the log file. +.PARAMETER Path + The path to the log file to which you would like to write. By default the function will + create the path and file if it does not exist. +.PARAMETER Level + Specify the criticality of the log information being written to the log (i.e. Error, Warning, Informational) +.PARAMETER NoClobber + Use NoClobber if you do not wish to overwrite an existing file. +.EXAMPLE + Write-Log -Message 'Log message' + Writes the message to c:\Logs\PowerShellLog.log. +.EXAMPLE + Write-Log -Message 'Restarting Server.' -Path c:\Logs\Scriptoutput.log + Writes the content to the specified log file and creates the path and file specified. +.EXAMPLE + Write-Log -Message 'Folder does not exist.' -Path c:\Logs\Script.log -Level Error + Writes the message to the specified log file as an error message, and writes the message to the error pipeline. +.LINK + https://gallery.technet.microsoft.com/scriptcenter/Write-Log-PowerShell-999c32d0 +#> +function Write-Log +{ + [CmdletBinding()] + Param + ( + [Parameter(Mandatory=$true, + ValueFromPipelineByPropertyName=$true)] + [ValidateNotNullOrEmpty()] + [Alias("LogContent")] + [string]$Message, + + [Parameter(Mandatory=$false)] + [Alias('LogPath')] + [string]$Path='C:\Logs\PowerShellLog.log', + + [Parameter(Mandatory=$false)] + [ValidateSet("Error","Warn","Info")] + [string]$Level="Info", + + [Parameter(Mandatory=$false)] + [switch]$NoClobber + ) + + Begin + { + # Set VerbosePreference to Continue so that verbose messages are displayed. + $VerbosePreference = 'Continue' + } + Process + { + + # If the file already exists and NoClobber was specified, do not write to the log. + if ((Test-Path $Path) -AND $NoClobber) { + Write-Error "Log file $Path already exists, and you specified NoClobber. Either delete the file or specify a different name." + Return + } + + # If attempting to write to a log file in a folder/path that doesn't exist create the file including the path. + elseif (!(Test-Path $Path)) { + Write-Verbose "Creating $Path." + $NewLogFile = New-Item $Path -Force -ItemType File + } + + else { + # Nothing to see here yet. + } + + # Format Date for our Log File + $FormattedDate = Get-Date -Format "yyyy-MM-dd HH:mm:ss" + + # Write message to error, warning, or verbose pipeline and specify $LevelText + switch ($Level) { + 'Error' { + Write-Error $Message + $LevelText = 'ERROR:' + } + 'Warn' { + Write-Warning $Message + $LevelText = 'WARNING:' + } + 'Info' { + Write-Verbose $Message + $LevelText = 'INFO:' + } + } + + # Write log entry to $Path + "$FormattedDate $LevelText $Message" | Out-File -FilePath $Path -Append + } + End + { + } +} # Allow SMTP with SSL and SMTP Auth # see: http://petermorrissey.blogspot.de/2013/01/sending-smtp-emails-with-powershell.html function Send-Email([String]$body) { - Write-Host "Sending email: $emailToAddress, $body" - try { $smtp = New-Object System.Net.Mail.SmtpClient($emailMailServer, $emailPort) @@ -86,7 +203,7 @@ function Send-Email([String]$body) { $smtp.Send($emailFromAddress, $emailToAddress, $emailSubject, $body) } catch { - Write-Host "Could not send email: $_.Exception.Message" + Write-Log "Could not send email: $_.Exception.Message" -Path $logFile -Level Error } } @@ -94,28 +211,23 @@ function Rotate-Backup { if($keepMonths -lt 0) { return } - - Write-Host "Rotating" $keepMonthsCount = $keepMonths Get-ChildItem $backupDir -Directory | Where-Object {($_.Name -ne $currMonth) -and ($_.Name -match "^\d{4,}-\d{2}$")} | Sort-Object -Descending | Foreach-Object { - Write-Host $_ "=>" $_.FullName - if($keepMonthsCount -ge 0) { $keepMonthsCount-- } if($keepMonthsCount -eq -1) { - Write-Host "Deleting $_" + Write-Log "Deleting backup $($_.FullName)" -Path $logFile -Level Info Remove-Item -Recurse -Force $_.FullName } } } -$startTime = Get-Date -format "yyyy-MM-dd HH:mm:ss" -Write-Host "Started at $startTime" +$dsAdditionalArgs = @("--UseVSS") $errorMessages = @() @@ -124,157 +236,167 @@ $success = $False $isDiff = $False $dsCommand = "" -if($smbDrive) { - try { - Write-Host "Connecting network drive" - - if($smbUser -and $smbPassword) { - Write-Host "With credentials" - - $secSmbPassword = $smbPassword | ConvertTo-SecureString -asPlainText -Force - $smbCredential = New-Object System.Management.Automation.PSCredential($smbUser, $secSmbPassword) +$currMonth = Get-Date -format "yyyy-MM" +$currDay = Get-Date -format "yyyy-MM-dd" - New-PSDrive -Name $smbDrive -PSProvider "FileSystem" -Root $smbPath -Credential $smbCredential -Persist -ErrorAction Stop | Out-Null - } - else { - Write-Host "Without credentials" - - New-PSDrive -Name $smbDrive -PSProvider "FileSystem" -Root $smbPath -Persist -ErrorAction Stop | Out-Null - } - - $smbConnected = $True + +if(!(Test-Path $logDir)) { + try { + New-Item -ItemType directory -Path $logDir -ErrorAction Stop | Out-Null } catch { - Write-Host "Could not connect to network drive $smbDrive`: $_.Exception.Message" - $errorMessages += "Could not connect to network drive $smbDrive`: $_.Exception.Message" + Write-Error "Could not create log directory $logDir`: $_.Exception.Message" + $errorMessages += "Could not create log directory $logDir`: $_.Exception.Message" } } -if(!(Test-Path $backupDir)) { - Write-Host "Directory $backupDir does not exist!" - $errorMessages += "Directory $backupDir does not exist!" -} +$logFile = "$logDir\$currMonth.log" if($errorMessages.Count -eq 0) { - $currMonth = Get-Date -format "yyyy-MM" - $currDay = Get-Date -format "yyyy-MM-dd" - - Write-Host $currMonth + $startTime = Get-Date -format "yyyy-MM-dd HH:mm:ss" + Write-Log "Started at $startTime" -Path $logFile - $backupTarget = $backupDir + "\" + $currMonth - $backupTargetFull = $backupTarget + "\" + "Full" + if($smbDrive) { + Write-Log "Connecting network drive $smbDrive to $smbPath" -Path $logFile - $backupTargetDiff = $backupTarget + "\" + "Diff-" + $currDay - - Write-Host $backupTarget + try { + if($smbUser -and $smbPassword) { + Write-Log "Connecting to network drive with credentials" -Path $logFile + + $secSmbPassword = $smbPassword | ConvertTo-SecureString -asPlainText -Force + $smbCredential = New-Object System.Management.Automation.PSCredential($smbUser, $secSmbPassword) - if((Test-Path $backupTarget) -and (Test-Path $backupTargetFull) -and (Test-Path "$backupTargetFull\*.hsh")) { - Write-Host "Differential backup" - - $isDiff = $True - - if(!(Test-Path $backupTargetDiff)) { - Write-Host "Creating directory $backupTargetDiff" - - try { - New-Item -ItemType directory -Path $backupTargetDiff -ErrorAction Stop | Out-Null - } - catch { - Write-Host "Could not create directory $backupTargetDiff`: $_.Exception.Message" - $errorMessages += "Could not create directory $backupTargetDiff`: $_.Exception.Message" + New-PSDrive -Name $smbDrive -PSProvider "FileSystem" -Root $smbPath -Credential $smbCredential -Persist -ErrorAction Stop | Out-Null } + else { + Write-Log "Connecting to network drive without credentials" -Path $logFile - if($errorMessages.Count -eq 0) { - $dsLogPath = if($dsLogFileToBackup) { "$backupTargetDiff\$dsLogFile" } else { $dsLogFile } - - $dsArgs = @($disksToBackup, "--logfile:$dsLogPath", "$backupTargetDiff\`$disk.sna", "-h$backupTargetFull\`$disk.hsh") + $dsAdditionalArgs - Write-Host $dsPath ($dsArgs -join " ") - - $dsCommand = "$dsPath $dsArgs" - - & $dsPath $dsArgs - - if($LastExitCode -ne 0) { - Write-Host "Drive Snapshot failed to backup! Exit code: $LastExitCode" - $errorMessages += "Drive Snapshot failed to backup! Exit code: $LastExitCode" - } - else { - $success = $True - } + New-PSDrive -Name $smbDrive -PSProvider "FileSystem" -Root $smbPath -Persist -ErrorAction Stop | Out-Null } + + $smbConnected = $True } - else { - Write-Host "Directory $backupTargetDiff already exists!" - $errorMessages += "Directory $backupTargetDiff already exists!" + catch { + Write-Log "Could not connect to network drive $smbDrive`: $_.Exception.Message" -Path $logFile -Level Error + $errorMessages += "Could not connect to network drive $smbDrive`: $_.Exception.Message" } } - else { - Write-Host "Full backup" - if(!(Test-Path $backupTarget)) { - Write-Host "Creating directory $backupTarget" + if(!(Test-Path $backupDir)) { + Write-Log "Directory $backupDir does not exist!" -Path $logFile -Level Error + $errorMessages += "Directory $backupDir does not exist!" + } + + if($errorMessages.Count -eq 0) { + $backupTarget = $backupDir + "\" + $currMonth + $backupTargetFull = $backupTarget + "\" + "Full" + + $backupTargetDiff = $backupTarget + "\" + "Diff-" + $currDay + + if((Test-Path $backupTarget) -and (Test-Path $backupTargetFull) -and (Test-Path "$backupTargetFull\*.hsh")) { + Write-Log "Doing a differential backup" -Path $logFile - try { - New-Item -ItemType directory -Path $backupTarget -ErrorAction Stop | Out-Null + $isDiff = $True + + if(!(Test-Path $backupTargetDiff)) { + try { + New-Item -ItemType directory -Path $backupTargetDiff -ErrorAction Stop | Out-Null + } + catch { + Write-Log "Could not create directory $backupTargetDiff`: $_.Exception.Message" -Path $logFile -Level Error + $errorMessages += "Could not create directory $backupTargetDiff`: $_.Exception.Message" + } + + if($errorMessages.Count -eq 0) { + $dsLogPath = if($dsLogFileToBackup) { "$backupTargetDiff\$dsLogFile" } else { $dsLogFile } + + $dsArgs = @($disksToBackup, "--logfile:$dsLogPath", "$backupTargetDiff\`$disk.sna", "-h$backupTargetFull\`$disk.hsh") + $dsAdditionalArgs + $dsCommand = "$dsPath $dsArgs" + + Write-Log $dsCommand -Path $logFile + + & $dsPath $dsArgs + + if($LastExitCode -ne 0) { + Write-Log "Drive Snapshot failed to backup! Exit code: $LastExitCode" -Path $logFile -Level Error + $errorMessages += "Drive Snapshot failed to backup! Exit code: $LastExitCode" + } + else { + Write-Log "Drive Snapshot succeeded!" -Path $logFile + $success = $True + } + } } - catch { - Write-Host "Could not create directory $backupTarget`: $_.Exception.Message" - $errorMessages += "Could not create directory $backupTarget`: $_.Exception.Message" + else { + Write-Log "Directory $backupTargetDiff already exists!" -Path $logFile -Level Error + $errorMessages += "Directory $backupTargetDiff already exists!" } } - - if($errorMessages.Count -eq 0) { - if(!(Test-Path $backupTargetFull)) { - Write-Host "Creating directory $backupTargetFull" - + else { + Write-Log "Doing a full backup" -Path $logFile + + if(!(Test-Path $backupTarget)) { try { - New-Item -ItemType directory -Path $backupTargetFull -ErrorAction Stop | Out-Null + New-Item -ItemType directory -Path $backupTarget -ErrorAction Stop | Out-Null } catch { - Write-Host "Could not create directory $backupTargetFull`: $_.Exception.Message" - $errorMessages += "Could not create directory $backupTargetFull`: $_.Exception.Message" + Write-Log "Could not create directory $backupTarget`: $_.Exception.Message" -Path $logFile -Level Error + $errorMessages += "Could not create directory $backupTarget`: $_.Exception.Message" } } if($errorMessages.Count -eq 0) { - if($rotateBeforeBackup) { - Rotate-Backup + if(!(Test-Path $backupTargetFull)) { + try { + New-Item -ItemType directory -Path $backupTargetFull -ErrorAction Stop | Out-Null + } + catch { + Write-Log "Could not create directory $backupTargetFull`: $_.Exception.Message" -Path $logFile -Level Error + $errorMessages += "Could not create directory $backupTargetFull`: $_.Exception.Message" + } } - $dsLogPath = if($dsLogFileToBackup) { "$backupTargetFull\$dsLogFile" } else { $dsLogFile } + if($errorMessages.Count -eq 0) { + if($rotateBeforeBackup) { + Rotate-Backup + } + + $dsLogPath = if($dsLogFileToBackup) { "$backupTargetFull\$dsLogFile" } else { $dsLogFile } - $dsArgs = @($disksToBackup, "--logfile:$dsLogPath", "$backupTargetFull\`$disk.sna") + $dsAdditionalArgs - Write-Host $dsPath ($dsArgs -join " ") - - $dsCommand = "$dsPath $dsArgs" - - & $dsPath $dsArgs - - if($LastExitCode -ne 0) { - Write-Host "Drive Snapshot failed to backup! Exit code: $LastExitCode" - $errorMessages += "Drive Snapshot failed to backup! Exit code: $LastExitCode" - } - else { - $success = $True - } - - if($rotateBeforeBackup -eq $False -and $success -eq $True) { - Rotate-Backup + $dsArgs = @($disksToBackup, "--logfile:$dsLogPath", "$backupTargetFull\`$disk.sna") + $dsAdditionalArgs + $dsCommand = "$dsPath $dsArgs" + + Write-Log $dsCommand -Path $logFile + + & $dsPath $dsArgs + + if($LastExitCode -ne 0) { + Write-Log "Drive Snapshot failed to backup! Exit code: $LastExitCode" -Path $logFile -Level Error + $errorMessages += "Drive Snapshot failed to backup! Exit code: $LastExitCode" + } + else { + Write-Log "Drive Snapshot succeeded!" -Path $logFile + $success = $True + } + + if($rotateBeforeBackup -eq $False -and $success -eq $True) { + Rotate-Backup + } } } } } -} -if($smbConnected) { - Write-Host "Disconnecting network drive" - - try { - Remove-PSDrive $smbDrive -ErrorAction Stop - } - catch { - Write-Host "Could not disconnect network drive $smbDrive`: $_.Exception.Message" - $errorMessages += "Could not disconnect network drive $smbDrive`: $_.Exception.Message" + if($smbConnected) { + Write-Log "Disconnecting network drive" -Path $logFile + + try { + Remove-PSDrive $smbDrive -ErrorAction Stop + } + catch { + Write-Log "Could not disconnect network drive $smbDrive`: $_.Exception.Message" -Path $logFile -Level Error + $errorMessages += "Could not disconnect network drive $smbDrive`: $_.Exception.Message" + } } } @@ -282,6 +404,8 @@ if($emailOnError -and $errorMessages.Count -gt 0) { $emailBody = "This is DSMonRot on $env:computername, started at $startTime.`n" $emailBody += "An error occured while performing a backup. Below are the error messages and some status information.`n`n" $emailBody += "Backup directory: $backupDir`n" + $emailBody += "Log directory: $logDir`n" + $emailBody += "Current log file: $logFile`n" $emailBody += "Differential backup: $isDiff`n" $emailBody += "Backup successful: $success`n" $emailBody += "Drive Snapshot command: $dsCommand`n`n" @@ -291,4 +415,4 @@ if($emailOnError -and $errorMessages.Count -gt 0) { } $endTime = Get-Date -format "yyyy-MM-dd HH:mm:ss" -Write-Host "Ended at $endTime" \ No newline at end of file +Write-Log "Ended at $endTime" -Path $logFile \ No newline at end of file