PartitionTestSuite.TestHandleAssetsNoHardwareYaml fails: "open /writable/cache/hardware.yaml: permission denied
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Snappy |
Fix Released
|
Undecided
|
Leo Arias |
Bug Description
One one of my machines I started getting one failure from the unit tests:
FAIL: bootloader_
FAIL: bootloader_
bootloader_
c.Assert(
... value *os.PathError = &os.PathError{
On my other machine it doesn't fail, so I went and looked into the permissions of that dir:
$ sudo ls -lah /writable
totalo 12K
drwxr-x--- 3 root root 4,0K Jun 2 04:53 .
drwxr-xr-x 24 root root 4,0K Jun 2 04:53 ..
drwxr-x--- 3 root root 4,0K Jun 2 04:53 cache
On the other machine, the directory is readable by my user.
Now, the problem is that we are relying in a /writable directory, that I'm not sure if will always exist. Looking at the test, I see that the directory is being patched with /tmp, but too late.
Related branches
- Michael Vogt (community): Approve
-
Diff: 17 lines (+2/-2)1 file modifiedpartition/bootloader_uboot_test.go (+2/-2)
- John Lenton (community): Approve
-
Diff: 11 lines (+1/-0)1 file modifiedpartition/bootloader_uboot_test.go (+1/-0)
Changed in snappy: | |
status: | In Progress → Fix Committed |
Changed in snappy: | |
status: | In Progress → Fix Committed |
Changed in snappy: | |
status: | Fix Committed → Fix Released |
I think that as a result of a bad merge here: https:/ /code.launchpad .net/~mvo/ snappy/ snappy- improved- developer- mode2/+ merge/261692
the patch of the cache dir was removed. So this test bug is back.