Skip to content

Add execution level to telemetry summary and logs#6195

Merged
JohnMcPMS merged 1 commit intomicrosoft:masterfrom
JohnMcPMS:user-context-telemetry
May 7, 2026
Merged

Add execution level to telemetry summary and logs#6195
JohnMcPMS merged 1 commit intomicrosoft:masterfrom
JohnMcPMS:user-context-telemetry

Conversation

@JohnMcPMS
Copy link
Copy Markdown
Member

@JohnMcPMS JohnMcPMS commented May 1, 2026

Change

Adds the execution level (user/admin/system) to the summary event and startup log lines.

Validation

Manually confirmed presence in user/admin logs.

Microsoft Reviewers: Open in CodeFlow

@JohnMcPMS JohnMcPMS requested a review from a team as a code owner May 1, 2026 20:59
@JohnMcPMS
Copy link
Copy Markdown
Member Author

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines will not run the associated pipelines, because the pull request was updated after the run command was issued. Review the pull request again and issue a new run command.


// LogStartup
bool IsCOMCall = false;
ExecutionLevel ExecutionLevel = ExecutionLevel::User;
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

comment says LogStartup but it's only logged to summary?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

But it is initialized and logged in the LogStartup function. I didn't want to update a critical event for multiple reasons.

@JohnMcPMS
Copy link
Copy Markdown
Member Author

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@JohnMcPMS JohnMcPMS merged commit bd40bc0 into microsoft:master May 7, 2026
9 checks passed
@JohnMcPMS JohnMcPMS deleted the user-context-telemetry branch May 7, 2026 20:18
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