partials fail to be created with "filename too long"
| Affects | Status | Importance | Assigned to | Milestone | ||
|---|---|---|---|---|---|---|
| Ubuntu One Client | Status tracked in Trunk | |||||
| Stable-1-4 |
Fix Released
|
High
|
Facundo Batista | |||
| Trunk |
Fix Released
|
High
|
Facundo Batista | |||
| ubuntuone-client (Ubuntu) |
Fix Released
|
High
|
Facundo Batista | |||
| Maverick |
Fix Released
|
High
|
dobey | |||
Bug Description
When syncdaemon creates a partial we add stuff to the filename (uuid for uniqueness and the .u1partial extension). This makes the file name longer and sometimes fails to create the file. We should be able to find another name to the file in this case and not fail.
(see #657195 for logs and side effects)
-------
TEST CASE:
Ubuntu One client should be connected to the service.
1. Copy test file with name that is one character below the max supported by the filesystem to ~/Ubuntu One (e.g. unecrypted filesystem: "aaaaaaaaaaaaaa
Result: File is not synced
Expected result: File is synced
Related branches
- Lucio Torre (community): Approve
- Guillermo Gonzalez: Approve
-
Diff: 216 lines (+151/-6)2 files modifiedtests/syncdaemon/test_fsm.py (+125/-2)
ubuntuone/syncdaemon/filesystem_manager.py (+26/-4)
- John O'Brien (community): Approve
- Guillermo Gonzalez: Approve
-
Diff: 216 lines (+151/-6)2 files modifiedtests/syncdaemon/test_fsm.py (+125/-2)
ubuntuone/syncdaemon/filesystem_manager.py (+26/-4)
| Lucio Torre (lucio.torre) wrote | #1 |
| Changed in ubuntuone-client: | |
| assignee: | nobody → Facundo Batista (facundo) |
| importance: | Undecided → High |
| status: | New → Triaged |
| description: | updated |
| Changed in ubuntuone-client (Ubuntu): | |
| status: | New → Triaged |
| Changed in ubuntuone-client: | |
| status: | Triaged → In Progress |
| Changed in ubuntuone-client (Ubuntu): | |
| importance: | Undecided → High |
| assignee: | nobody → Facundo Batista (facundo) |
| tags: | added: chicharra u1-maverick-sru |
| Changed in ubuntuone-client: | |
| status: | In Progress → Fix Committed |
| Changed in ubuntuone-client (Ubuntu): | |
| status: | Triaged → Fix Released |
| Changed in ubuntuone-client (Ubuntu Maverick): | |
| assignee: | nobody → Rodney Dawes (dobey) |
| importance: | Undecided → High |
| status: | New → In Progress |
| Changed in ubuntuone-client (Ubuntu Maverick): | |
| milestone: | none → maverick-updates |
| Martin Pitt (pitti) wrote Please test proposed package | #2 |
| Changed in ubuntuone-client (Ubuntu Maverick): | |
| status: | In Progress → Fix Committed |
| tags: | added: verification-needed |
| Joshua Hoover (joshuahoover) wrote | #3 |
| tags: |
added: verification-done removed: verification-needed |
| Launchpad Janitor (janitor) wrote | #4 |
| Changed in ubuntuone-client (Ubuntu Maverick): | |
| status: | Fix Committed → Fix Released |
| tags: | added: testcase |
This bug is high because in file systems like encryptfs where filenames are shorter we will fail a lot.