-
Notifications
You must be signed in to change notification settings - Fork 313
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 role metadata to node-stats telemetry device #1654
Conversation
@elasticmachine run rally/it-python38 |
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! LGTM. I added a small nit that is optional and does not require another round of review if you do implement it.
@elasticmachine run rally/it-python310 |
esrally/actor.py
Outdated
@@ -259,6 +259,7 @@ def bootstrap_actor_system(try_join=False, prefer_local_only=False, local_ip=Non | |||
if coordinator_ip: | |||
# Make the coordinator node the convention leader | |||
capabilities["Convention Address.IPv4"] = "%s:1900" % coordinator_ip | |||
print(system_base, capabilities) |
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.
leftover print
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 raised #1655 to track the CI failure, since it's not the first time we've seen it. |
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 iterating! LGTM.
It's useful to know what roles a node was configured for when analysing metrics in a remote metrics store.
For example:
I've tested this with a local 8.5 metrics store:
esrally race --track=geopoint --pipeline=benchmark-only --target-hosts=https://localhost:9222 --challenge=append-fast-with-conflicts --telemetry=node-stats --client-options="timeout:180,verify_certs:false,basic_auth_user:'elastic',basic_auth_password:'changeme'" --kill-running-processes --test-mode