-
Notifications
You must be signed in to change notification settings - Fork 155
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
Cannot set DNS server in Docker image #452
Comments
Thanks, those seen like issues we should correct. |
EDIT: I'm sorry, I just realized this should rather be a new issue. |
Is there anything specific you had to do to make this work? I've tried adding |
I have not tried |
This is a pebble bug, not a container one. Positional (non Providing a flag properly "fails":
So it is not the source of |
I can confirm. I spent two days debugging this now. :-( Docker image for pebble does not respect anymore CLI arguments. So it is not just Furthermore, pebble-challtestsrv Docker image also ignores CLI arguments. E.g., |
I'm sorry, but I can't replicate.
You'll need to share specific steps to reproduce. |
Running challtestsrv with ports mapped:
Running pebble in a container with ports mapped:
Running an arbitrary ACME client test against pebble in a container succeeds.
You can see the random domain Edit: I just saw that the following 2 port map lines were removed from the original
As for
|
@mcpherrinm: I am running it as:
This is how README has it documented, so with |
Thanks, I see the problem now. I think we can both fix the Pebble CLI to error in that case, and also fix the README. In the meantime, you can remove the |
Fixes #452 by rejecting the incorrect args, and fixes the readme to have a correct invocation.
After upgrading to
ghcr.io/letsencrypt/pebble:2.5.1
, I am seeing the following error in our test suite:This suggests that the
-dnsserver
setting is not honoured. As a workaround, I was able to specify a custom DNS server for the entire Docker container, but this required changing the port to 53.After further investigation, the issue seems to be that container parameters are being ignored altogether. The following invocation should fail, but the invalid parameter goes unnoticed:
The container listing shows that the command is translated into
/app abc
:Another issue I found with the new Docker image is that it does not expose any ports, requiring users to override the
ExposedPorts
setting when using the Docker daemon directly. Previously, theDockerfile
contained these two lines:The text was updated successfully, but these errors were encountered: