Skip to content
This repository has been archived by the owner on Jun 20, 2024. It is now read-only.

fix: multiple docker builder searcher support #41

Merged
merged 2 commits into from
Oct 6, 2023
Merged

Conversation

iowar
Copy link
Collaborator

@iowar iowar commented Oct 6, 2023

docker-compose up -d --scale builder=3

@kant777
Copy link
Contributor

kant777 commented Oct 6, 2023

yeah thought about this but this is not a real scenario since it uses different folder under same volume. so if log into the container I will access to all keys. It is ok for a play env but I am trying to come up with some thing that is as close as possible to the real scenarios.

Copy link
Contributor

@aloknerurkar aloknerurkar left a comment

Choose a reason for hiding this comment

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

I dont mind this as an interim solution unless @kant777 already has something in mind!. The PR changes LGTM!

@iowar
Copy link
Collaborator Author

iowar commented Oct 6, 2023

remove old containers, volumes and images
docker rm -f $(docker ps -qa)
docker volume rm mev-commit_bootnode-keys mev-commit_builder-keys mev-commit_searcher-keys
docker image rm -f $(docker images|grep "mev-com"|awk -F " " '{print $3}'|head -n1)

@iowar iowar merged commit af975f7 into main Oct 6, 2023
1 check passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants