-
Notifications
You must be signed in to change notification settings - Fork 529
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
Add terraform module to benchmark Elasticsearch on Elastic Cloud with locally built APM-Server and Integration package #8986
Conversation
2cc16dc
to
36575cf
Compare
This pull request does not have a backport label. Could you fix it @lahsivjar? 🙏
NOTE: |
📚 Go benchmark reportDiff with the
report generated with https://pkg.go.dev/golang.org/x/perf/cmd/benchstat |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks great!
Regarding the need to run make
in testing/cloud
: perhaps instead of building a custom Kibana image, we would be better off using the default image, and uploading a custom integration package to Kibana after it starts like we do in the system tests:
apm-server/systemtest/kibana.go
Line 137 in fd81083
if err := Fleet.InstallPackageByUpload(f); err != nil { |
This pull request is now in conflicts. Could you fix it @lahsivjar? 🙏
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice - uploading the custom package is much more sensible than bundling in a custom Kibana. We should probably update testing/cloud later to make use of this too.
It would be great if we could combine the two new vars into one.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks great :)
/test |
/test |
I tested this earlier. Not with a BC, since it's independent of the stack release. |
Motivation/summary
Allows running rally to benchmark a locally built Elastic Cloud cluster. This allows developers to benchmark changes introduced in ES schema used by APM-Server.
Checklist
- [ ] Update CHANGELOG.asciidoc- [ ] Update package changelog.yml (only if changes toapmpackage
have been made)- [ ] Documentation has been updatedHow to test these changes
testing/rally-cloud
and runmake init
if you are running for the firsttime
EC_API_KEY
by following this guide.testing/rally-cloud/variables.tf
to customize your benchmark.EC_API_KEY=<ec_api_key> make apply
to create the required infra and run rally.The rally benchmark will execute everytime
make apply
is called however, the infrastucturewill be created only once. After testing is done make sure to destroy all infrastructure using
EC_API_KEY=<ec_api_key> make destroy
Related issues
Closes #8754