This repository has been archived by the owner on Jul 11, 2023. It is now read-only.
ref(test): Improve e2e test framework using Fortio as load test tool #4631
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Codecov Report
@@ Coverage Diff @@
## main #4631 +/- ##
==========================================
+ Coverage 68.97% 68.99% +0.02%
==========================================
Files 213 213
Lines 14861 14861
==========================================
+ Hits 10250 10253 +3
+ Misses 4558 4555 -3
Partials 53 53
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report at Codecov.
|
allenlsy
added
the
do-not-merge/hold
allenlsy
force-pushed
the
e2e-fortio
branch
27 times, most recently
from
ef850b5 to
b1b3a60
Compare
allenlsy
force-pushed
the
e2e-fortio
branch
3 times, most recently
from
6a15408 to
9e37950
Compare
allenlsy
requested review from
shashankram,
snehachhabria,
nojnhuh,
draychev,
jaellio and
trstringer
as code owners
allenlsy
force-pushed
the
e2e-fortio
branch
5 times, most recently
from
2cabc53 to
5d867b2
Compare
allenlsy
changed the title
allenlsy
removed
the
do-not-merge/hold
allenlsy
force-pushed
the
e2e-fortio
branch
2 times, most recently
from
e6f6929 to
777bc72
Compare
* Improve e2e test framework to support running Fortio load tests * Migrate testing application images to Fortio client & server * Update some e2e test case for Fortio feature verification Signed-off-by: Allen Leigh <[email protected]>
shashankram
approved these changes
Apr 20, 2022
There was a problem hiding this comment.
@allenlsy thanks for this significant enhancement. Great to see all the e2e tests pass.
trstringer
approved these changes
Apr 20, 2022
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description:
Below is a list of services in the e2e tests that are used to test different web request protocols:
They are right now testing in single request mode. As our project develops, we need to a more customizable request sending solution to test various scenarios.
Fortio is a load testing tool that comes with both the server (web UI included) and the load generating client. They are packaged in a single Docker image.
In this PR:
Partially resolves #4621 .
TODO after this PR:
Testing done:
✅ Updated unit test
Affected area:
Please answer the following questions with yes/no.
No
Is this a breaking change?
Has documentation corresponding to this change been updated in the osm-docs repo (if applicable)?