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

impose start up constraints between endhosts and their dependencies #179

Open
wants to merge 2 commits into
base: development
Choose a base branch
from

Conversation

amdfxlucas
Copy link
Contributor

start the endhost containers ('hnode') only after the routers ('rnode') and cs_services ('csnode') and any other infrastructure it depends on are up.
This puts an end to many annoying wait-conditions in container start-scripts especially for scion-endhosts to avoid nasty 'no path to ISD-AS ' errors when they are started before i.e. the border routers.

In order to achieve this the containers X,Y,Z that provide the infrastructure of an Autonomous System hosting a given node N are listed under N's 'depends_on:' section in the docker-compose file.
Currently the type of dependency is 'service-started' but it could be improved to 'service-healthy' if we come up with suitable HEALTHCHECKS for all the different components.

Known potential drawbacks:

  • with many routers ( as in #177issue) docker-compose file can easily reach 68'000 lines+

seedemu/compiler/Docker.py Outdated Show resolved Hide resolved
start the endhost containers only after the routers and csservices  are up
@amdfxlucas
Copy link
Contributor Author

@wonkr Hey, what has become of this ?! :)

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 this pull request may close these issues.

2 participants