From: "Daniel Kral" <d.kral@proxmox.com>
To: "Proxmox VE development discussion" <pve-devel@lists.proxmox.com>
Subject: Re: [pve-devel] applied: [PATCH ha-manager 0/3] fix mixed resource affinity precedence
Date: Fri, 10 Oct 2025 07:48:50 +0200 [thread overview]
Message-ID: <DDEEMS7K434P.2NM4VDZLMPE8P@proxmox.com> (raw)
In-Reply-To: <20250919140856.1361124-1-d.kral@proxmox.com>
On Fri Sep 19, 2025 at 4:08 PM CEST, Daniel Kral wrote:
> Some last-minute changes to the positive resource affinity's heuristic
> (which is described in the next paragraph) in the initial implementation
> broke the mixed usage of resource affinity rules.
>
> Strict positive resource affinity rules narrow down the possible nodes
> to a single candidate for a HA resource A, which is the node, where the
> most of the HA resources in the positive affinity rule are already
> running on and in case of a tie the alphabetically first node is chosen.
>
> If the chosen node contains a HA resource B, which is in negative
> affinity with the HA resource A, then there will be no possible node
> left as it is removed now as well and won't resolve the incorrect node
> placement (see the test cases for more details).
For completeness, these patches are already applied:
6db0df7c test: add additional mixed resource affinity rule test cases
9ff6a5e1 manager: fix precedence in mixed resource affinity rules usage
adae0e3e tests: add regression tests for mixed resource affinity rules
_______________________________________________
pve-devel mailing list
pve-devel@lists.proxmox.com
https://lists.proxmox.com/cgi-bin/mailman/listinfo/pve-devel
prev parent reply other threads:[~2025-10-10 5:49 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2025-09-19 14:08 [pve-devel] " Daniel Kral
2025-09-19 14:08 ` [pve-devel] [PATCH ha-manager 1/3] tests: add regression tests for mixed resource affinity rules Daniel Kral
2025-09-19 14:08 ` [pve-devel] [PATCH ha-manager 2/3] manager: fix precedence in mixed resource affinity rules usage Daniel Kral
2025-09-19 14:08 ` [pve-devel] [PATCH ha-manager 3/3] test: add additional mixed resource affinity rule test cases Daniel Kral
2025-10-10 5:48 ` Daniel Kral [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=DDEEMS7K434P.2NM4VDZLMPE8P@proxmox.com \
--to=d.kral@proxmox.com \
--cc=pve-devel@lists.proxmox.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
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.