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

bugfix: Modify docker run error after agent folder move #458

Merged
merged 1 commit into from
Nov 26, 2024

Conversation

THtianhao
Copy link
Contributor

@THtianhao THtianhao commented Nov 20, 2024

Relates to:

Docker running error

Risks

Low, The scope of influence is the operation of docker

Background

What does this PR do?

After the author moved the agent folder, docker could not map the agent directory in docker, so docker volumn mapping incorrectly. I re-mapped the agent directory

docker has the following error:

sudo docker compose -f docker-compose-waifu.yaml up
[+] Running 1/0
✔ Container waifu  Created                                                                                               0.0 s
Attaching to waifu
waifu  |
waifu  | >  eliza@ start /app
waifu  | >  pnpm --filter "@ai16z/agent" start --isRoot "--characters=characters/waifu.character.json"
waifu  |
waifu  | No projects matched the filters in "/app"

What kind of change is this?

Bug fixes (non-breaking change which fixes an issue)

Documentation changes needed?

My changes do not require a change to the project documentation.

Testing

Where should a reviewer start?

Carry out

. ./scripts/docker.sh build
. ./scripts/docker.sh run

Try to see if the docker log is error free

Copy link
Collaborator

@monilpat monilpat left a comment

Choose a reason for hiding this comment

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

LGTM - thanks so much for fixing this

@THtianhao
Copy link
Contributor Author

LGTM - thanks so much for fixing this

This is what I should do, hope that the open source project will get better and better

@THtianhao THtianhao closed this Nov 21, 2024
@THtianhao THtianhao reopened this Nov 21, 2024
@lalalune lalalune merged commit c144a13 into elizaOS:main Nov 26, 2024
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.

3 participants