Uploads fail due to starvation
Bug #767466 reported by
Lucio Torre
This bug affects 1 person
Affects | Status | Importance | Assigned to | Milestone | ||
---|---|---|---|---|---|---|
Ubuntu One Client | Status tracked in Trunk | |||||
Stable-1-6 |
Fix Released
|
Critical
|
Facundo Batista | |||
Trunk |
Fix Released
|
Critical
|
Facundo Batista | |||
Ubuntu One storage protocol | Status tracked in Trunk | |||||
Stable-1-6 |
Fix Released
|
Critical
|
Facundo Batista | |||
Trunk |
Fix Released
|
Critical
|
Facundo Batista | |||
ubuntuone-client (Ubuntu) |
Fix Released
|
Critical
|
dobey | |||
Natty |
Fix Released
|
Critical
|
dobey | |||
Oneiric |
Fix Released
|
Undecided
|
Unassigned | |||
ubuntuone-storage-protocol (Ubuntu) |
Fix Released
|
Critical
|
dobey | |||
Natty |
Fix Released
|
Critical
|
dobey | |||
Oneiric |
Fix Released
|
Undecided
|
Unassigned |
Bug Description
When uploading files with a slow uplink and using the current payload size we dont write to each transfer often enough to avoid s3 starvation. We need to write more often from each of the current transfers and the way to do it is to have a smaller payload size. We write less but more often, as required to avoid starvation.
---
TEST CASE:
Need an Ubuntu One account setup and a slow uplink to test this properly.
1. Copy 25 files at least 2 MB in size into the ~/Ubuntu One folder
2. Allow files to sync
Expected result: Files upload properly rather than never uploading and showing TRY_AGAIN repeatedly in the ~/.cache/
Related branches
lp:~facundo/ubuntuone-storage-protocol/use-tuneable-maxsize
- Guillermo Gonzalez: Approve
- Lucio Torre (community): Approve
-
Diff: 176 lines (+59/-12)4 files modifiedtests/test_bytesproducer.py (+2/-4)
tests/test_client.py (+50/-2)
tests/test_putcontent.py (+4/-5)
ubuntuone/storageprotocol/client.py (+3/-1)
lp:~facundo/ubuntuone-client/configurable-two-values
- Guillermo Gonzalez: Approve
- Lucio Torre (community): Approve
-
Diff: 272 lines (+117/-10)6 files modifieddata/syncdaemon.conf (+9/-0)
tests/platform/linux/eventlog/test_zg_listener.py (+5/-2)
tests/syncdaemon/test_action_queue.py (+22/-4)
tests/syncdaemon/test_config.py (+50/-0)
ubuntuone/syncdaemon/action_queue.py (+11/-4)
ubuntuone/syncdaemon/config.py (+20/-0)
lp:~facundo/ubuntuone-storage-protocol/stable-1-6--use-tuneable-maxsize
- dobey (community): Approve
- Lucio Torre (community): Approve
-
Diff: 176 lines (+59/-12)4 files modifiedtests/test_bytesproducer.py (+2/-4)
tests/test_client.py (+50/-2)
tests/test_putcontent.py (+4/-5)
ubuntuone/storageprotocol/client.py (+3/-1)
lp:~facundo/ubuntuone-client/stable-1-6--configurable-two-values
- dobey (community): Approve
- Lucio Torre (community): Approve
-
Diff: 271 lines (+117/-9)6 files modifieddata/syncdaemon.conf (+9/-0)
tests/platform/linux/eventlog/test_zg_listener.py (+5/-2)
tests/syncdaemon/test_action_queue.py (+22/-4)
tests/syncdaemon/test_config.py (+50/-0)
ubuntuone/syncdaemon/action_queue.py (+11/-3)
ubuntuone/syncdaemon/config.py (+20/-0)
Changed in ubuntuone-client: | |
importance: | Undecided → Critical |
status: | New → Triaged |
assignee: | nobody → Facundo Batista (facundo) |
Changed in ubuntuone-storage-protocol: | |
status: | New → Triaged |
importance: | Undecided → Critical |
assignee: | nobody → Facundo Batista (facundo) |
Changed in ubuntuone-storage-protocol: | |
status: | Triaged → In Progress |
Changed in ubuntuone-client: | |
status: | Triaged → In Progress |
Changed in ubuntuone-storage-protocol: | |
status: | In Progress → Fix Committed |
Changed in ubuntuone-client: | |
status: | In Progress → Fix Committed |
Changed in ubuntuone-client (Ubuntu Natty): | |
assignee: | nobody → Rodney Dawes (dobey) |
importance: | Undecided → Critical |
status: | New → In Progress |
Changed in ubuntuone-storage-protocol (Ubuntu Natty): | |
assignee: | nobody → Rodney Dawes (dobey) |
importance: | Undecided → Critical |
status: | New → In Progress |
description: | updated |
To post a comment you must log in.
Accepted ubuntuone- storage- protocol into natty-proposed, the package will build now and be available in a few hours. Please test and give feedback here. See https:/ /wiki.ubuntu. com/Testing/ EnableProposed for documentation how to enable and use -proposed. Thank you in advance!