Removed console output

[psmysqlbackup.git] / psmysqlbackup.ps1

diff --git a/psmysqlbackup.ps1 b/psmysqlbackup.ps1
index cdc6afa26f7bae0d0685cd8ae074069bbf6c6903..3592213577ac231771a11e1092448e702366bb19 100644 (file)
--- a/psmysqlbackup.ps1
+++ b/psmysqlbackup.ps1
@@ -25,9 +25,12 @@ $configMysqldumpCli = "C:\Program Files\MariaDB 10.5\bin\mysqldump.exe"
 # Directory where to store the backups
 $configBackupDir = "backup"
 # Number of backups to keep, set to 0 to keep all backups
 # Directory where to store the backups
 $configBackupDir = "backup"
 # Number of backups to keep, set to 0 to keep all backups

-$configRotate = 7
+$configBackupRotate = 7

 
 

+# Directory where to store the logfiles

 $configLogDir = "log"
 $configLogDir = "log"

+# Number of logfiles to keep, set to 0 to keep all logfiles
+# You should set this to at least the same as $configBackupRotate

 $configLogRotate = 7
 
 # Databases to backup, leave empty to backup all databases
 $configLogRotate = 7
 
 # Databases to backup, leave empty to backup all databases


@@ -174,23 +177,32 @@ function Create-Backup([String]$database, [String]$target) {
         throw "mysqldump exited with Exit code $LastExitCode"
     }
 }
         throw "mysqldump exited with Exit code $LastExitCode"
     }
 }

-
-function Rotate-Backups($backupDir) {
-    if($configRotate -le 0) {
+function Invoke-FileRotation {
+    Param (
+        $Dir,
+        $MaxFiles,
+        [Parameter(Mandatory=$false)]
+        $Pattern,
+        [Parameter(Mandatory=$false)]
+        $LogFile
+    )
+
+    if($MaxFiles -le 0) {

         return
     }
         return
     }

-    
-    $keepBackupsCount = $configRotate

 
 

-    Get-ChildItem $backupDir -File | Where-Object {($_.Name -match "^backup-.+-\d{8,}-\d{6}\.sql$")} | Sort-Object -Descending |
+    $keepFilesCount = $MaxFiles
+
+    Get-ChildItem $Dir -File | Where-Object {($null -eq $Pattern -or $_.Name -match $Pattern)} | Sort-Object -Descending |

     Foreach-Object {
     Foreach-Object {

-        if($keepBackupsCount -ge 0) {
-            $keepBackupsCount--
+        if($keepFilesCount -ge 0) {
+            $keepFilesCount--

         }
 
         }
 

-        if($keepBackupsCount -eq -1) {
-            Write-Output "Deleting backup $($_.FullName)"
-            #Write-Log "Deleting backup $($_.FullName)"
+        if($keepFilesCount -eq -1) {
+            if($null -ne $LogFile) {
+                Write-Log "Deleting file $($_.FullName)" -Path $LogFile
+            }

 
             Remove-Item -Force $_.FullName
         }
 
             Remove-Item -Force $_.FullName
         }


@@ -199,6 +211,9 @@ function Rotate-Backups($backupDir) {
 
 $defaultDbExclude = @("information_schema", "performance_schema")
 
 
 $defaultDbExclude = @("information_schema", "performance_schema")
 

+$patternBackupFile = "^backup-.+-\d{8,}-\d{6}\.sql$"
+$patternLogFile = "^log-\d{8,}-\d{6}\.log$"
+

 $currDaytime = Get-Date -format "yyyyMMdd-HHmmss"
 
 $logFile = "$configLogDir\log-$currDaytime.log"
 $currDaytime = Get-Date -format "yyyyMMdd-HHmmss"
 
 $logFile = "$configLogDir\log-$currDaytime.log"


@@ -215,9 +230,6 @@ catch {
     Write-Log $_ -Path $logFile -Level Error
     Write-Log "Exiting" -Path $logFile -Level Error
 
     Write-Log $_ -Path $logFile -Level Error
     Write-Log "Exiting" -Path $logFile -Level Error
 

-    Write-Output "Failed to get list of databases"
-    Write-Output $_
-

     exit 1
 }
 
     exit 1
 }
 


@@ -232,7 +244,6 @@ if($configDbBackup -and $configDbBackup.count -gt 0) {
         }
         else {
             Write-Log "Not backing up database $cDb, because it does not exist" -Path $logFile -Level Warn
         }
         else {
             Write-Log "Not backing up database $cDb, because it does not exist" -Path $logFile -Level Warn

-            Write-Warning "Not backing up database $cDb, because it does not exist"

         }
     }
 }
         }
     }
 }


@@ -266,9 +277,6 @@ foreach($d in $databasesToBackup) {
             Write-Log $_ -Path $logFile -Level Error
             Write-Log "Exiting" -Path $logFile -Level Error
 
             Write-Log $_ -Path $logFile -Level Error
             Write-Log "Exiting" -Path $logFile -Level Error
 

-            Write-Output "Failed to create directory $databaseBackupDir"
-            Write-Output $_
-

             exit 1
         }
     }
             exit 1
         }
     }


@@ -276,20 +284,18 @@ foreach($d in $databasesToBackup) {
     $databaseBackupFile = Join-Path -Path $databaseBackupDir -ChildPath "backup-$d-$currDaytime.sql"
 
     Write-Log "Backing up $d to $databaseBackupFile..." -Path $logFile
     $databaseBackupFile = Join-Path -Path $databaseBackupDir -ChildPath "backup-$d-$currDaytime.sql"
 
     Write-Log "Backing up $d to $databaseBackupFile..." -Path $logFile

-    Write-Output "Backing up $d to $databaseBackupFile..."

     
     try {
         Create-Backup $d $databaseBackupFile
     
     try {
         Create-Backup $d $databaseBackupFile

-        Rotate-Backups $databaseBackupDir
+        Invoke-FileRotation -Dir $databaseBackupDir -MaxFiles $configBackupRotate -Pattern $patternBackupFile -LogFile $logFile

     }
     catch {
         Write-Log "Could not backup database $d to $databaseBackupFile" -Path $logFile -Level Error
         Write-Log $_ -Path $logFile -Level Error
     }
     catch {
         Write-Log "Could not backup database $d to $databaseBackupFile" -Path $logFile -Level Error
         Write-Log $_ -Path $logFile -Level Error

-
-        Write-Output "Could not backup database $d to $databaseBackupFile"
-        Write-Output $_

     }
 }
 
     }
 }
 

+Invoke-FileRotation -Dir $configLogDir -MaxFiles $configLogRotate -Pattern $patternLogFile -LogFile $logFile
+

 $endTime = Get-Date -format "yyyy-MM-dd HH:mm:ss"
 Write-Log "Ended at $endTime" -Path $logFile
\ No newline at end of file
 $endTime = Get-Date -format "yyyy-MM-dd HH:mm:ss"
 Write-Log "Ended at $endTime" -Path $logFile
\ No newline at end of file