tests: re-enable tun/tap test on Debian #4213

Merged
merged 1 commit into from Nov 14, 2017

Conversation

Projects
None yet
3 participants
Contributor

zyga commented Nov 14, 2017

This test used to fail and was disabled to unblock master but upon
inspection it does not fail anymore. The reason of the earlier failure
remains unknown.

Signed-off-by: Zygmunt Krynicki zygmunt.krynicki@canonical.com

tests: re-enable tun/tap test on Debian
This test used to fail and was disabled to unblock master but upon
inspection it does not fail anymore. The reason of the earlier failure
remains unknown.

Signed-off-by: Zygmunt Krynicki <zygmunt.krynicki@canonical.com>

codecov-io commented Nov 14, 2017

Codecov Report

Merging #4213 into master will decrease coverage by <.01%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #4213      +/-   ##
==========================================
- Coverage   75.83%   75.83%   -0.01%     
==========================================
  Files         439      439              
  Lines       38176    38176              
==========================================
- Hits        28952    28951       -1     
- Misses       7214     7215       +1     
  Partials     2010     2010
Impacted Files Coverage Δ
overlord/ifacestate/helpers.go 59.6% <0%> (-0.67%) ⬇️
cmd/snap/cmd_aliases.go 95% <0%> (+1.66%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 960abc1...a8a279e. Read the comment docs.

mvo5 approved these changes Nov 14, 2017

@mvo5 mvo5 merged commit 6b35474 into snapcore:master Nov 14, 2017

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details

@zyga zyga deleted the zyga:fix/re-enable-tuntap-on-debian branch Nov 14, 2017

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