[pve-devel] [PATCH manager v2] ui: dc/Cluster: show errors on 'join' info in the view
Dominik Csapak
d.csapak at proxmox.com
Fri Oct 29 11:14:08 CEST 2021
so instead of getting the 'standalone node' message, the grid
is masked with the actual api error
Signed-off-by: Dominik Csapak <d.csapak at proxmox.com>
---
changes from v1:
* only show error if it does not match the 'standalone' error
not sure if matching the *exact* error message is very robust, but AFAICS,
this did not change for a while now, and even if we'd change it, it
would be very apparent, as we could not create a cluster in the gui
anymore.
www/manager6/dc/Cluster.js | 8 +++++++-
1 file changed, 7 insertions(+), 1 deletion(-)
diff --git a/www/manager6/dc/Cluster.js b/www/manager6/dc/Cluster.js
index 12fd9468..10834b66 100644
--- a/www/manager6/dc/Cluster.js
+++ b/www/manager6/dc/Cluster.js
@@ -62,9 +62,15 @@ Ext.define('PVE.ClusterAdministration', {
view.on('destroy', view.store.stopUpdate);
},
- onLoad: function(store, records, success) {
+ onLoad: function(store, records, success, operation) {
let vm = this.getViewModel();
if (!success || !records || !records[0].data) {
+ let error = operation.getError();
+ let msg = Proxmox.Utils.getResponseErrorMessage(error);
+ if (msg !== 'node is not in a cluster, no join info available! (500)') {
+ // show the real message
+ Proxmox.Utils.setErrorMask(this.getView(), msg);
+ }
vm.set('totem', {});
vm.set('isInCluster', false);
vm.set('nodelist', []);
--
2.30.2
More information about the pve-devel
mailing list