]> git.sur5r.net Git - bacula/bacula/blobdiff - bacula/src/dird/ua_cmds.c
correct date
[bacula/bacula] / bacula / src / dird / ua_cmds.c
index acdf92a3b9349e901fe63a8f618a412d9bd3a1ee..45f62fd5ec9f216a5f4f5407e3e0b7fca0bbcac9 100644 (file)
@@ -37,32 +37,32 @@ extern int r_first;
 extern int r_last;
 extern struct s_res resources[];
 extern char my_name[];
-#ifndef USE_SEMAPHORE 
-extern workq_t job_wq;               /* work queue */
-#endif
+extern jobq_t job_queue;             /* job queue */
 
 extern char *list_pool;
 
 /* Imported functions */
-extern int statuscmd(UAContext *ua, char *cmd);
-extern int listcmd(UAContext *ua, char *cmd);
-extern int llistcmd(UAContext *ua, char *cmd);
+extern int status_cmd(UAContext *ua, char *cmd);
+extern int list_cmd(UAContext *ua, char *cmd);
+extern int llist_cmd(UAContext *ua, char *cmd);
 extern int show_cmd(UAContext *ua, char *cmd);
 extern int messagescmd(UAContext *ua, char *cmd);
 extern int autodisplaycmd(UAContext *ua, char *cmd);
 extern int sqlquerycmd(UAContext *ua, char *cmd);
 extern int querycmd(UAContext *ua, char *cmd);
-extern int runcmd(UAContext *ua, char *cmd);
+extern int run_cmd(UAContext *ua, char *cmd);
 extern int retentioncmd(UAContext *ua, char *cmd);
 extern int prunecmd(UAContext *ua, char *cmd);
 extern int purgecmd(UAContext *ua, char *cmd);
-extern int restorecmd(UAContext *ua, char *cmd);
-extern int labelcmd(UAContext *ua, char *cmd);
-extern int relabelcmd(UAContext *ua, char *cmd);
+extern int restore_cmd(UAContext *ua, char *cmd);
+extern int label_cmd(UAContext *ua, char *cmd);
+extern int relabel_cmd(UAContext *ua, char *cmd);
 extern int update_slots(UAContext *ua);  /* ua_label.c */
 
 /* Forward referenced functions */
-static int add_cmd(UAContext *ua, char *cmd),  createcmd(UAContext *ua, char *cmd), cancelcmd(UAContext *ua, char *cmd);
+static int add_cmd(UAContext *ua, char *cmd);  
+static int create_cmd(UAContext *ua, char *cmd); 
+static int cancel_cmd(UAContext *ua, char *cmd); 
 static int setdebug_cmd(UAContext *ua, char *cmd);
 static int var_cmd(UAContext *ua, char *cmd);
 static int estimate_cmd(UAContext *ua, char *cmd);
@@ -75,6 +75,7 @@ static int update_volume(UAContext *ua);
 static int update_pool(UAContext *ua);
 static int delete_volume(UAContext *ua);
 static int delete_pool(UAContext *ua);
+static int delete_job(UAContext *ua);
 static int mount_cmd(UAContext *ua, char *cmd);
 static int release_cmd(UAContext *ua, char *cmd);
 static int update_cmd(UAContext *ua, char *cmd);
