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

custom field in locust report #1592

Closed
stanislawskwark opened this issue Oct 14, 2020 · 5 comments
Closed

custom field in locust report #1592

stanislawskwark opened this issue Oct 14, 2020 · 5 comments
Labels
feature request non-critical stale Issue had no activity. Might still be worth fixing, but dont expect someone else to fix it

Comments

@stanislawskwark
Copy link
Contributor

Is your feature request related to a problem? Please describe.

It would be helpful to have possibility to add custom fields to locust report

Describe the solution you'd like

I tried to show more or less what I mean on image:
image

so X would be a custom field and v1 v2 v3 some values. values of x field could be also accessible from code

Describe alternatives you've considered

Using custom separator like "|" in name works as a workaroud (i.e. "/blog|v2") but it's not very elegant

@cyberw
Copy link
Collaborator

cyberw commented Oct 14, 2020

Yes, this would be nice.

I have been thinking about adding a tags parameter to requests, which kind of relates to this. Tags could be general (like in your example), or more unique (like customer id).

The more difficult part (although maybe not that difficult) would be how this should be aggregated. You'd need to be able to select what tags should be considered when aggregating (probably in code), to control how requests are aggregated on workers, forwarded to the master and lastly displayed in the UI (in my example version is something that we'd want to aggregate, but customer id is probably not). If we want to do it "right", we should probably change "type" (post, get, etc) into a tag.

Tags should of course be passed to the request_success/failure methods. While we are at it, I think we should also pass the User instance on which they are called, and (as suggested in #1413) the full request/response object.

@github-actions
Copy link

This issue is stale because it has been open 30 days with no activity. Remove stale label or comment or this will be closed in 10 days.

@github-actions github-actions bot added the stale Issue had no activity. Might still be worth fixing, but dont expect someone else to fix it label Apr 11, 2021
@cyberw cyberw removed the stale Issue had no activity. Might still be worth fixing, but dont expect someone else to fix it label Apr 11, 2021
@github-actions
Copy link

This issue is stale because it has been open 30 days with no activity. Remove stale label or comment or this will be closed in 10 days.

@github-actions github-actions bot added the stale Issue had no activity. Might still be worth fixing, but dont expect someone else to fix it label Jun 11, 2021
@cyberw
Copy link
Collaborator

cyberw commented Jun 11, 2021

This was partially addressed in #1750. Somebody could add aggregation and gui support.

@github-actions
Copy link

This issue was closed because it has been stalled for 10 days with no activity. This does not necessarily mean that the issue is bad, but it most likely means that nobody is willing to take the time to fix it. If you have found Locust useful, then consider contributing a fix yourself!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature request non-critical stale Issue had no activity. Might still be worth fixing, but dont expect someone else to fix it
Projects
None yet
Development

No branches or pull requests

2 participants