Merge lp:~ted/ubuntu-app-launch/zg-no-error into lp:ubuntu-app-launch/15.10
Proposed by
Ted Gould
Status: | Merged |
---|---|
Approved by: | Charles Kerr |
Approved revision: | 196 |
Merged at revision: | 199 |
Proposed branch: | lp:~ted/ubuntu-app-launch/zg-no-error |
Merge into: | lp:ubuntu-app-launch/15.10 |
Diff against target: |
11 lines (+1/-1) 1 file modified
zg-report-app.c (+1/-1) |
To merge this branch: | bzr merge lp:~ted/ubuntu-app-launch/zg-no-error |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Charles Kerr (community) | Approve | ||
PS Jenkins bot (community) | continuous-integration | Approve | |
Review via email: mp+260297@code.launchpad.net |
Commit message
Don't error on ZG failure
Description of the change
We're overreacting here. It seems that ZG will detect this error and clean up the dirty DB when it happens, so it's not a big deal. We should still report it, but it's not worth an entry in errors.u.c to track it.
To post a comment you must log in.
PASSED: Continuous integration, rev:196 jenkins. qa.ubuntu. com/job/ ubuntu- app-launch- ci/2/ jenkins. qa.ubuntu. com/job/ ubuntu- app-launch- wily-amd64- ci/2 jenkins. qa.ubuntu. com/job/ ubuntu- app-launch- wily-armhf- ci/2 jenkins. qa.ubuntu. com/job/ ubuntu- app-launch- wily-i386- ci/2
http://
Executed test runs:
SUCCESS: http://
SUCCESS: http://
SUCCESS: http://
Click here to trigger a rebuild: s-jenkins. ubuntu- ci:8080/ job/ubuntu- app-launch- ci/2/rebuild
http://