Merge lp:~ken-vandine/content-hub/lp1326535 into lp:content-hub
Status: | Merged | ||||
---|---|---|---|---|---|
Approved by: | Michael Sheldon | ||||
Approved revision: | 111 | ||||
Merged at revision: | 112 | ||||
Proposed branch: | lp:~ken-vandine/content-hub/lp1326535 | ||||
Merge into: | lp:content-hub | ||||
Diff against target: |
11 lines (+2/-0) 1 file modified
src/com/ubuntu/content/detail/service.cpp (+2/-0) |
||||
To merge this branch: | bzr merge lp:~ken-vandine/content-hub/lp1326535 | ||||
Related bugs: |
|
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Michael Sheldon (community) | Approve | ||
PS Jenkins bot | continuous-integration | Approve | |
Review via email: mp+222549@code.launchpad.net |
Commit message
Don't abort downloads that haven't been charged yet when a new transfer
is created. This will leave multiple transfers from downloads which haven't
been charged remain in active_transfers until they have been
charged. (LP: #1326535)
Having multiple downloaded transfers in active_transfers means when the
destination starts or resumes, then all the completed downloads will get charged
which will send multiple signals to the destination.
Description of the change
Don't abort downloads that haven't been charged yet when a new transfer
is created. This will leave multiple transfers from downloads which haven't
been charged remain in active_transfers until they have been
charged. (LP: #1326535)
Having multiple downloaded transfers in active_transfers means when the
destination starts or resumes, then all the completed downloads will get charged
which will send multiple signals to the destination.
PASSED: Continuous integration, rev:111 jenkins. qa.ubuntu. com/job/ content- hub-ci/ 165/ jenkins. qa.ubuntu. com/job/ content- hub-utopic- amd64-ci/ 22 jenkins. qa.ubuntu. com/job/ content- hub-utopic- armhf-ci/ 22 jenkins. qa.ubuntu. com/job/ content- hub-utopic- armhf-ci/ 22/artifact/ work/output/ *zip*/output. zip jenkins. qa.ubuntu. com/job/ content- hub-utopic- i386-ci/ 22
http://
Executed test runs:
SUCCESS: http://
SUCCESS: http://
deb: http://
SUCCESS: http://
Click here to trigger a rebuild: s-jenkins. ubuntu- ci:8080/ job/content- hub-ci/ 165/rebuild
http://