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

feat: docker compose file #33

Merged
merged 1 commit into from
Dec 15, 2023
Merged

feat: docker compose file #33

merged 1 commit into from
Dec 15, 2023

Conversation

conorsch
Copy link
Contributor

Based on the upstream Penumbra setup, but here we add a postgres container and of course the cuiloa container itself. Significantly, we munge the pd-generated cometbft config to enable indexing to postgres, which is a requirement for cuiloa.

This PR resurrects #1 and adapts that work for the latest version of the cuiloa setup.

Testing

So far, I've only used just podman-compose myself. @ejmg I'd appreciate if you tried out the just compose target and report back on how it works for you.

Based on the upstream Penumbra setup, but here we add
a postgres container and of course the cuiloa container itself.
Significantly, we munge the pd-generated cometbft config to enable
indexing to postgres, which is a requirement for cuiloa.
@conorsch conorsch requested a review from ejmg December 14, 2023 23:57
@ejmg ejmg linked an issue Dec 15, 2023 that may be closed by this pull request
@ejmg
Copy link
Collaborator

ejmg commented Dec 15, 2023

I've tested this using both the just recipe and also manual docker-compose and both build & run successfully on my machine. The just recipe did somehow lose control of stdio/err but the images were still running just fine so I consider this something to follow up later as the PR, as is, gives everything a new user with Docker/Podman needs to run cuiloa 😎

@ejmg ejmg merged commit f406874 into main Dec 15, 2023
Copy link
Collaborator

@ejmg ejmg left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Already merged. It works 👍

@ejmg ejmg deleted the compose-part-deux branch August 5, 2024 17:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
No open projects
Status: Done
Development

Successfully merging this pull request may close these issues.

update README
2 participants