Skip to content
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

Store track-related meta-data in results #771

Merged

Conversation

danielmitterdorfer
Copy link
Member

Rally allows to specify meta-data on track, challenge, task and
operation level. This information is stored for raw metrics records but
not for results. With this commit we ensure that any meta-data are
properly set also on the respective results documents. This allows to
use this information also e.g. in Kibana visualizations.

Rally allows to specify meta-data on track, challenge, task and
operation level. This information is stored for raw metrics records but
not for results. With this commit we ensure that any meta-data are
properly set also on the respective results documents. This allows to
use this information also e.g. in Kibana visualizations.
@danielmitterdorfer danielmitterdorfer added enhancement Improves the status quo :Metrics How metrics are stored, calculated or aggregated labels Sep 12, 2019
@danielmitterdorfer danielmitterdorfer added this to the 1.4.0 milestone Sep 12, 2019
Copy link
Contributor

@dliappis dliappis left a comment

Choose a reason for hiding this comment

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

Looks great! Left a minor comment for a clarification, but otherwise LGTM.

esrally/reporter.py Show resolved Hide resolved
@@ -327,23 +340,32 @@ def as_dict(self):
return self.__dict__

def as_flat_list(self):
def op_metrics(op_item, key, single_value=False):
Copy link
Contributor

Choose a reason for hiding this comment

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

Nice refactoring here!

@danielmitterdorfer danielmitterdorfer merged commit 06e0b3e into elastic:master Sep 12, 2019
@danielmitterdorfer danielmitterdorfer deleted the meta-for-results branch September 12, 2019 08:40
@danielmitterdorfer
Copy link
Member Author

Thanks for the review! :)

novosibman pushed a commit to novosibman/rally that referenced this pull request Oct 2, 2019
Rally allows to specify meta-data on track, challenge, task and
operation level. This information is stored for raw metrics records but
not for results. With this commit we ensure that any meta-data are
properly set also on the respective results documents. This allows to
use this information also e.g. in Kibana visualizations.

Relates elastic#771
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Improves the status quo :Metrics How metrics are stored, calculated or aggregated
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants