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
=== modified file 'docs/frameworks.md'
--- docs/frameworks.md 2015-06-03 11:48:37 +0000
+++ docs/frameworks.md 2015-10-26 21:32:22 +0000
@@ -162,7 +162,7 @@
162 description: "desc for qux service"162 description: "desc for qux service"
163 start: bin/qux163 start: bin/qux
164 caps:164 caps:
165 - networking165 - network-client
166 - foo_bar-client166 - foo_bar-client
167167
168See `security.md` for more information on specifying `caps` and a168See `security.md` for more information on specifying `caps` and a
169169
=== modified file 'snappy/security.go'
--- snappy/security.go 2015-09-25 15:27:11 +0000
+++ snappy/security.go 2015-10-26 21:32:22 +0000
@@ -52,7 +52,7 @@
5252
53const defaultTemplate = "default"53const defaultTemplate = "default"
5454
55var defaultPolicyGroups = []string{"networking"}55var defaultPolicyGroups = []string{"network-client"}
5656
57// TODO: autodetect, this won't work for personal57// TODO: autodetect, this won't work for personal
58const defaultPolicyVendor = "ubuntu-core"58const defaultPolicyVendor = "ubuntu-core"
5959
=== modified file 'snappy/security_test.go'
--- snappy/security_test.go 2015-09-25 15:27:11 +0000
+++ snappy/security_test.go 2015-10-26 21:32:22 +0000
@@ -90,7 +90,7 @@
90 a.verifyApparmorFile(c, `{90 a.verifyApparmorFile(c, `{
91 "template": "default",91 "template": "default",
92 "policy_groups": [92 "policy_groups": [
93 "networking"93 "network-client"
94 ],94 ],
95 "policy_vendor": "ubuntu-core",95 "policy_vendor": "ubuntu-core",
96 "policy_version": 15.0496 "policy_version": 15.04
@@ -221,7 +221,7 @@
221 "--policy-vendor=ubuntu-core",221 "--policy-vendor=ubuntu-core",
222 "--policy-version=15.04",222 "--policy-version=15.04",
223 "--template=something",223 "--template=something",
224 "--policy-groups=networking",224 "--policy-groups=network-client",
225 })225 })
226}226}
227227
228228
=== modified file 'snappy/snapp_test.go'
--- snappy/snapp_test.go 2015-10-20 07:03:39 +0000
+++ snappy/snapp_test.go 2015-10-26 21:32:22 +0000
@@ -1018,7 +1018,7 @@
1018 stop: bin/testme-service.stop1018 stop: bin/testme-service.stop
1019 description: "testme service"1019 description: "testme service"
1020 caps:1020 caps:
1021 - "networking"1021 - "network-client"
1022 - "foo_group"1022 - "foo_group"
1023 security-template: "foo_template"1023 security-template: "foo_template"
1024`)1024`)
@@ -1031,7 +1031,7 @@
1031 c.Assert(m.ServiceYamls[0].Start, Equals, "bin/testme-service.start")1031 c.Assert(m.ServiceYamls[0].Start, Equals, "bin/testme-service.start")
1032 c.Assert(m.ServiceYamls[0].Stop, Equals, "bin/testme-service.stop")1032 c.Assert(m.ServiceYamls[0].Stop, Equals, "bin/testme-service.stop")
1033 c.Assert(m.ServiceYamls[0].SecurityCaps, HasLen, 2)1033 c.Assert(m.ServiceYamls[0].SecurityCaps, HasLen, 2)
1034 c.Assert(m.ServiceYamls[0].SecurityCaps[0], Equals, "networking")1034 c.Assert(m.ServiceYamls[0].SecurityCaps[0], Equals, "network-client")
1035 c.Assert(m.ServiceYamls[0].SecurityCaps[1], Equals, "foo_group")1035 c.Assert(m.ServiceYamls[0].SecurityCaps[1], Equals, "foo_group")
1036 c.Assert(m.ServiceYamls[0].SecurityTemplate, Equals, "foo_template")1036 c.Assert(m.ServiceYamls[0].SecurityTemplate, Equals, "foo_template")
1037}1037}

Subscribers

People subscribed via source and target branches