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

installer-tests: Use a more specific cleanup function #2650

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

paulproteus
Copy link
Collaborator

Earlier, I had added purging postfix to cleanup for all tests. However, not all of our test VMs run Debian,
so this was a pretty silly idea in hindsight.

This Sandstorm commit uses a new cleanup function defined here:
paulproteus/stodgy-tester@51ed3de
so that Postfix is successfully removed from the VM to avoid disruption to
other installer-tests.

Testing done: Currently running this on asheeshdev.sandcats.io with the latest version of stodgy-tester. Once that passes, I'll merge this, and re-enable https://build.sandstorm.io/job/Sandstorm-installer/ and do a test build on Jenkins to make sure all is well.

Earlier, I had added purging postfix to cleanup for all tests. However, not all of our test VMs run Debian,
so this was a pretty silly idea in hindsight.

This Sandstorm commit uses a new cleanup function defined here:
paulproteus/stodgy-tester@51ed3de
so that Postfix is successfully removed from the VM to avoid disruption to
other installer-tests.
@ocdtrekkie ocdtrekkie added the sandstorm-dev Issues hacking on Sandstorm label Feb 22, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
sandstorm-dev Issues hacking on Sandstorm
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants