chore(test): rearrange e2e structure folder #4072
Merged
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
|
oscerd
approved these changes
Feb 22, 2023
squakez
force-pushed
the
chore/speed_up_tests
branch
2 times, most recently
from
e730847 to
aba1924
Compare
Common workflow
squakez
force-pushed
the
chore/speed_up_tests
branch
from
d72316e to
0f90c1b
Compare
squakez
added
the
trigger native test
christophd
approved these changes
Feb 27, 2023
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
The goal of this PR is to rearrange the structure of the E2E test as they had grown untidily. We also address the need to reduce the time to execute the
commonsmoke test which is used for nightly releases. The idea is to install an operator ahead of time for all those tests (80% of them) which do not require a brand new installation every time instead of creating one from scratch at every test. With this we'll leverage also the reuse of kits and dependencies stored. I've also merged most of the previousinstalltests in thecommonas well (so, the gain is even higher).With this PR we should go from the average 3 hours to execute such a test to about 45 minutes (tested in my fork).
Closes #3847
Release Note