perf(hooks): reduce service reconciliation overhead#205
Open
gboutry wants to merge 2 commits into
Open
Conversation
Avoid snapctl service tasks when the requested enabled and active state is already satisfied. Keep corrective operations for every partially converged state. Treat file-transfer as unavailable until its TLS inputs exist so faster reconciliation does not expose a start-limit timing dependency. Assisted-By: Codex (gpt-5-6) Signed-off-by: Guillaume Boutry <guillaume.boutry@canonical.com>
Combine service operations within each existing configure-hook phase to reduce snapctl round trips. Preserve pre-command state filtering; snapd applies the queued service operations after the hook exits. Assisted-By: Codex (gpt-5-6) Signed-off-by: Guillaume Boutry <guillaume.boutry@canonical.com>
gboutry
force-pushed
the
fix/hook-fastness
branch
from
July 22, 2026 15:10
ccec17f to
eb762ec
Compare
gboutry
requested review from
ahmad-can and
freyes
and removed request for
ahmad-can
July 23, 2026 08:16
gboutry
marked this pull request as ready for review
July 23, 2026 09:41
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Reduce
openstack-hypervisorconfigure-hook runtime by avoiding serviceoperations that are already converged and batching the remaining operations.
Assisted-By: Codex (gpt-5-6)
QA steps
The result of this branch is available under
2026.1/edge/fastto test it. This snap should be able to be used in place of the regular2026.1/edgefor any type of operation.Methodology
I've personally run deployments, and could see a significant improvement.
The baseline was commit
759615fand the measured candidate snap was builtfrom
2484ab83. The candidate snap payload is identical to the current branchhead for the runtime code under test; the subsequent history rewrite only
reworded the commits.
Both modes were measured using separate, freshly provisioned Ubuntu LXD
virtual machines. The local snap was installed strictly after a devmode
bootstrap used to connect the required interfaces, and a post-install snapshot
was restored before each first-configuration sample.