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

Inline infrastructure module and reorganize into core stage #21

Open
jpolchlo opened this issue Oct 20, 2022 · 1 comment · May be fixed by #45
Open

Inline infrastructure module and reorganize into core stage #21

jpolchlo opened this issue Oct 20, 2022 · 1 comment · May be fixed by #45
Labels
kubernetes Kubernetes project work

Comments

@jpolchlo
Copy link
Collaborator

Issue #1 suggested that the right way to go about architecting this infrastructure was to use a Terraform module to provision an EKS cluster. This was done in #6, but it's unclear if this module structure was a good idea. In particular, because some of the steps to set up the infrastructure require the use of the kubernetes Terraform provider, and configuring that provider relies on the output of the EKS module, which creates a circular dependency when using the infrastructure module as currently set up, but not when the same code is inlined (for some idiosyncratic Terraform reason, no doubt), I had to separate the core into the 0-hardware and 1-services stages, rather than simply offer a core stage. This hasn't been and continues not to be a particularly big deal, but configuring IRSA for some core add-ons (notably EBS CSI, as introduced in #17 / #18) is now split across stages—the add-on is installed in 0-hardware and the IRSA role is attached in 1-services. This has a smell, and I think the better way might be to inline the infrastructure module code, and merge the two core deployment stages.

@BryanQuigley
Copy link

Agreed, I think we just want to have those handled separately. Having the modules in a separate place doesn't add anything afaict.

We should use the AWS provider and that's it.

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

Successfully merging a pull request may close this issue.

2 participants