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

examples/front-proxy tutorial is broken #11506

Closed
ahmetb opened this issue Jun 8, 2020 · 4 comments · Fixed by #11523
Closed

examples/front-proxy tutorial is broken #11506

ahmetb opened this issue Jun 8, 2020 · 4 comments · Fixed by #11523
Labels
question Questions that are neither investigations, bugs, nor enhancements

Comments

@ahmetb
Copy link

ahmetb commented Jun 8, 2020

I'm using Docker for Mac and for some reason the example proxy in examples/front-proxy cannot bind to 0.0.0.0:80 inside the container.

front-envoy_1  | [2020-06-08 21:04:27.772][7][critical][main] [source/server/server.cc:100] error initializing configuration '/etc/front-envoy.yaml': cannot bind '0.0.0.0:80': Permission denied

docker run --rm -p 80:80 nginx works fine.

So it makes me think something’s wrong with the docker-compose config. I unfortunately don't know enough to contribute.

@dio
Copy link
Member

dio commented Jun 8, 2020

Hum, I think all examples that bound to 80 (and other privileged ports) need to be updated since #11323

Meanwhile:

    environment:
      - "ENVOY_UID=0"

@ahmetkotan
Copy link

CMD command runs with envoy user because of this configuration. I tried adding to USER root to Dockerfile and user: root to docker-compose.yaml, but it didn't work because of this entrypoint.

If you are add to ENVOY_UID=0 to environment, you can run as root. Also see,

exec "${@}"

@mattklein123 mattklein123 added the question Questions that are neither investigations, bugs, nor enhancements label Jun 9, 2020
@phlax
Copy link
Member

phlax commented Jun 9, 2020

CMD command runs with envoy user because of this configuration.

its the entrypoint which changes the user.

@mattklein123 would you like me to PR to update the examples that expose 80 ?

@mattklein123
Copy link
Member

Sure any fixes to improve the examples appreciated.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
question Questions that are neither investigations, bugs, nor enhancements
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants