Updated TODO

[psmysqlbackup.git] / psmysqlbackup.ps1

diff --git a/psmysqlbackup.ps1 b/psmysqlbackup.ps1
index bbfc0755de09620824ce54917429fb7faaf6fcb9..6177abdf5b6d30f784c8bfbce38df3a1ecba9df5 100644 (file)
--- a/psmysqlbackup.ps1
+++ b/psmysqlbackup.ps1
@@ -9,33 +9,37 @@
 # Config
 
 # MySQL host
-$configMysqlHost = "localhost"
+$configMysqlHost = 'localhost'
 # Port of MySQL host
 $configMysqlPort = 3306
 # MySQL user using to connect to MySQL
-$configMysqlUser = "backup"
+$configMysqlUser = 'backup'
 # Password for MySQL user
-$configMysqlPassword = "backup"
+$configMysqlPassword = 'backup'
 
 # Path to MySQL CLI program
-$configMysqlCli = "C:\Program Files\MariaDB 10.5\bin\mysql.exe"
+$configMysqlCli = 'mysql.exe'
 # Path to mysqldump CLI program
-$configMysqldumpCli = "C:\Program Files\MariaDB 10.5\bin\mysqldump.exe"
+$configMysqldumpCli = 'mysqldump.exe'
 
 # Directory where to store the backups
-$configBackupDir = "backup"
+$configBackupDir = 'backup'
 # Number of backups to keep, set to 0 to keep all backups
-$configRotate = 7
+$configBackupRotate = 7
+
+# Compress backups (limited to 2 GB due to usage of Compress-Archive)
+$configBackupCompress = $False
 
 # 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
 $configDbBackup = @()
 # If $configDbBackup is empty, don't backup the databases defined here
-$configDbExclude = @("test")
+$configDbExclude = @('test')
 # If $configDbBackup is empty, don't backup the databases matching these patterns
 $configDbExcludePattern = @()
 
@@ -175,6 +179,11 @@ function Create-Backup([String]$database, [String]$target) {
     if($LastExitCode -ne 0) {
         throw "mysqldump exited with Exit code $LastExitCode"
     }
+
+    if($configBackupCompress) {
+        Compress-Archive -Path $target -DestinationPath "$target.zip"
+        Remove-Item -Path $target
+    }
 }
 function Invoke-FileRotation {
     Param (
@@ -199,8 +208,6 @@ function Invoke-FileRotation {
         }
 
         if($keepFilesCount -eq -1) {
-            Write-Output "Deleting file $($_.FullName)"
-            
             if($null -ne $LogFile) {
                 Write-Log "Deleting file $($_.FullName)" -Path $LogFile
             }
@@ -210,13 +217,16 @@ function Invoke-FileRotation {
     }
 }
 
-$defaultDbExclude = @("information_schema", "performance_schema")
+$defaultDbExclude = @('information_schema', 'performance_schema')
 
-$currDaytime = Get-Date -format "yyyyMMdd-HHmmss"
+$patternBackupFile = '^backup-.+-\d{8,}-\d{6}\.sql(\.zip)?$'
+$patternLogFile = '^log-\d{8,}-\d{6}\.log$'
+
+$currDaytime = Get-Date -format 'yyyyMMdd-HHmmss'
 
 $logFile = "$configLogDir\log-$currDaytime.log"
 
-$startTime = Get-Date -format "yyyy-MM-dd HH:mm:ss"
+$startTime = Get-Date -format 'yyyy-MM-dd HH:mm:ss'
 Write-Log "Started at $startTime" -Path $logFile
 
 # Get a list of all databases
@@ -224,12 +234,9 @@ try {
     $databases = Get-Databases | Where-Object {!($_ -in $defaultDbExclude)}
 }
 catch {
-    Write-Log "Failed to get list of databases" -Path $logFile -Level Error
+    Write-Log 'Failed to get list of databases' -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 $_
+    Write-Log 'Exiting' -Path $logFile -Level Error
 
     exit 1
 }
@@ -245,7 +252,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
-            Write-Warning "Not backing up database $cDb, because it does not exist"
         }
     }
 }
@@ -277,10 +283,7 @@ foreach($d in $databasesToBackup) {
         catch {
             Write-Log "Failed to create directory $databaseBackupDir" -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 $_
+            Write-Log 'Exiting' -Path $logFile -Level Error
 
             exit 1
         }
@@ -288,21 +291,24 @@ foreach($d in $databasesToBackup) {
 
     $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..."
+    if($configBackupCompress) {
+        Write-Log "Backing up $d to compressed file $databaseBackupFile.zip..." -Path $logFile
+    }
+    else {
+        Write-Log "Backing up $d to $databaseBackupFile..." -Path $logFile
+    }
     
     try {
         Create-Backup $d $databaseBackupFile
-        Invoke-FileRotation -Dir $databaseBackupDir -MaxFiles $configRotate -Pattern "^backup-.+-\d{8,}-\d{6}\.sql$" -LogFile $logFile
+        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
-
-        Write-Output "Could not backup database $d to $databaseBackupFile"
-        Write-Output $_
     }
 }
 
-$endTime = Get-Date -format "yyyy-MM-dd HH:mm:ss"
+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