Skip to content

Conversation

@peytonr18
Copy link
Contributor

@peytonr18 peytonr18 commented Oct 13, 2025

Despite my earlier PR to enable pre-HTTP telemetry emission, _report still fails to emit to .kvp because it closes immediately after the Wireserver connection is established and the root span exits. By creating a custom span for the _report function, it ensures that health::status events are included, regardless of the main span exiting.

Solution

  • Ensure the kvp_writer has an active span for _report to use outside of the PROVISIONING_REPORT.
    • Achieved by adding a function-level span on _report called health_status.
  • Added message "Provisioning report: {}" to the health_report event in _report() for visibility in azure-init.log

@peytonr18 peytonr18 changed the title Fix: Adding and updating report ready telemetry Fix: Guarantee health::status KVP emission via function-level span Oct 13, 2025
@peytonr18 peytonr18 force-pushed the probertson-libazureinit-kvp2 branch from 125b337 to 0f9187c Compare October 14, 2025 00:03
@peytonr18 peytonr18 force-pushed the probertson-libazureinit-kvp2 branch from 0f9187c to 74edee3 Compare October 14, 2025 00:06
@peytonr18 peytonr18 marked this pull request as ready for review October 14, 2025 17:33
@peytonr18 peytonr18 merged commit 2a90978 into Azure:main Oct 14, 2025
6 checks passed
@peytonr18 peytonr18 deleted the probertson-libazureinit-kvp2 branch October 14, 2025 21:53
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