Skip to content

[RUM-9457] - Automated regression detecting benchmark system#949

Merged
sbarrio merged 1 commit into
developfrom
sbarrio/RUM-9457/automated-regression-detecting-benchmark-system
Jul 30, 2025
Merged

[RUM-9457] - Automated regression detecting benchmark system#949
sbarrio merged 1 commit into
developfrom
sbarrio/RUM-9457/automated-regression-detecting-benchmark-system

Conversation

@sbarrio
Copy link
Copy Markdown
Contributor

@sbarrio sbarrio commented Jul 28, 2025

What does this PR do?

This PR merges the feature branch implementing the Benchmark app and its related CI tooling into develop.

It includes the following PRs:

[RUM 9617] Base Benchmark app with support for Test Scenarios and external config
[RUM 9598] Add Log test scenarios to the Benchmark app
[RUM 9600] Add Traces scenario to Benchmark app
[RUM 9602] Add Manual RUM test scenario to Benchmark app
[RUM 9603] Add RUM Auto test scenario to Benchmark app
[RUM 9660] Add session-replay-scenario to Benchmark app
[RUM 9618] Add CI tools for Base Benchmark app

Motivation

We should have a a set of automated scenarios ran and profiled on our benchmark app with every push to develop so we can monitor the impact and overhead that integrating the SDK into a host app supposes.

Review checklist (to be filled by reviewers)

  • Feature or bugfix MUST have appropriate tests
  • Make sure you discussed the feature or bugfix with the maintaining team in an Issue
  • Make sure each commit and the PR mention the Issue number (cf the CONTRIBUTING doc)
  • If this PR is auto-generated, please make sure also to manually update the code related to the change

@sbarrio sbarrio force-pushed the sbarrio/RUM-9457/automated-regression-detecting-benchmark-system branch from b9988bf to eced028 Compare July 28, 2025 12:44
@sbarrio sbarrio marked this pull request as ready for review July 28, 2025 14:35
@sbarrio sbarrio requested review from a team as code owners July 28, 2025 14:35
@maycmlee maycmlee self-assigned this Jul 28, 2025
@iadjivon
Copy link
Copy Markdown
Contributor

Created a Jira card for review: /DOCS-11646

@maycmlee maycmlee removed their assignment Jul 28, 2025
@maycmlee maycmlee added the editorial review Waiting on a more in-depth review from the docs team label Jul 28, 2025
maycmlee
maycmlee previously approved these changes Jul 28, 2025
Copy link
Copy Markdown
Contributor

@maycmlee maycmlee left a comment

Choose a reason for hiding this comment

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

Just a couple of small suggestions, but approving

Comment thread benchmarks/README.md Outdated
Comment thread benchmarks/README.md Outdated
Comment thread benchmarks/README.md Outdated
Comment thread benchmarks/README.md Outdated
Comment thread benchmarks/README.md Outdated
Comment thread benchmarks/README.md Outdated
@sbarrio sbarrio force-pushed the sbarrio/RUM-9457/automated-regression-detecting-benchmark-system branch from 0ab09a3 to 4d3b578 Compare July 29, 2025 07:53
@sbarrio sbarrio requested a review from maycmlee July 29, 2025 07:53
Copy link
Copy Markdown
Member

@marco-saia-datadog marco-saia-datadog left a comment

Choose a reason for hiding this comment

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

🎉

@sbarrio sbarrio merged commit 98fef81 into develop Jul 30, 2025
10 checks passed
@sbarrio sbarrio deleted the sbarrio/RUM-9457/automated-regression-detecting-benchmark-system branch July 30, 2025 13:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

editorial review Waiting on a more in-depth review from the docs team

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants