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 |
Revision history for this message
| Clint Byrum (clint-fewbar) wrote Please test proposed package | #1 |
| Changed in ubuntuone-storage-protocol (Ubuntu Natty): | |
| status: | In Progress → Fix Committed |
| tags: | added: verification-needed |
Revision history for this message
| Clint Byrum (clint-fewbar) wrote | #2 |
Revision history for this message
| Clint Byrum (clint-fewbar) wrote | #3 |
| Changed in ubuntuone-client (Ubuntu Natty): | |
| status: | In Progress → Fix Committed |
| description: | updated |
Revision history for this message
| Joshua Hoover (joshuahoover) wrote | #4 |
| tags: |
added: u1-natty-sru verification-done removed: verification-needed |
Revision history for this message
| Launchpad Janitor (janitor) wrote | #5 |
| Changed in ubuntuone-client (Ubuntu Natty): | |
| status: | Fix Committed → Fix Released |
Revision history for this message
| Launchpad Janitor (janitor) wrote | #6 |
| Changed in ubuntuone-storage-protocol (Ubuntu Natty): | |
| status: | Fix Committed → Fix Released |
Revision history for this message
| Martin Pitt (pitti) wrote | #7 |
| Changed in ubuntuone-storage-protocol (Ubuntu Oneiric): | |
| status: | New → Fix Released |
| Changed in ubuntuone-client (Ubuntu Oneiric): | |
| status: | New → Fix Released |
To post a comment you must log in.
Accepted ubuntuone-