Skip to content
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/core/failover: enable the test on core18 #11175

Merged
merged 2 commits into from Dec 16, 2021

Conversation

bboozzoo
Copy link
Collaborator

Enable the test on core18

Signed-off-by: Maciej Borzecki <maciej.zenon.borzecki@canonical.com>
Signed-off-by: Maciej Borzecki <maciej.zenon.borzecki@canonical.com>
@codecov-commenter
Copy link

Codecov Report

Merging #11175 (3550828) into master (0190c13) will decrease coverage by 0.00%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master   #11175      +/-   ##
==========================================
- Coverage   78.33%   78.33%   -0.01%     
==========================================
  Files         920      920              
  Lines      104882   104882              
==========================================
- Hits        82159    82156       -3     
- Misses      17597    17598       +1     
- Partials     5126     5128       +2     
Flag Coverage Δ
unittests 78.33% <ø> (-0.01%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
sandbox/cgroup/tracking.go 92.72% <0.00%> (-1.27%) ⬇️
overlord/ifacestate/handlers.go 64.87% <0.00%> (-0.15%) ⬇️
osutil/synctree.go 79.24% <0.00%> (+2.83%) ⬆️

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 0190c13...3550828. Read the comment docs.

Copy link
Contributor

@mardy mardy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, just a question for you to double check :-)

@@ -109,7 +122,7 @@ execute: |
fi

# check boot env vars
readlink /snap/core/current > failBoot
readlink "/snap/$core_name/current" > failBoot
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do we want $core_name or $TARGET_SNAP_NAME here?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good point $TARGET_SNAP_NAME, but it works by accident since there was only ever running for the core* snaps. I'm working on making it work for the kernel snap too, so I'll fix it there.

Copy link
Collaborator

@sergiocazzolato sergiocazzolato left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice to see this enabled on uc18, thanks

@mvo5 mvo5 merged commit 7174d6e into snapcore:master Dec 16, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
5 participants