-
Notifications
You must be signed in to change notification settings - Fork 3
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
Launch additional services in separate container #119
Comments
It looks like a similar approach for AiiDA only was implemented here: https://github.com/aiidateam/aiida_docker_compose, however the repository appears unmaintained. |
@yakutovicha I've made a proposal on how to break up this task into multiple sub-steps in the description above. Please feel free to propose revisions to those. Thank you! |
I think I have the first somehow working version. Here is a bunch of PRs that I am preparing in order to switch to the docker compose approach:
More information will follow. |
@yakutovicha Could you please briefly document the remaining tasks for this? |
So, here is the report of what has been done and what needs to be done: [DONE] [TODO]
|
I would suggest to add a TODO item "Create rough estimate about how much of the API will need to be changed (which concepts are invalidated)". I would like to see some planning of those changes before we move into the implementation phase (even prototyping). |
Edit by @csadorf : this task list is no longer valid. Here is what needs to be done:
|
@yakutovicha Can you please create an issue for each individual unit of work and link them here as well as in the roadmap? |
I add the stack image aiidalab/aiidalab-docker-stack#319 and tested that it can be started with the services. |
The additional services (like Postgres or RabbitMQ) should be started as additional Docker services. This could for example be achieved via docker-compose.
AiiDAlab-launch would likely need some major revision to then support this, since it currently uses docker directly to manage the container(s).
Sub-tasks:
Dependencies
The text was updated successfully, but these errors were encountered: