Skip to content

[zoom] Add support for the operation data stream#19523

Merged
brijesh-elastic merged 6 commits into
elastic:mainfrom
brijesh-elastic:zoom-operation-datastream
Jul 7, 2026
Merged

[zoom] Add support for the operation data stream#19523
brijesh-elastic merged 6 commits into
elastic:mainfrom
brijesh-elastic:zoom-operation-datastream

Conversation

@brijesh-elastic

@brijesh-elastic brijesh-elastic commented Jun 15, 2026

Copy link
Copy Markdown
Contributor

Proposed commit message

zoom: add support for operation data stream

This data stream collects audit admin and user activity, such as adding a new user,
changing account settings, and deleting recordings.
API documentation is available here[1] and test samples were derived from documentation.

It also updates the default values for the rate limiting and page size parameters in the
`activity` data stream

[1] https://developers.zoom.us/docs/api/meetings/#tag/reports/get/report/operationlogs

Note

To Reviewers:

  • This data stream has not been tested with a live instance.

Checklist

  • I have reviewed tips for building integrations and this pull request is aligned with them.
  • I have verified that all data streams collect metrics or logs.
  • I have added an entry to my package's changelog.yml file.
  • I have verified that Kibana version constraints are current according to guidelines.
  • I have verified that any added dashboard complies with Kibana's Dashboard good practices

How to test this PR locally

  • Clone integrations repo.
  • Install elastic package locally.
  • Start elastic stack using elastic-package.
  • Move to integrations/packages/zoom directory.
  • Run the following command to run tests.

elastic-package test -v

Related issues

@brijesh-elastic brijesh-elastic self-assigned this Jun 15, 2026
@brijesh-elastic brijesh-elastic added documentation Improvements or additions to documentation. Applied to PRs that modify *.md files. enhancement New feature or request Integration:zoom Zoom Team:Security-Service Integrations Security Service Integrations team [elastic/security-service-integrations] Team:SDE-Crest Crest developers on the Security Integrations team [elastic/sit-crest-contractors] labels Jun 15, 2026
@github-actions

Copy link
Copy Markdown
Contributor

Elastic Docs Style Checker (Vale)

Summary: 2 warnings found

⚠️ Warnings (2): Fix when the suggestion improves clarity or correctness.
File Line Rule Message
packages/zoom/_dev/build/docs/README.md 5 Elastic.EndPuntuaction Don't end headings with punctuation.
packages/zoom/manifest.yml 71 Elastic.DontUse Don't use 'Please'.

The Vale linter checks documentation changes against the Elastic Docs style guide. To use Vale locally or report issues, refer to Elastic style guide for Vale.

@elastic-vault-github-plugin-prod

Copy link
Copy Markdown
Contributor

🚀 Benchmarks report

To see the full report comment with /test benchmark fullreport

@brijesh-elastic brijesh-elastic added the dashboard Relates to a Kibana dashboard bug, enhancement, or modification. label Jul 6, 2026
@brijesh-elastic
brijesh-elastic marked this pull request as ready for review July 6, 2026 11:12
@brijesh-elastic
brijesh-elastic requested review from a team as code owners July 6, 2026 11:12
@infra-vault-gh-plugin-prod

Copy link
Copy Markdown

Pinging @elastic/security-service-integrations (Team:Security-Service Integrations)

Comment thread packages/zoom/data_stream/activity/manifest.yml Outdated
@vera-review-bot

Copy link
Copy Markdown

Review summary

Issues found across the latest commits 7175142 — 1 low
  • 🔵 Rate Limit description is a run-on sentence — insert a period after 'on the account' so the two sentences read '...on the account. Leave empty to disable client-side rate limiting.' (link) (Unresolved)

A new commit triggers another review — at most once every 15 minutes. I skip the PR while it's approved or has merge conflicts.

🤖 AI-Generated Review | Vera Review Bot | 📚 Knowledge base: integration-skills

⚠️ Automated review — verify suggestions before applying.

@kcreddy kcreddy 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.

Nit only

Comment thread packages/zoom/data_stream/operation/fields/fields.yml
@elastic-vault-github-plugin-prod

Copy link
Copy Markdown
Contributor

✅ All changelog entries have the correct PR link.

@kcreddy kcreddy 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.

LGTM

@infra-vault-gh-plugin-prod

Copy link
Copy Markdown

💚 Build Succeeded

History

cc @brijesh-elastic

@mergify

mergify Bot commented Jul 6, 2026

Copy link
Copy Markdown
Contributor

Tick the box to add this pull request to the merge queue (same as @mergifyio queue).

  • Queue this pull request

@brijesh-elastic
brijesh-elastic merged commit a93b191 into elastic:main Jul 7, 2026
14 checks passed
@elastic-vault-github-plugin-prod

Copy link
Copy Markdown
Contributor

Package zoom - 1.25.0 containing this change is available at https://epr.elastic.co/package/zoom/1.25.0/

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dashboard Relates to a Kibana dashboard bug, enhancement, or modification. documentation Improvements or additions to documentation. Applied to PRs that modify *.md files. enhancement New feature or request Integration:zoom Zoom Team:SDE-Crest Crest developers on the Security Integrations team [elastic/sit-crest-contractors] Team:Security-Service Integrations Security Service Integrations team [elastic/security-service-integrations]

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants