Skip to content

feat: tag high-volume telemetry events as LogLevel.info (cont.) - #27450

Open
MarioJGMsoft wants to merge 2 commits into
microsoft:mainfrom
MarioJGMsoft:marioja/eventTagging
Open

feat: tag high-volume telemetry events as LogLevel.info (cont.)#27450
MarioJGMsoft wants to merge 2 commits into
microsoft:mainfrom
MarioJGMsoft:marioja/eventTagging

Conversation

@MarioJGMsoft

Copy link
Copy Markdown
Contributor

Description

This PR is a follow up to #27126; it tags high volume FluidFramework telemetry events as LogLevel.info. The goal is to let clients safely filter out info-tagged events without losing the signal needed to diagnose Fluid issues.

Events were selected by observed telemetry volume and span both generic and performance events. For consumers today, the only observable difference is that tagged events now carry logLevel: "info" on their payload - no filtering or delivery behavior changes were made.

Reviewer Guidance

The review process is outlined on this wiki page.

  • Each taggeed event has a PR comment specifying which event is being marked as info. The authors have been identified via git blame and tagged on the corresponding PR comment - please confirm info is correct for events that you have context on. If it is correct, please resolve the comment to know that I can go forward with the change.
  • If you don't have context on the event, please point me to who could know more about the event.
  • If you see an event in the diff you believe should remain untagged, flag it.
  • With the addition of the LogLevel.info parameter the formatting for various pieces of the code was updated. I highly recommend turning on the hide whitespace changes option when reviewing this PR.

Fixes: AB#52417

@github-actions

github-actions Bot commented Jun 1, 2026

Copy link
Copy Markdown
Contributor

Hi! Thank you for opening this PR. Want me to review it?

Based on the diff (606 lines, 14 files), I've queued these reviewers:

  • Correctness — logic errors, race conditions, lifecycle issues
  • Security — vulnerabilities, secret exposure, injection
  • API Compatibility — breaking changes, release tags, type design
  • Performance — algorithmic regressions, memory leaks
  • Testing — coverage gaps, hollow tests

How this works

  • Adjust the reviewer set by ticking/unticking boxes above. Reviewer toggles alone don't trigger anything.

  • Tick Start review below to dispatch the review fleet.

  • After review finishes, tick Start review again to request another run — it auto-resets after each dispatch.

  • This comment updates as new commits land; your reviewer selections are preserved.

  • Start review

@MarioJGMsoft MarioJGMsoft changed the title feat: set branch in same state as it used to be feat: tag high-volume telemetry events as LogLevel.info (cont.) Jun 17, 2026
@github-actions

github-actions Bot commented Jun 17, 2026

Copy link
Copy Markdown
Contributor

Bundle size comparison

Base commit: e2fc39bf6ad79006c2c6779bfad563fa42ec8eec
Head commit: 76e57e0ccdd77da3d4d6f00a6b371328b180ab99

Pending — Build - client packages is running. Results will appear here when the build completes.

@MarioJGMsoft
MarioJGMsoft marked this pull request as ready for review July 15, 2026 23:05
Copilot AI review requested due to automatic review settings July 15, 2026 23:05

Copilot AI left a comment

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.

Copilot reviewed 14 out of 14 changed files in this pull request and generated no comments.

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