Merge lp://staging/~doanac/ubuntu-ci-services-itself/bug1285386-cleaner-not-starting into lp://staging/ubuntu-ci-services-itself
Proposed by
Andy Doan
| Status: | Merged | ||||
|---|---|---|---|---|---|
| Approved by: | Andy Doan | ||||
| Approved revision: | 352 | ||||
| Merged at revision: | 352 | ||||
| Proposed branch: | lp://staging/~doanac/ubuntu-ci-services-itself/bug1285386-cleaner-not-starting | ||||
| Merge into: | lp://staging/ubuntu-ci-services-itself | ||||
| Diff against target: |
38 lines (+11/-0) 2 files modified
juju-deployer/production-only.yaml (+4/-0) ppa-assigner/ppa_assigner/api.py (+7/-0) |
||||
| To merge this branch: | bzr merge lp://staging/~doanac/ubuntu-ci-services-itself/bug1285386-cleaner-not-starting | ||||
| Related bugs: |
|
| Reviewer | Review Type | Date Requested | Status |
|---|---|---|---|
| PS Jenkins bot (community) | continuous-integration | Approve | |
| Francis Ginther | Approve | ||
|
Review via email:
|
|||
Commit message
ppa-cleaner: only start if ppa_django is running
We have seen a flaky issue where this daemon is started before
postgres is ready. This makes the daemon only start when django
is running which should only happen when postgres is ready.
It also provides a status check so the user can be aware of the issue
Description of the change
This hopefully fixes the problem we've seen where the ppa-cleaner isn't running. I also through in a diagnostic helper so people can more easily see this is happening.
To post a comment you must log in.
Revision history for this message
| PS Jenkins bot (ps-jenkins) wrote | # |
review:
Approve
(continuous-integration)
Preview Diff
[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
Looks good.