overlord/ifacestate: refresh all security backends on startup #4039

Merged
merged 1 commit into from Oct 13, 2017

Conversation

Projects
None yet
3 participants
Contributor

zyga commented Oct 13, 2017

This patch changes the initially conservative selection of backends that
are refreshed on startup (apparmor and seccomp and just recently added
udev) to accommodate all backends.

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

overlord/ifacestate: refresh all security backends on startup
This patch changes the initially conservative selection of backends that
are refreshed on startup (apparmor and seccomp and just recently added
udev) to accommodate all backends.

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

mvo5 approved these changes Oct 13, 2017

Codecov Report

Merging #4039 into master will decrease coverage by <.01%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #4039      +/-   ##
==========================================
- Coverage   75.86%   75.86%   -0.01%     
==========================================
  Files         431      431              
  Lines       36926    36922       -4     
==========================================
- Hits        28014    28010       -4     
  Misses       6958     6958              
  Partials     1954     1954
Impacted Files Coverage Δ
overlord/ifacestate/helpers.go 62.5% <100%> (-0.5%) ⬇️

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 5cd43ce...54ffe8c. Read the comment docs.

@mvo5 mvo5 merged commit 0754830 into snapcore:master Oct 13, 2017

1 of 7 checks passed

artful-amd64 autopkgtest running
Details
artful-i386 autopkgtest running
Details
xenial-amd64 autopkgtest running
Details
xenial-i386 autopkgtest running
Details
xenial-ppc64el autopkgtest running
Details
zesty-amd64 autopkgtest running
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details

@zyga zyga deleted the zyga:fix/regen-all-profiles branch Oct 13, 2017

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