-
Notifications
You must be signed in to change notification settings - Fork 119
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
Persist VU settings across restarts #696
Persist VU settings across restarts #696
Conversation
cc9a3b1
to
4f83d39
Compare
Many thanks for the pull request, I will take a look and review as soon as time allows. |
I've done a quick overview and some changes are needed, for the GUI it makes more sense to update the entire dict rather than every value with a single command and for the cli the command updates the SQLite data to persist over restarts but does not update the dict so if you do print generic after doing vuset it still shows the old value. I will do a commit to the PR that corrects these and then it should be good. |
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.
As per comment added change 5ff2f8a to update GUI persistence with a single command and to persist CLI changes also in the current genericdict
Automatically update the VU generic settings inside the
vuset
command (CLI) and inside the Virtual User Options (GUI) to make it easier to persist VU settings across restarts. Write the values to SQLite storage when set or the user presses OK.Example of the new vuset (CLI) behavior across the restarts:
Virtual Users Options menu (GUI) is the same before and after the restart: