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

/opt/docspell.conf isn't used #303

Closed
totti4ever opened this issue Oct 3, 2020 · 3 comments
Closed

/opt/docspell.conf isn't used #303

totti4ever opened this issue Oct 3, 2020 · 3 comments

Comments

@totti4ever
Copy link
Contributor

totti4ever commented Oct 3, 2020

It still works on the last release - just not on the latest/self-compiled version!

Mapping for local one is done in docker-compose.ymland seems to work properly:

volumes
	- ./docspell.conf:/opt/docspell.conf`

execing ls in the container also shows that all rights are there:

-rwxrwxrwx    1 root     root           871 Oct  3 17:35 /opt/docspell.conf

But still, the default one is used :-/

[main] INFO  d.r.Main - Using config file from system properties: /opt/docspell-restserver/bin/../conf/docspell-server.conf
@eikek
Copy link
Owner

eikek commented Oct 4, 2020

Is the code on your branch? Or where can I find it for playing around?

@totti4ever
Copy link
Contributor Author

Not committed, yet. Wanted to keep the commit history clean for the PR ;-)

But figured out the problem: When combining entrypoint and cmd one has to use [] notation for them, but then again ENV variables cannot be used (for JVM params). Because I switched to exec notation in order to have the var read properly, docker went on to ignore the cmd and therefore the path to the conf file!

I would go with setting the JVM params by JAVA_OPTS ENV variable now and not passing it directly to the docspell binary calls. From my understanding and little testing, this seems to work.

@eikek
Copy link
Owner

eikek commented Oct 5, 2020

great! don't worry about unclean commits too much. you can always squash them together afterwards

seems strange that it's not possible to have env variables in the command, but I have no clue:) if it can be solved better or differently thats fine. consumedir image also needs an env Variable currently in the command.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants