Skip to content

loadtest: load test the new registry #261

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

Open
matifali opened this issue Dec 20, 2024 · 2 comments
Open

loadtest: load test the new registry #261

matifali opened this issue Dec 20, 2024 · 2 comments
Labels
help wanted Extra attention is needed project/notifications

Comments

@matifali
Copy link
Member

We have recently moved the Coder registry https://registry.coder.com to a new refactored one. We would like to load test it.

It could be as simple as including some modules in the load-testing template.

module "jetbrains_gateway" {
  source         = "registry.coder.com/modules/jetbrains-gateway/coder"
  version        = "1.0.25"
  agent_id       = coder_agent.example.id
  agent_name     = "example"
  folder         = "/home/coder/example"
  jetbrains_ides = ["CL", "GO", "IU", "PY", "WS"]
  default        = "GO"
}

cc: @bcpeinhardt to coordinate with @f0ssel when this happens. We may need to keep an eye on the registry.
Ideally, we should have monitoring #182 and #137 before we do this.

@matifali matifali added this to the C210K: Phase 1 milestone Dec 20, 2024
@coder-labeler coder-labeler bot added help wanted Extra attention is needed project/notifications labels Dec 20, 2024
@bcpeinhardt
Copy link

@Parkreiner and I did some "load testing" prior to moving prod over. We ran something like 3000 requests over the course of a couple minutes and nothing was too bothered or too slow.
As it stands now I'm not convinced we need anything like regular load testing, as things are running pretty smoothly under the current production load, but maybe a load testing script in the repo for running against dev if we make changes we think may impact performance? Could be useful for detecting any introduction of memory leaks etc.
My instinct is with the existing architecture (cloud run deployed to us-central serving statically generated files), distance and the possibility of a cold start will effect performance more than anything else, and a regularly running load test would cost as much as the service itself.

@matifali
Copy link
Member Author

matifali commented Jan 7, 2025

I agree that we do not need this now but should include it in our regular load testing @f0ssel is working on. Moving it to low priority now.

@johnstcn johnstcn changed the title laodtest: load test the new registry loadtest: load test the new registry Jan 8, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
help wanted Extra attention is needed project/notifications
Projects
None yet
Development

No branches or pull requests

2 participants