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

Speed up testsuite #5373

Merged
merged 9 commits into from Dec 20, 2018

Conversation

2 participants
@stgraber
Copy link
Member

stgraber commented Dec 20, 2018

No description provided.

stgraber added some commits Dec 20, 2018

tests: Add env variable to skip static analysis
Signed-off-by: Stéphane Graber <stgraber@ubuntu.com>
tests: Reduce sleeps in proxy tests
Signed-off-by: Stéphane Graber <stgraber@ubuntu.com>
tests: Remove sleep in console test
Signed-off-by: Stéphane Graber <stgraber@ubuntu.com>
tests: Reduce delays in devlxd test
Signed-off-by: Stéphane Graber <stgraber@ubuntu.com>
tests: Speed up basic tests
Signed-off-by: Stéphane Graber <stgraber@ubuntu.com>
tests: Reduce sleep in network test
Signed-off-by: Stéphane Graber <stgraber@ubuntu.com>
tests: Reduce teardown delays
Signed-off-by: Stéphane Graber <stgraber@ubuntu.com>
tests: Drop startup sleep for cluster
Signed-off-by: Stéphane Graber <stgraber@ubuntu.com>
tests: Reduce clustering delays
Signed-off-by: Stéphane Graber <stgraber@ubuntu.com>

@stgraber stgraber force-pushed the stgraber:master branch from 9c53922 to bef26be Dec 20, 2018

@@ -478,10 +478,6 @@ test_basic_usage() {

[ "${REBOOTED}" = "true" ]

# Workaround for LXC bug which causes LXD to double-start containers
# on reboot
sleep 2

This comment has been minimized.

@brauner

brauner Dec 20, 2018

Member

What the hell was that all about? :) (just curious)

This comment has been minimized.

@stgraber

stgraber Dec 20, 2018

Member

IIRC, that was LXC still restarting the container when a post-stop hook failed

This comment has been minimized.

@brauner

brauner Dec 20, 2018

Member

Oh right, I think I fixed that a while ago.

@brauner brauner merged commit d7b0053 into lxc:master Dec 20, 2018

5 checks passed

Branch target Branch target is correct
Details
DCO All commits signed-off
Details
Testsuite Testsuite passed
Details
continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
@brauner

This comment has been minimized.

Copy link
Member

brauner commented Dec 20, 2018

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment