Skip to content

Improve logging to kusto for better debugging#77

Merged
frank-pang-msft merged 4 commits intofeature/v2/bootstrapVMWatchfrom
dev/frankpang/improveLogging
May 21, 2024
Merged

Improve logging to kusto for better debugging#77
frank-pang-msft merged 4 commits intofeature/v2/bootstrapVMWatchfrom
dev/frankpang/improveLogging

Conversation

@frank-pang-msft
Copy link
Collaborator

@frank-pang-msft frank-pang-msft commented May 21, 2024

Some info is missing in the kusto logs that are present in the local logs that makes it difficult to debug.

  • Log specific command being executed at startup (install/enable/update/etc)
  • Include extension sequence number and pid at startup for debugging from GuestAgent logs when extension logs are missing or seqNum.status file is missing
  • Log overall status file so we have better debugging when VMExtensionProvisioning fails. This status is only sent when extension transitions between Transitioning -> Success/Error or whenever extension starts up.
  • Update azure-extension-platform package to pull in change to increase precision of event timestamp to include milliseconds/nanoseconds, Previously it was RFC3339, which is in format yyyy-mm-ddThh:mm:ssZ, which causes issue in sorting timestamps. Use RFC3339Nano for better timestamp precision in extension events azure-extension-platform#34

@frank-pang-msft frank-pang-msft force-pushed the dev/frankpang/improveLogging branch from b1655e8 to 6cccdb9 Compare May 21, 2024 18:22
@frank-pang-msft frank-pang-msft marked this pull request as ready for review May 21, 2024 20:22
Copy link
Collaborator

@klugorosado klugorosado left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@frank-pang-msft frank-pang-msft merged commit aefdba0 into feature/v2/bootstrapVMWatch May 21, 2024
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.

3 participants