Merge lp://staging/~pwlars/ubuntu-ci-services-itself/private-images into lp://staging/ubuntu-ci-services-itself

Proposed by Paul Larson
Status: Merged
Approved by: Paul Larson
Approved revision: 307
Merged at revision: 310
Proposed branch: lp://staging/~pwlars/ubuntu-ci-services-itself/private-images
Merge into: lp://staging/ubuntu-ci-services-itself
Diff against target: 12 lines (+1/-1)
1 file modified
ci-utils/ci_utils/image_store/__init__.py (+1/-1)
To merge this branch: bzr merge lp://staging/~pwlars/ubuntu-ci-services-itself/private-images
Reviewer Review Type Date Requested Status
Ursula Junque (community) Approve
PS Jenkins bot (community) continuous-integration Approve
Review via email: mp+209342@code.staging.launchpad.net

Commit message

Make images uploaded private by default.

Description of the change

We don't all have rights to make images public, so image upload will fail without this. It can still be overridden if you really want an image to be public, and have the rights to do that.

To post a comment you must log in.
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :

PASSED: Continuous integration, rev:307
http://s-jenkins.ubuntu-ci:8080/job/uci-engine-ci/292/
Executed test runs:

Click here to trigger a rebuild:
http://s-jenkins.ubuntu-ci:8080/job/uci-engine-ci/292/rebuild

review: Approve (continuous-integration)
Revision history for this message
Ursula Junque (ursinha) wrote :

+1, I've tested this change by updating the code manually and restarting imagebuild_worker and it worked fine.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches