Skip to content

Conversation

Saviq
Copy link
Collaborator

@Saviq Saviq commented Dec 13, 2019

No description provided.

@Saviq Saviq requested a review from ricab December 13, 2019 11:01
@multipass-ci-bot

This comment has been minimized.

@codecov
Copy link

codecov bot commented Dec 13, 2019

Codecov Report

Merging #1228 into master will increase coverage by <.01%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #1228      +/-   ##
==========================================
+ Coverage   71.32%   71.33%   +<.01%     
==========================================
  Files         203      203              
  Lines        7478     7480       +2     
==========================================
+ Hits         5334     5336       +2     
  Misses       2144     2144
Impacted Files Coverage Δ
...rc/platform/backends/qemu/qemu_vm_process_spec.cpp 98.63% <ø> (ø) ⬆️
...rc/platform/backends/qemu/dnsmasq_process_spec.cpp 93.54% <ø> (ø) ⬆️
...form/backends/shared/sshfs_server_process_spec.cpp 54.05% <ø> (ø) ⬆️
...orm/backends/shared/linux/qemuimg_process_spec.cpp 100% <ø> (ø) ⬆️
src/utils/snap_utils.cpp 100% <100%> (ø) ⬆️

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 03e6588...bfa9cca. Read the comment docs.

@Saviq Saviq force-pushed the fix-symlinked-snap branch from b818d62 to b79a62f Compare December 13, 2019 12:06
@Saviq Saviq force-pushed the fix-symlinked-snap branch from b79a62f to fb06e79 Compare December 13, 2019 12:07
@multipass-ci-bot

This comment has been minimized.

Just hardcoding /var/lib/snapd since this goes away with strict anyway.
@Saviq Saviq force-pushed the fix-symlinked-snap branch from fb06e79 to bfa9cca Compare December 13, 2019 15:11
@multipass-ci-bot
Copy link
Collaborator

Snap build available: snap refresh multipass --channel edge/pr1228

Copy link
Collaborator

@ricab ricab left a comment

Choose a reason for hiding this comment

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

Looks good. Nice tests.

I had quite a few testing glitches but in the end they all seem unrelated to this.

bors r+

bors bot added a commit that referenced this pull request Dec 13, 2019
1228: [utils] resolve symlinks to snap directories (Fixes #1223) r=ricab a=Saviq



Co-authored-by: Michał Sawicz <[email protected]>
@bors
Copy link
Contributor

bors bot commented Dec 13, 2019

Build failed

@Saviq Saviq merged commit bfa9cca into master Dec 13, 2019
@bors bors bot deleted the fix-symlinked-snap branch December 13, 2019 20:10
Saviq added a commit that referenced this pull request Dec 13, 2019
1228: [utils] resolve symlinks to snap directories (Fixes #1223) r=ricab a=Saviq



Co-authored-by: Michał Sawicz <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants