Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(69)

Issue 55300043: Upload custom image metadata (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
10 years, 3 months ago by wallyworld
Modified:
10 years, 3 months ago
Reviewers:
mp+202565, thumper
Visibility:
Public.

Description

Upload custom image metadata This branch automates the use of custom image metadata at bootstrap and at the same time solves the issue of bootstrapping against a private cloud without http access to cloud storage. The bootstrap --source parameter is renamed --metadata-source and the directory is used to look for both tools and image metadata. If image metadata is found, which is the case after running juju metadata generate-image, the metadata is uploaded to cloud storage which automatically adds it to the simplestreams search path. Previously, the user had to upload manually. https://code.launchpad.net/~wallyworld/juju-core/image-metadata-upload/+merge/202565 (do not edit description out of merge proposal)

Patch Set 1 #

Total comments: 2
Unified diffs Side-by-side diffs Delta from patch set Stats (+335 lines, -47 lines) Patch
A [revision details] View 1 chunk +2 lines, -0 lines 0 comments Download
M cmd/juju/bootstrap.go View 6 chunks +26 lines, -10 lines 0 comments Download
M cmd/juju/bootstrap_test.go View 4 chunks +51 lines, -2 lines 0 comments Download
M cmd/plugins/juju-metadata/imagemetadata.go View 3 chunks +8 lines, -8 lines 0 comments Download
M cmd/plugins/juju-metadata/toolsmetadata_test.go View 5 chunks +5 lines, -5 lines 0 comments Download
M environs/imagemetadata/generate.go View 2 chunks +3 lines, -2 lines 0 comments Download
M environs/imagemetadata/generate_test.go View 4 chunks +4 lines, -4 lines 0 comments Download
M environs/imagemetadata/testing/testing.go View 2 chunks +22 lines, -8 lines 0 comments Download
A environs/imagemetadata/upload.go View 1 chunk +62 lines, -0 lines 0 comments Download
A environs/imagemetadata/upload_test.go View 1 chunk +98 lines, -0 lines 2 comments Download
M environs/imagemetadata/urls.go View 2 chunks +33 lines, -2 lines 0 comments Download
M environs/imagemetadata/urls_test.go View 1 chunk +14 lines, -0 lines 0 comments Download
M environs/simplestreams/simplestreams.go View 3 chunks +4 lines, -3 lines 0 comments Download
M environs/tools/simplestreams_test.go View 3 chunks +3 lines, -3 lines 0 comments Download

Messages

Total messages: 3
wallyworld
Please take a look.
10 years, 3 months ago (2014-01-21 22:32:00 UTC) #1
thumper
LGTM - one suggestion, but not binding. https://codereview.appspot.com/55300043/diff/1/environs/imagemetadata/upload_test.go File environs/imagemetadata/upload_test.go (right): https://codereview.appspot.com/55300043/diff/1/environs/imagemetadata/upload_test.go#newcode29 environs/imagemetadata/upload_test.go:29: func createImageMetadata(c ...
10 years, 3 months ago (2014-01-22 04:41:51 UTC) #2
wallyworld
10 years, 3 months ago (2014-01-22 04:53:55 UTC) #3
https://codereview.appspot.com/55300043/diff/1/environs/imagemetadata/upload_...
File environs/imagemetadata/upload_test.go (right):

https://codereview.appspot.com/55300043/diff/1/environs/imagemetadata/upload_...
environs/imagemetadata/upload_test.go:29: func createImageMetadata(c *gc.C)
(string, string, storage.Storage, *imagemetadata.ImageMetadata) {
On 2014/01/22 04:41:51, thumper wrote:
> If you are returning multiple parameters is it often nice to name them so
> someone reading the signature can get it without having to look at the return
> statements.

Just for you, done.
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b