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

Benchmark rally add flags #1576

Merged
merged 10 commits into from
Dec 1, 2023
Merged

Conversation

aspacca
Copy link
Contributor

@aspacca aspacca commented Nov 30, 2023

Add package-from-registry and use-corpus-at-path flags.
See updated docs for usage.

@aspacca aspacca requested review from jsoriano, ruflin and gizas November 30, 2023 03:13
@aspacca aspacca self-assigned this Nov 30, 2023
@aspacca aspacca mentioned this pull request Nov 30, 2023
docs/howto/rally_benchmarking.md Outdated Show resolved Hide resolved
internal/cobraext/flags.go Outdated Show resolved Hide resolved
internal/benchrunner/runners/rally/runner.go Outdated Show resolved Hide resolved
internal/benchrunner/runners/rally/runner.go Outdated Show resolved Hide resolved
internal/benchrunner/runners/rally/runner.go Show resolved Hide resolved
internal/benchrunner/runners/rally/runner.go Outdated Show resolved Hide resolved
internal/benchrunner/runners/rally/runner.go Outdated Show resolved Hide resolved
aspacca and others added 2 commits November 30, 2023 19:09

## Loading a package from registry
If the `package-from-registry` flag is used, the package installed in Kibana and its assets referenced in the track will be loaded from registry. The format of the flag value is `%packageName%-%packageVersion%`.
You still need to run the command from the root of the local package in order to read the benchmark scenario (`benchmark` flag).
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Does this mean I need to run two commands? I tried the following:

elastic-package benchmark rally --benchmark pod-benchmark -v --package-from-registry=kubernetes-1.55.0

Can you share a working example that users can just copy/paste?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

your example is correct: I've reworded the docs, please let me know if it's clearer now

cmd/benchmark.go Outdated Show resolved Hide resolved
internal/cobraext/flags.go Show resolved Hide resolved
cmd/benchmark.go Outdated Show resolved Hide resolved
@elasticmachine
Copy link
Collaborator

💚 Build Succeeded

History

cc @aspacca

@aspacca aspacca merged commit 5f59b9e into elastic:main Dec 1, 2023
4 checks passed
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.

4 participants