@@ -85,38 +86,38 @@ int quit_cmd(UAContext *ua, char *cmd);
 
 struct cmdstruct { char *key; int (*func)(UAContext *ua, char *cmd); char *help; }; 
 static struct cmdstruct commands[] = {
- { N_("add"),        add_cmd,       _("add media to a pool")},
+ { N_("add"),        add_cmd,         _("add media to a pool")},
  { N_("autodisplay"), autodisplaycmd, _("autodisplay [on/off] -- console messages")},
- { N_("automount"),   automount_cmd,   _("automount [on/off] -- after label")},
- { N_("cancel"),     cancelcmd,     _("cancel job=nnn -- cancel a job")},
- { N_("create"),     createcmd,     _("create DB Pool from resource")},  
+ { N_("automount"),   automount_cmd,  _("automount [on/off] -- after label")},
+ { N_("cancel"),     cancel_cmd,    _("cancel job=nnn -- cancel a job")},
+ { N_("create"),     create_cmd,    _("create DB Pool from resource")},  
  { N_("delete"),     delete_cmd,    _("delete [pool=<pool-name> | media volume=<volume-name>]")},    
+ { N_("estimate"),   estimate_cmd,  _("performs FileSet estimate, listing gives full listing")},
+ { N_("exit"),       quit_cmd,      _("exit = quit")},
  { N_("help"),       help_cmd,      _("print this command")},
- { N_("label"),      labelcmd,      _("label a tape")},
- { N_("relabel"),    relabelcmd,    _("relabel a tape")},
- { N_("list"),       listcmd,       _("list [pools | jobs | jobtotals | media <pool> | files job=<nn>]; from catalog")},
- { N_("llist"),      llistcmd,      _("full or long list like list command")},
+ { N_("list"),       list_cmd,      _("list [pools | jobs | jobtotals | media <pool> | files jobid=<nn>]; from catalog")},
+ { N_("label"),      label_cmd,     _("label a tape")},
+ { N_("llist"),      llist_cmd,     _("full or long list like list command")},
  { N_("messages"),   messagescmd,   _("messages")},
  { N_("mount"),      mount_cmd,     _("mount <storage-name>")},
- { N_("restore"),    restorecmd,    _("restore files")},
  { N_("prune"),      prunecmd,      _("prune expired records from catalog")},
  { N_("purge"),      purgecmd,      _("purge records from catalog")},
- { N_("run"),        runcmd,        _("run <job-name>")},
+ { N_("quit"),       quit_cmd,      _("quit")},
+ { N_("query"),      querycmd,      _("query catalog")},
+ { N_("restore"),    restore_cmd,   _("restore files")},
+ { N_("relabel"),    relabel_cmd,   _("relabel a tape")},
+ { N_("release"),    release_cmd,   _("release <storage-name>")},
+ { N_("run"),        run_cmd,       _("run <job-name>")},
+ { N_("status"),     status_cmd,    _("status [storage | client]=<name>")},
  { N_("setdebug"),   setdebug_cmd,  _("sets debug level")},
- { N_("estimate"),   estimate_cmd,  _("performs FileSet estimate debug=1 give full listing")},
- { N_("var"),        var_cmd,       _("does variable expansion")},
  { N_("show"),       show_cmd,      _("show (resource records) [jobs | pools | ... | all]")},
  { N_("sqlquery"),   sqlquerycmd,   _("use SQL to query catalog")}, 
- { N_("status"),     statuscmd,     _("status [storage | client]=<name>")},
+ { N_("time"),       time_cmd,      _("print current time")},
  { N_("unmount"),    unmount_cmd,   _("unmount <storage-name>")},
  { N_("update"),     update_cmd,    _("update Volume or Pool")},
  { N_("use"),        use_cmd,       _("use catalog xxx")},
+ { N_("var"),        var_cmd,       _("does variable expansion")},
  { N_("version"),    version_cmd,   _("print Director version")},
- { N_("quit"),       quit_cmd,      _("quit")},
- { N_("query"),      querycmd,      _("query catalog")},
- { N_("release"),    release_cmd,   _("release <storage-name>")},
- { N_("time"),       time_cmd,      _("print current time")},
- { N_("exit"),       quit_cmd,      _("exit = quit")},
  { N_("wait"),       wait_cmd,      _("wait until no jobs are running")},
             };
 #define comsize (sizeof(commands)/sizeof(struct cmdstruct))
