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.

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

[tests] and different operation modes to integration testing script #27

Merged

Conversation

timothytrippel
Copy link
Collaborator

Previously, ./run_integration_tests.sh would build a release tarball of all the provisioning binaries, along with container images containing each provisioning binary, and then unpack the binaries into the deployment staging area, even if only the containers were run. This was slow when debugging because compressing and uncompressing binaries (that are not even used) is slow.

This adds a -c mode to the ./run_integration_tests.sh to only build an deploy the container images to save time when debugging.

Additionally, a -d (debug) mode is added that prevents the containers from being torn down on script exit to enable debugging test failures.

Previously, `./run_integration_tests.sh` would build a release tarball
of all the provisioning binaries, along with container images containing
each provisioning binary, and then unpack the binaries into the deployment
staging area, even if only the containers were run. This was slow when
debugging because compressing and uncompressing binaries (that are not
even used) is slow.

This adds a `-c` mode to the `./run_integration_tests.sh` to only build
an deploy the container images to save time when debugging.

Additionally, a `-d` (debug) mode is added that prevents the containers
from being torn down on script exit to enable debugging test failures.

Signed-off-by: Tim Trippel <[email protected]>
@timothytrippel timothytrippel merged commit e21e9dc into lowRISC:main Oct 2, 2024
3 checks passed
@timothytrippel timothytrippel deleted the add-fast-integration-test-mode branch October 2, 2024 21:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants