-
Notifications
You must be signed in to change notification settings - Fork 0
Custom CoreDNS for GCP and Azure #132
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
| } | ||
|
|
||
| # Custom CoreDNS Deployment | ||
| resource "kubernetes_deployment" "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 will move the Coredns ConfigMap, Deployment and Autoscaler to Common Kubernetes Module, since these components are cloud agnostic.
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.
On a second thought, I am thinking if we should avoid using coredns eks addon in aws as well. That way all the coredns config will be a common kubernetes modules, wdyt @jubrad ?
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.
Discussed on slack, I agree!
| ] | ||
| } | ||
|
|
||
| # TODO add autoscaler for coredns, should we use cluster-proportional autoscaler or HPA? |
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 looked into this today. coredns/coredns#5915 found a good link that shades some light over using HPA instead of Cluster Proportional Autoscaler.
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.
discussed on slack, HPA seems like the right call
|
Autoscaler changes are a part of separate PR, meanwhile this PR should be good to review. |
|
Closing this PR, Addressed in #134 |
Uh oh!
There was an error while loading. Please reload this page.