Merge lp://staging/~gz/charms/precise/juju-reports/cidata-url into lp://staging/~juju-qa/charms/precise/juju-reports/trunk
Proposed by
Martin Packman
| Status: | Merged |
|---|---|
| Approved by: | Martin Packman |
| Approved revision: | 49 |
| Merged at revision: | 49 |
| Proposed branch: | lp://staging/~gz/charms/precise/juju-reports/cidata-url |
| Merge into: | lp://staging/~juju-qa/charms/precise/juju-reports/trunk |
| Diff against target: |
47 lines (+9/-3) 2 files modified
config.yaml (+4/-0) hooks/common.py (+5/-3) |
| To merge this branch: | bzr merge lp://staging/~gz/charms/precise/juju-reports/cidata-url |
| Related bugs: |
| Reviewer | Review Type | Date Requested | Status |
|---|---|---|---|
| Aaron Bentley (community) | Approve | ||
|
Review via email:
|
|||
Description of the change
Adds new config value for the location of the S3 mirror.
Maybe quibble over whether we want all the path like this, or just the scheme and domain?
To post a comment you must log in.
Revision history for this message
| Aaron Bentley (abentley) | # |
review:
Approve
Preview Diff
[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
