-
Notifications
You must be signed in to change notification settings - Fork 2k
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
docker: use Nomad managed resolv.conf when DNS options are set #8600
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
Did this fix make it into the enterprise release? We are currently hitting this error ( |
@baxor yes, that'd be in enterprise as well. Can you open a new issue describing the problem you've got? Thanks. |
Thanks, I opened #11857 -- happy to provide any further details. |
I'm going to lock this pull request because it has been closed for 120 days ⏳. This helps our maintainers find and focus on the active contributions. |
When network DNS configuration was initially added, the implementation plumbed the config into the docker driver's DNS related field. This caused issues when using
bridge
orcni
Nomad networking modes as docker will fail to create the container because these DNS options cannot be set when using a non-default network mode. In this case Nomad's bridge networking mode sets the docker network mode tocontainer
.This PR fixes this issue by reusing logic in exec and other drivers to manage the DNS configuration and mount it into the task. This way Nomad generates the resolv.conf file and mounts it into the task automatically.
fixes #8431