Skip to content

add input and vars in rally scenario spec, make input and corpora man…#653

Merged
ruflin merged 14 commits intoelastic:mainfrom
francescayeye:rally-benchmark-take-2
Nov 6, 2023
Merged

add input and vars in rally scenario spec, make input and corpora man…#653
ruflin merged 14 commits intoelastic:mainfrom
francescayeye:rally-benchmark-take-2

Conversation

@francescayeye
Copy link
Copy Markdown

@francescayeye francescayeye commented Oct 25, 2023

…datory

What does this PR do?

Defines better the spec for the rally scenario

Why is it important?

When we developed elastic/elastic-package#1522 we realized the spec need some update

Checklist

Related issues

@francescayeye francescayeye self-assigned this Oct 25, 2023
@francescayeye francescayeye requested a review from a team as a code owner October 25, 2023 06:14
@elasticmachine
Copy link
Copy Markdown

💚 Build Succeeded

History

cc @aspacca

Copy link
Copy Markdown
Collaborator

@ruflin ruflin left a comment

Choose a reason for hiding this comment

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

LGTM but we need @jsoriano or @mrodm for final confirmation.

Copy link
Copy Markdown
Member

@jsoriano jsoriano left a comment

Choose a reason for hiding this comment

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

👍

@ruflin ruflin merged commit 1a7d2cb into elastic:main Nov 6, 2023
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