-
Notifications
You must be signed in to change notification settings - Fork 32
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
Include package provision for system test docker environment #3966
Conversation
Testing
|
90e8d62
to
bfde2bf
Compare
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.
How will the new parameters be set? Can you update the system README
with the instructions for provisioning with packages?
b84558f
to
7e9b5ed
Compare
To provide a system environment using the built package, it is required to use the
The decision to specify the package using those multiple parameters instead of a single one with the full package URL was motivated by the following conditions:
Also, it has tested the improvement of speed in environment provision. For this example, it has been used the basic_cluster environment. For this environment, the use of packages for system test provisioning is significantly faster compared to using sources. While the sources take around 23 minutes, the packages take only 9 minutes to complete the provisioning process. Therefore, I recommend shifting to using packages for system test provisioning when possible. |
Evidences |
Tests have been found to be stable with this change.
Description
Include package support for system tests.
Added
Testing performed
Environment provision: #3966 (comment)