@@ -162,7 +163,7 @@ int do_a_command(UAContext *ua, char *cmd)
 void set_pool_dbr_defaults_in_media_dbr(MEDIA_DBR *mr, POOL_DBR *pr)
 {
    mr->PoolId = pr->PoolId;
-   strcpy(mr->VolStatus, "Append");
+   bstrncpy(mr->VolStatus, "Append", sizeof(mr->VolStatus));
    mr->Recycle = pr->Recycle;
    mr->VolRetention = pr->VolRetention;
    mr->VolUseDuration = pr->VolUseDuration;
@@ -183,7 +184,7 @@ static int add_cmd(UAContext *ua, char *cmd)
    int first_id = 0;
    char name[MAX_NAME_LENGTH];
    STORE *store;
-   int slot = 0;
+   int Slot = 0, InChanger = 0;
 
    bsendmsg(ua, _(
 "You probably don't want to be using this command since it\n"
@@ -286,13 +287,18 @@ getVolName:
       if (!get_pint(ua, _("Enter slot (0 for none): "))) {
         return 1;
       }
-      slot = ua->pint32_val;
+      Slot = ua->pint32_val;
+      if (!get_yesno(ua, _("InChanger? yes/no: "))) {
+        return 1;
+      }
+      InChanger = ua->pint32_val;
    }
           
    set_pool_dbr_defaults_in_media_dbr(&mr, &pr);
    for (i=startnum; i < num+startnum; i++) { 
       bsnprintf(mr.VolumeName, sizeof(mr.VolumeName), name, i);
-      mr.Slot = slot++;
+      mr.Slot = Slot++;
+      mr.InChanger = InChanger;
       Dmsg1(200, "Create Volume %s\n", mr.VolumeName);
       if (!db_create_media_record(ua->jcr, ua->db, &mr)) {
         bsendmsg(ua, db_strerror(ua->db));
@@ -340,7 +346,7 @@ int automount_cmd(UAContext *ua, char *cmd)
 /*
  * Cancel a job
  */
-static int cancelcmd(UAContext *ua, char *cmd)
+static int cancel_cmd(UAContext *ua, char *cmd)
 {
    int i;
    int njobs = 0;
@@ -425,15 +431,14 @@ static int cancelcmd(UAContext *ua, char *cmd)
    case JS_WaitJobRes:
    case JS_WaitClientRes:
    case JS_WaitStoreRes:
+   case JS_WaitPriority:
    case JS_WaitMaxJobs:
    case JS_WaitStartTime:
       set_jcr_job_status(jcr, JS_Canceled);
       bsendmsg(ua, _("JobId %d, Job %s marked to be canceled.\n"),
              jcr->JobId, jcr->Job);
-#ifndef USE_SEMAPHORE
-      workq_remove(&job_wq, jcr->work_item); /* attempt to remove it from queue */
-#endif
-      free_jcr(jcr);
+      jobq_remove(&job_queue, jcr); /* attempt to remove it from queue */
+      free_jcr(jcr);                 /* this decrements the use count only */
       return 1;
         
    default:
@@ -561,7 +566,7 @@ int create_pool(JCR *jcr, B_DB *db, POOL *pool, e_pool_op op)
  * Create a Pool Record in the database.
  *  It is always created from the Resource record.
  */
-static int createcmd(UAContext *ua, char *cmd) 
+static int create_cmd(UAContext *ua, char *cmd) 
 {
    POOL *pool;
 
@@ -617,18 +622,18 @@ static int update_cmd(UAContext *ua, char *cmd)
    }
 
    switch (find_arg_keyword(ua, kw)) {
-      case 0:
-      case 1:
-        update_volume(ua);
-        return 1;
-      case 2:
-        update_pool(ua);
-        return 1;
-      case 3:
-        update_slots(ua);
-        return 1;
-      default:
-        break;
+   case 0:
+   case 1:
+      update_volume(ua);
+      return 1;
+   case 2:
+      update_pool(ua);
+      return 1;
+   case 3:
+      update_slots(ua);
+      return 1;
+   default:
+      break;
    }
     
    start_prompt(ua, _("Update choice:\n"));
@@ -636,21 +641,192 @@ static int update_cmd(UAContext *ua, char *cmd)
    add_prompt(ua, _("Pool from resource"));
    add_prompt(ua, _("Slots from autochanger"));
    switch (do_prompt(ua, _("item"), _("Choose catalog item to update"), NULL, 0)) {
-      case 0:
-        update_volume(ua);
-        break;
-      case 1:
-        update_pool(ua);
-        break;
-      case 2:
-        update_slots(ua);
-        break;
-      default:
-        break;
+   case 0:
+      update_volume(ua);
+      break;
+   case 1:
+      update_pool(ua);
+      break;
+   case 2:
+      update_slots(ua);
+      break;
+   default:
+      break;
    }
    return 1;
 }
 
+static void update_volstatus(UAContext *ua, char *val, MEDIA_DBR *mr)
+{
+   POOLMEM *query = get_pool_memory(PM_MESSAGE);
+   char *kw[] = {
+      "Append",
+      "Archive",
+      "Disabled",
+      "Full",
+      "Used", 
+      "Cleaning", 
+      "Recycle",
+      "Read-Only",
+      NULL};
+   bool found = false;
+   int i;
+
+   for (i=0; kw[i]; i++) {
+      if (strcasecmp(val, kw[i]) == 0) {
+        found = true;
+        break;
+      }
+   }
+   if (!found) {
+      bsendmsg(ua, _("Invalid VolStatus specified: %s\n"), val);
+   } else {
+      bstrncpy(mr->VolStatus, kw[i], sizeof(mr->VolStatus));
+      Mmsg(&query, "UPDATE Media SET VolStatus='%s' WHERE MediaId=%u",
+        mr->VolStatus, mr->MediaId);
+      if (!db_sql_query(ua->db, query, NULL, NULL)) {  
+         bsendmsg(ua, "%s", db_strerror(ua->db));
+      } else {
+         bsendmsg(ua, _("New Volume status is: %s\n"), mr->VolStatus);
+      }
+   }
+   free_pool_memory(query);
+}
+
+static void update_volretention(UAContext *ua, char *val, MEDIA_DBR *mr)
+{
+   char ed1[50];
+   POOLMEM *query;
+   if (!duration_to_utime(val, &mr->VolRetention)) {
+      bsendmsg(ua, _("Invalid retention period specified: %s\n"), val);
+      return;
+   }
+   query = get_pool_memory(PM_MESSAGE);
+   Mmsg(&query, "UPDATE Media SET VolRetention=%s WHERE MediaId=%u",
+      edit_uint64(mr->VolRetention, ed1), mr->MediaId);
+   if (!db_sql_query(ua->db, query, NULL, NULL)) {  
+      bsendmsg(ua, "%s", db_strerror(ua->db));
+   } else {
+      bsendmsg(ua, _("New retention seconds is: %s\n"),
+        edit_utime(mr->VolRetention, ed1));
+   }
+   free_pool_memory(query);
+}
+
+static void update_voluseduration(UAContext *ua, char *val, MEDIA_DBR *mr)
+{
+   char ed1[50];
+   POOLMEM *query;
+
+   if (!duration_to_utime(val, &mr->VolUseDuration)) {
+      bsendmsg(ua, _("Invalid use duration specified: %s\n"), val);
+      return;
+   }
+   query = get_pool_memory(PM_MESSAGE);
+   Mmsg(&query, "UPDATE Media SET VolUseDuration=%s WHERE MediaId=%u",
+      edit_uint64(mr->VolUseDuration, ed1), mr->MediaId);
+   if (!db_sql_query(ua->db, query, NULL, NULL)) {  
+      bsendmsg(ua, "%s", db_strerror(ua->db));
+   } else {
+      bsendmsg(ua, _("New use duration is: %s\n"),
+        edit_utime(mr->VolUseDuration, ed1));
+   }
+   free_pool_memory(query);
+}
+
+static void update_volmaxjobs(UAContext *ua, char *val, MEDIA_DBR *mr)
+{
+   POOLMEM *query = get_pool_memory(PM_MESSAGE);
+   Mmsg(&query, "UPDATE Media SET MaxVolJobs=%s WHERE MediaId=%u",
+      val, mr->MediaId);
+   if (!db_sql_query(ua->db, query, NULL, NULL)) {  
+      bsendmsg(ua, "%s", db_strerror(ua->db));
+   } else {
+      bsendmsg(ua, _("New max jobs is: %s\n"), val);
+   }
+   free_pool_memory(query);
+}
+
+static void update_volmaxfiles(UAContext *ua, char *val, MEDIA_DBR *mr)
+{
+   POOLMEM *query = get_pool_memory(PM_MESSAGE);
+   Mmsg(&query, "UPDATE Media SET MaxVolFiles=%s WHERE MediaId=%u",
+      val, mr->MediaId);
+   if (!db_sql_query(ua->db, query, NULL, NULL)) {  
+      bsendmsg(ua, "%s", db_strerror(ua->db));
+   } else {
+      bsendmsg(ua, _("New max files is: %s\n"), val);
+   }
+   free_pool_memory(query);
+}
+
+static void update_volmaxbytes(UAContext *ua, char *val, MEDIA_DBR *mr)
+{
+   uint64_t maxbytes;
+   char ed1[50];
+   POOLMEM *query;
+
+   if (!size_to_uint64(val, strlen(val), &maxbytes)) {
+      bsendmsg(ua, _("Invalid max. bytes specification: %s\n"), val);
+      return;
+   } 
+   query = get_pool_memory(PM_MESSAGE);
+   Mmsg(&query, "UPDATE Media SET MaxVolBytes=%s WHERE MediaId=%u",
+      edit_uint64(maxbytes, ed1), mr->MediaId);
+   if (!db_sql_query(ua->db, query, NULL, NULL)) {  
+      bsendmsg(ua, "%s", db_strerror(ua->db));
+   } else {
+      bsendmsg(ua, _("New Max bytes is: %s\n"), edit_uint64(maxbytes, ed1));
+   }
+   free_pool_memory(query);
+}
+
+static void update_volrecycle(UAContext *ua, char *val, MEDIA_DBR *mr)
+{
+   int recycle;
+   POOLMEM *query;
+   if (strcasecmp(val, _("yes")) == 0) {
+      recycle = 1;
+   } else if (strcasecmp(val, _("no")) == 0) {
+      recycle = 0;
+   } else {
+      bsendmsg(ua, _("Invalid value. It must by yes or no.\n"));
+      return;
+   }
+   query = get_pool_memory(PM_MESSAGE);
+   Mmsg(&query, "UPDATE Media SET Recycle=%d WHERE MediaId=%u",
+      recycle, mr->MediaId);
+   if (!db_sql_query(ua->db, query, NULL, NULL)) {  
+      bsendmsg(ua, "%s", db_strerror(ua->db));
+   } else {      
+      bsendmsg(ua, _("New Recycle flag is: %s\n"),
+         mr->Recycle==1?_("yes"):_("no"));
+   }
+   free_pool_memory(query);
+}
+
+/* Modify the Pool in which this Volume is located */
+static void update_volpool(UAContext *ua, char *val, MEDIA_DBR *mr)
+{
+   POOL_DBR pr;
+
+   memset(&pr, 0, sizeof(pr));
+   bstrncpy(pr.Name, val, sizeof(pr.Name));
+   if (!get_pool_dbr(ua, &pr)) {
+      return;
+   }
+   mr->PoolId = pr.PoolId;           /* set new PoolId */
+   /*
+    * Make sure to use db_update... rather than doing this directly,
+    *  so that any Slot is handled correctly. 
+    */
+   if (!db_update_media_record(ua->jcr, ua->db, mr)) {
+      bsendmsg(ua, _("Error updating media record Pool: ERR=%s"), db_strerror(ua->db));
+   } else {
+      bsendmsg(ua, _("New Pool is: %s\n"), pr.Name);
+   }
+}
+
 /*
  * Update a media record -- allows you to change the
  *  Volume status. E.g. if you want Bacula to stop
@@ -660,11 +836,57 @@ static int update_cmd(UAContext *ua, char *cmd)
 static int update_volume(UAContext *ua)
 {
    MEDIA_DBR mr;
+   POOL_DBR pr;
    POOLMEM *query;
    char ed1[30];
+   bool done = false;
+   char *kw[] = {
+      N_("VolStatus"),                /* 0 */
+      N_("VolRetention"),             /* 1 */
+      N_("VolUse"),                   /* 2 */
+      N_("MaxVolJobs"),               /* 3 */
+      N_("MaxVolFiles"),              /* 4 */
+      N_("MaxVolBytes"),              /* 5 */
+      N_("Recycle"),                  /* 6 */
+      N_("Pool"),                     /* 7 */
+      NULL };
+
+   for (int i=0; kw[i]; i++) {
+      int j;
+      if ((j=find_arg_with_value(ua, kw[i])) > 0) {
+        if (!select_media_dbr(ua, &mr)) {
+           return 0;
+        }
+        switch (i) {
+        case 0:
+           update_volstatus(ua, ua->argv[j], &mr);
+           break;
+        case 1:
+           update_volretention(ua, ua->argv[j], &mr);
+           break;
+        case 2:
+           update_voluseduration(ua, ua->argv[j], &mr);
+           break;
+        case 3:
+           update_volmaxjobs(ua, ua->argv[j], &mr);
+           break;
+        case 4:
+           update_volmaxfiles(ua, ua->argv[j], &mr);
+           break;
+        case 5:
+           update_volmaxbytes(ua, ua->argv[j], &mr);
+           break;
+        case 6:
+           update_volrecycle(ua, ua->argv[j], &mr);
+           break;
+        case 7:
+           update_volpool(ua, ua->argv[j], &mr);
+        }
+        done = true;
+      }
+   }
 
-
-   for (int done=0; !done; ) {
+   for ( ; !done; ) {
       if (!select_media_dbr(ua, &mr)) {
         return 0;
       }
@@ -678,7 +900,9 @@ static int update_volume(UAContext *ua)
       add_prompt(ua, _("Maximum Volume Bytes"));
       add_prompt(ua, _("Recycle Flag"));
       add_prompt(ua, _("Slot"));
+      add_prompt(ua, _("InChanger Flag"));
       add_prompt(ua, _("Volume Files"));
+      add_prompt(ua, _("Pool"));
       add_prompt(ua, _("Done"));
       switch (do_prompt(ua, "", _("Select parameter to modify"), NULL, 0)) {
       case 0:                        /* Volume Status */
@@ -698,16 +922,7 @@ static int update_volume(UAContext *ua)
          if (do_prompt(ua, "", _("Choose new Volume Status"), ua->cmd, sizeof(mr.VolStatus)) < 0) {
            return 1;
         }
-        bstrncpy(mr.VolStatus, ua->cmd, sizeof(mr.VolStatus));
-        query = get_pool_memory(PM_MESSAGE);
-         Mmsg(&query, "UPDATE Media SET VolStatus='%s' WHERE MediaId=%u",
-           mr.VolStatus, mr.MediaId);
-        if (!db_sql_query(ua->db, query, NULL, NULL)) {  
-            bsendmsg(ua, "%s", db_strerror(ua->db));
-        } else {
-            bsendmsg(ua, _("New Volume status is: %s\n"), mr.VolStatus);
-        }
-        free_pool_memory(query);
+        update_volstatus(ua, ua->cmd, &mr);
         break;
       case 1:                        /* Retention */
          bsendmsg(ua, _("Current retention seconds is: %s\n"),
@@ -715,20 +930,7 @@ static int update_volume(UAContext *ua)
          if (!get_cmd(ua, _("Enter Volume Retention period: "))) {
            return 0;
         }
-        if (!duration_to_utime(ua->cmd, &mr.VolRetention)) {
-            bsendmsg(ua, _("Invalid retention period specified.\n"));
-           break;
-        }
-        query = get_pool_memory(PM_MESSAGE);
-         Mmsg(&query, "UPDATE Media SET VolRetention=%s WHERE MediaId=%u",
-           edit_uint64(mr.VolRetention, ed1), mr.MediaId);
-        if (!db_sql_query(ua->db, query, NULL, NULL)) {  
-            bsendmsg(ua, "%s", db_strerror(ua->db));
-        } else {
-            bsendmsg(ua, _("New retention seconds is: %s\n"),
-              edit_utime(mr.VolRetention, ed1));
-        }
-        free_pool_memory(query);
+        update_volretention(ua, ua->cmd, &mr);
         break;
 
       case 2:                        /* Use Duration */
@@ -737,103 +939,45 @@ static int update_volume(UAContext *ua)
          if (!get_cmd(ua, _("Enter Volume Use Duration: "))) {
            return 0;
         }
-        if (!duration_to_utime(ua->cmd, &mr.VolUseDuration)) {
-            bsendmsg(ua, _("Invalid use duration specified.\n"));
-           break;
-        }
-        query = get_pool_memory(PM_MESSAGE);
-         Mmsg(&query, "UPDATE Media SET VolUseDuration=%s WHERE MediaId=%u",
-           edit_uint64(mr.VolUseDuration, ed1), mr.MediaId);
-        if (!db_sql_query(ua->db, query, NULL, NULL)) {  
-            bsendmsg(ua, "%s", db_strerror(ua->db));
-        } else {
-            bsendmsg(ua, _("New use duration is: %s\n"),
-              edit_utime(mr.VolUseDuration, ed1));
-        }
-        free_pool_memory(query);
+        update_voluseduration(ua, ua->cmd, &mr);
         break;
 
       case 3:                        /* Max Jobs */
-        int32_t maxjobs;
          bsendmsg(ua, _("Current max jobs is: %u\n"), mr.MaxVolJobs);
          if (!get_pint(ua, _("Enter new Maximum Jobs: "))) {
            return 0;
         }
-        maxjobs = ua->pint32_val;
-        query = get_pool_memory(PM_MESSAGE);
-         Mmsg(&query, "UPDATE Media SET MaxVolJobs=%u WHERE MediaId=%u",
-           maxjobs, mr.MediaId);
-        if (!db_sql_query(ua->db, query, NULL, NULL)) {  
-            bsendmsg(ua, "%s", db_strerror(ua->db));
-        } else {
-            bsendmsg(ua, _("New max jobs is: %u\n"), maxjobs);
-        }
-        free_pool_memory(query);
+        update_volmaxjobs(ua, ua->cmd, &mr);
         break;
 
       case 4:                        /* Max Files */
-        int32_t maxfiles;
          bsendmsg(ua, _("Current max files is: %u\n"), mr.MaxVolFiles);
          if (!get_pint(ua, _("Enter new Maximum Files: "))) {
            return 0;
         }
-        maxfiles = ua->pint32_val;
-        query = get_pool_memory(PM_MESSAGE);
-         Mmsg(&query, "UPDATE Media SET MaxVolFiles=%u WHERE MediaId=%u",
-           maxfiles, mr.MediaId);
-        if (!db_sql_query(ua->db, query, NULL, NULL)) {  
-            bsendmsg(ua, "%s", db_strerror(ua->db));
-        } else {
-            bsendmsg(ua, _("New max files is: %u\n"), maxfiles);
-        }
-        free_pool_memory(query);
+        update_volmaxfiles(ua, ua->cmd, &mr);
         break;
 
       case 5:                        /* Max Bytes */
-        uint64_t maxbytes;
          bsendmsg(ua, _("Current value is: %s\n"), edit_uint64(mr.MaxVolBytes, ed1));
          if (!get_cmd(ua, _("Enter new Maximum Bytes: "))) {
            return 0;
         }
-        if (!size_to_uint64(ua->cmd, strlen(ua->cmd), &maxbytes)) {
-            bsendmsg(ua, _("Invalid byte size specification.\n"));
-           break;
-        } 
-        query = get_pool_memory(PM_MESSAGE);
-         Mmsg(&query, "UPDATE Media SET MaxVolBytes=%s WHERE MediaId=%u",
-           edit_uint64(maxbytes, ed1), mr.MediaId);
-        if (!db_sql_query(ua->db, query, NULL, NULL)) {  
-            bsendmsg(ua, "%s", db_strerror(ua->db));
-        } else {
-            bsendmsg(ua, _("New Max bytes is: %s\n"), edit_uint64(maxbytes, ed1));
-        }
-        free_pool_memory(query);
+        update_volmaxbytes(ua, ua->cmd, &mr);
         break;
 
 
       case 6:                        /* Recycle */
-        int recycle;
          bsendmsg(ua, _("Current recycle flag is: %s\n"),
             mr.Recycle==1?_("yes"):_("no"));
          if (!get_yesno(ua, _("Enter new Recycle status: "))) {
            return 0;
         }
-        recycle = ua->pint32_val;
-        query = get_pool_memory(PM_MESSAGE);
-         Mmsg(&query, "UPDATE Media SET Recycle=%d WHERE MediaId=%u",
-           recycle, mr.MediaId);
-        if (!db_sql_query(ua->db, query, NULL, NULL)) {  
-            bsendmsg(ua, "%s", db_strerror(ua->db));
-        } else {       
-            bsendmsg(ua, _("New recycle flag is: %s\n"),
-               mr.Recycle==1?_("yes"):_("no"));
-        }
-        free_pool_memory(query);
+        update_volrecycle(ua, ua->cmd, &mr);
         break;
 
       case 7:                        /* Slot */
-        int slot;
-        POOL_DBR pr;
+        int Slot;
 
         memset(&pr, 0, sizeof(POOL_DBR));
         pr.PoolId = mr.PoolId;
@@ -845,24 +989,43 @@ static int update_volume(UAContext *ua)
          if (!get_pint(ua, _("Enter new Slot: "))) {
            return 0;
         }
-        slot = ua->pint32_val;
-        if (pr.MaxVols > 0 && slot > (int)pr.MaxVols) {
+        Slot = ua->pint32_val;
+        if (pr.MaxVols > 0 && Slot > (int)pr.MaxVols) {
             bsendmsg(ua, _("Invalid slot, it must be between 0 and %d\n"),
               pr.MaxVols);
            break;
         }
-        query = get_pool_memory(PM_MESSAGE);
-         Mmsg(&query, "UPDATE Media SET Slot=%d WHERE MediaId=%u",
-           slot, mr.MediaId);
-        if (!db_sql_query(ua->db, query, NULL, NULL)) {  
-            bsendmsg(ua, "%s", db_strerror(ua->db));
+        mr.Slot = Slot;
+        /*
+         * Make sure to use db_update... rather than doing this directly,
+         *   so that any Slot is handled correctly. 
+         */
+        if (!db_update_media_record(ua->jcr, ua->db, &mr)) {
+            bsendmsg(ua, _("Error updating media record Slot: ERR=%s"), db_strerror(ua->db));
         } else {
-            bsendmsg(ua, "New Slot is: %d\n", slot);
+            bsendmsg(ua, _("New Slot is: %s\n"), mr.Slot);
         }
-        free_pool_memory(query);
         break;
 
-      case 8:                        /* Volume Files */
+      case 8:                        /* InChanger */
+         bsendmsg(ua, _("Current InChanger flag is: %d\n"), mr.InChanger);
+         if (!get_yesno(ua, _("Set InChanger flag? yes/no: "))) {
+           return 0;
+        }
+        mr.InChanger = ua->pint32_val;
+        /*
+         * Make sure to use db_update... rather than doing this directly,
+         *   so that any Slot is handled correctly. 
+         */
+        if (!db_update_media_record(ua->jcr, ua->db, &mr)) {
+            bsendmsg(ua, _("Error updating media record Slot: ERR=%s"), db_strerror(ua->db));
+        } else {
+            bsendmsg(ua, _("New InChanger flag is: %s\n"), mr.InChanger);
+        }
+        break;
+
+
+      case 9:                        /* Volume Files */
         int32_t VolFiles;
          bsendmsg(ua, _("Warning changing Volume Files can result\n"
                         "in loss of data on your Volume\n\n"));
@@ -888,6 +1051,20 @@ static int update_volume(UAContext *ua)
         free_pool_memory(query);
         break;
 
+      case 10:                        /* Volume's Pool */
+        memset(&pr, 0, sizeof(POOL_DBR));
+        pr.PoolId = mr.PoolId;
+        if (!db_get_pool_record(ua->jcr, ua->db, &pr)) {
+            bsendmsg(ua, "%s", db_strerror(ua->db));
+           return 0;
+        }
+         bsendmsg(ua, _("Current Pool is: %s\n"), pr.Name);
+         if (!get_cmd(ua, _("Enter new Pool name: "))) {
+           return 0;
+        }
+        update_volpool(ua, ua->cmd, &mr);
+        return 1;
+
       default:                       /* Done or error */
          bsendmsg(ua, "Selection done.\n");
         return 1;
@@ -906,7 +1083,6 @@ static int update_pool(UAContext *ua)
    POOL *pool;
    POOLMEM *query;      
    
-   
    pool = get_pool_resource(ua);
    if (!pool) {
       return 0;
@@ -981,7 +1157,6 @@ static void do_client_setdebug(UAContext *ua, CLIENT *client, int level)
    bnet_sig(fd, BNET_TERMINATE);
    bnet_close(fd);
    ua->jcr->file_bsock = NULL;
-
    return;  
 }
 
@@ -1098,7 +1273,8 @@ static int setdebug_cmd(UAContext *ua, char *cmd)
         debug_level = level;
         return 1;
       }
-      if (strcasecmp(ua->argk[i], _("client")) == 0) {
+      if (strcasecmp(ua->argk[i], _("client")) == 0 ||
+          strcasecmp(ua->argk[i], _("fd")) == 0) {
         client = NULL;
         if (ua->argv[i]) {
            client = (CLIENT *)GetResWithName(R_CLIENT, ua->argv[i]);
@@ -1115,7 +1291,8 @@ static int setdebug_cmd(UAContext *ua, char *cmd)
       }
 
       if (strcasecmp(ua->argk[i], _("store")) == 0 ||
-          strcasecmp(ua->argk[i], _("storage")) == 0) {
+          strcasecmp(ua->argk[i], _("storage")) == 0 ||
+          strcasecmp(ua->argk[i], _("sd")) == 0) {
         store = NULL;
         if (ua->argv[i]) {
            store = (STORE *)GetResWithName(R_STORAGE, ua->argv[i]);
@@ -1141,39 +1318,48 @@ static int setdebug_cmd(UAContext *ua, char *cmd)
    add_prompt(ua, _("Client"));
    add_prompt(ua, _("All"));
    switch(do_prompt(ua, "", _("Select daemon type to set debug level"), NULL, 0)) {
-      case 0:                        /* Director */
-        debug_level = level;
-        break;
-      case 1:
-        store = get_storage_resource(ua, 0);
-        if (store) {
-           do_storage_setdebug(ua, store, level);
-        }
-        break;
-      case 2:
-        client = select_client_resource(ua);
-        if (client) {
-           do_client_setdebug(ua, client, level);
-        }
-        break;
-      case 3:
-        do_all_setdebug(ua, level);
-        break;
-      default:
-        break;
+   case 0:                        /* Director */
+      debug_level = level;
+      break;
+   case 1:
+      store = get_storage_resource(ua, 0);
+      if (store) {
+        do_storage_setdebug(ua, store, level);
+      }
+      break;
+   case 2:
+      client = select_client_resource(ua);
+      if (client) {
+        do_client_setdebug(ua, client, level);
+      }
+      break;
+   case 3:
+      do_all_setdebug(ua, level);
+      break;
+   default:
+      break;
    }
    return 1;
 }
 
 static int var_cmd(UAContext *ua, char *cmd)
 {
-   POOLMEM *var = get_pool_memory(PM_FNAME);
-   for (int i=1; i<ua->argc; i++) {
-      if (ua->argk[i] && variable_expansion(ua->jcr, ua->argk[i], &var)) {
-         bsendmsg(ua, "%s\n", var);
-      }
+   POOLMEM *val = get_pool_memory(PM_FNAME);
+   char *var;
+
+   if (!open_db(ua)) {
+      return 1;
+   }
+   for (var=ua->cmd; *var != ' '; ) {    /* skip command */
+      var++;
+   }
+   while (*var == ' ') {                 /* skip spaces */
+      var++;
    }
-   free_pool_memory(var);
+   Dmsg1(100, "Var=%s:\n", var);
+   variable_expansion(ua->jcr, var, &val);  
+   bsendmsg(ua, "%s\n", val);
+   free_pool_memory(val);
    return 1;
 }
 
@@ -1187,7 +1373,8 @@ static int estimate_cmd(UAContext *ua, char *cmd)
    char since[MAXSTRING];
 
    for (int i=1; i<ua->argc; i++) {
-      if (strcasecmp(ua->argk[i], _("client")) == 0) {
+      if (strcasecmp(ua->argk[i], _("client")) == 0 ||
+          strcasecmp(ua->argk[i], _("fd")) == 0) {
         if (ua->argv[i]) {
            client = (CLIENT *)GetResWithName(R_CLIENT, ua->argv[i]);
            continue;
@@ -1296,40 +1483,73 @@ static int delete_cmd(UAContext *ua, char *cmd)
    static char *keywords[] = {
       N_("volume"),
       N_("pool"),
+      N_("job"),
       NULL};
 
    if (!open_db(ua)) {
       return 1;
    }
 
-   bsendmsg(ua, _(
-"In general it is not a good idea to delete either a\n"
-"Pool or a Volume since they may contain data.\n\n"));
      
    switch (find_arg_keyword(ua, keywords)) {
-      case 0:
-        delete_volume(ua);     
-        return 1;
-      case 1:
-        delete_pool(ua);
-        return 1;
-      default:
-        break;
+   case 0:
+      delete_volume(ua);     
+      return 1;
+   case 1:
+      delete_pool(ua);
+      return 1;
+   case 2:
+      delete_job(ua);
+      return 1;
+   default:
+      break;
    }
+
+   bsendmsg(ua, _(
+"In general it is not a good idea to delete either a\n"
+"Pool or a Volume since they may contain data.\n\n"));
+
    switch (do_keyword_prompt(ua, _("Choose catalog item to delete"), keywords)) {
-      case 0:
-        delete_volume(ua);
-        break;
-      case 1:
-        delete_pool(ua);
-        break;
-      default:
-         bsendmsg(ua, _("Nothing done.\n"));
-        break;
+   case 0:
+      delete_volume(ua);
+      break;
+   case 1:
+      delete_pool(ua);
+      break;
+   case 2:
+      delete_job(ua);
+      return 1;
+   default:
+      bsendmsg(ua, _("Nothing done.\n"));
+      break;
    }
    return 1;
 }
 
+static int delete_job(UAContext *ua)
+{
+   POOLMEM *query = get_pool_memory(PM_MESSAGE);
+   JobId_t JobId;
+
+   int i = find_arg_with_value(ua, "jobid");
+   if (i >= 0) {
+      JobId = str_to_int64(ua->argv[i]);
+   } else if (!get_pint(ua, _("Enter JobId to delete: "))) {
+      return 0;
+   } else {
+      JobId = ua->pint32_val; 
+   }
+   Mmsg(&query, "DELETE FROM Job WHERE JobId=%u", JobId);
+   db_sql_query(ua->db, query, NULL, (void *)NULL);
+   Mmsg(&query, "DELETE FROM File WHERE JobId=%u", JobId);
+   db_sql_query(ua->db, query, NULL, (void *)NULL);
+   Mmsg(&query, "DELETE FROM JobMedia WHERE JobId=%u", JobId);
+   db_sql_query(ua->db, query, NULL, (void *)NULL);
+   free_pool_memory(query);
+   bsendmsg(ua, _("Job %u and associated records deleted from the catalog.\n"), JobId);
+   return 1;
+}
+
 /*
  * Delete media records from database -- dangerous 
  */
@@ -1502,12 +1722,11 @@ static int help_cmd(UAContext *ua, char *cmd)
 {
    unsigned int i;
 
-/* usage(); */
    bsendmsg(ua, _("  Command    Description\n  =======    ===========\n"));
    for (i=0; i<comsize; i++) {
       bsendmsg(ua, _("  %-10s %s\n"), _(commands[i].key), _(commands[i].help));
    }
-   bsendmsg(ua, "\n");
+   bsendmsg(ua, _("\nWhen at a prompt, entering a period cancels the command.\n\n"));
    return 1;
 }
 
@@ -1546,9 +1765,12 @@ int open_db(UAContext *ua)
    ua->db = db_init_database(ua->jcr, ua->catalog->db_name, ua->catalog->db_user,
                             ua->catalog->db_password, ua->catalog->db_address,
                             ua->catalog->db_port, ua->catalog->db_socket);
-   if (!db_open_database(ua->jcr, ua->db)) {
-      bsendmsg(ua, _("Could not open DB %s: ERR=%s"), 
-        ua->catalog->db_name, db_strerror(ua->db));
+   if (!ua->db || !db_open_database(ua->jcr, ua->db)) {
+      bsendmsg(ua, _("Could not open database \"%s\".\n"),
+                ua->catalog->db_name);
+      if (ua->db) {
+         bsendmsg(ua, "%s", db_strerror(ua->db));
+      }
       close_db(ua);
       return 0;
    }
@@ -1561,7 +1783,9 @@ void close_db(UAContext *ua)
 {
    if (ua->db) {
       db_close_database(ua->jcr, ua->db);
+      ua->db = NULL;
+      if (ua->jcr) {
+        ua->jcr->db = NULL;
+      }
    }
-   ua->db = NULL;
-   ua->jcr->db = NULL;
 }