Merge lp:~renatofilho/sync-monitor/fix-1605122 into lp:sync-monitor
- fix-1605122
- Merge into trunk
Status: | Merged | ||||
---|---|---|---|---|---|
Approved by: | Bill Filler | ||||
Approved revision: | 74 | ||||
Merged at revision: | 76 | ||||
Proposed branch: | lp:~renatofilho/sync-monitor/fix-1605122 | ||||
Merge into: | lp:sync-monitor | ||||
Prerequisite: | lp:~renatofilho/sync-monitor/fix-pot-build | ||||
Diff against target: |
58 lines (+19/-9) 2 files modified
src/sync-account.cpp (+8/-2) src/sync-daemon.cpp (+11/-7) |
||||
To merge this branch: | bzr merge lp:~renatofilho/sync-monitor/fix-1605122 | ||||
Related bugs: |
|
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
system-apps-ci-bot | continuous-integration | Approve | |
Ubuntu Phablet Team | Pending | ||
Review via email: mp+304546@code.launchpad.net |
This proposal supersedes a proposal from 2016-07-27.
Commit message
Retry sync in case of error 500.
Description of the change
system-apps-ci-bot (system-apps-ci-bot) wrote : Posted in a previous version of this proposal | # |
system-apps-ci-bot (system-apps-ci-bot) wrote : Posted in a previous version of this proposal | # |
FAILED: Continuous integration, rev:69
https:/
Executed test runs:
FAILURE: https:/
FAILURE: https:/
Click here to trigger a rebuild:
https:/
system-apps-ci-bot (system-apps-ci-bot) wrote : Posted in a previous version of this proposal | # |
FAILED: Continuous integration, rev:70
https:/
Executed test runs:
SUCCESS: https:/
FAILURE: https:/
SUCCESS: https:/
SUCCESS: https:/
SUCCESS: https:/
SUCCESS: https:/
SUCCESS: https:/
deb: https:/
SUCCESS: https:/
deb: https:/
SUCCESS: https:/
deb: https:/
SUCCESS: https:/
deb: https:/
SUCCESS: https:/
deb: https:/
SUCCESS: https:/
deb: https:/
SUCCESS: https:/
deb: https:/
SUCCESS: https:/
deb: https:/
SUCCESS: https:/
deb: https:/
Click here to trigger a rebuild:
https:/
system-apps-ci-bot (system-apps-ci-bot) wrote : Posted in a previous version of this proposal | # |
PASSED: Continuous integration, rev:71
https:/
Executed test runs:
SUCCESS: https:/
SUCCESS: https:/
SUCCESS: https:/
SUCCESS: https:/
SUCCESS: https:/
SUCCESS: https:/
SUCCESS: https:/
deb: https:/
SUCCESS: https:/
deb: https:/
SUCCESS: https:/
deb: https:/
SUCCESS: https:/
deb: https:/
SUCCESS: https:/
deb: https:/
SUCCESS: https:/
deb: https:/
SUCCESS: https:/
deb: https:/
SUCCESS: https:/
deb: https:/
SUCCESS: https:/
deb: https:/
Click here to trigger a rebuild:
https:/
system-apps-ci-bot (system-apps-ci-bot) wrote : Posted in a previous version of this proposal | # |
PASSED: Continuous integration, rev:72
https:/
Executed test runs:
SUCCESS: https:/
SUCCESS: https:/
SUCCESS: https:/
SUCCESS: https:/
SUCCESS: https:/
SUCCESS: https:/
SUCCESS: https:/
deb: https:/
SUCCESS: https:/
deb: https:/
SUCCESS: https:/
deb: https:/
SUCCESS: https:/
deb: https:/
SUCCESS: https:/
deb: https:/
SUCCESS: https:/
deb: https:/
SUCCESS: https:/
deb: https:/
SUCCESS: https:/
deb: https:/
SUCCESS: https:/
deb: https:/
Click here to trigger a rebuild:
https:/
- 74. By Renato Araujo Oliveira Filho
-
Merged: ~renatofilho/
sync-monitor/ fix-pot- build
system-apps-ci-bot (system-apps-ci-bot) wrote : | # |
PASSED: Continuous integration, rev:74
https:/
Executed test runs:
SUCCESS: https:/
SUCCESS: https:/
SUCCESS: https:/
SUCCESS: https:/
SUCCESS: https:/
SUCCESS: https:/
SUCCESS: https:/
deb: https:/
SUCCESS: https:/
deb: https:/
SUCCESS: https:/
deb: https:/
SUCCESS: https:/
deb: https:/
SUCCESS: https:/
deb: https:/
SUCCESS: https:/
deb: https:/
SUCCESS: https:/
deb: https:/
SUCCESS: https:/
deb: https:/
SUCCESS: https:/
deb: https:/
Click here to trigger a rebuild:
https:/
Preview Diff
1 | === modified file 'src/sync-account.cpp' |
2 | --- src/sync-account.cpp 2016-08-18 22:59:09 +0000 |
3 | +++ src/sync-account.cpp 2016-08-31 17:59:26 +0000 |
4 | @@ -269,10 +269,16 @@ |
5 | case 204: |
6 | case 207: |
7 | // status ok |
8 | + case 500: |
9 | + // server side generic error (just retry the sync) |
10 | + case 1500: |
11 | + // server side generic error (just retry the sync) |
12 | + case 403: |
13 | + // "Forbidden / access denied"; |
14 | + case 10403: |
15 | + // "Forbidden / access denied"; |
16 | return "two-way"; |
17 | case 401: |
18 | - case 403: |
19 | - // "Forbidden / access denied"; |
20 | case 404: |
21 | // "Object not found / unassigned field"; |
22 | case 405: |
23 | |
24 | === modified file 'src/sync-daemon.cpp' |
25 | --- src/sync-daemon.cpp 2016-07-07 19:56:56 +0000 |
26 | +++ src/sync-daemon.cpp 2016-08-31 17:59:26 +0000 |
27 | @@ -710,6 +710,7 @@ |
28 | // this is a bug on SyncEvolution sometimes it fail to read the correct address book |
29 | // FIXME: we should fix that on SyncEvolution |
30 | whiteListStatus << QStringLiteral("10500"); |
31 | + whiteListStatus << QStringLiteral("500"); |
32 | } |
33 | |
34 | // check if we are going re-sync due a know problem |
35 | @@ -731,13 +732,16 @@ |
36 | } else if (!errorMessage.isEmpty()) { |
37 | fail = true; |
38 | errorCode = 0; |
39 | - NotifyMessage *notify = new NotifyMessage(true, this); |
40 | - notify->show(_("Synchronization"), |
41 | - QString(_("Fail to sync calendar %1 from account %2.\n%3")) |
42 | - .arg(source) |
43 | - .arg(acc->displayName()) |
44 | - .arg(errorMessage), |
45 | - acc->iconName(CALENDAR_SERVICE_TYPE)); |
46 | + // only show error message if is the first sync or if error is not on whitelist |
47 | + if (firstSync || !whiteListStatus.contains(status)) { |
48 | + NotifyMessage *notify = new NotifyMessage(true, this); |
49 | + notify->show(_("Synchronization"), |
50 | + QString(_("Fail to sync calendar %1 from account %2.\n%3")) |
51 | + .arg(source) |
52 | + .arg(acc->displayName()) |
53 | + .arg(errorMessage), |
54 | + acc->iconName(CALENDAR_SERVICE_TYPE)); |
55 | + } |
56 | } |
57 | |
58 | if (saveLog && !source.isEmpty()) { |
PASSED: Continuous integration, rev:68 /jenkins. canonical. com/system- apps/job/ lp-sync- monitor- ci/12/ /jenkins. canonical. com/system- apps/job/ build/1048 /jenkins. canonical. com/system- apps/job/ test-0- autopkgtest/ label=phone- armhf,release= vivid+overlay, testname= default/ 207 /jenkins. canonical. com/system- apps/job/ build-0- fetch/1048 /jenkins. canonical. com/system- apps/job/ build-1- sourcepkg/ release= vivid+overlay/ 945 /jenkins. canonical. com/system- apps/job/ build-1- sourcepkg/ release= xenial+ overlay/ 945 /jenkins. canonical. com/system- apps/job/ build-1- sourcepkg/ release= yakkety/ 945 /jenkins. canonical. com/system- apps/job/ build-2- binpkg/ arch=amd64, release= vivid+overlay/ 936 /jenkins. canonical. com/system- apps/job/ build-2- binpkg/ arch=amd64, release= vivid+overlay/ 936/artifact/ output/ *zip*/output. zip /jenkins. canonical. com/system- apps/job/ build-2- binpkg/ arch=amd64, release= xenial+ overlay/ 936 /jenkins. canonical. com/system- apps/job/ build-2- binpkg/ arch=amd64, release= xenial+ overlay/ 936/artifact/ output/ *zip*/output. zip /jenkins. canonical. com/system- apps/job/ build-2- binpkg/ arch=amd64, release= yakkety/ 936 /jenkins. canonical. com/system- apps/job/ build-2- binpkg/ arch=amd64, release= yakkety/ 936/artifact/ output/ *zip*/output. zip /jenkins. canonical. com/system- apps/job/ build-2- binpkg/ arch=armhf, release= vivid+overlay/ 936 /jenkins. canonical. com/system- apps/job/ build-2- binpkg/ arch=armhf, release= vivid+overlay/ 936/artifact/ output/ *zip*/output. zip /jenkins. canonical. com/system- apps/job/ build-2- binpkg/ arch=armhf, release= xenial+ overlay/ 936 /jenkins. canonical. com/system- apps/job/ build-2- binpkg/ arch=armhf, release= xenial+ overlay/ 936/artifact/ output/ *zip*/output. zip /jenkins. canonical. com/system- apps/job/ build-2- binpkg/ arch=armhf, release= yakkety/ 936 /jenkins. canonical. com/system- apps/job/ build-2- binpkg/ arch=armhf, release= yakkety/ 936/artifact/ output/ *zip*/output. zip /jenkins. canonical. com/system- apps/job/ build-2- binpkg/ arch=i386, release= vivid+overlay/ 936 /jenkins. canonical. com/system- apps/job/ build-2- binpkg/ arch=i386, release= vivid+overlay/ 936/artifact/ output/ *zip*/output. zip /jenkins. canonical. com/system- apps/job/ build-2- binpkg/ arch=i386, release= xenial+ overlay/ 936 /jenkins. canonical. com/system- apps/job/ build-2- binpkg/ arch=i386, release= xenial+ overlay/ 936/artifact/ output/ *zip*/output. zip /jenkins. canonical. com/system- apps/job/ build-2- binpkg/ arch=i386, release= yakkety/ 936 /jenkins. canonical. com/system- apps/job/ build-2- binpkg/ arch=i386, release= yakkety/ 936/artifact/ output/ *zip*/output. zip
https:/
Executed test runs:
SUCCESS: https:/
SUCCESS: https:/
SUCCESS: https:/
SUCCESS: https:/
SUCCESS: https:/
SUCCESS: https:/
SUCCESS: https:/
deb: https:/
SUCCESS: https:/
deb: https:/
SUCCESS: https:/
deb: https:/
SUCCESS: https:/
deb: https:/
SUCCESS: https:/
deb: https:/
SUCCESS: https:/
deb: https:/
SUCCESS: https:/
deb: https:/
SUCCESS: https:/
deb: https:/
SUCCESS: https:/
deb: https:/
Click here to trigger a rebuild: /jenkins. canonical. com/system- apps/job/ lp-sync- monitor- ci/12/rebuild
https:/