Merge lp:~ahayzen/music-app/remix-confinement-back-002 into lp:music-app/remix

Proposed by Andrew Hayzen
Status: Merged
Approved by: Victor Thompson
Approved revision: 672
Merged at revision: 681
Proposed branch: lp:~ahayzen/music-app/remix-confinement-back-002
Merge into: lp:music-app/remix
Diff against target: 23 lines (+12/-2)
1 file modified
click/apparmor.json (+12/-2)
To merge this branch: bzr merge lp:~ahayzen/music-app/remix-confinement-back-002
Reviewer Review Type Date Requested Status
Victor Thompson Approve
Ubuntu Phone Apps Jenkins Bot continuous-integration Approve
Review via email: mp+238638@code.launchpad.net

Commit message

* Add back confinement

Description of the change

* Add back confinement

Please test thoroughly, especially content-hub, url-dispatcher, thumbnailer.

To post a comment you must log in.
Revision history for this message
Andrew Hayzen (ahayzen) wrote :

Note we were waiting for bug 1381102 to land, it has in utopic not so sure about rtm.

Revision history for this message
Ubuntu Phone Apps Jenkins Bot (ubuntu-phone-apps-jenkins-bot) wrote :
review: Approve (continuous-integration)
Revision history for this message
Andrew Hayzen (ahayzen) wrote :

Tested in rtm, only the following denials appear.

Oct 20 04:41:08 ubuntu-phablet kernel: [17638.562210] type=1400 audit(1413776468.504:177): apparmor="DENIED" operation="open" profile="com.ubuntu.music_music_1.3.672" name="/home/phablet/.cache/QML/Apps/" pid=28161 comm="qmlscene" requested_mask="r" denied_mask="r" fsuid=32011 ouid=32011
Oct 20 04:41:11 ubuntu-phablet dbus[1586]: apparmor="DENIED" operation="dbus_bind" bus="session" name="org.freedesktop.Application" mask="bind" pid=28161 profile="com.ubuntu.music_music_1.3.672"

$ system-image-cli -i
current build number: 99
device name: mako
channel: ubuntu-touch/ubuntu-rtm/devel-proposed
alias: ubuntu-touch/ubuntu-rtm/14.09-proposed
last update: 2014-10-19 07:24:09
version version: 99
version ubuntu: 20141019
version device: 20141017.2
version custom: mako-1.1

Revision history for this message
Victor Thompson (vthompson) wrote :

Jdstrand has stated that a bug exists for the error we are seeing (lp:1381620) and that it is essentially noise.

Looks great! We are now mere mortals again--sort of.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'click/apparmor.json'
2--- click/apparmor.json 2014-07-01 23:13:50 +0000
3+++ click/apparmor.json 2014-10-17 00:18:21 +0000
4@@ -1,7 +1,17 @@
5 {
6 "policy_version": 1.2,
7- "template": "unconfined",
8 "policy_groups": [
9- "content_exchange"
10+ "audio",
11+ "content_exchange",
12+ "music_files_read",
13+ "networking",
14+ "usermetrics"
15+ ],
16+ "read_path": [
17+ "@{HOME}/.cache/media-art/",
18+ "@{HOME}/.cache/mediascanner-2.0/"
19+ ],
20+ "write_path": [
21+ "@{HOME}/Music/Imported/"
22 ]
23 }

Subscribers

People subscribed via source and target branches