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: run spread tests in fedora-35 #11120

Merged
merged 4 commits into from Dec 9, 2021

Conversation

sergiocazzolato
Copy link
Collaborator

This change adds support to fedora-35 in spread tests and move to manual fedora-33

@codecov-commenter
Copy link

codecov-commenter commented Nov 29, 2021

Codecov Report

Merging #11120 (b13410b) into master (5f5e786) will decrease coverage by 0.00%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master   #11120      +/-   ##
==========================================
- Coverage   78.26%   78.25%   -0.01%     
==========================================
  Files         918      918              
  Lines      104403   104403              
==========================================
- Hits        81706    81703       -3     
- Misses      17580    17582       +2     
- Partials     5117     5118       +1     
Flag Coverage Δ
unittests 78.25% <ø> (-0.01%) ⬇️

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

Impacted Files Coverage Δ
osutil/synctree.go 76.41% <0.00%> (-2.84%) ⬇️
daemon/api_connections.go 93.04% <0.00%> (-0.54%) ⬇️
cmd/snap/cmd_debug_state.go 70.64% <0.00%> (+0.45%) ⬆️

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 5f5e786...b13410b. 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!

@bboozzoo
Copy link
Collaborator

bboozzoo commented Dec 2, 2021

I'll try to find out what's going on with the failing tests.

@bboozzoo
Copy link
Collaborator

bboozzoo commented Dec 3, 2021

Ok, so it's as I suspected, it's enough to run just 2 tests, one after another, this is the sequence that triggers it:

google:fedora-35-64:tests/main/parallel-install-services
google:fedora-35-64:tests/main/snap-mgmt

I think that it's caused by the fact that s-c will remount /var/lib/snapd/snap onto itself to make it a mountpoint and then change the propagation. Need to investigate this further.

@bboozzoo
Copy link
Collaborator

bboozzoo commented Dec 3, 2021

Yup:

google:fedora-35-64 .../tests/main/snap-mgmt# findmnt 
TARGET                        SOURCE                              FSTYPE     OPTIONS
/                             /dev/sda5[/root]                    btrfs      rw,relatime,seclabel,compress=zstd:1,space_cache,subvolid=256,subvol=/root
├─/proc                       proc                                proc       rw,nosuid,nodev,noexec,relatime
│ └─/proc/sys/fs/binfmt_misc  systemd-1                           autofs     rw,relatime,fd=36,pgrp=1,timeout=0,minproto=5,maxproto=5,direct,pipe_ino=15352
├─/sys                        sysfs                               sysfs      rw,nosuid,nodev,noexec,relatime,seclabel
│ ├─/sys/kernel/security      securityfs                          securityfs rw,nosuid,nodev,noexec,relatime
│ ├─/sys/fs/cgroup            cgroup2                             cgroup2    rw,nosuid,nodev,noexec,relatime,seclabel,nsdelegate,memory_recursiveprot
│ ├─/sys/fs/pstore            pstore                              pstore     rw,nosuid,nodev,noexec,relatime,seclabel
│ ├─/sys/firmware/efi/efivars efivarfs                            efivarfs   rw,nosuid,nodev,noexec,relatime
│ ├─/sys/fs/bpf               none                                bpf        rw,nosuid,nodev,noexec,relatime,mode=700
│ ├─/sys/fs/selinux           selinuxfs                           selinuxfs  rw,nosuid,noexec,relatime
│ ├─/sys/kernel/debug         debugfs                             debugfs    rw,nosuid,nodev,noexec,relatime,seclabel
│ ├─/sys/kernel/tracing       tracefs                             tracefs    rw,nosuid,nodev,noexec,relatime,seclabel
│ ├─/sys/fs/fuse/connections  fusectl                             fusectl    rw,nosuid,nodev,noexec,relatime
│ └─/sys/kernel/config        configfs                            configfs   rw,nosuid,nodev,noexec,relatime
├─/dev                        devtmpfs                            devtmpfs   rw,nosuid,seclabel,size=1851656k,nr_inodes=462914,mode=755,inode64
│ ├─/dev/shm                  tmpfs                               tmpfs      rw,nosuid,nodev,seclabel,inode64
│ ├─/dev/pts                  devpts                              devpts     rw,nosuid,noexec,relatime,seclabel,gid=5,mode=620,ptmxmode=000
│ ├─/dev/hugepages            hugetlbfs                           hugetlbfs  rw,relatime,seclabel,pagesize=2M
│ └─/dev/mqueue               mqueue                              mqueue     rw,nosuid,nodev,noexec,relatime,seclabel
├─/run                        tmpfs                               tmpfs      rw,nosuid,nodev,seclabel,size=749372k,nr_inodes=819200,mode=755,inode64
│ └─/run/user/0               tmpfs                               tmpfs      rw,nosuid,nodev,relatime,seclabel,size=374684k,nr_inodes=93671,mode=700,inode64
├─/boot                       /dev/sda2                           ext4       rw,relatime,seclabel
│ └─/boot/efi                 /dev/sda1                           vfat       rw,relatime,fmask=0077,dmask=0077,codepage=437,iocharset=ascii,shortname=winnt,errors=remount-ro
├─/home                       /dev/sda5[/home]                    btrfs      rw,relatime,seclabel,compress=zstd:1,space_cache,subvolid=258,subvol=/home
├─/tmp                        tmpfs                               tmpfs      rw,nosuid,nodev,seclabel,size=1873428k,nr_inodes=409600,inode64
├─/var/lib/nfs/rpc_pipefs     sunrpc                              rpc_pipefs rw,relatime
├─/var/lib/snapd/snap         /dev/sda5[/root/var/lib/snapd/snap] btrfs      rw,relatime,seclabel,compress=zstd:1,space_cache,subvolid=256,subvol=/root
└─/var/snap                   /dev/sda5[/root/var/snap]           btrfs      rw,relatime,seclabel,compress=zstd:1,space_cache,subvolid=256,subvol=/root

The path-in-filesystem for bind mounts done on btrfs are a bit different from
what we normally expect and include the subvolume ID as a prefix, eg:

749 63 0:31 /root/var/lib/snapd/snap /var/lib/snapd/snap rw,relatime shared:1 - btrfs /dev/sda5 rw,seclabel,compress=zstd:1,space_cache,subvolid=256,subvol=/root

Tweak the grep expression to match that.

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

bboozzoo commented Dec 8, 2021

What's left to fix is the SELinux denials that popped up after merging current master. I've fixed those in #11158

Copy link
Collaborator

@bboozzoo bboozzoo left a comment

Choose a reason for hiding this comment

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

LGTM for the bits I didn't touch

@mvo5 mvo5 merged commit f56272b into snapcore:master Dec 9, 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