all lists on lists.proxmox.com
 help / color / mirror / Atom feed
* [pve-devel] [PATCH widget-toolkit] node/APTRepositories: make 'no pve repositories' an error
@ 2021-07-05 14:06 Dominik Csapak
  2021-07-05 14:41 ` [pve-devel] applied: " Thomas Lamprecht
  0 siblings, 1 reply; 2+ messages in thread
From: Dominik Csapak @ 2021-07-05 14:06 UTC (permalink / raw)
  To: pve-devel

since we also show it as error in the node summary

Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>
---
 src/node/APTRepositories.js | 7 ++++++-
 1 file changed, 6 insertions(+), 1 deletion(-)

diff --git a/src/node/APTRepositories.js b/src/node/APTRepositories.js
index e8f807d..e30396b 100644
--- a/src/node/APTRepositories.js
+++ b/src/node/APTRepositories.js
@@ -462,7 +462,12 @@ Ext.define('Proxmox.node.APTRepositories', {
 	    let wrongSuites = vm.get('suitesWarning');
 
 	    if (!enterprise && !nosubscription && !test) {
-		addWarn(Ext.String.format(gettext('No {0} repository is enabled, you do not get any updates!'), vm.get('product')));
+		status = 'critical';
+		text = gettext('Error');
+		store.add({
+		    status: 'critical',
+		    message: Ext.String.format(gettext('No {0} repository is enabled, you do not get any updates!'), vm.get('product')),
+		});
 	    } else if (enterprise && !nosubscription && !test && activeSubscription) {
 		addGood(Ext.String.format(gettext('You get supported updates for {0}'), vm.get('product')));
 	    } else if (nosubscription || test) {
-- 
2.30.2





^ permalink raw reply	[flat|nested] 2+ messages in thread

* [pve-devel] applied: [PATCH widget-toolkit] node/APTRepositories: make 'no pve repositories' an error
  2021-07-05 14:06 [pve-devel] [PATCH widget-toolkit] node/APTRepositories: make 'no pve repositories' an error Dominik Csapak
@ 2021-07-05 14:41 ` Thomas Lamprecht
  0 siblings, 0 replies; 2+ messages in thread
From: Thomas Lamprecht @ 2021-07-05 14:41 UTC (permalink / raw)
  To: Proxmox VE development discussion, Dominik Csapak

On 05.07.21 16:06, Dominik Csapak wrote:
> since we also show it as error in the node summary
> 
> Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>
> ---
>  src/node/APTRepositories.js | 7 ++++++-
>  1 file changed, 6 insertions(+), 1 deletion(-)
> 
>

applied, thanks!




^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2021-07-05 14:41 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-07-05 14:06 [pve-devel] [PATCH widget-toolkit] node/APTRepositories: make 'no pve repositories' an error Dominik Csapak
2021-07-05 14:41 ` [pve-devel] applied: " Thomas Lamprecht

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.
Service provided by Proxmox Server Solutions GmbH | Privacy | Legal