[tahoe-lafs-trac-stream] [tahoe-lafs] #1333: 'private/' subdir of BASEDIR should be created during 'tahoe create-alias' and friends
tahoe-lafs
trac at tahoe-lafs.org
Wed Jan 4 21:48:09 UTC 2012
#1333: 'private/' subdir of BASEDIR should be created during 'tahoe create-alias'
and friends
--------------------------------+-----------------------
Reporter: warner | Owner:
Type: defect | Status: new
Priority: minor | Milestone: undecided
Component: code-nodeadmin | Version: 1.8.1
Resolution: | Keywords: easy
Launchpad Bug: |
--------------------------------+-----------------------
Comment (by warner):
discussing this on IRC just now, we agreed that both {{{create-alias}}}
and {{{add-alias}}} are allowed to require the pre-existence of
{{{NODEDIR}}} and {{{NODEDIR/node.url}}}, but both should
{{{os.mkdir(NODEDIR/private)}}} when necessary.
It'd be polite if both commands, when faced with a missing NODEDIR or
NODEDIR/node.url, were to tell the user (in the error message) that they
should create them, rather than throwing a weird exception that's hard to
understand. But that's not strictly necessary to resolve this ticket.
The main code changes will go into
{{{src/allmydata/scripts/tahoe_add_alias.py}}}. The unit test should
probably be added to the end of {{{CreateAlias.test_create}}} in
source:src/allmydata/test/test_cli.py , and should clobber {{{private/}}}
and then re-run {{{tahoe create-alias}}} and verify the results (by
reading the alias file). It might be a good idea to test {{{add-alias}}}
too, which is a bit easier (it doesn't need to talk to an actual node).
--
Ticket URL: <https://tahoe-lafs.org/trac/tahoe-lafs/ticket/1333#comment:2>
tahoe-lafs <https://tahoe-lafs.org>
secure decentralized storage
More information about the tahoe-lafs-trac-stream
mailing list