Skip to content

telemetry: add stream_image_success and clean_install_finalized completion metrics - #783

Closed
bfjelds (bfjelds) wants to merge 1 commit into
user/bfjelds/mjolnir/testimages-telemetry-optinfrom
user/bfjelds/mjolnir/streaming-completion-telemetry
Closed

telemetry: add stream_image_success and clean_install_finalized completion metrics#783
bfjelds (bfjelds) wants to merge 1 commit into
user/bfjelds/mjolnir/testimages-telemetry-optinfrom
user/bfjelds/mjolnir/streaming-completion-telemetry

Conversation

@bfjelds

@bfjelds bfjelds (bfjelds) commented Sep 8, 2026

Copy link
Copy Markdown
Member

@bfjelds
bfjelds (bfjelds) requested a review from a team as a code owner September 8, 2026 21:43
@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines:
1 pipeline(s) were filtered out due to trigger conditions.
There may be pipelines that require an authorized user to comment /azp run to run.

stream_image_start fires with no matching completion signal. Added
stream_image_success, fired in Trident::stream_image() once the
underlying install() call returns Ok, mirroring the existing _start
metric.

(clean_install_finalized was considered for the streaming clean-install
completion gap too, but dropped -- the clean_install/finalize_clean_install
functions are already #[tracing::instrument]-ed spans, which already
give start/end + duration for free; an additional boolean metric would
just be redundant with that.)
@bfjelds
bfjelds (bfjelds) force-pushed the user/bfjelds/mjolnir/streaming-completion-telemetry branch from 26f5eb2 to 9196167 Compare September 8, 2026 21:53
@bfjelds

Copy link
Copy Markdown
Member Author

Folded into #778 (stream_image_success is now part of that commit and merged forward through 779/781). Closing since this branch is now redundant.

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.

1 participant