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

Simplify/fix basic docker setup for single instances #4221

Closed
philippotto opened this issue Aug 7, 2019 · 5 comments · Fixed by #4368
Closed

Simplify/fix basic docker setup for single instances #4221

philippotto opened this issue Aug 7, 2019 · 5 comments · Fixed by #4368
Assignees

Comments

@philippotto
Copy link
Member

If I simply run wk with docker on a server with ip X, I'd like to be able to visit http://X:9000 and everything should just work. Instead, the communication with the datastore fails since it wants to communicate with localhost.

At the moment, I change the datastores manually within the database that gets reset by wk sometimes.

It would be cool if we could make this scenario more hassle-free.

@normanrz
Copy link
Member

To implement this, I believe, we need an additional publicUrl for data and tracing stores. webknossos and the stores would continue to use the url and the frontend would use the publicUrl. The rest can be done with some docker-compose magic.
The data and tracing stores probably don't need to know the publicUrl of webKnossos since they are not creating user-facing links?

pinging @fm3 @youri-k

@fm3
Copy link
Member

fm3 commented Sep 10, 2019

sounds about right to me

@youri-k
Copy link
Contributor

youri-k commented Sep 18, 2019

@normanrz @fm3 the backend part should be finished and ready for review

@fm3
Copy link
Member

fm3 commented Sep 18, 2019

@youri-k that’s #4286, right? could you fill in the template so that we know what to look for?

@normanrz
Copy link
Member

I think your PR can be merged w/o the work that I still need to do.

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

Successfully merging a pull request may close this issue.

4 participants