Merge pull request #809 from aaannz/status_check_requires_database_main

Correctly check backup status after database is started (bsc#1268229)

Author: mcalmer

mgradm/cmd/backup/db/disable.go

@@ -59,6 +59,9 @@ func Disable(flags *Flagpole) error {
 		if err := systemd.StartService(podman.DBService); err != nil {
 			return err
 		}
+		if err := cnx.WaitForHealthcheck(); err != nil {
+			return err
+		}
 	}
 	return Status()
 }

mgradm/cmd/backup/db/enable.go

@@ -58,11 +58,6 @@ func Enable(force bool) error {
 		return err
 	}
 
-	// Check if all is well configured
-	if err := CheckStatus(); err != nil {
-		return err
-	}
-
 	// Prepare initial snapshot. For this container either must be running or we need to start container database
 	log.Info().Msg(L("Starting database…"))
 	if err := systemd.StartService(podman.DBService); err != nil {
@@ -72,6 +67,11 @@ func Enable(force bool) error {
 		return err
 	}
 
+	// Check if all is well configured
+	if err := CheckStatus(); err != nil {
+		return err
+	}
+
 	if err := RunBaseBackup(cnx); err != nil {
 		return err
 	}

uyuni-tools.changes.oholecek.status_check_requires_database

@@ -0,0 +1 @@
+- Check backup status only after database is started (bsc#1262492)