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.

Sign up for GitHub

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

Jump to bottom

Load tests #854

Open
wants to merge 6 commits into
base: master
Choose a base branch
from
Open

Load tests #854

wants to merge 6 commits into from
+2,417 −0

Conversation

kiniry
Copy link
Contributor

@kiniry kiniry commented Oct 26, 2017 •
edited
Loading

Closes #350 , #351.

@kiniry kiniry added this to the fourth and final release milestone Oct 26, 2017
@kiniry kiniry requested review from dmzimmerman and nealmcb October 26, 2017 05:10
@nealmcb
Copy link
Contributor

nealmcb commented Oct 26, 2017

.pyc files should not be included.

@nealmcb
Copy link
Contributor

nealmcb commented Oct 26, 2017

It seems like we should not capture the reset time in the timing for each script.

@kiniry
Copy link
Contributor Author

kiniry commented Oct 26, 2017

External timing on the scripts presumes an empty DB. I have been using some shell scripting to narrow log messages to get the precise timings you have seen in the load testing spreadsheet. I will add a someday issue to improve this and make it fully automated.

dmzimmerman
dmzimmerman reviewed Oct 26, 2017
View reviewed changes
test/load_tests/2-cdos.sh
@@ -0,0 +1,8 @@
#!/bin/sh
./main.py -u http://192.168.24.43/api/ reset dos_init > 0.log
Copy link
Member

Choose a reason for hiding this comment

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

Would it make sense to replace these scripts with scripts that take a command line parameter for the hostname?

Copy link
Contributor

Choose a reason for hiding this comment

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

A parameter would be better. Or my approach in autotest2.sh is to pass an environment variable, $URL.

nealmcb
nealmcb approved these changes Dec 4, 2017
View reviewed changes
Copy link
Contributor

@nealmcb nealmcb left a comment •
edited
Loading

Choose a reason for hiding this comment

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

This PR provides a variety of scripts and data files for testing performance of the full system. Future enhancements should focus on parallel entry of ACVRs from multiple counties at once.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@dmzimmerman dmzimmerman dmzimmerman left review comments

@nealmcb nealmcb nealmcb approved these changes

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
assurance review server
Projects
None yet
Milestone
fourth and final release
Development

Successfully merging this pull request may close these issues.
3 participants
@kiniry @nealmcb @dmzimmerman