-
Notifications
You must be signed in to change notification settings - Fork 9
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
Add Containerization To Load Test #1130
Conversation
Co-authored-by: James Herr<[email protected]>
…nerating and linking placer order id, replaced in message
…CDCgov/trusted-intermediary into story/1121/dockerize-load-tests
…CDCgov/trusted-intermediary into story/1121/dockerize-load-tests
Bring back silence
Co-Authored-by: Luis Pabon<[email protected]>
…CDCgov/trusted-intermediary into story/1121/dockerize-load-tests
docker-load-execute.sh
Outdated
warm_up_api() { | ||
echo 'Warming up API...' | ||
pem=$(pwd)/mock_credentials/organization-trusted-intermediary-private-key-local.pem | ||
token=$(jwt encode --exp='+5min' --jti $(uuidgen) --alg RS256 --no-iat -S@${pem}) |
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.
You could read the JWT from the mock_credentials .jwt file to remove the dependency with the jwt
CLI
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.
Part of the reason for this is to also warm up the Auth endpoint. I found that individually calling the endpoints helps those endpoints perform faster.
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.
What I mean is that you could replace the jwt
CLI command, for something like this: cat mock_credentials/trusted-intermediary-valid-token.jwt
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.
Ah I see, for some reason I thought the mock_credentials/trusted-intermediary-valid-token.jwt
was the latter step in auth but that makes sense. I'll change this asap.
Quality Gate passedIssues Measures |
Run API in Docker During Load Test
This PR adds the logic to run the API in a docker container when performing the load test.
Issue
#1121