-
Notifications
You must be signed in to change notification settings - Fork 31
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
Add environment variable for tdb.dataPath #48
Comments
Unless you're specifically talking about the eddb sub-folder, this has
already been done. The environment variable TD_DATA, if set, specifies the
location to store the data at. So if it equals "/home/test/td", for
example, that's where the database is built, with the dump files being
stored in "/home/test/td/eddb".
…On Thu, Feb 14, 2019, 08:45 Stefan Morrell ***@***.*** wrote:
I know we previously discussed this, but given the change to a pip model,
I think we should switch up from this hardcoded
self.dataPath = tdb.dataPath / Path("eddb")
to having a default (as above) but overridable with a $ENV. It will still
only be relevant for server operation, I would think, but with automated
upgrades, I think it's the right way to deal with it from now on.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
<#48>, or mute the thread
<https://github.com/notifications/unsubscribe-auth/ABpz4Mv8ncLSbexmK5X-ArKX5MoMBD45ks5vNYSkgaJpZM4a734s>
.
|
I am talking about the eddb subfolder. On the server I redirect that into the web accessible part of the file heirarchy, e.g.
|
My ISP have (in their infinite wisdom) forgotten that I have a fixed IP and so now I can't connect through the office firewall as my IP has randomised. So I'll get this rolled out tomorrow and go live on the new server once I can (again) get shell access. |
So, the env-var is TD_EDDB once Travis finishes building the latest commit. |
I know we previously discussed this, but given the change to a pip model, I think we should switch up from this hardcoded
to having a default (as above) but overridable with a $ENV. It will still only be relevant for server operation, I would think, but with automated upgrades, I think it's the right way to deal with it from now on.
The text was updated successfully, but these errors were encountered: