[tahoe-lafs-trac-stream] [tahoe-lafs] #641: tahoe backup should be able to backup symlinks

[tahoe-lafs]

#641: tahoe backup should be able to backup symlinks
-------------------------+-------------------------------------------------
     Reporter:           |      Owner:
  francois               |     Status:  new
         Type:  defect   |  Milestone:  undecided
     Priority:  normal   |    Version:  1.3.0
    Component:  code-    |   Keywords:  tahoe-backup symlink reliability
  frontend-cli           |  news-done review-needed design-review-needed
   Resolution:           |
Launchpad Bug:           |
-------------------------+-------------------------------------------------
Changes (by jg71):

 * cc: jg71@… (added)


Old description:

> Running tahoe backup on a directory containing a symbolic link currently
> doesn't work. It raises the following exception instead.
>
> {{{
> Traceback (most recent call last):
>   File "/home/francois/dev/tahoe/support/bin/tahoe", line 8, in <module>
>     load_entry_point('allmydata-tahoe==1.2.0-r3615', 'console_scripts',
> 'tahoe')()
>   File "/home/francois/dev/tahoe/src/allmydata/scripts/runner.py", line
> 91, in run
>     rc = runner(sys.argv[1:])
>   File "/home/francois/dev/tahoe/src/allmydata/scripts/runner.py", line
> 78, in runner
>     rc = cli.dispatch[command](so)
>   File "/home/francois/dev/tahoe/src/allmydata/scripts/cli.py", line 359,
> in backup
>     rc = tahoe_backup.backup(options)
>   File "/home/francois/dev/tahoe/src/allmydata/scripts/tahoe_backup.py",
> line 353, in backup
>     return bu.run()
>   File "/home/francois/dev/tahoe/src/allmydata/scripts/tahoe_backup.py",
> line 198, in run
>     new_backup_dircap = self.process(options.from_dir,
> latest_backup_dircap)
>   File "/home/francois/dev/tahoe/src/allmydata/scripts/tahoe_backup.py",
> line 245, in process
>     newchilddircap = self.process(childpath, oldchildcap)
>   File "/home/francois/dev/tahoe/src/allmydata/scripts/tahoe_backup.py",
> line 245, in process
>     newchilddircap = self.process(childpath, oldchildcap)
>   File "/home/francois/dev/tahoe/src/allmydata/scripts/tahoe_backup.py",
> line 245, in process
>     newchilddircap = self.process(childpath, oldchildcap)
>   File "/home/francois/dev/tahoe/src/allmydata/scripts/tahoe_backup.py",
> line 251, in process
>     raise RuntimeError("how do I back this up?" % childpath)
> RuntimeError: how do I back this up?
> }}}

New description:

 Running tahoe backup on a directory containing a symbolic link currently
 doesn't work. It raises the following exception instead.

 {{{
 Traceback (most recent call last):
   File "/home/francois/dev/tahoe/support/bin/tahoe", line 8, in <module>
     load_entry_point('allmydata-tahoe==1.2.0-r3615', 'console_scripts',
 'tahoe')()
   File "/home/francois/dev/tahoe/src/allmydata/scripts/runner.py", line
 91, in run
     rc = runner(sys.argv[1:])
   File "/home/francois/dev/tahoe/src/allmydata/scripts/runner.py", line
 78, in runner
     rc = cli.dispatch[command](so)
   File "/home/francois/dev/tahoe/src/allmydata/scripts/cli.py", line 359,
 in backup
     rc = tahoe_backup.backup(options)
   File "/home/francois/dev/tahoe/src/allmydata/scripts/tahoe_backup.py",
 line 353, in backup
     return bu.run()
   File "/home/francois/dev/tahoe/src/allmydata/scripts/tahoe_backup.py",
 line 198, in run
     new_backup_dircap = self.process(options.from_dir,
 latest_backup_dircap)
   File "/home/francois/dev/tahoe/src/allmydata/scripts/tahoe_backup.py",
 line 245, in process
     newchilddircap = self.process(childpath, oldchildcap)
   File "/home/francois/dev/tahoe/src/allmydata/scripts/tahoe_backup.py",
 line 245, in process
     newchilddircap = self.process(childpath, oldchildcap)
   File "/home/francois/dev/tahoe/src/allmydata/scripts/tahoe_backup.py",
 line 245, in process
     newchilddircap = self.process(childpath, oldchildcap)
   File "/home/francois/dev/tahoe/src/allmydata/scripts/tahoe_backup.py",
 line 251, in process
     raise RuntimeError("how do I back this up?" % childpath)
 RuntimeError: how do I back this up?
 }}}

--

Comment:

 Replying to [comment:18 zooko]:

 imho, it would be a good idea to keep backing up symlinks optional (well,
 /make/ it optional)

-- 
Ticket URL: <https://tahoe-lafs.org/trac/tahoe-lafs/ticket/641#comment:19>
tahoe-lafs <https://tahoe-lafs.org>
secure decentralized storage