Skip to content

Webui: adding profile argument and display in html report #3093

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

Merged
merged 3 commits into from
Apr 11, 2025

Conversation

schwannden
Copy link
Contributor

Introduced a --profile CLI flag to allow users to specify a custom profile name.
This value is now displayed in the generated HTML report, making it easier to distinguish between test runs targeting different deployment strategies. Useful for organizing large sets of experiments.

image

@schwannden schwannden force-pushed the feature/add-profile-config branch 3 times, most recently from 20f9fec to beb781b Compare April 9, 2025 10:24
@schwannden schwannden force-pushed the feature/add-profile-config branch from beb781b to 4086097 Compare April 9, 2025 10:31
@cyberw
Copy link
Collaborator

cyberw commented Apr 9, 2025

I think the locustfile.py was accidentally commited, right?

@andrewbaldwin44 wdyt?

I'm thinking we could make the profile editable under the advanced options in the new test dialog box too, and visisble somewhere (maybe in the "current ratio" tab, if we can't find a better place for it)

Copy link
Collaborator

@andrewbaldwin44 andrewbaldwin44 left a comment

Choose a reason for hiding this comment

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

Looks good to me now! Maybe you can check @cyberw's comments? Being able to change the profile from the web ui / displaying the value somewhere in the UI while the test is running?

@schwannden
Copy link
Contributor Author

Looks good to me now! Maybe you can check @cyberw's comments? Being able to change the profile from the web ui / displaying the value somewhere in the UI while the test is running?

For that, I think we could use another pull request. I would prefer to keep this pull request small and simple. What do you think @cyberw ?

@cyberw cyberw merged commit 0f80721 into locustio:master Apr 11, 2025
16 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants