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

window10 cmd 执行docker-compose up -d nginx mysql #3063

Closed
zhangyang34 opened this issue Oct 1, 2021 · 10 comments
Closed

window10 cmd 执行docker-compose up -d nginx mysql #3063

zhangyang34 opened this issue Oct 1, 2021 · 10 comments

Comments

@zhangyang34
Copy link

unexpected character "-" in variable name near "-S\nWEAVER_MAX_WORKERS=10\nWEAVER_MAX_CONVERSION_QUEUE=50\nWEAVER_WORKER_TIMEOUT=90\nWEAVER_CONVERSION_FALLBACK=false\n\n# CloudConvert\n# API key can be found: https://cloudconvert.com/use
r/profile\n# CLOUDCONVERT_API=https://api.cloudconvert.com\n# CLOUDCONVERT_KEY=\n\n# Monitoring\n# STATSD_ADDRESS=\n# STATSD_PREFIX=\n# SENTRY_DSN="

@abelschooleman
Copy link

I have the same problem on linux kernel 5.14.6-arch1-1. This started when docker-compose was upgraded to 2.0.1.
I'm still running laradock with docker-compose 1.29.2 on another machine without problems.

@davidjr82
Copy link

Same issue in windows after docker upgrade. Workaround to fix it: comment (or delete) the weaver section in docker-compose.yml

@dhtmdgkr123
Copy link
Contributor

I have same issue in mac os
image

in weaver/conf/sample.env file
add double quotation
image
and it will work again!

davidjr82 added a commit to davidjr82/laradock that referenced this issue Oct 4, 2021
@davidjr82 davidjr82 mentioned this issue Oct 4, 2021
3 tasks
@dhtmdgkr123 dhtmdgkr123 mentioned this issue Oct 4, 2021
3 tasks
@garbinmarcelo
Copy link
Contributor

garbinmarcelo commented Oct 4, 2021

I got the same error when trying to up apache2 php-worker postgres

I believe this bug is of high priority as it is not possible to initialize the environment. More attention to this issue would be necessary.

Edit:
Using only double quotes as pull from #3066 fixed this bug.

@garbinmarcelo
Copy link
Contributor

One question I'm in doubt, have the prefixes for container names changed (underline replaced by dash)?

Previous example: website_apache2_1
Current: website-apache2-1

@dhtmdgkr123
Copy link
Contributor

dhtmdgkr123 commented Oct 4, 2021

@garbinmarcelo
yup u can change underline to dash see this document
also if u change underline to dash you can not use docker-compose scale

@garbinmarcelo
Copy link
Contributor

@garbinmarcelo yup u can change underline to dash see this document also if u change underline to dash you can not use docker-compose scale

Right, I didn't understand the documentation very well. I don't know if there was any update to the docker that changed the name.
Because I always used it with underline and now I went up laradock in an old project and saw that everything changed to dash (I tested it on one that already had laradock and now also has a dash.) .
That's why I was in doubt if it was the laradock or the docker, apparently it was the docker anyway, but I didn't find the option to set underline as default. 😕

@dhtmdgkr123
Copy link
Contributor

dhtmdgkr123 commented Oct 5, 2021

@garbinmarcelo umm... i don't know about this change...ToT
my environment(Mac BigSur(11.5.2), Docker version 20.10.8, docker-compose version 1.29.2)
can not reproduce your issue,,,

@stale
Copy link

stale bot commented Jan 3, 2022

Hi 👋 this issue has been automatically marked as stale 📌 because it has not had recent activity 😴. It will be closed if no further activity occurs. Thank you for your contributions ❤️.

@stale stale bot added the Stale label Jan 3, 2022
@stale
Copy link

stale bot commented Feb 20, 2022

Hi again 👋 we would like to inform you that this issue has been automatically closed 🔒 because it had not recent activity during the stale period. We really really appreciate your contributions, and looking forward for more in the future 🎈.

@stale stale bot closed this as completed Feb 20, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants