Skip to content
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

Remote first approach to take benchmarking #6

Merged
merged 10 commits into from
Dec 24, 2024
Merged

Conversation

nachocodoner
Copy link
Member

@nachocodoner nachocodoner commented Oct 14, 2024

OSS-585

The PR is to start using the Galaxy container as the approach to measure performance (only for Meteor staff since we have access to the Galaxy apps involving this repository).

This PR is also a preparation of a proper artillery config that reproduces the regressions found as an action to address community’s feedback on this thread.

If we have this configuration done, other members of the community can also use their infrastructures to deploy the apps and use the script and configuration to compare Meteor 3 and 2 and provide feedback and performance improvements.

The inital approach was using a local-first approach with a fixed machine. It had 64GB of RAM (software capped this memory at its limit), and we noticed the issue reported: Meteor 3 performs worse than Meteor 2, with methods running faster and subscriptions being the slowest. Testing with a remote machine confirmed this behavior, and later we gathered more metrics using APM here. Shifting to a remote-first approach would better align results with the Meteor team and community.

@nachocodoner nachocodoner changed the base branch from main to report-3.1.1 December 24, 2024 15:44
@nachocodoner nachocodoner marked this pull request as ready for review December 24, 2024 15:44
@nachocodoner nachocodoner merged commit 6ebc296 into report-3.1.1 Dec 24, 2024
1 check passed
@nachocodoner nachocodoner deleted the remote-first branch December 24, 2024 15:44
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.

1 participant