Skip to content

Conversation

@jshiwamV
Copy link
Collaborator

@jshiwamV jshiwamV commented Dec 19, 2025

}

# Custom CoreDNS Deployment
resource "kubernetes_deployment" "coredns" {
Copy link
Collaborator Author

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.

Copy link
Collaborator Author

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 ?

Copy link
Collaborator

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?
Copy link
Collaborator Author

@jshiwamV jshiwamV Dec 22, 2025

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.

Copy link
Collaborator

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

@jshiwamV
Copy link
Collaborator Author

Autoscaler changes are a part of separate PR, meanwhile this PR should be good to review.

@jshiwamV
Copy link
Collaborator Author

Closing this PR, Addressed in #134

@jshiwamV jshiwamV closed this Dec 23, 2025
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