debian: chmod created config files to 0644 #1504

Closed
wants to merge 1 commit into
from

Conversation

Projects
None yet
2 participants
Contributor

zyga commented Jul 7, 2016

This is a shot in the dark, currently autopackage tests fail on yakkety
with systemd 230 and snapd > 2.0.2. One of the error messages found is
that systemd ignores the created environment files due to incorrect
permissions, presumably causing various other things to misbehave.

Fixes: https://bugs.launchpad.net/ubuntu/+source/snapd/+bug/1599799
Signed-off-by: Zygmunt Krynicki zygmunt.krynicki@canonical.com

Contributor

snappy-m-o commented Jul 7, 2016

Can one of the admins verify this patch?

debian: chmod created config files to 0644
This is a shot in the dark, currently autopackage tests fail on yakkety
with systemd 230 and snapd > 2.0.2. One of the error messages found is
that systemd ignores the created environment files due to incorrect
permissions, presumably causing various other things to misbehave.

Fixes: https://bugs.launchpad.net/ubuntu/+source/snapd/+bug/1599799
Signed-off-by: Zygmunt Krynicki <zygmunt.krynicki@canonical.com>
Contributor

zyga commented Jul 7, 2016

Nope, doesn't help:

integration-tests/tests/service_test.go:90:
    installSnapWithService(c)
integration-tests/testutils/cli/cli.go:42:
    c.Assert(err, check.IsNil, check.Commentf("Error for %v: %v", cmds, output))
... value *exec.ExitError = &exec.ExitError{ProcessState:(*os.ProcessState)(0xc8202b4f80), Stderr:[]uint8(nil)} ("exit status 1")
... Error for [sudo snap install integration-tests/data/snaps/basic-service/basic-service_1.0_all.
error: cannot perform the following tasks:
- Mount snap "basic-service" ([start snap-basic\x2dservice-x1.mount] failed with exit status 1: Warning: snap-basic\x2dservice-x1.mount changed on disk. Run 'systemctl daemon-reload' to reload units.
A dependency job for snap-basic\x2dservice-x1.mount failed. See 'journalctl -xe' for details.
)


START: integration-tests/tests/service_test.go:48: serviceSuite.TearDownTest
integration-tests/tests/service_test.go:50:
    common.RemoveSnap(c, data.BasicServiceSnapName)
integration-tests/testutils/cli/cli.go:42:
    c.Assert(err, check.IsNil, check.Commentf("Error for %v: %v", cmds, output))
... value *exec.ExitError = &exec.ExitError{ProcessState:(*os.ProcessState)(0xc820307640), Stderr:[]uint8(nil)} ("exit status 1")
... Error for [sudo snap remove basic-service]: error: cannot remove "basic-service": cannot find snap "basic-service"

@zyga zyga closed this Jul 7, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment