Merge lp://staging/~kelvin.li/rnr-server/charm-supports into lp://staging/rnr-server

Proposed by Kelvin Li
Status: Merged
Approved by: Michael Nelson
Approved revision: 360
Merged at revision: 303
Proposed branch: lp://staging/~kelvin.li/rnr-server/charm-supports
Merge into: lp://staging/rnr-server
Diff against target: 20 lines (+2/-1)
1 file modified
django_project/settings_base.py (+2/-1)
To merge this branch: bzr merge lp://staging/~kelvin.li/rnr-server/charm-supports
Reviewer Review Type Date Requested Status
Wes Mason (community) Approve
Review via email: mp+262401@code.staging.launchpad.net

Commit message

emty revision to trigger rollout

Description of the change

emty revision to trigger rollout

To post a comment you must log in.
360. By Kelvin Li

changed the rnr.log generate location to fit log sync pattern in metal

Revision history for this message
Kelvin Li (kelvin.li) wrote :

In mental staging environment, the rnr.log was generated in /srv/xxx/staging-logs/rnr.log,(changed from revno290[0])
and that doesn't fit the metal sync pattern rules, we lost the rnr.log from 20141105[1]

[0]http://bazaar.launchpad.net/~rnr-developers/rnr-server/trunk/revision/290
[1]@vanadium:/srv/admin.isd.canonical.com/isd-logs/waxapple/reviews/www-logs

Revision history for this message
Kelvin Li (kelvin.li) wrote :

Rollout trigger is because we added the BzrRevnoMiddleware[0], which relies on the versioninfo that introduced for juju, it didn't existed in the metal env, we added it into the puppet build step and need to land it.

[0]http://bazaar.launchpad.net/~rnr-developers/rnr-server/trunk/view/head:/src/core/wsgi.py#L43

Revision history for this message
Wes Mason (wesmason) wrote :

lgtm

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

to all changes: