-
Notifications
You must be signed in to change notification settings - Fork 62
Add Elasticsearch Rally workload artifacts #611
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
base: main
Are you sure you want to change the base?
Add Elasticsearch Rally workload artifacts #611
Conversation
Signed-off-by: v-safilho <v-safilho@microsoft.com>
…ticsearch package download. New profile parameters to let the user choose Elasticsearch and Rally versions. Implemented the control to let wget or apt to be used for Linux, and WebRequest or parallel download for Windows. New unit tests were built.
…ps://github.com/v-safilho/VirtualClient into users/v-safilho/Add-ElasticsearchRally-Workload
…ps://github.com/v-safilho/VirtualClient into users/v-safilho/Add-ElasticsearchRally-Workload
| using global::VirtualClient.Contracts; | ||
| using Microsoft.VisualBasic; | ||
|
|
||
| internal class ElasticsearchMetricReader |
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.
This looks like it should be a metrics parser, for our other workloads we extend from the MetricsParser abstract class. It seems to do what you're doing here with this reader.
| using System.Threading; | ||
| using System.Threading.Tasks; | ||
|
|
||
| internal class ParallelDownloadHandler |
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.
This seems generic and not related to elastic search/rally? I think VirtualClient.Core or VirtualClient.Dependencies might be a better place for this.
Add the Linux client/server workload to evaluate hardware performance using Elasticsearch.
related to PBI #28841299