public inbox for pve-devel@lists.proxmox.com
 help / color / mirror / Atom feed
* [pve-devel] [PATCH docs 1/2] ha: crs: fix typo
@ 2022-11-21  8:31 Fiona Ebner
  2022-11-21  8:31 ` [pve-devel] [PATCH docs 2/2] ha: crs: clarify that non-HA-managed services are currently not counted Fiona Ebner
  2022-11-21 11:14 ` [pve-devel] applied: [PATCH docs 1/2] ha: crs: fix typo Thomas Lamprecht
  0 siblings, 2 replies; 3+ messages in thread
From: Fiona Ebner @ 2022-11-21  8:31 UTC (permalink / raw)
  To: pve-devel

Signed-off-by: Fiona Ebner <f.ebner@proxmox.com>
---
 ha-manager.adoc | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/ha-manager.adoc b/ha-manager.adoc
index e69db1a..2da6a99 100644
--- a/ha-manager.adoc
+++ b/ha-manager.adoc
@@ -958,14 +958,14 @@ the future.
 Basic
 ~~~~~
 
-The number of active HA serivces on each node is used to choose a recovery node.
+The number of active HA services on each node is used to choose a recovery node.
 
 Static
 ~~~~~~
 
 IMPORTANT: The static mode is still a technology preview.
 
-Static usage information from HA serivces on each node is used to choose a
+Static usage information from HA services on each node is used to choose a
 recovery node.
 
 For this selection, each node in turn is considered as if the service was
-- 
2.30.2





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

* [pve-devel] [PATCH docs 2/2] ha: crs: clarify that non-HA-managed services are currently not counted
  2022-11-21  8:31 [pve-devel] [PATCH docs 1/2] ha: crs: fix typo Fiona Ebner
@ 2022-11-21  8:31 ` Fiona Ebner
  2022-11-21 11:14 ` [pve-devel] applied: [PATCH docs 1/2] ha: crs: fix typo Thomas Lamprecht
  1 sibling, 0 replies; 3+ messages in thread
From: Fiona Ebner @ 2022-11-21  8:31 UTC (permalink / raw)
  To: pve-devel

Without explicitly stating this, it's rather easily missed.

Reported-by: Dominik Csapak <d.csapak@proxmox.com>
Signed-off-by: Fiona Ebner <f.ebner@proxmox.com>
---

As Thomas said off-list, it's better to start tracking this
information in HA manager too. For now, clarify the status-quo.

 ha-manager.adoc | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/ha-manager.adoc b/ha-manager.adoc
index 2da6a99..ccbb7f8 100644
--- a/ha-manager.adoc
+++ b/ha-manager.adoc
@@ -959,6 +959,7 @@ Basic
 ~~~~~
 
 The number of active HA services on each node is used to choose a recovery node.
+Non-HA-managed services are currently not counted.
 
 Static
 ~~~~~~
@@ -966,7 +967,7 @@ Static
 IMPORTANT: The static mode is still a technology preview.
 
 Static usage information from HA services on each node is used to choose a
-recovery node.
+recovery node. Usage of non-HA-managed services is currently not considered.
 
 For this selection, each node in turn is considered as if the service was
 already running on it, using CPU and memory usage from the associated guest
-- 
2.30.2





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

* [pve-devel] applied:  [PATCH docs 1/2] ha: crs: fix typo
  2022-11-21  8:31 [pve-devel] [PATCH docs 1/2] ha: crs: fix typo Fiona Ebner
  2022-11-21  8:31 ` [pve-devel] [PATCH docs 2/2] ha: crs: clarify that non-HA-managed services are currently not counted Fiona Ebner
@ 2022-11-21 11:14 ` Thomas Lamprecht
  1 sibling, 0 replies; 3+ messages in thread
From: Thomas Lamprecht @ 2022-11-21 11:14 UTC (permalink / raw)
  To: Proxmox VE development discussion, Fiona Ebner

Am 21/11/2022 um 09:31 schrieb Fiona Ebner:
> Signed-off-by: Fiona Ebner <f.ebner@proxmox.com>
> ---
>  ha-manager.adoc | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
>

applied both patches, thanks!




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

end of thread, other threads:[~2022-11-21 11:14 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-11-21  8:31 [pve-devel] [PATCH docs 1/2] ha: crs: fix typo Fiona Ebner
2022-11-21  8:31 ` [pve-devel] [PATCH docs 2/2] ha: crs: clarify that non-HA-managed services are currently not counted Fiona Ebner
2022-11-21 11:14 ` [pve-devel] applied: [PATCH docs 1/2] ha: crs: fix typo Thomas Lamprecht

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
Service provided by Proxmox Server Solutions GmbH | Privacy | Legal