-
Notifications
You must be signed in to change notification settings - Fork 127
Support system benchmarking with the logstash-output in the benchmarking agent policies #1615
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
Conversation
|
|
||
| // Assign the data_output_id to the agent policy to configure the output to logstash. The value is inferred from stack/_static/kibana.yml.tmpl | ||
| if r.options.Profile.Config("stack.logstash_enabled", "false") == "true" { | ||
| p.DataOutputID = "fleet-logstash-output" |
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.
Could we add a test for this case?
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.
@jsoriano You mean a test package? The test-check-packages has regular benchmarks being tested. Should we add another package like benchmarks-with-logstash?
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.
It would be a test that runs benchmarks with logstash, it can be a new package or a new script and an existing package, not sure what would be the best approach.
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.
I can add a regression test. As currently logstash does not provide relevant stats that are comparable to ingest node stats. But I just want to make sure we can run system benchmarks even with logstash enabled.
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.
@jsoriano Added a test package
983edad to
57f0264
Compare
57f0264 to
9aef4d8
Compare
💚 Build Succeeded
History
cc @bhapas |
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.
Thanks for adding a test case!
Support system benchmarking with the logstash-output in the benchmarking agent policies.
If
stack.logstash_enabled: trueis set in the current profile's config use the logstash output over the default.