ui: datastore/DataStoreListSummary: catch and show errors per datastore
so that the update does not get canceled because of a bad datastore hide the irrelevant fields in that case Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>
This commit is contained in:
		
				
					committed by
					
						 Thomas Lamprecht
						Thomas Lamprecht
					
				
			
			
				
	
			
			
			
						parent
						
							3420029b5e
						
					
				
				
					commit
					9594362e35
				
			| @ -21,6 +21,7 @@ Ext.define('PBS.datastore.DataStoreListSummary', { | ||||
| 	    full: "N/A", | ||||
| 	    stillbad: 0, | ||||
| 	    deduplication: 1.0, | ||||
| 	    error: "", | ||||
| 	}, | ||||
|     }, | ||||
|     setTasks: function(taskdata, since) { | ||||
| @ -32,6 +33,13 @@ Ext.define('PBS.datastore.DataStoreListSummary', { | ||||
| 	let me = this; | ||||
| 	let vm = me.getViewModel(); | ||||
|  | ||||
| 	if (statusData.error !== undefined) { | ||||
| 	    vm.set('error', statusData.error); | ||||
| 	    return; | ||||
| 	} else { | ||||
| 	    vm.set('error', ""); | ||||
| 	} | ||||
|  | ||||
| 	let usage = statusData.used/statusData.total; | ||||
| 	let usagetext = Ext.String.format(gettext('{0} of {1}'), | ||||
| 	    Proxmox.Utils.format_size(statusData.used), | ||||
| @ -83,12 +91,34 @@ Ext.define('PBS.datastore.DataStoreListSummary', { | ||||
| 	    }, | ||||
|  | ||||
| 	    items: [ | ||||
| 		{ | ||||
| 		    xtype: 'box', | ||||
| 		    reference: 'errorBox', | ||||
| 		    hidden: true, | ||||
| 		    tpl: [ | ||||
| 			'<center>', | ||||
| 			`<h3>${gettext("Error")}</h3>`, | ||||
| 			'<i class="fa fa-5x fa-exclamation-circle critical"></i>', | ||||
| 			'<br /><br/>', | ||||
| 			'{text}', | ||||
| 			'</center>', | ||||
| 		    ], | ||||
| 		    bind: { | ||||
| 			visible: '{error}', | ||||
| 			data: { | ||||
| 			    text: '{error}', | ||||
| 			}, | ||||
| 		    }, | ||||
| 		}, | ||||
| 		{ | ||||
| 		    xtype: 'proxmoxGauge', | ||||
| 		    warningThreshold: 0.8, | ||||
| 		    criticalThreshold: 0.95, | ||||
| 		    flex: 1, | ||||
| 		    reference: 'usage', | ||||
| 		    bind: { | ||||
| 			visible: '{!error}', | ||||
| 		    }, | ||||
| 		}, | ||||
| 		{ | ||||
| 		    xtype: 'pmxInfoWidget', | ||||
| @ -99,6 +129,7 @@ Ext.define('PBS.datastore.DataStoreListSummary', { | ||||
| 			data: { | ||||
| 			    text: '{full}', | ||||
| 			}, | ||||
| 			visible: '{!error}', | ||||
| 		    }, | ||||
| 		}, | ||||
| 		{ | ||||
| @ -110,6 +141,7 @@ Ext.define('PBS.datastore.DataStoreListSummary', { | ||||
| 			data: { | ||||
| 			    text: '{deduplication}', | ||||
| 			}, | ||||
| 			visible: '{!error}', | ||||
| 		    }, | ||||
| 		}, | ||||
| 		{ | ||||
| @ -143,6 +175,9 @@ Ext.define('PBS.datastore.DataStoreListSummary', { | ||||
| 		    reference: 'historychart', | ||||
| 		    title: gettext('Usage History'), | ||||
| 		    height: 100, | ||||
| 		    bind: { | ||||
| 			visible: '{!error}', | ||||
| 		    }, | ||||
| 		}, | ||||
| 		{ | ||||
| 		    xtype: 'container', | ||||
|  | ||||
		Reference in New Issue
	
	Block a user