Skip to content

Add new property deployer for system benchmarks#1099

Merged
mrodm merged 3 commits intoelastic:mainfrom
kcreddy:add-deployer-option-system-bench
Mar 2, 2026
Merged

Add new property deployer for system benchmarks#1099
mrodm merged 3 commits intoelastic:mainfrom
kcreddy:add-deployer-option-system-bench

Conversation

@kcreddy
Copy link
Copy Markdown
Contributor

@kcreddy kcreddy commented Feb 27, 2026

What does this PR do?

Add deployer option to system benchmark configuration

Add support for the deployer property in system benchmark configuration
files, allowing users to specify which service deployer (docker, tf, or k8s) to
use for benchmarks. This aligns system benchmarks with the existing system
test configuration capabilities.

Includes validation test packages for both valid and invalid deployer
configurations.

Why is it important?

Checklist

Related issues

@kcreddy kcreddy requested a review from a team as a code owner February 27, 2026 13:56
@kcreddy kcreddy self-assigned this Feb 27, 2026
@kcreddy kcreddy added enhancement New feature or request Team:Ecosystem Label for the Packages Ecosystem team labels Feb 27, 2026
@kcreddy kcreddy requested a review from mrodm February 27, 2026 13:58
@teresaromero
Copy link
Copy Markdown
Contributor

@coderabbitai review

@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Feb 27, 2026

✅ Actions performed

Review triggered.

Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.

@kcreddy kcreddy requested a review from teresaromero February 27, 2026 17:14
@elasticmachine
Copy link
Copy Markdown

💚 Build Succeeded

History

cc @kcreddy

Copy link
Copy Markdown
Contributor

@teresaromero teresaromero left a comment

Choose a reason for hiding this comment

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

👍🏻 lgtm

@kcreddy
Copy link
Copy Markdown
Contributor Author

kcreddy commented Mar 2, 2026

@teresaromero, could you please merge this? I don't have privileges to merge.

@teresaromero
Copy link
Copy Markdown
Contributor

@teresaromero, could you please merge this? I don't have privileges to merge.

i would like @mrodm to take a final look before merging

Copy link
Copy Markdown
Contributor

@mrodm mrodm left a comment

Choose a reason for hiding this comment

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

👍
Thanks @kcreddy !

@kcreddy
Copy link
Copy Markdown
Contributor Author

kcreddy commented Mar 2, 2026

@mrodm, thanks for the approval. Please feel free to merge, as I don't have merge access.

@mrodm mrodm merged commit a2aa38b into elastic:main Mar 2, 2026
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request Team:Ecosystem Label for the Packages Ecosystem team

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants