View backup error

app/classes/shared/server.py

@@ -1108,10 +1108,14 @@ class ServerInstance:
         self.run_threaded_server(user_id)
 
     def server_backup_threader(self, backup_id, update=False):
+        # Check to see if we're already backing up
+        if self.check_backup_by_id(backup_id):
+            return False
+
         backup_thread = threading.Thread(
             target=self.backup_server,
             daemon=True,
-            name=f"backup_{self.name}",
+            name=f"backup_{backup_id}",
             args=[backup_id, update],
         )
         logger.info(
@@ -1123,19 +1127,11 @@ class ServerInstance:
                 "Backup Thread - Local server path not defined. "
                 "Setting local server path variable."
             )
-        # checks if the backup thread is currently alive for this server
+
-        if not self.is_backingup:
+        try:
-            try:
+            backup_thread.start()
-                backup_thread.start()
+        except Exception as ex:
-                self.is_backingup = True
+            logger.error(f"Failed to start backup: {ex}")
-            except Exception as ex:
-                logger.error(f"Failed to start backup: {ex}")
-                return False
-        else:
-            logger.error(
-                f"Backup is already being processed for server "
-                f"{self.settings['server_name']}. Canceling backup request"
-            )
             return False
         logger.info(f"Backup Thread started for server {self.settings['server_name']}.")
 
@@ -1157,7 +1153,6 @@ class ServerInstance:
         backup_location = conf["backup_location"]
         if not backup_location:
             Console.critical("No backup path found. Canceling")
-            self.is_backingup = False
             return None
         if conf["before"]:
             if self.check_running():
@@ -1215,7 +1210,6 @@ class ServerInstance:
                 logger.info(f"Removing old backup '{oldfile['path']}'")
                 os.remove(Helpers.get_os_understandable_path(oldfile_path))
 
-            self.is_backingup = False
             logger.info(f"Backup of server: {self.name} completed")
             results = {
                 "percent": 100,
@@ -1247,7 +1241,6 @@ class ServerInstance:
                 )
                 self.run_threaded_server(HelperUsers.get_user_id_by_name("system"))
             time.sleep(3)
-            self.last_backup_failed = False
             if conf["after"]:
                 if self.check_running():
                     logger.debug(
@@ -1256,7 +1249,7 @@ class ServerInstance:
                     self.send_command(conf["after"])
             # pause to let people read message.
             time.sleep(5)
-        except:
+        except Exception as e:
             logger.exception(
                 f"Failed to create backup of server {self.name} (ID {self.server_id})"
             )
@@ -1273,17 +1266,28 @@ class ServerInstance:
                     "backup_status",
                     results,
                 )
-            self.is_backingup = False
             if was_server_running:
                 logger.info(
                     "Backup complete. User had shutdown preference. Starting server."
                 )
                 self.run_threaded_server(HelperUsers.get_user_id_by_name("system"))
-            self.last_backup_failed = True
+            HelpersManagement.update_backup_config(
+                backup_id, {"status": json.dumps({"status": "Failed", "message": f"{e}"})}
+            )
+        self.set_backup_status()
 
     def last_backup_status(self):
         return self.last_backup_failed
 
+    def set_backup_status(self):
+        backups = HelpersManagement.get_backups_by_server(self.server_id, True)
+        alert = False
+        for backup in backups:
+            print(backup.status)
+            if json.loads(backup.status)["status"] == "Failed":
+                alert = True
+        self.last_backup_failed = alert
+
     def list_backups(self, backup_location):
         if not backup_location:
             logger.info(
@@ -1426,12 +1430,17 @@ class ServerInstance:
         except FileNotFoundError:
             logger.error("Could not create backup of jarfile. File not found.")
 
+        backing_up = True
         # wait for backup
-        while self.is_backingup:
+        while backing_up:
-            time.sleep(10)
+            # Check to see if we're already backing up
+            backing_up = self.check_backup_by_id(backup_config["backup_id"])
 
         # check if backup was successful
-        if self.last_backup_failed:
+        backup_status = json.load(
+            HelpersManagement.get_backup_config(backup_config["backup_id"])["status"]
+        )["status"]
+        if backup_status == "Failed":
             for user in server_users:
                 WebSocketManager().broadcast_user(
                     user,
@@ -1638,6 +1647,14 @@ class ServerInstance:
                 except:
                     Console.critical("Can't broadcast server status to websocket")
 
+    def check_backup_by_id(_self, backup_id: str) -> bool:
+        # Check to see if we're already backing up
+        for thread in threading.enumerate():
+            if thread.getName() == f"backup_{backup_id}":
+                Console.debug(f"Backup with id {backup_id} already running!")
+                return True
+        return False
+
     def get_servers_stats(self):
         server_stats = {}
 

app/frontend/templates/panel/server_backup.html

@@ -95,8 +95,9 @@
                           </td>
                           <td>
                             <div id="{{backup.backup_id}}_status">
-                              <span class="badge-pill badge-outline-success backup-status"
+                              <button class="btn btn-outline-success backup-status" data-status="{{ backup.status }}"
-                                data-status="{{ backup.status }}"></span>
+                                data-Standby="{{ translate('serverBackups', 'standby', data['lang'])}}"
+                                data-Failed="{{ translate('serverBackups', 'failed', data['lang'])}}"></button>
                             </div>
                           </td>
                           <td id="{{backup.backup_location}}" class="type">
@@ -292,6 +293,11 @@
       bootbox.alert($(this).data("explain"));
     });
 
+    $(".backup-status").on("click", function () {
+      if ($(this).data('message') != "") {
+        bootbox.alert($(this).data('message'));
+      }
+    });
     $('.backup-status').each(function () {
       // Get the JSON string from the element's text
       var data = $(this).data('status');
@@ -304,12 +310,16 @@
         $(this).text(data.status);
 
         // Optionally, add classes based on status to style the element
+        $(this).attr('data-message', data["message"]);
         if (data.status === 'Active') {
-          $(this).addClass('badge-pill badge-outline-success');
+          $(this).removeClass();
+          $(this).addClass('badge-pill badge-outline-success btn');
         } else if (data.status === 'Failed') {
-          $(this).addClass('badge-pill badge-outline-danger');
+          $(this).removeClass();
+          $(this).addClass('badge-pill badge-outline-danger btn');
         } else if (data.status === 'Standby') {
-          $(this).addClass('badge-pill badge-outline-secondary');
+          $(this).removeClass();
+          $(this).addClass('badge-pill badge-outline-secondary btn');
         }
       } catch (e) {
         console.error('Invalid JSON string:', jsonString);

app/translations/en_EN.json

@@ -338,7 +338,9 @@
         "backups": "Server Backups",
         "default": "Default Backup",
         "defaultExplain": "The backup that Crafty will use before updates. This cannot be changed or deleted.",
-        "status": "Status"
+        "status": "Status",
+        "standby": "Standby",
+        "failed": "Failed"
     },
     "serverConfig": {
         "bePatientDelete": "Please be patient while we remove your server from the Crafty panel. This screen will close in a few moments.",