Merge lp:~jdstrand/snappy/snappy.network-client into lp:~snappy-dev/snappy/snappy-moved-to-github

Proposed by Jamie Strandboge
Status: Needs review
Proposed branch: lp:~jdstrand/snappy/snappy.network-client
Merge into: lp:~snappy-dev/snappy/snappy-moved-to-github
Diff against target: 69 lines (+6/-6)
4 files modified
docs/frameworks.md (+1/-1)
snappy/security.go (+1/-1)
snappy/security_test.go (+2/-2)
snappy/snapp_test.go (+2/-2)
To merge this branch: bzr merge lp:~jdstrand/snappy/snappy.network-client
Reviewer Review Type Date Requested Status
Snappy Developers Pending
Review via email: mp+275771@code.launchpad.net

Commit message

Use the network-client cap instead of the old, deprecated networking cap. network-client was available on 15.04 so this is safe to backport to the stable branch as well.

Description of the change

Use the network-client cap instead of the old, deprecated networking cap. network-client was available on 15.04 so this is safe to backport to the stable branch as well.

To post a comment you must log in.
Revision history for this message
Michael Vogt (mvo) wrote :

Thanks for the branch! We move to github the other day but no worries, I converted the branch and created a github pull request: https://github.com/ubuntu-core/snappy/pull/25

Revision history for this message
Michael Vogt (mvo) wrote :

I commented in LP.

Revision history for this message
Michael Vogt (mvo) wrote :

Eh, commented in github.

Unmerged revisions

799. By Jamie Strandboge

use the network-client cap instead of the old, deprecated networking cap.
network-client was available on 15.04 so this is safe to backport to the
stable branch as well.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'docs/frameworks.md'
2--- docs/frameworks.md 2015-06-03 11:48:37 +0000
3+++ docs/frameworks.md 2015-10-26 21:32:22 +0000
4@@ -162,7 +162,7 @@
5 description: "desc for qux service"
6 start: bin/qux
7 caps:
8- - networking
9+ - network-client
10 - foo_bar-client
11
12 See `security.md` for more information on specifying `caps` and a
13
14=== modified file 'snappy/security.go'
15--- snappy/security.go 2015-09-25 15:27:11 +0000
16+++ snappy/security.go 2015-10-26 21:32:22 +0000
17@@ -52,7 +52,7 @@
18
19 const defaultTemplate = "default"
20
21-var defaultPolicyGroups = []string{"networking"}
22+var defaultPolicyGroups = []string{"network-client"}
23
24 // TODO: autodetect, this won't work for personal
25 const defaultPolicyVendor = "ubuntu-core"
26
27=== modified file 'snappy/security_test.go'
28--- snappy/security_test.go 2015-09-25 15:27:11 +0000
29+++ snappy/security_test.go 2015-10-26 21:32:22 +0000
30@@ -90,7 +90,7 @@
31 a.verifyApparmorFile(c, `{
32 "template": "default",
33 "policy_groups": [
34- "networking"
35+ "network-client"
36 ],
37 "policy_vendor": "ubuntu-core",
38 "policy_version": 15.04
39@@ -221,7 +221,7 @@
40 "--policy-vendor=ubuntu-core",
41 "--policy-version=15.04",
42 "--template=something",
43- "--policy-groups=networking",
44+ "--policy-groups=network-client",
45 })
46 }
47
48
49=== modified file 'snappy/snapp_test.go'
50--- snappy/snapp_test.go 2015-10-20 07:03:39 +0000
51+++ snappy/snapp_test.go 2015-10-26 21:32:22 +0000
52@@ -1018,7 +1018,7 @@
53 stop: bin/testme-service.stop
54 description: "testme service"
55 caps:
56- - "networking"
57+ - "network-client"
58 - "foo_group"
59 security-template: "foo_template"
60 `)
61@@ -1031,7 +1031,7 @@
62 c.Assert(m.ServiceYamls[0].Start, Equals, "bin/testme-service.start")
63 c.Assert(m.ServiceYamls[0].Stop, Equals, "bin/testme-service.stop")
64 c.Assert(m.ServiceYamls[0].SecurityCaps, HasLen, 2)
65- c.Assert(m.ServiceYamls[0].SecurityCaps[0], Equals, "networking")
66+ c.Assert(m.ServiceYamls[0].SecurityCaps[0], Equals, "network-client")
67 c.Assert(m.ServiceYamls[0].SecurityCaps[1], Equals, "foo_group")
68 c.Assert(m.ServiceYamls[0].SecurityTemplate, Equals, "foo_template")
69 }

Subscribers

People subscribed via source and target branches