HomeClusterLabs Projects

Feature: executor: Don't handle start actions in action_complete()
5cb9381f0e38Unpublished

Unpublished Commit ยท Learn More

Not On Permanent Ref: This commit is not an ancestor of any permanent ref.
This commit no longer exists in the repository. It may have been part of a branch which was deleted.This commit has been deleted in the repository: it is no longer reachable from any branch, tag, or ref.

Description

Feature: executor: Don't handle start actions in action_complete()

The action_complete function is called when a systemd action finishes.
Unfortunately, StartUnit actions finish and return a value immediately,
not when the unit we asked it to start is actually up and running.

For this reason, action_complete handles start actions by then
scheduling a follow-up monitor action to poll the service and see if
it's actually up and running yet. It then takes the appropriate action
when this happens.

Since we're moving towards waiting for systemd to tells us a service is
running, we don't need to do any of this anymore. However, stop actions
and monitors still need this code. For the moment, it's easiest to just
bail out for start actions.

Details

Provenance
clumensAuthored on Jan 16 2025, 4:21 PM
nrwahl2Committed on Feb 5 2025, 3:46 PM

Event Timeline

Commit No Longer Exists

This commit no longer exists in the repository.