Skip to content
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

taky won't run without a config file #58

Open
tkuester opened this issue Jan 4, 2022 · 3 comments
Open

taky won't run without a config file #58

tkuester opened this issue Jan 4, 2022 · 3 comments
Assignees
Labels
bug Something isn't working fix-committed
Milestone

Comments

@tkuester
Copy link
Owner

tkuester commented Jan 4, 2022

➜  ~ taky -l debug
INFO:root:taky v0.9a1
usage: taky [-h] [-l {debug,info,warning,error,critical}] [-c CFG_FILE] [-d] [--version]
taky: error: Unable to load './taky.conf' or '/etc/taky.conf'

Thanks to our users in Discord for finding this bug!

@tkuester tkuester added the bug Something isn't working label Jan 4, 2022
@tkuester tkuester added this to the Release 0.9 milestone Jan 4, 2022
@tkuester tkuester self-assigned this Jan 4, 2022
tkuester added a commit that referenced this issue Jan 5, 2022
@tkuester
Copy link
Owner Author

There is another issue that has come up related to this. With the new features (ie: management sock), the config option for taky.root_dir has become more critical. Having discussed this issue with the helpful users in the Discord, we have come to the conclusion that the solution is to create a directory in the working path, and proceed from there.

A brief recount of the discussion, for posterity:

One option is to have the COT server simply not create the management socket when running "configuration-less". However, what should the data package server do? Without a directory to place uploaded data packages, the data package server is effectively useless. This expands the problem size.

We could simply do away with the "configuration-less" feature, but the users voted against this, as they found it to be useful.

Another option is to create a temporary directory with tempfile -- but this creates more problems than it solves. Should the files be deleted after usage? Leaving them will clutter the user's system with obscure paths, but deleting them may get rid of valuable user data.

The simplest option, proposed by the userbase, is to create a directory in the current working directory, and notify the user. And so that is what we will adopt!

@tkuester
Copy link
Owner Author

Commits 63e4f68..a3679a0 have the fix for this -- but has only been preliminarily tested.

@tkuester
Copy link
Owner Author

Due to my sloppy development practices, I can't confirm this fix has fully made it into 0.9. I have checked to confirm that taky runs without a config file, but this needs a little more careful review.

@tkuester tkuester modified the milestones: Release 0.9, Release 1.0 Jan 21, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working fix-committed
Projects
None yet
Development

No branches or pull requests

1 participant