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 Optional Helm Install Support #1

Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

Conversation

bobbyiliev
Copy link
Collaborator

@bobbyiliev bobbyiliev commented Jan 2, 2025

Similar to the AWS module PR here: MaterializeInc/terraform-aws-materialize#15

Uses the unified Helm chart terraform module: github.com/MaterializeInc/terraform-helm-materialize?ref=v0.1.0

  • Introduced support for enabling Helm installation using:
    install_materialize_operator = true
  • Added configuration for Materialize instances with optional instance-specific details, such as namespace, database credentials, and host. Example:
 materialize_instances = [
   {
     name           = "analytics"
     namespace      = "materialize-environment"
     database_name  = "analytics_db"
     cpu_request    = "2"
     memory_request = "4Gi"
     memory_limit   = "4Gi"
   },
   {
     name           = "production"
     namespace      = "materialize-environment"
     database_name  = "production_db"
     cpu_request    = "4"
     memory_request = "8Gi"
     memory_limit   = "8Gi"
   }
 ]

One thing to keep in mind is that the materialize_instances are using the kubernetes_manifest resource which has a couple of limitations:

  • It requires the Kubernetes cluster to be running, otherwise it will fail to connect during the plan stage
  • For CRDs it requires the operator to be installed in the cluster first, otherwise it will fail during the plan stage

The workaround for the above is to not define the materialize_instances initially, once the EKS cluster is ready along with the operator installed, then you can do a second run to setup the materialize_instances.

@bobbyiliev bobbyiliev requested review from pH14, kay-kim and def- January 15, 2025 14:58
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.

2 participants