FluidFramework icon indicating copy to clipboard operation
FluidFramework copied to clipboard

Improve DDS stress testing: introduce enum stress mode

Open clarenceli-msft opened this issue 1 year ago • 2 comments

AB#2729, AB#2730

This PR is a prototype for improving the Test - DDS Stress line by using a numeric stressLevel instead of a boolean isStress. This change allows for more granular control over whether to use stress runs and to what degree.

The follow-up tasks include:

  1. Integrating this functionality (potentially as a stage) into the CI build pipeline with a light stress level mode, so it can be triggered in the PR gateway.

  2. Creating a long-running pipeline to execute the DDS stress tests with a heavy stress level mode, scheduled to run periodically during off-peak hours, similar to the test stability pipeline.

Guidelines for Pull Requests.

Reviewer Guidance

The review process is outlined on this wiki page.

List any specific things you want to get reviewer opinions on, and anything a reviewer would need to know to review this PR effectively. Things you might want to include:

  • Questions about how to properly make automated tests for your changes.
  • Questions about design choices you made.
  • Descriptions of how to manually test the changes (and how much of that you have done).
  • etc.

If you have any questions in this section, consider making the PR a draft until all questions have been resolved.

clarenceli-msft avatar Jun 28 '24 20:06 clarenceli-msft

@fluid-example/bundle-size-tests: +245 Bytes
Metric NameBaseline SizeCompare SizeSize Diff
aqueduct.js 460.31 KB 460.34 KB +35 Bytes
azureClient.js 557.29 KB 557.34 KB +49 Bytes
connectionState.js 724 Bytes 724 Bytes No change
containerRuntime.js 259.58 KB 259.6 KB +14 Bytes
fluidFramework.js 405.95 KB 405.96 KB +14 Bytes
loader.js 134.34 KB 134.36 KB +14 Bytes
map.js 42.46 KB 42.46 KB +7 Bytes
matrix.js 148.63 KB 148.64 KB +7 Bytes
odspClient.js 524.25 KB 524.3 KB +49 Bytes
odspDriver.js 97.84 KB 97.86 KB +21 Bytes
odspPrefetchSnapshot.js 42.81 KB 42.82 KB +14 Bytes
sharedString.js 164.82 KB 164.83 KB +7 Bytes
sharedTree.js 396.41 KB 396.41 KB +7 Bytes
Total Size 3.31 MB 3.31 MB +245 Bytes

Baseline commit: f57799c060d45f2356eef26b3a221f72c071c426

Generated by :no_entry_sign: dangerJS against d3a3368b6824385f861a5a49615f55c3b7397dbb

msfluid-bot avatar Jun 28 '24 21:06 msfluid-bot

⚠️ No Changeset found

Latest commit: d7ca1da523b081e0dc8cf6fb17c044ef3d77c51c

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

changeset-bot[bot] avatar Aug 02 '24 15:08 changeset-bot[bot]