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

Streamline startup #612

Open
wants to merge 6 commits into
base: master
Choose a base branch
from
Open

Streamline startup #612

wants to merge 6 commits into from

Conversation

zardus
Copy link
Contributor

@zardus zardus commented Oct 27, 2024

While writing ARCHITECTURE.md, I realized that the way we start the dojo is very convoluted (docker calls dojo init which calls dojo-init and then starts systemd, which in turn calls dojo up. I changed the startup flow to:

  • Outer docker entrypoint is systemd
  • Systemd has a dojo-init.service that calls dojo-init.
  • After this, systemd has a dojo-up.service that calls dojo-up.

This makes things more sane, and also turns us into a two-stop-shop for the most important logs (journalctl for -u dojo-init and -u dojo-up and docker logs ctfd for ctfd logs).

@zardus zardus force-pushed the wip/dojo-init branch 2 times, most recently from b798a57 to a6befbe Compare October 27, 2024 08:18
@zardus
Copy link
Contributor Author

zardus commented Oct 27, 2024

Failing with:

ctfd               | docker.errors.APIError: 400 Client Error for http+docker://localhost/v1.47/containers/0503714327bde0a6d7d8658cca4a539f4faceca6f16ac8a94711d3973eeed3f2/start: Bad Request ("failed to create task for container: failed to create shim task: OCI runtime create failed: runc create failed: unable to start container process: error during container init: error setting cgroup config for procHooks process: openat2 /sys/fs/cgroup/system.slice/docker-0503714327bde0a6d7d8658cca4a539f4faceca6f16ac8a94711d3973eeed3f2.scope/pids.max: no such file or directory: unknown")```

No idea why yet...

@zardus zardus force-pushed the wip/dojo-init branch 2 times, most recently from e830493 to 82a8ef6 Compare October 27, 2024 09:45
@zardus
Copy link
Contributor Author

zardus commented Oct 27, 2024

This is fucked.

@zardus zardus changed the title Streamline startup and document architecture Streamline startup Oct 27, 2024
dojo/dojo Outdated Show resolved Hide resolved
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