Skip to content

push ZoomReports To GA#14146

Merged
TwistedAlex merged 6 commits intoAzure:masterfrom
TwistedAlex:alexli/zommGA
Apr 28, 2026
Merged

push ZoomReports To GA#14146
TwistedAlex merged 6 commits intoAzure:masterfrom
TwistedAlex:alexli/zommGA

Conversation

@TwistedAlex
Copy link
Copy Markdown
Contributor

Required items, please complete

Change(s):

  • Updated parser, remove preview labels

Reason for Change(s):

  • GA

Version Updated:
Yes

Testing Completed:
Yes

Checked that the validations are passing and have addressed any issues that are present:

@TwistedAlex TwistedAlex requested review from a team as code owners April 27, 2026 21:54
@TwistedAlex
Copy link
Copy Markdown
Contributor Author

@microsoft-github-policy-service agree company="Microsoft"

Comment thread Solutions/ZoomReports/Parsers/Zoom.yaml Outdated
EventOriginalTime=column_ifexists('Time', ''),
EventOriginalMessage=column_ifexists('OperationDetail', ''),
EventResult=column_ifexists('ActivityType', ''),
UserType=column_ifexists('ActivityType', ''),
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.

Should this be checking if UserType exists?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

They share the same data source, just an additional column whose name is derived from the api response schema, however, since the EventResult and UserType in the original parser already covered the use case of the data source, to simplifying the parser, I've removed ActivityType from the parser, we will just reuse existing ones to simplify it and introduce no breaking changes.

Comment thread Solutions/ZoomReports/Parsers/Zoom.yaml Outdated
Date=column_ifexists('ReportDate', ''),
EventDay=column_ifexists('ReportDate', ''),
NewUsersCount=tostring(column_ifexists('NewUsers', '')),
MeetingsCount=tostring(column_ifexists('Meetings', '')),
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.

Should this be tostring?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

same

Comment thread Solutions/ZoomReports/Parsers/Zoom.yaml Outdated
| extend
Date=column_ifexists('ReportDate', ''),
EventDay=column_ifexists('ReportDate', ''),
NewUsersCount=tostring(column_ifexists('NewUsers', '')),
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.

should this be tostring?

Copy link
Copy Markdown
Contributor Author

@TwistedAlex TwistedAlex Apr 28, 2026

Choose a reason for hiding this comment

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

Thank you for the review Jason, I just tested, it works as well after I remove tostring.

Looking at the original parser, NewUsersCount, MeetingsCount, ParticipantsCount use column_ifexists with a string default '', so I added tostring here, just tested without it, also works fine.
I tested the parser in 3 different scenarios: 1) only ccf, 2) only azure function, and 3) both exists in the same workspace(alex-prd-rg/providers/Microsoft.OperationalInsights/workspaces/alexZoomCCFGAws/logs), this parser works as expected.

@TwistedAlex TwistedAlex merged commit 11a71be into Azure:master Apr 28, 2026
33 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants