]> git.proxmox.com Git - pve-manager.git/blobdiff - www/manager6/dc/Backup.js
ui: dc/Backup: never show id input field, autogenerate id
[pve-manager.git] / www / manager6 / dc / Backup.js
index 7d2c0381841593344e4374559f533a900da0e21a..3f250a76329809c941d9b76104c6b2aa0736763e 100644 (file)
 Ext.define('PVE.dc.BackupEdit', {
-    extend: 'PVE.window.Edit',
+    extend: 'Proxmox.window.Edit',
     alias: ['widget.pveDcBackupEdit'],
 
-    initComponent : function() {
-       /*jslint confusion: true */
-         var me = this;
+    defaultFocus: undefined,
 
-        me.create = !me.jobid;
+    initComponent: function() {
+       let me = this;
 
-       var url;
-       var method;
+       me.isCreate = !me.jobid;
 
-       if (me.create) {
-            url = '/api2/extjs/cluster/backup';
-            method = 'POST';
-        } else {
-            url = '/api2/extjs/cluster/backup/' + me.jobid;
-            method = 'PUT';
-        }
+       let url, method;
+       if (me.isCreate) {
+           url = '/api2/extjs/cluster/backup';
+           method = 'POST';
+       } else {
+           url = '/api2/extjs/cluster/backup/' + me.jobid;
+           method = 'PUT';
+       }
 
-       var vmidField = Ext.create('Ext.form.field.Hidden', {
-           name: 'vmid'
+       let vmidField = Ext.create('Ext.form.field.Hidden', {
+           name: 'vmid',
        });
 
-       var selModeField =  Ext.create('PVE.form.KVComboBox', {
-           xtype: 'pveKVComboBox',
+       // 'value' can be assigned a string or an array
+       let selModeField = Ext.create('Proxmox.form.KVComboBox', {
+           xtype: 'proxmoxKVComboBox',
            comboItems: [
                ['include', gettext('Include selected VMs')],
                ['all', gettext('All')],
-               ['exclude', gettext('Exclude selected VMs')]
+               ['exclude', gettext('Exclude selected VMs')],
+               ['pool', gettext('Pool based')],
            ],
            fieldLabel: gettext('Selection mode'),
            name: 'selMode',
-           value: ''
+           value: '',
        });
 
-       var insideUpdate = false;
-       
-       var sm = Ext.create('Ext.selection.CheckboxModel', {
+       let sm = Ext.create('Ext.selection.CheckboxModel', {
            mode: 'SIMPLE',
            listeners: {
                selectionchange: function(model, selected) {
-                   if (!insideUpdate) { // avoid endless loop
-                       var sel = [];
-                       Ext.Array.each(selected, function(record) {
-                           sel.push(record.data.vmid);
-                       });
-
-                       insideUpdate = true;
-                       vmidField.setValue(sel);
-                       insideUpdate = false;
-                   }
-               }
-           }
+                   var sel = [];
+                   Ext.Array.each(selected, function(record) {
+                       sel.push(record.data.vmid);
+                   });
+
+                   // to avoid endless recursion suspend the vmidField change
+                   // event temporary as it calls us again
+                   vmidField.suspendEvent('change');
+                   vmidField.setValue(sel);
+                   vmidField.resumeEvent('change');
+               },
+           },
        });
 
-       var storagesel = Ext.create('PVE.form.StorageSelector', {
+       let storagesel = Ext.create('PVE.form.StorageSelector', {
            fieldLabel: gettext('Storage'),
-           nodename: 'localhost',
+           clusterView: true,
            storageContent: 'backup',
            allowBlank: false,
-           name: 'storage'
+           name: 'storage',
+           listeners: {
+               change: function(f, v) {
+                   let store = f.getStore();
+                   let rec = store.findRecord('storage', v, 0, false, true, true);
+                   let compressionSelector = me.down('pveCompressionSelector');
+
+                   if (rec && rec.data && rec.data.type === 'pbs') {
+                       compressionSelector.setValue('zstd');
+                       compressionSelector.setDisabled(true);
+                   } else if (!compressionSelector.getEditable()) {
+                       compressionSelector.setDisabled(false);
+                   }
+               },
+           },
        });
 
-       var store = new Ext.data.Store({
+       let store = new Ext.data.Store({
            model: 'PVEResources',
-           sorters: { 
-               property: 'vmid', 
-               order: 'ASC' 
-           }
+           sorters: {
+               property: 'vmid',
+               order: 'ASC',
+           },
        });
 
-       var vmgrid = Ext.createWidget('grid', {
+       let vmgrid = Ext.createWidget('grid', {
            store: store,
            border: true,
            height: 300,
            selModel: sm,
            disabled: true,
            columns: [
-               { 
+               {
                    header: 'ID',
                    dataIndex: 'vmid',
-                   width: 60
+                   width: 60,
                },
-               { 
+               {
                    header: gettext('Node'),
-                   dataIndex: 'node'
+                   dataIndex: 'node',
                },
-               { 
+               {
                    header: gettext('Status'),
                    dataIndex: 'uptime',
                    renderer: function(value) {
                        if (value) {
-                           return PVE.Utils.runningText;
+                           return Proxmox.Utils.runningText;
                        } else {
-                           return PVE.Utils.stoppedText;
+                           return Proxmox.Utils.stoppedText;
                        }
-                   }
+                   },
                },
-               { 
-                   header: gettext('Name'), 
+               {
+                   header: gettext('Name'),
                    dataIndex: 'name',
-                   flex: 1 
+                   flex: 1,
                },
-               { 
-                   header: gettext('Type'), 
-                   dataIndex: 'type'
-               }
-           ]
+               {
+                   header: gettext('Type'),
+                   dataIndex: 'type',
+               },
+           ],
        });
 
-       var nodesel = Ext.create('PVE.form.NodeSelector', {
+       let selectPoolMembers = function(poolid) {
+           if (!poolid) {
+               return;
+           }
+           sm.deselectAll(true);
+           store.filter([
+               {
+                   id: 'poolFilter',
+                   property: 'pool',
+                   value: poolid,
+               },
+           ]);
+           sm.selectAll(true);
+       };
+
+       let selPool = Ext.create('PVE.form.PoolSelector', {
+           fieldLabel: gettext('Pool to backup'),
+           hidden: true,
+           allowBlank: true,
+           name: 'pool',
+           listeners: {
+               change: function(selpool, newValue, oldValue) {
+                   selectPoolMembers(newValue);
+               },
+           },
+       });
+
+       let nodesel = Ext.create('PVE.form.NodeSelector', {
            name: 'node',
            fieldLabel: gettext('Node'),
            allowBlank: true,
@@ -119,90 +159,97 @@ Ext.define('PVE.dc.BackupEdit', {
            emptyText: '-- ' + gettext('All') + ' --',
            listeners: {
                change: function(f, value) {
-                   storagesel.setNodename(value || 'localhost');
-                   var mode = selModeField.getValue();
+                   storagesel.setNodename(value);
+                   let mode = selModeField.getValue();
                    store.clearFilter();
                    store.filterBy(function(rec) {
-                       return (!value || rec.get('node') === value);
+                       return !value || rec.get('node') === value;
                    });
                    if (mode === 'all') {
                        sm.selectAll(true);
                    }
-               }
-           }
+                   if (mode === 'pool') {
+                       selectPoolMembers(selPool.value);
+                   }
+               },
+           },
        });
 
-       var column1 = [
+       let column1 = [
            nodesel,
            storagesel,
            {
-               xtype: 'pveDayOfWeekSelector',
-               name: 'dow',
-               fieldLabel: gettext('Day of week'),
-               multiSelect: true,
-               value: ['sat'],
-               allowBlank: false
-           },
-           {
-               xtype: 'timefield',
-               fieldLabel: gettext('Start Time'),
-               name: 'starttime',
-               format: 'H:i',
-               value: '00:00',
-               allowBlank: false
+               xtype: 'pveCalendarEvent',
+               fieldLabel: gettext('Schedule'),
+               allowBlank: false,
+               name: 'schedule',
            },
-           selModeField
+           selModeField,
+           selPool,
        ];
 
-       var column2 = [
+       let column2 = [
            {
                xtype: 'textfield',
                fieldLabel: gettext('Send email to'),
-               name: 'mailto'
+               name: 'mailto',
            },
            {
                xtype: 'pveEmailNotificationSelector',
-               fieldLabel: gettext('Email notification'),
+               fieldLabel: gettext('Email'),
                name: 'mailnotification',
-               deleteEmpty: me.create ? false : true,
-               value: me.create ? 'always' : ''
+               deleteEmpty: !me.isCreate,
+               value: me.isCreate ? 'always' : '',
            },
            {
                xtype: 'pveCompressionSelector',
                fieldLabel: gettext('Compression'),
                name: 'compress',
-               deleteEmpty: me.create ? false : true,
-               value: 'lzo'
+               deleteEmpty: !me.isCreate,
+               value: 'zstd',
            },
            {
                xtype: 'pveBackupModeSelector',
                fieldLabel: gettext('Mode'),
                value: 'snapshot',
-               name: 'mode'
+               name: 'mode',
            },
            {
-               xtype: 'pvecheckbox',
+               xtype: 'proxmoxcheckbox',
                fieldLabel: gettext('Enable'),
                name: 'enabled',
                uncheckedValue: 0,
                defaultValue: 1,
-               checked: true
+               checked: true,
            },
-           vmidField
+           vmidField,
        ];
 
-       var ipanel = Ext.create('PVE.panel.InputPanel', {
+       let ipanel = Ext.create('Proxmox.panel.InputPanel', {
+           onlineHelp: 'chapter_vzdump',
            column1: column1,
-           column2:  column2,
+           column2: column2,
+           columnB: [
+               {
+                   xtype: 'proxmoxtextfield',
+                   name: 'comment',
+                   fieldLabel: gettext('Comment'),
+                   deleteEmpty: !me.isCreate,
+               },
+           ],
            onGetValues: function(values) {
                if (!values.node) {
-                   if (!me.create) {
-                       PVE.Utils.assemble_field_data(values, { 'delete': 'node' }); 
+                   if (!me.isCreate) {
+                       Proxmox.Utils.assemble_field_data(values, { 'delete': 'node' });
                    }
                    delete values.node;
                }
 
-               var selMode = values.selMode;
+               if (!values.id && me.isCreate) {
+                   values.id = 'backup-' + Ext.data.identifier.Uuid.Global.generate().slice(0, 13);
+               }
+
+               let selMode = values.selMode;
                delete values.selMode;
 
                if (selMode === 'all') {
@@ -213,87 +260,138 @@ Ext.define('PVE.dc.BackupEdit', {
                    values.all = 1;
                    values.exclude = values.vmid;
                    delete values.vmid;
+               } else if (selMode === 'pool') {
+                   delete values.vmid;
+               }
+
+               if (selMode !== 'pool') {
+                   delete values.pool;
                }
                return values;
-           }
+           },
        });
 
-       var update_vmid_selection = function(list, mode) {
-           if (insideUpdate) {
-               return; // should not happen - just to be sure
-           }
-           insideUpdate = true;
-           if (mode !== 'all') {
+       let update_vmid_selection = function(list, mode) {
+           if (mode !== 'all' && mode !== 'pool') {
                sm.deselectAll(true);
                if (list) {
                    Ext.Array.each(list.split(','), function(vmid) {
-                       var rec = store.findRecord('vmid', vmid);
+                       var rec = store.findRecord('vmid', vmid, 0, false, true, true);
                        if (rec) {
                            sm.select(rec, true);
                        }
                    });
                }
            }
-           insideUpdate = false;
        };
 
        vmidField.on('change', function(f, value) {
-           var mode = selModeField.getValue();
+           let mode = selModeField.getValue();
            update_vmid_selection(value, mode);
        });
 
        selModeField.on('change', function(f, value, oldValue) {
+           if (oldValue === 'pool') {
+               store.removeFilter('poolFilter');
+           }
+
+           if (oldValue === 'all') {
+               sm.deselectAll(true);
+               vmidField.setValue('');
+           }
+
            if (value === 'all') {
                sm.selectAll(true);
                vmgrid.setDisabled(true);
            } else {
                vmgrid.setDisabled(false);
            }
-           if (oldValue === 'all') {
-               sm.deselectAll(true);
+
+           if (value === 'pool') {
+               vmgrid.setDisabled(true);
                vmidField.setValue('');
+               selPool.setVisible(true);
+               selPool.allowBlank = false;
+               selectPoolMembers(selPool.value);
+           } else {
+               selPool.setVisible(false);
+               selPool.allowBlank = true;
            }
-           var list = vmidField.getValue();
+           let list = vmidField.getValue();
            update_vmid_selection(list, value);
        });
-                
-       var reload = function() {
+
+       let reload = function() {
            store.load({
-               params: { type: 'vm' },
+               params: {
+                   type: 'vm',
+               },
                callback: function() {
-                   var node = nodesel.getValue();
+                   let node = nodesel.getValue();
                    store.clearFilter();
-                   store.filterBy(function(rec) {
-                       return (!node || rec.get('node') === node);
-                   });
-                   var list = vmidField.getValue();
-                   var mode = selModeField.getValue();
+                   store.filterBy(rec => !node || node.length === 0 || rec.get('node') === node);
+                   let list = vmidField.getValue();
+                   let mode = selModeField.getValue();
                    if (mode === 'all') {
                        sm.selectAll(true);
+                   } else if (mode === 'pool') {
+                       selectPoolMembers(selPool.value);
                    } else {
                        update_vmid_selection(list, mode);
                    }
-               }
+               },
            });
        };
 
-        Ext.applyIf(me, {
-            subject: gettext("Backup Job"),
-            url: url,
-            method: method,
-           items: [ ipanel, vmgrid ]
-        });
+       Ext.applyIf(me, {
+           subject: gettext("Backup Job"),
+           url: url,
+           method: method,
+           bodyPadding: 0,
+           items: [
+               {
+                   xtype: 'tabpanel',
+                   region: 'center',
+                   layout: 'fit',
+                   bodyPadding: 10,
+                   items: [
+                       {
+                           xtype: 'container',
+                           title: gettext('General'),
+                           region: 'center',
+                           layout: {
+                               type: 'vbox',
+                               align: 'stretch',
+                           },
+                           items: [
+                               ipanel,
+                               vmgrid,
+                           ],
+                       },
+                       {
+                           xtype: 'pveBackupJobPrunePanel',
+                           title: gettext('Retention'),
+                           isCreate: me.isCreate,
+                           keepAllDefaultForCreate: false,
+                           showPBSHint: false,
+                           fallbackHintHtml: gettext('Without any keep option, the storage\'s configuration or node\'s vzdump.conf is used as fallback'),
+                       },
+                   ],
+               },
+           ],
+
+       });
 
-        me.callParent();
+       me.callParent();
 
-        if (me.create) {
+       if (me.isCreate) {
            selModeField.setValue('include');
        } else {
             me.load({
                success: function(response, options) {
-                   var data = response.result.data;
+                   let data = response.result.data;
 
-                   data.dow = data.dow.split(',');
+                   data.dow = (data.dow || '').split(',');
 
                    if (data.all || data.exclude) {
                        if (data.exclude) {
@@ -303,106 +401,311 @@ Ext.define('PVE.dc.BackupEdit', {
                            data.vmid = '';
                            data.selMode = 'all';
                        }
+                   } else if (data.pool) {
+                       data.selMode = 'pool';
+                       data.selPool = data.pool;
                    } else {
                        data.selMode = 'include';
                    }
 
+                   if (data['prune-backups']) {
+                       Object.assign(data, data['prune-backups']);
+                       delete data['prune-backups'];
+                   } else if (data.maxfiles !== undefined) {
+                       if (data.maxfiles > 0) {
+                           data['keep-last'] = data.maxfiles;
+                       } else {
+                           data['keep-all'] = 1;
+                       }
+                       delete data.maxfiles;
+                   }
+
                    me.setValues(data);
-               }
-            });
-        }
+               },
+           });
+       }
 
        reload();
-    }
+    },
 });
 
-
 Ext.define('PVE.dc.BackupView', {
     extend: 'Ext.grid.GridPanel',
 
     alias: ['widget.pveDcBackupView'],
 
+    onlineHelp: 'chapter_vzdump',
+
     allText: '-- ' + gettext('All') + ' --',
-    allExceptText: gettext('All except {0}'),
 
-    initComponent : function() {
-       var me = this;
+    initComponent: function() {
+       let me = this;
 
-       var store = new Ext.data.Store({
+       let store = new Ext.data.Store({
            model: 'pve-cluster-backup',
            proxy: {
-                type: 'pve',
-               url: "/api2/json/cluster/backup"
-           }
+               type: 'proxmox',
+               url: "/api2/json/cluster/backup",
+           },
        });
 
-       var reload = function() {
+       let not_backed_store = new Ext.data.Store({
+           sorters: 'vmid',
+           proxy: {
+               type: 'proxmox',
+               url: 'api2/json/cluster/backup-info/not-backed-up',
+           },
+       });
+
+       let noBackupJobWarning, noBackupJobInfoButton;
+       let reload = function() {
            store.load();
+           not_backed_store.load({
+               callback: function(records, operation, success) {
+                   noBackupJobWarning.setVisible(records.length > 0);
+                   noBackupJobInfoButton.setVisible(records.length > 0);
+               },
+           });
        };
 
-       var sm = Ext.create('Ext.selection.RowModel', {});
+       let sm = Ext.create('Ext.selection.RowModel', {});
 
-       var run_editor = function() {
-           var rec = sm.getSelection()[0];
+       let run_editor = function() {
+           let rec = sm.getSelection()[0];
            if (!rec) {
                return;
            }
 
-            var win = Ext.create('PVE.dc.BackupEdit',{
-                jobid: rec.data.id
-            });
-            win.on('destroy', reload);
-            win.show();
+           let win = Ext.create('PVE.dc.BackupEdit', {
+               jobid: rec.data.id,
+           });
+           win.on('destroy', reload);
+           win.show();
+       };
+
+       let run_detail = function() {
+           let record = sm.getSelection()[0];
+           if (!record) {
+               return;
+           }
+           Ext.create('Ext.window.Window', {
+               modal: true,
+               width: 800,
+               height: 600,
+               stateful: true,
+               stateId: 'backup-detail-view',
+               resizable: true,
+               layout: 'fit',
+               title: gettext('Backup Details'),
+               items: [
+                   {
+                       xtype: 'panel',
+                       region: 'center',
+                       layout: {
+                           type: 'vbox',
+                           align: 'stretch',
+                       },
+                       items: [
+                           {
+                               xtype: 'pveBackupInfo',
+                               flex: 0,
+                               layout: 'fit',
+                               record: record.data,
+                           },
+                           {
+                               xtype: 'pveBackupDiskTree',
+                               title: gettext('Included disks'),
+                               flex: 1,
+                               jobid: record.data.id,
+                           },
+                       ],
+                   },
+               ],
+           }).show();
+       };
+
+       let run_backup_now = function(job) {
+           job = Ext.clone(job);
+
+           let jobNode = job.node;
+           // Remove properties related to scheduling
+           delete job.enabled;
+           delete job.starttime;
+           delete job.dow;
+           delete job.id;
+           delete job.schedule;
+           delete job.type;
+           delete job.node;
+           job.all = job.all === true ? 1 : 0;
+
+           if (job['prune-backups']) {
+               job['prune-backups'] = PVE.Parser.printPropertyString(job['prune-backups']);
+           }
+
+           let allNodes = PVE.data.ResourceStore.getNodes();
+           let nodes = allNodes.filter(node => node.status === 'online').map(node => node.node);
+           let errors = [];
+
+           if (jobNode !== undefined) {
+               if (!nodes.includes(jobNode)) {
+                   Ext.Msg.alert('Error', "Node '"+ jobNode +"' from backup job isn't online!");
+                   return;
+               }
+               nodes = [jobNode];
+           } else {
+               let unkownNodes = allNodes.filter(node => node.status !== 'online');
+               if (unkownNodes.length > 0) {errors.push(unkownNodes.map(node => node.node + ": " + gettext("Node is offline")));}
+           }
+           let jobTotalCount = nodes.length, jobsStarted = 0;
+
+           Ext.Msg.show({
+               title: gettext('Please wait...'),
+               closable: false,
+               progress: true,
+               progressText: '0/' + jobTotalCount,
+           });
+
+           let postRequest = function() {
+               jobsStarted++;
+               Ext.Msg.updateProgress(jobsStarted / jobTotalCount, jobsStarted + '/' + jobTotalCount);
+
+               if (jobsStarted === jobTotalCount) {
+                   Ext.Msg.hide();
+                   if (errors.length > 0) {
+                       Ext.Msg.alert('Error', 'Some errors have been encountered:<br />' + errors.join('<br />'));
+                   }
+               }
+           };
+
+           nodes.forEach(node => Proxmox.Utils.API2Request({
+               url: '/nodes/' + node + '/vzdump',
+               method: 'POST',
+               params: job,
+               failure: function(response, opts) {
+                   errors.push(node + ': ' + response.htmlStatus);
+                   postRequest();
+               },
+               success: postRequest,
+           }));
+       };
+
+       let run_show_not_backed = function() {
+           Ext.create('Ext.window.Window', {
+               modal: true,
+               width: 600,
+               height: 500,
+               resizable: true,
+               layout: 'fit',
+               title: gettext('Guests without backup job'),
+               items: [
+                   {
+                       xtype: 'panel',
+                       region: 'center',
+                       layout: {
+                           type: 'vbox',
+                           align: 'stretch',
+                       },
+                       items: [
+                           {
+                               xtype: 'pveBackedGuests',
+                               flex: 1,
+                               layout: 'fit',
+                               store: not_backed_store,
+                           },
+                       ],
+                   },
+               ],
+           }).show();
        };
 
-       var edit_btn = new PVE.button.Button({
+       var edit_btn = new Proxmox.button.Button({
            text: gettext('Edit'),
            disabled: true,
            selModel: sm,
-           handler: run_editor
+           handler: run_editor,
        });
 
-       var remove_btn = new PVE.button.Button({
-           text: gettext('Remove'),
+       var run_btn = new Proxmox.button.Button({
+           text: gettext('Run now'),
            disabled: true,
            selModel: sm,
-           confirmMsg: gettext('Are you sure you want to remove this entry'),
-           handler: function(btn, event, rec) {
-               PVE.Utils.API2Request({
-                   url: '/cluster/backup/' + rec.data.id,
-                   method: 'DELETE',
-                   waitMsgTarget: me,
-                   callback: function() {
-                       reload();
+           handler: function() {
+               var rec = sm.getSelection()[0];
+               if (!rec) {
+                   return;
+               }
+
+               Ext.Msg.show({
+                   title: gettext('Confirm'),
+                   icon: Ext.Msg.QUESTION,
+                   msg: gettext('Start the selected backup job now?'),
+                   buttons: Ext.Msg.YESNO,
+                   callback: function(btn) {
+                       if (btn !== 'yes') {
+                           return;
+                       }
+                       run_backup_now(rec.data);
                    },
-                   failure: function (response, opts) {
-                       Ext.Msg.alert(gettext('Error'), response.htmlStatus);
-                   }
                });
-           }
+           },
+       });
+
+       var remove_btn = Ext.create('Proxmox.button.StdRemoveButton', {
+           selModel: sm,
+           baseurl: '/cluster/backup',
+           callback: function() {
+               reload();
+           },
        });
 
-       PVE.Utils.monStoreErrors(me, store);
+       var detail_btn = new Proxmox.button.Button({
+           text: gettext('Job Detail'),
+           disabled: true,
+           tooltip: gettext('Show job details and which guests and volumes are affected by the backup job'),
+           selModel: sm,
+           handler: run_detail,
+       });
+
+       noBackupJobWarning = Ext.create('Ext.toolbar.TextItem', {
+           html: '<i class="fa fa-fw fa-exclamation-circle"></i>' + gettext('Some guests are not covered by any backup job.'),
+           hidden: true,
+       });
+
+       noBackupJobInfoButton = new Proxmox.button.Button({
+           text: gettext('Show'),
+           hidden: true,
+           handler: run_show_not_backed,
+       });
+
+       Proxmox.Utils.monStoreErrors(me, store);
 
        Ext.apply(me, {
            store: store,
            selModel: sm,
-           stateful: false,
+           stateful: true,
+           stateId: 'grid-dc-backup',
            viewConfig: {
-               trackOver: false
+               trackOver: false,
            },
            tbar: [
                {
                    text: gettext('Add'),
                    handler: function() {
-                       var win = Ext.create('PVE.dc.BackupEdit',{});
+                       var win = Ext.create('PVE.dc.BackupEdit', {});
                        win.on('destroy', reload);
                        win.show();
-                   }
+                   },
                },
+               '-',
                remove_btn,
-               edit_btn
-           ],          
+               edit_btn,
+               detail_btn,
+               '-',
+               run_btn,
+               '->',
+               noBackupJobWarning,
+               noBackupJobInfoButton,
+           ],
            columns: [
                {
                    header: gettext('Enabled'),
@@ -412,7 +715,12 @@ Ext.define('PVE.dc.BackupView', {
                    sortable: true,
                    disabled: true,
                    disabledCls: 'x-item-enabled',
-                   stopSelection: false
+                   stopSelection: false,
+               },
+               {
+                   header: gettext('ID'),
+                   dataIndex: 'id',
+                   hidden: true,
                },
                {
                    header: gettext('Node'),
@@ -424,108 +732,58 @@ Ext.define('PVE.dc.BackupView', {
                            return value;
                        }
                        return me.allText;
-                   }
-               },
-               {
-                   header: gettext('Day of week'),
-                   width: 200,
-                   sortable: false,
-                   dataIndex: 'dow',
-                   renderer: function(val) {
-                       var dows = ['sun', 'mon', 'tue', 'wed', 'thu', 'fri', 'sat'];
-                       var selected = [];
-                       var cur = -1;
-                       val.split(',').forEach(function(day){
-                           cur++;
-                           dow = (dows.indexOf(day)+6)%7;
-                           if (cur === dow) {
-                               if (selected.length === 0 || selected[selected.length-1] === 0) {
-                                   selected.push(1);
-                               } else {
-                                   selected[selected.length-1]++;
-                               }
-                           } else {
-                               while (cur < dow) {
-                                   cur++;
-                                   selected.push(0);
-                               }
-                               selected.push(1);
-                           }
-                       });
-
-                       cur = -1;
-                       var days = [];
-                       selected.forEach(function(item) {
-                           cur++;
-                           if (item > 2) {
-                               days.push(Ext.Date.dayNames[(cur+1)] + '-' + Ext.Date.dayNames[(cur+item)%7]);
-                               cur += item-1;
-                           } else if (item == 2) {
-                               days.push(Ext.Date.dayNames[cur+1]);
-                               days.push(Ext.Date.dayNames[(cur+2)%7]);
-                               cur++;
-                           } else if (item == 1) {
-                               days.push(Ext.Date.dayNames[(cur+1)%7]);
-                           }
-                       });
-                       return days.join(', ');
-                   }
+                   },
                },
                {
-                   header: gettext('Start Time'),
-                   width: 60,
-                   sortable: true,
-                   dataIndex: 'starttime'
+                   header: gettext('Schedule'),
+                   width: 150,
+                   dataIndex: 'schedule',
                },
                {
                    header: gettext('Storage'),
                    width: 100,
                    sortable: true,
-                   dataIndex: 'storage'
+                   dataIndex: 'storage',
                },
                {
-                   header: gettext('Selection'),
+                   header: gettext('Comment'),
+                   dataIndex: 'comment',
+                   renderer: Ext.htmlEncode,
                    flex: 1,
+               },
+               {
+                   header: gettext('Selection'),
+                   flex: 2,
                    sortable: false,
                    dataIndex: 'vmid',
-                   renderer: function(value, metaData, record) {
-                       /*jslint confusion: true */
-                       if (record.data.all) {
-                           if (record.data.exclude) {
-                               return Ext.String.format(me.allExceptText, record.data.exclude);
-                           }
-                           return me.allText;
-                       }
-                       if (record.data.vmid) {
-                           return record.data.vmid;
-                       }
-
-                       return "-";
-                   }
-               }
+                   renderer: PVE.Utils.render_backup_selection,
+               },
            ],
            listeners: {
                activate: reload,
-               itemdblclick: run_editor
-           }
+               itemdblclick: run_editor,
+           },
        });
-       
+
        me.callParent();
-    }
+    },
 }, function() {
-
     Ext.define('pve-cluster-backup', {
        extend: 'Ext.data.Model',
-       fields: [ 
-           'id', 'starttime', 'dow',
-           'storage', 'node', 'vmid', 'exclude',
+       fields: [
+           'id',
+           'compress',
+           'dow',
+           'exclude',
            'mailto',
+           'mode',
+           'node',
+           'pool',
+           'starttime',
+           'storage',
+           'vmid',
            { name: 'enabled', type: 'boolean' },
            { name: 'all', type: 'boolean' },
-           { name: 'snapshot', type: 'boolean' },
-           { name: 'stop', type: 'boolean' },
-           { name: 'suspend', type: 'boolean' },
-           { name: 'compress', type: 'boolean' }
-       ]
+       ],
     });
 });