sync-monitor

Merge lp://qastaging/~renatofilho/sync-monitor/start-after-unity8 into lp://qastaging/sync-monitor

  1. start-after-unity8
  2. Merge into trunk
Proposed by Renato Araujo Oliveira Filho
Status: Merged
Approved by: Bill Filler
Approved revision: 47
Merged at revision: 39
Proposed branch: lp://qastaging/~renatofilho/sync-monitor/start-after-unity8
Merge into: lp://qastaging/sync-monitor
Prerequisite: lp://qastaging/~renatofilho/sync-monitor/check-network
Diff against target: 354 lines (+84/-66)
14 files modified
CMakeLists.txt (+1/-2)
Ubuntu/SyncMonitor/syncmonitor-qml.cpp (+2/-0)
data/CMakeLists.txt (+0/-8)
data/com.canonical.SyncMonitor.service.in (+0/-3)
debian/sync-monitor.install (+0/-1)
src/eds-helper.cpp (+40/-31)
src/eds-helper.h (+1/-0)
src/sync-daemon.cpp (+14/-3)
src/sync-daemon.h (+2/-0)
src/sync-dbus.cpp (+14/-1)
src/sync-dbus.h (+7/-0)
tests/unittest/eds-helper-mock.h (+3/-1)
upstart/CMakeLists.txt (+0/-7)
upstart/sync-monitor.conf.in (+0/-9)
To merge this branch: bzr merge lp://qastaging/~renatofilho/sync-monitor/start-after-unity8
Reviewer Review Type Date Requested Status
Bill Filler (community) Approve
PS Jenkins bot continuous-integration Approve
Review via email: mp+240278@code.qastaging.launchpad.net

This proposal supersedes a proposal from 2014-10-22.

Commit message

Do not sync on startup, only after a client registers with sync-monitor (apps, or online accounts).

To post a comment you must log in.
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote : Posted in a previous version of this proposal

PASSED: Continuous integration, rev:38
http://jenkins.qa.ubuntu.com/job/sync-monitor-ci/70/
Executed test runs:
    SUCCESS: http://jenkins.qa.ubuntu.com/job/sync-monitor-utopic-amd64-ci/18
    SUCCESS: http://jenkins.qa.ubuntu.com/job/sync-monitor-utopic-armhf-ci/18
        deb: http://jenkins.qa.ubuntu.com/job/sync-monitor-utopic-armhf-ci/18/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/sync-monitor-utopic-i386-ci/18

Click here to trigger a rebuild:
http://s-jenkins.ubuntu-ci:8080/job/sync-monitor-ci/70/rebuild

review: Approve (continuous-integration)
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote : Posted in a previous version of this proposal

PASSED: Continuous integration, rev:39
http://jenkins.qa.ubuntu.com/job/sync-monitor-ci/77/
Executed test runs:
    SUCCESS: http://jenkins.qa.ubuntu.com/job/sync-monitor-utopic-amd64-ci/25
    SUCCESS: http://jenkins.qa.ubuntu.com/job/sync-monitor-utopic-armhf-ci/25
        deb: http://jenkins.qa.ubuntu.com/job/sync-monitor-utopic-armhf-ci/25/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/sync-monitor-utopic-i386-ci/25

Click here to trigger a rebuild:
http://s-jenkins.ubuntu-ci:8080/job/sync-monitor-ci/77/rebuild

review: Approve (continuous-integration)
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote : Posted in a previous version of this proposal

FAILED: Continuous integration, rev:41
http://jenkins.qa.ubuntu.com/job/sync-monitor-ci/80/
Executed test runs:
    FAILURE: http://jenkins.qa.ubuntu.com/job/sync-monitor-utopic-amd64-ci/28/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/sync-monitor-utopic-armhf-ci/28/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/sync-monitor-utopic-i386-ci/28/console

Click here to trigger a rebuild:
http://s-jenkins.ubuntu-ci:8080/job/sync-monitor-ci/80/rebuild

review: Needs Fixing (continuous-integration)
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote : Posted in a previous version of this proposal

FAILED: Continuous integration, rev:44
http://jenkins.qa.ubuntu.com/job/sync-monitor-ci/81/
Executed test runs:
    FAILURE: http://jenkins.qa.ubuntu.com/job/sync-monitor-utopic-amd64-ci/29/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/sync-monitor-utopic-armhf-ci/29/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/sync-monitor-utopic-i386-ci/29/console

Click here to trigger a rebuild:
http://s-jenkins.ubuntu-ci:8080/job/sync-monitor-ci/81/rebuild

review: Needs Fixing (continuous-integration)
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote : Posted in a previous version of this proposal

PASSED: Continuous integration, rev:45
http://jenkins.qa.ubuntu.com/job/sync-monitor-ci/82/
Executed test runs:
    SUCCESS: http://jenkins.qa.ubuntu.com/job/sync-monitor-utopic-amd64-ci/30
    SUCCESS: http://jenkins.qa.ubuntu.com/job/sync-monitor-utopic-armhf-ci/30
        deb: http://jenkins.qa.ubuntu.com/job/sync-monitor-utopic-armhf-ci/30/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/sync-monitor-utopic-i386-ci/30

Click here to trigger a rebuild:
http://s-jenkins.ubuntu-ci:8080/job/sync-monitor-ci/82/rebuild

review: Approve (continuous-integration)
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :

PASSED: Continuous integration, rev:46
http://jenkins.qa.ubuntu.com/job/sync-monitor-ci/83/
Executed test runs:
    SUCCESS: http://jenkins.qa.ubuntu.com/job/sync-monitor-utopic-amd64-ci/31
    SUCCESS: http://jenkins.qa.ubuntu.com/job/sync-monitor-utopic-armhf-ci/31
        deb: http://jenkins.qa.ubuntu.com/job/sync-monitor-utopic-armhf-ci/31/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/sync-monitor-utopic-i386-ci/31

Click here to trigger a rebuild:
http://s-jenkins.ubuntu-ci:8080/job/sync-monitor-ci/83/rebuild

review: Approve (continuous-integration)
lp://qastaging/~renatofilho/sync-monitor/start-after-unity8 updated
47. By Renato Araujo Oliveira Filho

Merged parent branch.

Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :

PASSED: Continuous integration, rev:47
http://jenkins.qa.ubuntu.com/job/sync-monitor-ci/91/
Executed test runs:
    SUCCESS: http://jenkins.qa.ubuntu.com/job/sync-monitor-vivid-amd64-ci/2
    SUCCESS: http://jenkins.qa.ubuntu.com/job/sync-monitor-vivid-armhf-ci/2
        deb: http://jenkins.qa.ubuntu.com/job/sync-monitor-vivid-armhf-ci/2/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/sync-monitor-vivid-i386-ci/2

Click here to trigger a rebuild:
http://s-jenkins.ubuntu-ci:8080/job/sync-monitor-ci/91/rebuild

review: Approve (continuous-integration)
Revision history for this message
Bill Filler (bfiller) wrote :

tested according to "Test Network" section of test plan. All working.

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