Skip to content

Conversation

@akshaydeo
Copy link
Contributor

@akshaydeo akshaydeo commented Dec 10, 2025

Summary

Added a load testing script for Bifrost HTTP transport with accompanying results files to measure performance overhead.

Changes

  • Added load-test.sh script that builds bifrost-http, sets up a mocker provider, and runs load tests
  • Created results output files in both JSON and Markdown formats
  • Script runs a 3000 RPS load test for 60 seconds against a mocker with 10-second base latency
  • Updated .gitignore to exclude bifrost-benchmarking directory

Type of change

  • Bug fix
  • Feature
  • Refactor
  • Documentation
  • Chore/CI

Affected areas

  • Core (Go)
  • Transports (HTTP)
  • Providers/Integrations
  • Plugins
  • UI (Next.js)
  • Docs

How to test

Run the load test script to validate Bifrost's performance:

# Make the script executable if needed
chmod +x .github/workflows/scripts/load-test.sh

# Run the load test
.github/workflows/scripts/load-test.sh

The script will:

  1. Build bifrost-http if not present
  2. Set up a mocker provider with 10s latency
  3. Run a 3000 RPS load test for 60 seconds
  4. Generate performance reports showing Bifrost's overhead

Security considerations

The script only runs locally and doesn't expose any sensitive information.

Checklist

  • I read docs/contributing/README.md and followed the guidelines
  • I added/updated tests where appropriate
  • I updated documentation where needed
  • I verified builds succeed (Go and UI)
  • I verified the CI pipeline passes locally if applicable

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Dec 10, 2025

Important

Review skipped

Draft detected.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch 12-11-load_test_as_part_of_ci

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link
Contributor Author

This stack of pull requests is managed by Graphite. Learn more about stacking.

@akshaydeo akshaydeo force-pushed the 12-11-load_test_as_part_of_ci branch from acbb0bb to 3166a42 Compare December 10, 2025 21:46
@github-actions
Copy link
Contributor

🧪 Test Suite Available

This PR can be tested by a repository admin.

Run tests for PR #1044

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