-
Notifications
You must be signed in to change notification settings - Fork 246
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Configurable storage path #472
Configurable storage path #472
Conversation
On the [storage] section of the tahoe.cfg now there is a field 'storage_dir' where the path to the storage folder can be configured.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great, LGTM. One inline comment.
docs/configuration.rst
Outdated
``storage_dir = (string, optional)`` | ||
|
||
This specifies a storing directory. | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Maybe something like "This is where share files and other state pertaining to storage-servers will be kept".
Codecov Report
@@ Coverage Diff @@
## master #472 +/- ##
==========================================
+ Coverage 90.09% 90.11% +0.02%
==========================================
Files 144 144
Lines 27316 27321 +5
Branches 3926 3927 +1
==========================================
+ Hits 24609 24619 +10
+ Misses 1964 1959 -5
Partials 743 743
Continue to review full report at Codecov.
|
Wow, nice. I'm happy to see it merged. Thanks :) |
Relates to #2045 but does not completely resolve it.
This introduces a new configuration item to the
storage
section oftahoe.cfg
,storage_dir
. This item tells a storage node where in the filesystem the directory to use for share storage can be found.For example,
Now, instead of using
$BASEDIR/storage
this node will use/foo/bar
.These changes were in part pulled out of #104 (which includes more changes for making more things configurable). I hope that pulling one specific feature out the chances of having it merged quickly are greater (and may even improve the chances of getting the rest of #104 merged).