-
Notifications
You must be signed in to change notification settings - Fork 170
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
Review and update DNS-resolver doc #1373
Comments
Hi @mayorova, how about add in If this looks good to you, I can add this to the project. |
@hector-vido I think the purpose of this ticket is to review and remove dnsmasq from the documentation since we no longer rely on dnsmasq to resolve DNS queries. Therefore, re-adding the dnsmasq container makes no sense to me. |
@tkan145 sorry, I totally misunderstood and forgot that the containers will relay in docker internal DNS resolver and adding entries in our local |
No need to sorry 😃 fyi, the resolver logic is located here https://github.com/3scale/APIcast/blob/master/gateway/src/resty/resolver.lua |
Just FYI, adding entries to |
The document https://github.com/3scale/APIcast/blob/bcd00e0/doc/dns-resolver.md mentions
dnsmasq
, howerverdnsmasq
is not forming part of the APIcast image anymore, see #1090 and https://issues.redhat.com/browse/THREESCALE-1555.Some other info in the doc might still be relevant though, but it needs to be reviewed and updated according to the current state.
The text was updated successfully, but these errors were encountered: