Merge ~canonical-kernel-team/+git/overlay-shiftfs-tests:systemd-tmpfiles-race into ~canonical-kernel-team/+git/overlay-shiftfs-tests:main

Proposed by Thadeu Lima de Souza Cascardo
Status: Merged
Merge reported by: Po-Hsu Lin
Merged at revision: d9b1b680581c4142eb0aa23189852b29ce3e2612
Proposed branch: ~canonical-kernel-team/+git/overlay-shiftfs-tests:systemd-tmpfiles-race
Merge into: ~canonical-kernel-team/+git/overlay-shiftfs-tests:main
Diff against target: 47 lines (+15/-14)
1 file modified
tests/ovl-refcnt.sh (+15/-14)
Reviewer Review Type Date Requested Status
Po-Hsu Lin Approve
Sean Feole Approve
Review via email: mp+454970@code.launchpad.net

Description of the change

I see a failure on our test that look very similar to the one we had for ovl-mkdir-xattrs. Trying to apply the same type of fix here.

To post a comment you must log in.
Revision history for this message
Sean Feole (sfeole) wrote :

+1

review: Approve
Revision history for this message
Po-Hsu Lin (cypressyew) wrote :

Looking good, tested on a VM and it has passed.
Thanks!

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1diff --git a/tests/ovl-refcnt.sh b/tests/ovl-refcnt.sh
2index bb32f01..36baead 100755
3--- a/tests/ovl-refcnt.sh
4+++ b/tests/ovl-refcnt.sh
5@@ -9,27 +9,28 @@ if [ ! -x "$(which lxc)" ]; then
6 fi
7
8 cat > test.sh <<EOF
9+dir=`mktemp -p "${HOME}" -d`
10 if [ "$(mount -t proc | grep '/proc')" == "" ]; then
11 mount -t proc proc /proc
12 fi
13-mkdir -p /tmp/ovlissue/mountpointtmp
14-mount -t tmpfs tmpfs /tmp/ovlissue/mountpointtmp
15+mkdir -p $dir/ovlissue/mountpointtmp
16+mount -t tmpfs tmpfs $dir/ovlissue/mountpointtmp
17
18-mkdir -p /tmp/ovlissue/mountpointtmp/firstupperdir
19-mkdir -p /tmp/ovlissue/mountpointtmp/firstworkdir
20-mkdir -p /tmp/ovlissue/mountpointtmp/firstunion
21-mount -t overlay overlay -o lowerdir=/,upperdir=/tmp/ovlissue/mountpointtmp/firstupperdir,workdir=/tmp/ovlissue/mountpointtmp/firstworkdir /tmp/ovlissue/mountpointtmp/firstunion
22+mkdir -p $dir/ovlissue/mountpointtmp/firstupperdir
23+mkdir -p $dir/ovlissue/mountpointtmp/firstworkdir
24+mkdir -p $dir/ovlissue/mountpointtmp/firstunion
25+mount -t overlay overlay -o lowerdir=/,upperdir=$dir/ovlissue/mountpointtmp/firstupperdir,workdir=$dir/ovlissue/mountpointtmp/firstworkdir $dir/ovlissue/mountpointtmp/firstunion
26
27-mkdir -p /tmp/ovlissue/mountpointtmp/firstunion/tmp/ovlissue/mountpointtmp
28-mount --bind /tmp/ovlissue/mountpointtmp /tmp/ovlissue/mountpointtmp/firstunion/tmp/ovlissue/mountpointtmp
29+mkdir -p $dir/ovlissue/mountpointtmp/firstunion/tmp/ovlissue/mountpointtmp
30+mount --bind $dir/ovlissue/mountpointtmp $dir/ovlissue/mountpointtmp/firstunion/tmp/ovlissue/mountpointtmp
31
32-mkdir /tmp/ovlissue/mountpointtmp/firstunion/tmp/ovlissue/mountpointtmp/secondupperdir
33-mkdir /tmp/ovlissue/mountpointtmp/firstunion/tmp/ovlissue/mountpointtmp/workdir
34-mkdir /tmp/ovlissue/mountpointtmp/firstunion/tmp/ovlissue/mountpointtmp/secondunion
35-mount -t overlay overlay -o lowerdir=/tmp/ovlissue/mountpointtmp/firstunion,upperdir=/tmp/ovlissue/mountpointtmp/firstunion/tmp/ovlissue/mountpointtmp/secondupperdir,workdir=/tmp/ovlissue/mountpointtmp/firstunion/tmp/ovlissue/mountpointtmp/workdir /tmp/ovlissue/mountpointtmp/firstunion/tmp/ovlissue/mountpointtmp/secondunion
36+mkdir $dir/ovlissue/mountpointtmp/firstunion/tmp/ovlissue/mountpointtmp/secondupperdir
37+mkdir $dir/ovlissue/mountpointtmp/firstunion/tmp/ovlissue/mountpointtmp/workdir
38+mkdir $dir/ovlissue/mountpointtmp/firstunion/tmp/ovlissue/mountpointtmp/secondunion
39+mount -t overlay overlay -o lowerdir=$dir/ovlissue/mountpointtmp/firstunion,upperdir=$dir/ovlissue/mountpointtmp/firstunion/tmp/ovlissue/mountpointtmp/secondupperdir,workdir=$dir/ovlissue/mountpointtmp/firstunion/tmp/ovlissue/mountpointtmp/workdir $dir/ovlissue/mountpointtmp/firstunion/tmp/ovlissue/mountpointtmp/secondunion
40
41-chroot /tmp/ovlissue/mountpointtmp/firstunion/tmp/ovlissue/mountpointtmp/secondunion true
42-umount /tmp/ovlissue/mountpointtmp/firstunion/tmp/ovlissue/mountpointtmp/secondunion
43+chroot $dir/ovlissue/mountpointtmp/firstunion/tmp/ovlissue/mountpointtmp/secondunion true
44+umount $dir/ovlissue/mountpointtmp/firstunion/tmp/ovlissue/mountpointtmp/secondunion
45 EOF
46
47 sudo lxd init --auto

Subscribers

People subscribed via source and target branches

to all changes: