Merge lp://qastaging/~pwlars/ubuntu-ci-services-itself/no-service-start-on-install into lp://qastaging/ubuntu-ci-services-itself
Proposed by
Paul Larson
Status: | Merged |
---|---|
Approved by: | Andy Doan |
Approved revision: | 360 |
Merged at revision: | 365 |
Proposed branch: | lp://qastaging/~pwlars/ubuntu-ci-services-itself/no-service-start-on-install |
Merge into: | lp://qastaging/ubuntu-ci-services-itself |
Diff against target: |
39 lines (+16/-0) 1 file modified
image-builder/imagebuilder/cloud_image.py (+16/-0) |
To merge this branch: | bzr merge lp://qastaging/~pwlars/ubuntu-ci-services-itself/no-service-start-on-install |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
PS Jenkins bot (community) | continuous-integration | Approve | |
Andy Doan (community) | Approve | ||
Paul Larson | Needs Resubmitting | ||
Chris Johnston (community) | Needs Fixing | ||
Para Siva (community) | Approve | ||
Review via email:
|
Commit message
Create a policy file to prevent services from starting (Fixes LP:1289249)
Description of the change
This should fix up https:/
I'm not in a good position to run a full ticket through with this right now, but I've tested it by adding the same policy file, installing whoopsie, and verifying that whoopsie is prevented from starting.
To post a comment you must log in.
wow -this is great. i'll sleep better tonight.
textwrap.dedent could be nice for stuff like:
8 + policyfile="""\
9 +#!/bin/sh
10 + while true; do
11 + case "$1" in
12 + -*) shift ;;
13 + makedev) exit 0;;
14 + x11-common) exit 0;;
15 + *) exit 101;;
16 + esac
17 +done"""
but i know you are nearing vacation time, so feel free to merge as is if needed.