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: remove test-snapd-with-configure on restore #6201

Closed
wants to merge 1 commit into from

Conversation

zyga
Copy link
Collaborator

@zyga zyga commented Nov 23, 2018

The tests/main/ubuntu-core-gadget-config-defaults test, when executed,
leaves the mount namespace of the test-snapd-with-configure snap. For
some tests that is not a problem so the mistake was left unnoticed for a
long time. For a new test I was writing it does matter though hence the
chase for leftovers in the test suite.

Signed-off-by: Zygmunt Krynicki me@zygoon.pl

The tests/main/ubuntu-core-gadget-config-defaults test, when executed,
leaves the mount namespace of the test-snapd-with-configure snap. For
some tests that is not a problem so the mistake was left unnoticed for a
long time. For a new test I was writing it does matter though hence the
chase for leftovers in the test suite.

Signed-off-by: Zygmunt Krynicki <me@zygoon.pl>
@zyga zyga added the Simple 😃 A small PR which can be reviewed quickly label Nov 23, 2018
@zyga
Copy link
Collaborator Author

zyga commented Nov 23, 2018

Eh, the snap is not removable. Let's see what we can do instead.

@zyga zyga closed this Nov 23, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Simple 😃 A small PR which can be reviewed quickly
Projects
None yet
3 participants