cartesi / cli

Cartesi CLI tool
Apache License 2.0
3 stars 3 forks source link

feat(cli): use cartesi as compose project name #32

Open endersonmaia opened 1 month ago

endersonmaia commented 1 month ago

This PT will make the docker compose use cartesi as project name.

This way, instead of having a random string as the prefix for containers we'll have cartesi as prefix.

This will make it easier for us to grab logs or execute commands on a specific container, like:

docker logs -f cartesi-validator-1
docker exec -ti cartesi-validator-1 ls -lha /
changeset-bot[bot] commented 1 month ago

🦋 Changeset detected

Latest commit: ed35df596991ae38eccd87571bc0aac441babc0e

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package | Name | Type | | ------------ | ----- | | @cartesi/cli | Patch |

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

tuler commented 1 month ago

The drawback is that if you try to run two projects simultaneously it will fail. Running two projects may already fail because of ports, but I think we provide a way to change ports. We need to test the behavior and see if the error is graceful.