Skip to content

scaletest: use new format (no config file required) #5202

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.

Sign up for GitHub

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

Jump to bottom
Closed
Tracked by #4748
deansheather opened this issue Nov 30, 2022 · 4 comments · Fixed by #5310
Closed
Tracked by #4748

scaletest: use new format (no config file required) #5202

deansheather opened this issue Nov 30, 2022 · 4 comments · Fixed by #5310
Assignees
@deansheather

Comments

@deansheather
Copy link
Member

deansheather commented Nov 30, 2022 •
edited
Loading

Remove the old JSON configuration format and convert the individual job types into subcommands of coder loadtest.

$ coder scaletest create-workspaces \
  --count 1000 \
  --template "name or ID" \
  [--parameters-file "path/to/file"] \
  [--parameter key="value"] \
  [--no-cleanup] \
  [--no-cleanup-failures] \
  [--no-wait-for-agents] \
  [--run-command "some command to run over reconnecting-pty in each workspace in the default shell"] \
  [--run-pty-size "80x24"] \
  [--run-timeout "5m"] \
  [--run-expect-timeout] \
  [--run-expect-output "expect this in command output"] \
  [--run-log-output] \
  [--connect-mode "derp|direct"] \
  [--connect-hold] \
  [--connect-hold-duration "5m"] \
  [--connect-url "http://localhost:4/"] \
  [--connect-interval "1s"] \
  [--connect-timeout "10s"]
@deansheather deansheather self-assigned this Nov 30, 2022
@bpmct bpmct mentioned this issue Nov 30, 2022
Closed
21 tasks
@bpmct
Copy link
Member

bpmct commented Nov 30, 2022

@deansheather this is going to be coder scaletest, right?

@deansheather
Copy link
Member Author

Sure, I'll change it from loadtest to scaletest

@bpmct bpmct changed the title Update coder loadtest command to new format Update coder scaletest command to new format Dec 1, 2022
@bpmct bpmct mentioned this issue Dec 1, 2022
Closed
@bpmct
Copy link
Member

bpmct commented Dec 1, 2022

From #5206 (comment)

I actually forgot to specify the concurrency flags in the issue, my bad. We'll have flags for --concurrency, --cleanup-concurrency, --timeout and --cleanup-timeout

@bpmct bpmct changed the title Update coder scaletest command to new format scaletest: use new format (no config file required) Dec 1, 2022
@deansheather
Copy link
Member Author

I have removed --no-cleanup-failures as it can be implemented later, and --run-pty-size as it's not very useful.

@deansheather deansheather mentioned this issue Dec 6, 2022
Merged
4 tasks
@bpmct bpmct mentioned this issue Dec 6, 2022
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@deansheather deansheather

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

feat: remove loadtest cmd, add new scaletest cmd coder/coder
2 participants
@deansheather @bpmct