Skip to content

Ensure sentry passes JS integration tests and benchmarks #257

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
Ivshti opened this issue Feb 24, 2020 · 4 comments
Closed

Ensure sentry passes JS integration tests and benchmarks #257

Ivshti opened this issue Feb 24, 2020 · 4 comments
Assignees
Labels
High High priority task tracking Tracking issue for a bigger task
Milestone

Comments

@Ivshti
Copy link
Member

Ivshti commented Feb 24, 2020

ensure both pass

please report the results of the benchmark here, the release mode benchmark of the sentry should be at least a few times better

@samparsky @elpiel you can split duties between the benchmark suite and the integration tests

@elpiel
Copy link
Member

elpiel commented Feb 26, 2020

I can confirm that @samparsky made the integration tests run and pass! Now I am looking at the benchmark and @samparsky is looking into the failing routes tests

@elpiel
Copy link
Member

elpiel commented Feb 27, 2020

I am debugging sentry to see why we have such a low req/sec from the benchmark.

@elpiel elpiel pinned this issue Feb 27, 2020
@elpiel elpiel added this to the sentry-v0.2.0 milestone Jan 5, 2021
@elpiel
Copy link
Member

elpiel commented Jan 8, 2021

I am changing the priority of this task since the Rust Sentry is lacking behind the JS one and we need to make sure all integration tests are passing.

Currently known issues to catch up with JS Sentry are:

@simzzz Please double check if additional routes, features, etc are still missing from the Rust Sentry implementation and share them in order to be able to prioritize and track them

@elpiel elpiel added High High priority task tracking Tracking issue for a bigger task labels Jan 8, 2021
@elpiel elpiel unpinned this issue Jun 9, 2022
@elpiel
Copy link
Member

elpiel commented Aug 22, 2022

V5 & #377 have introduced both the test_harness and additional tests for both sentry & worker and benchmarks for sentry (events submission)

@elpiel elpiel closed this as completed Aug 22, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
High High priority task tracking Tracking issue for a bigger task
Projects
None yet
Development

No branches or pull requests

3 participants