/snapd

skills/types: teach bool-file about udev security #489

Merged
merged 2 commits into from Feb 23, 2016

Conversation

Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
3 participants
@zyga @mvo5 @niemeyer
@zyga
Contributor

zyga commented Feb 16, 2016

Since bool-file doesn't need any additional rules all the change does is
recognizes udev and returns nil snippets. Tests are extended to check
for this condition.

Signed-off-by: Zygmunt Krynicki zygmunt.krynicki@canonical.com
@zyga
skills/types: teach bool-file about udev security 
Since bool-file doesn't need any additional rules all the change does is
recognizes udev and returns nil snippets. Tests are extended to check
for this condition.

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

Some checks were not successful

1 failing and 2 successful checks
@snappy-m-o
Integration tests — 19 tests run, 0 skipped, 7 failed.
Details
continuous-integration/travis-ci/pr — The Travis CI build passed
Details
coverage/coveralls — Coverage decreased (-0.0%) to 70.13%
Details
3dc457d

@zyga zyga referenced this pull request Feb 16, 2016

Merged

skills,skills/types: pass slot to SlotSecuritySnippet() #490

@mvo5
Collaborator

mvo5 commented Feb 22, 2016

Looks fine, 👍
@zyga
Merge branch 'master' of github.com:ubuntu-core/snappy into skills-bo… 
…ol-file-udev

All checks have passed

3 successful checks
@snappy-m-o
Integration tests — Success 64 tests run, 0 skipped, 0 failed.
Details
continuous-integration/travis-ci/pr — The Travis CI build passed
Details
coverage/coveralls — First build on master at 70.276%
Details
fb866a4
@zyga
Contributor

zyga commented Feb 23, 2016

Merged with master to re-test with integration test fixes
@niemeyer
Contributor

niemeyer commented Feb 23, 2016

👍

niemeyer added a commit that referenced this pull request Feb 23, 2016

@niemeyer
Merge pull request #489 from zyga/skills-bool-file-udev 
skills/types: teach bool-file about udev security
10d1ef7

@niemeyer niemeyer merged commit 10d1ef7 into snapcore:master Feb 23, 2016
3 checks passed

3 checks passed

Integration tests Success 64 tests run, 0 skipped, 0 failed.
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
coverage/coveralls First build on master at 70.276%
Details

@zyga zyga deleted the zyga:skills-bool-file-udev branch Mar 8, 2016

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