Skip to content

pacemaker: write preferred_host as its own constraint - #102

Merged
insatomcat merged 1 commit into
mainfrom
preferred-host-constraint
Sep 29, 2026
Merged

insatomcat merged 1 commit into
mainfrom
preferred-host-constraint

Conversation

@insatomcat

Copy link
Copy Markdown
Member

preferred_host is applied with crm resource move, whose cli-prefer-<vm> constraint cannot be told apart from a manual move and is removed by crm resource clear. This writes seapath-preferred-<vm> with the same infinite score instead.

crm resource move names its constraint cli-prefer-<vm>, which looks
like a manual move and is removed by crm resource clear. Write
seapath-preferred-<vm> with the same infinite score instead.

Signed-off-by: Florent Carli <florent.carli@rte-france.com>
@insatomcat
insatomcat force-pushed the preferred-host-constraint branch from a3f20a8 to 33cc162 Compare September 27, 2026 09:59
@sonarqubecloud

Copy link
Copy Markdown

insatomcat added a commit to insatomcat/seapath-webui that referenced this pull request Sep 27, 2026
vm_manager now writes preferred_host as seapath-preferred-<guest>
(seapath/vm_manager#102) instead of a crm resource move, so the
declared placement and an operator's move are two constraints, and a
clear leaves the first.

The VMs and Cluster pages take the rule in force as the move's
cli-prefer where there is one, and the deployment's rule otherwise.
Return is the clear alone beside that rule, and still writes
preferred_host back on a guest an older vm_manager deployed. D34 is
amended.

Signed-off-by: Florent Carli <florent.carli@rte-france.com>

@eroussy eroussy left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

very good, thanks

@insatomcat
insatomcat merged commit 630917b into main Sep 29, 2026
5 checks passed
@insatomcat
insatomcat deleted the preferred-host-constraint branch September 29, 2026 13:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants