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

add cching #3333

Closed
wants to merge 5 commits into from
Closed

Conversation

louieheaton
Copy link

No description provided.

@boromisp
Copy link
Contributor

FYI, dnscache seems to be abandoned: yahoo/dnscache#28

@louieheaton
Copy link
Author

FYI, dnscache seems to be abandoned: yahoo/dnscache#28

Yeah @boromisp - got any alternatives ? We need a caching strategy and this is really the only one that seemingly works. We already have NodeLocal on our K8s environments.

@boromisp
Copy link
Contributor

FYI, dnscache seems to be abandoned: yahoo/dnscache#28

Yeah @boromisp - got any alternatives ? We need a caching strategy and this is really the only one that seemingly works. We already have NodeLocal on our K8s environments.

Sorry, I've never used anything like this before. There are a bunch of options but none of them stands out.

@charmander
Copy link
Collaborator

I think this should be implemented in the form of an option to provide any user-defined DNS lookup function.

@louieheaton
Copy link
Author

We've solved this now internally with a custom approach, but may not be a bad thing to think about. I'll close for now as I think there are some workarounds that can be done externally to the package. But the ability to provide a DNS lookup function is not a bad shout

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants