]> git.sur5r.net Git - bacula/bacula/blobdiff - bacula/src/dird/ua_purge.c
Fix bug #2343 where truncate of explicit Volume name truncates non-purged volumes
[bacula/bacula] / bacula / src / dird / ua_purge.c
index f31b577af8c93423921baee57d1f6314a0bd1fb8..6d85b91950ef93510ddf0935bbd9c9ce00a45652 100644 (file)
@@ -1,29 +1,20 @@
 /*
-   Bacula® - The Network Backup Solution
-
-   Copyright (C) 2002-2008 Free Software Foundation Europe e.V.
-
-   The main author of Bacula is Kern Sibbald, with contributions from
-   many others, a complete list can be found in the file AUTHORS.
-   This program is Free Software; you can redistribute it and/or
-   modify it under the terms of version two of the GNU General Public
-   License as published by the Free Software Foundation and included
-   in the file LICENSE.
-
-   This program is distributed in the hope that it will be useful, but
-   WITHOUT ANY WARRANTY; without even the implied warranty of
-   MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU
-   General Public License for more details.
-
-   You should have received a copy of the GNU General Public License
-   along with this program; if not, write to the Free Software
-   Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA
-   02110-1301, USA.
-
-   Bacula® is a registered trademark of Kern Sibbald.
-   The licensor of Bacula is the Free Software Foundation Europe
-   (FSFE), Fiduciary Program, Sumatrastrasse 25, 8006 Zürich,
-   Switzerland, email:ftf@fsfeurope.org.
+   Bacula(R) - The Network Backup Solution
+
+   Copyright (C) 2000-2017 Kern Sibbald
+
+   The original author of Bacula is Kern Sibbald, with contributions
+   from many others, a complete list can be found in the file AUTHORS.
+
+   You may use this file and others of this release according to the
+   license defined in the LICENSE file, which includes the Affero General
+   Public License, v3.0 ("AGPLv3") and some additional permissions and
+   terms pursuant to its AGPLv3 Section 7.
+
+   This notice must be preserved when any source code is
+   conveyed and/or propagated.
+
+   Bacula(R) is a registered trademark of Kern Sibbald.
 */
 /*
  *
@@ -35,7 +26,6 @@
  *
  *     Kern Sibbald, February MMII
  *
- *   Version $Id$
  */
 
 #include "bacula.h"
@@ -44,6 +34,7 @@
 /* Forward referenced functions */
 static int purge_files_from_client(UAContext *ua, CLIENT *client);
 static int purge_jobs_from_client(UAContext *ua, CLIENT *client);
+int truncate_cmd(UAContext *ua, const char *cmd);
 
 static const char *select_jobsfiles_from_client =
    "SELECT JobId FROM Job "
@@ -59,15 +50,18 @@ static const char *select_jobs_from_client =
  *
  *     Purge Files (from) [Job|JobId|Client|Volume]
  *     Purge Jobs  (from) [Client|Volume]
+ *     Purge Volumes
  *
  *  N.B. Not all above is implemented yet.
  */
-int purgecmd(UAContext *ua, const char *cmd)
+int purge_cmd(UAContext *ua, const char *cmd)
 {
    int i;
    CLIENT *client;
    MEDIA_DBR mr;
    JOB_DBR  jr;
+   memset(&jr, 0, sizeof(jr));
+
    static const char *keywords[] = {
       NT_("files"),
       NT_("jobs"),
@@ -86,15 +80,21 @@ int purgecmd(UAContext *ua, const char *cmd)
       NT_("Volume"),
       NULL};
 
-   ua->warning_msg(_(
-      "\nThis command is can be DANGEROUS!!!\n\n"
-      "It purges (deletes) all Files from a Job,\n"
-      "JobId, Client or Volume; or it purges (deletes)\n"
-      "all Jobs from a Client or Volume without regard\n"
-      "for retention periods. Normally you should use the\n"
-      "PRUNE command, which respects retention periods.\n"));
+   /* Special case for the "Action On Purge", this option is working only on
+    * Purged volume, so no jobs or files will be purged.
+    * We are skipping this message if "purge volume action=xxx"
+    */
+   if (!(find_arg(ua, "volume") >= 0 && find_arg(ua, "action") >= 0)) {
+      ua->warning_msg(_(
+        "\nThis command can be DANGEROUS!!!\n\n"
+        "It purges (deletes) all Files from a Job,\n"
+        "JobId, Client or Volume; or it purges (deletes)\n"
+        "all Jobs from a Client or Volume without regard\n"
+        "to retention periods. Normally you should use the\n"
+        "PRUNE command, which respects retention periods.\n"));
+   }
 
-   if (!open_db(ua)) {
+   if (!open_new_client_db(ua)) {
       return 1;
    }
    switch (find_arg_keyword(ua, keywords)) {
@@ -110,7 +110,8 @@ int purgecmd(UAContext *ua, const char *cmd)
          }
          return 1;
       case 2:                         /* client */
-         client = get_client_resource(ua);
+         /* We restrict the client list to ClientAcl, maybe something to change later */
+         client = get_client_resource(ua, JT_SYSTEM);
          if (client) {
             purge_files_from_client(ua, client);
          }
@@ -125,7 +126,8 @@ int purgecmd(UAContext *ua, const char *cmd)
    case 1:
       switch(find_arg_keyword(ua, jobs_keywords)) {
       case 0:                         /* client */
-         client = get_client_resource(ua);
+         /* We restrict the client list to ClientAcl, maybe something to change later */
+         client = get_client_resource(ua, JT_SYSTEM);
          if (client) {
             purge_jobs_from_client(ua, client);
          }
@@ -138,6 +140,11 @@ int purgecmd(UAContext *ua, const char *cmd)
       }
    /* Volume */
    case 2:
+      /* Perform ActionOnPurge (action=truncate) */
+      if (find_arg(ua, "action") >= 0) {
+         return truncate_cmd(ua, ua->cmd);
+      }
+
       while ((i=find_arg(ua, NT_("volume"))) >= 0) {
          if (select_media_dbr(ua, &mr)) {
             purge_jobs_from_volume(ua, &mr, /*force*/true);
@@ -151,13 +158,15 @@ int purgecmd(UAContext *ua, const char *cmd)
    }
    switch (do_keyword_prompt(ua, _("Choose item to purge"), keywords)) {
    case 0:                            /* files */
-      client = get_client_resource(ua);
+      /* We restrict the client list to ClientAcl, maybe something to change later */
+      client = get_client_resource(ua, JT_SYSTEM);
       if (client) {
          purge_files_from_client(ua, client);
       }
       break;
    case 1:                            /* jobs */
-      client = get_client_resource(ua);
+      /* We restrict the client list to ClientAcl, maybe something to change later */
+      client = get_client_resource(ua, JT_SYSTEM);
       if (client) {
          purge_jobs_from_client(ua, client);
       }
@@ -204,11 +213,11 @@ static int purge_files_from_client(UAContext *ua, CLIENT *client)
 
    purge_files_from_job_list(ua, del);
 
-   if (del.num_ids == 0) {
+   if (del.num_del == 0) {
       ua->warning_msg(_("No Files found for client %s to purge from %s catalog.\n"),
          client->name(), client->catalog->name());
    } else {
-      ua->info_msg(_("Files for %d Jobs for client \"%s\" purged from %s catalog.\n"), del.num_ids,
+      ua->info_msg(_("Files for %d Jobs for client \"%s\" purged from %s catalog.\n"), del.num_del,
          client->name(), client->catalog->name());
    }
 
@@ -245,7 +254,7 @@ static int purge_jobs_from_client(UAContext *ua, CLIENT *client)
    del.max_ids = 1000;
    del.JobId = (JobId_t *)malloc(sizeof(JobId_t) * del.max_ids);
    del.PurgedFiles = (char *)malloc(del.max_ids);
-   
+
    ua->info_msg(_("Begin purging jobs from Client \"%s\"\n"), cr.Name);
 
    Mmsg(query, select_jobs_from_client, edit_int64(cr.ClientId, ed1));
@@ -254,11 +263,11 @@ static int purge_jobs_from_client(UAContext *ua, CLIENT *client)
 
    purge_job_list_from_catalog(ua, del);
 
-   if (del.num_ids == 0) {
-      ua->warning_msg(_("No Files found for client %s to purge from %s catalog.\n"),
+   if (del.num_del == 0) {
+      ua->warning_msg(_("No Jobs found for client %s to purge from %s catalog.\n"),
          client->name(), client->catalog->name());
    } else {
-      ua->info_msg(_("%d Jobs for client %s purged from %s catalog.\n"), del.num_ids,
+      ua->info_msg(_("%d Jobs for client %s purged from %s catalog.\n"), del.num_del,
          client->name(), client->catalog->name());
    }
 
@@ -287,6 +296,10 @@ void purge_files_from_jobs(UAContext *ua, char *jobs)
    db_sql_query(ua->db, query.c_str(), NULL, (void *)NULL);
    Dmsg1(050, "Delete BaseFiles sql=%s\n", query.c_str());
 
+   Mmsg(query, "DELETE FROM PathVisibility WHERE JobId IN (%s)", jobs);
+   db_sql_query(ua->db, query.c_str(), NULL, (void *)NULL);
+   Dmsg1(050, "Delete PathVisibility sql=%s\n", query.c_str());
+
    /*
     * Now mark Job as having files purged. This is necessary to
     * avoid having too many Jobs to process in future prunings. If
@@ -380,28 +393,16 @@ void purge_files_from_job_list(UAContext *ua, del_ctx &del)
 void upgrade_copies(UAContext *ua, char *jobs)
 {
    POOL_MEM query(PM_MESSAGE);
-   
+   int dbtype = ua->db->bdb_get_type_index();
+
    db_lock(ua->db);
-   /* Do it in two times for mysql */
-   Mmsg(query, "CREATE TEMPORARY TABLE cpy_tmp AS "
-                  "SELECT MIN(JobId) AS JobId FROM Job "     /* Choose the oldest job */
-                   "WHERE Type='%c' "
-                     "AND ( PriorJobId IN (%s) "
-                         "OR "
-                          " PriorJobId IN ( "
-                             "SELECT PriorJobId "
-                               "FROM Job "
-                              "WHERE JobId IN (%s) "
-                               " AND Type='B' "
-                            ") "
-                         ") "
-                   "GROUP BY PriorJobId ",           /* one result per copy */
-        JT_JOB_COPY, jobs, jobs);
+
+   Mmsg(query, uap_upgrade_copies_oldest_job[dbtype], JT_JOB_COPY, jobs, jobs);
    db_sql_query(ua->db, query.c_str(), NULL, (void *)NULL);
    Dmsg1(050, "Upgrade copies Log sql=%s\n", query.c_str());
 
    /* Now upgrade first copy to Backup */
-   Mmsg(query, "UPDATE Job SET Type='B' "           /* JT_JOB_COPY => JT_BACKUP  */
+   Mmsg(query, "UPDATE Job SET Type='B' "      /* JT_JOB_COPY => JT_BACKUP  */
                 "WHERE JobId IN ( SELECT JobId FROM cpy_tmp )");
 
    db_sql_query(ua->db, query.c_str(), NULL, (void *)NULL);
@@ -430,6 +431,16 @@ void purge_jobs_from_catalog(UAContext *ua, char *jobs)
    db_sql_query(ua->db, query.c_str(), NULL, (void *)NULL);
    Dmsg1(050, "Delete Log sql=%s\n", query.c_str());
 
+   Mmsg(query, "DELETE FROM RestoreObject WHERE JobId IN (%s)", jobs);
+   db_sql_query(ua->db, query.c_str(), NULL, (void *)NULL);
+   Dmsg1(050, "Delete RestoreObject sql=%s\n", query.c_str());
+
+   /* The JobId of the Snapshot record is no longer usable
+    * TODO: Migth want to use a copy for the jobid?
+    */
+   Mmsg(query, "UPDATE Snapshot SET JobId=0 WHERE JobId IN (%s)", jobs);
+   db_sql_query(ua->db, query.c_str(), NULL, (void *)NULL);
+
    upgrade_copies(ua, jobs);
 
    /* Now remove the Job record itself */
@@ -449,12 +460,11 @@ void purge_files_from_volume(UAContext *ua, MEDIA_DBR *mr )
 bool purge_jobs_from_volume(UAContext *ua, MEDIA_DBR *mr, bool force)
 {
    POOL_MEM query(PM_MESSAGE);
-   struct del_ctx del;
+   db_list_ctx lst_all, lst;
+   char *jobids=NULL;
    int i;
    bool purged = false;
    bool stat;
-   JOB_DBR jr;
-   char ed1[50];
 
    stat = strcmp(mr->VolStatus, "Append") == 0 ||
           strcmp(mr->VolStatus, "Full")   == 0 ||
@@ -467,42 +477,43 @@ bool purge_jobs_from_volume(UAContext *ua, MEDIA_DBR *mr, bool force)
       return 0;
    }
 
-   memset(&jr, 0, sizeof(jr));
-   memset(&del, 0, sizeof(del));
-   del.max_ids = 1000;
-   del.JobId = (JobId_t *)malloc(sizeof(JobId_t) * del.max_ids);
-
    /*
     * Check if he wants to purge a single jobid
     */
    i = find_arg_with_value(ua, "jobid");
-   if (i >= 0) {
-      del.num_ids = 1;
-      del.JobId[0] = str_to_int64(ua->argv[i]);
+   if (i >= 0 && is_a_number_list(ua->argv[i])) {
+      jobids = ua->argv[i];
+
    } else {
+      POOL_MEM query;
       /*
        * Purge ALL JobIds
        */
-      Mmsg(query, "SELECT DISTINCT JobId FROM JobMedia WHERE MediaId=%s", 
-           edit_int64(mr->MediaId, ed1));
-      if (!db_sql_query(ua->db, query.c_str(), file_delete_handler, (void *)&del)) {
+      if (!db_get_volume_jobids(ua->jcr, ua->db, mr, &lst_all)) {
          ua->error_msg("%s", db_strerror(ua->db));
          Dmsg0(050, "Count failed\n");
          goto bail_out;
       }
-   }
 
-   purge_job_list_from_catalog(ua, del);
-
-   ua->info_msg(_("%d File%s on Volume \"%s\" purged from catalog.\n"), del.num_del,
-      del.num_del==1?"":"s", mr->VolumeName);
+      if (lst_all.count > 0) {
+         Mmsg(query, "SELECT JobId FROM Job WHERE JobId IN (%s) AND JobStatus NOT IN ('R', 'C')",
+              lst_all.list);
+         if (!db_sql_query(ua->db, query.c_str(), db_list_handler, &lst)) {
+            ua->error_msg("%s", db_strerror(ua->db));
+            goto bail_out;
+         }
+      }
+      jobids = lst.list;
+   }
 
-   purged = is_volume_purged(ua, mr, force); 
+   if (*jobids) {
+      purge_jobs_from_catalog(ua, jobids);
+      ua->info_msg(_("%d Job%s on Volume \"%s\" purged from catalog.\n"),
+                   lst.count, lst.count<=1?"":"s", mr->VolumeName);
+   }
+   purged = is_volume_purged(ua, mr, force);
 
 bail_out:
-   if (del.JobId) {
-      free(del.JobId);
-   }
    return purged;
 }
 
@@ -531,13 +542,14 @@ bool is_volume_purged(UAContext *ua, MEDIA_DBR *mr, bool force)
    }
 
    if (strcmp(mr->VolStatus, "Purged") == 0) {
+      Dmsg1(100, "Volume=%s already purged.\n", mr->VolumeName);
       purged = true;
       goto bail_out;
    }
 
    /* If purged, mark it so */
    cnt.count = 0;
-   Mmsg(query, "SELECT count(*) FROM JobMedia WHERE MediaId=%s", 
+   Mmsg(query, "SELECT 1 FROM JobMedia WHERE MediaId=%s LIMIT 1",
         edit_int64(mr->MediaId, ed1));
    if (!db_sql_query(ua->db, query.c_str(), del_count_handler, (void *)&cnt)) {
       ua->error_msg("%s", db_strerror(ua->db));
@@ -548,6 +560,8 @@ bool is_volume_purged(UAContext *ua, MEDIA_DBR *mr, bool force)
    if (cnt.count == 0) {
       ua->warning_msg(_("There are no more Jobs associated with Volume \"%s\". Marking it purged.\n"),
          mr->VolumeName);
+      Dmsg1(100, "There are no more Jobs associated with Volume \"%s\". Marking it purged.\n",
+         mr->VolumeName);
       if (!(purged = mark_media_purged(ua, mr))) {
          ua->error_msg("%s", db_strerror(ua->db));
       }
@@ -556,6 +570,178 @@ bail_out:
    return purged;
 }
 
+/*
+ * Called here to send the appropriate commands to the SD
+ *  to do truncate on purge.
+ */
+static void truncate_volume(UAContext *ua, MEDIA_DBR *mr,
+                            char *pool, char *storage,
+                            int drive, BSOCK *sd)
+{
+   bool ok = false;
+   uint64_t VolBytes = 0;
+   uint64_t VolABytes = 0;
+   uint32_t VolType = 0;
+
+   if (!mr->Recycle) {
+      return;
+   }
+
+   /* Do it only if action on purge = truncate is set */
+   if (!(mr->ActionOnPurge & ON_PURGE_TRUNCATE)) {
+      ua->error_msg(_("\nThe option \"Action On Purge = Truncate\" was not defined in the Pool resource.\n"
+                      "Unable to truncate volume \"%s\"\n"), mr->VolumeName);
+      return;
+   }
+
+   /*
+    * Send the command to truncate the volume after purge. If this feature
+    * is disabled for the specific device, this will be a no-op.
+    */
+
+   /* Protect us from spaces */
+   bash_spaces(mr->VolumeName);
+   bash_spaces(mr->MediaType);
+   bash_spaces(pool);
+   bash_spaces(storage);
+
+   /* Do it by relabeling the Volume, which truncates it */
+   sd->fsend("relabel %s OldName=%s NewName=%s PoolName=%s "
+             "MediaType=%s Slot=%d drive=%d\n",
+             storage,
+             mr->VolumeName, mr->VolumeName,
+             pool, mr->MediaType, mr->Slot, drive);
+
+   unbash_spaces(mr->VolumeName);
+   unbash_spaces(mr->MediaType);
+   unbash_spaces(pool);
+   unbash_spaces(storage);
+
+   /* Check for valid response. With cloud volumes, the upload of the part.1 can
+    * generate a dir_update_volume_info() message that is handled by bget_dirmsg()
+    */
+   while (bget_dirmsg(sd) >= 0) {
+      ua->send_msg("%s", sd->msg);
+      if (sscanf(sd->msg, "3000 OK label. VolBytes=%llu VolABytes=%lld VolType=%d ",
+                 &VolBytes, &VolABytes, &VolType) == 3) {
+
+         ok=true;
+         mr->VolBytes = VolBytes;
+         mr->VolABytes = VolABytes;
+         mr->VolType = VolType;
+         mr->VolFiles = 0;
+         mr->VolParts = 1;
+         mr->VolCloudParts = 0;
+         mr->LastPartBytes = VolBytes;
+
+         set_storageid_in_mr(NULL, mr);
+         if (!db_update_media_record(ua->jcr, ua->db, mr)) {
+            ua->error_msg(_("Can't update volume size in the catalog\n"));
+         }
+         ua->send_msg(_("The volume \"%s\" has been truncated\n"), mr->VolumeName);
+      }
+   }
+   if (!ok) {
+      ua->warning_msg(_("Unable to truncate volume \"%s\"\n"), mr->VolumeName);
+   }
+}
+
+/*
+ * Implement Bacula bconsole command  purge action
+ *     purge action=truncate pool= volume= storage= mediatype=
+ * or
+ *     truncate [cache] pool= volume= storage= mediatype=
+ *
+ * If the keyword "cache:  is present, then we use the truncate
+ *   command rather than relabel so that the driver can decide
+ *   whether or not it wants to truncate.  Note: only the
+ *   Cloud driver permits truncating the cache.
+ *
+ * Note, later we might want to rename this action_on_purge_cmd() as
+ *  was the original, but only if we add additional actions such as
+ *  erase, ... For the moment, we only do a truncate.
+ *
+ */
+int truncate_cmd(UAContext *ua, const char *cmd)
+{
+   int drive = -1;
+   int nb = 0;
+   uint32_t *results = NULL;
+   const char *action = "truncate";
+   MEDIA_DBR mr;
+   POOL_DBR pr;
+   BSOCK *sd;
+   char storage[MAX_NAME_LENGTH];
+
+   if (find_arg(ua, "cache") > 0) {
+      return cloud_volumes_cmd(ua, cmd, "truncate cache");
+   }
+
+   memset(&pr, 0, sizeof(pr));
+
+   /*
+    * Look for all Purged volumes that can be recycled, are enabled and
+    *  have more than 1,000 bytes (i.e. actually have data).
+    */
+   mr.Recycle = 1;
+   mr.Enabled = 1;
+   mr.VolBytes = 1000;
+   bstrncpy(mr.VolStatus, "Purged", sizeof(mr.VolStatus));
+   /* Get list of volumes to truncate */
+   if (!scan_storage_cmd(ua, cmd, true, /* allfrompool */
+                         &drive, &mr, &pr, &action, storage, &nb, &results)) {
+      goto bail_out;
+   }
+
+   if ((sd=open_sd_bsock(ua)) == NULL) {
+      Dmsg0(100, "Can't open connection to sd\n");
+      goto bail_out;
+   }
+
+   /*
+    * Loop over the candidate Volumes and actually truncate them
+    */
+   for (int i=0; i < nb; i++) {
+      mr.clear();
+      mr.MediaId = results[i];
+      if (db_get_media_record(ua->jcr, ua->db, &mr)) {
+         if (strcasecmp(mr.VolStatus, "Purged") != 0) {
+            ua->send_msg(_("Truncate Volume \"%s\" skipped. Status is \"%s\", but must be \"Purged\".\n"),
+               mr.VolumeName, mr.VolStatus);
+            continue;
+         }
+         if (drive < 0) {
+            STORE *store = (STORE*)GetResWithName(R_STORAGE, storage);
+            drive = get_storage_drive(ua, store);
+         }
+
+         /* Must select Pool if not already done */
+         if (pr.PoolId == 0) {
+            pr.PoolId = mr.PoolId;
+            if (!db_get_pool_record(ua->jcr, ua->db, &pr)) {
+               goto bail_out;   /* free allocated memory */
+            }
+         }
+         if (strcasecmp("truncate", action) == 0) {
+            truncate_volume(ua, &mr, pr.Name, storage,
+                            drive, sd);
+         }
+      } else {
+         Dmsg1(0, "Can't find MediaId=%lu\n", mr.MediaId);
+      }
+   }
+
+bail_out:
+   close_db(ua);
+   close_sd_bsock(ua);
+   ua->jcr->wstore = NULL;
+   if (results) {
+      free(results);
+   }
+
+   return 1;
+}
+
 /*
  * IF volume status is Append, Full, Used, or Error, mark it Purged
  *   Purged volumes can then be recycled (if enabled).
@@ -568,12 +754,12 @@ bool mark_media_purged(UAContext *ua, MEDIA_DBR *mr)
        strcmp(mr->VolStatus, "Used")   == 0 ||
        strcmp(mr->VolStatus, "Error")  == 0) {
       bstrncpy(mr->VolStatus, "Purged", sizeof(mr->VolStatus));
+      set_storageid_in_mr(NULL, mr);
       if (!db_update_media_record(jcr, ua->db, mr)) {
          return false;
       }
       pm_strcpy(jcr->VolumeName, mr->VolumeName);
-      generate_job_event(jcr, "VolumePurged");
-      generate_plugin_event(jcr, bEventVolumePurged);
+      generate_plugin_event(jcr, bDirEventVolumePurged);
       /*
        * If the RecyclePool is defined, move the volume there
        */
@@ -583,12 +769,11 @@ bool mark_media_purged(UAContext *ua, MEDIA_DBR *mr)
          memset(&newpr, 0, sizeof(POOL_DBR));
          newpr.PoolId = mr->RecyclePoolId;
          oldpr.PoolId = mr->PoolId;
-         if (   db_get_pool_record(jcr, ua->db, &oldpr) 
-             && db_get_pool_record(jcr, ua->db, &newpr)) 
-         {
+         if (   db_get_pool_numvols(jcr, ua->db, &oldpr)
+             && db_get_pool_numvols(jcr, ua->db, &newpr)) {
             /* check if destination pool size is ok */
             if (newpr.MaxVols > 0 && newpr.NumVols >= newpr.MaxVols) {
-               ua->error_msg(_("Unable move recycled Volume in full " 
+               ua->error_msg(_("Unable move recycled Volume in full "
                               "Pool \"%s\" MaxVols=%d\n"),
                         newpr.Name, newpr.MaxVols);
 
@@ -599,10 +784,11 @@ bool mark_media_purged(UAContext *ua, MEDIA_DBR *mr)
             ua->error_msg("%s", db_strerror(ua->db));
          }
       }
-      /* Send message to Job report, if it is a *real* job */           
+
+      /* Send message to Job report, if it is a *real* job */
       if (jcr && jcr->JobId > 0) {
          Jmsg(jcr, M_INFO, 0, _("All records pruned from Volume \"%s\"; marking it \"Purged\"\n"),
-            mr->VolumeName); 
+            mr->VolumeName);
       }
       return true;
    } else {