-
Notifications
You must be signed in to change notification settings - Fork 0
Custom Core dns with HPA for all Cloud Providers #134
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
Conversation
aws ttl config remove azure changes from this pr add azure module back
aws/examples/simple/main.tf
Outdated
| module.eks, | ||
| module.base_node_group, | ||
| module.networking, | ||
| module.coredns, |
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.
I actually don't think Karpenter or any of its EC2NodeClasses or NodePools need coredns, since Karpenter is configured for Default DNS mode, which doesn't use coredns. (Default is to point at the cloud provider's DNS server outside the cluster. The default DNS mode for pods is ClusterFirst, not Default).
This is fine, I won't block the PR for this one. It's just unnecessary.
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.
(Default is to point at the cloud provider's DNS server outside the cluster. The default DNS mode for pods is ClusterFirst, not Default).
how about the daemonsets that would run on karpenter nodes, those would rely on Coredns isn't it? Since Pod DNS mode is ClusterFirst. So it makes sense to bring up coredns before provisioning karpenter nodes?
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.
I guess you're right. Karpenter and the node classes don't need it, but the nodepools do.
Closing Disable TTL cache in coredns AWS #128 and Custom CoreDNS for GCP and Azure #132 in favour of this PR since it covers for both of them