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: add basic test for docker #2421
tests: add basic test for docker #2421
Conversation
def4d67
to
aeabc2f
Compare
aeabc2f
to
8537058
Compare
Loic told me that we need extra kernel modules for this to work on linode. So probably |
I just re-tested this. Docker needs the |
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.
Looks good, posted a question about running this on core. Even if we don't want to / cannot do that now it'd be nice to describe this in the test.
tests/main/docker/task.yaml
Outdated
@@ -0,0 +1,20 @@ | |||
summary: Check that the docker snap works | |||
|
|||
systems: [ubuntu-16.04-64] |
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.
Should we also test this on core?
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.
Yes!
This seems to be ok now:
Would be nice of course if we could run it without |
As sugested on the malinlist. Limited to classic currently but we want to expand to core once its fully working there.