Skip to content

pacemaker: fix runtime validation from present state - #12538

Draft
munchtoast wants to merge 3 commits into
ansible-collections:mainfrom
munchtoast:fix_pacemaker_runtime_validation
Draft

pacemaker: fix runtime validation from present state#12538
munchtoast wants to merge 3 commits into
ansible-collections:mainfrom
munchtoast:fix_pacemaker_runtime_validation

Conversation

@munchtoast

@munchtoast munchtoast commented Aug 7, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor
SUMMARY

Fixes #11574

ISSUE TYPE
  • Bugfix Pull Request
COMPONENT NAME

pacemaker_stonith
pacemaker_resource

ADDITIONAL INFORMATION

@munchtoast
pacemaker_resource: split configuration vs runtime state semantics
bb5ccee 
state=present and state=cloned now only mutate the CIB and return;
they no longer poll pcs resource status after the change. Runtime
state assertion moves to state=enabled (polls Started/Promoted/
Unpromoted) and state=disabled (polls Stopped). Setting wait on a
configuration state emits a warning. wait default changes from 300
to unset; enabled/disabled fall back to 300s internally when wait
is unset.

Adds _STOPPED_READY_STATES to _pacemaker and generalises the wait
timeout message to cover both directions.
@munchtoast
pacemaker_stonith: split configuration vs runtime state semantics
8f5ef37 
state=present no longer polls pcs stonith status after creating the
device. Runtime state assertion moves to state=enabled (polls
Started) and state=disabled (polls Stopped). Setting wait on
state=present or state=absent emits a warning. wait default changes
from 300 to unset; enabled/disabled fall back to 300s internally
when wait is unset.
@ansibullbot ansibullbot added WIP Work in progress bug This issue/PR relates to a bug module module module_utils module_utils plugins plugin (any type) tests tests unit tests/unit labels Aug 7, 2026
@felixfontein felixfontein added check-before-release PR will be looked at again shortly before release and merged if possible. backport-12 Automatically create a backport for the stable-12 branch backport-13 Automatically create a backport for the stable-13 branch breaking_change This PR contains a breaking change that MUST NOT be backported and removed backport-12 Automatically create a backport for the stable-12 branch backport-13 Automatically create a backport for the stable-13 branch labels Aug 8, 2026
@ansibullbot ansibullbot added the stale_ci CI is older than 7 days, rerun before merging label Aug 16, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

breaking_change This PR contains a breaking change that MUST NOT be backported bug This issue/PR relates to a bug check-before-release PR will be looked at again shortly before release and merged if possible. module_utils module_utils module module plugins plugin (any type) stale_ci CI is older than 7 days, rerun before merging tests tests unit tests/unit WIP Work in progress

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

pacemaker_stonith: Resource polling timeout

3 participants

@munchtoast @felixfontein @ansibullbot