1.0.41-0ubuntu1 ftbfs on amd64, i386
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| snap-confine |
Fix Released
|
High
|
Zygmunt Krynicki | ||
| snap-confine (Ubuntu) |
Fix Released
|
Undecided
|
Unassigned | ||
| Xenial |
Fix Released
|
Undecided
|
Unassigned | ||
Bug Description
[Impact]
snap-confine contains sanity unit tests that check if the kernel behaves in a certain, particular way that subsequent runtime behaviour depends on. As it happens to be launchpad package building machinery runs on an older kernel with a xenial/yakkety chroot and the sanity test fails.
The relevant unit tests now check and parse kernel version and unless a recent enough version is available, those tests are skipped.
[Test Case]
N/A
Technically if someone wants to verify this change then it is sufficient to rebuild the package on 14.04 release (not the LTS updated) kernel like 3.19.
[Regression Potential]
None, this only affects unit tests in a safe way.
[Other Info]
* This bug is a part of a major SRU that brings snap-confine in Ubuntu 16.04 in line with the current upstream release 1.0.42.
* snap-confine is technically an integral part of snapd which has an SRU exception and is allowed to introduce new features and take advantage of accelerated procedure. For more information see https:/
== # Pre-SRU bug description follows # ==
Probably because of the old kernel version, a test fails like this:
/ns/sc_
/ns/sc_
**
ERROR:ns-
/ns/nsfs_fs_id: FAIL
12:44 < mwhudson> 40864 seems to be PROC_SUPER_MAGIC
12:44 < mwhudson> zyga: https:/
12:44 < zyga> mwhudson: thanks
12:45 < mwhudson> and NSFS_MAGIC was only added to the kernel in a commit from Sat Nov 1 10:57:28 2014 -0400
12:45 < mwhudson> so that's not going to be in the 14.04 release kernel
12:46 < mwhudson> 3.19+
12:46 < zyga> mwhudson: so even the kernel headers have that macro, the relevant files in the kernel don't use it?
12:47 < mwhudson> zyga: well the kernel headers you are building against are from yakkety presumably
12:47 < zyga> mwhudson: yes, I just checked that :/
12:47 < zyga> mwhudson: the mount namespace file in the kernel is indeed procfs
12:47 < zyga> mwhudson: I guess this test needs to be skipped
12:47 < zyga> mwhudson: and we might need a separate check for this in snap-confine proper
12:48 < zyga> so that if we open and see PROC_SUPER_MAGIC we can die()
| Michael Hudson-Doyle (mwhudson) wrote | #1 |
| Changed in snap-confine (Ubuntu): | |
| status: | New → In Progress |
| Michael Hudson-Doyle (mwhudson) wrote | #2 |
| description: | updated |
| Changed in snap-confine: | |
| status: | New → In Progress |
| assignee: | nobody → Zygmunt Krynicki (zyga) |
| milestone: | none → 1.0.42 |
| importance: | Undecided → High |
| Zygmunt Krynicki (zyga) wrote | #3 |
| Changed in snap-confine: | |
| status: | In Progress → Fix Committed |
| Changed in snap-confine (Ubuntu): | |
| status: | In Progress → Fix Released |
| Changed in snap-confine: | |
| status: | Fix Committed → Fix Released |
| Andy Whitcroft (apw) wrote Please test proposed package | #4 |
| Changed in snap-confine (Ubuntu Xenial): | |
| status: | New → Fix Committed |
| tags: | added: verification-needed |
| description: | updated |
| Andy Whitcroft (apw) wrote | #5 |
| Leo Arias (elopio) wrote | #6 |
| tags: |
added: verification-done removed: verification-needed |
| Launchpad Janitor (janitor) wrote | #7 |
| Changed in snap-confine (Ubuntu Xenial): | |
| status: | Fix Committed → Fix Released |
| Steve Langasek (vorlon) wrote Update Released | #8 |
40864 seems to be PROC_SUPER_MAGIC