overlord/ifacestate: refresh udev backend on startup #4037

Merged
merged 1 commit into from Oct 13, 2017

Conversation

Projects
None yet
4 participants
Contributor

zyga commented Oct 13, 2017

With the recent fixes to the udev snippets of various interfaces we need
to refresh udev profiles on startup. Note that like all profiles, those
are only refreshed if they differ from the values we intend have.

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

overlord/ifacestate: refresh udev backend on startup
With the recent fixes to the udev snippets of various interfaces we need
to refresh udev profiles on startup. Note that like all profiles, those
are only refreshed if they differ from the values we intend have.

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

@mvo5 mvo5 changed the base branch from release/2.28 to master Oct 13, 2017

@mvo5 mvo5 changed the base branch from master to release/2.28 Oct 13, 2017

Codecov Report

Merging #4037 into release/2.28 will increase coverage by <.01%.
The diff coverage is 100%.

Impacted file tree graph

@@               Coverage Diff                @@
##           release/2.28    #4037      +/-   ##
================================================
+ Coverage         75.68%   75.68%   +<.01%     
================================================
  Files               413      413              
  Lines             35565    35565              
================================================
+ Hits              26918    26919       +1     
  Misses             6742     6742              
+ Partials           1905     1904       -1
Impacted Files Coverage Δ
overlord/ifacestate/helpers.go 62.33% <100%> (ø) ⬆️
cmd/snap/cmd_aliases.go 93.33% <0%> (-1.67%) ⬇️
interfaces/sorting.go 98.71% <0%> (-1.29%) ⬇️
overlord/snapstate/snapstate.go 80.4% <0%> (+0.25%) ⬆️

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 4eafe60...1a8fc46. Read the comment docs.

mvo5 approved these changes Oct 13, 2017

LGTM

@stolowski stolowski merged commit efedcc5 into snapcore:release/2.28 Oct 13, 2017

7 checks passed

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

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

@mvo5 mvo5 added this to the 2.28 milestone Oct 17, 2017

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