tests: copy snap-confine apparmor profile into testbed #3194

Merged
merged 2 commits into from Apr 13, 2017

Conversation

Projects
None yet
3 participants
Contributor

zyga commented Apr 13, 2017

Ever since we started executing snap-confine from the core snap we rely
on a copy of the apparmor profile from the Debian package to generate
the apparmor profile for snap-confine in the core snap.

My recent changes to the snap-confine required an update to the profile
and I noticed that all the test have failed because apparmor profile
in the core snap was not kept up-to-date.

Signed-off-by: Zygmunt Krynicki zygmunt.krynicki@canonical.com

tests: copy snap-confine apparmor profile into testbed
Ever since we started executing snap-confine from the core snap we rely
on a copy of the apparmor profile from the Debian package to generate
the apparmor profile for snap-confine in the core snap.

My recent changes to the snap-confine required an update to the profile
and I noticed that all the test have failed because apparmor profile
in the core snap was not kept up-to-date.

Signed-off-by: Zygmunt Krynicki <zygmunt.krynicki@canonical.com>

Approving, but it would be nice if this was closer to the snap-confine copy.

Contributor

zyga commented Apr 13, 2017

@jdstrand one of @chipaca's branches are converting that to cp -a with simpler set of files to manage. I will update it then.

ogra1 approved these changes Apr 13, 2017

looks fine

tests: use wildcard to cover both .real and plain
Signed-off-by: Zygmunt Krynicki <zygmunt.krynicki@canonical.com>

@zyga zyga merged commit 6b2d8cf into snapcore:master Apr 13, 2017

1 of 6 checks passed

xenial-amd64 autopkgtest running
Details
xenial-i386 autopkgtest running
Details
xenial-ppc64el autopkgtest running
Details
yakkety-amd64 autopkgtest running
Details
zesty-amd64 autopkgtest running
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details

@zyga zyga deleted the zyga:fix-aa-profile branch Apr 13, 2017

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